Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-09-29
21:24:04 mriedem but overall good
21:24:27 mriedem superdan: ok, that was something i mentioned in ^ as a missing use case
21:24:35 mriedem since it seemed like a pretty critical issue for ironic
21:29:29 openstackgerrit ayoung proposed openstack/nova master: Admin API Policy contingent on is_admin_project https://review.openstack.org/384148
21:41:13 mriedem leakypipes: fried_rice: is there a reason why the traits parameter in alex's spec for GET /allocation_candidates is just called "required" rather than "required_traits"?
21:41:34 mriedem "Is the 'required' parameter required? No, it's optional. OK?"
21:43:24 figleaf mriedem: thanks
21:45:12 figleaf mriedem: and I wasn't at Boston, but I remember it from Atlanta
21:45:33 mriedem ok
21:45:40 mriedem let's combine our forgetfulness
21:45:43 mriedem for max recollection
21:46:06 figleaf It was in Austin!!
21:50:42 mriedem leakypipes: figleaf: superdan: johnthetubaguy: fried_rice: left some comments in the traits spec https://review.openstack.org/#/c/497713/ - two things i don't like, but i'm assuming have been discussed to death already
21:51:08 superdan I won't get back to it until monday
21:51:12 superdan but... cool
21:51:58 figleaf mriedem: I don't like 'required' or 'requires' either
21:52:17 superdan required= is the only way to go, IMHO
21:52:31 figleaf mriedem: heading out soon; will comment by Monday
21:58:39 mriedem ?required_foobars
22:05:44 mriedem ok seeing the bike shedding on this in the previous comments
22:06:41 fried_rice mriedem I abstain. I'd be fine with whatever y'all land on wrt those two points.
22:28:16 mriedem tested out GET /servers?status=''
22:28:28 mriedem it definitely has a behavior change from what the spec is proposing
22:28:39 mriedem it applies that as the filter, which means you don't get any instances back that have a vm_state set
22:41:08 fried_rice mriedem But does status= take a list?
22:41:23 fried_rice It doesn't appear to, in my trials.
22:41:59 fried_rice mriedem Which makes it apples:oranges compared to traits.
22:44:30 thingee mriedem: there are 69 total forum sessions
22:44:43 thingee mriedem: was waiting all day for that answer myself :)
22:45:55 mriedem ok and 73 proposed
22:46:11 mriedem so i guess chances are good to get stuff in, especially since there are some duplicates
22:46:33 thingee hey I'll cheers my beer to that
23:21:57 openstackgerrit Eric Fried proposed openstack/nova master: Use ksa adapter for cinder client https://review.openstack.org/508345
23:22:32 fried_rice mriedem Since you're the cinder guy? ^
23:37:43 superdan mriedem: they just restarted zuul with some fixes including one that makes it not do the thing your patch was trying to avoid
23:37:47 superdan so we might be unblocked
23:51:33 superdan mriedem: strike that.
#openstack-nova - 2017-09-30
01:25:14 openstackgerrit Liping Mao proposed openstack/nova stable/ocata: Stop _undefine_domain erroring if domain not found https://review.openstack.org/508640
01:26:04 openstackgerrit Liping Mao proposed openstack/nova stable/newton: Stop _undefine_domain erroring if domain not found https://review.openstack.org/508641
03:41:12 openstackgerrit Dan Smith proposed openstack/nova master: Use improved instance_list module in compute API https://review.openstack.org/505418
03:41:12 openstackgerrit Dan Smith proposed openstack/nova master: Fix a pagination logic bug in test_bug_1689692 https://review.openstack.org/505661
03:41:13 openstackgerrit Dan Smith proposed openstack/nova master: Fix minor input items from previous patches https://review.openstack.org/506416
03:41:13 openstackgerrit Dan Smith proposed openstack/nova master: Fix CellDatabases fixture swallowing exceptions https://review.openstack.org/506312
03:41:14 openstackgerrit Dan Smith proposed openstack/nova master: Merge build requests into the sortmaster 9000 https://review.openstack.org/508595
05:31:41 openstackgerrit Merged openstack/nova master: Move cell0 marker test to Cellsv1DeprecatedTestMixIn https://review.openstack.org/508314
09:45:45 openstackgerrit Lei Zhang proposed openstack/nova master: placement: add traits support to get resource_providers with shared https://review.openstack.org/478464
09:45:45 openstackgerrit Lei Zhang proposed openstack/nova master: placement: the common method to filter resource providers by traits https://review.openstack.org/489206
09:45:46 openstackgerrit Lei Zhang proposed openstack/nova master: placement: add traits to the 'GET /allocation_candidates' endpoint https://review.openstack.org/479776
09:45:46 openstackgerrit Lei Zhang proposed openstack/nova master: placement: add traits support in the AllocationCandidates object https://review.openstack.org/479766
09:45:47 openstackgerrit Lei Zhang proposed openstack/nova master: placement: extract traits from flavor extra spec https://review.openstack.org/492026
13:26:22 openstackgerrit Ed Leafe proposed openstack/nova-specs master: Return Alternate Hosts https://review.openstack.org/504275
14:33:48 openstackgerrit Lei Zhang proposed openstack/nova master: placement: add traits support to get resource_providers with shared https://review.openstack.org/478464
14:33:48 openstackgerrit Lei Zhang proposed openstack/nova master: placement: the common method to filter resource providers by traits https://review.openstack.org/489206
14:33:49 openstackgerrit Lei Zhang proposed openstack/nova master: placement: add traits to the 'GET /allocation_candidates' endpoint https://review.openstack.org/479776
14:33:49 openstackgerrit Lei Zhang proposed openstack/nova master: placement: add traits support in the AllocationCandidates object https://review.openstack.org/479766
16:11:16 openstackgerrit Matt Riedemann proposed openstack/nova master: Set migration object attributes for source/dest during live migrate https://review.openstack.org/508699
16:11:16 openstackgerrit Matt Riedemann proposed openstack/nova master: Refactor duplicate code for looking up the compute node name https://review.openstack.org/508604
16:11:44 mriedem superdan: ^ is the thing i was talking about with live migration not setting the source/dest nodes in the migration object, and my fix for that
16:12:04 mriedem also, looks like we have jobs failing with the "wait for compute host discovery by hostname" thing in devstack again...
16:12:23 superdan mriedem: okay, we are merging stuff again though
16:12:57 mriedem alright, i saw the backscroll in infra
16:13:03 mriedem i've rechecked a bunch of stuff this morning
16:14:28 dansmith cool
16:15:07 mriedem huh, wtf http://logs.openstack.org/16/507816/1/check/legacy-novaclient-dsvm-functional/806ee46/logs/screen-n-cpu.txt.gz#_Sep_30_16_08_29_812276
16:15:33 dansmith mriedem: your comment about the live migration thing is what I was saying, I think we're missing coverage on live migration failures after the task in conductor is done,
16:15:37 mriedem zuulv3 job reconfig might have f'ed up the novaclient functional jobs for newton
16:15:44 dansmith so I'm not actually calling that remove thing for live migrations yet
16:15:55 mriedem i also have a fix for that
16:15:58 mriedem brother man
16:16:15 mriedem https://review.openstack.org/#/c/507687/
16:16:29 mriedem ^ is also needed for pike
16:16:51 dansmith fix for what?
16:17:42 dansmith that patch doesn't add tests for allocation correctness after a fail
16:19:27 dansmith oh nevermind
16:19:31 dansmith I see the one below it now
16:19:53 mriedem yeah
16:20:00 mriedem fixes 2 bugs actually
16:20:52 mriedem i've also realized that our rollback live migration method is really odd, i.e. it does the notification stuff in the wrong places, and it sets the instance.task_state to None before it's done rolling stuff back, which is messy if we're polling the instance task_state at all
16:21:07 dansmith meh, live migrations never fail
16:41:39 openstackgerrit Matt Riedemann proposed openstack/python-novaclient stable/newton: Fix aggregate_update name and availability_zone clash https://review.openstack.org/507816
16:54:15 openstackgerrit Merged openstack/nova master: Fix a pagination logic bug in test_bug_1689692 https://review.openstack.org/505661
16:55:11 mriedem wow code is merging
17:35:59 efried_thbagh mriedem I slept on it.
17:56:43 openstackgerrit Merged openstack/nova master: Add recreate test for live migrate rollback not cleaning up dest allocs https://review.openstack.org/507677
22:42:51 openstackgerrit Moshe Levi proposed openstack/nova master: Don't overwrite binding-profile https://review.openstack.org/505613
#openstack-nova - 2017-10-01
09:04:26 Lehtu Hey guys, I updated today my setup from Ocata->Pike and now getting this error on "openstack server list" : ERROR (BadRequest): This service is older (v9) than the minimum (v22) version of the rest of the deployment. Unable to continue. (HTTP 400)
09:04:38 Lehtu I've run all db_sync commands and such
09:34:36 Lehtu Ahh nevermind, I had some deprecated services still at v9. Deleting them from DB solved the issue.
13:51:28 openstackgerrit Matt Riedemann proposed openstack/python-novaclient stable/newton: Fix aggregate_update name and availability_zone clash https://review.openstack.org/507816
23:48:04 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
#openstack-nova - 2017-10-02
00:31:31 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
04:24:10 openstackgerrit Hongbin Lu proposed openstack/nova master: [WIP] Fix wrapping of neutron Forbidden error https://review.openstack.org/508802
04:46:07 openstackgerrit Hongbin Lu proposed openstack/nova master: Fix wrapping of neutron forbidden error https://review.openstack.org/508802
07:42:35 openstackgerrit edan david proposed openstack/os-traits master: Add NIC Switchdev feature https://review.openstack.org/508817
08:40:19 openstackgerrit Merged openstack/nova-specs master: Request traits in Nova https://review.openstack.org/468797
08:41:31 openstackgerrit Ghanshyam Mann proposed openstack/nova-specs master: Spec for API extensions policy removal https://review.openstack.org/508101
09:18:26 openstackgerrit Merged openstack/nova master: Support qemu >= 2.10 https://review.openstack.org/505673
09:39:16 openstackgerrit Radoslav Gerganov proposed openstack/nova-specs master: VMware: add support for live migration https://review.openstack.org/299207
09:48:28 openstackgerrit sahid proposed openstack/nova master: pci: add for PciDevice object new field mdev https://review.openstack.org/508176
09:48:28 openstackgerrit sahid proposed openstack/nova master: pci: update PciDevice object field 'address' to accept NULL https://review.openstack.org/508175

Earlier   Later