Skip to content

make the pipelines pass, and other TLC#1189

Merged
sudoforge merged 4 commits intomasterfrom
I957c6de57b3a9151e6d9ce8caba87f4e2c714846
Jul 20, 2024
Merged

make the pipelines pass, and other TLC#1189
sudoforge merged 4 commits intomasterfrom
I957c6de57b3a9151e6d9ce8caba87f4e2c714846

Conversation

@sudoforge
Copy link
Contributor

@sudoforge sudoforge commented Jul 19, 2024

- feat: update action library versions
- feat: add concurrency limits to all pipelines
- fix: bump to go v1.22.5
- fix: correct typo: acceps => accepts

This is a stack of logically independent commits and should be rebased instead of merged with a merge commit, or squashed.

@sudoforge sudoforge changed the title chore: correct typo: acceps => accepts pipeline corrections Jul 19, 2024
@sudoforge sudoforge force-pushed the I957c6de57b3a9151e6d9ce8caba87f4e2c714846 branch from 4810695 to 45d6eb7 Compare July 19, 2024 04:46
@sudoforge
Copy link
Contributor Author

sudoforge commented Jul 19, 2024

@MichaelMure i'm getting test failures due to expired bridge tokens. I can update the pipeline to skip this, however, I'd appreciate it if you rotated the token so that I can ensure the test suite passes (especially because I updated the pipeline to use go 1.22.5).


edit: i connected with michael on matrix; this is now resolved

@sudoforge sudoforge force-pushed the I957c6de57b3a9151e6d9ce8caba87f4e2c714846 branch from 1488e5c to 43cb897 Compare July 19, 2024 04:57
@sudoforge sudoforge changed the title pipeline corrections A lil' TLC for the pipelines Jul 19, 2024
@sudoforge sudoforge changed the title A lil' TLC for the pipelines A lil' bit of TLC for the pipelines Jul 19, 2024
@sudoforge sudoforge force-pushed the I957c6de57b3a9151e6d9ce8caba87f4e2c714846 branch from 43cb897 to 89ffb37 Compare July 19, 2024 14:02
The GHA build pipeline is failing with an error:
    package crypto/ecdh is not in GOROOT (/opt/hostedtoolcache/go/1.19.4/x64/src/crypto/ecdh)

crypto/ecdh was added in Go 1.21 [0]. This change updates the matrix of
Go versions that the pipelines use to the latest, in order to resolve this
error.

[0]: https://tip.golang.org/doc/go1.21
@sudoforge sudoforge force-pushed the I957c6de57b3a9151e6d9ce8caba87f4e2c714846 branch from 89ffb37 to d22b095 Compare July 20, 2024 05:22
This change restricts the number of active pipeline executions for a
given tree to one, so that new pushes to a tree that trigger a pipeline
will cancel older (but still running) pipelines.

Change-Id: I1deda682d39cadbce97f8df4ca7c3110a96b4549
Change-Id: I957c6de57b3a9151e6d9ce8caba87f4e2c714846
@sudoforge sudoforge force-pushed the I957c6de57b3a9151e6d9ce8caba87f4e2c714846 branch from d22b095 to 6b65227 Compare July 20, 2024 05:35
@sudoforge sudoforge marked this pull request as draft July 20, 2024 06:04
@sudoforge sudoforge marked this pull request as ready for review July 20, 2024 14:43
@sudoforge sudoforge merged commit eda0d67 into master Jul 20, 2024
@sudoforge sudoforge deleted the I957c6de57b3a9151e6d9ce8caba87f4e2c714846 branch July 20, 2024 14:44
@sudoforge sudoforge mentioned this pull request Jul 20, 2024
11 tasks
@sudoforge sudoforge changed the title A lil' bit of TLC for the pipelines Get the pipelines to a good state Jul 21, 2024
@sudoforge sudoforge changed the title Get the pipelines to a good state Make the pipelines pass Jul 21, 2024
@sudoforge sudoforge changed the title Make the pipelines pass make the pipelines pass, and other TLC Jul 21, 2024
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.

1 participant