-
Notifications
You must be signed in to change notification settings - Fork 13
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
Restore Image.repoDigest output #530
Conversation
PR is now waiting for a maintainer to run the acceptance tests. |
PR is now waiting for a maintainer to run the acceptance tests. |
PR is now waiting for a maintainer to run the acceptance tests. |
491c635
to
bc43269
Compare
PR is now waiting for a maintainer to run the acceptance tests. |
bc43269
to
1b0edeb
Compare
PR is now waiting for a maintainer to run the acceptance tests. |
ExtraRuntimeValidation: func(t *testing.T, stack integration.RuntimeValidationStackInfo) { | ||
digest, ok := stack.Outputs["repoDigest"].(string) | ||
assert.True(t, ok) | ||
assert.NotEmpty(t, digest) | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@guineveresaenger added an assertion here
/run-acceptance-tests |
Please view the PR build: https://github.com/pulumi/pulumi-docker/actions/runs/4440579373 |
1b0edeb
to
cf34ff4
Compare
PR is now waiting for a maintainer to run the acceptance tests. |
/run-acceptance-tests |
Please view the PR build: https://github.com/pulumi/pulumi-docker/actions/runs/4441968511 |
I looked at the test again, and the output is definitely v4 output now. |
cf34ff4
to
4961814
Compare
PR is now waiting for a maintainer to run the acceptance tests. |
The Image.repoDigest property was removed in v4 as part of the transition to use the Docker SDK instead of the Docker CLI. Fortunately the SDK exposes a method that makes it easy to construct the repo digest. Fix pulumi#507. Co-authored-by: Aaron Friel <mayreply@aaronfriel.com>
4961814
to
db12af4
Compare
PR is now waiting for a maintainer to run the acceptance tests. |
/run-acceptance-tests |
Please view the PR build: https://github.com/pulumi/pulumi-docker/actions/runs/4464049883 |
The Image.repoDigest property was removed in v4 as part of the transition to use the Docker SDK instead of the Docker CLI. Fortunately the SDK exposes a method that makes it easy to construct the repo digest.
Fix #507.