Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-04-03
20:05:41 cdent jaypipes, efried: can we pretty please get forbidden traits merged before https://review.openstack.org/#/c/554529 ?
20:06:11 efried heh
20:06:52 efried cdent: I was actually just working up granular and thinking much the same thing. Would force jaypipes to make nested work for granular, rather than me having to do the reverse :P
20:07:04 cdent 'xactly
20:09:20 cfriesen andrewbogott: see subject line for the channel. that said, did you add the [api_database]/connection information to nova.conf?
20:09:43 andrewbogott cfriesen: Yep! But, I'll try over there.
20:16:28 andrewbogott cfriesen: is there a doc you can link me to that explains what that new database actually does? Is it just a cache?
20:24:26 cdent edleafe, jaypipes: i'm pretty sure the data model in https://review.openstack.org/#/c/557958/ (consumer generation) is wrong. even if I'm wrong about being wrong, then we need to make it clear in the commit why we're doing it weirdly.
20:27:59 cdent are we trying to log generation on the allocation as a sort of historical reference? that won't work because we delete all allocations for a cosumer when they change
20:28:16 cdent efried_schoolrun: you'll also be interested in ^
20:38:56 cfriesen andrewbogott: it's related to the cellsv2 work. the nova DB became per-cell, and the nova-api DB spans the cells and has higher-level stuff
20:39:21 andrewbogott ah, that makes sense, and might be a clue. thanks.
21:16:01 jaypipes cdent: you're absolutely right. no idea why I hadn't seen that. :(
21:16:48 cdent ETOOMUCHNUMA
21:18:17 cdent I think I'll use that as my cue to retire
21:18:19 cdent night all
21:31:45 openstackgerrit Merged openstack/nova master: Fix N332 api_version decorator hacking check https://review.openstack.org/557743
21:33:07 arvindn051 jaypipes: https://review.openstack.org/#/c/557795/ wanted to discuss comments on the review...good time?
21:34:54 jaypipes arvindn051: I need to re-review that one...
21:36:02 arvindn051 k
21:36:48 arvindn051 no rush...i am able to make progress on future patches...but wanted to get feedback on using a list of strings vs dict
21:45:24 openstackgerrit Michael Still proposed openstack/nova master: Move configurable mkfs to privsep. https://review.openstack.org/551921
21:45:24 openstackgerrit Michael Still proposed openstack/nova master: Move xenapi xenstore_read's to privsep. https://review.openstack.org/552241
21:45:25 openstackgerrit Michael Still proposed openstack/nova master: Move xenapi disk resizing to privsep. https://review.openstack.org/552242
21:45:25 openstackgerrit Michael Still proposed openstack/nova master: Sync xenapi and libvirt on what flags to pass e2fsck. https://review.openstack.org/554078
21:45:26 openstackgerrit Michael Still proposed openstack/nova master: Move xenapi partition copies to privsep. https://review.openstack.org/553605
21:45:26 openstackgerrit Michael Still proposed openstack/nova master: Move image conversion to privsep. https://review.openstack.org/554437
21:45:27 openstackgerrit Michael Still proposed openstack/nova master: We don't need utils.trycmd any more. https://review.openstack.org/554439
21:45:27 openstackgerrit Michael Still proposed openstack/nova master: We no longer need rootwrap. https://review.openstack.org/554438
21:58:32 arvindn051 jaypipes: thx for comments....i can revert the prefix
21:59:11 arvindn051 how do you propose we handle preferred traits, numbered traits etc in future?
21:59:26 arvindn051 seperate fields for each of them?
22:01:52 jaypipes arvindn051: well, preferred traits will have a value of "prefer" not "require", right?
22:02:35 arvindn051 but we changed the image meta prop to be a list of strings in the object
22:02:50 arvindn051 so no way to introduce prefer for the value...
22:03:08 jaypipes arvindn051: that's just the outward production of the stuff stored in the image_properties table in glance, though...
22:03:09 arvindn051 i am talking about the ojbect definition...not the image property key value
22:03:33 jaypipes arvindn051: the object definition is entirely temporal... it never saves to a DB.
22:05:25 arvindn051 yup...it just parses the properties into usable object format...i got that.
22:06:04 arvindn051 if we had to introduce preferred traits to this model, we would first add trait:foo=preffered in the image property
22:06:38 arvindn051 when we parse it into the object, we would need to introduce antoher field preferred_traits with these values right?
22:07:15 jaypipes arvindn051: yes, precisely.
22:07:36 arvindn051 how about if we do numbered traits like trait:CUSTOM_foo, trait1:foo etc...how would we model this here?
22:08:16 jaypipes arvindn051: don't worry about that.
22:08:45 arvindn051 my intial thought was to just parse the image properties as a dict into the object and have the consumer figure out which keys/values makes sense to him
22:08:50 jaypipes arvindn051: for now, all we need to do is produce the non-granular required traits as a list of trait strings.
22:10:04 arvindn051 is there a possiblity to support the need for granular traits in the future? if so, how would we handle is what i am thinking about
22:10:37 arvindn051 if you think there is no possiblity of this, then we can ignore and go ahead with having specific fields
22:10:55 jaypipes arvindn051: like I said, don't worry about it. :)
22:11:47 jaypipes arvindn051: there's nothing about the proposed solution here that prohibits granular traits being parsed from strings like "trait1:HW_CPU_X86_AVX2" in the future.
22:13:21 arvindn051 ok :) i guess we could introduce a granular_traits dict of strings fields in the future if needed...
22:13:35 arvindn051 thanks...will make the updates in a couple of mins
22:23:05 openstackgerrit Arvind Nadendla proposed openstack/nova master: Update ImageMetaProp object to expose traits https://review.openstack.org/557795
22:24:51 arvindn051 jaypipes: made the changes...should be good now
22:28:32 jaypipes arvindn051: cool, thank you sir.
23:02:24 edleafe jaypipes: sorry, was at a doc appt. So we don't want the consumer_generation in the Allocation at all? That's cool, and I think that was my original idea, as written in the spec. I don't remember the conversation (there are so many) where it was requested.
23:14:59 openstackgerrit melanie witt proposed openstack/nova master: Default to py3 for the pep8 tox env because it's stricter https://review.openstack.org/558648
23:32:31 openstackgerrit melanie witt proposed openstack/nova master: Default to py3 for the pep8 tox env because it's stricter https://review.openstack.org/558648
23:59:52 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Fix AttributeError in getting a resource ID https://review.openstack.org/558328
#openstack-nova - 2018-04-04
00:01:16 tetsuro good morning!
00:09:08 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Replace GB with GiB https://review.openstack.org/558364
01:18:49 openstackgerrit Merged openstack/nova master: [placement] Add test for provider summaries https://review.openstack.org/558044
01:18:59 openstackgerrit Merged openstack/nova master: Unit test framework: common FakeResponse https://review.openstack.org/556551
01:19:15 openstackgerrit Merged openstack/nova master: [placement] Add to contributor docs about handler testing https://review.openstack.org/557355
01:21:04 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Noauth should also use request_id from compute_req_id.py https://review.openstack.org/555266
02:19:09 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240
02:19:19 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340
02:19:32 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344
02:58:02 openstackgerrit Yikun Jiang (Kero) proposed openstack/osc-placement master: Should we remove 'provider' in resource provider allocation https://review.openstack.org/558682
03:19:53 openstackgerrit Jay Pipes proposed openstack/nova-specs master: Handle nested providers for allocation candidates https://review.openstack.org/556873
03:21:06 openstackgerrit Takashi NATSUME proposed openstack/nova master: Replace mox stubs with stub_out https://review.openstack.org/558685
03:22:38 openstackgerrit Takashi NATSUME proposed openstack/nova master: Replace mox stubs with stub_out in test_extended_volumes.py https://review.openstack.org/558685
03:25:16 openstackgerrit Jay Pipes proposed openstack/nova-specs master: mirror nova host aggregates to placement API https://review.openstack.org/545057
03:44:37 openstackgerrit Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
03:44:37 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
03:57:04 openstackgerrit Merged openstack/nova master: api-ref: add a note about volume-backed rescue not being supported https://review.openstack.org/556996
03:57:38 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387
03:57:39 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
03:57:39 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240
03:57:40 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340
03:57:40 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344
04:04:41 openstackgerrit Merged openstack/nova master: ironic: Get correct inventory for deployed node https://review.openstack.org/553367
05:32:22 openstackgerrit jichenjc proposed openstack/nova master: Avoid showing password in log https://review.openstack.org/558694
06:03:31 openstackgerrit jichenjc proposed openstack/nova master: Remove a outdated warning https://review.openstack.org/557898
06:16:11 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/virt/xenapi/test_vm_utils.py (1) https://review.openstack.org/558704
06:49:19 openstackgerrit sahid proposed openstack/nova-specs master: update: isolate guests emulthreads on CONF.cpu_shared_set https://review.openstack.org/511188
06:49:21 openstackgerrit Merged openstack/nova master: PowerVM Driver: Network interface attach/detach https://review.openstack.org/546813
06:57:49 openstackgerrit sahid proposed openstack/nova-specs master: update: isolate guests emulthreads on CONF.cpu_shared_set https://review.openstack.org/511188
07:06:22 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/548772
07:45:54 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387
07:45:55 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
07:45:55 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240
07:45:56 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340
07:45:56 openstackgerrit jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344
07:53:26 openstackgerrit jichenjc proposed openstack/nova master: Avoid showing password in log https://review.openstack.org/558694
07:53:29 kashyap johnthetubaguy: Morning; this is ready, if it's merged, can get going with the backports: https://review.openstack.org/#/c/534384/
08:01:03 gmann johnthetubaguy: alex_xu_ gibi_away looking for feedback on this - https://review.openstack.org/#/c/547850/
09:46:41 openstackgerrit sahid proposed openstack/nova-specs master: update: isolate guests emulthreads on CONF.cpu_shared_set https://review.openstack.org/511188
09:50:18 naichuans Hi, dansmith, do you have the time to review below patches: https://review.openstack.org/#/c/554154/, https://review.openstack.org/#/c/489451/. It all about use xapi-pool instead of aggregate pool. We have finished internal review and the test. Thank you very much.
09:59:57 openstackgerrit Rajesh Tailor proposed openstack/nova master: Fix host validity check for live-migration https://review.openstack.org/401009

Earlier   Later