Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-09-22
01:49:48 mriedem there is a surprise
01:50:20 openstackgerrit Dan Smith proposed openstack/nova master: Fix minor input items from previous patches https://review.openstack.org/506416
01:50:28 dansmith mriedem: the mogan stuff?
01:50:32 dansmith melwitt: ^
01:50:42 mriedem no
01:50:44 mriedem better
01:50:50 mriedem an old set of friends
01:50:54 mriedem coming together at last
01:51:09 melwitt dansmith: cool, thanks
01:52:31 melwitt I was thinking between the tests earlier in the series to now, everything stitches together, but wasn't totally sure
01:52:59 dansmith melwitt: yeah, I think it's all the same enough that it wouldn't matter, but good to have them in there
01:54:37 melwitt lol mriedem
01:55:22 dansmith oh, the shelve and file injection thing?
01:55:28 mriedem yeah
01:55:42 mriedem oh come on
01:55:53 melwitt I chuckled
01:55:56 mriedem shelve is one of my most favorite things to shit on in the ML on a semi-monthly basis
01:56:05 mriedem melwitt: you said you "lol"ed
01:56:14 melwitt I chuckled out loud
01:56:28 mriedem ok
01:56:29 melwitt a chuckle is a laugh right?
01:56:32 dansmith issubclass('chuckle', 'laugh') == True
01:56:36 mriedem ok
01:56:41 mriedem didn't know the rules
01:56:41 melwitt haha yeeeah
01:56:48 dansmith rules ^
01:56:58 mriedem you know,
01:57:09 mriedem if we allowed you to just specify personality files on unshelve...
01:57:13 mriedem basically,
01:57:26 dansmith you know, if we took shelve and file injection out back and set them on fire...
01:57:29 mriedem if we just allowed you to specify everything you can specify on initial create, but on rebuild and unshelve, we'd be good
01:57:42 melwitt well yeah, that's what everyone wants
01:57:42 mriedem well we're doing that with file injection this release, i just need to spec that up
01:58:01 mriedem maybe shelve/unshelve should be a topic at the forum
01:58:06 mriedem "can we please just remove this? please?"
01:58:25 dansmith getting input on who actually uses it and for what seems fairly legit to me, fwiw
01:58:40 dansmith like if nobody cares if it keeps working, I care a lot less than I already do
01:58:54 dansmith and I care very little
01:59:05 mriedem yeah we still kick that idea around on figuring out who is using the APIs
01:59:08 mriedem maybe that's a brainstormer
01:59:48 melwitt based on the amount of bugs I see with it, I will guess a fair number of people use it
01:59:56 melwitt *bug reports
02:00:28 dansmith I want to know how many people use it for real, vs. use tempest to verify their cloud and continually find it broken and think it's worth reporting
02:00:45 dansmith but, also getting 'why do you want to use this' answered is useful I think
02:00:53 dansmith because if they're using it instead of snapshot for dumb reasons...
02:00:56 melwitt yeah. just musing
02:01:34 melwitt talkin it up
02:01:38 melwitt talkin bout topics
02:03:15 mriedem the tempest tests are super simple too
02:04:25 openstackgerrit wanghongtaozz proposed openstack/nova master: Fix typos https://review.openstack.org/506437
02:04:47 dansmith oooh, typos
02:05:03 dansmith fix _all_ the typos
02:06:19 mriedem -1'ed with fury
02:07:22 mriedem https://review.openstack.org/#/q/owner:wanghongtaozz+status:open
02:07:23 mriedem a nice pattern
02:08:07 melwitt ... yeah
02:17:52 openstackgerrit Merged openstack/nova master: Add db.instance_get_by_sort_filters() https://review.openstack.org/504985
02:27:39 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Remove allocations when unshelve fails on host https://review.openstack.org/506458
03:17:48 alex_xu dansmith: just found the default dir is 'desc', not the 'asc', the confusing thing is we pass a different different_dir to process_sort_params...https://review.openstack.org/#/c/504983/8/nova/compute/instance_list.py@87
03:19:13 dansmith alex_xu: I don't think it matters anyway, right?
03:20:30 dansmith some default sort order helps for consistency, but if the user didn't ask for something specific I don't think we have any requirement to give them something specific
03:22:57 alex_xu dansmith: yes, the only thing is the default order changed, the API behaviour should be stable?
03:24:04 dansmith alex_xu: if you think it's important we can change it to desc by default, but I don't think it matters
03:24:18 dansmith if you don't ask for a sort order, I don't think you can complain about which way we sort it :)
03:24:34 dansmith sorting desc by default also seems weird to me, but..
03:26:31 alex_xu dansmith: emm...only can complain is the api-ref talk about default order https://developer.openstack.org/api-ref/compute/#id4 :)
03:27:16 dansmith alex_xu: where? I don't see "sort" or "order" on that page
03:27:42 dansmith the default sort includes db id, so I hope we're not saying that in our api ref :)
03:28:12 alex_xu dansmith: the description of sort_key, "Sorts by a server attribute. Default attribute is created.", but yea, there is mistake, it is created_at, not created
03:28:27 alex_xu dansmith: yea, good luck, we didn't mention that :)
03:29:02 dansmith alex_xu: I don't see where you see that in that doc
03:29:30 alex_xu dansmith: the request parameters of https://developer.openstack.org/api-ref/compute/#list-servers
03:30:00 dansmith okay I see it
03:30:02 alex_xu no anchor to specific parameter :(
03:30:14 dansmith still, it says desc by created (at)
03:30:20 dansmith I wish we hadn't codified that :/
03:30:51 dansmith anyway, I'll change it back to desc in the fixes patch at the end of that series
03:34:11 alex_xu dansmith: ok, thanks, it cure my OCD
03:34:28 dansmith heh
04:11:40 openstackgerrit Michael Still proposed openstack/nova master: Use privsep.path in create_ploop_image https://review.openstack.org/504429
05:09:10 openstackgerrit Rajesh Tailor proposed openstack/nova master: Host addition host-aggregate should be case-sensitive https://review.openstack.org/498334
06:20:14 openstackgerrit Merged openstack/nova master: Support pagination in instance_list https://review.openstack.org/504986
06:46:06 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Mention API behavior change when over quota limit https://review.openstack.org/506544
07:26:41 openstackgerrit Rajesh Tailor proposed openstack/nova master: Host addition host-aggregate should be case-sensitive https://review.openstack.org/498334
07:49:21 openstackgerrit Zhenyu Zheng proposed openstack/nova-specs master: Add API to allow update keypair for instance https://review.openstack.org/506552
07:57:24 openstackgerrit Elod Illes proposed openstack/nova master: Add instance.interface_detach notification https://review.openstack.org/506284
07:59:54 openstackgerrit Zhenyu Zheng proposed openstack/nova-specs master: Add API to allow update keypair for instance https://review.openstack.org/506552
08:19:01 ioggstream hi@all
08:41:37 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Add datapath type information to OVS vif objects https://review.openstack.org/474892
08:51:33 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257
08:51:34 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777
08:51:34 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051
08:52:41 ratailor alex_xu, Can you pl review https://review.openstack.org/#/c/504885/
08:54:02 alex_xu ratailor: can we do collate change online?
08:55:43 ratailor alex_xu, I am not sure about that whether it would work online or not.
09:00:23 alex_xu I remember that will lead to temorary table copy
09:00:43 alex_xu ratailor: maybe jaypipes is right person to answer that
09:01:08 ratailor alex_xu, Thanks, will try to contact him.
09:04:15 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add memoize function using oslo.cache https://review.openstack.org/472773
09:24:17 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451

Earlier   Later