Skip to content

deps: Bump bazel-buildtools -> 4.2.2#18401

Merged
mattklein123 merged 5 commits intoenvoyproxy:mainfrom
phlax:bump-bazel-buildtools
Oct 7, 2021
Merged

deps: Bump bazel-buildtools -> 4.2.2#18401
mattklein123 merged 5 commits intoenvoyproxy:mainfrom
phlax:bump-bazel-buildtools

Conversation

@phlax
Copy link
Copy Markdown
Member

@phlax phlax commented Oct 4, 2021

Signed-off-by: Ryan Northey ryan@synca.io

Commit Message: deps: Bump bazel-buildtools -> 4.2.0
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue] Fix #18347
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).

🐱

Caused by: #18401 was opened by phlax.

see: more, trace.

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Oct 4, 2021
Signed-off-by: Ryan Northey <ryan@synca.io>
@phlax phlax force-pushed the bump-bazel-buildtools branch from 593b97d to 45c522c Compare October 4, 2021 13:34
Comment on lines +35 to +37
version = "4.2.0",
sha256 = "d49976b0b1e81146d79072f10cabe6634afcd318b1bd86b0102d5967121c43c1",
release_date = "2021-09-06",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yep, missed that, thanks, ill update

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@moderation it turns out that this is correct the current release is 4.2.0 - not sure why but 4.2.1 is only a tag not a release

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The release tagged latest is 4.2.0 but the actual latest release is 4.2.1 - https://github.com/bazelbuild/buildtools/releases. I would bump this to 4.2.1 and be consistent with #18302

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

my point is its a tag not a release - probs an oversight - but my guess is that the links are different

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

actually - nm - the link works - ill update

@wrowe
Copy link
Copy Markdown
Contributor

wrowe commented Oct 4, 2021

You are aware 4.2.1 has been current for some time?

@phlax
Copy link
Copy Markdown
Member Author

phlax commented Oct 4, 2021

for bazel yes, but this is buildozer/buildifier so i didnt make the connection

i updated to 4.2.0 because that was the version on the ticket

we are just improving the issue creator so that it will have upstream links, which will make it easier to check what is current (#18374)

@wrowe
Copy link
Copy Markdown
Contributor

wrowe commented Oct 6, 2021

I believe this is fixed if you want to merge main to pass CI

phlax added 2 commits October 6, 2021 17:17
Signed-off-by: Ryan Northey <ryan@synca.io>
Signed-off-by: Ryan Northey <ryan@synca.io>
release_date = "2021-03-01",
version = "4.2.1",
sha256 = "b8b69615e8d9ade79f3612311b8d0c4dfe01017420c90eed11db15e9e7c9ff3c",
release_date = "2021-09-10",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As per CI failure the UTC for this release is 2021-09-11. The way I find this date is right click on tagged date text on Github, inspect the element in either Firefox or Chrome and find the datetime attribute below which is the UTC of the release. This is the date the checkers use

<relative-time datetime="2021-09-11T01:38:17Z" class="no-wrap" title="Sep 10, 2021, 6:38 PM PDT">26 days ago</relative-time>

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks - github ui uses "ago" and localizes dates, both of which are deeply unhelpful when trying to check the dates

our new issue creator mostly resolves this, but in this case its showing the latest release so not showing the date for this tag version

@phlax phlax force-pushed the bump-bazel-buildtools branch from ee21105 to 7394447 Compare October 7, 2021 09:18
Signed-off-by: Ryan Northey <ryan@synca.io>
@phlax phlax force-pushed the bump-bazel-buildtools branch from 7394447 to e76a8b1 Compare October 7, 2021 09:20
@phlax phlax requested a review from moderation October 7, 2021 18:12
@moderation
Copy link
Copy Markdown
Contributor

@phlax good news or terrible news? 4.2.2 is out https://github.com/bazelbuild/buildtools/releases/tag/4.2.2

@phlax
Copy link
Copy Markdown
Member Author

phlax commented Oct 7, 2021

8)

ill update...

Signed-off-by: Ryan Northey <ryan@synca.io>
@phlax phlax changed the title deps: Bump bazel-buildtools -> 4.2.0 deps: Bump bazel-buildtools -> 4.2.2 Oct 7, 2021
@moderation
Copy link
Copy Markdown
Contributor

/lgtm deps

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Oct 7, 2021
@mattklein123 mattklein123 merged commit fffc1c3 into envoyproxy:main Oct 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Newer release available com_github_bazelbuild_buildtools: 4.2.0

4 participants