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

Please make proper releases and keep a Changelog #489

Open
dserodio opened this issue Aug 21, 2020 · 8 comments
Open

Please make proper releases and keep a Changelog #489

dserodio opened this issue Aug 21, 2020 · 8 comments
Assignees
Labels
keepalive Use to prevent automatic closing

Comments

@dserodio
Copy link

Making proper tagged releases and keeping a changelog would make managing and upgrading kubernetes-mixin much easier.

I suggest following the keepachangelog.com standard

Thanks!

@povilasv
Copy link
Contributor

What do you mean by proper releases? Keep in mind we need to make a release for each minor release of k8s if it changes metrics, etc.

At the moment, our minor releases follow that. You can see more info in this discussion: https://kubernetes.slack.com/archives/CAX9GU941/p1596108610030400

@s-urbaniak
Copy link
Contributor

I think what is meant to maintain a changelog and a release in the release page of github https://github.com/kubernetes-monitoring/kubernetes-mixin/releases ?

If yes, then I am definitely 👍 on compiling a list of changes for a given release. In addition to bumping a Kubernetes compatibility version we merge in a lot of changes which are worth explicitly being called out.

wdyt @brancz @csmarchbanks @metalmatze @tomwilkie ?

@povilasv
Copy link
Contributor

I think what is meant to maintain a changelog and a release in the release page of github https://github.com/kubernetes-monitoring/kubernetes-mixin/releases ?

If yes, then I am definitely on compiling a list of changes for a given release. In addition to bumping a Kubernetes compatibility version we merge in a lot of changes which are worth explicitly being called out.

That would be awesome!

So do we then require people to add stuff to changelog and how do we make sure that happens?

@s-urbaniak
Copy link
Contributor

@povilasv probably two options:

  1. "light-weight": we compile a list of changes which includes simply all merged PRs and their titles since the last branch cut ourselfes.
  2. "heavy-weight": we ask contributors to explicitly fill out a changelog.

I feel option 1. is good enough for a starter.

@povilasv
Copy link
Contributor

povilasv commented Sep 1, 2020

That would work for one-off, what about new PRs and cherry picks to other branches?

@metalmatze
Copy link
Member

Yes, I'm definitely for the first option. We never really intended for these mixins to have any release really. They are mostly a collection of working rules, alerts, and dashboards. We try to maintain compatibility with the last 3 Kubernetes versions and that's about it.
As we have a few releases already, we should try to have a proper changelog indeed.
These changes were pretty much only made by contributors from Red Hat, as we want to have something to reference for each OpenShift version. Other than that, most people (including the kube-prometheus) project just rolls with master anyway.

@povilasv
Copy link
Contributor

povilasv commented Sep 2, 2020

👍 first option SGTM.

Copy link

github-actions bot commented Nov 3, 2024

This issue has not had any activity in the past 30 days, so the
stale label has been added to it.

  • The stale label will be removed if there is new activity
  • The issue will be closed in 7 days if there is no new activity
  • Add the keepalive label to exempt this issue from the stale check action

Thank you for your contributions!

@github-actions github-actions bot added the stale label Nov 3, 2024
@skl skl added keepalive Use to prevent automatic closing and removed stale labels Nov 4, 2024
@skl skl self-assigned this Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
keepalive Use to prevent automatic closing
Projects
None yet
Development

No branches or pull requests

5 participants