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

Resource Option Import not respected in YAML #359

Closed
Tracked by #269
rdbartram opened this issue Oct 2, 2022 · 1 comment · Fixed by #367
Closed
Tracked by #269

Resource Option Import not respected in YAML #359

rdbartram opened this issue Oct 2, 2022 · 1 comment · Fixed by #367
Assignees
Labels
area/expr Syntax, expressions, and functions in YAML kind/bug Some behavior is incorrect or out of spec resolution/fixed This issue was fixed

Comments

@rdbartram
Copy link

What happened?

When using resource options within Pulumi YAML, pulumi doesnt seem to even try to perform a lookup and instead just jumps to creation

Steps to reproduce

{
  "resources": {
    "bob": {
      "properties": {
        "password": "P@ssw0rd1234",
        "userPrincipalName": "bob@xxx.onmicrosoft.com",
        "givenName": "bob",
        "displayName": "Bob"
      },
      "type": "azuread:index/user:User",
      "options": {
        "import": "bc6993d9-fe39-4f02-b32d-104e8e0a621b",
        "deleteBeforeReplace": false,
        "ignoreChanges": [
          "password"
        ],
        "protect": false,
        "retainOnDelete": false
      }
    }
  },
  "variables": {},
  "outputs": {}
}

pulumi up

Expected Behavior

Type Name Plan
pulumi:pulumi:Stack psyamlpulumi.ad-ad-dev
= ├─ azuread:index:User bob import

Actual Behavior

Type Name Plan
pulumi:pulumi:Stack psyamlpulumi.ad-ad-dev

  • ├─ azuread:index:User bob create

Output of pulumi about

CLI
Version 3.40.2
Go Version go1.19.1
Go Compiler gc

Plugins
NAME VERSION
azuread unknown
yaml unknown

Host
OS alpine
Version 3.14.8
Arch x86_64

This project is written in yaml

Backend
Name pulumi.com
URL https://app.pulumi.com/rdbartram
User rdbartram
Organizations rdbartram, worxspace

No dependencies found

Pulumi locates its logs in /tmp by default
warning: Failed to get information about the current stack: No current stack

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

@rdbartram rdbartram added kind/bug Some behavior is incorrect or out of spec needs-triage Needs attention from the triage team labels Oct 2, 2022
@Frassle Frassle transferred this issue from pulumi/pulumi Oct 3, 2022
@Frassle Frassle removed the needs-triage Needs attention from the triage team label Oct 4, 2022
@AaronFriel AaronFriel mentioned this issue Oct 4, 2022
43 tasks
@AaronFriel
Copy link
Member

@rdbartram thank you for reporting this! I've added it to our epic (#269) for GA. This is an important gap to close.

@AaronFriel AaronFriel added the area/expr Syntax, expressions, and functions in YAML label Oct 4, 2022
@AaronFriel AaronFriel self-assigned this Oct 4, 2022
@aq17 aq17 self-assigned this Oct 14, 2022
@aq17 aq17 closed this as completed in #367 Oct 14, 2022
@pulumi-bot pulumi-bot added the resolution/fixed This issue was fixed label Oct 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/expr Syntax, expressions, and functions in YAML kind/bug Some behavior is incorrect or out of spec resolution/fixed This issue was fixed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants