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

go 1.17.5 #90350

Closed
wants to merge 1 commit into from
Closed

go 1.17.5 #90350

wants to merge 1 commit into from

Conversation

Porkepix
Copy link
Contributor

@Porkepix Porkepix commented Dec 3, 2021

Created with brew bump-formula-pr.

resource blocks may require updates.

@BrewTestBot BrewTestBot added bump-formula-pr PR was created using `brew bump-formula-pr` CI-build-dependents-from-source Pass --build-dependents-from-source to brew test-bot. CI-linux-self-hosted Build on Linux self-hosted runner labels Dec 3, 2021
@Loyalsoldier
Copy link
Contributor

Why were tests cancelled?

@bevanjkay
Copy link
Member

bevanjkay commented Dec 3, 2021

@Loyalsoldier Tests have a timeout of 60 minute unless maintainers allow longer CI runs (by labelling the PR), to ensure CI machines aren't blocked by too many concurrent long runs.

@carlocab
Copy link
Member

carlocab commented Dec 3, 2021

We already have two long-running CI jobs at the moment. A third one is going to cause huge delays, so we can restart this when those jobs are done.

@carlocab carlocab added CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. and removed CI-linux-self-hosted Build on Linux self-hosted runner labels Dec 5, 2021
@kidoman
Copy link

kidoman commented Dec 5, 2021

Go 1.17.4 is now officially released. It fixes a nasty crash on mac OS Monterey (apparent kernel bug.) How can I help get this released?

@SMillerDev
Copy link
Member

Check all the failing dependent formulae and help fix them

@gonzaloserrano
Copy link

@SMillerDev where can we see that list?

@SMillerDev
Copy link
Member

In the pull request "Checks"

@iMichka
Copy link
Member

iMichka commented Dec 7, 2021

I opened a PR for imgproxy and bosh-cli: #90670, #90669

@carlocab carlocab mentioned this pull request Dec 8, 2021
6 tasks
carlocab added a commit to carlocab/homebrew-core that referenced this pull request Dec 8, 2021
This seems to make the test hang indefinitely (e.g. Homebrew#90350). Let's also
sleep for longer since short `sleep`s tend to cause trouble in CI.
@carlocab carlocab removed the CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. label Dec 8, 2021
@jidicula
Copy link
Contributor

jidicula commented Dec 9, 2021

fyi Go 1.17.5 was just released. Does that mean this PR should be closed and a new one opened?

@SMillerDev
Copy link
Member

This one can also be updated

stefanb added a commit to stefanb/homebrew-core that referenced this pull request Dec 10, 2021
This was referenced Dec 10, 2021
@SMillerDev SMillerDev changed the title go 1.17.4 go 1.17.5 Dec 10, 2021
@BrewTestBot BrewTestBot added the CI-linux-self-hosted Build on Linux self-hosted runner label Dec 10, 2021
@SMillerDev SMillerDev added the CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. label Dec 10, 2021
@stefanb
Copy link
Member

stefanb commented Dec 10, 2021

Most of tests

exceeded the maximum execution time of 60 minutes.

@stefanb
Copy link
Member

stefanb commented Dec 10, 2021

🤔 Q: why do all dependancies need to be checked and rebuilt?
Yes, it is true, that they depend on go, but upgrading go will not break them, as those binaries are already built and statically linked with NO external dependancies on go. They will continue to be functioning as before.

I have noticed this problem with the last few go upgrades, most notably with the major revisions, eg #83413 with a huge list of fixes to dependants, which should be up to dependants to sort out, and giving them upgraded go binary would actually help them detect and fix those problems earlier.

@SMillerDev
Copy link
Member

Yes, it is true, that they depend on go, but upgrading go will not break them, as those binaries are already built and statically linked with NO external dependancies on go. They will continue to be functioning as before.

But how will we know they don't break with the newest Go version if we don't try and build them with that version?

I have noticed this problem with the last few go upgrades, most notably with the major revisions, eg #83413 with a huge list of fixes to dependants, which should be up to dependants to sort out, and giving them upgraded go binary would actually help them detect and fix those problems earlier.

It is up to dependents to sort out, but if nobody checks if it works how would they ever know?

and giving them upgraded go binary would actually help them detect and fix those problems earlier.

They can always run CI with pre-release Go builds to see if their software will stop working soon. Depending on Homebrew to suddendly break your build when you upgrade is just lazy and irrisponsible engineering.

@stefanb
Copy link
Member

stefanb commented Dec 10, 2021

Yes, it is true, that they depend on go, but upgrading go will not break them, as those binaries are already built and statically linked with NO external dependancies on go. They will continue to be functioning as before.

But how will we know they don't break with the newest Go version if we don't try and build them with that version?

Do we need to know that when upgrading Go? Yes it is nice to know, but not essential for go upgrade and that can be dealt with later. Their current, already build version won't be affected in any way.

I have noticed this problem with the last few go upgrades, most notably with the major revisions, eg #83413 with a huge list of fixes to dependants, which should be up to dependants to sort out, and giving them upgraded go binary would actually help them detect and fix those problems earlier.

It is up to dependents to sort out, but if nobody checks if it works how would they ever know?

It will become obvious when somebody will eventually want to build the new dependant binary (either same or newer version, locally or when trying to bump the version in homebrew).

and giving them upgraded go binary would actually help them detect and fix those problems earlier.

They can always run CI with pre-release Go builds to see if their software will stop working soon. Depending on Homebrew to suddendly break your build when you upgrade is just lazy and irrisponsible engineering.

Yes, it is chicken and egg problem in a way, but current approach is blocking gradual evolution progress.

@andig
Copy link
Contributor

andig commented Dec 10, 2021

I think we've had the argument before and it is probably better suited for discussions. I'd suggest here: Homebrew/discussions#2618

@iMichka iMichka removed the CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. label Dec 10, 2021
@Bo98 Bo98 added the CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. label Dec 11, 2021
@chenrui333 chenrui333 removed the CI-long-timeout [DEPRECATED] Use longer GitHub Actions CI timeout. label Dec 12, 2021
@iMichka
Copy link
Member

iMichka commented Dec 12, 2021

@BrewTestBot
Copy link
Member

:shipit: @iMichka has triggered a merge.

@iMichka
Copy link
Member

iMichka commented Dec 13, 2021

Let's ship this. A few issues have been fixed, I'll make a second pass for go 1.17.6 ...

@iMichka
Copy link
Member

iMichka commented Dec 13, 2021

Thanks @Porkepix for the pull request.

@AlekSi
Copy link
Contributor

AlekSi commented Dec 13, 2021

Thanks, everyone

@stefanb stefanb mentioned this pull request Dec 15, 2021
@github-actions github-actions bot added the outdated PR was locked due to age label Jan 13, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 13, 2022
@Porkepix Porkepix deleted the bump-go-1.17.4 branch July 28, 2022 00:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bump-formula-pr PR was created using `brew bump-formula-pr` CI-build-dependents-from-source Pass --build-dependents-from-source to brew test-bot. CI-linux-self-hosted Build on Linux self-hosted runner CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.