| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-27 | |||
| 15:02:32 | artom | derekh, remember https://bugs.launchpad.net/nova/+bug/1622545 ? Do you by any chance have similar dumps for older openstack versions? The one you provided in comment #6 has been very useful for our downstream testing, and we'd like to backport the fix all the way back to Juno | |
| 15:02:32 | openstack | Launchpad bug 1622545 in OpenStack Compute (nova) "archive_deleted_rows isn't archiving instances" [Medium,Fix released] - Assigned to Dan Smith (danms) | |
| 15:03:16 | mriedem | johnthetubaguy: i've left comments in your cinder v3 spec https://review.openstack.org/#/c/373203/ | |
| 15:03:17 | sean-k-mooney | sfinucan: or rather there ml2 drivers not ovn/odl themselves | |
| 15:03:39 | dansmith | mriedem: did you check cinder? might be in there | |
| 15:03:56 | mriedem | ooo looking | |
| 15:04:07 | derekh | artom: Hi, afraid not, this was a redeploy and the data from the old installation is long gone | |
| 15:04:14 | dansmith | mriedem: well, we have a BFV | |
| 15:04:27 | artom | derekh, figured. Oh well, it was worth a shot - thanks :) | |
| 15:04:33 | mriedem | ah https://github.com/openstack-dev/grenade/blob/master/projects/70_cinder/resources.sh#L130 | |
| 15:04:37 | sdague | mriedem: we have a bvf on the cinder side, but not an explicit attach after iirc | |
| 15:04:58 | mriedem | ok, | |
| 15:05:12 | mriedem | for johnthetubaguy's cinder v3 spec i think we want a grenade scenario where we attach on the old side and detach on the new side, | |
| 15:05:13 | derekh | artom: its always worth a shot, sorry I don't have anything useful | |
| 15:05:24 | mriedem | because the detach flow is going to depend on how we attached the volume in the first place | |
| 15:07:45 | mriedem | johnthetubaguy: on https://review.openstack.org/#/c/373203/ my main concern was around the swap volume steps setting bdm.attachment_id, | |
| 15:07:57 | mriedem | which would trigger the new detach flow before we actually wanted that i think | |
| 15:08:11 | mriedem | johnthetubaguy: otherwise just some questions and clarifications but i'd be +2 | |
| 15:08:14 | johnthetubaguy | mriedem: yeah, +1 the grenade these, that sounds good | |
| 15:08:42 | johnthetubaguy | mriedem: oh, thats the new swap flow, so it only applies when attachement_id is already set in the BDM right? | |
| 15:08:44 | mriedem | and i think on the shared connections section you could be more clear that we're not dealing with that in this spec, and why we are not of course | |
| 15:09:34 | mriedem | johnthetubaguy: it doesn't say that https://review.openstack.org/#/c/373203/21/specs/pike/approved/cinder-new-attach-apis.rst@265 | |
| 15:09:53 | johnthetubaguy | mriedem: yeah, it should say that | |
| 15:10:27 | johnthetubaguy | mriedem: basically, like all the other ones, keep the old flow if the attachment_id is not there | |
| 15:11:01 | mriedem | ok i didn't see that in the swap volume part, really threw me off | |
| 15:11:16 | mriedem | feel free to point out where i'm blind, otherwise we should probably clarify | |
| 15:11:22 | sfinucan | sean-k-mooney: Hmm, OK can you leave that in the review (if you haven't done so already)_ | |
| 15:11:46 | sfinucan | Not sure if that's worth doing. It might be. We should probably add a TODO in the code in any case | |
| 15:11:50 | mriedem | sdague: dansmith: johnthetubaguy: also, fyi, i'm traveling to CA for orientation this week, leaving in less than an hour, getting back home thursday night. | |
| 15:12:00 | mriedem | so i probably won't be around much the next few days | |
| 15:12:02 | sdague | ok, cool | |
| 15:12:04 | sdague | enjoy | |
| 15:12:12 | dansmith | mriedem: ack | |
| 15:12:19 | mriedem | i will not enjoy tucking my shirt in | |
| 15:12:25 | mriedem | or you know, wearing pants | |
| 15:12:30 | dansmith | heh | |
| 15:19:13 | johnthetubaguy | mriedem: good luck with orientation! | |
| 15:19:31 | johnthetubaguy | mriedem: sounds like a very good clarification we need in the spec | |
| 15:19:38 | sdague | don't forget your compass | |
| 15:22:08 | bauzas | mriedem: North is in that direction => | |
| 15:22:42 | sean-k-mooney | sfinucan: sure, i was planning on requing it as part of the migration to os-vif objects so that would be a logical time to implement it. | |
| 15:23:16 | mriedem | don't worry guys, i've seen The Edge several times, | |
| 15:23:27 | mriedem | i know how to survive in the wild with a pine needle compass | |
| 15:24:04 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: use context mgr in instance.delete https://review.openstack.org/443764 | |
| 15:25:27 | bauzas | mriedem: I hugely watched McGyver when I was a child | |
| 15:25:46 | bauzas | so, nothing is scaring me about the wild | |
| 15:27:22 | bauzas | mriedem: just make sure you got you swiss army knife and you'll be good | |
| 15:29:17 | mriedem | i'm less worried about the bears than i am the tight connection i have | |
| 15:35:21 | dims | mriedem : safe travels! | |
| 15:37:50 | openstackgerrit | Jens Rosenboom proposed openstack/nova master: WIP: Fix network metadata information - part 3/3 https://review.openstack.org/450297 | |
| 15:42:58 | openstackgerrit | Maciej Szankin proposed openstack/nova-specs master: Add remove-nova-cert spec https://review.openstack.org/448882 | |
| 15:45:36 | openstackgerrit | Sujitha proposed openstack/nova master: Add description to policies in aggregates.py https://review.openstack.org/445230 | |
| 15:50:19 | openstackgerrit | Sujitha proposed openstack/nova master: Add description to policies in cells_scheduler.py https://review.openstack.org/448260 | |
| 16:19:30 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Add Cinder V3 Detach calls https://review.openstack.org/438750 | |
| 16:26:11 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Rename '[neutron] ovs_bridge' https://review.openstack.org/450266 | |
| 16:26:11 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Move 'floating_ips' opts into 'network' https://review.openstack.org/431039 | |
| 16:26:12 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Deprecate 'default_floating_pool' https://review.openstack.org/431038 | |
| 16:26:12 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Add neutron.default_floating_pool https://review.openstack.org/431037 | |
| 16:34:22 | openstackgerrit | Anthony PERARD proposed openstack/nova master: Add exclusion list for tempest for a libvirt+xen job https://review.openstack.org/449687 | |
| 16:43:10 | openstackgerrit | dane-fichter proposed openstack/nova master: Use cursive for signature verification https://review.openstack.org/351232 | |
| 16:53:29 | openstackgerrit | Gábor Antal proposed openstack/nova master: Add json style checking for sample notifications https://review.openstack.org/443677 | |
| 16:55:56 | oomichi | andreaf: about https://review.openstack.org/#/c/448851/ , cinder v2 api is deprecated in this Pike cycle and we don't have enough v3 clients. | |
| 16:56:19 | oomichi | andreaf: the above patch itself seems good for me | |
| 16:56:29 | openstackgerrit | Maxim Nestratov proposed openstack/nova master: libvirt: Virtuozzo containers config drive support https://review.openstack.org/449818 | |
| 16:56:42 | oomichi | andreaf: oops, sorry here is wrong channel.. should use qa channel | |
| 16:56:51 | abhi89 | Hi All.. does anybody know which option was introduced in openstack when they deprecared 'verbose' option (used for INFO logging) form /etc/nova.conf | |
| 16:57:31 | abhi89 | 'verbose' option in /etc/nova.conf in default section, was deprecated from mitaka | |
| 16:57:46 | jaypipes | ralonsoh: https://review.openstack.org/448282 fixed per your request. | |
| 16:58:42 | ralonsoh | jaypipes: I'll review again the patch | |
| 16:58:58 | jaypipes | ralonsoh: cheers | |
| 17:00:06 | jaypipes | johnthetubaguy, dansmith: would be great to get your feedback on https://review.openstack.org/#/c/435954/. | |
| 17:02:56 | openstackgerrit | Béla Vancsics proposed openstack/nova master: Transform instance.trigger_crash_dump notification https://review.openstack.org/396225 | |
| 17:02:57 | openstackgerrit | Béla Vancsics proposed openstack/nova master: Ensure instance is in active state after notification test https://review.openstack.org/448655 | |
| 17:12:39 | openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach notification https://review.openstack.org/401992 | |
| 17:19:35 | efried | Can anyone tell me if this call is actually doing something? https://github.com/openstack/nova/blob/master/nova/tests/unit/compute/test_compute.py#L10628-L10629 | |
| 17:20:00 | efried | ...cause the 'uuid' param isn't a UUID. It's the uuid lib. Which is... weird. | |
| 17:20:24 | efried | The test still passes when I change that param to a random string. Or when I comment out the statement. | |
| 17:21:01 | efried | shaohe_feng ^^ | |
| 17:37:12 | moshele | sean-k-mooney: hi | |
| 17:37:31 | sean-k-mooney | moshele: o/ | |
| 17:37:49 | moshele | sean-k-mooney: please see my comments on https://review.openstack.org/#/c/447269 | |
| 17:38:20 | moshele | sean-k-mooney: adding port to the bridge didn't bring up the port on Ubuntu | |
| 17:38:57 | sean-k-mooney | the code are disableing should is not bring up the port | |
| 17:39:02 | sean-k-mooney | its bring up the bridge | |
| 17:39:27 | sean-k-mooney | you need to bring up the veth pair port but the bridge port should not need to be brought up | |
| 17:41:35 | sean-k-mooney | processutils.execute('ip', 'link', 'set', bridge, 'up') if bridge was dev it would make sense to me but i dont know why we would bring upu the linux bridge prot when adding a port to it | |
| 17:42:10 | sean-k-mooney | that is what was been done before in https://review.openstack.org/#/c/447269/1/vif_plug_ovs/linux_net.py | |
| 17:42:10 | moshele | sean-k-mooney: sorry, let me try it again :) | |
| 17:42:37 | sean-k-mooney | moshele: sure :) | |
| 17:54:00 | openstackgerrit | Sujitha proposed openstack/nova master: [WIP] Add BDM to InstancePayload https://review.openstack.org/448779 | |
| 17:55:27 | moshele | sean-k-mooney: I added a local port which in up state to the brdige but the bridge is not up see http://paste.openstack.org/show/604360/ | |
| 17:56:41 | sean-k-mooney | moshele: ok and with that configuration what do you observe | |
| 17:56:55 | sean-k-mooney | does the linux bridge work? | |
| 17:59:01 | moshele | sean-k-mooney: I don't have a full openstack setup to verify, but I this that why tempest is failing in https://review.openstack.org/#/c/447269, I can try to do it tomorrow and see if it working | |
| 18:00:38 | sean-k-mooney | moshele: you dont need a full openstack. but the issue that we observed is that if you bring up the bridge everything works but if the vms sends an arp and you tcp dump on the bridge(test) interface the host recives an untagged arp requrest on the bridge interface | |
| 18:01:33 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: power_on/off and reboot https://review.openstack.org/427380 | |
| 18:01:36 | efried | thorst ^^ | |
| 18:02:25 | openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #3: TaskFlow https://review.openstack.org/438729 | |
| 18:02:45 | openstackgerrit | Sujitha proposed openstack/nova master: [WIP] Add BDM to InstancePayload https://review.openstack.org/448779 | |
| 18:02:47 | sean-k-mooney | moshele: if the linux bridge wont switch packets unless the bridge is up then im fine to close the bug as we have no choice but to bring up the bridge interface | |