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

LTPA Manager does not deep copy the short-hand applicationImage syntax from wlapp to the LTPA Job #600

Open
2 of 3 tasks
kabicin opened this issue Dec 8, 2023 · 2 comments
Assignees
Labels
bug Something isn't working zenhub-dev

Comments

@kabicin
Copy link
Contributor

kabicin commented Dec 8, 2023

The WebSphereLibertyApplication .spec.applicationImage supports a short-hand syntax i.e. applicationImage: my-app that allows the operator to search for an OpenShift ImageStream named my-app to retrieve the image reference.

When enabling the .spec.manageLTPA: true the operator simply passes the .spec.applicationImage value from the WebSphereLibertyApplication to the LTPA Job. Instead, it should do a deep parsing of image reference from the ImageStreamTag object before passing the applicationImage for the Job to use.

  • Delivered to WLO
  • Delivered to OLO
  • Add tests
@kabicin kabicin added the bug Something isn't working label Dec 8, 2023
@kabicin kabicin self-assigned this Dec 8, 2023
@kabicin
Copy link
Contributor Author

kabicin commented Dec 8, 2023

Current workaround for 1.3.0 is to copy the .status.imageReference image into the .spec.applicationImage field for all WebSphereLibertyApplications setting .spec.manageLTPA to true.

@mingcyu
Copy link
Contributor

mingcyu commented Mar 11, 2024

Waiting on tests to get merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working zenhub-dev
Projects
None yet
Development

No branches or pull requests

2 participants