search

StackStorm / st2-packages

StackStorm deb/rpm packages (automated docker build pipeline)
https://stackstorm.com/
27 stars 59 forks source link

Remove st2 resultstracker #683

Closed winem closed 3 years ago

winem commented 3 years ago

This PR removes all references to st2resultstracker in the package specs and service definitions as well as the corresponding service file.

closes #668

winem commented 3 years ago

The workflow fails because st2ctl still checks the state of the st2resultstracker and can't find it any more.

See https://github.com/StackStorm/st2/blob/746ae505e77126af6b7473a91eb1e98614a67568/st2common/bin/st2ctl#L17

The corresponding issue to be solved first: https://github.com/StackStorm/st2/issues/5070

winem commented 3 years ago

Now there is a circular dependency between this PR and https://github.com/StackStorm/st2/pull/5108

Agreed with @blag to keep this PR as it is and merge both PRs on monday. The Exchange pack CI builds do have a higher priority and we want them to succeed over the weekend.

See https://stackstorm-community.slack.com/archives/CSBELJ78A/p1608330230290400

arm4b commented 3 years ago

@winem Following this PR change, we'll need a fix in Ansible-st2 repository (https://github.com/stackstorm/ansible-st2) which started to fail because it included the st2resultstracker in the list of services. Could you please submit a small PR for that?