Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-02-02
06:40:34 openstackgerrit Merged openstack/nova stable/pike: Fix instance lookup in hide_server_addresses extension https://review.openstack.org/518996
06:40:46 openstackgerrit Merged openstack/nova stable/pike: Only log not correcting allocation once per period https://review.openstack.org/526477
06:45:04 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (3) https://review.openstack.org/531280
06:46:04 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in get https://review.openstack.org/531299
06:56:15 openstackgerrit Ameed Ashour proposed openstack/nova stable/ocata: add "--until-complete" option for nova-manage db archive_deleted_rows. https://review.openstack.org/540274
07:03:47 openstackgerrit Ameed Ashour proposed openstack/nova stable/pike: add "--until-complete" option for nova-manage db archive_deleted_rows. https://review.openstack.org/539880
07:45:52 openstackgerrit James E. Blair proposed openstack/python-novaclient master: Zuul: Remove project name https://review.openstack.org/540304
07:47:58 openstackgerrit Merged openstack/nova master: Pass limit to /allocation_requests https://review.openstack.org/531517
08:38:00 openstackgerrit Ameed Ashour proposed openstack/nova master: detach instance volumes when VM creation fails https://review.openstack.org/528385
08:38:23 bauzas good Friday everyone
08:38:30 bauzas let's take my casual dress
08:41:06 ameeda bauzas: good Friday :)
08:41:29 ameeda bauwser: also :D
08:41:51 Spazmotic Morning bauwser
08:41:53 Spazmotic Oi...
08:41:58 Spazmotic not you.. I take back my morning to you!
08:42:12 Spazmotic :)
08:42:48 gibi happy Friday everyone :)
08:42:55 Spazmotic Morning gibi
08:43:54 Spazmotic If anyone gets some time to review this today I would appreciate it. https://review.openstack.org/#/c/538415/ Would like to try to pack this one away before I leave for my trip
08:45:29 Spazmotic johnthetubaguy if you're working today and have time, that is the one that we discussed before, with live migration assert_can_migrate
08:56:00 ameeda cloud you please help me to figure out this issue https://review.openstack.org/#/c/526900/ ? the error of functional test appears after I added
08:56:00 ameeda https://review.openstack.org/#/c/526900/28/nova/db/sqlalchemy/migrate_repo/versions/378_instance_system_metadata_sqlite_upgrade.sql
09:19:26 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Encode libvirt domain XML in UTF-8 https://review.openstack.org/522161
09:43:29 gibi edleafe: thanks for simplification in https://review.openstack.org/#/c/539323 , LGTM
09:48:06 openstackgerrit James E. Blair proposed openstack/nova master: Zuul: Remove project name https://review.openstack.org/540326
09:55:21 openstackgerrit Surya Seetharaman proposed openstack/nova master: Extending delete_cell --force to delete instance_mappings https://review.openstack.org/540073
09:55:52 openstackgerrit Merged openstack/nova stable/ocata: Stop _undefine_domain erroring if domain not found https://review.openstack.org/508640
09:56:00 openstackgerrit Merged openstack/nova master: doc: placement upgrade notes for queens https://review.openstack.org/539658
09:59:03 openstackgerrit Surya Seetharaman proposed openstack/nova master: Marker reset option for nova-manage map_instances https://review.openstack.org/539501
10:18:11 lyarwood mdbooth / kashyap ; would you mind taking a look at the following LM tempest changes regarding encrypted volumes - https://review.openstack.org/#/q/status:open+project:openstack/tempest+branch:master+topic:bp/libvirt-qemu-native-luks
10:18:53 kashyap lyarwood: Hiya. Racing to do the slides for FOSDEM tomm :-(
10:18:57 lyarwood just want to ensure I'm not confusing live migration, live block migration, live migration with volumes attached etc
10:19:05 lyarwood k np
10:19:06 kashyap Sure, will look
10:19:09 ildikov gibi: hi :)
10:19:13 mdbooth lyarwood: Sure
10:19:22 kashyap But Monday I'll have full concentration back here
10:19:36 ildikov gibi: if you have a minute this multi-attach related change needs one more +2: https://review.openstack.org/#/c/532689/ :)
10:19:54 Spazmotic Opened page for FOSDEM and thought it said Bruges
10:20:19 lyarwood urgh so the cleanup duplicates the above test, sorry missed that
10:20:52 lyarwood ah no it doesn't, think I might need more coffee
10:20:56 lyarwood brb
10:22:31 kashyap Spazmotic: It is Brussels, not Bruges :-)
10:22:47 Spazmotic Hehe yeah I saw that after.. but made me audible go, "In Bruges!?"
10:49:08 andreaf mriedem_parent, ildikov: wow, https://review.openstack.org/#/c/266605/ merged! 266605! I didn't see a number in the 200k in a while :]
10:49:29 andreaf it surely is a pain to merge a patch in Tempest :P
10:50:12 gibi ildikov: hello :) I will try to look at it before end of today
10:59:43 ildikov andreaf: lol, I would say cross-project work has its challenges :)
11:00:01 ildikov andreaf: I mean overall
11:00:22 ildikov gibi: sounds good, thanks :)
11:01:03 ildikov gibi: the patch is small, but an important one for testing the new multi-attach stuff constantly
11:01:08 andreaf ildikov yes especially if QA is one of the projects :D
11:01:35 ildikov andreaf: that was the easy part this time :D
11:01:45 Spazmotic Ugh just found a more simple way to do my own commit
11:01:48 Spazmotic damn you eyes
11:04:16 openstackgerrit Chris Dent proposed openstack/nova master: Remove microversion fallback code from report client https://review.openstack.org/528794
11:10:27 openstackgerrit Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766
11:18:25 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Move resource provider objects into placement hierarchy https://review.openstack.org/540049
12:17:34 openstackgerrit Claudiu Belu proposed openstack/nova master: hyper-v: autospec classes before they are instantiated https://review.openstack.org/342211
12:45:36 openstackgerrit Lee Yarwood proposed openstack/nova master: DNM: Test LM with encrypted volumes https://review.openstack.org/536350
13:09:25 openstackgerrit Surya Seetharaman proposed openstack/nova master: Marker reset option for nova-manage map_instances https://review.openstack.org/539501
13:19:17 Spazmotic giblet always makes me think of a tiny pig.
13:19:23 Spazmotic like real tiny, like ant sized
13:20:33 efried @/
13:21:58 giblet Spazmotic: this giblet is full of birthday cake today so it is not so small of a pig now
13:22:16 Spazmotic Birthday pig.. sounds delicious.
13:22:39 Spazmotic Happy birthday man, if it is your own cake :)
13:22:40 giblet the cake was delicious too. :)
13:22:57 giblet Spazmotic: Yes, it is. Thank you
13:23:24 cdent happy birthday giblet
13:24:31 giblet cdent: thanks
13:25:13 openstackgerrit Eric Fried proposed openstack/nova-specs master: Granular Resource Request Syntax (Rocky) https://review.openstack.org/540179
13:26:03 efried Wow, is it Friday already??
13:30:44 Spazmotic That spec made my eyes fall out of my head so I assume it's super good
13:32:30 fried_rice Which one?
13:32:50 Spazmotic Yours.. impressive I feel dumber for even being too close to it.. I'll save that for when I want to make one later hehe
13:34:17 fried_rice Spazmotic: I wouldn't model your bp off of that one. *Some* folks thought it was too long.
13:38:12 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Move resource provider objects into placement hierarchy https://review.openstack.org/540049
13:57:22 openstackgerrit Balazs Gibizer proposed openstack/nova master: Escalate UUID validation warning to error in test https://review.openstack.org/540386
13:59:13 giblet edleafe: I figured out how can we ensure that no new UUID validation warning is merged to nova: https://review.openstack.org/#/c/540386
14:08:35 openstackgerrit Eric Fried proposed openstack/nova-specs master: Granular Resource Request Syntax (Rocky) https://review.openstack.org/540179
14:11:42 mriedem lyarwood: mdbooth: can you comment on melwitt's question here https://review.openstack.org/#/c/539739/2/nova/virt/libvirt/driver.py@7430
14:11:43 openstackgerrit Eric Fried proposed openstack/nova master: Avoid inventory DELETE API (no conflict detection) https://review.openstack.org/539712
14:11:51 mriedem i don't see anything ever setting connection_info['volume_id']
14:12:44 figleaf giblet: nice
14:13:08 mdbooth mriedem: When I was working on an HLR for a telco many, many moons ago, we'd have called that sort of request tromboning :)
14:13:27 figleaf giblet: working on fixing the merge conflict
14:13:59 mriedem mdbooth: ok i don't know what that means
14:14:10 ildikov mriedem: when we did the connection_info translation for the new attach API we didn't explicitly add the volume_id field to 'data' there so it doesn't seem to come from Cinder either
14:14:18 lyarwood mriedem: yup looking now
14:15:33 mdbooth mriedem: It was a request sent externally only to be bounced back to an internal source :)
14:16:16 mdbooth mriedem: Ah, that's interesting. I made some comments about that in review.
14:16:23 mriedem mdbooth: ok? i asked you guys because you were reviewing/working on the luks patch that i'm trying to fix.
14:16:54 mdbooth mriedem: Sorry, IRC failure. My mind was just wandering...
14:17:12 mdbooth mriedem: So I don't think we should be fetching stuff out of connection_info at all
14:17:27 mriedem this line specifically can be using bdm.volume_id
14:17:33 mdbooth I let it pass in review because I believed we always set volume_id in practise
14:17:38 mdbooth in connection_info
14:17:40 mriedem i was just asking if there was some specific logic as to why it was using connection_info

Earlier   Later