Skip to content

Update GitHub Actions workflows.#1120

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

Update GitHub Actions workflows.#1120
pulumi-provider-automation[bot] wants to merge 1 commit into
masterfrom
update-github-actions-workflows-694

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.

@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.

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>
@codecov

codecov Bot commented Mar 17, 2026

Copy link
Copy Markdown

Codecov Report

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

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #1120   +/-   ##
======================================
  Coverage    0.00%       0           
======================================
  Files           2       0    -2     
  Lines           5       0    -5     
======================================
+ Misses          5       0    -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.

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