-
Notifications
You must be signed in to change notification settings - Fork 62
Inconsistencies in locally generated and docker-hub F8UI deploy images #2368
Comments
In order to work out a list of necessary and sufficient In the 2 CI pipelines used in
It's important to notice, the artifacts built on CICO aren't meant to run the app, and the app is never validated to run as well. It's only purpose is to run the tests, and it does just that. Meaning, the configs from this cite can't be used as reference as it doesn't build into a functional app. F8CI, on the other hand, is meant to build a functional app and it pushes the generated image to registry as well as deploys it to stage. However, it does so with some undocumented & hard-to-discover behind-the-scene-magic (which may or may not be) resulting in the image that doesn't work out of the box, as it'd commonly be expected to. |
From the list of configs (not Following is a draft grouping of configs to classify them based off of this:
NOTE: still unresolved whether the required configs mentioned here are sufficient for the app to run. |
@debloper The next line https://github.com/fabric8-ui/fabric8-ui-openshift-nginx/blob/master/root/template.sh#L9, actually substitutes the environment variable values in the file. When you run fabric8-ui image as
But if you run this as
All the environment variables are optional as the defaults are set.
|
The point of this issue was, it wasn't happening as the way it should. You're explaining how it should. Now, there can be two things right now:
Can't tell which one is it, and at thins point it might just be worth more to just ignore this whole planner-as-a-library-built-with-f8ui-creating-devops-bugs-whack-a-mole-nightmare and move on to standalone planner devops and focus more on making it better than what we have right now. |
/usr/share/nginx/html
contains a_config
directory in CI images, whereas local builds haveconfig
directory_config/fabric8-ui.env.js
are blank string in docker-hub images, where local one has weird values to them.In locally built depoly image:
compared to, docker-hub deploy image:
The text was updated successfully, but these errors were encountered: