Skip to content

Update rules_apple#16404

Merged
lizan merged 1 commit intoenvoyproxy:mainfrom
vladmos:update_rules_apple
May 12, 2021
Merged

Update rules_apple#16404
lizan merged 1 commit intoenvoyproxy:mainfrom
vladmos:update_rules_apple

Conversation

@vladmos
Copy link
Copy Markdown
Contributor

@vladmos vladmos commented May 10, 2021

Commit Message: build: update rules_apple to the latest version
Additional Description: The current version of rules_apple is not compatible with future versions of Bazel and will cause breakages
Risk Level: Low
Testing: CI
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label May 10, 2021
@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: #16404 was opened by vladmos.

see: more, trace.

Signed-off-by: Vladimir Moskva <vladmos@google.com>
@vladmos
Copy link
Copy Markdown
Contributor Author

vladmos commented May 10, 2021

"envoy-presubmit (precheck dependencies)" is failing because GitHub returned 502, how do I rerun the tests?

sha256 = "c84962b64d9ae4472adfb01ec2cf1aa73cb2ee8308242add55fa7cc38602d882",
urls = ["https://github.com/bazelbuild/rules_apple/releases/download/{version}/rules_apple.{version}.tar.gz"],
release_date = "2019-10-10",
release_date = "2021-05-04",
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 UTC for this release is 2021-05-07 however it's possible that the last commit for the release is 2021-05-04

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I initially submitted it with "2021-05-07" but a test failed saying that it should have been "2021-05-04".

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.

No problem. I'll fix-up when I update the release date check.

@moderation
Copy link
Copy Markdown
Contributor

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #16404 (comment) was created by @moderation.

see: more, trace.

@moderation
Copy link
Copy Markdown
Contributor

/lgtm deps
Looks like a timeout on Window CI
/retest

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label May 11, 2021
@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #16404 (comment) was created by @moderation.

see: more, trace.

@lizan lizan merged commit ce071f6 into envoyproxy:main May 12, 2021
@vladmos vladmos deleted the update_rules_apple branch May 14, 2021 18:13
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.

3 participants