Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-08-10
17:57:48 openstackgerrit Merged openstack/nova master: Quota details for key_pair "in_use" is 0. https://review.openstack.org/590081
18:09:08 openstackgerrit Marcus Furlong proposed openstack/nova master: improve migration script https://review.openstack.org/591030
18:29:39 melwitt SteelyDan: added comments to the revert patch. I think some unit test coverage specific to the speed limit patch was missed for reverting. let me know if I'm mistaken
18:30:27 SteelyDan looking through them now
18:31:08 SteelyDan most of this was conflicted out since these tests were all modified since this patch
18:31:33 openstackgerrit Konstantinos Samaras-Tsakiris proposed openstack/nova-specs master: Placement model for PCI devices https://review.openstack.org/591037
18:31:41 SteelyDan it's not a straight revert regardless, so I guess I'm not sure I see the point of putting things back like the bandwidth thing as that pattern then got copied in the params stuff afterwards in some places
18:31:57 SteelyDan and like the virtapi thing makes no difference
18:32:04 SteelyDan and the networkinfo
18:32:13 SteelyDan but the things to collect the events are probably dead now
18:32:30 SteelyDan not sure I'm going to get to this before I have to relocate for the day but I'll try
18:32:35 melwitt okay, it just seems like test_live_migration_main and test_live_migration_ovs_vif are duplicates of one another, essentially
18:33:18 SteelyDan probably yeah
18:33:46 melwitt since the networkinfo part is taken out, that was the difference between them, I thought. ok, I can add a follow up to clean that part up since you've got to run soon
18:34:19 melwitt since it's just extra duplicated unit test coverage
18:34:25 SteelyDan I'll take a shot and let you know if I don't push something up
18:34:33 melwitt ok
18:34:41 SteelyDan yeah, the second test being a duplicate now makes sense
18:36:53 openstackgerrit Chris Dent proposed openstack/nova stable/ocata: [placement] Retry allocation writes server side https://review.openstack.org/591042
18:38:34 SteelyDan maybe that will all just come out easy, lemme stash and try
18:45:43 kosamara efried: Hi! I've asked you some more questions on your spec. In the meantime I've published my take for review and expect to change it depending on your answers.
18:46:16 kosamara efried: https://review.openstack.org/#/c/591037/
18:46:17 fried_rolls kosamara: Neat, this one: https://review.openstack.org/591037 ?
18:46:20 fried_rolls okay, cool.
18:48:06 openstackgerrit Dan Smith proposed openstack/nova master: Revert "libvirt: slow live-migration to ensure network is ready" https://review.openstack.org/590538
19:18:46 openstackgerrit Dan Smith proposed openstack/nova master: Revert "libvirt: slow live-migration to ensure network is ready" https://review.openstack.org/590538
19:25:30 openstackgerrit Matthew Edmonds proposed openstack/nova master: Make Xen code py3-compatible https://review.openstack.org/591061
19:43:50 openstackgerrit Eric Fried proposed openstack/nova master: Test case for multiple forbidden traits https://review.openstack.org/591070
19:59:40 openstackgerrit Merged openstack/nova master: Add additional info to resource provider aggregates update API https://review.openstack.org/590243
20:04:23 openstackgerrit Matthew Edmonds proposed openstack/nova master: Make Xen code py3-compatible https://review.openstack.org/591061
20:11:15 openstackgerrit Jay Pipes proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041
20:11:42 openstackgerrit Eric Fried proposed openstack/nova master: DNM: demo https://review.openstack.org/591073
20:12:57 leakypipes fried_rice: one sec. pulling your test patch onto the bottom.
20:13:26 fried_rice leakypipes: Cool beans. I just posted my comments, as well as --^ another patch demoing some of them
20:13:30 openstackgerrit Jay Pipes proposed openstack/nova master: placement: use simple code paths when possible https://review.openstack.org/590388
20:13:31 openstackgerrit Jay Pipes proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041
20:13:32 openstackgerrit Jay Pipes proposed openstack/nova master: Adds a test for _get_provider_ids_matching() https://review.openstack.org/590150
20:13:33 openstackgerrit Jay Pipes proposed openstack/nova master: Test case for multiple forbidden traits https://review.openstack.org/591070
20:13:36 leakypipes fried_rice: et voila ^
20:14:36 fried_rice leakypipes: You put it on top
20:15:19 fried_rice leakypipes: You also reordered the other test patch to the top.
20:17:32 fried_rice leakypipes: For ease of backportage, perhaps we should squash both test patches into Gigantor.
20:19:58 openstackgerrit Jay Pipes proposed openstack/nova master: Adds a test for _get_provider_ids_matching() https://review.openstack.org/590150
20:19:59 openstackgerrit Jay Pipes proposed openstack/nova master: Test case for multiple forbidden traits https://review.openstack.org/591070
20:20:00 openstackgerrit Jay Pipes proposed openstack/nova master: placement: use simple code paths when possible https://review.openstack.org/590388
20:20:01 openstackgerrit Jay Pipes proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041
20:20:11 leakypipes fried_rice: sorry bout that. fixed.
20:20:27 fried_rice leakypipes: Okay, please see comments in PS6 - at least one blocker remaining.
20:20:44 fried_rice s/at least//
20:35:59 openstackgerrit melanie witt proposed openstack/nova master: Handle unicode characters in migration params https://review.openstack.org/591080
21:00:45 openstackgerrit Merged openstack/nova master: Remove patching the mock lib https://review.openstack.org/590709
21:47:23 openstackgerrit Jay Pipes proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041
21:48:05 leakypipes fried_rice: ^^
21:48:22 fried_rice ack
22:02:12 fried_rice leakypipes: Just a few minor things.
22:31:00 openstackgerrit Jay Pipes proposed openstack/nova master: split gigantor SQL placement query into multiple https://review.openstack.org/590041
#openstack-nova - 2018-08-11
07:29:45 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/591129
#openstack-nova - 2018-08-12
07:33:06 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/591129
09:59:29 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Trivial fix to remove extra 'will' on microversion doc https://review.openstack.org/591235
14:41:10 tbachman anyone here used heat templates with nova much? I was trying to use them to create instances in a project other than admin, but I didn’t see a parameter that I could use to pass in the project ID when creating the server
14:41:22 tbachman hopefully I’m just missing something
14:44:20 tbachman I guess I should say - it looks like it just uses the project ID that was used for the heat-template
14:45:40 tbachman would have been nice if this could be a parameter
14:57:02 openstackgerrit Balazs Gibizer proposed openstack/nova master: Use placement 1.28 in scheduler report client https://review.openstack.org/583667
22:27:58 openstackgerrit Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289
22:28:27 openstackgerrit Takashi NATSUME proposed openstack/nova master: Making consistent used of GiB and MiB in API ref https://review.openstack.org/559985
22:48:10 openstackgerrit Takashi NATSUME proposed openstack/nova master: Update contributor guide for Stein https://review.openstack.org/591258
#openstack-nova - 2018-08-13
01:57:50 mriedem muwahahaha
02:01:08 openstackgerrit Matt Riedemann proposed openstack/nova master: add zvm into support matrix https://review.openstack.org/532720
03:58:28 openstackgerrit Merged openstack/nova master: Revert "libvirt: slow live-migration to ensure network is ready" https://review.openstack.org/590538
03:58:35 openstackgerrit Merged openstack/nova master: Trivial fix to remove extra 'will' on microversion doc https://review.openstack.org/591235
03:58:42 openstackgerrit Merged openstack/nova master: Adds a test for _get_provider_ids_matching() https://review.openstack.org/590150
03:58:49 openstackgerrit Merged openstack/nova master: Test case for multiple forbidden traits https://review.openstack.org/591070
04:05:20 openstackgerrit Merged openstack/nova master: placement: use simple code paths when possible https://review.openstack.org/590388
04:05:26 openstackgerrit Merged openstack/nova master: Handle unicode characters in migration params https://review.openstack.org/591080
04:05:33 openstackgerrit Merged openstack/nova master: improve migration script https://review.openstack.org/591030
06:20:06 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Revert "libvirt: slow live-migration to ensure network is ready" https://review.openstack.org/591275
06:23:33 prometheanfire :D
06:23:41 prometheanfire one step at a time
07:34:35 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform volume.usage notification https://review.openstack.org/580345
07:39:47 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform compute_task notifications https://review.openstack.org/482629
07:44:12 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform libvirt.error notification https://review.openstack.org/484851
07:57:37 openstackgerrit Konstantinos Samaras-Tsakiris proposed openstack/nova-specs master: Placement model for PCI devices https://review.openstack.org/591037
08:34:16 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP scheduler: check request_spec.instance_group before updating https://review.openstack.org/590733
09:11:13 mriedem gibi: oh hi
09:11:18 mriedem we are timezone buddies!
09:12:07 gibi mriedem: Ohh welcome in this side of the globe!
09:12:46 gibi mriedem: are you in China?
09:18:06 openstackgerrit garyk proposed openstack/nova master: Docs: remove invalid link for remote debugging https://review.openstack.org/591316
09:23:07 mriedem mdbooth: http://lists.openstack.org/pipermail/openstack-dev/2018-August/133198.html - the functional test job runs mysql (and postgresql) "opportunistic" tests
09:23:11 mriedem with mysql and pg backends
09:24:15 mdbooth mriedem: So both are configured? IIRC the opportunistic tests run only if the specific DB is actually available.
09:24:34 mdbooth mriedem: You're up early, btw. In Europe somewhere?
09:25:54 mriedem_ mdbooth: china
09:26:10 mdbooth Ah, so you've been up for a while :)
09:26:26 mriedem_ https://github.com/openstack/nova/blob/master/tools/test-setup.sh sets up the mysql and pg dbs for the opportunistic tests
09:26:47 mdbooth mriedem_: Thank you!
09:33:24 mriedem mdbooth: btw is rajesh aware of the spec for that metadata case sensitivity issue?
09:33:42 mdbooth mriedem: I was just putting a +1 on it and I noticed your comments. Reading...
09:34:02 mdbooth ratailor: ^^^ ?

Earlier   Later