Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

terrascan 1.19.6 #184483

Closed
wants to merge 1 commit into from
Closed

terrascan 1.19.6 #184483

wants to merge 1 commit into from

Conversation

BrewTestBot
Copy link
Member

Created by brew bump


Created with brew bump-formula-pr.

@github-actions github-actions bot added go Go use is a significant feature of the PR or issue bump-formula-pr PR was created using `brew bump-formula-pr` labels Sep 10, 2024
@chenrui333
Copy link
Member

chenrui333 commented Sep 10, 2024

==> /opt/homebrew/Cellar/terrascan/1.19.6/bin/terrascan scan -f /private/tmp/terrascan-test-20240910-33760-vlpqhj/ami.tf -t aws
panic: proto: file "extensions/extension.proto" is already registered
		previously from: "github.com/google/gnostic-models/extensions"
		currently from:  "github.com/googleapis/gnostic/extensions"
	See https://protobuf.dev/reference/go/faq#namespace-conflict


goroutine 1 [running]:
google.golang.org/protobuf/reflect/protoregistry.init.func1({0x107151700?, 0x10570f640?}, {0x1057b3720, 0x140001b2320})
	google.golang.org/[email protected]/reflect/protoregistry/registry.go:56 +0x1f8
google.golang.org/protobuf/reflect/protoregistry.(*Files).RegisterFile(0x140002224f8, {0x1057f13f8, 0x140001836c0})
	google.golang.org/[email protected]/reflect/protoregistry/registry.go:130 +0x9b0
google.golang.org/protobuf/internal/filedesc.Builder.Build({{0x104caf585, 0x28}, {0x1071b87e0, 0x282, 0x282}, 0x0, 0x4, 0x0, 0x0, {0x1057c1a98, ...}, ...})
	google.golang.org/[email protected]/internal/filedesc/build.go:112 +0x1a0
google.golang.org/protobuf/internal/filetype.Builder.Build({{{0x104caf585, 0x28}, {0x1071b87e0, 0x282, 0x282}, 0x0, 0x4, 0x0, 0x0, {0x0, ...}, ...}, ...})
	google.golang.org/[email protected]/internal/filetype/build.go:138 +0x158
github.com/googleapis/gnostic/extensions.file_extensions_extension_proto_init()
	github.com/googleapis/[email protected]/extensions/extension.pb.go:456 +0x144
github.com/googleapis/gnostic/extensions.init.0()
	github.com/googleapis/[email protected]/extensions/extension.pb.go:388 +0x1c

/opt/homebrew/Library/Homebrew/ignorable.rb:27:in `block in raise'
Minitest::Assertion: Expected: 0
  Actual: 2


@chenrui333 chenrui333 added upstream issue An upstream issue report is needed test failure CI fails while running the test-do block labels Sep 10, 2024
terrascan: add ldflag config for protobuf

Signed-off-by: Rui Chen <[email protected]>
@chenrui333 chenrui333 force-pushed the bump-terrascan-1.19.6 branch from 8f17148 to 19874a3 Compare September 10, 2024 14:09
@chenrui333 chenrui333 added ready to merge PR can be merged once CI is green pre-release Artifact is pre-release and removed upstream issue An upstream issue report is needed test failure CI fails while running the test-do block ready to merge PR can be merged once CI is green labels Sep 10, 2024
@chenrui333
Copy link
Member

image

@chenrui333 chenrui333 closed this Sep 10, 2024
@github-actions github-actions bot deleted the bump-terrascan-1.19.6 branch September 10, 2024 17:23
@stefanb stefanb mentioned this pull request Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump-formula-pr PR was created using `brew bump-formula-pr` go Go use is a significant feature of the PR or issue pre-release Artifact is pre-release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants