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

[sdk-wide] go get -u all + go mod tidy automation #22852

Open
richardpark-msft opened this issue May 6, 2024 · 2 comments
Open

[sdk-wide] go get -u all + go mod tidy automation #22852

richardpark-msft opened this issue May 6, 2024 · 2 comments
Assignees
Labels
EngSys This issue is impacting the engineering system.

Comments

@richardpark-msft
Copy link
Member

richardpark-msft commented May 6, 2024

Our goal here is just to have it happen on a weekly cadence, with the merge happening automatically and the testing in the weekly pipeline to validate.

Weekly pipelines for a package should do this:

  1. go get -u all
  2. go mod tidy
  3. Run tests
  4. git commit go.mod go.sum
  5. git push

Steps 4 and 5 are probably new, but we believe some teams must have some process like this in place (for instance, Java) so hopefully we can lean a bit on what they've done.

CC: @benbp, @alzimmermsft (who probably has heard of something like this for Java)

@github-actions github-actions bot added the needs-triage This is a new issue that needs to be triaged to the appropriate team. label May 6, 2024
@benbp
Copy link
Member

benbp commented May 6, 2024

@richardpark-msft @jhendrixMSFT we have a pretty similar example of this already working in the go repo for the post-release steps: https://github.com/Azure/azure-sdk-for-go/blob/main/eng/pipelines/templates/jobs/archetype-go-release.yml#L112-L120

@benbp
Copy link
Member

benbp commented May 6, 2024

If you want to push to main directly we have this template: https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/pipelines/templates/steps/git-push-changes.yml

The weekly pipelines already appear to be wired up with github permissions.

@jhendrixMSFT jhendrixMSFT added EngSys This issue is impacting the engineering system. and removed needs-triage This is a new issue that needs to be triaged to the appropriate team. labels May 6, 2024
@richardpark-msft richardpark-msft self-assigned this May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EngSys This issue is impacting the engineering system.
Projects
None yet
Development

No branches or pull requests

3 participants