-
Notifications
You must be signed in to change notification settings - Fork 154
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
Beanstalk test failing #5
Milestone
Comments
lukehoban
added a commit
that referenced
this issue
Jul 25, 2017
Adds `make all` target to run integration tests. Adds `make all` as the default target for Travis. Temporarily disable the `beanstalk` integration test which is failing (opened #5 to track). Fix LumiJS error in examples/cpuwatch.
joeduffy
added a commit
to pulumi/pulumi-terraform
that referenced
this issue
Aug 4, 2017
Our existing code is a little confused on which resource property snapshot goes where, when it comes to Terraform states and diffs. As a result, depending on what the provider attempts to do, we don't always do the correct thing. For instance, because our Create implementation populated the *new* state (rather than *existing*) on the state attributes, and gave an empty diff, should a provider work by calling APIs like d.HasChange("foo"), rather than simply d.GetOk("foo"), it would get misled. This concrerely affected several of our existing AWS tests: in fact, this fix helps to unblock pulumi/pulumi-aws#5.
corymhall
added a commit
that referenced
this issue
Jul 11, 2024
# This is the 1st commit message: Fix import resources with provider default tags We have special logic around applying default provider tags to resources. This logic only applied to the `Check` call which means it was not applied when you were importing resources. This PR extends that logic to also run during the `Read` call. fix #4030, fix 4080 # This is the commit message #2: skip test # This is the commit message #3: fixing test # This is the commit message #4: Adding more tests # This is the commit message #5: Upgrade pulumi-terraform-bridge to v3.86.0 (#4160) This PR was generated via `$ upgrade-provider pulumi/pulumi-aws --kind=bridge --pr-reviewers=guineveresaenger`. Fixes #4091 Fixes #4137 --- - Upgrading pulumi-terraform-bridge from v3.85.0 to v3.86.0. - Upgrading pulumi-terraform-bridge/pf from v0.38.0 to v0.39.0. # This is the commit message #6: chore: run upstream provider-lint (#4120) This adds a step for running the upstream `provider-lint` make target. As part of this I had to fix some of the patches which violated some lint rules. **0009-Add-ECR-credentials_data_source.patch** - `ForceNew` does not apply to data sources **0032-DisableTagSchemaCheck-for-PF-provider.patch** - Schema have to have a `Type` - Also needed to add a ignore for `S013` which forces `Computed`, `Optional` or `Required` to be set. Looks like it can't recognize the `tagsComputed` var **0034-Fail-fast-when-PF-resources-are-dropped.patch** - Added a lint ignore for a rule which doesn't allow panics **0050-Normalize-retentionDays-in-aws_controltower_landing_.patch** - This test doesn't actually need a region or partition so replacing with a placeholder closes #4110 # This is the commit message #7: fix: CVE-2024-24791 (#4175) Fixes #4163 Upgrades minimally required Go versions to those unaffected by CVE-2024-24791.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The
examples/beanstalk
test is currently failing with the following output:The text was updated successfully, but these errors were encountered: