| opendevreview | Dale Smith proposed openstack/magnum-specs master: Propose control plane resize feature. https://review.opendev.org/c/openstack/magnum-specs/+/905281 | 02:01 |
|---|---|---|
| opendevreview | Dale Smith proposed openstack/magnum-specs master: Propose control plane resize feature. https://review.opendev.org/c/openstack/magnum-specs/+/905281 | 04:00 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 08:25 |
| opendevreview | Merged openstack/magnum master: Migrate to importlib.metadata https://review.opendev.org/c/openstack/magnum/+/901901 | 09:09 |
| mnasiadka | elodilles: if we don't want to include https://opendev.org/openstack/magnum-capi-helm-charts in 2024.1 release - do we need to do anything? (that repo is needed for a new driver, that we're not ready to merge in C, but plan for getting it in D) | 09:21 |
| elodilles | mnasiadka: no, nothing to do in that case, thanks for the info :) release management team will ping you around same time (Milestone-2) next cycle :) so good to prepare for that ;) | 09:43 |
| mnasiadka | elodilles: thanks :) | 09:43 |
| elodilles | np :) | 09:44 |
| jakeyip | does anyone else have the issue of magnum occm breaking when the user who spins a up a cluster leaves the project? how are you handling this case? | 10:00 |
| mnasiadka | jakeyip: I think everybody has been there, https://review.opendev.org/c/openstack/magnum/+/806441 this is a bit related - but I guess does not help | 12:18 |
| opendevreview | Michal Nasiadka proposed openstack/magnum master: Regenerate trust when operate cluster https://review.opendev.org/c/openstack/magnum/+/806441 | 12:19 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 16:22 |
| *** dalees0 is now known as dalees | 16:59 | |
| *** travisholton9 is now known as travisholton | 16:59 | |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 17:46 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 20:47 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 20:48 |
| dalees | jakeyip: yes, we have this issue - it will be present in both trusts and application credentials and also occurs if any **roles** used in the trust are later revoked. we: 1) recommend customers create with a service account. 2) have a db-is-my-api method to update trusts to refer to new trustors, or remove keystone roles. | 20:50 |
| dalees | one problem with regenerating the trust or application credentials is some k8s resources are one-shot applied in Magnum Heat. So there is no current method to update them. | 20:51 |
| dalees | In CAPI Helm I am ensuring the application credentials are created with only a subset of user roles (`_member_`), and the Helm charts should mean rolling an application credential does the right things. That will still need some attention to ensure it works, and some way to trigger the update. | 20:53 |
| mnasiadka | ok, I found out why I can't get node_addresses and master_addresses from a failed cluster in the CI job - we just populate it after the creation is success | 20:58 |
| mnasiadka | so I guess the only sensible option to get it, is to define a HeatClient class and get that data directly from Heat, ugh | 20:59 |
| mnasiadka | dalees, jakeyip: If you have better ideas - I'm open to proposals - but I'll probably work on this in Feb - for now we have a job that passes - but it can't be used for 1.27 or 1.28, because for some reason that does not pass through CI yet | 21:00 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 21:03 |
| dalees | mnasiadka: do you know where that sync happens? without looking at the code I wouldn't be opposed to having it sync those from the Heat stack earlier. | 21:03 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 21:03 |
| mnasiadka | dalees: looks like somewhere here: https://github.com/openstack/magnum/blob/master/magnum/conductor/handlers/cluster_conductor.py | 21:06 |
| dalees | mnasiadka: also, hope you have a good break! see you when you return | 21:06 |
| mnasiadka | dalees: thanks :) | 21:08 |
| opendevreview | Michal Nasiadka proposed openstack/magnum master: WIP: conductor: save nodegroup info even on failed clusters https://review.opendev.org/c/openstack/magnum/+/905357 | 21:10 |
| opendevreview | Michal Nasiadka proposed openstack/magnum master: Add kubernetes fedora coreos v1 jobs https://review.opendev.org/c/openstack/magnum/+/896244 | 21:10 |
| mnasiadka | I don't believe it's as simple as this, but will check tomorrow in the morning | 21:10 |
| dalees | Nah, I expect that is too early. The conductor won't be blocking waiting for the heat stack to fail. The `allow_update_status` is interesting, but still have to check it's polling that method during creation - and even then there is complex logic in there that might prevent what we want. | 21:14 |
| opendevreview | Dale Smith proposed openstack/magnum-specs master: Propose control plane resize feature. https://review.opendev.org/c/openstack/magnum-specs/+/905281 | 21:25 |
| opendevreview | Michal Nasiadka proposed openstack/magnum master: WIP: conductor: save nodegroup info even on failed clusters https://review.opendev.org/c/openstack/magnum/+/905357 | 21:56 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 22:01 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 22:02 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 22:13 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!