| tonyb | karolinku[m], jcapitao[m]: Okay at this point you have a basically functional build. The problem now is that glean doesn't work with the NetworkManager only style networking in CS-10: https://paste.opendev.org/show/bjlMgJrC6Jru7uNkmane/ | 01:38 |
|---|---|---|
| tonyb | I don't know enough about it to do the right thing here. I also don't know if there is a short-term crutch you could employ to "network-scripts" back | 01:41 |
| tonyb | I don't really recommend that even if it exists | 01:41 |
| tonyb | I'll clean up my nodepool change I think the rest is up to you for now. | 01:48 |
| tonyb | clarkb: it was deprecated in 9 and compat via the plugin needed to be explicitly enabled, removed in 10. I feel like that's reasonable | 02:04 |
| tonyb | that file you pointed to is part of the centos-9 package not not centos-10 | 02:05 |
| tonyb | https://gitlab.com/redhat/centos-stream/rpms/NetworkManager/-/blob/c10s/NetworkManager.spec?ref_type=heads#L168 | 02:05 |
| tonyb | https://gitlab.com/redhat/centos-stream/rpms/NetworkManager/-/blob/c10s/NetworkManager.spec?ref_type=heads#L794 | 02:06 |
| tonyb | https://gitlab.com/redhat/centos-stream/rpms/NetworkManager/-/blob/c10s/NetworkManager.spec?ref_type=heads#L214 | 02:06 |
| jcapitao[m] | tonyb: we've already hit that issue, lemme take a look | 08:20 |
| tonyb | jcapitao[m]: okay cool | 08:25 |
| opendevreview | Rafal Lewandowski proposed openstack/diskimage-builder master: Prevent from overwriting grub defaults if no variables are set https://review.opendev.org/c/openstack/diskimage-builder/+/937684 | 08:42 |
| jcapitao[m] | tonyb: so it's another issue this time, with same error message | 08:43 |
| tonyb | jcapitao[m]: ah okay | 08:44 |
| jcapitao[m] | in the older patches we were passing that step (build was ok) | 08:44 |
| jcapitao[m] | but functional tests were failing | 08:44 |
| tonyb | jcapitao[m]: actually the build is still fine. this is the last check .... we've never been here before | 08:45 |
| tonyb | the image builds and boots and is available via ssh | 08:45 |
| tonyb | but glean failed to configure networking | 08:45 |
| tonyb | this is progress not a regression | 08:45 |
| jcapitao[m] | oh right | 08:46 |
| jcapitao[m] | it's in unlocked state | 08:47 |
| jcapitao[m] | that's nice :) | 08:47 |
| tonyb | yeah. #progress | 08:47 |
| opendevreview | Rafal Lewandowski proposed openstack/diskimage-builder master: Set grub default values according to its project defaults https://review.opendev.org/c/openstack/diskimage-builder/+/937684 | 08:47 |
| tonyb | it should be easy enough to iterate on glean and get it working | 08:48 |
| jcapitao[m] | ok I caught up all the thread | 08:51 |
| tonyb | I'll clean up the nodepool change and the cpu-level one too. | 08:51 |
| jcapitao[m] | so I think we have to do the NetworkManager migration | 08:51 |
| jcapitao[m] | the network profiles I meant from ifcfg to keyfile format | 08:52 |
| jcapitao[m] | tonyb: ack | 08:52 |
| tonyb | jcapitao[m]: that sounds correct to me | 08:53 |
| jcapitao[m] | but looks like glean project is not very active | 08:54 |
| tonyb | it's as active as it "needs" to be | 08:55 |
| opendevreview | Rafal Lewandowski proposed openstack/diskimage-builder master: Change grub variables for style and timeout https://review.opendev.org/c/openstack/diskimage-builder/+/937684 | 08:55 |
| jcapitao[m] | yeah right, as it's pretty stable and haven't need new features to be added until now | 08:56 |
| tonyb | jcapitao[m]: yup | 08:56 |
| jcapitao[m] | fair enough, thank you tonyb for the heads up on that. We'll continue the work | 08:57 |
| tonyb | jcapitao[m]: cool. ping me if you need anything | 08:57 |
| clarkb | tonyb: the file I linked to is from the centos 10 stream branch of the package aiui. The content of that file also differs compared to c9s. My point was more that the way it is written it is ambiguous as to whether ir is merely deprecated or if it is actually removed | 15:27 |
| clarkb | I think it is reasonable to deprecate and rmove things. I just wish it was more clearly documented | 15:28 |
| clarkb | There is probably a different changelog file somewhere that does make it explicit | 15:32 |
| clarkb | tonyb: jcapitao[m]: the default behavior of network manager is to dhcp the interfaces which is why we were still able to ssh in and check things. The reason for the explicit check against glean reporting success vs failure is that not all clouds dhcp for network setup | 15:33 |
| clarkb | for example rackspace uses a static configuration and for those clouds you need glean to properly configure the network decides. We' retrying to catch these problems as ealry as possible in dib and glean testing so that we're not scratching our heads when half the clouds "work" and the other half do not | 15:34 |
| clarkb | the good news is we have succeeded at catchign the problem early | 15:34 |
| tonyb | clarkb: that's more or less what I thought. So really what we need to do is help glean understand than in the scenario it's been told to use networkmanager and the OS doesn't support the older style ifcfg network configuration instead write an appropriate nmconnection file in /etc/NetworkManager/system-connections | 20:38 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!