*** gyee has quit IRC | 00:04 | |
*** slaweq has joined #openstack-nova | 00:11 | |
*** slaweq has quit IRC | 00:16 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (15) https://review.openstack.org/576031 | 00:34 |
---|---|---|
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (16) https://review.openstack.org/576299 | 00:34 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (17) https://review.openstack.org/576344 | 00:34 |
*** markvoelker has quit IRC | 00:44 | |
*** hoangcx has joined #openstack-nova | 01:00 | |
*** brinzhang has joined #openstack-nova | 01:06 | |
naichuans_ | bauzas: got it, thanks. Please ping me when you online. | 01:20 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (18) https://review.openstack.org/576673 | 01:25 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (19) https://review.openstack.org/576676 | 01:26 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (20) https://review.openstack.org/576689 | 01:26 |
openstackgerrit | fupingxie proposed openstack/nova master: Delete allocations for instances that is not in this node https://review.openstack.org/582899 | 01:32 |
*** mrsoul has quit IRC | 01:35 | |
*** mschuppert has quit IRC | 01:37 | |
openstackgerrit | fupingxie proposed openstack/nova master: Delete allocations for instances that is not in this node https://review.openstack.org/582899 | 01:37 |
*** harlowja has quit IRC | 01:46 | |
*** bhagyashris has joined #openstack-nova | 01:48 | |
*** mriedem_away is now known as mriedem | 01:55 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (21) https://review.openstack.org/576709 | 02:08 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (22) https://review.openstack.org/576712 | 02:09 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (6) https://review.openstack.org/571330 | 02:09 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7) https://review.openstack.org/571992 | 02:10 |
*** slaweq has joined #openstack-nova | 02:11 | |
*** slaweq has quit IRC | 02:16 | |
*** tetsuro has joined #openstack-nova | 02:26 | |
*** hongbin has joined #openstack-nova | 02:37 | |
*** mriedem has quit IRC | 02:38 | |
*** tetsuro has quit IRC | 02:38 | |
*** tetsuro has joined #openstack-nova | 02:38 | |
cburgess | mriedem Oh I'm still lurking about. | 02:56 |
*** lbragstad has quit IRC | 03:04 | |
*** bhagyashris has quit IRC | 03:05 | |
*** lbragstad has joined #openstack-nova | 03:11 | |
*** yingjun has joined #openstack-nova | 03:15 | |
*** hongbin has quit IRC | 03:25 | |
*** Dinesh_Bhor has joined #openstack-nova | 03:40 | |
*** erlon has joined #openstack-nova | 03:42 | |
*** erlon has quit IRC | 03:48 | |
*** lbragstad has quit IRC | 03:48 | |
*** udesale has joined #openstack-nova | 03:53 | |
*** sorrison_ has joined #openstack-nova | 03:58 | |
*** sorrison_ has quit IRC | 03:58 | |
*** tetsuro has quit IRC | 04:01 | |
*** Dinesh_Bhor has quit IRC | 04:02 | |
*** slaweq has joined #openstack-nova | 04:11 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_compute_api.py (4) https://review.openstack.org/568462 | 04:11 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform volume.usage notification https://review.openstack.org/580345 | 04:11 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform compute_task notifications https://review.openstack.org/482629 | 04:13 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153 | 04:14 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8) https://review.openstack.org/571993 | 04:14 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform libvirt.error notification https://review.openstack.org/484851 | 04:15 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289 | 04:15 |
*** yingjun has quit IRC | 04:15 | |
*** slaweq has quit IRC | 04:16 | |
*** tetsuro has joined #openstack-nova | 04:20 | |
*** cfriesen has quit IRC | 04:27 | |
*** Dinesh_Bhor has joined #openstack-nova | 04:28 | |
openstackgerrit | Merged openstack/nova master: Fail heal_allocations if placement is borked https://review.openstack.org/597237 | 04:39 |
*** jaosorior has joined #openstack-nova | 04:54 | |
*** slaweq has joined #openstack-nova | 05:02 | |
*** slaweq has quit IRC | 05:06 | |
*** Dinesh_Bhor has quit IRC | 05:08 | |
*** ratailor has joined #openstack-nova | 05:23 | |
*** Dinesh_Bhor has joined #openstack-nova | 05:26 | |
*** belmoreira has joined #openstack-nova | 05:30 | |
*** Luzi has joined #openstack-nova | 05:35 | |
*** yingjun has joined #openstack-nova | 05:38 | |
*** holser_ has joined #openstack-nova | 05:45 | |
*** tetsuro has quit IRC | 05:49 | |
*** tetsuro has joined #openstack-nova | 05:50 | |
*** sorrison has joined #openstack-nova | 05:53 | |
*** tetsuro has quit IRC | 06:03 | |
*** tetsuro has joined #openstack-nova | 06:04 | |
*** psachin has joined #openstack-nova | 06:05 | |
*** janki has joined #openstack-nova | 06:08 | |
*** slaweq has joined #openstack-nova | 06:19 | |
*** sorrison has quit IRC | 06:27 | |
*** sorrison has joined #openstack-nova | 06:28 | |
*** sorrison has quit IRC | 06:32 | |
*** pvc has joined #openstack-nova | 06:41 | |
pvc | hi bauzas | 06:42 |
pvc | im the one who ask you a question about the vgpu | 06:42 |
*** dpawlik has quit IRC | 06:56 | |
*** dpawlik has joined #openstack-nova | 06:57 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/601047 | 07:02 |
pvc | can i ask sir bauzas | 07:02 |
bauzas | good morning novaers | 07:02 |
bauzas | pvc: hi | 07:02 |
pvc | hi sir after running lsmod | grep vfio | 07:02 |
pvc | it doesnt the show the nvidia_vgpu_vfio | 07:03 |
pvc | i already run the nvidia NVIDIA-Linux-x86_64-367.128-grid.run | 07:03 |
*** rcernin has quit IRC | 07:03 | |
bauzas | you don't need VFIO | 07:04 |
bauzas | that's only for passthrough | 07:04 |
pvc | also the mdev bus | 07:05 |
bauzas | the driver looks good tho | 07:05 |
bauzas | you don't see the bus ? | 07:05 |
bauzas | remember me which physical card you have ? | 07:05 |
pvc | Nvidia Tesla P100 | 07:05 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform libvirt.error notification https://review.openstack.org/484851 | 07:06 |
pvc | i enable mdev using modprove mdev | 07:06 |
pvc | then reboot the server wait | 07:06 |
*** Dinesh_Bhor has quit IRC | 07:09 | |
bauzas | pvc: you don't need to enable the mdev bus by modprobe | 07:16 |
pvc | what i need to do sir bauzas? | 07:16 |
bauzas | pvc: in theory (and what I did on my dev box), you just need to disable the 'nouveau' module, then install the nvidia driver | 07:17 |
bauzas | and then reboot and you should see the mdev structure | 07:17 |
pvc | i already disable my nouveau | 07:17 |
bauzas | pvc: do you still have 'nouveau' ? | 07:17 |
bauzas | sure? lsmod | grep nouveau ? | 07:17 |
pvc | [root@ops-gpgpu1 ~]# lsmod | grep nou [root@ops-gpgpu1 ~]# | 07:17 |
pvc | http://paste.openstack.org/show/730230/ | 07:18 |
bauzas | lemme check my own driver | 07:21 |
pvc | Verify that the libvirtd service is active and running. | 07:23 |
*** helenafm has joined #openstack-nova | 07:23 | |
pvc | my libvirtd is not running laso | 07:23 |
bauzas | wait a sec | 07:24 |
bauzas | I'm not very familiar with the nvidia packaging, but IIRC the "-grid" module is for the guests, not the host | 07:24 |
bauzas | my own driver is named "-kvm" | 07:24 |
bauzas | yup, that's correct, the "-grid" driver is for graphics https://docs.nvidia.com/grid/6.0/grid-vgpu-user-guide/index.html#installing-vgpu-drivers-linux | 07:26 |
pvc | i download the -grid on the 90 days trial site | 07:26 |
*** Dinesh_Bhor has joined #openstack-nova | 07:27 | |
bauzas | pvc: see https://docs.nvidia.com/grid/6.0/grid-vgpu-user-guide/index.html#install-vgpu-package-generic-linux-kvm for the install docs | 07:28 |
pvc | it doesnt have kvm :( NVIDIA-Linux-x86_64-367.128-grid.run | 07:29 |
pvc | i think i download the wrong driver | 07:30 |
openstackgerrit | zhufl proposed openstack/nova master: Fix some typos in nova api ref doc https://review.openstack.org/603306 | 07:30 |
pvc | ongoing download of version 384 | 07:32 |
pvc | i cant curl or wget it on my linux machine :( | 07:32 |
bauzas | ok, I have another meeting, but let me know async | 07:35 |
*** pvc_ has joined #openstack-nova | 07:36 | |
pvc_ | hi bauzas im downloaing again another driver | 07:36 |
*** pvc has quit IRC | 07:38 | |
*** priteau has joined #openstack-nova | 07:41 | |
*** maciejjozefczyk has joined #openstack-nova | 07:45 | |
*** yingjun has quit IRC | 07:48 | |
*** jpena|off is now known as jpena | 07:50 | |
*** pvc_ has quit IRC | 07:50 | |
openstackgerrit | Tao Li proposed openstack/python-novaclient master: Remove the unused instance-name https://review.openstack.org/602520 | 07:55 |
*** takashin has left #openstack-nova | 08:06 | |
naichuans_ | bauzas: please ping me when your are free. | 08:08 |
naichuans_ | bauzas: About n-rp and reshaper, which one will be merged first? | 08:08 |
*** pvc has joined #openstack-nova | 08:12 | |
*** belmoreira has quit IRC | 08:15 | |
*** brinzhang has quit IRC | 08:15 | |
*** brinzhang has joined #openstack-nova | 08:16 | |
*** luksky has joined #openstack-nova | 08:19 | |
*** yingjun has joined #openstack-nova | 08:19 | |
*** psachin has quit IRC | 08:22 | |
*** markvoelker has joined #openstack-nova | 08:36 | |
*** alexchadin has joined #openstack-nova | 08:41 | |
*** tetsuro has quit IRC | 08:41 | |
*** Dinesh_Bhor has quit IRC | 08:45 | |
*** cdent has joined #openstack-nova | 08:46 | |
*** derekh has joined #openstack-nova | 08:49 | |
*** Dinesh_Bhor has joined #openstack-nova | 08:53 | |
kashyap | johnthetubaguy: gibi: Morning, can we put this change out of its misery -- https://review.openstack.org/#/c/565517 ("libvirt: Drop MIN_LIBVIRT_PF_WITH_NO_VFS_CAP_VERSION") | 08:58 |
bauzas | naichuans_: hello :) | 08:59 |
kashyap | It's a dead-simple clean-up | 08:59 |
naichuans_ | bauzas: Hi~ | 08:59 |
*** lbragstad has joined #openstack-nova | 08:59 | |
naichuans_ | Anything share with me about n-rp disscussion during PTG meeting? | 09:00 |
bauzas | naichuans_: sure, we could be discussing by Bluejeans | 09:00 |
bauzas | or Hangout ? | 09:00 |
*** Dinesh_Bhor has quit IRC | 09:01 | |
naichuans_ | haha~ | 09:01 |
pvc | hi | 09:01 |
naichuans_ | bauzas: In truth I'm interested in the name mangling and reshaper | 09:02 |
naichuans_ | bauzas: for multiple vgpu type support | 09:02 |
pvc | anyone experience in using cyborg for vgpu? | 09:03 |
bauzas | naichuans_: that's why I think we could be discussing by Hangout directly | 09:05 |
*** belmoreira has joined #openstack-nova | 09:05 | |
naichuans_ | bauzas: Ah~ but how I reach you? | 09:05 |
naichuans_ | bauzas: Maybe I could give you a call? | 09:06 |
bauzas | naichuans_: so, we agreed during the PTG about names, but the problem is that when discussing with dansmith, we thought about modifying the children and rather have children RP that would be physical GPUs and not GPU types | 09:06 |
bauzas | hence mriedem's -W on https://review.openstack.org/#/c/599208/ | 09:06 |
naichuans_ | bauzas: got it. Then there maybe multiple vgpu rps with same type? | 09:08 |
naichuans_ | for they may have more than one gpu with same type on a host? | 09:08 |
*** belmoreira has quit IRC | 09:10 | |
bauzas | naichuans_: the problem is about NUMA affinity | 09:10 |
*** Dinesh_Bhor has joined #openstack-nova | 09:10 | |
bauzas | naichuans_: see https://review.openstack.org/#/c/552924/ | 09:11 |
*** markvoelker has quit IRC | 09:11 | |
naichuans_ | bauzas: checking | 09:11 |
bauzas | naichuans_: if we have NUMA children, then pGPUs should be grand-children | 09:11 |
*** belmoreira has joined #openstack-nova | 09:11 | |
bauzas | for knowing the type, it'd then be a trait | 09:11 |
bauzas | and tbh, I think it's better | 09:11 |
naichuans_ | Oh, understood, bauzas: | 09:12 |
bauzas | because resource capacity is about some resource provider, which is a pGPU | 09:12 |
bauzas | the type itself isn't a provider of resources, just a feature | 09:13 |
bauzas | naichuans_: so I'm working on providing a new revision of https://review.openstack.org/#/c/599208/ by creating children being pGPUs | 09:13 |
naichuans_ | bauzas: Agree, it is more natural. But I'm not sure if there would be some issues involved in. | 09:13 |
naichuans_ | bauzas: I made a patch on xen according to Matt's patch:https://review.openstack.org/#/c/599208/. reshape works well | 09:17 |
naichuans_ | bauzas: https://review.openstack.org/#/c/521041/ | 09:18 |
naichuans_ | bauzas: but only for single gpu rp support | 09:18 |
*** idlemind has quit IRC | 09:18 | |
naichuans_ | bauzas: Do we have a plane for reshaper on multiple gpu rp support? | 09:19 |
openstackgerrit | Chen proposed openstack/nova stable/queens: Fix soft deleting vm fails after "nova resize" vm https://review.openstack.org/603340 | 09:21 |
naichuans_ | bauzas: about gpu rp name mangling, it make no difference on xenserver, because gpu with same type would be put into the same gpu group. So base on gpu or gpu type is same | 09:22 |
bauzas | naichuans_: I'll look at your change for xen | 09:22 |
naichuans_ | bauzas: thanks. | 09:22 |
bauzas | naichuans_: we don't need to reshape for multiple GPU types | 09:23 |
bauzas | again, see my upgrade impact section on the spec | 09:23 |
naichuans_ | bauzas: will check it | 09:24 |
*** alexchadin has quit IRC | 09:26 | |
naichuans_ | bauzas: you mean this patch: https://review.openstack.org/#/c/552924/? | 09:26 |
bauzas | nope, the reproposed 'multiple GPU types' one | 09:27 |
naichuans_ | bauzas: will check it | 09:28 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge extended_volumes extension response into server view builder https://review.openstack.org/596285 | 09:29 |
*** tetsuro has joined #openstack-nova | 09:30 | |
naichuans_ | bauzas: We restrict the user scenario to avoid those troubles~ Fair enough :) | 09:33 |
bauzas | naichuans_: which troubles ? :) | 09:33 |
*** alexchadin has joined #openstack-nova | 09:34 | |
openstackgerrit | Wenran Xiao proposed openstack/nova-specs master: Move juno approved specs https://review.openstack.org/603344 | 09:34 |
naichuans_ | bauzas: Directly upgrade the old vgpu allocation to multiple vgpu rp support is troublesome~ | 09:35 |
naichuans_ | bauzas: As I mentioned in the email, we have difficluties to decide the old allocation belongs. | 09:35 |
*** dtantsur|afk is now known as dtantsur | 09:36 | |
bauzas | naichuans_: ah this | 09:36 |
bauzas | naichuans_: yeah | 09:36 |
bauzas | naichuans_: well, about finding which is which, I don't see it difficult | 09:36 |
*** sorrison has joined #openstack-nova | 09:36 | |
bauzas | but I could be wrong | 09:37 |
bauzas | I mean, allocations are done with a consumer UUID | 09:37 |
bauzas | we know that the consumer UUID is either an instance or a migration | 09:37 |
bauzas | given we don't support migrations yet in libvirt, that simplifies | 09:37 |
bauzas | so, the idea is to lookup the VGPU allocations, find for each which instance it is, check the libvirt XML to know which card is attached to, and give it back | 09:38 |
naichuans_ | bauzas: Yes, we need to talk with hypervisor, looks not so natural for me~ | 09:38 |
bauzas | naichuans_: I already do that for rebooting the host | 09:39 |
bauzas | naichuans_: given mdevs aren't persisted on reboots, every compute restart checks the existing allocations and recreates the mdevs | 09:39 |
naichuans_ | bauzas: awesome~ | 09:40 |
*** hoangcx has quit IRC | 09:40 | |
*** yingjun has quit IRC | 09:40 | |
bauzas | naichuans_: I need to go to the gym, but I can help you by showing you the libvirt codee | 09:40 |
bauzas | give me a sec | 09:41 |
naichuans_ | bauzas: Got it, thank you very much~ And for so much information :) | 09:41 |
bauzas | naichuans_: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L566 | 09:42 |
naichuans_ | bauzas: will check it | 09:43 |
*** sambetts_ is now known as sambetts | 09:44 | |
naichuans_ | bauzas: According to your spec, multiple vgpu support would not be implemented in stein, right? | 09:49 |
naichuans_ | https://review.openstack.org/#/c/602474/2/specs/stein/approved/vgpu-stein.rst | 09:50 |
kashyap | tox -epep8 -- -HEAD | 09:52 |
kashyap | Oeps | 09:52 |
openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_PARALLELS_SET_ADMIN_PASSWD https://review.openstack.org/562313 | 09:54 |
*** hoangcx has joined #openstack-nova | 09:56 | |
pvc | hi bauzas | 09:59 |
pvc | it is okay now :) | 09:59 |
pvc | i have 13 supported types for my one p100 | 09:59 |
pvc | do i need to do something more bauzas? | 10:04 |
*** sorrison has quit IRC | 10:05 | |
*** markvoelker has joined #openstack-nova | 10:07 | |
*** sorrison has joined #openstack-nova | 10:08 | |
*** Dinesh_Bhor has quit IRC | 10:10 | |
*** Dinesh_Bhor has joined #openstack-nova | 10:18 | |
*** sorrison has quit IRC | 10:21 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Spec: Support filtering by forbidden aggregate https://review.openstack.org/603352 | 10:21 |
openstackgerrit | Vlad Gusev proposed openstack/nova stable/queens: Remove mox in libvirt/test_driver.py (3) https://review.openstack.org/603353 | 10:32 |
*** alexchadin has quit IRC | 10:33 | |
*** vivsoni has quit IRC | 10:36 | |
stephenfin | gibi: Any chance I could get a copy of the commands you were using in the bandwidth-aware scheduling demo last week? | 10:38 |
*** vivsoni has joined #openstack-nova | 10:40 | |
*** markvoelker has quit IRC | 10:41 | |
openstackgerrit | Vlad Gusev proposed openstack/nova stable/queens: libvirt: Reduce calls to qemu-img during update_available_resource https://review.openstack.org/603358 | 10:45 |
*** udesale has quit IRC | 10:47 | |
*** tbachman has quit IRC | 10:48 | |
*** s10 has joined #openstack-nova | 10:48 | |
*** Dinesh_Bhor has quit IRC | 10:49 | |
*** litao__ has quit IRC | 10:51 | |
*** mhen has joined #openstack-nova | 10:54 | |
*** sorrison has joined #openstack-nova | 10:57 | |
Luzi | Hello, I'm new to nova. How do you propose patches or ideas? Do you use Launchpad or Storyboard? | 10:58 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Spec: Support filtering by forbidden aggregate https://review.openstack.org/603352 | 10:59 |
*** tetsuro has quit IRC | 11:00 | |
*** sorrison has quit IRC | 11:05 | |
s10 | Luzi: Hello. We should use gerrit for patches proposal, see https://docs.openstack.org/infra/manual/developers.html . AFAIK, bugs and RFE are proposed in Nova by Launchpad. | 11:05 |
Luzi | yeah, i work in a team, which implements security enhancements and we want to contribute to openstack | 11:07 |
*** alexchadin has joined #openstack-nova | 11:07 | |
Luzi | for now we want to propose a signature check after an image is cached on a compute node | 11:07 |
*** sorrison has joined #openstack-nova | 11:07 | |
*** alexchadin has quit IRC | 11:11 | |
gibi | stephenfin: hi | 11:11 |
gibi | stephenfin: let me talk to rubasov if he have an easy to digest version of the commands | 11:12 |
*** sorrison has quit IRC | 11:12 | |
gibi | stephenfin: hi did promise to write up the whole thing as a blogpost but I don't know the timing of that | 11:13 |
s10 | Luzi: So basically you want to implement signature check that is described in https://docs.openstack.org/glance/latest/user/signature.html ? | 11:13 |
*** alexchadin has joined #openstack-nova | 11:14 | |
Luzi | s10, well nova already checks the signature when downloading an image from glance | 11:14 |
Luzi | but not when it was cached on the compute node | 11:14 |
*** artom has joined #openstack-nova | 11:15 | |
s10 | Luzi: I think we should wait for nova cores to wake up (15:00-16:00 UTC+0) for their thoughts. I believe good starting point is to open a bug in https://bugs.launchpad.net/nova describing this issue | 11:18 |
Luzi | okay, thank you s10 :) | 11:19 |
*** imacdonn has quit IRC | 11:19 | |
*** imacdonn has joined #openstack-nova | 11:19 | |
*** tssurya has joined #openstack-nova | 11:26 | |
*** sorrison has joined #openstack-nova | 11:29 | |
*** ratailor has quit IRC | 11:34 | |
*** ratailor has joined #openstack-nova | 11:34 | |
*** ratailor has quit IRC | 11:34 | |
*** sorrison has quit IRC | 11:36 | |
*** alexchadin has quit IRC | 11:36 | |
*** sorrison has joined #openstack-nova | 11:37 | |
*** jpena is now known as jpena|lunch | 11:38 | |
*** alexchadin has joined #openstack-nova | 11:40 | |
kashyap | 'Citrix XenServer CI' is a 'voting' job, is it? | 11:44 |
*** sorrison has quit IRC | 11:51 | |
sean-k-mooney | kashyap: i think so | 11:55 |
gibi | stephenfin: does it work for you if there will be a blogpost still this week? | 11:56 |
kashyap | sean-k-mooney: Trying to figure out if this change is failing because of it: https://review.openstack.org/#/c/602592/ ("Use 'virt' as the default machine type for ARMv7") | 11:56 |
sean-k-mooney | kashyap: well you are chaing the libvirt driver the cirtx xenserver ci should be using the native xen driver | 11:57 |
kashyap | sean-k-mooney: Also that 'neutron-grenade' failure is irrelevant | 11:57 |
kashyap | sean-k-mooney: Exactly | 11:57 |
sean-k-mooney | kashyap: the neutron-grenade job is voting hence the zuul -1 | 11:57 |
kashyap | sean-k-mooney: That Grenade thing was failing due to the presence of untracked working tree files: http://logs.openstack.org/92/602592/3/check/neutron-grenade/bfeb3e6/logs/devstack-gate-setup-workspace-old.txt | 11:58 |
kashyap | Yesterday mriedem was saying he saw it, but no clue why we're hitting it | 11:58 |
kashyap | Who can I bother about "Grenade issues"? | 11:58 |
sean-k-mooney | kashyap: infra but i would do a recheck first | 11:58 |
kashyap | A 'recheck' today didn't fix it; and weirdly enough, it didn't result in fresh logs. | 11:58 |
kashyap | (As in, it's still posted yesterday's 'neutron-grenade' logs) | 11:59 |
sean-k-mooney | has it actully run again? | 11:59 |
johnthetubaguy | Luzi: curious about the image signature issue, Is this because you are worried about on disk changes happening after the image gets in the cache? | 11:59 |
sean-k-mooney | kashyap: zuul has not commented back since the recheck | 11:59 |
kashyap | sean-k-mooney: I ran at 11:33 AM ("Gerrit Time"). And it's showing '-1' isn't it? | 11:59 |
sean-k-mooney | that is from before the recheck | 12:00 |
sean-k-mooney | kashyap: your job is still running | 12:00 |
Luzi | johnthetubaguy, yes the image could be replaced, that's because a recheck would be fine | 12:00 |
kashyap | sean-k-mooney: Hmm, in that case, I thought it'd "reset" the -1. | 12:00 |
johnthetubaguy | Luzi: if you didn't bump across it yet, this was the original design spec: http://specs.openstack.org/openstack/nova-specs/specs/rocky/implemented/nova-validate-certificates.html | 12:01 |
kashyap | sean-k-mooney: You showed me a live console in the past, can you point me to that again? | 12:01 |
* kashyap checks the IRC logs, meanwhile | 12:01 | |
johnthetubaguy | Luzi: well it references the original bit at the bottom of there | 12:01 |
sean-k-mooney | kashyap: that is the last job that is running http://zuul.openstack.org/stream.html?uuid=e576a55aa1d1464fbbecbdea0cd574d8&logfile=console.log | 12:01 |
sean-k-mooney | it just finished devstack and is tarting the tempest and will be starting the tempest run soon | 12:02 |
kashyap | sean-k-mooney: Most excellent. I _guessed_ so, but thought it shouldn't take 3+ hours. Apparently it does :-) | 12:02 |
sean-k-mooney | kashyap: if you go to http://zuul.openstack.org/ and filter by 602592 you will see the rest | 12:02 |
kashyap | sean-k-mooney: You're clairvoyant. That was my next question :-) To remind how I can figure it out next time | 12:03 |
kashyap | Thanks! | 12:03 |
*** markvoelker has joined #openstack-nova | 12:04 | |
johnthetubaguy | Luzi: this is an email thread that might be worth reading through, around when we used to do image verification: http://lists.openstack.org/pipermail/openstack-operators/2016-May/010554.html | 12:07 |
johnthetubaguy | Luzi: largely some basic checks we removed because once you have access to the image cache, its basically game over for a heap of reasons. Could well be something wrong in that analysis, but its worth a read of the thread for context | 12:09 |
Luzi | johnthetubaguy, thank you, i will read through | 12:09 |
*** priteau has quit IRC | 12:14 | |
*** priteau has joined #openstack-nova | 12:17 | |
*** dave-mccowan has joined #openstack-nova | 12:23 | |
openstackgerrit | Merged openstack/os-traits master: import zuul job settings from project-config https://review.openstack.org/601377 | 12:24 |
*** dave-mccowan has quit IRC | 12:28 | |
openstackgerrit | Merged openstack/os-traits master: switch documentation job to new PTI https://review.openstack.org/601378 | 12:33 |
openstackgerrit | Merged openstack/os-traits master: add python 3.6 unit test job https://review.openstack.org/601379 | 12:33 |
*** yingjun has joined #openstack-nova | 12:36 | |
*** jpena|lunch is now known as jpena | 12:38 | |
*** sorrison has joined #openstack-nova | 12:38 | |
*** dtantsur is now known as dtantsur|brb | 12:40 | |
*** sorrison has quit IRC | 12:43 | |
bauzas | naichuans_: nope, just the fact that we would only support multiple types once reshape is fully done | 12:44 |
bauzas | pvc: nope, you're all good, enjoy | 12:44 |
*** sorrison has joined #openstack-nova | 12:45 | |
pvc | thank you so much bauzas :) | 12:47 |
stephenfin | gibi: Blogpost this week is a-ok. A dump of the commands, assuming there wasn't anything confidential in them, is also a-ok :) | 12:47 |
gibi | stephenfin: then everything is set :) rubasov will make a post and distribute it on the ML | 12:48 |
naichuans_ | bauzas: got it, thanks | 12:48 |
stephenfin | gibi: Sweet! | 12:48 |
stephenfin | gibi: Totally unrelated, fancy grabbing this patch at some point? Will unblock it's successor https://review.openstack.org/#/c/565472/ | 12:48 |
stephenfin | *its | 12:48 |
gibi | stephenfin: it was so unconfidental that you saw the demo earlier than anybody in the company ;) | 12:48 |
stephenfin | :D Touché | 12:49 |
gibi | stephenfin: ohh I +2d the successor earlier :) | 12:52 |
gibi | stephenfin: done | 12:53 |
*** plestang has joined #openstack-nova | 12:55 | |
*** sorrison has quit IRC | 13:00 | |
openstackgerrit | Christian Berendt proposed openstack/nova master: Add missing backticks in nova-manage docs https://review.openstack.org/603370 | 13:02 |
bauzas | stephenfin: looks like someone saw your presentation ^ | 13:03 |
*** tbachman has joined #openstack-nova | 13:04 | |
openstackgerrit | Michele Baldessari proposed openstack/nova master: Ignore ArgsAlreadyParsedError exceptions when registering logging options https://review.openstack.org/603372 | 13:05 |
*** efried_off is now known as efried | 13:05 | |
efried | ō/ | 13:05 |
*** mriedem has joined #openstack-nova | 13:07 | |
mriedem | brinzhang: hi | 13:07 |
brinzhang | mriedem: Hi | 13:07 |
mriedem | you had some questions? | 13:07 |
*** tbachman has quit IRC | 13:08 | |
bauzas | mriedem: FWIW, working on your change locally | 13:08 |
brinzhang | yes. | 13:08 |
brinzhang | 1. about this https://review.openstack.org/#/c/599276/23/api-guide/source/polling_changes-since_parameter.rst | 13:08 |
mriedem | yeah jay is right, git mv would be better to preserve the history | 13:09 |
brinzhang | But I think "polling_changes-since_and_changes-before_parameters.rst" already contains the contents of "polling_changes-since_parameter.rst", so I think this should be replaced by current doc. As mriedem said, I need to do a redirectmatch for old links. | 13:10 |
brinzhang | ignore the last sentence. ^^ | 13:11 |
brinzhang | I think this do a rediecetmatch for old links is better | 13:12 |
mriedem | why not just rename the file as jaypipes suggested to preserve the history? | 13:13 |
mriedem | git history i mean | 13:13 |
mriedem | you have to add a redirect link either way | 13:13 |
mriedem | then it's also easier, as a reviewer, to see what you changed about the document when adding the changes-before conent | 13:14 |
mriedem | *content | 13:14 |
brinzhang | I don't think I understand the meaning of jaypipes :-( | 13:15 |
*** tbachman has joined #openstack-nova | 13:16 | |
mriedem | just git mv the file to rename it | 13:16 |
mriedem | and then add the changes-before content | 13:16 |
mriedem | $ git mv api-guide/source/polling_changes-since_parameter.rst api-guide/source/polling_changes.rst | 13:16 |
*** sorrison has joined #openstack-nova | 13:18 | |
brinzhang | Ok, after do this, can I replace the contents of the polling_changes-since_and_changes-before_parameters.rst file directly with polling_changes.rst? | 13:18 |
brinzhang | Not using git mv yet :-( | 13:19 |
mriedem | polling_changes-since_and_changes-before_parameters.rst becomes polling_changes.rst - it's just a file rename | 13:19 |
mriedem | and then you can edit polling_changes.rst to add changes-before content | 13:19 |
brinzhang | OK. I know. | 13:20 |
*** alexchadin has quit IRC | 13:22 | |
stephenfin | bauzas: :D | 13:25 |
*** awaugama has joined #openstack-nova | 13:31 | |
openstackgerrit | Brin Zhang proposed openstack/nova master: Resource retrieving: add changes-before filter https://review.openstack.org/599276 | 13:31 |
*** yingjun has quit IRC | 13:32 | |
*** alexchadin has joined #openstack-nova | 13:32 | |
*** efried is now known as efried_afak | 13:32 | |
*** efried_afak is now known as efried_afk | 13:32 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: test resize reschedule up-calls https://review.openstack.org/603389 | 13:39 |
sean-k-mooney | looks like tetsuro has already written up the negitive memberof spec before i started https://review.openstack.org/#/c/603352/ :) | 13:43 |
*** yingjun has joined #openstack-nova | 13:43 | |
*** alexchadin has quit IRC | 13:45 | |
*** sorrison has quit IRC | 13:47 | |
*** efried_afk is now known as efried | 13:47 | |
gibi | sean-k-mooney: as far as I remember tetsuro grabbed that on the yesterday's scheduler meeting | 13:48 |
*** sorrison has joined #openstack-nova | 13:49 | |
*** erlon has joined #openstack-nova | 13:51 | |
openstackgerrit | Brin Zhang proposed openstack/nova master: Resource retrieving: add changes-before filter https://review.openstack.org/599276 | 13:52 |
*** lbragstad has quit IRC | 13:53 | |
s10 | Hello. I have a question about backport https://review.openstack.org/#/c/603358 . Clean cherry-pick to stable/queens is impossible because of the https://review.openstack.org/#/q/Ic853743573aa0b74d5d2c5b8b47252b875d5f7ef . Could I just cherry-pick this mox to mock change to stable/queens and build this backport on top of it or backported fix should use mox in tests? | 13:53 |
*** sorrison has quit IRC | 13:53 | |
*** lbragstad has joined #openstack-nova | 13:54 | |
*** alexchadin has joined #openstack-nova | 13:54 | |
*** awaugama has quit IRC | 13:55 | |
mriedem | brinzhang: that's still not a rename, | 13:55 |
mriedem | brinzhang: it's getting late for you - i can pull your change down and fix the rename | 13:56 |
brinzhang | mriedem: thank you. | 13:56 |
*** awaugama has joined #openstack-nova | 13:57 | |
mriedem | s10: so that queens backport is going to fail tests or you've already modified the cherry pick? | 13:59 |
dpawlik | hi, I :a | 13:59 |
mriedem | s10: because it looks like a clean backport | 13:59 |
*** brinzhang has quit IRC | 13:59 | |
s10 | mriedem: I built cherry-pick fix on top of the mock to mox cherry-pick | 13:59 |
mriedem | oh so you're asking if backporting https://review.openstack.org/#/c/603353/ is ok | 14:00 |
mriedem | yet another reason why mox removal is a pain in the ass.... | 14:00 |
mriedem | s10: personally i'd prefer that we don't backport these mox->mock changes since they are usually large and hairy, | 14:01 |
mriedem | and instead just modify the conflicting backports that you need and mention the conflict in the commit message | 14:01 |
mriedem | e.g. https://review.openstack.org/#/c/603358 commit message would mention the conflict because of https://review.openstack.org/#/q/Ic853743573aa0b74d5d2c5b8b47252b875d5f7ef | 14:01 |
mriedem | s10: left some comments on both of those backports | 14:02 |
mriedem | let me know if you need help | 14:03 |
sean-k-mooney | mriedem: isnt this yet another reasong to finish the mox->mock stuff this cycle | 14:04 |
dpawlik | Hi, I can't find where is a problem when I deploy fresh installation of Nova. I create DB's: nova, nova_api, nova_cell0, nova_cell1 and nova_placement. On controller I execute: db sync, api_db sync, then cell_v2 map_cell0 where I set --database_connection to "nova_cell0" database. Then db online_data_migrations and I create new cell1: nova-manage cell_v2 create_cell --name cell1 where I set --database_connection to nova_c | 14:04 |
dpawlik | ell1 DB and also I change transport_url to new vhost: cell1. After creation, I rerun DB, API_DB sync, then cell_v2 discover_hosts, but still I have one issue: nova service-list shows me duplicated services. Is it something related to Queens release or I make something bad? | 14:04 |
dansmith | dpawlik: duplicate listing is almost always because you have two cell mappings pointing at the same database | 14:05 |
dpawlik | hmm | 14:05 |
openstackgerrit | Vlad Gusev proposed openstack/nova stable/queens: libvirt: Reduce calls to qemu-img during update_available_resource https://review.openstack.org/603358 | 14:06 |
dpawlik | dansmith: But in nova.conf in section [database] I have connection to "nova_cell1" DB, for nova_api connection is to nova_api DB, | 14:06 |
*** mdrabe has joined #openstack-nova | 14:07 | |
dansmith | dpawlik: the [database]/connection doesn't get used for this, what matters is what is in the db for cell mappings | 14:07 |
dpawlik | I got a feeling that something is missing in cells doc | 14:07 |
bauzas | gibi: those rechecks aren't super happy when you're willing to review :) | 14:08 |
dansmith | dpawlik: or that you're just reading the docs and pasting commands that don't need to be run every time and thus you now have two mappings for the same DB :) | 14:09 |
*** dtantsur|brb is now known as dtantsur | 14:09 | |
dansmith | mriedem: are you able to still push a queens release? sounds like it's been a long time since we've had one | 14:09 |
dansmith | noticed because we're missing some juicy fixes from upstream in our branch that tracks upstream releases | 14:10 |
dpawlik | how can I check what mappings are done? | 14:10 |
s10 | mriedem: the problem with not backporting mox->mock change is that every other related fix will require resolving a conflict. For example, if I want consequent change https://review.openstack.org/#/c/589567/ be backported to Queens, this conflict resolve will be needed again... | 14:10 |
gibi | bauzas: you mean I should go and fix the unrelated gate issues instead? ;) | 14:10 |
dansmith | s10: this is why mox removal is a fool's errand, IMHO :) | 14:10 |
mriedem | dansmith: i am, or anyone is really able to propose them, | 14:10 |
bauzas | gibi: nah, just that it's sad | 14:10 |
dpawlik | dansmith: how can I check what mappings are done? | 14:10 |
mriedem | i'm taking over stable release review duties while tony is out | 14:10 |
dansmith | dpawlik: nova-manage cell_v2 list_cells (or whatever) | 14:10 |
mriedem | sean-k-mooney: you're missing my point | 14:11 |
bauzas | dansmith: FWIW, I turned the problem multiple ways | 14:11 |
mriedem | it was the mox->mock removal in rocky that is causing backport problems in queens now | 14:11 |
bauzas | dansmith: about NUMA affinity | 14:11 |
bauzas | dansmith: and the fact we don't model yet pGPUs | 14:11 |
*** alexchadin has quit IRC | 14:11 | |
bauzas | dansmith: the fact is, we said at the PTG that NUMA topology could be a thing for Stein but affinity isn't | 14:11 |
bauzas | so I need to cut my spec in tow | 14:12 |
bauzas | two | 14:12 |
dansmith | bauzas: so we're going to reshape now and then reshape again? | 14:12 |
bauzas | dansmith: my point is, we could model pGPUs as resource providers, but it will need a reshape anyway once we have NUMA nodes | 14:12 |
*** alexchadin has joined #openstack-nova | 14:12 | |
sean-k-mooney | mriedem: yes i know its just the loger we have partial mox removal and the more releases we have this in the longer we will have mox backport issues | 14:12 |
dpawlik | dansmith: http://paste.openstack.org/show/730274/ | 14:13 |
bauzas | dansmith: yup, because if I begin to model NUMA nodes as children and pGPUs as grand children, placement wouldn't find all the resources or would constraint the query | 14:13 |
dansmith | bauzas: you don't have to do that, you can just make them children of the compute.. reparenting is easier than a full reshape | 14:13 |
bauzas | that's the whole purpose of the later spec I have to write for describing queries when it's all about chidren and grand-children relationship | 14:13 |
dansmith | but whatever | 14:13 |
mriedem | sean-k-mooney: we could have just, oh idk, never done it... | 14:14 |
sean-k-mooney | mriedem: we are going to have them either way so there is not much we can do unless nova is going to make mox a deliverable of the project and maintain it going forward | 14:14 |
mriedem | sean-k-mooney: which i believe dansmith said he'd do if it became a problem, | 14:14 |
mriedem | which it never has | 14:14 |
dansmith | dpawlik: looks like only one mapping in there, so I guess you're good | 14:14 |
bauzas | dansmith: yup, that's my thoughts, I'm just about to model pGPUs as children, then they would just be parented to a NUMA node in a reshape | 14:14 |
dansmith | mox3 has never broken, so yeah I signed up to fix breakages :) | 14:14 |
dansmith | bauzas: okay | 14:14 |
bauzas | just wanted to follow-up on the situation since you made a valid point :) | 14:15 |
sean-k-mooney | i guess going against a openstack wide goal did not seam to be a vailable option to me but ya if it was i guess we could have never started it | 14:15 |
mriedem | it was always a viable option | 14:16 |
dpawlik | dansmith: but what if I have in [database] coonection set cell0 DB so after creating cell1 they are duplicatd? | 14:16 |
mriedem | dansmith: re the queens release question, i think mel was holding up on some fixes for queens regressions but i don't have a list handy, so would just have to go through what's been proposed and see if there is anything critical we should target | 14:16 |
sean-k-mooney | mriedem: it was always a technical option i just didnt think it was an option to explcitly go against a openstack release goal from a governance point of view | 14:17 |
dansmith | mriedem: it's just been a really long time (three months it seems) | 14:17 |
mriedem | yeah i know | 14:17 |
sean-k-mooney | mriedem: anyway i guss that ship has sailed. | 14:17 |
dansmith | we're missing things like the compute disable -> weigher thing | 14:17 |
dansmith | mriedem: are you saying you think I shouldn't just propose one to unblock us because there may be things pending that need to be merged first? | 14:19 |
mriedem | dansmith: yes, i'd want to go through the open reviews first to see if there is anything we really need to get in | 14:19 |
dansmith | sigh | 14:19 |
dansmith | I don't really understand.. all the things proposed are likely things we want to get in.. is there some reason not to just cut one and then cut another? | 14:20 |
dansmith | kinda seems like we should just cut one any time the delta hits 10 or the delta is >1mo old or something.. not sure why it needs more thought than that | 14:21 |
dansmith | lyarwood says the delta is 55 patches right now which is.. a lot | 14:21 |
mriedem | i realize we haven't done stable releases in a while, it was a known thing, mel was holding up for something, i can't remember what atm, if i can get out from under the other 3 things people asked me to do before we started talking about this i can take a look | 14:22 |
mriedem | my butt is starting to hurt | 14:22 |
*** maciejjozefczyk has quit IRC | 14:24 | |
dansmith | sorry, I'm not trying to dump on you, I'm just saying... I'm not sure what the point of waiting is | 14:24 |
mriedem | the release team asked me to review the open stable release requests, so let me get through that first and then i'll take a look at our stable releases to come | 14:25 |
sean-k-mooney | dansmith: i mean to your point if there was something we were waiting for and we cut now without it its not hard to have another z stream in a week when we rememeber what it was | 14:25 |
mriedem | sean-k-mooney: yes i get it | 14:26 |
sean-k-mooney | mriedem: sorry i was not trying to take a dig at you. | 14:26 |
*** mlavalle has joined #openstack-nova | 14:27 | |
mriedem | it probably doesn't help that this rotated in just now https://www.youtube.com/watch?v=0XoyDqFy5pU | 14:27 |
sean-k-mooney | release automation is one of the tings that infra/release team desgined really well. its alot less of a pain then it was when you had to manually push and sign everything to pypi by hand | 14:28 |
* sean-k-mooney clicks | 14:28 | |
*** sorrison has joined #openstack-nova | 14:28 | |
sean-k-mooney | haha it certenly set a tone | 14:28 |
*** Luzi has quit IRC | 14:30 | |
mriedem | dansmith: you wanna wade through some of the +2d rocky backports while waiting https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/rocky ? | 14:32 |
dansmith | mriedem: you know I'm asking about queens right? | 14:32 |
mriedem | yes | 14:32 |
dansmith | looks like there is a lot on queens that needs just a +W | 14:32 |
dansmith | but yeah I can try to hit both | 14:32 |
dansmith | ah, maybe some are still pending on the rocky ones | 14:33 |
dansmith | like the service list thing | 14:33 |
lyarwood | yeah sorry that's my bad, I +2'd a few waiting on stable/rocky so I wouldn't block anything while I was offline yesterday | 14:37 |
dansmith | gdi lyarwood, stop being proactive! | 14:39 |
*** dpawlik has quit IRC | 14:40 | |
dansmith | okay I just sent a raft of rocky stuff to meet its maker | 14:41 |
openstackgerrit | Vlad Gusev proposed openstack/nova stable/queens: WIP libvirt: Reduce calls to qemu-img during update_available_resource https://review.openstack.org/603358 | 14:54 |
mriedem | btw, i think that's part of the ordering, we want to make sure stuff we release in queens is also released in rocky, so someone doesn't upgrade from queens to rocky in that window and lose some fixes - probably rare that would happen though | 14:55 |
mriedem | so we'll do a rocky->queens->pike->ocata release stack | 14:55 |
*** hamzy has quit IRC | 14:56 | |
dansmith | if we release by the clock, frequently enough, and keep the deps in gerrit, | 14:58 |
dansmith | then it should never really happen | 14:58 |
*** hamzy has joined #openstack-nova | 14:58 | |
dansmith | but if we have three-month windows, it's far more possible | 14:58 |
*** luksky has quit IRC | 14:59 | |
*** tbachman has quit IRC | 15:02 | |
*** s10 has quit IRC | 15:04 | |
*** sorrison has quit IRC | 15:06 | |
*** sorrison has joined #openstack-nova | 15:08 | |
*** dpawlik has joined #openstack-nova | 15:10 | |
*** yingjun has quit IRC | 15:11 | |
openstackgerrit | Chen proposed openstack/nova master: doc trivial: additional info to admin-password-injection https://review.openstack.org/603414 | 15:11 |
*** belmoreira has quit IRC | 15:15 | |
*** dpawlik has quit IRC | 15:15 | |
*** sorrison has quit IRC | 15:19 | |
openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Add 'disallow-isolcpus-with-overcommit-vcpus' spec https://review.openstack.org/603417 | 15:20 |
*** sorrison has joined #openstack-nova | 15:22 | |
*** MrWatson is now known as NostawRm | 15:26 | |
*** sorrison has quit IRC | 15:31 | |
openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Specify doctree directory https://review.openstack.org/603421 | 15:32 |
*** sorrison has joined #openstack-nova | 15:33 | |
*** afazekas has joined #openstack-nova | 15:34 | |
*** cfriesen has joined #openstack-nova | 15:41 | |
*** vivsoni has quit IRC | 15:43 | |
*** tbachman has joined #openstack-nova | 15:43 | |
*** luksky has joined #openstack-nova | 15:45 | |
*** sorrison has quit IRC | 15:45 | |
*** sorrison has joined #openstack-nova | 15:46 | |
*** sorrison has quit IRC | 15:55 | |
stephenfin | mriedem, bauzas, dansmith: Can one of you put this python3-first patch through, please? https://review.openstack.org/#/c/601373/ | 15:57 |
bauzas | stephenfin: easy review, thanks to your help | 15:58 |
stephenfin | bauzas: Cheers :) | 15:59 |
bauzas | stephenfin: no, ``Cheers`` | 15:59 |
bauzas | it's a string, dude | 15:59 |
stephenfin | Think I've found my presentation topic for Denver 2019 | 16:00 |
bauzas | :) | 16:00 |
*** hamzy has quit IRC | 16:01 | |
*** ralonsoh has joined #openstack-nova | 16:01 | |
stephenfin | bauzas: If you're game, there are also these. Think it's the last of them https://review.openstack.org/#/q/topic:python3-first+status:open+(project:openstack/os-traits+OR+project:openstack/os-vif) | 16:02 |
*** helenafm has quit IRC | 16:02 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: Merge extended_volumes extension response into server view builder https://review.openstack.org/596285 | 16:04 |
openstackgerrit | Merged openstack/nova master: Merge security groups extension response into server view builder https://review.openstack.org/585475 | 16:06 |
openstackgerrit | Merged openstack/nova master: Merge extended_status extension response into server view builder https://review.openstack.org/592092 | 16:06 |
openstackgerrit | Merged openstack/nova-specs master: import zuul job settings from project-config https://review.openstack.org/601373 | 16:12 |
*** holser_ has quit IRC | 16:23 | |
*** bnemec has quit IRC | 16:28 | |
*** dtantsur is now known as dtantsur|afk | 16:28 | |
*** bnemec has joined #openstack-nova | 16:28 | |
*** sorrison has joined #openstack-nova | 16:33 | |
cburgess | dansmith Apparently you made a few people thing something bad had happened to me. :) | 16:35 |
*** sambetts is now known as sambetts|afk | 16:35 | |
cburgess | I guess it nice to know people still care. | 16:35 |
cdent | you were missed | 16:36 |
cburgess | cdent I missed you all. Its weird not attending the summits and PTGs. I've been doing it since 2012 so yeah... makes the whole "I don't work with OpenStack anymore" thing real. | 16:37 |
*** gyee has joined #openstack-nova | 16:38 | |
*** sean-k-mooney has quit IRC | 16:41 | |
openstackgerrit | Vladyslav Drok proposed openstack/nova stable/pike: Fix resize_instance rpcapi call https://review.openstack.org/603439 | 16:42 |
*** sorrison has quit IRC | 16:42 | |
*** sean-k-mooney has joined #openstack-nova | 16:43 | |
*** sorrison has joined #openstack-nova | 16:46 | |
*** jpena is now known as jpena|off | 17:02 | |
*** derekh has quit IRC | 17:02 | |
*** sorrison has quit IRC | 17:07 | |
*** ralonsoh has quit IRC | 17:11 | |
*** jchhatbar has joined #openstack-nova | 17:14 | |
*** janki has quit IRC | 17:17 | |
*** dpawlik has joined #openstack-nova | 17:23 | |
*** dpawlik has quit IRC | 17:24 | |
*** dpawlik has joined #openstack-nova | 17:24 | |
*** alexchadin has quit IRC | 17:25 | |
*** sorrison has joined #openstack-nova | 17:33 | |
*** sorrison has quit IRC | 17:38 | |
mriedem | dansmith: would be good to get this into pike before we release https://review.openstack.org/#/c/594184/ | 17:38 |
*** sorrison has joined #openstack-nova | 17:39 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: Merge extended_volumes extension response into server view builder https://review.openstack.org/596285 | 17:39 |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: Making instance/migration listing skipping down cells configurable https://review.openstack.org/592428 | 17:39 |
*** jchhatbar has quit IRC | 17:40 | |
mriedem | we have a few pike backports with +2s actually https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/pike+label:Code-Review=2 | 17:41 |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: Add get_by_cell_and_project() method to InstanceMappingList https://review.openstack.org/591656 | 17:45 |
mhen | hello guys, I'm from the same team as Luzi and would like to pick up on her topic regarding the image signature checking again, hoping that some of the core members are present right now. | 17:46 |
*** sapd1_ has joined #openstack-nova | 17:47 | |
*** dpawlik has quit IRC | 17:48 | |
*** hemna_ has joined #openstack-nova | 17:48 | |
mhen | we created a launchpad entry as advised: https://bugs.launchpad.net/nova/+bug/1793159 | 17:49 |
openstack | Launchpad bug 1793159 in OpenStack Compute (nova) "no signature check for cached images" [Undecided,New] | 17:49 |
mriedem | dansmith: actually i'll just push a pike release since we have a lot of stuff unreleased in there http://paste.openstack.org/show/730308/ | 17:49 |
mhen | so we propose to do an additional image signature verification for signed images after retrieving it from a local compute's cache while spawning an instance. Any opinions on this? | 17:52 |
mriedem | mhen: i left a question in the bug | 17:52 |
mriedem | i believe this is also a duplicate of https://bugs.launchpad.net/nova/+bug/1785668 | 17:52 |
openstack | Launchpad bug 1785668 in OpenStack Compute (nova) "nova-compute doesn't check image signature if imagecache exists" [Undecided,New] | 17:52 |
mriedem | also, mdbooth is probably the person to talk with about image cache stuff, but he's not around right now | 17:53 |
mriedem | i'll subscribe him to the bugs | 17:53 |
mhen | mriedem, thank you for your input! You are right about the duplicate. Should I answer your question in our bug entry or is it gonna be closed as a duplicate now anyway? | 17:54 |
mriedem | mhen: might as well reply and then we'll duplicate it | 17:56 |
mhen | mriedem, roger that! | 17:57 |
mriedem | it wasn't clear to me that the other bug had sufficient recreate details | 17:57 |
mhen | the other bug actually follows a different reasoning | 17:59 |
*** sorrison has quit IRC | 17:59 | |
mhen | they are talking about the image signature check should fail in their scenario, since the user in question should not be able to retrieve the corresponding validation certificate from Barbican due to the user's permissions | 18:00 |
mhen | since a cached image is not checked in Nova, the cert retrieval is not even attempted and the permission problem does not surface | 18:01 |
mhen | our reasoning is more about the possibility of tampering with the cached image data, but I'll elaborate on that in our bug entry | 18:01 |
mriedem | dansmith: we should get this into ocata given it's a regression in ocata https://review.openstack.org/#/c/588077/ | 18:02 |
mriedem | that and the one below it. at this rate, we're going to have to start doing the old "stable core backport proxies as a +2" thing | 18:02 |
dansmith | mriedem: okay just got back, give me a few and I'll look | 18:08 |
*** sorrison has joined #openstack-nova | 18:11 | |
*** sorrison has quit IRC | 18:15 | |
*** sorrison has joined #openstack-nova | 18:17 | |
*** alexchadin has joined #openstack-nova | 18:20 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: Return a minimal construct for nova list when a cell is down https://review.openstack.org/567785 | 18:23 |
*** alexchadin has quit IRC | 18:24 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: Add scatter-gather-single-cell utility https://review.openstack.org/594947 | 18:26 |
*** priteau has quit IRC | 18:28 | |
mhen | mriedem, I replied to your question on https://bugs.launchpad.net/nova/+bug/1793159 - I hope this clarifies our reasoning a bit more | 18:30 |
openstack | Launchpad bug 1793159 in OpenStack Compute (nova) "no signature check for cached images" [Undecided,New] | 18:30 |
*** sorrison has quit IRC | 18:32 | |
*** alexchadin has joined #openstack-nova | 18:37 | |
dansmith | mriedem: were you implying I should slam this? https://review.openstack.org/#/c/588077/2 | 18:37 |
mriedem | dansmith: yes and the one below it | 18:38 |
dansmith | are we actually going to release any of this? :) | 18:39 |
*** alexchadin has quit IRC | 18:40 | |
*** alexchadin has joined #openstack-nova | 18:41 | |
*** luksky11 has joined #openstack-nova | 18:42 | |
*** alexchadin has quit IRC | 18:44 | |
*** luksky has quit IRC | 18:46 | |
*** sorrison has joined #openstack-nova | 18:48 | |
*** sorrison has quit IRC | 18:55 | |
*** sorrison has joined #openstack-nova | 18:55 | |
mriedem | oh for ocata... | 19:00 |
mriedem | good question | 19:00 |
mriedem | i guess that's something we need to sort out with the STABLE MAINT SIG!!! https://releases.openstack.org/ | 19:00 |
mriedem | ocata was supposed to go into extended maintenance on aug 27 | 19:01 |
*** sorrison has quit IRC | 19:02 | |
*** sorrison has joined #openstack-nova | 19:03 | |
*** sapd1_ has quit IRC | 19:03 | |
dansmith | mriedem: I actually meant on any of our branches, re: previous discussion :) | 19:03 |
*** hamzy has joined #openstack-nova | 19:04 | |
mriedem | i have release patches up for rocky and queens | 19:04 |
mriedem | WIPs while things merge | 19:04 |
mriedem | and one for pike stacked on top of that | 19:04 |
mriedem | https://review.openstack.org/#/q/topic:nova-stable-releases-sept-2018+(status:open+OR+status:merged) | 19:04 |
*** sorrison has quit IRC | 19:17 | |
*** sorrison has joined #openstack-nova | 19:18 | |
*** sorrison has quit IRC | 19:27 | |
*** priteau has joined #openstack-nova | 19:30 | |
*** alexchadin has joined #openstack-nova | 19:34 | |
*** alexchadin has quit IRC | 19:38 | |
*** awaugama has quit IRC | 19:45 | |
*** sorrison has joined #openstack-nova | 19:46 | |
*** oikiki has joined #openstack-nova | 19:48 | |
*** slowrie has joined #openstack-nova | 19:48 | |
*** dpawlik has joined #openstack-nova | 19:48 | |
*** slowrie has left #openstack-nova | 19:48 | |
*** dpawlik has quit IRC | 19:52 | |
*** alexchadin has joined #openstack-nova | 19:54 | |
*** sorrison has quit IRC | 19:59 | |
*** gouthamr has quit IRC | 20:00 | |
*** sorrison has joined #openstack-nova | 20:02 | |
*** david-lyle is now known as dklyle | 20:02 | |
*** gouthamr has joined #openstack-nova | 20:03 | |
*** alexchadin has quit IRC | 20:04 | |
*** s10 has joined #openstack-nova | 20:07 | |
*** sorrison has quit IRC | 20:08 | |
*** dave-mccowan has joined #openstack-nova | 20:08 | |
openstackgerrit | Merged openstack/nova master: libvirt: Use os.stat and os.path.getsize for RAW disk inspection https://review.openstack.org/589567 | 20:13 |
openstackgerrit | Merged openstack/nova master: Add functional test for live migrate with anti-affinity group https://review.openstack.org/588935 | 20:13 |
openstackgerrit | Merged openstack/nova master: Explicitly fail if trying to attach SR-IOV port https://review.openstack.org/591898 | 20:13 |
openstackgerrit | Vlad Gusev proposed openstack/nova stable/rocky: libvirt: Use os.stat and os.path.getsize for RAW disk inspection https://review.openstack.org/603488 | 20:15 |
*** erlon has quit IRC | 20:16 | |
*** NostawRm has quit IRC | 20:17 | |
openstackgerrit | Vlad Gusev proposed openstack/nova stable/queens: WIP libvirt: Reduce calls to qemu-img during update_available_resource https://review.openstack.org/603358 | 20:19 |
*** NostawRm has joined #openstack-nova | 20:20 | |
*** nicolasbock has joined #openstack-nova | 20:25 | |
*** sorrison has joined #openstack-nova | 20:25 | |
*** munimeha1 has joined #openstack-nova | 20:31 | |
*** med_ has joined #openstack-nova | 20:33 | |
*** med_ has quit IRC | 20:34 | |
*** sorrison has quit IRC | 20:39 | |
*** cdent has quit IRC | 20:39 | |
*** sorrison has joined #openstack-nova | 20:40 | |
*** efried has quit IRC | 20:44 | |
*** efried has joined #openstack-nova | 20:44 | |
*** sorrison has quit IRC | 20:45 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Merge extended_volumes extension response into server view builder https://review.openstack.org/596285 | 20:46 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Making instance/migration listing skipping down cells configurable https://review.openstack.org/592428 | 20:46 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add get_by_cell_and_project() method to InstanceMappingList https://review.openstack.org/591656 | 20:46 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Return a minimal construct for nova list when a cell is down https://review.openstack.org/567785 | 20:46 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add scatter-gather-single-cell utility https://review.openstack.org/594947 | 20:46 |
tssurya | mriedem: was just about to change the nits | 20:46 |
tssurya | thanks | 20:46 |
*** hamzy has quit IRC | 20:47 | |
mriedem | you're supposed to be sleeping or something | 20:47 |
tssurya | mriedem: hehe still living in denver TZ | 20:48 |
*** sorrison has joined #openstack-nova | 20:50 | |
*** jiapei has joined #openstack-nova | 20:54 | |
*** mrsoul has joined #openstack-nova | 20:56 | |
*** priteau has quit IRC | 20:57 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Merge extended_volumes extension response into server view builder https://review.openstack.org/596285 | 20:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Making instance/migration listing skipping down cells configurable https://review.openstack.org/592428 | 20:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add get_by_cell_and_project() method to InstanceMappingList https://review.openstack.org/591656 | 20:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Return a minimal construct for nova list when a cell is down https://review.openstack.org/567785 | 20:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add scatter-gather-single-cell utility https://review.openstack.org/594947 | 20:58 |
mriedem | tssurya: can we abandon https://review.openstack.org/#/c/594947/ ? i don't think anything is using it now | 20:58 |
tssurya | mriedem: originally it was created for https://review.openstack.org/#/c/591658/ | 20:59 |
tssurya | so let's keep it (see the comments from dan there) | 20:59 |
*** priteau has joined #openstack-nova | 20:59 | |
mriedem | oh, ok | 20:59 |
tssurya | I am currently updating the test cases for this one | 20:59 |
tssurya | it will come on top | 20:59 |
*** sorrison has quit IRC | 21:08 | |
*** sorrison has joined #openstack-nova | 21:09 | |
*** tssurya has quit IRC | 21:10 | |
*** sorrison has quit IRC | 21:13 | |
mriedem | dansmith: so if i want to kick the tires on your db migration script in a devstack env, i should be able to just create an empty placement db (but with schema) and then just create some instances and do the migration? | 21:15 |
dansmith | mriedem: if you have a devstack, easiest thing is to just create some instances, then create an empty placement db and then point it at the server with root/$pass | 21:16 |
dansmith | if not, you | 21:16 |
dansmith | will need a nova db with all the right tables, plus something in inventories | 21:16 |
mriedem | yeah i was thinking of creating a stable/rocky devstack, creating some instances so there is data (allocations), and then create a placement db and run the script | 21:17 |
mriedem | essentially what grenade is going to have to do | 21:18 |
dansmith | yup | 21:18 |
*** gouthamr_ has joined #openstack-nova | 21:18 | |
dansmith | run --mkconfig and then put root and the DATABASE_PASSWORD from your localrc in there | 21:19 |
dansmith | I should probably have mkconfig assume placement and nova_api as db names.. dunno why I didn't | 21:19 |
*** sorrison has joined #openstack-nova | 21:20 | |
dansmith | done | 21:26 |
*** gouthamr has quit IRC | 21:26 | |
*** sorrison has quit IRC | 21:28 | |
openstackgerrit | Ben Nemec proposed openstack/nova master: WIP: Migrade upgrade checks to oslo.upgradecheck https://review.openstack.org/603499 | 21:29 |
*** sorrison has joined #openstack-nova | 21:30 | |
openstackgerrit | Ben Nemec proposed openstack/nova master: WIP: Migrade upgrade checks to oslo.upgradecheck https://review.openstack.org/603499 | 21:34 |
mriedem | bnemec: fancy | 21:35 |
mriedem | is that library in the openstack/requirements repo yet? | 21:36 |
mriedem | nope https://github.com/openstack/requirements/blob/master/upper-constraints.txt | 21:36 |
bnemec | No, the repo just got created. | 21:36 |
bnemec | There is a release on pypi from when I reserved the name though. | 21:36 |
bnemec | I'm also wondering if we want to rename the UpgradeCheck* classes to just Code and Result for the sake of brevity. | 21:37 |
bnemec | When used from the lib they'll be upgradecheck.Code and upgradecheck.Result anyway. | 21:37 |
dansmith | gdi, mriedem already got my -1 | 21:37 |
bnemec | lol | 21:38 |
mriedem | i didn't -1 b/c i'm too nice | 21:39 |
mriedem | bnemec: that works for me - i named them that way in nova b/c we have the whole subcommand thing | 21:40 |
bnemec | dansmith: If you want, you could -1 for the flagrant pep8 violations. ;-) | 21:40 |
mriedem | so theoretically we could have nova-status foo bar | 21:40 |
mriedem | nova-status upgrade baz | 21:40 |
dansmith | nah, now I'm demotivated | 21:40 |
bnemec | mriedem: Yeah, makes sense. Since the library is specialized we should be able to use the more generic names. | 21:41 |
bnemec | I'll get that proposed and update the Nova patch. | 21:41 |
*** sorrison has quit IRC | 21:45 | |
*** sorrison has joined #openstack-nova | 21:47 | |
*** oikiki has quit IRC | 21:47 | |
*** sorrison has quit IRC | 21:55 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Resource retrieving: add changes-before filter https://review.openstack.org/599276 | 21:56 |
*** sorrison has joined #openstack-nova | 21:58 | |
*** sorrison has quit IRC | 22:03 | |
*** sorrison has joined #openstack-nova | 22:04 | |
mriedem | wah wah https://etherpad.openstack.org/p/nova-forum-stein | 22:05 |
mriedem | we have 8 days to submit forum sessions | 22:05 |
*** priteau has quit IRC | 22:05 | |
*** sorrison has quit IRC | 22:08 | |
*** sorrison has joined #openstack-nova | 22:10 | |
*** sorrison has quit IRC | 22:14 | |
*** sorrison has joined #openstack-nova | 22:16 | |
*** luksky11 has quit IRC | 22:16 | |
dansmith | this always confuses me, | 22:18 |
dansmith | but you mean non-presentation forum sessions I guess? | 22:19 |
dansmith | like projecty design summity topics | 22:19 |
dansmith | right? | 22:19 |
*** sorrison has quit IRC | 22:21 | |
*** sorrison has joined #openstack-nova | 22:22 | |
mriedem | i guess so yeah, | 22:23 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Resource retrieving: add changes-before filter https://review.openstack.org/599276 | 22:23 |
mriedem | although they changed to not use the old forum submission tool, | 22:23 |
mriedem | and instead use something that looks like the normal call for presentations thing for the big recorded rooms | 22:23 |
mriedem | https://www.openstack.org/summit/berlin-2018/call-for-presentations | 22:23 |
dansmith | yeah, which .. seems weird, since those are all picked already, AFAIK | 22:24 |
mriedem | right, and the one thing i did submit as a presentation has somehow carried over to the forum review i think, | 22:24 |
mriedem | which is not what i wanted | 22:24 |
dansmith | meaning, didn't get accepted as a presentation so they made it a forum session instead? | 22:25 |
mriedem | because i'm pretty sure you all don't want to hear me talking about nfv at the edge in fusionsphere during the forum... | 22:25 |
dansmith | I think jaypipes would | 22:25 |
dansmith | he <3's edge | 22:25 |
mriedem | it would be on jay's top 4 watch list | 22:25 |
mriedem | or whatever | 22:25 |
dansmith | Pipes' Picks | 22:25 |
*** slaweq has quit IRC | 22:26 | |
mriedem | so my thing wasn't accepted as a forum session, | 22:26 |
mriedem | but i don't want it to be a candidate i guess | 22:26 |
mriedem | asking for clarification | 22:30 |
mriedem | dansmith: maybe you want to do a session on oschown | 22:38 |
mriedem | s/want/should/ | 22:38 |
dansmith | I'd like someone to have ever tried it ever first | 22:38 |
dansmith | but I could lie | 22:38 |
mriedem | dude, it's already in production in starlingx | 22:39 |
dansmith | haha | 22:39 |
*** sorrison has quit IRC | 22:40 | |
mriedem | kablam http://logs.openstack.org/24/602124/4/experimental/grenade-postgresql/c27d517/ | 22:42 |
mriedem | passing | 22:42 |
dansmith | I don't know why, but.. congrats :) | 22:42 |
*** jaosorior_ has joined #openstack-nova | 22:42 | |
*** sorrison has joined #openstack-nova | 22:44 | |
*** tetsuro has joined #openstack-nova | 22:45 | |
*** jaosorior has quit IRC | 22:45 | |
*** rcernin has joined #openstack-nova | 22:47 | |
*** janki has joined #openstack-nova | 22:47 | |
*** dpawlik has joined #openstack-nova | 22:49 | |
openstackgerrit | Merged openstack/nova master: Revert "libvirt: add method to configure migration speed" https://review.openstack.org/590814 | 22:49 |
openstackgerrit | Merged openstack/nova master: Add missing backticks in nova-manage docs https://review.openstack.org/603370 | 22:49 |
*** dpawlik has quit IRC | 22:53 | |
*** sorrison has quit IRC | 22:54 | |
*** sorrison has joined #openstack-nova | 22:56 | |
*** munimeha1 has quit IRC | 22:57 | |
*** mriedem has quit IRC | 22:59 | |
*** sorrison has quit IRC | 23:00 | |
*** slaweq has joined #openstack-nova | 23:11 | |
*** slaweq has quit IRC | 23:16 | |
*** sorrison has joined #openstack-nova | 23:22 | |
*** s10 has quit IRC | 23:36 | |
*** mlavalle has quit IRC | 23:39 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!