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

Jira SO-53: Remove non-groupified API from jboss-fuse/application-templates #217

Merged
merged 3 commits into from
Jul 13, 2022

Conversation

fbm3307
Copy link

@fbm3307 fbm3307 commented May 6, 2022

Non-groupified APIs were deprecated in OCP 4.7.

This PR only handles the imagestreams/Templates which are bundled in OpenShift. If any of the other imagestreams or templates are meant to still be used, they should also be similarly updated. A complete API list is available at https://docs.openshift.com/container-platform/4.10/rest_api/index.html .

Signed-off-by: Feny Mehta fbm3307@gmail.com

…plates

Signed-off-by: Feny Mehta <fbm3307@gmail.com>
@fbm3307
Copy link
Author

fbm3307 commented May 6, 2022

/assign @yselkowitz

@yselkowitz
Copy link

Templates contain subsidiary objects which themselves have apiVersions which also need to be updated, e.g. Route v1 => route.openshift.io, etc. Only those listed here as core/v1 should remain v1 here, all others should be updated accordingly.

…plates

Signed-off-by: Feny Mehta <fbm3307@gmail.com>
@fbm3307
Copy link
Author

fbm3307 commented May 13, 2022

/cc @yselkowitz Updated the PR accordingly

@@ -1,6 +1,6 @@
{
"kind": "List",
"apiVersion": "v1",
"apiVersion": "image.openshift.io/v1",

Choose a reason for hiding this comment

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

Isn't List just v1?

Copy link
Author

Choose a reason for hiding this comment

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

Changed it now

Updated the List Api as v1
Copy link

@yselkowitz yselkowitz left a comment

Choose a reason for hiding this comment

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

LGTM

@yselkowitz
Copy link

ping @cunningt @tadayosi

@tadayosi
Copy link
Member

tadayosi commented Jun 3, 2022

Looks good to me, but master is not maintained.
@cunningt Should we port this to 2.1.x.sb2.redhat-7-x?

@yselkowitz
Copy link

Ultimately, we need this in the GA branch, how should we go about getting this in there (directly, or indirectly)?

@fbm3307
Copy link
Author

fbm3307 commented Jul 13, 2022

ping @cunningt @tadayosi

@tadayosi tadayosi merged commit 18481e8 into jboss-fuse:master Jul 13, 2022
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.

None yet

3 participants