Consider changing release strategy

[onedr0p]

Team, I love using reloader but since I am using renovate to update it in my clusters the churn of releases are getting overwhelming. Any chance this can be addressed to not release on every merged PR to the main branch? Does every merged PR warrant a new release? Thanks.

EDIT: I understand the current way lessens the burden of having to manually craft new releases and makes it so this project is pretty much auto-piloted however not every merged PR justifies a release IMO.

[IdanAdar]

Would like to highlight this part from your comment:

since I am using renovate to update it in my clusters

I could understand maybe doing this in development environment, but do you update your production clusters the same way, too? This seems very specific to your use case...

[onedr0p]

It doesn't matter, whether dev, stage or production or whatever. Changing the release strategy to being manually curated would stop things like this from being released.

b73b04d

This is the second time the release has been broken in the past couple months

[buroa]

Would like to highlight this part from your comment:

since I am using renovate to update it in my clusters

I could understand maybe doing this in development environment, but do you update your production clusters the same way, too? This seems very specific to your use case...

@IdanAdar This is how many organizations update, and it just broke a few of my production work clusters.

[IdanAdar]

Puts in question your production environment IMO...

[buroa]

Puts in question your production environment IMO...

Cause I use renovate? How are you an SRE Team Lead... olol

[IdanAdar]

Because you update directly to production lolol. 🤷‍♂️

[onedr0p]

Because you update directly to production lolol. 🤷‍♂️

I never updated dev, stage or production, the PR comes from renovate for my GitOps repo, runs tests, and I can see that it fails so I don't merge it.

[IdanAdar]

Your production broke because you pushed directly to it before testing it and you're complaining on someone else's release process instead of your own release process? Please...

[joryirving]

We also get notifications in slack at work when reloader is updated. Because it's a manual process for us to update charts (because production), it becomes a chore to manually comb through a bunch of release notes to only find out the README was updated, and functionally nothing has changed.

It would be nice to only have releases when the code is changed.

[buroa]

Because you update directly to production lolol. 🤷‍♂️

You really don't understand my environment, but tried to directly criticize how I update. What's a fact here, is this merging pipeline needs to change.

Your production broke because you pushed directly to it before testing it and you're complaining on someone else's release process instead of your own release process? Please...

I have CI tests that check if image is even present, so it fails to merge. Fix this garbage release on merge pipeline.

[onedr0p]

Your production broke because you pushed directly to it before testing it and you're complaining on someone else's release process instead of your own release process? Please...

The fact of the matter this project should not be releasing builds like this in an automated fashion because there obviously isn't enough testing going on to prevent broken builds from being released, they should be manually curated like most of the other software on the internet.