Earlier  
Posted Nick Remark
#openstack-nova - 2018-11-15
13:45:55 coreycb mriedem: placement's been split from nova in stein, is that right?
13:46:19 coreycb mriedem: if so, yes we can do that in the stein cycle
13:47:14 cdent coreycb: yeah, that's right
13:47:45 cdent coreycb: people are hoping to have packages sooner, if possible, to make testing with ansible+packages more possible
13:47:50 cdent not a blocker though
13:48:22 coreycb cdent: mriedem: ok thanks for the nudge. we'll try to get it done soon. feel free to keep nudging if we don't get to it soon.
13:50:01 cdent awesome, thanks
14:00:47 openstackgerrit Jack Ding proposed openstack/nova master: Improve libvirt image and snapshot handling https://review.openstack.org/616692
14:01:02 aspiers[m] mriedem: got any time for a 5 min sync up on SEV in the next few hours?
14:02:42 mriedem aspiers[m]: i've got about 10 minutes right now outside of M-4 on level 3
14:02:48 mriedem or inside
14:03:13 openstackgerrit Ghanshyam Mann proposed openstack/nova stable/rocky: DNM: Testing tempest-full keep testing on xenial on stable brnach https://review.openstack.org/618176
14:03:20 aspiers[m] I think I'm next door
14:03:58 aspiers[m] I'll nip out of this Vitrage session
14:18:45 openstackgerrit Ghanshyam Mann proposed openstack/nova stable/pike: DNM: Testing tempest-full keep testing on xenial on stable brnach https://review.openstack.org/618181
15:09:18 openstackgerrit Emmet Hikory proposed openstack/nova master: Execute Nova functional jobs on arm64 https://review.openstack.org/618191
15:25:44 openstackgerrit Chris Dent proposed openstack/nova master: Use external placement in functional tests https://review.openstack.org/617941
15:26:08 cdent gibi, mriedem, dansmith ^ that's fairly real now.
15:31:48 sean-k-mooney cdent: so this remvoes all the placement api unit and fucntional tests
15:32:07 sean-k-mooney cdent: but i dont se the removal of the placement api code
15:32:24 sean-k-mooney was that intentional
15:33:23 cdent sean-k-mooney: yes, I left the placement code in place so that devstack/tempest will still work without needing to depends on yet more stuff, but leaving the tests in would have meant many changes in those files to keep them working in the face of the "use the external placement fixtures" changes
15:33:35 cdent I explain that in the commit message, but probably not well enough
15:33:58 sean-k-mooney i have not read it yet :)
15:34:05 sean-k-mooney i just was looking at the file list
15:36:07 sean-k-mooney i dont know why i would feel more comfortable with haveing a second patch that jsut deleted the nova/api/openstack/placement folder after this but i would
15:36:33 cdent sean-k-mooney: happy to do that
15:43:37 gibi cdent: great work! I left one comment inline
15:43:53 cdent thanks gibi
15:47:47 cdent gibi: I left a response, summary is: not quite sure what to do
15:49:47 gibi cdent: ohh so tox-siblings is the problem there. I did not get that earlier when you mentioned a similar thing
15:51:09 cdent well, I tried it using git and one of the test runs in the gate did not work, and when I switched it back to package, it did
15:51:28 cdent but I _may_ have misinterpreted the problem, lemme look at the logs
15:51:56 gibi cdent: I think it worth to ask the zuul devs
15:52:11 cdent If you see one :)
15:52:18 gibi :)
16:00:29 sean-k-mooney gibi: cdent im not sure if -e git+https://github.com/openstack/placement.git@master#egg=placement would work to install from master
16:00:54 sean-k-mooney i guess i can try locally
16:01:37 cdent sean-k-mooney: it does, both gibi and i have done things like that (from git.o.o) but it breaks the tox-siblings concept
16:02:09 cdent just talked to fungi about it
16:02:19 cdent brb
16:16:22 openstackgerrit Chris Dent proposed openstack/nova master: DNM: confirm deny tox siblings behavior https://review.openstack.org/618206
16:32:18 cfriesen maybe a weird question, but does anyone know which virtual device is mapped to the noVNC console?
16:37:18 sean-k-mooney cfriesen: i belive there is a vitrual display device in the vm that it connects to
16:38:22 sean-k-mooney im useing spice in this vm but for vnc the type just chagnes
16:38:24 sean-k-mooney <graphics type='spice' port='5900' autoport='yes' listen='127.0.0.1'>
16:38:26 sean-k-mooney <listen type='address' address='127.0.0.1'/>
16:38:28 sean-k-mooney <image compression='off'/>
16:38:30 sean-k-mooney </graphics>
16:39:24 sean-k-mooney i assume qemu just maps that graphic device to the first video output e.g. <video>
16:39:26 sean-k-mooney <model type='qxl' ram='65536' vram='65536' vgamem='16384' heads='1' primary='yes'/>
16:39:28 sean-k-mooney <alias name='video0'/>
16:39:30 sean-k-mooney <address type='pci' domain='0x0000' bus='0x00' slot='0x01' function='0x0'/>
16:39:32 sean-k-mooney </video>
16:39:35 cfriesen sean-k-mooney: based on some experimenting, with my current guest it looks like /dev/tty and /dev/tty0 go to the VNC console, and /dev/console and /dev/ttyS0 go to the console.log file.
16:39:46 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Delete the placement code https://review.openstack.org/618215
16:40:07 sean-k-mooney cfriesen: that would kind of make sense
16:42:31 sean-k-mooney i honestly have never really looked into it but i had assumed the first serial tty would go to the console log file
16:43:07 cfriesen yeah, normally it just works but someone was asking questions on how to set up a FreeBSD guest.
16:43:12 sean-k-mooney i would not be surpried if /dev/console was just a symlink to /dev/ttyS0
16:44:32 sean-k-mooney i rememebr a few years ago there was a short push to get openstack to install and run on netbsd or freebsd
16:44:57 sean-k-mooney i think that died out but it would have been nice to have it run there too
16:54:23 fungi cdent: sean-k-mooney: it would help to corner mordred to get some ideas, but in short tox-siblings performs multiple steps (including multiple tox invocations and other tasks in between) to work around what could be considered a missing feature in tox. maybe we could somehow reinterpret git protocol requirement entries as their egg name equivalents, but short of that i don't know how to go about
16:54:25 fungi making a single tox invocation on a dev's machine do the same thing tox-siblings does
16:56:15 sean-k-mooney fungi: ya im not that familar with tox-siblings so i cant really comment but the feeling i got form cdent was it would be non trivial
16:57:44 fungi because, otherwise, you know, that's what we would have just done rather than making the convoluted tox-siblings implementation
16:57:52 sean-k-mooney fungi: do you know of any easy way bar using the pip inside the venv created by tox to install a local copy of placemet fo dev to do this?
16:59:04 fungi writing a wrapper script to do the same, i guess
16:59:23 fungi (not called from tox, but a wrapper which calls tox and also runs those other commands)
16:59:34 fungi or working to get a feature for this implemented in tox itself
16:59:34 sean-k-mooney most developers should be using the released package anyway so its only going to affect people working on nova integrating placemetn
17:00:53 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Delete the placement code https://review.openstack.org/618215
17:02:35 sean-k-mooney fungi: its kind of an edgecase so im not sure if tox would accepet it as a feature
17:03:23 sean-k-mooney unless we coudl have like a requriemets-overrides.txt file that if pressent would be used to change the deps
17:04:10 sean-k-mooney so at the end of all our deps install the overrides form requriemets-overrides.txt if it existis and gitignore the file by default so devs can just add the deps they want to redirect locally
17:16:28 sean-k-mooney bauzas: FYI i have pushed the commit to do the 1.12.1 release of os-vif https://review.openstack.org/#/c/618239/ however we still need to merge https://review.openstack.org/#/c/612534 and its parent change before we can do the release. it would be nice to get this done before the end of the month if not sooner
18:32:54 openstackgerrit zhouxinyong proposed openstack/osc-placement master: Replacing the link in HACKING.rst https://review.openstack.org/618270
19:27:19 openstackgerrit zhouxinyong proposed openstack/python-ganttclient master: Optimizing the safety of the http link site in HACKING.rst https://review.openstack.org/618282
22:17:53 openstackgerrit Vlad Gusev proposed openstack/nova master: [WIP] libvirt: Add a default Virtio RNG device to guests https://review.openstack.org/616603
23:14:52 openstackgerrit Vlad Gusev proposed openstack/nova master: [WIP] libvirt: Add a default Virtio RNG device to guests https://review.openstack.org/616603
23:29:46 openstackgerrit Vlad Gusev proposed openstack/nova master: [WIP] libvirt: Add a default Virtio RNG device to guests https://review.openstack.org/616603
23:37:03 openstackgerrit Vlad Gusev proposed openstack/nova master: [WIP] libvirt: Add a default Virtio RNG device to guests https://review.openstack.org/616603
#openstack-nova - 2018-11-16
00:02:12 fungi sean-k-mooney: i don't know that it's any more of an edge case than some of the other features in tox ;)
00:02:18 fungi but sure, worth a discussion
01:28:07 openstackgerrit Vlad Gusev proposed openstack/nova master: [WIP] libvirt: Add a default Virtio RNG device to guests https://review.openstack.org/616603
01:33:57 openstackgerrit Vlad Gusev proposed openstack/nova master: [WIP] libvirt: Add a default Virtio RNG device to guests https://review.openstack.org/616603
03:47:41 openstackgerrit zhouxinyong proposed openstack/osc-placement master: Replacing the link in HACKING.rst https://review.openstack.org/618428
04:09:08 openstackgerrit zhouxinyong proposed openstack/python-ganttclient master: Optimizing the safety of the http link site in HACKING.rst https://review.openstack.org/618444
07:40:28 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP libvirt: Always cleanup the instance dir when using block image backends https://review.openstack.org/618478
09:11:20 openstackgerrit Balazs Gibizer proposed openstack/nova master: DNM: another try to use tox-siblings https://review.openstack.org/618490
09:28:00 openstackgerrit zhufl proposed openstack/nova master: Add missing ws seperator between words https://review.openstack.org/618491
11:33:29 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Delete the placement code https://review.openstack.org/618215
14:37:51 openstack Launchpad bug 1803717 in OpenStack Compute (nova) "Instance snapshot fails with rbd backend" [Undecided,New] - Assigned to Dr. Jens Harbott (j-harbott)
14:37:51 frickler melwitt: stephenfin: seems there was a regression that went unnoticed due to the ceph job being broken, see https://bugs.launchpad.net/nova/+bug/1803717 . if my suspicion is correct, the fix should be easy, though, working on it now
14:54:27 openstackgerrit Jens Harbott (frickler) proposed openstack/nova master: Fix regression in glance client call https://review.openstack.org/618534
14:56:15 frickler ^^ I merged the fix to devstack-plugin-ceph so this should verify things now. might be worth adding a unit test, if someone feels so inclined
14:57:47 frickler it also might be worthwhile considering to make that job voting
14:58:29 sean-k-mooney frickler: does teh final arg correspond to image metadata?
14:58:43 sean-k-mooney i am currently tyring to find where add_location is defiend
15:00:13 sean-k-mooney there is similar code in horizon http://git.openstack.org/cgit/openstack/horizon/tree/openstack_dashboard/api/glance.py#n542

Earlier   Later