Skip to content

feat(ci): upgrade to latest pulumi/ci-mgmt scaffolding - #350

Draft
ctreatma wants to merge 7 commits into
mainfrom
pulumi-ci-mgmt
Draft

ctreatma wants to merge 7 commits into
mainfrom
pulumi-ci-mgmt

Conversation

@ctreatma

Copy link
Copy Markdown
Contributor

The provider maintenance tools used in this repo have fallen far behind what is published by Pulumi. This brings us back in line with the latest pulumi/ci-mgmt tools so that we and our customers can benefit from the various improvements Pulumi has made over the past 18 months or more.

Unfortunately, because we let this get so outdated, this upgrade is guaranteed to lead to a lot of changes in SDK code. We will need to review those changes to confirm that new generated code is, as much as possible, functionally equivalent to previous generated code. That said, we may need to lean on our current v0 state to write off the changes so we can follow better provider maintenance practices going forward.

@ctreatma

Copy link
Copy Markdown
Contributor Author

I've successfully generated and reviewed the SDK changes locally using the devcontainer setup, but omitting those here for now so I can take a crack at testing the updated CI workflows.

One particular concern I have with the updated CI is that it breaks (I think?) our previous release and versioning approach; depending on how opinionated pulumi/ci-mgmt is about this stuff, we may need to give up on some of our previous release workflow behavior.

Comment on lines 3 to +7
on:
workflow_dispatch:
push:
tags:
- v*.*.*
- "!v*.*.*-**"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Our previous release workflow was a push-button workflow that determined the next release version by analyzing Conventional Commit tags in the git logs since the last release. Since this workflow file is autogenerated now, if we want to stick with push-button releases with version numbers determined by Conventional Commit tags, we can add a separate push-button workflow that determines the next release version and pushes that tag in order to trigger this workflow.

Comment thread .github/workflows/publish.yml Outdated
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