Skip to content

Conversation

@RoniKishner
Copy link
Contributor

@RoniKishner RoniKishner commented Dec 6, 2022

Short description:

Change the API call of processedtemplates to the current template namespace

More details:

The previous call used "openshift" namespace only, this change allow a template from any namespace to be processed

https://github.com/kubevirt/common-templates/blob/master/templates/README.md

@RoniKishner
Copy link
Contributor Author

!Verified

@RoniKishner
Copy link
Contributor Author

!Verified

@myakove
Copy link
Collaborator

myakove commented Dec 7, 2022

!NEED-REBASE

@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 7, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@myakove
Copy link
Collaborator

myakove commented Dec 7, 2022

!-NEED-REBASE

@RoniKishner
Copy link
Contributor Author

!Verified

@myakove
Copy link
Collaborator

myakove commented Dec 7, 2022

@rnetser Please review

@rnetser rnetser merged commit 1e8249a into RedHatQE:main Dec 7, 2022
@RoniKishner RoniKishner deleted the template_process_namespace branch December 7, 2022 13:47
@RoniKishner
Copy link
Contributor Author

!cherry-pick v4.12

cnv-qe-bot pushed a commit that referenced this pull request Dec 28, 2022
* Allow templates from all namespaces to be processed

* Remove TODO and add comment
@cnv-qe-bot
Copy link

Cherry-picked PR Allow templates from all namespaces to be processed into v4.12

rnetser pushed a commit that referenced this pull request Dec 28, 2022
* Allow templates from all namespaces to be processed

* Remove TODO and add comment

Co-authored-by: Roni Kishner <87259550+RoniKishner@users.noreply.github.com>
myakove pushed a commit that referenced this pull request Aug 11, 2025
* Allow templates from all namespaces to be processed

* Remove TODO and add comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants