Skip to content

envoy 1.24.0#114194

Closed
dio wants to merge 1 commit intoHomebrew:masterfrom
dio:envoy-1.24.0-update
Closed

envoy 1.24.0#114194
dio wants to merge 1 commit intoHomebrew:masterfrom
dio:envoy-1.24.0-update

Conversation

@dio
Copy link
Copy Markdown
Contributor

@dio dio commented Oct 28, 2022

  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change? I found envoy 1.24.0 #113451 but seems like that needs tweaking.
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

This supersedes #113451. This PR makes sure we build from the source code archive of a tagged release (vs. git clone) and set the build info to "Distribution" vs. "Modified" (via providing the SOURCE_VERSION file before building).

@BrewTestBot BrewTestBot added the automerge-skip `brew pr-automerge` will skip this pull request label Oct 28, 2022
@dio dio force-pushed the envoy-1.24.0-update branch from ea36bc5 to c253d87 Compare October 28, 2022 22:59
@dio
Copy link
Copy Markdown
Contributor Author

dio commented Oct 29, 2022

We do have the v1.23.2 release https://github.com/envoyproxy/envoy/releases/tag/v1.23.2. @carlocab @cho-m do you think we should aim for that (#114215) first?

@dio dio mentioned this pull request Oct 29, 2022
6 tasks
@dio
Copy link
Copy Markdown
Contributor Author

dio commented Oct 29, 2022

Seems like it requires the long-build label.

@SMillerDev SMillerDev added the long build Set a long timeout for formula testing label Oct 30, 2022
@dio
Copy link
Copy Markdown
Contributor Author

dio commented Oct 31, 2022

Thanks, @SMillerDev! Also, probably we need the CI-linux-self-hosted label as the previous PR: #110883

@dio dio force-pushed the envoy-1.24.0-update branch from c253d87 to 3362539 Compare October 31, 2022 09:27
@dio
Copy link
Copy Markdown
Contributor Author

dio commented Oct 31, 2022

Sorry I needed to update one of the pre-build script args.

@dio dio force-pushed the envoy-1.24.0-update branch 4 times, most recently from 61c2683 to 63cad2d Compare November 1, 2022 11:57
@chenrui333
Copy link
Copy Markdown
Member

We do have the v1.23.2 release https://github.com/envoyproxy/envoy/releases/tag/v1.23.2. @carlocab @cho-m do you think we should aim for that (#114215) first?

yeah, always doing the latest. :)

@dio dio force-pushed the envoy-1.24.0-update branch from 63cad2d to 0d72b82 Compare November 2, 2022 02:01
@dio
Copy link
Copy Markdown
Contributor Author

dio commented Nov 2, 2022

Thanks for the confirmation @chenrui333! Really appreciated.

Hope we can move this forward, with some additional labels.

@chenrui333 chenrui333 added CI-linux-self-hosted Build on Linux self-hosted runner CI-no-bottles Merge without publishing bottles CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. and removed CI-no-bottles Merge without publishing bottles labels Nov 12, 2022
@chenrui333
Copy link
Copy Markdown
Member

🏃

@dio
Copy link
Copy Markdown
Contributor Author

dio commented Nov 13, 2022

Thank you, @chenrui333!

@cho-m cho-m removed the CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. label Nov 13, 2022
@BrewTestBot
Copy link
Copy Markdown
Contributor

:shipit: @fxcoudert has triggered a merge.

@github-actions github-actions Bot added the outdated PR was locked due to age label Dec 14, 2022
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Dec 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

automerge-skip `brew pr-automerge` will skip this pull request CI-linux-self-hosted Build on Linux self-hosted runner CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. long build Set a long timeout for formula testing outdated PR was locked due to age

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants