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

Update GitHub Actions workflows. #3383

Closed
wants to merge 1 commit into from

Conversation

pulumi-bot
Copy link
Contributor

This PR was automatically generated by the rollout-single-native-provider-workflow workflow in the pulumi/ci-mgmt repo, from commit 0fde7cbb0da9104cfdb68e69fe5d0ddee2bc0ec7.

@pulumi-bot pulumi-bot added the impact/no-changelog-required This issue doesn't require a CHANGELOG update label Dec 19, 2024
Copy link

Does the PR have any schema changes?

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

Copy link

codecov bot commented Dec 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 41.14%. Comparing base (02c6a40) to head (f80f4ae).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3383      +/-   ##
==========================================
- Coverage   41.17%   41.14%   -0.03%     
==========================================
  Files          85       85              
  Lines       12764    12764              
==========================================
- Hits         5255     5252       -3     
- Misses       7120     7122       +2     
- Partials      389      390       +1     

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

@blampe blampe closed this Dec 19, 2024
github-merge-queue bot pushed a commit to pulumi/ci-mgmt that referenced this pull request Dec 19, 2024
p-k has a weird bug
pulumi/pulumi-kubernetes#3382 where we aren't
able to depend on the correct "Sentinel" check.

This updates the native sentinel job to more closely match the bridged
one:

https://github.com/pulumi/ci-mgmt/blob/1243b9e2dff8ae4d4391591bcec61a38daeac0af/provider-ci/test-providers/aws/.github/workflows/run-acceptance-tests.yml#L87-L109

Other native providers don't have this problem, so I don't actually
think this will resolve the issue, but it should at least help rule out
any permission weirdness.

Edit: OK I ran this against p-k here
pulumi/pulumi-kubernetes#3383 and suddenly the
Sentinel check is available to us! So it does seem there's something
wonky about the job's settings.

Before

![image](https://github.com/user-attachments/assets/f7f48998-275a-4d40-9a5b-1be669f3d9b0)

After

![image](https://github.com/user-attachments/assets/9b85be85-2093-4435-b975-4acc2c5a75ee)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact/no-changelog-required This issue doesn't require a CHANGELOG update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants