Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-07-07
06:29:12 Kevin_Zheng 4/4 also rebased to the latest version
06:29:47 mriedem cool, now just need to get alex_xu to look at https://review.openstack.org/#/c/459593/ again, i remember he was +2 on it before
06:31:36 Kevin_Zheng and the notification patch is also on the way, I messed up with a bad rebase last time ... will fix latter today
06:37:53 openstackgerrit Alex Xu proposed openstack/nova master: placement: ensure shared RP maps with correct root RP https://review.openstack.org/480379
06:47:28 openstackgerrit sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007
06:54:06 mriedem alex_xu: i've replied to your comments in https://review.openstack.org/#/c/471899/
06:54:14 mriedem thanks for going through that, it's a massive change
06:54:24 alex_xu mriedem: np, will check soon
06:54:50 ildikov mriedem: thank you!
07:01:58 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle Cinder 3.27 style attachments in swap_volume https://review.openstack.org/456971
07:01:59 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove check_detach https://review.openstack.org/446671
07:02:00 openstackgerrit Matt Riedemann proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
07:02:05 mriedem Kevin_Zheng: it would be good if you could review that bottom swap volume change as well ^
07:02:19 mriedem Kevin_Zheng: the new cinder v3 attach flow needs some more people involved on the nova side,
07:02:26 mriedem for reviews and understanding
07:02:43 Kevin_Zheng sure, will do
07:04:56 alex_xu mriedem: replied
07:06:08 openstackgerrit Alex Szarka proposed openstack/nova master: Refactor init_instance:resume_guests_state https://review.openstack.org/349582
07:06:22 openstackgerrit Alex Szarka proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660
07:07:46 mriedem alex_xu: got it, thanks again. i would say i will deal with that tomorrow, but it's already tomorrow :)
07:07:51 mriedem so i'll deal with it in a few hours
07:09:00 alex_xu mriedem: :) yea, really shock you still awake up now
07:10:06 openstackgerrit iswarya vakati proposed openstack/nova master: Fixed wrap from taking negative values https://review.openstack.org/481465
07:10:08 mriedem not for long, i'm signing off - see you in awhile
07:10:18 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add sample test for instance audit https://review.openstack.org/480955
07:44:28 openstackgerrit jichenjc proposed openstack/nova master: Add device tag support info in support matrix https://review.openstack.org/481478
07:44:28 openstackgerrit jichenjc proposed openstack/nova master: Add description for tag support in attach https://review.openstack.org/481479
07:51:37 openstackgerrit jichenjc proposed openstack/nova master: add description for supports_migrate_to_same_host https://review.openstack.org/481483
07:52:32 openstackgerrit KaiLi proposed openstack/nova master: Add "default" rule when generate policy.yaml.sample https://review.openstack.org/481443
08:11:11 openstackgerrit jichenjc proposed openstack/nova master: WIP: remove extension pop https://review.openstack.org/481491
08:15:03 openstackgerrit jichenjc proposed openstack/nova master: Remove nova.api.extensions.server.extensions usage https://review.openstack.org/481494
08:26:49 openstackgerrit jichenjc proposed openstack/nova master: add cli to support matrix https://review.openstack.org/481498
08:31:30 pabardina hello, Is it normal I don't see "locked_by" attribute when I'm using "nova show XX" on mitaka? I have "Locked: True" information, no more... But in the code there is the "locked_by" attr... any ideas ?
08:54:59 stephenfin johnthetubaguy: Next time you're about, would appreciate it if you could remove the -2 on this change as it no longer applies https://review.openstack.org/#/c/449584
08:58:26 openstackgerrit Alex Szarka proposed openstack/nova master: Reduce code complexity - manager.py https://review.openstack.org/359868
09:04:05 openstackgerrit Alex Szarka proposed openstack/nova master: Reduce code complexity - linux_net.py https://review.openstack.org/359871
09:04:17 openstackgerrit Alex Szarka proposed openstack/nova master: Reduce code complexity - libvirt/config.py https://review.openstack.org/359879
09:06:48 openstackgerrit Merged openstack/nova master: Add 'networks' quota in quota sample files https://review.openstack.org/479227
09:29:35 gmann pabardina, that will be present if you request API with microversion >2.9
09:30:23 gmann pabardina, and its there since liberty so mitaka is all fine to get that
09:32:22 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257
09:32:26 pabardina gmann, yes with microversion > 2.9, "locked" is present, but no "locked_by"
09:33:10 openstackgerrit Takashi NATSUME proposed openstack/nova master: Use URIOpt https://review.openstack.org/479215
09:34:09 gmann pabardina, ahh, yes its boolean 'locked'
09:35:03 pabardina gmann, ok, it's fine then. thanks :)
09:35:34 gmann pabardina, API never publish 'locked_by' i think it was only internally used
09:35:59 pabardina pabardina, alright
09:36:03 pabardina gmann, alright
09:48:13 openstackgerrit jichenjc proposed openstack/nova master: WIP: remove extension pop https://review.openstack.org/481491
09:48:44 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777
10:08:57 cdent jaypipes: another gabbi change you might be interested in: https://github.com/cdent/gabbi/pull/216
10:16:43 bauzas cdent: jaypipes: you knew that we had a problem with traits not synced in gabbi testing ?
10:16:54 bauzas that's fixed by Monday but just to make sure you knew it
10:25:53 cdent bauzas: this last monday? yeah
10:31:52 bauzas cdent: kewl
10:32:00 bauzas was just a FYI
10:32:13 cdent yeah, thanks
10:46:50 openstackgerrit sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007
10:59:03 openstackgerrit Dinesh Bhor proposed openstack/nova master: Trivial: Remove unnecessary format specifier https://review.openstack.org/481574
11:01:34 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051
11:02:15 jaypipes bauzas: yeah, that was the thing edleafe fixed, yeah?
11:05:09 bauzas jaypipes: nope, he missed something
11:05:31 bauzas jaypipes: https://review.openstack.org/#/c/480198/
11:17:28 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP traits https://review.openstack.org/474550
11:17:28 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocation_candidates https://review.openstack.org/481112
11:17:29 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for traits https://review.openstack.org/474186
11:17:29 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocations https://review.openstack.org/470933
11:17:30 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for usages https://review.openstack.org/480563
11:17:30 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Make placement_api_docs.py failing https://review.openstack.org/480924
11:17:31 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP usages https://review.openstack.org/450105
11:17:31 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP allocations https://review.openstack.org/450550
11:21:55 jaypipes cdent: nice on that full JSON gabbi enhancement. :)
11:22:33 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocation_candidates https://review.openstack.org/481112
11:22:34 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocations https://review.openstack.org/470933
11:22:34 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP traits https://review.openstack.org/474550
11:22:35 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for usages https://review.openstack.org/480563
11:22:35 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Make placement_api_docs.py failing https://review.openstack.org/480924
11:22:37 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP usages https://review.openstack.org/450105
11:22:37 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP allocations https://review.openstack.org/450550
11:23:10 cdent jaypipes: if you want to put a thumbs up of some kind on either or both of #215 and #216 that will help convince some of the other people.
11:23:28 jaypipes cdent: sure thing.
11:23:51 cdent thanks
11:24:27 jaypipes cdent: and... done. :)
11:24:37 cdent word
11:26:33 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocation_candidates https://review.openstack.org/481112
11:26:34 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Make placement_api_docs.py failing https://review.openstack.org/480924
11:27:43 jaypipes cdent: the bird.
11:27:53 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocation_candidates https://review.openstack.org/481112
11:27:54 openstackgerrit Andrey Volkov proposed openstack/nova master: [placement] Make placement_api_docs.py failing https://review.openstack.org/480924
11:28:11 jaypipes I wonder what avolkov` is working on ... :)
11:28:43 avolkov` jaypipes: sorry )
11:28:59 jaypipes avolkov`: why? it's a much-needed patch series! ++
11:29:19 jaypipes avolkov`: thank you very much for leading these doc efforts!
11:30:02 avolkov` jaypipes: I'm about all these rebases those spam chat ), always miss something
11:30:36 jaypipes avolkov`: heh, no worries :)
11:30:38 cdent avolkov`++
11:36:17 avolkov` cdent: jaypipes: btw, there is some initiative to migrate docs, probably placement api ref would be the first one, https://review.openstack.org/#/c/480991/
11:37:43 cdent avolkov`: yeah, saw that. I guess it makes sense given the docs migration spec (which feels a bit complicated but necessary)
11:38:00 cdent I reckon we should merge all the pending placement-api-ref before moving stuff?
11:39:24 jaypipes cdent: ++

Earlier   Later