add support for --import-file when using the Automation API

[charlie-haley]

Description

Fixes # (issue)

Checklist

• I have run make tidy to update any new dependencies
• I have run make lint to verify my code passes the lint check
  • I have formatted my code using gofumpt

• I have added tests that prove my fix is effective or that my feature works

• I have run make changelog and committed the changelog/pending/<file> documenting my change

• Yes, there are changes in this PR that warrants bumping the Pulumi Cloud API version

[github-actions]

PR is now waiting for a maintainer to take action.

Note for the maintainer: Commands available:

• /run-acceptance-tests - used to test run the acceptance tests for the project
• /run-codegen - used to test the Pull Request against downstream codegen
• /run-docs-gen - used to test the Pull Request against documentation generation

[pulumi-bot]

Changelog

[uncommitted] (2024-04-29)

Features

• [auto/{go,nodejs,python}] Add support for --import-file option on Preview with Automation API

[tgummerer]

/run-acceptance-tests
Please view the results of the acceptance tests Here

[tgummerer]

/run-acceptance-tests
Please view the results of the acceptance tests Here

[github-actions]

PR is now waiting for a maintainer to take action.

Note for the maintainer: Commands available:

• /run-acceptance-tests - used to test run the acceptance tests for the project
• /run-codegen - used to test the Pull Request against downstream codegen
• /run-docs-gen - used to test the Pull Request against documentation generation

[tgummerer]

Thanks for the PR, this looks like a good addition to the automation API.

Having a look at the code, I think this needs to be a string, to be able to pass an actual path argument to the CLI, I don't think this is just a binary flag.

[charlie-haley]

Thanks for the PR, this looks like a good addition to the automation API.

Having a look at the code, I think this needs to be a string, to be able to pass an actual path argument to the CLI, I don't think this is just a binary flag.

Ah, good point! The output of the help command is quite confusing as it says --import-file pulumi import instead of --import-file string, I believe that's happened because of pulumi import being in single quotes here:
https://github.com/pulumi/pulumi/blob/master/pkg/cmd/pulumi/preview.go#L542

I'll also update that as part of this PR unless you want me to raise a separate PR, also happy to do that.

[github-actions]

PR is now waiting for a maintainer to take action.

Note for the maintainer: Commands available:

• /run-acceptance-tests - used to test run the acceptance tests for the project
• /run-codegen - used to test the Pull Request against downstream codegen
• /run-docs-gen - used to test the Pull Request against documentation generation

[github-actions]

PR is now waiting for a maintainer to take action.

Note for the maintainer: Commands available:

• /run-acceptance-tests - used to test run the acceptance tests for the project
• /run-codegen - used to test the Pull Request against downstream codegen
• /run-docs-gen - used to test the Pull Request against documentation generation

[tgummerer]

/run-acceptance-tests
Please view the results of the acceptance tests Here

[tgummerer]

Thank you! Agreed on the docstring, it's nice to update that so the help output is less confusing. I'm fine with keeping that in the same PR, just left a minor suggestion on formatting there.

[github-actions]

PR is now waiting for a maintainer to take action.

Note for the maintainer: Commands available:

• /run-acceptance-tests - used to test run the acceptance tests for the project
• /run-codegen - used to test the Pull Request against downstream codegen
• /run-docs-gen - used to test the Pull Request against documentation generation

[tgummerer]

/run-acceptance-tests
Please view the results of the acceptance tests Here

[github-actions]

PR is now waiting for a maintainer to take action.

Note for the maintainer: Commands available:

• /run-acceptance-tests - used to test run the acceptance tests for the project
• /run-codegen - used to test the Pull Request against downstream codegen
• /run-docs-gen - used to test the Pull Request against documentation generation

[tgummerer]

/run-acceptance-tests
Please view the results of the acceptance tests Here

[tgummerer]

@charlie-haley I saw CI was still failing and had a quick look. It looks like there was an != "" missing in the Go if statement. I took the liberty to just push that to your branch, which will hopefully get the PR green, so we can go ahead and merge it.

[github-actions]

PR is now waiting for a maintainer to take action.

Note for the maintainer: Commands available:

• /run-acceptance-tests - used to test run the acceptance tests for the project
• /run-codegen - used to test the Pull Request against downstream codegen
• /run-docs-gen - used to test the Pull Request against documentation generation

[charlie-haley]

@charlie-haley I saw CI was still failing and had a quick look. It looks like there was an != "" missing in the Go if statement. I took the liberty to just push that to your branch, which will hopefully get the PR green, so we can go ahead and merge it.

Thanks! There was also an indentation problem in the stack.ts, it should hopefully all be green now

[tgummerer]

/run-acceptance-tests
Please view the results of the acceptance tests Here

[github-actions]

PR is now waiting for a maintainer to take action.

Note for the maintainer: Commands available:

• /run-acceptance-tests - used to test run the acceptance tests for the project
• /run-codegen - used to test the Pull Request against downstream codegen
• /run-docs-gen - used to test the Pull Request against documentation generation

[tgummerer]

/run-acceptance-tests
Please view the results of the acceptance tests Here

[tgummerer]

Thanks again for this contribution @charlie-haley! This is gonna go into the next pulumi release, which might happen this week (or potentially next, since we have a company offsite this week)