[Waiting for fedmsg EOL] Sunset github2fedmsg and fedmsg #2454
No reviewers
Labels
No labels
freeze-break-request
post-freeze
No milestone
No project
No assignees
4 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: Infrastructure/ansible#2454
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "fedmsg_sunset"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
This commit is removing all the fedmsg related stuff from ansible
repository.
Signed-off-by: Michal Konecny mkonecny@redhat.com
Merge Failed.
This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging pagure.io/fedora-infra/ansible for 2454,8294feeac7bf5691d5a41b6c6f682f3c9c54890d
rebased onto
04bc8ed4d2
Merge Failed.
This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging pagure.io/fedora-infra/ansible for 2454,62d294d4d6fbe1aaf707593e88bd86c5654b6e47
Strange, there is no conflict anymore, but zuul is still unable to merge it for testing :/
@kevin @abompard Could you check this PR? Hopefully I wasn't too overzealous when removing the fedmsg from ansible.
A pretty massive pr. ;)
I skimmed it and it all seemed pretty reasonable.
I think we can delete playbooks/manual/upgrade/packages.yml entirely. also roles/packages3/ I think.
We should confirm that pagure has no need for gitolite3 anymore. I don't think it does, but we should make sure.
@kevin I will update the PR to remove playbooks/manual/upgrade/packages.yml as well. Only place I needed to remove gitolite role from was playbooks/manual/upgrade/pkgs.yml and that only used the
gitolite/check_fedmsg_hooks
. It seems that pagure doesn't need it for anything.rebased onto
7880b13991
Merge Failed.
This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging pagure.io/fedora-infra/ansible for 2454,2e3f9bb2a042c556edf98bcf1ae6421a0c77d115
rebased onto
7880b13991
Merge Failed.
This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging pagure.io/fedora-infra/ansible for 2454,2d8fd26ab3b75a18270b304cc9541e831e5412ea
rebased onto
7880b13991
Removed also monitori-dashboard as part of this PR.
Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci
https://fedora.softwarefactory-project.io/zuul/buildset/a1e7694b7c684733a1f6f28a5e05ce95
This removes the dead
csi_*
vars, too (e.g.inventory/host_vars/bodhi-backend01.iad2.fedoraproject.org
). Do you want to keep this together or split into separate commits?I see some stuff being removed which I don’t understand:
bugz.fp.o
fromplaybooks/include/proxies-redirects.yml
. Intentional?playbooks/manual/upgrade/packages.yml
playbooks/openshift-apps/monitor_dashboard.yml
roles/gitolite/base/tasks/main.yml
roles/openshift-apps/monitor-dashboard/*
roles/packages3/bugz.fp.o/*
(see above)roles/packages3/web/*
I can remove them from this commit, it was just too close to fedmsg that I just removed them as well as CSI is no longer a thing.
That is because of the removal of packages3 role.
That was mentioned by @kevin to remove as it's no longer needed and it was related to packages3 role.
That one was too tied to fedmsg and I got confirmation that it should be removed anyway.
Another one that had plenty of fedmsg references and got confirmed that it should be removed.
Just the role for monitor-dashboard playbook.
Too tied to fedmsg and got confirmation that it could be deleted completelly.
Same as above.
Oh, my mistake. We do want to keep roles/packages3/bugz.fp.o/ just drop roles/packages3/web
rebased onto
0a0713ecb4
Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci
https://fedora.softwarefactory-project.io/zuul/buildset/0842c02a04304158b43ecbeed9987048
Removed
roles/packages3/bugz.fp.o
and the change inplaybooks/include/proxies-redirects.yaml
from the PR.Zuul is working again :-)
rebased onto
cdb7471dfe
Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci
https://fedora.softwarefactory-project.io/zuul/buildset/b54bdd461c304d10bcb65822370a88d0
rebased onto
2f5ee2da64
rebased onto
2f5ee2da64
Pull-Request has been merged by zlopez
Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci
https://fedora.softwarefactory-project.io/zuul/buildset/24a85c06c3ad40f7831c6276d99aed72