-
Notifications
You must be signed in to change notification settings - Fork 19
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
Have a more meaningful "name" in examples? #7
Comments
Based on my understanding, reviewing it again and talking to a few people. I agree this is a little confusing in this example. For SpringBoot:
For Thorntail:
Perhaps we could rename |
I am still confused by this use of Eg. with the old redhat MSA helloworld ( https://github.com/redhat-helloworld-msa/helloworld-msa ) app.kubernetes.io/part-of: helloworld-msa
app.kubernetes.io/name: aloha
app.kubernetes.io/version: 1.0.1
app.kubernetes.io/managed-by: odo
app.openshift.io/runtime: vert.x
app.openshift.io/runtime-version: 3.8 The So I could perfectly have a single instance of helloworld-msa, in which case I don't need to specify app.kubernetes.io/instance: aloha-prod
# or
app.kubernetes.io/instance: aloha-demo
# or
app.kubernetes.io/instance: aloha-summit We must read And finally there's the |
Hi @gorkem ,
In the examples provided here, "nodejs" is used both as app.openshift.io/runtime and as app.kubernetes.io/name . I think it can be confusing when people try to understand what exactly they should put under "name".
Having something that looks more business-related would perhaps help to remove any ambiguity. What about using names like there is in some tutorials like here : https://redhat-developer-demos.github.io/istio-tutorial/istio-tutorial/1.1.x/index.html ; that would gives "names" such as "customer", "preference", "recommendation".
What do you think?
The text was updated successfully, but these errors were encountered: