Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Manually pin Container instance for v3 #3912

Merged
merged 2 commits into from
Jan 30, 2025

Conversation

danielrbradley
Copy link
Member

  • Only ContainerGroup and ContainerGroupProfile are actually listed as live.
  • The latest live version with both resources is 2024-05-01-preview so we'll track that for v3.
  • Exclude the NGroup resource as it's not live and causes conflicts for now.

This lets us remove the manual type merging override.

- Only ContainerGroup and ContainerGroupProfile are actually listed as live.
- The latest live version with both resources is 2024-05-01-preview so we'll track that for v3.
- Exclude the NGroup resource as it's not live and causes conflicts for now.

This lets us remove the manual type merging override.
Copy link

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

Copy link

codecov bot commented Jan 30, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 57.03%. Comparing base (e78802a) to head (997c17e).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3912      +/-   ##
==========================================
- Coverage   57.04%   57.03%   -0.01%     
==========================================
  Files          79       79              
  Lines       12549    12549              
==========================================
- Hits         7158     7157       -1     
- Misses       4840     4841       +1     
  Partials      551      551              

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

Copy link
Contributor

@thomas11 thomas11 left a comment

Choose a reason for hiding this comment

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

2024-05 is also what the latest beta of the Go SDK uses (via). Latest stable is still on 2023-05, though.

@danielrbradley danielrbradley enabled auto-merge (squash) January 30, 2025 13:15
@danielrbradley danielrbradley merged commit f1cae56 into master Jan 30, 2025
23 checks passed
@danielrbradley danielrbradley deleted the pin-v3-container-instance branch January 30, 2025 13:47
@pulumi-bot
Copy link
Contributor

This PR has been shipped in release v2.86.0.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants