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

ProgramTest upgrade tests can fail right after recording #76

Open
VenelinMartinov opened this issue Mar 15, 2024 · 0 comments
Open

ProgramTest upgrade tests can fail right after recording #76

VenelinMartinov opened this issue Mar 15, 2024 · 0 comments
Labels
impact/reliability Something that feels unreliable or flaky kind/engineering Work that is not visible to an external user

Comments

@VenelinMartinov
Copy link
Contributor

What happened?

Looks like under some conditions, the upgrade tests fail right after being recorded, when no changes have happened.

In this example, the diff has:

json_match.go:135: [#["stables"]] unexpected value [
              "staging"
            ]

For the resource under test, the staging property is not available in the baseline version and has a default in the newer version. We probably compare a diff using the old version to a diff using the new version?

Example

https://github.com/pulumi/pulumi-aws/actions/runs/8265739539/job/22614819405?pr=3610

Output of pulumi about

pulumi about
CLI
Version      3.108.1
Go Version   go1.22.0
Go Compiler  gc

Host
OS       darwin
Version  14.1.1
Arch     arm64

Backend
Name           pulumi.com
URL            https://app.pulumi.com/venelin-pulumi-corp
User           venelin-pulumi-corp
Organizations  venelin-pulumi-corp, demo, pulumi
Token type     personal

tested on pulumi/pulumi-aws@17db4d7

Additional context

No response

Contributing

Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

@VenelinMartinov VenelinMartinov added kind/bug Some behavior is incorrect or out of spec needs-triage Needs attention from the triage team labels Mar 15, 2024
@mjeffryes mjeffryes added kind/engineering Work that is not visible to an external user impact/reliability Something that feels unreliable or flaky and removed needs-triage Needs attention from the triage team kind/bug Some behavior is incorrect or out of spec labels Mar 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact/reliability Something that feels unreliable or flaky kind/engineering Work that is not visible to an external user
Projects
None yet
Development

No branches or pull requests

2 participants