Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-08-10
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: ^^^ ?
09:35:33 ratailor mdbooth, I don't know about spec.
09:35:42 gibi mriedem: do you know if we can remove this in Stein? https://github.com/openstack/nova/blob/16f89fd093217d22530570e8277b561ea79f46ff/nova/virt/ironic/driver.py#L147-L149
09:36:00 mdbooth At first glance it doesn't sound like a very good idea to me, tbh.
09:36:58 ratailor mdbooth, let me go through the comments on patch.
09:36:58 mdbooth So.. we've allowed users to define their own metadata for however long, and now we want to change it just because of a mysql misconfiguration?
09:37:01 mriedem gibi: maybe...
09:37:24 mdbooth That seems user hostile
09:37:33 mriedem gibi: we have a nova-status upgrade check for the ironic instance / flavor migration to resource classes
09:37:42 ratailor mdbooth, mriedem in that case, should I abandon the patch ?
09:37:48 mriedem gibi: i think we have something in the ptg etherpad about this, removing support for granular resources with ironic
09:38:39 gibi mriedem: Is it L25 in the etherpad?
09:39:17 gibi mriedem: it seems so. So I'm plugging my pain (https://review.openstack.org/#/c/583667/19/nova/tests/functional/test_servers.py@5135) into that item. thanks
09:40:04 mdbooth ratailor: It's up to you, but personally I'd see if the bp and spec still have weight behind them, especially given that they're user hostile and your patch shows that fixing the underlying issue isn't hard.
09:40:45 mriedem ratailor: mdbooth: right it's obviously still an issue,
09:40:53 mriedem and the spec was merged at some point, so there was a plan with code,
09:41:03 mriedem but then auggy was gone and sdague was gone and that whole HP team
09:41:11 mriedem er IBM team or whatever
09:41:46 mriedem gibi: yes that's the one
09:41:54 mriedem definitely want to remove that in stein if we can
09:41:55 gibi mriedem: thanks. I've added a note
09:43:03 sean-k-mooney mriedem: are you in china right now or is this very early for you. im guessing its quite late but just wondering what time zone your in since your not normally on at this time
09:43:37 mriedem sean-k-mooney: i'm in china this week
09:44:02 mdbooth ratailor: Looks like the proposed code wasn't merged and was abandoned.
09:45:02 mdbooth mriedem: Thanks for pointing that out. I'll follow up on the ML with a question around if this is still something we want to do.

Earlier   Later