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

bazel: remove version warnings #24660

Merged
merged 4 commits into from
Jan 2, 2025
Merged

Conversation

rockwotj
Copy link
Contributor

This removes all the annoying warning logging when invoking bazel. Mostly these are due to version mismatches and we just ratchet everything forward.

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v24.3.x
  • v24.2.x
  • v24.1.x

Release Notes

  • none

To remove an annoying warning spam logging message.
To remove some warning logging
So they are the same and bazel stops complaining
To match RPK and get rid of annoying warnings in Bazel
@rockwotj
Copy link
Contributor Author

Force push: run go mod tidy

@vbotbuildovich
Copy link
Collaborator

CI test results

test results on build#60162
test_id test_kind job_url test_status passed
rptest.tests.datalake.partition_movement_test.PartitionMovementTest.test_cross_core_movements.cloud_storage_type=CloudStorageType.S3 ducktape https://buildkite.com/redpanda/redpanda/builds/60162#0194066c-8ff4-4b3f-97d0-3601fb63c585 FLAKY 4/6

@rockwotj
Copy link
Contributor Author

The only failure is the raft reconfiguration unit test, which is a known flake (and I will soon make logs accessible)

@rockwotj rockwotj merged commit afa9ad9 into redpanda-data:dev Jan 2, 2025
29 of 32 checks passed
rockwotj added a commit that referenced this pull request Jan 2, 2025
@rockwotj rockwotj deleted the bazel-warnings branch January 2, 2025 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants