| bauzas | fwiw, I'll attend the publicclould vPTG sessions today | 11:50 |
|---|---|---|
| bauzas | gibi: sean-k-mooney: we need a second core, please https://review.opendev.org/c/openstack/nova/+/875621 | 12:51 |
| sean-k-mooney | we are pas the RC phase so sure. | 12:52 |
| sean-k-mooney | bauzas: by the way we know what az the instance is on in the hostmanager but on in the filters | 12:54 |
| sean-k-mooney | bauzas: that is why we need the new filed | 12:55 |
| bauzas | sean-k-mooney: tbh, before discussing about it, I'd prefer to have a spec | 12:55 |
| bauzas | because I don't understand about the usecase | 12:55 |
| bauzas | (I mean AZ soft affinity) | 12:55 |
| sean-k-mooney | bauzas: mnaser was the first to bring it up | 12:56 |
| bauzas | but ok, if we need to create a field, meh then | 12:56 |
| sean-k-mooney | bauzas: basically the use the cross_az attach option in the vexhost public cloud | 12:56 |
| bauzas | for example, if you see, people don't even know about all the existing config options | 12:56 |
| sean-k-mooney | but they dont force user to set an az when the vm is created | 12:57 |
| bauzas | even if we document it in the config option | 12:57 |
| sean-k-mooney | and then if the user does a rezie it can fail and get reschedulerd | 12:57 |
| bauzas | sean-k-mooney: I think we said before that the fact that Cinder uses AZs is different from Nova | 12:57 |
| sean-k-mooney | thats not true however and also not the problem | 12:57 |
| bauzas | so I'm a bit not happy with the cross_az_attach option | 12:57 |
| sean-k-mooney | the propbelm is we have a config cross_az_attach that changes api/sechduler beahvior in a non discofverable way | 12:58 |
| bauzas | maybe we would rather need to discuss with Cinder what we could do if so | 12:58 |
| sean-k-mooney | right so this is a very very similar situration to the routed networks issue | 12:58 |
| sean-k-mooney | we have an addtional constratint when cross_az_attach=false (like the ip aviablity) which is not currently consider by placement or the schduler | 12:59 |
| bauzas | anyway, going to the vPTG now | 12:59 |
| sean-k-mooney | the correct longterm fix would be too model cinder volume backend avaibality as placmenet aggreates | 12:59 |
| bauzas | sean-k-mooney: yeah, so we should discuss this with Cinder | 13:00 |
| bauzas | indeed | 13:00 |
| bauzas | I agree with ^ | 13:00 |
| sean-k-mooney | ya so that woudl be good to do and i have a usecasue related to this that i would like us to also do | 13:00 |
| sean-k-mooney | ill add it to the adgenda | 13:00 |
| sean-k-mooney | clip notes version is, when using iamges_type RBD i would like to create an aggreate usign the ceph FSID and recored the fsid of cluster the instance is using in the instnace_system_metadta so that we can automatically sechdule ot those in teh came ceph cluster wehn moving RBD backed vms | 13:02 |
| sean-k-mooney | right now we have exactly the same problem that we can select hosts that do not have accces to the cluster based on there local confg | 13:02 |
| sean-k-mooney | so the live migration can succed but thten a hard reboot will fail | 13:03 |
| sean-k-mooney | its a differnt but simiarl probelm casue by the fact we dont model network storage in placment | 13:03 |
| sean-k-mooney | so there is both a cinder and nova only varient of the same issue | 13:03 |
| bauzas | gibi: you're listening to the publiccloud SIG session ? | 13:56 |
| bauzas | fwiw, they're discussing about interop flavors | 13:56 |
| gibi | bauzas: I try to follow but pretty distracted | 14:14 |
| sean-k-mooney | bauzas: i am not currently on that but i left a comment in the ether pad | 14:16 |
| bauzas | sean-k-mooney: tl;dr they have a pretty solid usecase about filtering flavors on attributes | 14:17 |
| bauzas | operator-defined attributes | 14:17 |
| sean-k-mooney | filtering yes but that is an interop issue if they are custom | 14:17 |
| opendevreview | Eric Harney proposed openstack/nova master: mypy: Fix implicit optional usage https://review.opendev.org/c/openstack/nova/+/878693 | 15:08 |
| bauzas | bim bam boom | 15:16 |
| bauzas | https://07de6a0c9e6ec0c6835f-ccccbfab26b1456f69293167016566bc.ssl.cf2.rackcdn.com/875621/10/gate/openstack-tox-pep8/e50f9f0/job-output.txt | 15:16 |
| bauzas | huzzah, mypy issue | 15:16 |
| bauzas | related to https://review.opendev.org/c/openstack/requirements/+/872065 | 15:17 |
| bauzas | gibi: sean-k-mooney: ^ | 15:17 |
| bauzas | working on a fix with no_implicit_optional=False | 15:18 |
| gibi | bauzas: I'm fine with https://review.opendev.org/c/openstack/nova/+/878693 as a fix for ^^ | 15:25 |
| sean-k-mooney | i agree with the mypi fix in nova | 15:28 |
| bauzas | gibi: damn, I was jinxed :) | 15:29 |
| bauzas | I was working on a patch | 15:29 |
| bauzas | anyway +Wd | 15:29 |
| kashyap | bauzas: Are there meeting minutes from the public-cloud WG? | 15:29 |
| bauzas | and sent an mail to -discuss to prevent rechecks | 15:29 |
| kashyap | Or Etherpad nodes | 15:29 |
| bauzas | kashyap: https://etherpad.opendev.org/p/march2023-ptg-publiccloud | 15:30 |
| bauzas | tl;dr: most of the discussion was about flavors API | 15:30 |
| bauzas | and how filtering extra specs is a need | 15:31 |
| bauzas | + some can of worms about some possible discoverable mechanism for describing a flavor | 15:31 |
| bauzas | they'll write a backlog spec so we'll nitpick over it | 15:31 |
| frickler | bauzas: regarding mypy, does it make sense to add nova pep8 job to reqs repo? I hope we will get more frequent updates again in the future, after getting almost none in the last cycle | 15:45 |
| bauzas | frickler: good question, I dunno what to say here | 15:47 |
| bauzas | because we use the pep8 tox target for mypy | 15:47 |
| bauzas | so either the pep8 openstack one should also use mypy or maybe we should use a different tox job | 15:47 |
| bauzas | I just don't want to test again pep89 twice | 15:47 |
| frickler | bauzas: I'm not sure I understand that, I was talking about adding cross-nova-pep8 to requirements repo. currently there only is cross-nova-py310 | 15:48 |
| bauzas | frickler: ok, then +1 | 15:49 |
| frickler | o.k., I'm feeling a bit guilty about having pushed the u-c updates yesterday, so I'll try to set that up as compensation ;-) | 15:51 |
| gibi | ^^ +1 from me as well | 15:52 |
| gmann | frickler: +1 | 15:58 |
| frickler | does any of you know why this files filter is only in place in gate and only for nova+placement? https://opendev.org/openstack/requirements/src/branch/master/.zuul.d/project.yaml#L126-L129 | 16:05 |
| bauzas | frickler: good question, I dunno https://opendev.org/openstack/requirements/commit/91eaa0cd7f75f5f1b93226e4bb2de64aa4fed2b3 | 16:32 |
| bauzas | looks like we just followed a pattern | 16:33 |
| ygk_12345 | Hi All | 17:29 |
| opendevreview | Merged openstack/placement master: Do not use coalesce for consumers.uuid https://review.opendev.org/c/openstack/placement/+/877852 | 18:32 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!