Skip to content

docs: add FUNC_USERNAME/PASSWORD env vars documentation#6543

Open
Kunal1522 wants to merge 1 commit intoknative:mainfrom
Kunal1522:knative-fast-b
Open

docs: add FUNC_USERNAME/PASSWORD env vars documentation#6543
Kunal1522 wants to merge 1 commit intoknative:mainfrom
Kunal1522:knative-fast-b

Conversation

@Kunal1522
Copy link
Contributor

Fixes knative/func#3299

Proposed Changes

  • Added documentation for FUNC_USERNAME and FUNC_PASSWORD environment variables in docs/snippets/proc-building-function.md.
  • Added documentation for FUNC_USERNAME and FUNC_PASSWORD environment variables in docs/snippets/proc-deploying-function.md.
  • Clarified that these environment variables now work with all builders (host, s2i, and pack) for private registry authentication, following the refactoring in knative/func#3298.
  • Since these are snippets, the updates will automatically appear in the "Building functions", "Deploying functions", and "Getting started" documentation pages.

@netlify
Copy link

netlify bot commented Dec 18, 2025

Deploy Preview for knative ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit e091285
🔍 Latest deploy log https://app.netlify.com/projects/knative/deploys/6985e41f49df0a0008fdee4f
😎 Deploy Preview https://deploy-preview-6543--knative.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Dec 18, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: Kunal1522 / name: Kunal Kashyap (e091285)

@knative-prow
Copy link

knative-prow bot commented Dec 18, 2025

Welcome @Kunal1522! It looks like this is your first PR to knative/docs 🎉

@knative-prow knative-prow bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Dec 18, 2025
@Cali0707
Copy link
Member

/cc @gauron99 @lkingland

@knative-prow knative-prow bot requested review from gauron99 and lkingland December 18, 2025 16:40
@gauron99
Copy link
Contributor

Thanks for the ping. I commented on the mentioned issue here

@Kunal1522 Kunal1522 marked this pull request as draft December 26, 2025 19:36
@knative-prow knative-prow bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 26, 2025
@dprotaso
Copy link
Member

dprotaso commented Feb 5, 2026

@gauron99 is this docs change still relevant?

@knative-prow
Copy link

knative-prow bot commented Feb 6, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Kunal1522
Once this PR has been reviewed and has the lgtm label, please assign leo6leo for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@gauron99
Copy link
Contributor

gauron99 commented Feb 6, 2026

i think the verify mkdocs strict check is unrelated
cc @dprotaso

@dprotaso
Copy link
Member

dprotaso commented Feb 6, 2026

@gauron99 it's unrelated - you can lgtm and approve when you're ready and it won't block the PR

@dprotaso
Copy link
Member

dprotaso commented Feb 6, 2026

The PR is still a draft

@Kunal1522
Copy link
Contributor Author

my bad i missed to mark it for review

@Kunal1522 Kunal1522 marked this pull request as ready for review February 6, 2026 17:52
@knative-prow knative-prow bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 6, 2026
@knative-prow knative-prow bot requested a review from Leo6Leo February 6, 2026 17:52
@gauron99
Copy link
Contributor

gauron99 commented Feb 8, 2026

@Kunal1522 could we mention this for func run as well?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add release note and docs to credentials refactoring.

4 participants