Skip to content

Update GitHub Actions workflows.#492

Closed
pulumi-provider-automation[bot] wants to merge 1 commit into
mainfrom
update-github-actions-workflows-677
Closed

Update GitHub Actions workflows.#492
pulumi-provider-automation[bot] wants to merge 1 commit into
mainfrom
update-github-actions-workflows-677

Conversation

@pulumi-provider-automation

Copy link
Copy Markdown
Contributor

This PR was triggered by @pose generated by the update-workflows-single-bridged-provider workflow in the pulumi/ci-mgmt repo, from commit 5eff2670d336ad2d30a5f2b7aec1b69646415f7f.

@pulumi-provider-automation pulumi-provider-automation Bot added the impact/no-changelog-required This issue doesn't require a CHANGELOG update label Mar 16, 2026
@github-actions

Copy link
Copy Markdown

Does the PR have any schema changes?

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

Maintainer note: consult the runbook for dealing with any breaking changes.

@codecov

codecov Bot commented Mar 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (14edd54) to head (e0d7114).

Additional details and impacted files
@@          Coverage Diff          @@
##            main    #492   +/-   ##
=====================================
  Coverage   0.00%   0.00%           
=====================================
  Files          3       1    -2     
  Lines        107     102    -5     
=====================================
+ Misses       107     102    -5     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

github-merge-queue Bot pushed a commit to pulumi/ci-mgmt that referenced this pull request Mar 16, 2026
…ailures (#2113)

Part of #2106

## Problem

After #2107 was merged, several providers started failing CI with:

```
FAIL github.com/pulumi/pulumi-postgresql/provider/v3/cmd/pulumi-resource-postgresql [setup failed]
```

Affected providers:
- pulumi/pulumi-postgresql#886
- pulumi/pulumi-newrelic#1312
- pulumi/pulumi-auth0#1098
- pulumi/pulumi-scm#474

## Root Cause

The `cmd/pulumi-resource-*` packages embed a generated `schema.json`
file (via `//go:embed`) that only exists after `make tfgen` runs. At
test time in CI, this file does not exist.

The old workflow ran `go test .` (root package of `provider/` only),
which never compiled the `cmd/` packages. After #2107, the `Run provider
tests` step was changed to call `make test_provider`, which uses `./...`
— and this is the first time the `cmd/` packages were compiled in CI,
exposing the pre-existing build failure.

## Fix

Change `test_provider_cmd` from `./...` to `.` in the base Makefile
template, restoring the original scope (root package of `provider/`
only). The `cmd/` packages are binary entrypoints with no tests; they do
not belong in the test scope.

## Test plan

- [ ] CI on this PR passes
- [ ] Re-run CI on one of the affected providers after this merges to
confirm the fix

## Verification PRs

PRs created from branch `pose/fix/test-provider-scope` for the 21
providers with open sub-tickets:

- 🔄 in-progress pulumi/pulumi-eks#2193
- ✅ passed pulumi/pulumi-sdwan#460
- ✅ passed pulumi/pulumi-ise#492
- ✅ passed pulumi/pulumi-snowflake#1136
- 🔄 in-progress pulumi/pulumi-cloudflare#1521
- ❌ failed pulumi/pulumi-tls#990 (unrelated:
certificate issue pulumi/pulumi-tls#962)
- ✅ passed pulumi/pulumi-dbtcloud#514
- ✅ passed pulumi/pulumi-datadog#1115
- ✅ passed pulumi/pulumi-scm#481
- ✅ passed pulumi/pulumi-digitalocean#1295
- ✅ passed pulumi/pulumi-harness#634
- ✅ passed pulumi/pulumi-gitlab#1146
- ✅ passed pulumi/pulumi-random#2027
- ✅ passed pulumi/pulumi-newrelic#1321
- ✅ passed pulumi/pulumi-postgresql#893
- ✅ passed pulumi/pulumi-auth0#1105
- ✅ passed pulumi/pulumi-meraki#508
- ✅ passed pulumi/pulumi-artifactory#1298
- ✅ passed pulumi/pulumi-junipermist#631
- ✅ passed pulumi/pulumi-okta#1120
- ✅ passed pulumi/pulumi-pagerduty#1048

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

1 participant