| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 04:38 |
|---|---|---|
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 04:38 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 04:55 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 05:28 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 05:46 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 06:11 |
| opendevreview | Merged openstack/magnum master: devstack: Add function to copy kubeconfig from d-p-c k8s-capi https://review.opendev.org/c/openstack/magnum/+/985519 | 07:36 |
| opendevreview | Michal Nasiadka proposed openstack/magnum stable/2026.1: devstack: Add function to copy kubeconfig from d-p-c k8s-capi https://review.opendev.org/c/openstack/magnum/+/989070 | 07:56 |
| opendevreview | Michal Nasiadka proposed openstack/magnum stable/2026.1: devstack: Add function to copy kubeconfig from d-p-c k8s-capi https://review.opendev.org/c/openstack/magnum/+/989070 | 07:56 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: Add support for OpenStackSDK in backwards compatible way https://review.opendev.org/c/openstack/magnum-capi-helm/+/989071 | 08:04 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: WIP: Migrate out from stack_id https://review.opendev.org/c/openstack/magnum-capi-helm/+/988930 | 08:05 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: Add support for OpenStackSDK in backwards compatible way https://review.opendev.org/c/openstack/magnum-capi-helm/+/989071 | 08:34 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: Add support for OpenStackSDK in backwards compatible way https://review.opendev.org/c/openstack/magnum-capi-helm/+/989071 | 08:34 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: Add support for OpenStackSDK in backwards compatible way https://review.opendev.org/c/openstack/magnum-capi-helm/+/989071 | 08:35 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: Add support for OpenStackSDK in backwards compatible way https://review.opendev.org/c/openstack/magnum-capi-helm/+/989071 | 08:35 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: WIP: Migrate out from stack_id https://review.opendev.org/c/openstack/magnum-capi-helm/+/988930 | 08:35 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: WIP: Migrate out from stack_id https://review.opendev.org/c/openstack/magnum-capi-helm/+/988930 | 08:36 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: WIP: Migrate out from stack_id https://review.opendev.org/c/openstack/magnum-capi-helm/+/988930 | 08:36 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 08:56 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 08:57 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 08:57 |
| opendevreview | Michal Nasiadka proposed openstack/magnum master: CI: Run tempest in mcapi job https://review.opendev.org/c/openstack/magnum/+/989075 | 09:01 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 09:08 |
| opendevreview | Merged openstack/magnum stable/2026.1: devstack: Add function to copy kubeconfig from d-p-c k8s-capi https://review.opendev.org/c/openstack/magnum/+/989070 | 09:15 |
| opendevreview | Rico Lin proposed openstack/magnum master: Omit null Barbican secret expiration https://review.opendev.org/c/openstack/magnum/+/989076 | 09:18 |
| opendevreview | Rico Lin proposed openstack/magnum master: Restore legacy Nova manager accessors https://review.opendev.org/c/openstack/magnum/+/989077 | 09:18 |
| opendevreview | Rico Lin proposed openstack/magnum master: Restore legacy Octavia load balancer methods https://review.opendev.org/c/openstack/magnum/+/989078 | 09:18 |
| opendevreview | Rico Lin proposed openstack/magnum master: Restore legacy Cinder volume type accessors https://review.opendev.org/c/openstack/magnum/+/989079 | 09:18 |
| opendevreview | Rico Lin proposed openstack/magnum master: Read image metadata from SDK properties https://review.opendev.org/c/openstack/magnum/+/989080 | 09:18 |
| opendevreview | Rico Lin proposed openstack/magnum master: Restore legacy Neutron network methods https://review.opendev.org/c/openstack/magnum/+/989081 | 09:18 |
| opendevreview | Rico Lin proposed openstack/magnum master: Preserve custom Glance image properties https://review.opendev.org/c/openstack/magnum/+/989082 | 09:18 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 09:22 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: Add support for OpenStackSDK in backwards compatible way https://review.opendev.org/c/openstack/magnum-capi-helm/+/989071 | 09:38 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 09:45 |
| opendevreview | Michal Nasiadka proposed openstack/magnum master: CI: Run tempest in mcapi job https://review.opendev.org/c/openstack/magnum/+/989075 | 09:48 |
| mnasiadka | ricolin: I’d like to first get 989075 in before your patches, apologies for breaking - but I somehow only enabled devstack without tempest tests in mcapi driver job :) | 10:03 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: WIP: Migrate out from stack_id https://review.opendev.org/c/openstack/magnum-capi-helm/+/988930 | 10:20 |
| opendevreview | Rico Lin proposed openstack/magnum master: Ensure Nova manager accessors https://review.opendev.org/c/openstack/magnum/+/989077 | 10:21 |
| opendevreview | Rico Lin proposed openstack/magnum master: Ensure Octavia load balancer methods https://review.opendev.org/c/openstack/magnum/+/989078 | 10:21 |
| opendevreview | Rico Lin proposed openstack/magnum master: Ensure Cinder volume type accessors https://review.opendev.org/c/openstack/magnum/+/989079 | 10:21 |
| opendevreview | Rico Lin proposed openstack/magnum master: Read image metadata from SDK properties https://review.opendev.org/c/openstack/magnum/+/989080 | 10:21 |
| opendevreview | Rico Lin proposed openstack/magnum master: Ensure Neutron network methods https://review.opendev.org/c/openstack/magnum/+/989081 | 10:21 |
| opendevreview | Rico Lin proposed openstack/magnum master: Preserve custom Glance image properties https://review.opendev.org/c/openstack/magnum/+/989082 | 10:21 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: WIP: Migrate out from stack_id https://review.opendev.org/c/openstack/magnum-capi-helm/+/988930 | 10:23 |
| ricolin | Hey mnasiadka jakeyip after fix the first barbican part, I realize there are actually more issue related to openstacksdk migrate. I finish some end to end test for create/update/delete cluseter and get those fixed, see https://bugs.launchpad.net/magnum/+bug/2153128 . we need to merge https://review.opendev.org/q/topic:%22fix-barbican-openstacksdk-bytes%22 asap | 10:26 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: Adapt to Magnum latest changes https://review.opendev.org/c/openstack/magnum-capi-helm/+/989094 | 10:28 |
| mnasiadka | ricolin: will have a look into that today, but first want to have tempest tests running in the mcapi job - so bear with me :) | 10:28 |
| ricolin | NP | 10:28 |
| opendevreview | Michal Nasiadka proposed openstack/magnum master: CI: Run tempest in mcapi job https://review.opendev.org/c/openstack/magnum/+/989075 | 10:29 |
| mnasiadka | Ok, now the stack is complete - I’ll wait for green jobs on 989075 and will review the rest | 10:30 |
| ricolin | mnaser ^^^ | 10:31 |
| ricolin | fyi | 10:31 |
| mnasiadka | ricolin: do you think we could remove these compatibility layers after 2027.1 and add a deprecation notice or does mcapi expect them to be there forever? ;-) | 10:42 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 10:52 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: Adapt to Magnum latest changes https://review.opendev.org/c/openstack/magnum-capi-helm/+/989094 | 11:09 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 11:12 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: Adapt to Magnum latest changes https://review.opendev.org/c/openstack/magnum-capi-helm/+/989094 | 11:24 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 11:25 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 11:25 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: Adapt to Magnum latest changes https://review.opendev.org/c/openstack/magnum-capi-helm/+/989094 | 11:27 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 11:27 |
| mnasiadka | ricolin: your stack is failing tempest test - can you check? | 11:27 |
| opendevreview | Merged openstack/magnum master: Omit null Barbican secret expiration https://review.opendev.org/c/openstack/magnum/+/989076 | 11:28 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 11:29 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add --diff to black https://review.opendev.org/c/openstack/magnum-capi-helm/+/989103 | 11:43 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add --diff to black https://review.opendev.org/c/openstack/magnum-capi-helm/+/989103 | 11:43 |
| jakeyip | mnasiadka: you around for meeting? | 13:01 |
| mnasiadka | Anybody else around? | 13:05 |
| mnasiadka | I’m a bit firefighting… | 13:05 |
| jakeyip | we can catch up at a separate time tomorrow or something, but prob good to book in a time | 13:09 |
| jakeyip | ricolin: does the stack_id change affect mcapi? how do you get capi cluster id otherwise? | 13:10 |
| mnasiadka | jakeyip: I have a patch to rework capi helm | 13:24 |
| mnasiadka | Although I also reworked openstacksdk calls in capi helm instead of doing what ricolin proposed | 13:25 |
| jakeyip | does the patch allow capi helm driver users to get the unique cluster name and how? | 13:26 |
| jakeyip | nvm I'll wait for your patch | 13:47 |
| ricolin | jakeyip: yes, mcapi still using stack_id to store k8s cluster id for magnum to have better info with k8s | 13:51 |
| opendevreview | Rico Lin proposed openstack/magnum master: Preserve custom Glance image properties https://review.opendev.org/c/openstack/magnum/+/989082 | 14:11 |
| andrewbogott__ | could someone take a glance at my issue here? https://github.com/vexxhost/magnum-cluster-api/issues/1028 -- the short version is that when I set docker_volume_size everything breaks. I could easily believe this is something missing from my capi worker and unrelated to the magnum driver but it's not clear. | 14:23 |
| mnasiadka | jakeyip: once I get some window in around 30 minutes I can share | 14:24 |
| mnasiadka | ricolin: well, heat driver has been deprecated long time ago, we can revert, but I expect to be able to merge it again this cycle | 14:24 |
| mnasiadka | jakeyip: capi helm was setting stack_id with a generated random bit - now it’s putting it in a label (in https://review.opendev.org/c/openstack/magnum-capi-helm/+/989094/4/magnum_capi_helm/driver.py) | 14:31 |
| ricolin | mnasiadka: agree to wipe out the heat driver, but regarding stack_id, maybe we can migrate the stack_id to something like "cluster_id" or something we can track resource from k8s? which is pretty useful when you have a k8s cluster behind magnum cluster, we can keep the info easier | 14:44 |
| mnasiadka | cluster_id makes sense, can write a patch to introduce that and switch to that in capi helm instead | 14:46 |
| mnasiadka | Agree that maybe using labels is something we should stop doing | 14:46 |
| ricolin | also for the legacy client method support, I think after 2027.1 is definitely okay | 14:51 |
| ricolin | for db migration path, ideally we can drop heat driver now but keep stack_id in DB, create something like cluster_id, and mark stack_id as deprecated. and finally drop stack_id once for all, so no one get supprised | 14:54 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 15:08 |
| opendevreview | Michal Nasiadka proposed openstack/magnum master: CI: Run tempest in mcapi job https://review.opendev.org/c/openstack/magnum/+/989075 | 15:18 |
| mnasiadka | let me merge the revert and follow up with cluster_id + deprecation | 15:18 |
| opendevreview | Matt Crees proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 15:26 |
| ricolin | mnasiadka: IIRC, like for nova, the db stays the same, but the api above expose it with new names too. so db is "stack_id" but the column shows up becomes cluster_id, but api layer still expose both. Or like what we did for "tenant" and "project" in keystone :) | 15:31 |
| mnasiadka | That’s an option as well, but I guess we should rename it in the DB at some point | 15:32 |
| opendevreview | Merged openstack/magnum master: Revert "Remove stack_id after removing Heat driver" https://review.opendev.org/c/openstack/magnum/+/988904 | 15:49 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Drop copy logs and heat reliance https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/989177 | 16:27 |
| andrewbogott__ | ricolin: on issue you ask how I create the cluster; can you tell me what you mean by 'how'? Do you mean, like, the openstack commandline? Or are you asking about my capi worker cluster? | 16:43 |
| andrewbogott__ | oh nm, looks like the issue is already diagnosed. Speedy! | 16:47 |
| mnasiadka | ricolin: any ideas why tempest is failing here? https://review.opendev.org/c/openstack/magnum/+/989075 | 17:38 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add magnum-capi initial CI job https://review.opendev.org/c/openstack/magnum-capi-helm/+/957702 | 17:41 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Bump cluster delete timeout to 60m and make it configurable https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/986495 | 17:41 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-capi-helm master: CI: Add --diff to black https://review.opendev.org/c/openstack/magnum-capi-helm/+/989103 | 18:05 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Bump cluster delete timeout to 60m and make it configurable https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/986495 | 19:39 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!