Skip to content

Conversation

qswinson
Copy link
Contributor

Adds documentation for Supporting using pod default credentials for Integration Source and Sink

Proposed Changes

This adds documentation for using an AWS IAM role as the credentials for IntegrationSource and IntegrationSink.

Copy link

netlify bot commented Sep 17, 2025

Deploy Preview for knative ready!

Name Link
🔨 Latest commit 2e15048
🔍 Latest deploy log https://app.netlify.com/projects/knative/deploys/68e69db300111700087fddde
😎 Deploy Preview https://deploy-preview-6394--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.

@knative-prow knative-prow bot requested review from creydr and pierDipi September 17, 2025 20:57
@knative-prow knative-prow bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Sep 17, 2025
@dprotaso
Copy link
Member

/hold for eventing PR - knative/eventing#8731

@knative-prow knative-prow bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 18, 2025
@dprotaso
Copy link
Member

/lgtm
/approve

@knative-prow knative-prow bot added the lgtm Indicates that a PR is ready to be merged. label Sep 18, 2025
@dprotaso dprotaso assigned creydr and unassigned dprotaso Sep 18, 2025
@knative-prow knative-prow bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 18, 2025
@evankanderson
Copy link
Member

Sorry about the merge -- in #6398 , I moved all the versioned documentation under versioned, to separate it from the blog, testimonials, community pages that are not version-specific. You should be able to just move these files to the new location; I didn't need to change any contents.

@knative-prow-robot knative-prow-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Sep 23, 2025
@knative-prow knative-prow bot removed the lgtm Indicates that a PR is ready to be merged. label Oct 8, 2025
@knative-prow-robot knative-prow-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 8, 2025
@dprotaso
Copy link
Member

dprotaso commented Oct 8, 2025

/retest

@dprotaso
Copy link
Member

dprotaso commented Oct 8, 2025

/ok-to-test

@knative-prow knative-prow bot added the ok-to-test Indicates a non-member PR verified by an org member that is safe to test. label Oct 8, 2025
@dprotaso
Copy link
Member

dprotaso commented Oct 8, 2025

interesting - hitting a github rate limit - ithub.GithubException.RateLimitExceededException: 403 {"message": "API rate limit exceeded for 3.141.103.128. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)", "documentation_url": "https://docs.github.com/rest/overview/resources-in-the-rest-api#rate-limiting"}

@dprotaso
Copy link
Member

dprotaso commented Oct 8, 2025

/hold cancel

@knative-prow knative-prow bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 8, 2025
@dprotaso
Copy link
Member

dprotaso commented Oct 8, 2025

GitHub rate limiting
/retest

@dprotaso
Copy link
Member

dprotaso commented Oct 8, 2025

I'm going to accept my own suggestions and then merge this PR

@dprotaso
Copy link
Member

dprotaso commented Oct 8, 2025

/lgtm
/approve

I just added links to AWS docs for IAM for SA and Pod Identity

@knative-prow knative-prow bot added the lgtm Indicates that a PR is ready to be merged. label Oct 8, 2025
Copy link

knative-prow bot commented Oct 8, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dprotaso, qswinson

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

The pull request process is described here

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

@knative-prow knative-prow bot merged commit 9d0e598 into knative:main Oct 8, 2025
19 checks passed
@dprotaso
Copy link
Member

dprotaso commented Oct 8, 2025

thanks @qswinson

@qswinson qswinson deleted the aws-default-creds branch October 8, 2025 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. ok-to-test Indicates a non-member PR verified by an org member that is safe to test. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants