Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-03-27
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
18:03:07 openstackgerrit Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #4: full flavor https://review.openstack.org/391288
18:03:27 sean-k-mooney moshele: if we do close it i would ask though that we add a comment in the code to that effect to capture why that code is needed.
18:03:42 openstackgerrit Eric Fried proposed openstack/nova master: PowerVM Driver: console https://review.openstack.org/409402
18:04:14 openstackgerrit Eric Fried proposed openstack/nova master: PowerVM Driver: SSP emphemeral disk support https://review.openstack.org/443189
18:04:26 moshele sean-k-mooney: sure, let me just verify it first :), I will update you tomorrow
18:04:32 openstackgerrit Eric Fried proposed openstack/nova master: PowerVM Driver: config drive https://review.openstack.org/409404
18:07:24 openstackgerrit Matthew Booth proposed openstack/nova master: libvirt: Pass instance to connect_volume and disconnect_volume https://review.openstack.org/450383
18:19:52 openstackgerrit Matthew Booth proposed openstack/nova master: libvirt: Fix races with nfs volume mount/umount https://review.openstack.org/383859
18:32:54 openstackgerrit Matthew Booth proposed openstack/nova master: libvirt: Fix races with nfs volume mount/umount https://review.openstack.org/383859
18:32:54 openstackgerrit Matthew Booth proposed openstack/nova master: libvirt: Pass instance to connect_volume and disconnect_volume https://review.openstack.org/450383
18:39:12 jaypipes gibi: guh, what's your gerrit userid?
19:03:30 openstackgerrit Sujitha proposed openstack/nova master: [WIP] Add BDM to InstancePayload https://review.openstack.org/448779
19:05:49 efried jaypipes Pretty sure it's Balazs Gibizer / balazs.gibizer@ericsson.com
19:14:53 jaypipes efried: damn it, I was spelling it balasz... :)
19:14:57 jaypipes efried: cheers.
19:15:06 efried jaypipes np.
19:47:44 jaypipes melwitt, johnthetubaguy, alex_xu, bauzas, dansmith: easy one for you: https://review.openstack.org/#/c/448225/
20:00:20 openstackgerrit Dan Smith proposed openstack/nova master: Teach HostAPI about cells https://review.openstack.org/442162
20:00:21 openstackgerrit Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891
20:00:22 openstackgerrit Dan Smith proposed openstack/nova master: Sort CellMappingList.get_all() for safety https://review.openstack.org/443174
20:00:22 openstackgerrit Dan Smith proposed openstack/nova master: Add get_by_instance_uuids() to InstanceMappingList https://review.openstack.org/443292

Earlier   Later