Skip to content

feat: adding KnativeServing CR ORCS support #2260

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

Merged
merged 39 commits into from
Jul 14, 2025
Merged

feat: adding KnativeServing CR ORCS support #2260

merged 39 commits into from
Jul 14, 2025

Conversation

Ani1357
Copy link
Contributor

@Ani1357 Ani1357 commented Jun 20, 2025

📌 Summary

APL-887
This PR configures the KnativeServing CR to use images from ORCS when useORCS is set to true.
Knative documentation

🔍 Reviewer Notes

🧹 Checklist

  • Code is readable, maintainable, and robust.
  • Unit tests added/updated

Copy link
Contributor

@j-zimnowoda j-zimnowoda left a comment

Choose a reason for hiding this comment

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

Some inconsistencies and typos.

@CasLubbers CasLubbers requested a review from j-zimnowoda June 30, 2025 14:35
@CasLubbers
Copy link
Contributor

Hardcoded the version in the gotmpl. Also there is already a version in there which needs to be updated. So I think its fine.

@merll merll requested a review from CasLubbers July 8, 2025 13:37
@j-zimnowoda j-zimnowoda dismissed their stale review July 14, 2025 14:31

others took over

@j-zimnowoda j-zimnowoda enabled auto-merge (squash) July 14, 2025 14:32
@j-zimnowoda j-zimnowoda merged commit 9871159 into main Jul 14, 2025
11 checks passed
@j-zimnowoda j-zimnowoda deleted the APL-887 branch July 14, 2025 14:32
j-zimnowoda added a commit that referenced this pull request Jul 16, 2025
merll pushed a commit that referenced this pull request Jul 16, 2025
Co-authored-by: svcAPLBot <174728082+svcAPLBot@users.noreply.github.com>
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