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

Clean up leftover GoReleaser files & config #1074

Merged
merged 2 commits into from
Sep 10, 2024
Merged

Conversation

ringods
Copy link
Member

@ringods ringods commented Sep 9, 2024

Cleanup last GoReleaser based files & config.

Copy link
Member

@danielrbradley danielrbradley left a comment

Choose a reason for hiding this comment

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

Based on how this is rolled-out, I think we need the ci-mgmt tool to actively delete these files. Only the .github folder is wiped out completely - all other files are maintained by default.

@ringods
Copy link
Member Author

ringods commented Sep 9, 2024

@danielrbradley add the delete of the obsolete files here?

- name: Delete existing workflows
# Bridged providers completely replace workflow files from ci-mgmt currently except for custom workflows
# specific to the provider, named `provider*.yml` such as `aws-run-tests.yml`.
if: inputs.bridged
run: |
find pulumi-${{ inputs.provider_name }}/.github/workflows/*.yml -type f ! -name '${{inputs.provider_name}}*.yml' -delete

@danielrbradley
Copy link
Member

@danielrbradley add the delete of the obsolete files here?

- name: Delete existing workflows
# Bridged providers completely replace workflow files from ci-mgmt currently except for custom workflows
# specific to the provider, named `provider*.yml` such as `aws-run-tests.yml`.
if: inputs.bridged
run: |
find pulumi-${{ inputs.provider_name }}/.github/workflows/*.yml -type f ! -name '${{inputs.provider_name}}*.yml' -delete

Yes there or here: https://github.com/pulumi/ci-mgmt/blob/master/provider-ci%2Finternal%2Fpkg%2Fgenerate.go#L78-L87

Copy link
Member

@danielrbradley danielrbradley left a comment

Choose a reason for hiding this comment

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

Looks good. The release workflow should be tested during the merge.

@ringods ringods added this pull request to the merge queue Sep 10, 2024
Merged via the queue into master with commit 8fc0b60 Sep 10, 2024
6 checks passed
@ringods ringods deleted the cleanup-goreleaser branch September 10, 2024 09:31
ringods added a commit to pulumi/pulumi-gcp that referenced this pull request Sep 12, 2024
Remove the flag `skipGoReleaserHooks` from `.ci-mgmt.yml` because
support for this was removed a while ago, with the last bits removed in
pulumi/ci-mgmt#1074

Change applied based on this section which was removed from
https://github.com/pulumi/ci-mgmt/blob/master/provider-ci/internal/pkg/templates/bridged-provider.config.yaml:

```
# Go releaser: skip the post cleaning steps
# Only used for gcp: https://github.com/search?q=org%3Apulumi+path%3A.ci-mgmt.yaml+%22skipGoReleaserHooks%3A%22&type=code
#skipGoReleaserHooks: false
```
ringods added a commit to pulumi/pulumi-minio that referenced this pull request Sep 12, 2024
Remove the flag `skipWindowsArmBuild` from `.ci-mgmt.yml` because
support for this was removed a while ago, with the last bits removed in
pulumi/ci-mgmt#1074

Change applied based on this section which was removed from
https://github.com/pulumi/ci-mgmt/blob/master/provider-ci/internal/pkg/templates/bridged-provider.config.yaml:

```
# Go releaser: add ignore for arm64 windows.
# Only used for minio: https://github.com/search?q=org%3Apulumi+path%3A.ci-mgmt.yaml+%22skipWindowsArmBuild%3A%22&type=code
#skipWindowsArmBuild: false
```
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.

3 participants