-
Notifications
You must be signed in to change notification settings - Fork 126
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
Update integrate with apps page #2497
Comments
Do you mean to delet the info about annotations vs. labels or to move it somewhere else. I realize that this is generic k8s stuff but I still think it's useful and may indeed go with the basic annotations. I don't object to moving it, just not sure where it should go. |
Should the keptn.sh/container annotation be moved to the "Deployment tasks" along with the pre- and post-deployment tasks? I don't think it's relevant if one is not doing tasks that have containers. Also, the guide and reference pages that discuss containers should mention/link to the info about this annotation. |
No I mostly want to move content around. The info about pre and post deployments could probably be removed since we have a full page on that already.
No, that's only relevant for the auto app discovery I think so I would move the content there. But be careful, this has nothing to do with the container runtime so those sections also don't need to be adapted for this. |
Goal
The "Integrate Keptn with your applications" page could use some updating.
Details
Depends on #2490
The text was updated successfully, but these errors were encountered: