search

projectatomic / ContainerApplicationGenericLabels

Default Containerized Application Labels used to document the application and/or image
67 stars 29 forks source link

A label for changelog #38

Closed rwngwn closed 9 years ago

rwngwn commented 9 years ago

Hi, I'm now working on a way how to put changelog information to OpenShift Xpaas JBoss MW images and I'd like to see some more standard way to attach some release notes/changelog to an image even for regular container application.

Ideally it should be two labels like:

Name Description
changelog a brief release notes/description whats new in image
changelog_url url to a webpage containing description of all new shiny stuff
goldmann commented 9 years ago

I do not necessarily like the changelog_url label. IMHO it would be better to have only changelog, where the content could be one of:

  1. Short description of the change (image respin is one example), or
  2. Link to release notes.
vpavlin commented 9 years ago

I'd suggest adding just changelog which would strictly contain a link to release notes. I envision labels more as machine-readable information and thus pasting arbitrary text doesn't make much sense to me.

goldmann commented 9 years ago

Sure, I'm cool with this. Should we send a PR for this?

rhatdan commented 9 years ago

I would prefer to update this pull request if that is the consensus.

goldmann commented 9 years ago

There is no PR submitted to fix this issue, yet. I'll prepare one.

jmtd commented 9 years ago

This looks good to me. I hope we can get this into the next release of xPaas images.