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

The st2ctl reload command is missing a purge option #3313

Open
pietervogelaar opened this issue Mar 28, 2017 · 10 comments
Open

The st2ctl reload command is missing a purge option #3313

pietervogelaar opened this issue Mar 28, 2017 · 10 comments

Comments

@pietervogelaar
Copy link

I discovered that when a rule, trigger, action or workflow is removed from a git repository and is no longer on the disk of a stackstorm server, the command sudo st2ctl reload --register-setup-virtualenvs --register-all does not purge these items from StackStorm. I have to manually delete them with st2 commands.

It would be very nice if there was a purge flag or something.

@arm4b
Copy link
Member

arm4b commented Mar 28, 2017

Yeah, sounds like a good feature request.
I remember we got several reports like that before and I believe we have to put everything together and reconsider, keeping in mind infrastructure as a code principles.

Similar Issues:

There are probably other related issues in search.

@hagay3
Copy link

hagay3 commented Mar 30, 2017

+1

@leevs
Copy link

leevs commented May 30, 2017

+1
I didn't get it why my aliases would still be present when we reload it.

@mbrannigan
Copy link

+1

@nmaludy
Copy link
Member

nmaludy commented Dec 8, 2017

+1

1 similar comment
@ankittyagi20
Copy link

+1

@matrulda
Copy link

+1

@punkrokk
Copy link
Member

Maybe someone who knows the st2ctl and pack install could post a list of what would need to happen to solve this ticket? @armab @nmaludy ?

@arm4b
Copy link
Member

arm4b commented Jun 12, 2020

There is some explanation in: #3143 (comment). I think that statement was made from the point that possible pack content could be added via APIs, that's not present on the file system.

I think the end goal to close a bunch of issues: make sure the registered content in DB is in sync with the file system state. Unload everything else. Otherwise it looks counter-intuitive.

@punkrokk
Copy link
Member

@armab I kind of agree. What if there was a --purge option that ran st2 pack remove? Would that solve for this without a ton of new logic?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

9 participants