Skip to content

ci: Update shellcheck and build image hashes#13362

Merged
mattklein123 merged 4 commits intoenvoyproxy:masterfrom
phlax:ci-update-shellcheck-and-build-hashes
Oct 5, 2020
Merged

ci: Update shellcheck and build image hashes#13362
mattklein123 merged 4 commits intoenvoyproxy:masterfrom
phlax:ci-update-shellcheck-and-build-hashes

Conversation

@phlax
Copy link
Copy Markdown
Member

@phlax phlax commented Oct 1, 2020

Commit Message: ci: Update shellcheck and build image hashes
Additional Description:

some shellcheck cleanups that are required once shellcheck is updated

this pr needs to /wait until envoyproxy/envoy-build-tools#103 lands and to have the new build image hashes added

Risk Level: low
Testing:
Docs Changes:
Release Notes:
[Optional Runtime guard:]
[Optional Fixes #Issue] Fixes #13228
[Optional Deprecated:]

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.

this will fail until shellcheck is updated

@mattklein123 mattklein123 self-assigned this Oct 1, 2020
@phlax
Copy link
Copy Markdown
Member Author

phlax commented Oct 2, 2020

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines, to retry CircleCI checks, use /retest-circle.
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #13362 (comment) was created by @phlax.

see: more, trace.

@phlax
Copy link
Copy Markdown
Member Author

phlax commented Oct 2, 2020

@mattklein123 i think the envoy-build-tools PR (envoyproxy/envoy-build-tools#103) didnt land yet

phlax added 3 commits October 3, 2020 09:57
Signed-off-by: Ryan Northey <ryan@synca.io>
Signed-off-by: Ryan Northey <ryan@synca.io>
Signed-off-by: Ryan Northey <ryan@synca.io>
@phlax phlax force-pushed the ci-update-shellcheck-and-build-hashes branch from 8deed43 to ba40377 Compare October 3, 2020 08:59
@phlax
Copy link
Copy Markdown
Member Author

phlax commented Oct 3, 2020

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines, to retry CircleCI checks, use /retest-circle.
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #13362 (comment) was created by @phlax.

see: more, trace.

@mattklein123
Copy link
Copy Markdown
Member

Hopefully this will work with the new image that is about to get built.

/wait

Signed-off-by: Ryan Northey <ryan@synca.io>
Copy link
Copy Markdown
Member

@mattklein123 mattklein123 left a comment

Choose a reason for hiding this comment

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

Thanks!

@mattklein123 mattklein123 merged commit f76c6b4 into envoyproxy:master Oct 5, 2020
@phlax phlax mentioned this pull request Oct 7, 2020
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.

Update shellcheck in CI

2 participants