| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-03 | |||
| 17:12:23 | hrw | https://bugs.launchpad.net/nova/+bug/1740824 already exists... | |
| 17:12:26 | hrw | will add to commit | |
| 17:12:29 | openstack | Launchpad bug 1740824 in OpenStack Compute (nova) "Make UEFI as the default properties for AArch64" [Undecided,New] | |
| 17:13:38 | stephenfin | hrw: Cool. If you were feeling generous, a little more info in that bug would be appreciated. Why is it essential, etc. | |
| 17:13:47 | hrw | sure | |
| 17:15:13 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: Use UEFI as the default boot for AArch64 https://review.openstack.org/489951 | |
| 17:19:03 | hrw | stephenfin: added comment to the bug | |
| 17:19:22 | hrw | stephenfin: please read and tell me is it enough or should I expand it | |
| 17:19:27 | claudiub | stephenfin: sure, will do. and btw, there are plenty of OVS users on Windows. :) We've added OVS vif plugging support in nova some time ago. I think it was Mitaka. | |
| 17:19:47 | claudiub | or newton | |
| 17:19:51 | hrw | Mitaka... my favourite openstack release. | |
| 17:19:57 | hrw | due to uefi support ;) | |
| 17:20:00 | stephenfin | claudiub: Ah, so I wonder how come we're not using os-vif code for that? | |
| 17:20:11 | claudiub | stephenfin: we are | |
| 17:20:40 | claudiub | stephenfin: https://github.com/openstack/nova/blob/master/nova/virt/hyperv/vif.py#L88 | |
| 17:20:44 | stephenfin | claudiub: I'm confused. How does that work with the code that's there. Surely that's the whole point of that patch? | |
| 17:21:34 | stephenfin | Ahhhh... http://git.openstack.org/cgit/openstack/compute-hyperv/tree/compute_hyperv/nova/vif.py | |
| 17:26:27 | stephenfin | hrw: So to summarize, if you don't set that property, the instance doesn't boot? | |
| 17:26:50 | hrw | stephenfin: unless it is setup to boot kernel+initrd which no one does nowadays | |
| 17:27:44 | stephenfin | hrw: Cool. +2d | |
| 17:27:47 | melwitt | uh, what's happened to tempest job logs? can't filter by log level anymore | |
| 17:28:27 | hrw | stephenfin: cool! thanks | |
| 17:29:39 | hrw | now have to find someone else to take a look ;D | |
| 17:29:41 | claudiub | stephenfin: hm, I think the main point of the patch is to make os-vif a bit more OS agnostic. If you look in the vif_plug_ovs.ovs module, linux_net.device_exists is being used, even though that is also executed on Windows | |
| 17:30:04 | claudiub | and it also adds a Windows specific implementation for device_exists, which is nice. | |
| 17:31:39 | openstackgerrit | Claudiu Belu proposed openstack/nova master: hyper-v: Deprecates support for Windows / Hyper-V Server 2012 https://review.openstack.org/530953 | |
| 17:32:16 | hrw | stephenfin: can you look at https://review.openstack.org/#/c/530965/ one too? It is probably wrong way of doing it and I am open for suggestions | |
| 17:39:09 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Raise MarkerNotFound if BuildRequestList.get_by_filters doesn't find marker https://review.openstack.org/530980 | |
| 17:45:16 | ralonsoh_ | stephenfin: I don't know why it's failing | |
| 17:45:25 | ralonsoh_ | stephenfin: I can't reproduce the error | |
| 17:48:23 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Raise MarkerNotFound if BuildRequestList.get_by_filters doesn't find marker https://review.openstack.org/530981 | |
| 17:49:48 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/newton: Raise MarkerNotFound if BuildRequestList.get_by_filters doesn't find marker https://review.openstack.org/530982 | |
| 17:54:06 | mriedem | stephenfin: https://review.openstack.org/#/c/514309/ is fine except it has to be renamed nova-lvm i guess | |
| 17:59:16 | mriedem | mmedvede: are you powerkvm ci? | |
| 17:59:44 | mriedem | yeah; https://review.openstack.org/#/c/530982/ on stable/newton failed but +1ed the change | |
| 17:59:48 | mmedvede | mriedem: I maintain kvm on power CI, yes | |
| 17:59:58 | mriedem | should we just not run pkvm ci on newton changes? | |
| 18:00:05 | mriedem | that branch is nearly eol | |
| 18:00:28 | mriedem | 2018-01-03 17:51:43.289 | error: pathspec 'stable/newton' did not match any file(s) known to git. | |
| 18:01:20 | mmedvede | mriedem: there is actually a fix for that in the pipeline, let me dig it out | |
| 18:01:39 | mmedvede | it affects all non-zuulv3 third-party CIs | |
| 18:03:02 | mmedvede | mriedem: https://review.openstack.org/#/c/523925/ | |
| 18:05:17 | mmedvede | the CI failing but still +1 the change is due to there being only non-voting job in the set | |
| 18:05:24 | mriedem | i feel like i recently made a similar change to devstack in newton | |
| 18:05:47 | mriedem | ah yes https://review.openstack.org/#/c/519758/ | |
| 18:06:11 | mriedem | and apparently it was my devstack-gate patch they are referencing | |
| 18:06:11 | mriedem | https://review.openstack.org/#/c/520095/ | |
| 18:07:54 | mriedem | sdague: you around for devstack-gate +2? https://review.openstack.org/#/c/523925/ | |
| 18:07:59 | mmedvede | d-g is accumulating lots of cruft due to having to support zuulv3/v2 at the same time :( | |
| 18:09:53 | mmedvede | mriedem: I'll watch the patch, if the fix would not land I'll disable our CI on stable/newton | |
| 18:11:44 | mriedem | ok | |
| 18:28:10 | mriedem | cdent: you want to take a crack at a release note for this? https://review.openstack.org/#/c/448034/ | |
| 18:30:37 | cdent | mriedem: ya, thanks, will look, I don't know much about that one (just did the rebase) but should be able to do it up | |
| 18:31:00 | mriedem | stephenfin: seeing a TypeError in testing the websocket proxy series http://logs.openstack.org/50/530950/1/check/legacy-tempest-dsvm-py35/1dffae7/logs/screen-n-novnc.txt.gz?level=TRACE#_Jan_03_16_09_08_830407 | |
| 18:31:13 | mriedem | in the py35 job | |
| 18:31:24 | cdent | will check with rado first | |
| 18:33:29 | mriedem | huzzah, i've done my job https://review.openstack.org/#/c/345398/ | |
| 18:37:43 | melwitt | mriedem: weird, I wonder how the tempest py35 passed on my DNM patch but not with yours on the console proxy with TLS stuff | |
| 18:38:32 | mriedem | melwitt: because you were configuring one specific job | |
| 18:38:35 | mriedem | which doesn't use py35 | |
| 18:38:41 | melwitt | oh, right. derp | |
| 18:55:07 | openstackgerrit | Chris Dent proposed openstack/nova master: VMware: add support for different firmwares https://review.openstack.org/448034 | |
| 18:56:46 | cdent | I had an amusing conversation with someone outside of openstack today about the complex bits of placement and scheduling and he said what many people have said before: maybe there needs to be an openstack for "give me a vm" and a different openstack for "I am a telco" | |
| 18:57:36 | jaypipes | cdent: bingo. | |
| 18:57:59 | cdent | I thought that might draw you out :) | |
| 19:01:06 | melwitt | hah, I like that. openstack Telco Edition(tm) | |
| 19:01:23 | cdent | now including a super upper ontology | |
| 19:07:10 | ildikov | and it's only the 3rd of January... :) | |
| 19:10:58 | hrw | guys: can you look at https://review.openstack.org/#/c/530965/ one? It is probably wrong way of doing it and I am open for suggestions | |
| 19:23:28 | openstackgerrit | Matt Riedemann proposed openstack/nova master: VMware: add support for different firmwares https://review.openstack.org/448034 | |
| 19:25:28 | stvnoyes | mriedem: Hi Matt, I was talking to ildikov and she suggested I could help with https://review.openstack.org/#/c/266605 , the tempest multiattach test. I see you've been working on it recently. Any objections to me taking it on? | |
| 19:31:58 | ildikov | stvnoyes: I mentioned it mriedem today that we talked about that test and that we can talk about the desired test coverage in Tempest on the meeting tomorrow | |
| 19:32:19 | mriedem | i'm debugging the failure now - can't create a multiattach volume | |
| 19:32:38 | ildikov | mriedem: it's disabled in Cinder | |
| 19:33:14 | ildikov | mriedem: https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/lvm.py#L279 | |
| 19:33:29 | mriedem | Jan 03 14:42:03.502967 ubuntu-xenial-citycloud-sto2-0001666901 cinder-scheduler[12541]: DEBUG cinder.scheduler.filters.capabilities_filter [None req-ecbc7305-ea6f-438b-af55-028c5b6eeb03 tempest-AttachVolumeMultiAttachTest-1915307512 None] Volume type extra spec requirement "multiattach=<is> True" does not match reported capability "False" {{(pid=12541) _satisfies_extra_specs /opt/stack/cinder/cinder/scheduler/filters/capabili | |
| 19:33:29 | mriedem | _filter.py:74}} | |
| 19:33:56 | stvnoyes | @page { margin: 0.79in } pre { direction: ltr; text-align: left; orphans: 2; widows: 2 } pre.western { font-family: "Times New Roman", serif; font-size: 12pt } pre.cjk { font-family: "SimSun"; font-size: 12pt; so-language: zh-CN } pre.ctl { font-family: "Times New Roman"; font-size: 12pt } p { margin-bottom: 0.1in; direction: ltr; line-height: 120%; text-align: left; orphans: 2; widows: 2 } p.western { fo | |
| 19:33:56 | stvnoyes | ||
| 19:34:03 | stvnoyes | ugh! | |
| 19:34:34 | stvnoyes | - Need the lvm driver to support MA - in cinder/volume/drivers/lvm.py , line 280 > | |
| 19:34:34 | stvnoyes | multiattach = True | |
| 19:35:05 | stvnoyes | there are also issues in tempest detach, I'm working on that now | |
| 19:35:36 | ildikov | stvnoyes: what's the issue with detach? | |
| 19:35:59 | mriedem | tempest assumes that when you detach the volume will eventually be 'available' | |
| 19:36:10 | mriedem | but with multiattach it will go from in-use to in-use | |
| 19:36:20 | mriedem | until the last attachment is gone | |
| 19:36:24 | ildikov | stvnoyes: there's a whole chain to address shared_targets issues: https://review.openstack.org/#/c/529695/ | |
| 19:36:31 | stvnoyes | from what I see so far, at the end of the test it tries to detach, but when it detaches the vol from the first server, the vol status doesn't go to abailable (as we would expect) but the test wants to see available | |
| 19:36:35 | ildikov | however I'm not sure that's applicable for lvm... | |
| 19:36:45 | mriedem | ildikov: different issues | |
| 19:37:00 | mriedem | i didn't realize that cinder hard-coded this in all drivers across the board | |
| 19:37:11 | ildikov | ah ok, sorry, I thought actual issues and not things we need to update | |
| 19:37:19 | mriedem | that should have probably been a global policy change in the API | |
| 19:37:48 | stvnoyes | I think we will need to pass in an expected status to detach so it knows what success means | |
| 19:37:53 | ildikov | I think they also wanted to ensure that driver maintainers think about it for at least two seconds before turning it on | |
| 19:38:20 | mriedem | stvnoyes: not sure that is necessary - we can just count the attachments first, and if the volume has multiattach=True | |
| 19:38:34 | stvnoyes | y, that should work too | |
| 19:38:52 | stvnoyes | and is simpler... | |
| 19:39:06 | mriedem | ildikov: so is https://review.openstack.org/#/c/428365/ going to be reverted at the same time that a policy rule is put in place? | |