https://pulumi.com
Join Slack
This message was deleted.
# general
s
This message was deleted.
b
I’m not following here, have you used 
pulumi import
to generate the code? if so it should have the correct name
e
No I am not using the generated code. I have written a little library and ideally it can be used by people who are converting from CDK (and therefore importing resources) and by people who start from scratch
b
ah, well the resource itself has an import resource option: https://www.pulumi.com/docs/intro/concepts/resources/options/import/
Copy code
new aws.ec2.SecurityGroup(
  "ruwentest",
  {
    description: "ruwentest",
    vpcId: "vpc-123",
    ingress: []
  },
  { import: <id> }
);
What you could do is have a configurable flag that can be toggled if you’re importing
a more advanced patten would be to optionally use a transformation to include the import id if some external thing detects you’re doing an import https://www.pulumi.com/docs/intro/concepts/resources/options/transformations/
e
Yes I am actually dping that.. But if I import it like this and if I run afterwards 
pulumi refreh
and 
pulumi up
pulumi wants to replace the security group because the name has changed
b
can you show me an example output of that? you probably want to alias to the old name: https://www.pulumi.com/docs/intro/concepts/resources/options/aliases/
e
gimme a sec, I run through it again and post my code here
my security group
my code
Copy code
new aws.ec2.SecurityGroup(
  "ruwentest",
  {
    description: "ruwentest",
    vpcId: standardAwsEnvironment.getVpcId(),
    ingress: [],
    // name: "ruwentest-52f4618",
  },
  { import: "sg-0c09aeb634822eed9" }
);
pulumi autogenerates the name 
ruwentest-5f93af2
and that clashes with the existing name 
ruwentest-52f4618
Does it make sense @billowy-army-68599?
b
Why can’t you just make the name match the resource?
e
It is generic library code. I would like to benefit from the auto-naming features like zero downtime deployments
b
Hmm, I’m not sure that’ll be possible, you can’t import a resource without the properties matching
You could use ignoreChanges for the name, but that presents its own problems
e
Since the name is normally auto-generated, I was hoping that I can somehow tweak pulumi to use the name form the imported resource instead of auto generating it. It would make the import experience much nicer
b
You can, but that requires setting the name property 😅
e
Logically I would expect something like that happening in pulumi: • generate a model how the resource has to look like based on the code • auto-generate names • read resource from cloud provider • diff And I would like it to be: • generate a model how the resource has to look like based on the code • read resources from cloud provider • if custom option X is set, fill in all auto generated names with values from the cloud provider • diff
but looking at the code, it might not be that simple 😉
But I would expect that from such a change various resources and their import process would benefit
If I specify the name during the import phase and remove it afterwards, pulumi still wants to generate a new name and therefore replace the resource
b
Yes that’s all expected behaviour, the way you want it to operate would be a feature request, so I’d recommend filing a GitHub issue
e
Cool I'll do it. I wanted to first check that I don't miss anything obvious given that I am new to pulumi. Thanks!
would it be on the pulumi project itself? or on teh aws-provider?
b
It’d be in the main repo I think the team will triage it as needed
e
thx
s
@echoing-address-44214 do you have a link to your GitHub issue?
e
20 Views
Open in Slack
PreviousNext
Powered by

Pulumi Community

No matter how you like to participate in developer communities, Pulumi wants to meet you there. If you want to meet other Pulumi users to share use-cases and best practices, contribute code or documentation, see us at an event, or just tell a story about something cool you did with Pulumi, you are part of our community.

Powered by