| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-11-15 | |||
| 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 | |
| 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 | |
| 15:00:14 | frickler | sean-k-mooney: yes. https://docs.openstack.org/python-glanceclient/latest/reference/api/glanceclient.v2.images.html#glanceclient.v2.images.Controller | |
| 15:01:22 | sean-k-mooney | hum ok baded on that this looks correct good catch | |
| 15:18:00 | openstackgerrit | Jack Ding proposed openstack/nova-specs master: Flavor Extra Spec and Image Properties Validation https://review.openstack.org/618542 | |
| 16:52:34 | mdbooth | Can I get tempest logs from a job which didn't fail? | |
| 16:53:26 | sean-k-mooney | you should be able to | |
| 16:57:35 | mdbooth | sean-k-mooney: Thanks ;) Any idea how? | |
| 16:58:35 | mdbooth | http://logs.openstack.org/33/618533/1/check/tempest-slow/bb92d7b/ | |
| 16:58:43 | mdbooth | Looking for tempest logs for that job | |
| 16:59:16 | mdbooth | Ah, nm. It's on 'controller' | |
| 17:09:08 | munimeha1 | I have a nova question for team here, i have a VM with a disk and some nics, a volume is attached. the VM OS has some persistence rules. The VM is hard rebooted. Nova generates a new virsh xml with new PCI map with a additional PCI for SCSI and the net persistence rules are messed. | |
| 17:10:19 | munimeha1 | is there a way around the issue to tell nova to pick PCI after the last one used on last reboot | |
| 17:11:09 | munimeha1 | right now nova is picking PCIs by device groups first disks and then nics | |
| 17:29:47 | mdbooth | In the -slow CI job, I can see that lvmdriver-1 is defined as a type on both controller and compute1. Is there any way that tempest can query cinder to find out which hosts a type is present on? | |
| 21:16:37 | openstackgerrit | Patricia Domingues proposed openstack/nova master: load up the volume drivers by checking architecture https://review.openstack.org/541393 | |
| #openstack-nova - 2018-11-17 | |||
| 16:32:46 | openstackgerrit | Dr. Clemens Hardewig proposed openstack/nova master: Fixes instance resizing/migration when swap volume is on lvm https://review.openstack.org/618621 | |
| 19:21:33 | openstackgerrit | Dr. Clemens Hardewig proposed openstack/nova master: Fixes instance resizing/migration when swap volume is on lvm https://review.openstack.org/618621 | |
| 21:26:42 | openstackgerrit | Dr. Clemens Hardewig proposed openstack/nova master: Fixes instance resizing/migration when swap volume is on lvm https://review.openstack.org/618621 | |
| #openstack-nova - 2018-11-18 | |||
| 08:01:14 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Consider root id is None in the database case https://review.openstack.org/613305 | |
| 08:02:31 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Consider root id is None in the database case https://review.openstack.org/613305 | |
| 12:16:33 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Calculate port_id rp_uuid mapping for binding https://review.openstack.org/616239 | |
| 12:16:34 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Pass allocations and traits to neturonv2 api https://review.openstack.org/616240 | |