Monday, 2020-06-08

*** tetsuro has joined #openstack-nova00:07
*** tetsuro has quit IRC00:08
*** bbowen has quit IRC00:10
*** brinzhang has joined #openstack-nova00:12
*** brinzhang_ has joined #openstack-nova00:18
*** brinzhang has quit IRC00:21
*** brinzhang0 has joined #openstack-nova00:21
*** brinzhang_ has quit IRC00:24
*** eandersson has quit IRC00:35
*** ccstone has quit IRC00:35
*** eandersson has joined #openstack-nova00:36
*** boxiang_ has quit IRC00:38
*** boxiang has joined #openstack-nova00:38
*** hongbin has quit IRC00:47
*** brinzhang0 has quit IRC00:48
*** brinzhang0 has joined #openstack-nova00:49
*** hongbin has joined #openstack-nova00:50
*** xiaolin has joined #openstack-nova01:07
*** tetsuro has joined #openstack-nova01:07
*** Liang__ has joined #openstack-nova01:11
*** yaawang has quit IRC01:11
*** bbowen has joined #openstack-nova01:21
*** songwenping_ has quit IRC01:35
*** songwenping_ has joined #openstack-nova01:36
*** songwenping_ has quit IRC01:45
*** brinzhang_ has joined #openstack-nova01:49
*** brinzhang0 has quit IRC01:51
*** sapd1 has joined #openstack-nova02:21
*** rcernin has quit IRC02:28
*** boxiang_ has joined #openstack-nova02:30
*** boxiang has quit IRC02:33
*** Liang__ has quit IRC02:46
*** rcernin has joined #openstack-nova02:58
*** tetsuro has quit IRC03:08
*** tetsuro has joined #openstack-nova03:08
*** ociuhandu has joined #openstack-nova03:12
*** sapd1 has quit IRC03:16
*** ociuhandu has quit IRC03:17
*** tetsuro has quit IRC03:18
*** sapd1 has joined #openstack-nova03:31
*** psachin has joined #openstack-nova03:34
*** jmlowe has quit IRC03:43
*** tetsuro has joined #openstack-nova03:44
*** jmlowe has joined #openstack-nova03:47
*** yaawang has joined #openstack-nova03:51
*** slaweq has joined #openstack-nova04:15
*** slaweq has quit IRC04:20
*** vishalmanchanda has joined #openstack-nova04:33
*** redrobot has quit IRC04:36
*** ratailor has joined #openstack-nova04:38
*** sapd1 has quit IRC04:54
*** Liang__ has joined #openstack-nova04:59
*** sapd1 has joined #openstack-nova05:10
*** hongbin has quit IRC05:12
*** Liang__ is now known as LiangFang05:15
*** redrobot has joined #openstack-nova05:20
*** links has joined #openstack-nova05:43
*** links has quit IRC05:49
*** links has joined #openstack-nova05:58
*** factor has quit IRC06:04
*** jawad_axd has joined #openstack-nova06:07
*** jraju__ has joined #openstack-nova06:11
*** links has quit IRC06:11
*** jawad_axd has quit IRC06:12
*** udesale has joined #openstack-nova06:19
*** jsuchome has joined #openstack-nova06:27
*** links has joined #openstack-nova06:32
*** jraju__ has quit IRC06:33
*** xiaolin has quit IRC06:35
*** ttsiouts has joined #openstack-nova06:42
*** lvdombrkr has joined #openstack-nova06:46
*** tesseract has joined #openstack-nova06:59
*** LiangFang has quit IRC07:02
*** Liang__ has joined #openstack-nova07:03
*** Liang__ is now known as LiangFang07:07
*** ttsiouts has quit IRC07:08
*** ircuser-1 has quit IRC07:10
*** iurygregory has joined #openstack-nova07:10
*** rpittau|afk is now known as rpittau07:14
*** ralonsoh has joined #openstack-nova07:15
*** slaweq has joined #openstack-nova07:17
bauzasgood morning Nova07:20
*** rcernin has quit IRC07:21
*** maciejjozefczyk has joined #openstack-nova07:25
lvdombrkrmorning nova. is there other ways to inject password except inject_password=true ?07:30
*** maciejjozefczyk_ has joined #openstack-nova07:31
*** maciejjozefczyk has quit IRC07:31
*** ccamacho has joined #openstack-nova07:32
*** jawad_axd has joined #openstack-nova07:34
*** xek_ has joined #openstack-nova07:37
*** tosky has joined #openstack-nova07:39
*** ttsiouts has joined #openstack-nova07:39
*** xek_ has quit IRC07:44
*** ttsiouts has quit IRC07:44
*** xek has joined #openstack-nova07:44
gibibauzas: good morning07:53
gibilvdombrkr: you can use cloud init in the guest and you can use the config driver or the metadata service to provide input for cloud init07:54
bauzasfirst time I don't feel jetlagged or fried after a PTG07:54
bauzasgibi: didn't we deprecated password injection btw. ?07:55
gibibauzas: I don't miss the jetlag part of the PTG :)07:55
bauzasgibi: next one, we won't have jetlag too07:55
bauzas:)07:55
gibibauzas: I think we did https://blueprints.launchpad.net/nova/+spec/deprecate-file-injection07:55
gibiohh this is file injection not password injection07:55
lvdombrkrgibi: thanks for answer. but if i will use metadata, then passwortd will be stored in metadata in plain text, right?07:56
bauzasah my bad07:56
bauzasI remembered we did deprecated some injection, but like you, it was the wrong way07:57
gibilvdombrkr: you can inject public keys07:58
*** xek_ has joined #openstack-nova07:58
lvdombrkrgibi: yes yes about public keys i know, but i need excatly password07:58
*** martinkennelly has joined #openstack-nova07:58
bauzaslvdombrkr: question, why can't you use a config tool like Ansible for injecting such things if you don't wanna use the tool we provide ?07:58
fricklerbauzas: because it is broken? see https://launchpad.net/bugs/188242107:59
openstackLaunchpad bug 1882421 in OpenStack Compute (nova) "inject_password fails with python3" [Undecided,In progress] - Assigned to Dr. Jens Harbott (j-harbott)07:59
lvdombrkrbauzas: i want use config you provided (inject_password) but its not working with Centos8 Python308:00
bauzasfrickler: ah, gtk08:00
*** xek has quit IRC08:00
* frickler built a patch yesterday but isn't too sure about all this bytes ./. str stuff08:00
bauzasthen, I'd recommend to go thru external tools until it's fixed08:00
*** maciejjozefczyk has joined #openstack-nova08:01
bauzasfrickler: I can take a look at your change if you will08:03
lvdombrkrbauzas frickler : thanks for answers. how long time can take fix this?08:03
*** maciejjozefczyk_ has quit IRC08:04
gibibauzas, lvdombrkr, frickler: I'm reviewing the fix now (while on a parallel phone meeting so it will take time)08:06
*** tkajinam has quit IRC08:07
*** ravsingh has joined #openstack-nova08:07
*** avolkov has joined #openstack-nova08:07
*** nightmare_unreal has joined #openstack-nova08:08
lvdombrkrgibi bauzas frickler:  perfect guys, keep me updated please :)08:10
gibilvdombrkr: sure08:11
aarentsgood morning nova08:13
aarentsgibi: bauzas just FYI, I've made some stupid max attachement contest on a devstack with machine-type q35:08:13
gibiaarents: good morning08:13
aarentsnova interface-list 6cc30912-c4ae-493d-8209-e19f16f101bc | grep -c ACTIVE08:13
aarents22008:13
aarentsit is only possible because I override in code num_pcie_ports.max to 224 instead of 28  (we force max to 28 because of limitation with aarch64/virt), without doing it max attachment on q35 is 24  (28 - 4 reserved) and on pc it is 26, so we are still better on pc than on q35 regarding max attachement with current code.08:13
bauzasaarents: good morning08:13
bauzasthat reminds me a bell, but I don't remember exactly what08:14
gibiaarents: so we need to make the num_pcie_ports.max dynamic based on the machine typ08:14
gibie08:14
aarentsgibi: I think08:14
bauzasdidn't we had an issue like this that was raised before ?08:15
bauzasor maybe it was for usb devices, I can't recall08:15
gibibauzas: I don't have memory about it08:17
*** ttsiouts has joined #openstack-nova08:19
*** sapd1 has quit IRC08:19
*** ttx has joined #openstack-nova08:19
bauzasfrickler: this could help you to understand the distinguo between the bytes and the str types in py3 https://eli.thegreenplace.net/2012/01/30/the-bytesstr-dichotomy-in-python-308:20
* bauzas unfortunately only has a good article in French about it08:20
bauzasfrickler: tl;dr you shouldn't explicitely tell a string as byte-coded08:21
fricklerbauzas: yes, this is clear, the question for me is, when a function needs bytes, is it better to make the caller supply bytes or have the function convert str arguments to bytes. that happens multiple times in the inject_* code08:31
*** sapd1 has joined #openstack-nova08:32
*** xiaolin has joined #openstack-nova08:36
bauzasfrickler: I definitely prefer the latter08:48
bauzasbytes type should only be used when you need08:48
*** xek_ has quit IRC08:50
*** ttsiouts has quit IRC08:52
*** xek_ has joined #openstack-nova08:53
*** ravsingh has quit IRC08:59
*** ravsingh has joined #openstack-nova09:00
bauzasfrickler: I think i found why you need to horribly encode all the strings09:02
bauzastl;dr: the guestfs import seems to be py309:03
bauzaspy2*09:03
bauzasthat should come from libguestfs not having the py3 bindings09:05
fricklerbauzas: well I tested locally on focal with python3-guestfs installed, surely no py2 involved there09:07
bauzaswhat I don't understand is why you get a bytes-encoded FS09:08
fricklerbauzas: I think that's pretty normal, files contain bytes, you'd have to tell guestfs the encoding in order to be able to convert them to strings09:09
bauzassure, of course09:10
bauzaseither way, I'm surprised we need to explicitely encode all the strings09:10
*** links has quit IRC09:16
*** xek_ has quit IRC09:17
*** mdbooth has joined #openstack-nova09:32
gibibauzas: I confirm that a py3 devstack setup has the reported byte / str problem with password injection (finally reproduced it)09:39
*** LiangFang has quit IRC09:49
*** whoami-rajat has joined #openstack-nova09:54
whoami-rajathi #openstack-nova i've a query, during reserve_volume call do we have a lock on the volume? if there's a multiattach volume and multiple instances issue reserve request on it then are we secure in that case?09:54
*** ccamacho has quit IRC09:55
*** iokiwi3 has quit IRC09:56
*** adriant has quit IRC09:56
*** adriant has joined #openstack-nova09:57
*** slaweq has quit IRC09:57
*** slaweq has joined #openstack-nova09:58
*** xiaolin has quit IRC10:12
*** rpittau is now known as rpittau|bbl10:15
*** sapd1 has quit IRC10:17
*** ttsiouts has joined #openstack-nova10:27
sean-k-mooneystephenfin: small docs review for you https://review.opendev.org/#/c/726936/810:29
*** ccamacho has joined #openstack-nova10:31
*** sapd1 has joined #openstack-nova10:32
*** tetsuro has quit IRC10:32
stephenfinsean-k-mooney: done10:33
*** nightmare_unreal has quit IRC10:33
openstackgerritMerged openstack/os-traits master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/73235510:37
sean-k-mooneystephenfin: cool thanks10:38
sean-k-mooneybauzas: want to finish the review it will take you 30 seconds https://review.opendev.org/#/c/726936/10:39
sean-k-mooneystephenfin: did you recongise the name by the way. i think they are a new contiributor10:40
stephenfinI suspect so too10:40
stephenfinProbably someone deploying things and finding the gaps10:40
sean-k-mooneylooks like ther other patches were to the sandbox project so ya likely.10:40
openstackgerritStephen Finucane proposed openstack/nova-specs master: Encrypted Emulated Virtual TPM  https://review.opendev.org/72850510:41
*** derekh has joined #openstack-nova10:46
*** lvdombrkr has quit IRC11:00
*** ttsiouts has quit IRC11:01
*** yaawang has quit IRC11:05
*** yaawang has joined #openstack-nova11:05
stephenfinbauzas: Can I get you to look at https://review.opendev.org/728505 this week, please?11:12
stephenfinbauzas: I also have a couple of code reviews I'd like some eyes on. Happy to horse trade ;)11:12
*** lvdombrkr has joined #openstack-nova11:15
*** nightmare_unreal has joined #openstack-nova11:32
*** ttsiouts has joined #openstack-nova11:47
bauzassean-k-mooney: stephenfin: sorry was at lunch11:52
*** ttsiouts has quit IRC11:52
*** Luzi has joined #openstack-nova11:53
*** raildo has joined #openstack-nova11:57
*** sapd1 has quit IRC11:57
bauzasstephenfin: sure, I wanna do some specs reviews this week anyway11:59
*** rpittau|bbl is now known as rpittau12:13
*** martinkennelly has quit IRC12:19
fricklerso if anyone is interesting in getting devstack testing on focal to proceed, I created this bug report. lyarwood had started to look at it earlier, I don't have enough clue myself, in the end it might be nova, libvirt, qemu, cinder or even tempest is the actual culprit https://bugs.launchpad.net/nova/+bug/188252112:21
openstackLaunchpad bug 1882521 in OpenStack Compute (nova) "Failing device detachments on Focal" [Undecided,New]12:21
*** martinkennelly has joined #openstack-nova12:22
sean-k-mooneyfrickler: are there any known blockers right now12:23
*** martinkennelly has quit IRC12:23
sean-k-mooneyfrickler: ill be setting up a devenv for numa stuff later today and was considering if i should try centos 8 or ubuntu 20.04 instaed of sticking with 18.0412:23
sean-k-mooneyDevice detach failed for vdb: Unable to detach the device from the live config.12:24
sean-k-mooneyfrickler: that looks like its comming from libvirt althoug hi have not looked at the code yet12:24
sean-k-mooneyfrickler: was the default lvm backend or ceph?12:25
fricklersean-k-mooney: in devstack it is lvm, although I tested locally with ceph and see similar issues. but there's also a different bug for that, one moment12:28
kashyapsean-k-mooney: Hey, do you know of any prior rejected work on memory ballooning?12:28
kashyap... in Nova12:28
fricklersean-k-mooney: also, I think that those test-failures are blocking for focal, I wouldn't want to merge focal support with them blacklisted12:28
sean-k-mooneynot specifcally but i know dansmit is not a fan of nova managing it12:28
sean-k-mooneyfrickler: oh i ment in terms of me swaping my daily dev env to focal rather then the ci12:29
sean-k-mooneyfrickler: if i start using it and hit error its more likely that im going to fix them12:29
sean-k-mooneykashyap: i know that at least one user wants us to remove the automatic memory balloan that is added by livirt12:30
sean-k-mooneykashyap: i also know that we are getting asked to consider added memory balloan managemnt downstream12:31
*** ratailor has quit IRC12:31
fricklersean-k-mooney: ah, I see, you'd need to use my patch https://review.opendev.org/704831 , apart from the tempest failures it has been running pretty well for me for some months now12:32
sean-k-mooneyfrickler: cool i was planning on swapping to it for the numa work im starting. the vms i have created are already on it but i had not got as far as stacking yey12:34
sean-k-mooneyso if i use that devstack patch it should be good12:34
fricklersean-k-mooney: the ceph thing seems to have been fixed with https://review.opendev.org/725812 , not sure I've tested with that patch in tempest12:34
fricklersean-k-mooney: yes, that would be great, let me know if you find any further issues12:35
sean-k-mooneyill set up the multi node env and before i start modifying code ill try and do a few volume tests i assuem you are hitting the issue locally too?12:36
sean-k-mooneye.g. its not just in the ci12:36
fricklersean-k-mooney: yes, although locally I didn't see it 100%, might be a bit load/memory pressure related12:36
sean-k-mooneyhehe its funny but i realised the other day i finally got https://opendev.org/opentack into my muscel memory instead of defaulting to github12:37
sean-k-mooneyfrickler: so while your here12:38
sean-k-mooneyubuntu@numa-1:/opt/repos/devstack$ python3 -m pip install --user git-review12:38
sean-k-mooney/usr/bin/python3: No module named pip12:38
sean-k-mooneyam i imagining things or is pip not part of the standard libary now12:38
sean-k-mooneyi know how to fix this12:38
sean-k-mooneybut im just wondering why on 3.8 i have to install a seperate package12:38
sean-k-mooneyi tought it was ment to be always present12:39
frickleryeah, they did strange things there, same for venv, no idea why. for git-review, I just use the pkg from universe, not pip12:39
sean-k-mooneyi have had issue on centos7 with it being old and busted12:40
sean-k-mooneyso i default to pip for it12:40
sean-k-mooneymainly to have a consitent behavior across distos to be honest12:41
fricklerI also had to create a symlink for pip3.8 since the distro pkg only provides pip3, but devstack wants it version specific12:43
sean-k-mooneyfrickler: i think they are treating pip and venv as dev tools12:43
sean-k-mooneyfrickler: we proably should modify devstack to not use the pip command but instaead do it the python3 -m pip  way12:44
*** ttsiouts has joined #openstack-nova12:44
sean-k-mooneyit will avoid that issue but also i think that is the new recomendation to ensure it installs where you think it will12:45
sean-k-mooneyobviously python3 can be python3.8 or the version specifc one too12:45
*** martinkennelly has joined #openstack-nova12:47
*** mgariepy has quit IRC12:48
*** udesale_ has joined #openstack-nova12:49
*** eharney has joined #openstack-nova12:52
*** udesale has quit IRC12:52
*** martinkennelly has quit IRC12:53
*** brinzhang0 has joined #openstack-nova12:54
*** martinkennelly has joined #openstack-nova12:55
*** brinzhang_ has quit IRC12:57
*** artom has joined #openstack-nova12:59
*** mgariepy has joined #openstack-nova13:03
*** ociuhandu has joined #openstack-nova13:05
sean-k-mooneygibi: bauzas if we set  map_az_to_placement_aggregate=true we nolonger need the avaiablity zone filter right13:06
*** lbragstad has joined #openstack-nova13:06
sean-k-mooneyhave we talked about when we might deprecate that and swap over to the placment way of doing it by default13:06
gibisean-k-mooney: only aggregate info is in placeent but not AZ info13:06
kashyapsean-k-mooney: libvirt doesn't do any automatic memory ballooning, does it?13:07
gibiI mean the aggregate metadata is not in placement and the AZ name is in the aggregate metadata13:07
sean-k-mooneyright but what i ment is could we start enabling the prefilter by default and deprecate the filter for removal13:07
sean-k-mooneygibi: specifclaly this prefilter https://github.com/openstack/nova/blob/8988316b8c132c9662dea6cf0345975e87ce7344/nova/scheduler/request_filter.py#L136-L16313:08
sean-k-mooneykashyap: no it does not13:08
kashyapsean-k-mooney: Then what do you mean by: "i know that at least one user wants us to remove the automatic memory balloan that is added by livirt"13:08
sean-k-mooneykashyap: you can makrk the domain to try an deflate on OOM but even then im not sure that it will work without something in the guest13:08
*** nweinber has joined #openstack-nova13:09
sean-k-mooneykashyap: libvirt adds a virtio-ballon device even if you dont ask for one by default13:09
kashyapThat "deflate-on-OOM" (which is a QEMU feature) is only for the guest to reclaim its mem back - nothing related to host13:09
sean-k-mooneykashyap: we had one user ask for a way to disable that13:09
kashyapsean-k-mooney: Ah, merely the adding of the ballooning device13:09
gibisean-k-mooney: I see now. Yes it make sense. I think we can deprecate the Filter and user the perfilter + placement13:09
kashyapsean-k-mooney: Yes, I know - the real-time usecase doesn't want it13:09
sean-k-mooneykashyap: actrully it was not for realtime13:10
sean-k-mooneykashyap: it was for a guest with 2TB of ram and the debian folk were wondering if that was causeing some qemu crashes13:10
sean-k-mooneybut i actully think it was related to the adress space not the memory ballaon13:10
sean-k-mooneygibi: ok ill file a blueprint and submit a patch13:11
stephenfingibi: replied on https://review.opendev.org/#/c/733668/113:11
sean-k-mooneygibi: i was just looking at my local.conf an noticed i was still enebling the retry filter which has notbeen needed for a few release and notice i was also enabling the az filter13:11
*** ociuhandu has quit IRC13:12
openstackgerritMerged openstack/nova master: [Nova] Add reference to Placement installation guide  https://review.opendev.org/72693613:13
sean-k-mooneykashyap: the issue they were hiting is by default qemu was not allcating enough virtual adressspace to create a guest larger than 2TB13:14
*** brinzhang_ has joined #openstack-nova13:16
*** ttsiouts has quit IRC13:18
*** brinzhang has joined #openstack-nova13:18
*** brinzhang0 has quit IRC13:19
kashyapsean-k-mooney: I see13:20
kashyapThx for the context13:20
*** iurygregory is now known as iurygregory|afk13:20
*** brinzhang_ has quit IRC13:20
sean-k-mooneykashyap: ya they just wanted to turn it off for debuging reasons they were hitting basically this bug https://bugzilla.redhat.com/show_bug.cgi?id=630975 or https://bugs.launchpad.net/ubuntu/+source/qemu/+bug/176905313:21
openstackbugzilla.redhat.com bug 630975 in seabios "KVM guest limited to 40bit of physical address space" [Medium,Closed: errata] - Assigned to aarcange13:21
*** brinzhang_ has joined #openstack-nova13:21
openstackLaunchpad bug 1769053 in libvirt (Ubuntu) "Ability to control phys-bits through libvirt" [High,Triaged]13:21
sean-k-mooneyi wonder if nova shoudl actully be setting the libvirt option13:21
sean-k-mooneykashyap: they had to create a custom machine type to work around the issue13:23
*** brinzhang has quit IRC13:24
sean-k-mooneyhum i guess this is still open https://bugzilla.redhat.com/show_bug.cgi?id=157827813:24
openstackbugzilla.redhat.com bug 1578278 in libvirt "Ability to control phys-bits through libvirt" [Unspecified,New] - Assigned to libvirt-maint13:24
sean-k-mooneyso maybe this can only be done at the qemu level13:24
gibisean-k-mooney: cool thanks.13:28
gibistephenfin: looking...13:28
*** ociuhandu has joined #openstack-nova13:30
gibistephenfin: sorry. totally my bad I guess I wasn't paying enough attention. Sorry again.13:34
stephenfinit's all good :)13:34
*** lbragstad has quit IRC13:34
gibi:)13:35
*** lbragstad has joined #openstack-nova13:37
*** ociuhandu has quit IRC13:38
openstackgerritStephen Finucane proposed openstack/nova master: Use 'Exception.__traceback__' for versioned notifications  https://review.opendev.org/73366813:39
stephenfingibi: actually, looks like I had a pep8 failure on that ^ Trying to figure out why pre-commit didn't catch it13:39
sean-k-mooneystephenfin: was it a hacking rule im not sure pre-commit runs those13:40
sean-k-mooneyit might i just have not checked13:40
stephenfinnot a hacking rule, no, but pre-commit does run hacking anyway13:41
*** Luzi has quit IRC13:41
*** martinkennelly has quit IRC13:45
*** brinzhang0 has joined #openstack-nova13:47
gibistephenfin: ack, reapplied my +213:47
stephenfinta13:47
*** ociuhandu has joined #openstack-nova13:49
*** brinzhang_ has quit IRC13:50
openstackgerritMerged openstack/os-resource-classes master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/73151213:50
*** ociuhandu has quit IRC13:57
*** martinkennelly has joined #openstack-nova13:59
*** amodi has joined #openstack-nova14:05
*** priteau has joined #openstack-nova14:06
*** ravsingh has quit IRC14:16
*** dklyle has joined #openstack-nova14:23
*** dklyle has quit IRC14:24
*** dklyle has joined #openstack-nova14:24
*** ociuhandu has joined #openstack-nova14:25
*** Liang__ has joined #openstack-nova14:26
*** jawad_axd has quit IRC14:31
*** ociuhandu has quit IRC14:33
*** ociuhandu has joined #openstack-nova14:34
*** iurygregory|afk is now known as iurygregory14:36
sean-k-mooneyfrickler: so im not seeing any detach errors but i am seeing this  http://paste.openstack.org/show/794474/14:43
fricklersean-k-mooney: yeah, that's related to eventlet and py>=3.7 it seems, see my note in the bug and the other bug linked. I'm not sure whether that's related or not, haven't been able to build an easier reproducer so far, either14:45
bauzasfrickler: we don't support yet py>=3.714:45
sean-k-mooneybauzas: well we kindo of do. there is an non voting job we can enable and i think its planned for victoria14:46
sean-k-mooneyyep it is https://opendev.org/openstack/governance/src/branch/master/reference/runtimes/victoria.rst#user-content-python-runtimes-for-victoria14:47
sean-k-mooneyso 3.6 and 3.8 will be the required supported version once we swap form 18.0414:47
sean-k-mooneyto 20.0414:48
sean-k-mooneyfrickler: the eventlet thing seams to be intermitent14:48
*** ociuhandu has quit IRC14:49
sean-k-mooneyi see the command are sometimes passing fine14:49
sean-k-mooneyfrickler: given i deployed master i would apper that https://review.opendev.org/#/c/707474/ does not actully work14:52
*** mlavalle has joined #openstack-nova14:53
fricklersean-k-mooney: you might want to test with a revert, but I was assuming that it would be only incomplete and we would have much more issues without it14:53
sean-k-mooneyyes it likely incomplete but i can try that quickly.14:54
*** beekneemech is now known as bnemec14:54
sean-k-mooneyme might need to do this in oslo as well14:54
openstackgerritBalazs Gibizer proposed openstack/nova master: DNM: Test the state of VMware NSX 3pp CI  https://review.opendev.org/73411415:00
*** ociuhandu has joined #openstack-nova15:10
*** ttsiouts has joined #openstack-nova15:15
*** gyee has joined #openstack-nova15:17
*** lyarwood has quit IRC15:21
*** larainema has joined #openstack-nova15:22
*** priteau has quit IRC15:22
*** martinkennelly has quit IRC15:28
*** iurygregory has quit IRC15:29
*** iurygregory has joined #openstack-nova15:29
*** brinzhang0 has quit IRC15:33
*** ociuhandu has quit IRC15:34
*** Liang__ has quit IRC15:38
*** ttsiouts has quit IRC15:48
*** gyee has quit IRC15:53
*** rpittau is now known as rpittau|afk16:08
*** udesale_ has quit IRC16:08
*** dtantsur is now known as dtantsur|afk16:10
*** gyee has joined #openstack-nova16:15
*** nightmare_unreal has quit IRC16:43
*** gyee has quit IRC16:54
*** gyee has joined #openstack-nova16:58
*** psachin has quit IRC16:59
*** gyee has quit IRC17:00
*** tesseract has quit IRC17:00
*** derekh has quit IRC17:01
*** jawad_axd has joined #openstack-nova17:05
*** vishalmanchanda has quit IRC17:20
*** ttsiouts has joined #openstack-nova17:25
*** artom has quit IRC17:26
*** ttsiouts has quit IRC17:29
*** artom has joined #openstack-nova17:45
*** artom has quit IRC17:54
*** jawad_axd has quit IRC18:02
*** jsuchome has quit IRC18:04
*** artom has joined #openstack-nova18:08
*** hamalq has joined #openstack-nova18:13
*** ralonsoh has quit IRC18:15
*** dustinc has joined #openstack-nova18:16
*** avolkov has quit IRC19:07
*** jawad_axd has joined #openstack-nova19:19
*** xek has joined #openstack-nova19:19
dansmithsean-k-mooney: around?19:19
*** xek has quit IRC19:25
*** jawad_axd has quit IRC19:27
sean-k-mooneydansmith: yes19:28
sean-k-mooneyjust traging a networking issue how can i help19:28
dansmithsean-k-mooney: I want to create a variant of the ceph job that has multiple glance stores, so some glance conf tweaking19:28
dansmithsean-k-mooney: I think I need a devstack_local_conf section in the job def in .zuul.conf yeah?19:28
dansmithI'm not quite sure what it needs to look like, can I pastebin something in a sec for you to look at?19:29
sean-k-mooneyso you likely need to set different values on the contoler/all-in-one node vs the compute only node19:29
sean-k-mooneyso you will need to set different value in different groups19:29
sean-k-mooneydansmith: sure give me the linke and ill take a look19:30
dansmiththis can be single-node fwiw19:30
sean-k-mooneywith multiple stores?19:30
dansmithyup19:31
sean-k-mooneyoh are you configuring mutile glance backends or something19:31
sean-k-mooneyi assume it cant advertises the single file store backend as multiple stores19:32
sean-k-mooneyor in this case ceph backends19:32
dansmithone file and one ceph, but yeah you can have multiple file backends too AFAIK19:32
sean-k-mooneyya im pretty sure you can have multipel file backedns19:33
sean-k-mooneyhttps://github.com/openstack/nova/blob/master/.zuul.yaml#L154-L16119:33
sean-k-mooneybasically you are doing that19:33
sean-k-mooneybut replacen $NOVA_CPU_CONF with $GLANCE_CONF19:34
sean-k-mooneythen each "[section]" becomes "section:"19:34
dansmithsean-k-mooney: https://pastebin.com/h8Z2hTQs19:34
dansmithokay I thought so, but I saw some in that file that didn't look like section names I was expecting19:34
dansmithso I thought maybe there was more to it19:35
sean-k-mooneyso the $WHATEVER19:35
sean-k-mooneyis being replaced by devstack19:35
dansmithI figured19:35
sean-k-mooneyalthoguh you are missing a bit19:35
sean-k-mooneyhttps://github.com/openstack/nova/blob/master/.zuul.yaml#L15519:35
sean-k-mooneyyou will notice its under post-config19:36
dansmithah okay thanks19:36
sean-k-mooneyso line 155 and 156 become [[post-config|$NOVA_CPU_CONF]]19:36
dansmithack19:37
sean-k-mooneyim sure you know this but the post-config section in devstack run after all the infrastucture serivces are start and after all the default config are generated but before any of the openstack services are started19:38
dansmithyeah that's what I want19:38
sean-k-mooneyyep its both the latest point you can set it and in this casse the earlist that makes sense19:38
dansmithsean-k-mooney: so what about the bit of parenting this from the existing job and replacing the existing thing under check to reference this... that look okay too?19:39
sean-k-mooneyyes although when im testing i normally disable the other check jobs untill i get it working19:39
dansmithby just deleting them?19:40
sean-k-mooneyhttps://review.opendev.org/#/c/681771/2/.zuul.yaml19:40
sean-k-mooneyi normally coment them out but ya deleting works too19:40
dansmithack19:40
dansmithsmart :)19:40
sean-k-mooneyactully thats slight a bad example since i enabled noop and use the experimental queue to test19:41
sean-k-mooneythe expermiental queue is faster19:41
sean-k-mooneyso it will report back sooner19:41
sean-k-mooneynormaly i just use check with the other disabeld19:41
sean-k-mooneythe goal is the same jsut run the one job i care about and not waste ci resouces19:42
dansmithack, I will leave it in check because it's easier19:42
dansmithgah, conflict in .zuul19:42
sean-k-mooneybut ya once you do that all you need to do is reference it by its new name and you should be good to go19:43
sean-k-mooneyya that is the downside to commenting19:43
sean-k-mooneyby the way i normally comment out all template bar       - check-requirements19:44
sean-k-mooneybut again that is not needed i just normally do that while im getting the job working to disable pep8 and the tox jobs19:44
openstackgerritDan Smith proposed openstack/nova master: Plumb image import functionality through our glance module  https://review.opendev.org/73155019:44
openstackgerritDan Smith proposed openstack/nova master: Make libvirt able to trigger a backend image copy when needed  https://review.opendev.org/65699819:44
openstackgerritDan Smith proposed openstack/nova master: WIP: Add libvirt RBD configuration doc  https://review.opendev.org/73157719:44
openstackgerritDan Smith proposed openstack/nova master: DNM: Try to make a glance multistore job  https://review.opendev.org/73418419:44
dansmithsean-k-mooney: ^19:45
dansmithsean-k-mooney: thanks for your help. it's so much easier to just hint at you doing it, but I have to do it myself *sometimes* :)19:45
sean-k-mooneyyou proably dont need the required-projects: nova but it wont break anything19:45
sean-k-mooneyyou will get that form devstack via inheritance19:46
dansmithI copied that from another job19:46
sean-k-mooneyya its fine19:46
dansmithblindly without knowing why.. seems silly in the nova tree19:46
sean-k-mooneyits already required by tempest or devstack19:46
dansmithack19:46
sean-k-mooneyso what that does is it adds nova to LIBS_FROM_GIT19:47
sean-k-mooneywhich for libs has the sideffect of make depens-on work19:47
sean-k-mooneybut its also used for zuul to know which poject to allow depens on to work with19:48
sean-k-mooneyin general that is19:48
sean-k-mooneyso if you want depens on to work say with ooo then somewhere in the job inheritance you would need a reqiured projects19:48
sean-k-mooneyso that he zuul cloner can prepare it properly19:49
dansmithah19:49
dansmithsounds like one of those things it's easy to forget until after a few hours of tearing out hair wondering why19:49
sean-k-mooneyyou normally dont need to know that19:49
sean-k-mooneyyep exactly19:50
sean-k-mooneyfor os-vif for example if you dont add it it will use the latest version allowed by upper-constraints.txt instead of master19:50
sean-k-mooneywhich is normaly the correct behavior unless you are creating an os-vif job19:51
dansmithwe should just squash all the projects back into one tree obviously.. this would be so much easier ;)19:52
sean-k-mooneyhehe it works for the kernel but i like gerrit not exploding19:52
sean-k-mooneyalthough to be fair im not sure if would have an issue with that19:52
dansmithwell, I'm joking of course19:53
sean-k-mooneyi know that gitea would cry whoever19:53
dansmithbut sometimes ... it'd be easier19:53
sean-k-mooneysometimes yes19:53
sean-k-mooneyi know that neutron-lib can be a pain for the neutron folks in that regard19:53
dansmithaight, well, time to go do something else while that runs so I can see how close it is19:54
dansmiththanks again19:54
sean-k-mooneyits partly why tempest lib is now dead19:54
sean-k-mooneyno worries o/19:54
sean-k-mooneydansmith: you have a syntax error19:54
dansmithnoo19:55
sean-k-mooneyoh i see19:55
sean-k-mooneyyou commented out the definition of dsvm-irrelevant-files19:55
sean-k-mooneybut you are still using it on line 37119:55
dansmithI thought that was defined somewhere magic.. where did I nuke it?19:56
sean-k-mooneyactully that is not the issue its defined at the top of the file i think19:57
sean-k-mooneyya line 1619:57
dansmithokay I thought something like that19:57
sean-k-mooneythe & is the defintion and * is a deference or alis19:57
dansmithdid something fail or you just eye-parsed it and thought it was broken?19:57
sean-k-mooneyoh you nuked a different one19:58
sean-k-mooneypolicies-irrelevant-files19:58
sean-k-mooneyon line 39019:58
sean-k-mooneycommented19:59
dansmithoh that wasn't in yours I guess so I just block-removed19:59
sean-k-mooneyya just comment out the jobs in gate and you shoudl be fine19:59
sean-k-mooneyit would be nice if there was a cleaner way to do this but it is what it is20:00
openstackgerritDan Smith proposed openstack/nova master: DNM: Try to make a glance multistore job  https://review.opendev.org/73418420:00
dansmiththanks20:01
*** jawad_axd has joined #openstack-nova20:01
sean-k-mooneyno worries that has bit me in the past too20:01
*** nweinber has quit IRC20:02
sean-k-mooneylooking at https://zuul.openstack.org/status its queue so i think it passed syntax checking20:02
sean-k-mooneyzuul is really quick to comment abck if there is a sysntax error20:03
sean-k-mooneyalthough it was a bit delayed that time20:03
*** jawad_axd has quit IRC20:31
*** ccamacho has quit IRC20:39
*** maciejjozefczyk has quit IRC21:15
*** lvdombrkr has quit IRC21:17
*** raildo has quit IRC21:18
sean-k-mooneydansmith: looks like your job jsut started running https://zuul.openstack.org/stream/98f29ccd122c4ff080aca6cd71be7d52?logfile=console.log so it will be a while but i proably will work at this point assuming the glance config options were correct21:19
dansmithsweet21:22
*** slaweq has quit IRC21:45
*** slaweq has joined #openstack-nova21:47
*** gyee has joined #openstack-nova22:07
*** efried has quit IRC22:07
*** threestrands has joined #openstack-nova22:15
*** threestrands has quit IRC22:16
*** threestrands has joined #openstack-nova22:17
*** rcernin has joined #openstack-nova22:17
*** threestrands has quit IRC22:18
*** threestrands has joined #openstack-nova22:18
*** threestrands has quit IRC22:19
*** threestrands has joined #openstack-nova22:20
*** threestrands has quit IRC22:21
*** threestrands has joined #openstack-nova22:21
*** threestrands has quit IRC22:22
*** threestrands has joined #openstack-nova22:23
*** threestrands has quit IRC22:24
*** threestrands has joined #openstack-nova22:24
*** threestrands has quit IRC22:25
*** threestrands has joined #openstack-nova22:26
*** threestrands has quit IRC22:27
*** threestrands has joined #openstack-nova22:27
*** threestrands has quit IRC22:28
*** threestrands has joined #openstack-nova22:29
*** threestrands has quit IRC22:30
*** threestrands has joined #openstack-nova22:30
*** tkajinam has joined #openstack-nova22:49
*** slaweq has quit IRC23:11
*** tosky has quit IRC23:13
*** slaweq has joined #openstack-nova23:17
*** mlavalle has quit IRC23:29
*** eharney has quit IRC23:43

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!