| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-05-06 | |||
| 08:53:57 | gibi | but the grammar is really different | |
| 08:54:05 | Uggla | gibi, ok. | |
| 08:54:39 | gibi | like slag = Schlauch = hose | |
| 08:56:07 | gibi | of course we have our own word for Schlauch that is not related: tömlő ;) | |
| 08:59:08 | gibi | as far as I see tömlő has Uralic roots so that one is an original :D | |
| 09:01:58 | opendevreview | Balazs Gibizer proposed openstack/nova stable/wallaby: Isolate PCI tracker unit tests https://review.opendev.org/c/openstack/nova/+/840841 | |
| 09:01:59 | opendevreview | Balazs Gibizer proposed openstack/nova stable/wallaby: Remove unavailable but not reported PCI devices at startup https://review.opendev.org/c/openstack/nova/+/840842 | |
| 09:02:00 | opendevreview | Balazs Gibizer proposed openstack/nova stable/wallaby: Simulate bug 1969496 https://review.opendev.org/c/openstack/nova/+/840843 | |
| 09:02:01 | opendevreview | Balazs Gibizer proposed openstack/nova stable/wallaby: Allow claiming PCI PF if child VF is unavailable https://review.opendev.org/c/openstack/nova/+/840844 | |
| 09:03:53 | Uggla | gibi, thanks for the hungarian lesson ! ;) | |
| 09:04:03 | gibi | you are welcome :) | |
| 09:12:21 | opendevreview | Balazs Gibizer proposed openstack/nova stable/victoria: Isolate PCI tracker unit tests https://review.opendev.org/c/openstack/nova/+/840845 | |
| 09:12:22 | opendevreview | Balazs Gibizer proposed openstack/nova stable/victoria: Remove unavailable but not reported PCI devices at startup https://review.opendev.org/c/openstack/nova/+/840846 | |
| 09:12:23 | opendevreview | Balazs Gibizer proposed openstack/nova stable/victoria: Simulate bug 1969496 https://review.opendev.org/c/openstack/nova/+/840847 | |
| 09:12:24 | opendevreview | Balazs Gibizer proposed openstack/nova stable/victoria: Allow claiming PCI PF if child VF is unavailable https://review.opendev.org/c/openstack/nova/+/840848 | |
| 09:14:35 | opendevreview | Balazs Gibizer proposed openstack/nova stable/yoga: Retry in CellDatabases fixture when global DB state changes https://review.opendev.org/c/openstack/nova/+/840734 | |
| 09:18:32 | opendevreview | Balazs Gibizer proposed openstack/nova stable/ussuri: Isolate PCI tracker unit tests https://review.opendev.org/c/openstack/nova/+/840850 | |
| 09:18:33 | opendevreview | Balazs Gibizer proposed openstack/nova stable/ussuri: Remove unavailable but not reported PCI devices at startup https://review.opendev.org/c/openstack/nova/+/840851 | |
| 09:18:34 | opendevreview | Balazs Gibizer proposed openstack/nova stable/ussuri: Simulate bug 1969496 https://review.opendev.org/c/openstack/nova/+/840852 | |
| 09:18:35 | opendevreview | Balazs Gibizer proposed openstack/nova stable/ussuri: Allow claiming PCI PF if child VF is unavailable https://review.opendev.org/c/openstack/nova/+/840853 | |
| 09:20:25 | opendevreview | Balazs Gibizer proposed openstack/nova stable/train: Remove unavailable but not reported PCI devices at startup https://review.opendev.org/c/openstack/nova/+/839717 | |
| 09:20:26 | opendevreview | Balazs Gibizer proposed openstack/nova stable/train: Simulate bug 1969496 https://review.opendev.org/c/openstack/nova/+/839718 | |
| 09:20:27 | opendevreview | Balazs Gibizer proposed openstack/nova stable/train: Allow claiming PCI PF if child VF is unavailable https://review.opendev.org/c/openstack/nova/+/839719 | |
| 09:20:28 | opendevreview | Balazs Gibizer proposed openstack/nova stable/train: Isolate PCI tracker unit tests https://review.opendev.org/c/openstack/nova/+/840854 | |
| 10:11:06 | sean-k-mooney | gibi: +w on https://review.opendev.org/c/openstack/nova/+/840822 | |
| 10:11:38 | gibi | sean-k-mooney: thansk | |
| 12:15:12 | opendevreview | ribaudr proposed openstack/nova-specs master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova-specs/+/831506 | |
| 12:17:35 | opendevreview | ribaudr proposed openstack/nova master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova/+/831507 | |
| 13:34:16 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Use network's dns_domain in metadata hostname https://review.opendev.org/c/openstack/nova/+/840899 | |
| 13:34:25 | artom | Now to write the spec :P | |
| 13:36:32 | sean-k-mooney | oh free -2 then :P | |
| 13:36:46 | sean-k-mooney | the spec should be fairly short i guess | |
| 13:37:36 | artom | I'll need to explain why our PTG thinking of hostname per network didn't pan out | |
| 13:37:43 | artom | tl;dr, no schema for it | |
| 13:38:00 | artom | And cloud-init doesn't know/care, it only cares about the single hostname in the "main" metadata | |
| 13:39:51 | sean-k-mooney | it was fqdn per port | |
| 13:40:25 | sean-k-mooney | well we are not touching hostname at all | |
| 13:40:28 | sean-k-mooney | just adding fqdn | |
| 13:41:12 | artom | And since the point is to make cloud-init not clobber the DHCP-provided hostname, I think we have to touch the hostname | |
| 13:41:26 | sean-k-mooney | then no | |
| 13:41:31 | sean-k-mooney | we should not do this | |
| 13:42:10 | sean-k-mooney | the point is not to change the value of /etc/hostname that shoudl not end up with the fqdn | |
| 13:42:41 | sean-k-mooney | it shoudl have the hostname but the domain name shoudl be avaiabel to used for generating /etc/hosts | |
| 13:43:02 | sean-k-mooney | or making hostname -f provide the fqdn | |
| 13:44:23 | artom | I mean we already have code in there that uses CONF.api.dhcp_domain, which defaults to 'novalocal', and joins that with the instance hostname and returns the whole thing as the hostname | |
| 13:44:32 | artom | So not having the FQDN in hostname has sailed | |
| 13:44:42 | sean-k-mooney | artom: if you going donw that pacth we ar eback to needing an nova api change to pass teh instance default domain name | |
| 13:44:50 | frickler | artom: is this related to https://review.opendev.org/c/openstack/neutron-specs/+/832658 ? | |
| 13:45:12 | artom | frickler, yep | |
| 13:45:29 | sean-k-mooney | well not really its relate dbut not dependent on that | |
| 13:45:43 | artom | sean-k-mooney, not necessarily. I mean, we can, but grabbing it from the network's dns_domain is also valid | |
| 13:45:55 | sean-k-mooney | only if doen at the per port level | |
| 13:45:58 | sean-k-mooney | not at the vm level | |
| 13:46:13 | artom | Yeah, so that's... not great | |
| 13:46:27 | artom | The assumption is that all of a VM's networks will have the same domain name | |
| 13:46:33 | sean-k-mooney | there are no port ordering garentees so theere is no such thing as first port even to say just take the first one | |
| 13:46:35 | artom | Which is true in the case of the operator who wants this | |
| 13:46:45 | sean-k-mooney | artom: right that is flat out not valid | |
| 13:46:51 | sean-k-mooney | we cannot assume that | |
| 13:46:59 | sean-k-mooney | becuae that is not how it works normally | |
| 13:47:19 | sean-k-mooney | take a ooo deployment each neutorn network typicaly has a differnt domain | |
| 13:47:41 | sean-k-mooney | e.g. our manament domain. internal api ectra | |
| 13:48:13 | artom | sean-k-mooney, so, backing up a bit, the purpose is to prevent cloud-init from clobbering the DHCP-provided domain name | |
| 13:48:22 | sean-k-mooney | no | |
| 13:48:26 | artom | Yes! | |
| 13:48:27 | sean-k-mooney | that is not the purpose | |
| 13:48:31 | artom | That is literally what the use case is | |
| 13:48:39 | sean-k-mooney | no it is not | |
| 13:48:52 | artom | Then we're talking about different use cases :) | |
| 13:48:55 | sean-k-mooney | the usecaue is to provide a way to pass domain info via the metadata service | |
| 13:49:17 | sean-k-mooney | cloud-init is one implemation but not the only one | |
| 13:49:26 | artom | We're talking about different use cases then | |
| 13:49:41 | sean-k-mooney | sure so my joking -2 shoudl actully be a really one | |
| 13:49:53 | artom | I mean it's WIP anyways | |
| 13:50:11 | sean-k-mooney | i ment on the spec | |
| 13:50:22 | artom | There is no spec yet | |
| 13:52:04 | sean-k-mooney | so if we look at https://cloudinit.readthedocs.io/en/latest/topics/instancedata.html | |
| 13:52:25 | artom | So there's public_hostname and local_hostname *per interface* in https://cloudinit.readthedocs.io/en/latest/topics/instancedata.html#example-output | |
| 13:52:25 | sean-k-mooney | we had talked about updaing the info in the interface section | |
| 13:52:36 | artom | But I couldn't find where that actually comes from | |
| 13:53:33 | artom | I couldn't find any existing schema in either ec2 or the openstack format for per-interface hostname | |
| 13:53:38 | sean-k-mooney | artom: this is based on aws's implemenation | |
| 13:53:44 | sean-k-mooney | not openstacks | |
| 13:54:11 | sean-k-mooney | so i dont think openstack genergates thsi today | |
| 13:54:45 | artom | Yeah, there's nothing that I could find in https://docs.openstack.org/nova/latest/user/metadata.html#openstack-format-metadata | |
| 13:54:54 | sean-k-mooney | i would have expected use to keep local host name as just the host naem and pulblic_hostname as i guess the fqdu in the interface section | |
| 13:56:33 | artom | So I don't think we generate any of that today | |
| 13:56:57 | sean-k-mooney | right we dont | |
| 13:57:07 | sean-k-mooney | which is what you were ment to be implementing | |
| 13:57:30 | artom | So what happens for the openstack formatted metadata? | |
| 13:57:37 | sean-k-mooney | so we coudl do this on a per port basis sicne we decide ot not take domain as a server paramter | |
| 13:57:49 | sean-k-mooney | artom: we extend it with this same info | |
| 13:57:57 | sean-k-mooney | with a new version number | |
| 13:58:10 | sean-k-mooney | well date sicne we use date for the version | |
| 14:06:25 | sean-k-mooney | bauzas: do you want to wait for the next team meetign for people ot review https://review.opendev.org/c/openstack/project-config/+/837595 or will i ask infra to review | |
| 14:13:05 | bauzas | sean-k-mooney: not really, I accepted the names | |
| 14:13:13 | bauzas | but if you want, we can wait until the meeting | |
| 14:13:25 | sean-k-mooney | i would be happy to proceed as is | |
| 14:13:57 | sean-k-mooney | i told infra we would wait already so lets leave it | |
| 14:14:01 | sean-k-mooney | we are not in a rush | |