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

[auto/python] Add new API to install the Pulumi CLI #15204

Merged
merged 1 commit into from Jan 26, 2024

Conversation

julienp
Copy link
Contributor

@julienp julienp commented Jan 22, 2024

Description

Provide a way for the Automation API to install the Pulumi CLI so that Automation API can be used in a more standalone manner.

#14987

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

@pulumi-bot
Copy link
Contributor

pulumi-bot commented Jan 22, 2024

Changelog

[uncommitted] (2024-01-26)

Features

  • [auto/python] Add new API to install the Pulumi CLI
    #15204

@julienp julienp force-pushed the julienp/auto-install-py branch 3 times, most recently from 5de0ada to 2c8cc8f Compare January 22, 2024 17:00
@julienp julienp marked this pull request as ready for review January 23, 2024 09:43
sdk/python/Makefile Outdated Show resolved Hide resolved
@julienp julienp requested a review from a team January 23, 2024 11:56
@julienp julienp force-pushed the julienp/auto-install-py branch 2 times, most recently from f7cf91a to ce1e6a5 Compare January 23, 2024 15:04
sdk/python/lib/pulumi/automation/_cmd.py Outdated Show resolved Hide resolved
sdk/python/lib/pulumi/automation/_cmd.py Outdated Show resolved Hide resolved
Copy link
Member

@Frassle Frassle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine to me

sdk/python/lib/pulumi/automation/_cmd.py Outdated Show resolved Hide resolved
sdk/python/lib/pulumi/automation/_cmd.py Outdated Show resolved Hide resolved
@julienp
Copy link
Contributor Author

julienp commented Jan 24, 2024

I just realised I forgot to use the new PulumiCommand in LocalWorkspace in this PR ... updating that soon.

Nevermind, I was looking at an old checkout, but I did forget to add a test for LocalWorkspace.

@julienp julienp force-pushed the julienp/auto-install-py branch 3 times, most recently from 4745da6 to 2d8aab1 Compare January 26, 2024 10:29
@julienp julienp added this pull request to the merge queue Jan 26, 2024
Merged via the queue into master with commit c26874d Jan 26, 2024
23 of 24 checks passed
@julienp julienp deleted the julienp/auto-install-py branch January 26, 2024 20:59
@justinvp justinvp mentioned this pull request Jan 31, 2024
github-merge-queue bot pushed a commit that referenced this pull request Jan 31, 2024
### Features

- [backend] Allows the PULUMI_SKIP_CHECKPOINTS environment variable to
be set without requiring the PULUMI_EXPERIMENTAL flag.
  [#15318](#15318)

- [auto/go] Add new API to install the Pulumi CLI from the Automation
API
  [#15049](#15049)

- [auto/nodejs] Allow disabling version validation when using a custom
PulumiCommand instance
  [#15242](#15242)

- [auto/python] Add new API to install the Pulumi CLI
  [#15204](#15204)

- [auto/python] Inline programs can now be defined as async functions.
  [#15278](#15278)

- [pkg/testing] Add RequireEmptyPreviewAfterRefresh option to
ProgramTest


### Bug Fixes

- [engine] Fix a hang in preview --import-file.
  [#15203](#15203)

- [engine] Engine will now error if aliases request a parent with no
URN.
  [#15232](#15232)

- [engine] Fix stack name validation when PULUMI_DISABLE_VALIDATION
isn't set.
  [#15245](#15245)

- [engine] Fix a performance issue doing unneeded secret encryption.
  [#15306](#15306)

- [sdk/go] Include error message in local workspace validation.
  [#15279](#15279)

- [sdkgen/go] Remove pulumix references from generated enums
  [#15280](#15280)

- [programgen/nodejs] Fixes generated expression for filebase64 function
to use fs.readFileSync directly with base64 encoding option
  [#15253](#15253)

- [sdkgen/python] Use importlib.metadata instead of pkg_resources
  [#15266](#15266)


### Miscellaneous

- [auto/go] Remove obsolete note from FullyQualifiedStackName comment.
  [#15281](#15281)

- [backend/filestate] Upgrade gocloud.dev to the latest version
  [#15202](#15202)

- [sdk/python] Remove compatability code for Python 3.7 and below.
  [#15276](#15276)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants