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

fix the moved explanation url #519

Merged
merged 5 commits into from Oct 5, 2022

Conversation

erolkskn
Copy link
Contributor

It's a simple one line change to config. Old explanation url moved to new one.
See: https://fedoraproject.org/wiki/Upstream_release_monitoring

Signed-off-by: Erol Keskin <erolkeskin.dev@gmail.com>
@softwarefactory-project-zuul
Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci

✔️ fi-tox-mypy SUCCESS in 5m 55s
fi-tox-lint FAILURE in 5m 07s
✔️ fi-tox-format SUCCESS in 5m 13s
fi-tox-python38 FAILURE in 5m 38s
fi-tox-python39 FAILURE in 5m 42s
fi-tox-python310 FAILURE in 5m 14s
✔️ fi-tox-docs SUCCESS in 5m 58s
✔️ fi-tox-bandit SUCCESS in 5m 49s
fi-tox-diff-cover FAILURE in 6m 24s

Signed-off-by: Erol Keskin <erolkeskin.dev@gmail.com>
@softwarefactory-project-zuul
Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci

✔️ fi-tox-mypy SUCCESS in 4m 51s
fi-tox-lint FAILURE in 5m 33s
✔️ fi-tox-format SUCCESS in 5m 33s
✔️ fi-tox-python38 SUCCESS in 5m 21s
✔️ fi-tox-python39 SUCCESS in 4m 56s
✔️ fi-tox-python310 SUCCESS in 4m 48s
✔️ fi-tox-docs SUCCESS in 4m 48s
✔️ fi-tox-bandit SUCCESS in 4m 45s
✔️ fi-tox-diff-cover SUCCESS in 6m 05s

@softwarefactory-project-zuul
Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci

✔️ fi-tox-mypy SUCCESS in 4m 51s
fi-tox-lint FAILURE in 4m 37s
fi-tox-format FAILURE in 5m 34s
✔️ fi-tox-python38 SUCCESS in 5m 21s
✔️ fi-tox-python39 SUCCESS in 4m 56s
✔️ fi-tox-python310 SUCCESS in 4m 44s
✔️ fi-tox-docs SUCCESS in 4m 40s
✔️ fi-tox-bandit SUCCESS in 5m 11s
✔️ fi-tox-diff-cover SUCCESS in 5m 54s

@softwarefactory-project-zuul
Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci

✔️ fi-tox-mypy SUCCESS in 4m 49s
✔️ fi-tox-lint SUCCESS in 4m 36s
fi-tox-format FAILURE in 4m 37s
✔️ fi-tox-python38 SUCCESS in 5m 03s
✔️ fi-tox-python39 SUCCESS in 4m 56s
✔️ fi-tox-python310 SUCCESS in 4m 47s
✔️ fi-tox-docs SUCCESS in 4m 43s
✔️ fi-tox-bandit SUCCESS in 5m 21s
✔️ fi-tox-diff-cover SUCCESS in 6m 21s

@softwarefactory-project-zuul
Copy link

Build succeeded.

✔️ fi-tox-mypy SUCCESS in 4m 54s
✔️ fi-tox-lint SUCCESS in 4m 32s
✔️ fi-tox-format SUCCESS in 5m 22s
✔️ fi-tox-python38 SUCCESS in 5m 01s
✔️ fi-tox-python39 SUCCESS in 4m 46s
✔️ fi-tox-python310 SUCCESS in 4m 42s
✔️ fi-tox-docs SUCCESS in 4m 43s
✔️ fi-tox-bandit SUCCESS in 5m 22s
✔️ fi-tox-diff-cover SUCCESS in 6m 59s

@Zlopez
Copy link
Contributor

Zlopez commented Sep 26, 2022

Could you add news file as well?

@Zlopez
Copy link
Contributor

Zlopez commented Sep 26, 2022

And it would be nice to change it here as well. This is the configuration file that is used for deployment in both https://stg.release-monitoring.org and https://release-monitoring.org

@erolkskn
Copy link
Contributor Author

And it would be nice to change it here as well. This is the configuration file that is used for deployment in both https://stg.release-monitoring.org and https://release-monitoring.org

Thanks for pointing it out Michal :-) I'm going to take a look at it now

Signed-off-by: Erol Keskin <erolkeskin.dev@gmail.com>
@softwarefactory-project-zuul
Copy link

Build succeeded.

✔️ fi-tox-mypy SUCCESS in 7m 46s
✔️ fi-tox-lint SUCCESS in 6m 07s
✔️ fi-tox-format SUCCESS in 6m 13s
✔️ fi-tox-python38 SUCCESS in 6m 56s
✔️ fi-tox-python39 SUCCESS in 7m 29s
✔️ fi-tox-python310 SUCCESS in 7m 10s
✔️ fi-tox-docs SUCCESS in 6m 08s
✔️ fi-tox-bandit SUCCESS in 5m 54s
✔️ fi-tox-diff-cover SUCCESS in 7m 39s

@erolkskn
Copy link
Contributor Author

erolkskn commented Sep 26, 2022

And it would be nice to change it here as well. This is the configuration file that is used for deployment in both https://stg.release-monitoring.org and https://release-monitoring.org

Created news entry for this in the latest commit.
Created a PR regarding the configuration change: https://pagure.io/fedora-infra/ansible/pull-request/1197

@softwarefactory-project-zuul
Copy link

Build succeeded.

✔️ fi-tox-mypy SUCCESS in 5m 12s
✔️ fi-tox-lint SUCCESS in 5m 33s
✔️ fi-tox-format SUCCESS in 4m 55s
✔️ fi-tox-python38 SUCCESS in 5m 16s
✔️ fi-tox-python39 SUCCESS in 5m 13s
✔️ fi-tox-python310 SUCCESS in 5m 04s
✔️ fi-tox-docs SUCCESS in 4m 58s
✔️ fi-tox-bandit SUCCESS in 5m 15s
✔️ fi-tox-diff-cover SUCCESS in 6m 10s

@mergify mergify bot merged commit 769e226 into fedora-infra:master Oct 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants