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.16.4 #76791

Closed
wants to merge 1 commit into from
Closed

go 1.16.4 #76791

wants to merge 1 commit into from

Conversation

Loyalsoldier
Copy link
Contributor

@Loyalsoldier Loyalsoldier commented May 6, 2021

Created with brew bump-formula-pr.

resource blocks may require updates.

@Loyalsoldier
Copy link
Contributor Author

These formulas failed on Big Sur on Apple Silicon:

Error: 62 failed steps!
brew test --retry --verbose akamai
brew install --build-from-source argo
brew test --retry --verbose awsweeper
brew test --retry --verbose bazelisk
brew test --retry --verbose bitrise
brew install --build-from-source cf-tool
brew test --retry --verbose chart-testing
brew install --build-from-source chronograf
brew install --build-from-source circleci
brew install --build-from-source --only-dependencies clair
brew install --build-from-source clair
brew install --build-from-source consul
brew install --build-from-source --only-dependencies docker-machine-driver-hyperkit
brew install --build-from-source docker-machine-driver-hyperkit
brew install --build-from-source --only-dependencies dockviz
brew install --build-from-source dockviz
brew install --build-from-source drone-cli
brew install --build-from-source dvm
brew install --only-dependencies --include-test envconsul
brew test --retry --verbose envconsul
brew install --build-from-source --only-dependencies envoy
brew install --build-from-source envoy
brew test --retry --verbose etcd
brew install --build-from-source --only-dependencies fabio
brew install --build-from-source fabio
brew test --retry --verbose fortio
brew install --build-from-source --only-dependencies geoipupdate
brew install --build-from-source geoipupdate
brew install --build-from-source --only-dependencies git-annex-remote-rclone
brew install --build-from-source git-annex-remote-rclone
brew install --build-from-source gotop
brew install --build-from-source ioctl
brew install --build-from-source --only-dependencies ironcli
brew install --build-from-source ironcli
brew install --build-from-source istioctl
brew test --retry --verbose jabba
brew test --retry --verbose keptn
brew install --build-from-source kube-linter
brew install --build-from-source kustomize
brew install --build-from-source logcli
brew install --build-from-source mpdviz
brew install --build-from-source pilosa
brew install --build-from-source promtail
brew test --retry --verbose server-go
brew install --build-from-source ship
brew install --build-from-source source-to-image
brew install --build-from-source --only-dependencies ssh-vault
brew install --build-from-source ssh-vault
brew install --only-dependencies --include-test stolon
brew test --retry --verbose stolon
brew install --build-from-source td
brew install --build-from-source teleconsole
brew install --build-from-source telegraf
brew test --retry --verbose terrascan
brew test --retry --verbose terrascan
brew install --build-from-source traefik@1
brew install --build-from-source --only-dependencies vault
brew install --build-from-source vault
brew install --build-from-source --only-dependencies vert
brew install --build-from-source vert
brew install --build-from-source virgil
brew fetch --retry vitess
Error: Process completed with exit code 1.

@carlocab
Copy link
Member

carlocab commented May 7, 2021

Most of the ARM failures haven't been bottled. These are the ones that have been:

chronograf
fortio
istioctl
kustomize
logcli
promtail
ship
td
telegraf
terrascan
traefik@1

@carlocab
Copy link
Member

carlocab commented May 7, 2021

Catalina:

Error: 15 failed steps!
brew install --build-from-source chronograf
brew test --retry --verbose darksky-weather
brew test --retry --verbose darksky-weather
brew test --retry --verbose fortio
brew install --build-from-source grafana
brew install --build-from-source kustomize
brew install --build-from-source logcli
brew install --build-from-source promtail
brew install --build-from-source ship
brew install --build-from-source td
brew install --build-from-source telegraf
brew test --retry --verbose terrascan
brew test --retry --verbose terrascan
brew install --build-from-source traefik@1
brew install --build-from-source vitess

Mojave:

Error: 12 failed steps!
brew install --build-from-source aws-iam-authenticator
brew install --build-from-source chronograf
brew install --build-from-source kustomize
brew install --build-from-source logcli
brew install --build-from-source promtail
brew install --build-from-source ship
brew install --build-from-source td
brew install --build-from-source telegraf
brew test --retry --verbose terrascan
brew test --retry --verbose terrascan
brew install --build-from-source traefik@1
brew install --build-from-source vitess

@Loyalsoldier
Copy link
Contributor Author

Then what's next?

@carlocab
Copy link
Member

carlocab commented May 7, 2021

Then what's next?

Investigate these test failures and try to fix as many as we can.

Big Sur:

Error: 12 failed steps!
brew install --build-from-source chronograf
brew test --retry --verbose fortio
brew install --build-from-source kustomize
brew install --build-from-source logcli
brew install --build-from-source promtail
brew install --build-from-source ship
brew install --build-from-source td
brew install --build-from-source telegraf
brew test --retry --verbose terrascan
brew test --retry --verbose terrascan
brew install --build-from-source traefik@1
brew install --build-from-source vitess

@andig
Copy link
Contributor

andig commented May 8, 2021

I find this a bit confusing. Go 1.16.4 has zero changes that should not work where 1.16.3 has worked?

@cho-m
Copy link
Member

cho-m commented May 8, 2021

Will update this post with some of my analysis of failures:

Go-related failures or in need of further triage

  • kustomize kustomize: fix build error due to multiple tags with HEAD #77023 - EDIT: git tag --contains HEAD is outputting multiple tags, breaking go build command.
  • fortio - somehow fortio (1.15.2) version command is outputting 1.15.0. EDIT: odd failure as locally building go 1.16.4 and fortio has no test issue for me. Also Mojave node passed. May need to rerun CI.

node@16 + node-sass failures (either need node@14 or node-sass@6.0.0)

SHA256 mismatch failures (need to confirm with upstream and then update formulae):

Broken Formula tests (not too sure why CI didn't detect on last Formula PR):

  • terrascan terrascan 1.6.0 #77325 - confirmed test failure with current bottle on local system:
    EDIT: Test fix added with linked 1.6.0 PR

Maybe Network Issue (Probably can just re-run CI):

  • darksky-weather - Only Catalina failed from 403 on https://geocode.jessfraz.com/forecast
  • aws-iam-authenticator - Only Mojave failed on git clone error: RPC failed; curl 56 LibreSSL SSL_read: error:06FFF064:digital envelope
  • grafana - Only Catalina failed with ECONNRESET

@jlk
Copy link
Contributor

jlk commented May 11, 2021

Folks - hey from terrascan team. :) Just saw this while passing through - we'll take a peek at this from our side, as well...

@cho-m
Copy link
Member

cho-m commented May 19, 2021

May want to rebase and rerun CI to test on latest Homebrew state.

From previous run, the failures that probably will reoccur are 4 SHA checksum mismatch errors (vitess, logcli, promtail, and td), which require updating the sha256 checksum in each formula after confirming with the upstream source that the new checksum is correct and not the result of a malicious actor.

It is possible that new failures will occur due to formulae being updated during the timeframe.

@BrewTestBot BrewTestBot added the bump-formula-pr PR was created using `brew bump-formula-pr` label May 19, 2021
@carlocab carlocab added the in progress Stale bot should stay away label May 19, 2021
@cho-m
Copy link
Member

cho-m commented May 19, 2021

Will update with latest failures here.
Can later decide if any are actually go-update blocking failures.

Failures:

  • grafana (10.15, 11) - may be network issue? - Also had recent update grafana 7.5.7 #77608 so might be okay now.
  • istioctl (11-arm) - Doesn't build on ARM as-is - Probably not blocking. Possible fix in istioctl 1.10.0 #77139
  • youtubedr (ALL Intel) - test failure - new version was merged after test youtubedr 2.7.0 #77619, so maybe fixed?
    Error: youtubedr: failed
    An exception occurred within a child process:
      Minitest::Assertion: Expected /Title:\ \ \ \ \ \ \ History\ of\ homebrew\-core/ to match nil.
    
  • inlets (10.14) - test failure
    2021/05/19 15:51:44 Starting client - version 3.0.1
    2021/05/19 15:51:44 Upstream:  => http://localhost:62671
    time="2021/05/19 15:51:44" level=info msg="Connecting to proxy" url="ws://localhost:62672/tunnel"
    time="2021/05/19 15:51:44" level=error msg="Failed to connect to proxy. Empty dialer response" error="dial tcp [::1]:62672: connect: connection refused"
    time="2021/05/19 15:51:44" level=error msg="Remotedialer proxy error" error="dial tcp [::1]:62672: connect: connection refused"
    

Checksum mismatches. May not be blocking merge?

  • linkerd (ALL) - commit mismatch
    ==> Checking out tag stable-2.10.2
    HEAD is now at 5535e9c4 Add service annotations fix to stable 2.10.2 (#6126)
    Error: stable-2.10.2 tag should be fd03538b2641c56f6e6dc3bb5de1669ccd96278a
    but is actually 5535e9c4edda7f5a8d8d21e351b61425bd3a8208
    
  • td (ALL) - SHA mismatch
  • teleport (ALL) - SHA mismatch
  • kops (ALL Intel) - SHA mismatch
  • vitess (ALL Intel) - SHA mismatch

@cho-m cho-m mentioned this pull request May 19, 2021
@cho-m
Copy link
Member

cho-m commented May 26, 2021

@carlocab Can you check if we can go ahead with merge here?
Pretty much all failures are checksum errors (linkerd, td, telegraf, and vitess).
Only other one is istioctl on Catalina, which was a network issue on CI node.


Big Sur ARM

brew install --build-from-source linkerd
brew install --build-from-source td
brew install --build-from-source telegraf

Big Sur

brew install --build-from-source linkerd
brew install --build-from-source td
brew install --build-from-source telegraf
brew install --build-from-source vitess

Catalina

brew install --build-from-source istioctl
brew install --build-from-source linkerd
brew install --build-from-source td
brew install --build-from-source telegraf
brew install --build-from-source vitess

Mojave

brew install --build-from-source linkerd
brew install --build-from-source td
brew install --build-from-source telegraf
brew install --build-from-source vitess

EDIT:
On side note, I can't believe we hit another checksum issue with telegraf.
We dealt with one for 1.18.2 and see another for 1.18.3.

Copy link
Member

@carlocab carlocab left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for working on this, @cho-m.

@BrewTestBot
Copy link
Member

:shipit: @carlocab has triggered a merge.

@ldemailly
Copy link
Contributor

#78670 maybe will eliminate the issue (though there shouldn't have been an issue at all so it's a bit mysterious)

@chenrui333 chenrui333 removed the in progress Stale bot should stay away label Jun 3, 2021
@AlekSi AlekSi mentioned this pull request Jun 4, 2021
@github-actions github-actions bot added the outdated PR was locked due to age label Jul 4, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 4, 2021
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` outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants