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

Revisit import issues #3875

Closed
t0yv0 opened this issue Apr 26, 2024 · 7 comments
Closed

Revisit import issues #3875

t0yv0 opened this issue Apr 26, 2024 · 7 comments
Assignees
Labels
area/import An issue related to `pulumi import` or the import resource option. kind/task Work that's part of an ongoing epic resolution/fixed This issue was fixed

Comments

@t0yv0
Copy link
Member

t0yv0 commented Apr 26, 2024

#2318 import subnet warnings
#3670
#3158 (awaiting/upstream)

#1528 ((awaiting/upstream)

#1940 (needs-repro) import LaunchTemplate warnings
#2410 (needs-repro)

#2318 conflicting arguments on subnet import (+13) bridge issue

#2852 quickSight.Analyiss

#2854 import (no repro)

#3311 tags import

#3986

#4054

@t0yv0 t0yv0 added kind/task Work that's part of an ongoing epic area/import An issue related to `pulumi import` or the import resource option. labels Apr 26, 2024
@flostadler
Copy link
Contributor

Another tags import issue: #4030

@t0yv0
Copy link
Member Author

t0yv0 commented Jul 22, 2024

Specifically looking here for opportunities/recommendations to fix import-related issues globally, in the bridge or in Pulumi itself.

pulumi/pulumi-terraform-bridge#2028 has an assortment of bridged provider import issues this can help inform.

@t0yv0
Copy link
Member Author

t0yv0 commented Jul 22, 2024

For example, perhaps if it is possible to fix #4054 generically by dropping Read entries that do not pass ValidateDiagFunc that could also improve the ecosystem as a whole, in e.g. linode.

@corymhall
Copy link
Contributor

Unfortunately I haven't found any themes among the issues linked in the ticket. Everything related to tags will hopefully be fixed with #4169, but everything else seems to be a unique issue or is no longer an issue.

@t0yv0
Copy link
Member Author

t0yv0 commented Jul 22, 2024

That's a great result too!

@mjeffryes mjeffryes modified the milestone: 0.107 Jul 24, 2024
@t0yv0
Copy link
Member Author

t0yv0 commented Sep 27, 2024

Counting this as done, thanks so much!

@t0yv0 t0yv0 closed this as completed Sep 27, 2024
@pulumi-bot pulumi-bot reopened this Sep 27, 2024
@pulumi-bot
Copy link
Contributor

Cannot close issue:

  • does not have required labels: resolution/

Please fix these problems and try again.

@t0yv0 t0yv0 added the resolution/fixed This issue was fixed label Sep 27, 2024
@t0yv0 t0yv0 closed this as completed Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/import An issue related to `pulumi import` or the import resource option. kind/task Work that's part of an ongoing epic resolution/fixed This issue was fixed
Projects
None yet
Development

No branches or pull requests

5 participants