Giter Club home page Giter Club logo

Comments (20)

IdanAdar avatar IdanAdar commented on July 17, 2024 8

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...

from reloader.

onedr0p avatar onedr0p commented on July 17, 2024 8

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.

from reloader.

MuneebAijaz avatar MuneebAijaz commented on July 17, 2024 7

Hi (wow this is a long thread), i personally agree with the change, it doesn't make sense to me why a small action upgrade or a small dependency upgrade would release versions so frequently, when it's not really of value to the end user.
But sadly, this is how the projects were maintained (from I dont know when). And it became quite noticeable now with renovate configured on the repo and actively merging multiple PRs every week

I apologize for the recent issues the community has faced due to broken Reloader releases.

I have strongly suggested the change internally, and once we have decided on how we want to proceed, we can take action on this.

I have suggested fortnightly manual releases, which would be done in one of the opensource calls which happen every week on Wednesday.

from reloader.

IdanAdar avatar IdanAdar commented on July 17, 2024 6

"and it just broke a few of my production work clusters."

I'm not saying Reloader's release process shouldn't change, just saying yours could too. You don't have to be sensitive about it. 🤷‍♂️

P.s., I'm not a developer of Reloader. Just a user of.

from reloader.

IdanAdar avatar IdanAdar commented on July 17, 2024 5

Puts in question your production environment IMO...

from reloader.

IdanAdar avatar IdanAdar commented on July 17, 2024 5

There's some confusion here -- I'm a user of Reloader, not a maintainer of, and only voiced an opinion. You don't have to agree or anything.

from reloader.

IdanAdar avatar IdanAdar commented on July 17, 2024 4

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

from reloader.

IdanAdar avatar IdanAdar commented on July 17, 2024 4

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...

from reloader.

buroa avatar buroa commented on July 17, 2024 4

There's some confusion here -- I'm a user of Reloader, not a maintainer of, and only voiced an opinion. You don't have to agree or anything.

Learn to refrain from commenting on things you truly don't understand.

from reloader.

smuda avatar smuda commented on July 17, 2024 4

Since this is a free and open source project, I think we as a community have a duty to fix problems we see, especially when we are affected ourselves 😀. Therefore, if the releases have been broken twice in the last couple of months the same way, please create a PR that checks against the problem in the release process. We will all benefit.

With that said, I'd appreciate to not have multiple releases per day since our manual process can't handle that anyway and it creates work to find all the changes.

from reloader.

joryirving avatar joryirving commented on July 17, 2024 3

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.

from reloader.

buroa avatar buroa commented on July 17, 2024 2

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.

from reloader.

Ornias1993 avatar Ornias1993 commented on July 17, 2024 1

Puts in question your production environment IMO...

Wait a second, you start snarking-off someones production environment...
While someone politely points out your release procedure is the equivalent of hot-garbage.

I think the last person that should be snarking about someones CD pipeline, is the one that has shown to be unable to provide a decent quality CI and release pipeline, isn't it?

from reloader.

IdanAdar avatar IdanAdar commented on July 17, 2024 1

You mean on things you could have explained better from the get-go instead of being exhaustive ("broke my work clusters" vs "broke my release process"? big difference between the two statements, and likely would've taken the conversation completely differently)? Anyway, to your own hilariously condescending comment, the answer is... No. 😄

Anyway, I'm still in favor. 🙄

from reloader.

MuneebAijaz avatar MuneebAijaz commented on July 17, 2024 1

Hi all, we have discussed this internally, and we welcome the change. We would, however, require help from the community to switch to manual releases. If someone is willing to do it, changes can be discussed in the same thread.
Changes by the members might be much slower in comparison to the community.

from reloader.

onedr0p avatar onedr0p commented on July 17, 2024

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

from reloader.

buroa avatar buroa commented on July 17, 2024

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.

from reloader.

buroa avatar buroa commented on July 17, 2024

Puts in question your production environment IMO...

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

from reloader.

onedr0p avatar onedr0p commented on July 17, 2024

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.

from reloader.

buroa avatar buroa commented on July 17, 2024

"and it just broke a few of my production work clusters."

I'm not saying Reloader's release process shouldn't change, just saying yours could too. You don't have to be sensitive about it. 🤷‍♂️

buroa/k8s-gitops#2489

Some checks were not successful

/end

from reloader.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.