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

Add missing CLI environment variables #12567

Merged
merged 1 commit into from
Aug 22, 2024
Merged

Conversation

mikhailshilkov
Copy link
Member

Proposed changes

Apparently, we miss many environment variables in our docs. I added all the ones that I could find (with Claude's help).

Unreleased product version (optional)

Related issues (optional)

@pulumi-bot
Copy link
Collaborator

Copy link
Contributor

@cnunciato cnunciato left a comment

Choose a reason for hiding this comment

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

Thanks @mikhailshilkov!

Do you know if it's possible to generate these programmatically? It'd be great to not have to remember to do this.

<p>
Specifies the endpoint for Pulumi AI service.
</p>
<pre><code class="text-xs">PULUMI_AI_SERVICE_ENDPOINT="https://ai.pulumi.com"</code></pre>
Copy link
Contributor

Choose a reason for hiding this comment

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

Actually .. is this URL a thing? Doesn't seem to resolve for me.

</dt>
<dd>
<p>
Enable features for hacking on pulumi itself.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Enable features for hacking on pulumi itself.
Enable features for hacking on Pulumi itself.

</dt>
<dd>
<p>
Disables provider preview and enabled previous more concervative preview behavior.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Disables provider preview and enabled previous more concervative preview behavior.
Disables provider preview and enables previous more conservative preview behavior.

Incidentally, what does "more conservative" mean"? Might be good to explain this a little.

</dt>
<dd>
<p>
Specifies overrides for Plugin Download URLs. The expected format is `regexp=URL`, and multiple pairs can be specified separated by commas.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Specifies overrides for Plugin Download URLs. The expected format is `regexp=URL`, and multiple pairs can be specified separated by commas.
Specifies overrides for plugin-download URLs. The expected format is <code>regexp=URL</code>, and multiple pairs can be specified separated by commas.

</dd>
<dt>
<span class="font-mono">
PULUMI_DISABLE_RESOURCE_REFERENCES
Copy link
Member

Choose a reason for hiding this comment

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

I don't think we need this anymore, so not sure we should even document it. Rather, we should clean up its usage.

</dd>
<dt>
<span class="font-mono">
PULUMI_DISABLE_OUTPUT_VALUES
Copy link
Member

Choose a reason for hiding this comment

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

Similar to PULUMI_DISABLE_RESOURCE_REFERENCES, though I'm not sure we're quite ready to get rid of it. I don't think we're going to want to maintain this long term, so not sure we should doc it here.

@dirien dirien merged commit 7976af4 into master Aug 22, 2024
19 checks passed
@dirien dirien deleted the mikhailshilkov/env-vars branch August 22, 2024 06:12
@dirien
Copy link
Contributor

dirien commented Aug 22, 2024

Hey @cnunciato and @mikhailshilkov : I accidentally clicked on merge as I thought I had a different PR open to review. (stupid mobile app).

Sorry you may need to reopen this.

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.

5 participants