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

Skip Pulumi CLI download if runner version satisfies requested CLI version #860

Merged
merged 16 commits into from
Apr 9, 2023

Conversation

ehafenmaier
Copy link
Contributor

Accidentally closed PR #837 when syncing fork.

Original PR

  • Add functionality to check GitHub runner for the existence of the Pulumi CLI.
  • If the CLI is already installed on the runner check if the runner CLI version matches the requested version from the action input.
  • If runner CLI version matches, skip downloading CLI.
  • If CLI not installed on runner or runner CLI does not match requested version, proceed to download CLI

NOTE: Skipping the CLI download will also skip the outbound network call in the getVersionObject() method, which was mentioned in #674

This resolves #789

src/libs/pulumi-cli.ts Outdated Show resolved Hide resolved
src/libs/pulumi-cli.ts Show resolved Hide resolved
@simenandre simenandre merged commit 6735f6b into pulumi:master Apr 9, 2023
@ehafenmaier ehafenmaier deleted the 789-skip-download-pulumi-cli branch April 9, 2023 15:34
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.

Optimization: Skip Download Pulumi CLI when already present
3 participants