| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-11-15 | |||
| 13:36:15 | sean-k-mooney | am im just going to grab lunch so brb | |
| 13:36:30 | coreycb | sean-k-mooney: thanks for pushing on this | |
| 13:41:26 | mriedem | coreycb: hey can we get some extracted placement debs? https://etherpad.openstack.org/p/BER-placement-extract | |
| 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 | sean-k-mooney | most developers should be using the released package anyway so its only going to affect people working on nova integrating placemetn | |
| 16:59:34 | fungi | or working to get a feature for this implemented in tox itself | |
| 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 | 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: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: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 | |