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

chore: gh action to upgrade provider dependency in Pulumi provider #452

Merged
merged 2 commits into from
Nov 13, 2023

Conversation

ocobles
Copy link
Contributor

@ocobles ocobles commented Nov 6, 2023

This action will open an Issue in equinix/pulumi-equinix when there is a new release, and this in turn will trigger https://github.com/equinix/pulumi-equinix/blob/main/.github/workflows/upgrade-provider.yml

ocobleseqx added 2 commits November 7, 2023 00:12
@codecov-commenter
Copy link

Codecov Report

Attention: 60 lines in your changes are missing coverage. Please review.

Comparison is base (26b2f3d) 60.11% compared to head (952291e) 60.01%.
Report is 41 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #452      +/-   ##
==========================================
- Coverage   60.11%   60.01%   -0.11%     
==========================================
  Files          99       99              
  Lines       20045    20076      +31     
==========================================
- Hits        12051    12048       -3     
- Misses       7691     7724      +33     
- Partials      303      304       +1     
Files Coverage Δ
equinix/config.go 64.36% <100.00%> (+1.06%) ⬆️
equinix/errors.go 86.95% <100.00%> (+0.59%) ⬆️
equinix/fabric_cloud_router_schema.go 100.00% <100.00%> (ø)
equinix/fabric_connection_schema.go 100.00% <100.00%> (ø)
equinix/helpers_device.go 82.26% <97.14%> (-2.65%) ⬇️
equinix/data_source_metal_device.go 88.33% <89.47%> (-0.71%) ⬇️
equinix/resource_fabric_cloud_router.go 9.69% <0.00%> (-0.05%) ⬇️
equinix/resource_metal_device.go 78.00% <87.50%> (-1.08%) ⬇️
equinix/resource_fabric_connection.go 5.75% <0.00%> (-0.09%) ⬇️
equinix/fabric_mapping_helper.go 0.74% <0.00%> (-0.03%) ⬇️

... and 5 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@displague displague left a comment

Choose a reason for hiding this comment

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

lgtm

As a next step, could we trigger a PR on Pulumi to build the update?

@ocobles
Copy link
Contributor Author

ocobles commented Nov 7, 2023

lgtm

As a next step, could we trigger a PR on Pulumi to build the update?

Yes @displague, It is already in place, I took it from Pulumi, it is the method that is used for all the providers that they maintain. https://github.com/equinix/pulumi-equinix/blob/c73136ee0a1a8f9c4e3ccb6c9edd834aa2f6f967/.github/workflows/upgrade-provider.yml#L6-L8

@ocobles ocobles merged commit 2c30ecf into main Nov 13, 2023
5 of 6 checks passed
@ocobles ocobles deleted the pulumi-releases branch November 13, 2023 21:28
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