Open bmildren opened 4 years ago
Any ETA on that? It would be really useful.
Hi. I've made such a module (totally inspired from proxysql.proxysql_replication_hostgroups module). Even if I'm not a maintainer, am I able to propose a PR ?
Hi. I've made such a module (totally inspired from proxysql.proxysql_replication_hostgroups module). Even if I'm not a maintainer, am I able to propose a PR ?
Hi @TheOignon, sure, everyone can and should! Please do it:)
You can also be interested in some of our contributor guidelines listed in https://github.com/ansible-collections/community.proxysql/blob/main/README.md. You can find info on how to run sanity test locally / add integration tests for your module and run them locally before pushing. Though you can push it as it is now and fix everything later.
Thanks for the quick answer !
Though you can push it as it is now and fix everything later.
Even though I didn't added sanity nor integration tests, should I create a PR right now ? It's pushed on a fork I made following the guidelines you linked.
Thanks for the quick answer !
Though you can push it as it is now and fix everything later.
Even though I didn't added sanity nor integration tests, should I create a PR right now ? It's pushed on a fork I made following the guidelines you linked.
yep, sounds good, though we need to have integration tests eventually in the PR (no need to do anything for sanity tests, they are just checks for formatting compliance, standards, etc.)
we need to have integration tests eventually in the PR
Those tests will be added, it's been running on a role I developped using this plugin, just need to rewrite correctly everything. I also need to add some documentations. I will create the PR soon, thanks for your answers !
@TheOignon sounds great, thank you!
@TheOignon here are a couple of docs you may be interested in:
Hi all, I have created a PR for this issue, so what's next ? I see that workflows of this repo have been disabled with message:
This scheduled workflow is disabled because there hasn't been activity in this repository for at least 60 days. Enable this workflow to resume scheduled runs.
How can it be enabled, and my PR get reviewed and merged ?
Thanks for any help.
I've enabled the workflow again, but it looks like that it needs a push on your PR to get it run again @tompal3
Thanks @markuman I did that, now need to wait for someone's approval for workflow to run I guess: https://github.com/ansible-collections/community.proxysql/actions/runs/6902866352 ?
Hi all, any news on this? It would be very useful.
@dpavlos my PR was approved https://github.com/ansible-collections/community.proxysql/pull/147 so its only need to be merged and new version should be released. @markuman any estimate on that?
We had some trouble with the backport branch.
I'll be away some days and try do it next week.
SUMMARY
Need to add a module to support changes to the proxysql_galera_hostgroups config, along with the associated integration tests.
ISSUE TYPE
COMPONENT NAME
ADDITIONAL INFORMATION