Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-04-28
01:22:20 openstackgerrit Jay Pipes proposed openstack/nova master: placement: get providers sharing capacity https://review.openstack.org/460798
01:22:28 jaypipes alex_xu, mriedem, dansmith, edleafe, bauzas: ^^
01:22:46 jaypipes more of dansmith's favourite SQL-fu.
01:25:27 openstackgerrit Matthew Treinish proposed openstack/nova master: Remove nova-cert https://review.openstack.org/459460
01:46:37 openstackgerrit Takashi NATSUME proposed openstack/nova master: Avoid forcing translation on logging calls https://review.openstack.org/413876
01:46:46 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix examples for add/removeFixedIp action https://review.openstack.org/456053
01:46:57 openstackgerrit Takashi NATSUME proposed openstack/nova master: Refactor a test method including 7 test cases https://review.openstack.org/395436
02:04:10 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(2/4) https://review.openstack.org/459569
02:06:49 openstackgerrit Dao Cong Tien proposed openstack/nova master: Stop using mox int unit/virt/xenapi/image/test_utils.py https://review.openstack.org/342626
02:10:11 openstackgerrit Kiseok Kim proposed openstack/nova master: Fixed a broken link in API Plugins document https://review.openstack.org/460398
02:28:10 openstackgerrit Matthew Treinish proposed openstack/nova master: Remove nova-cert https://review.openstack.org/459460
03:25:52 mriedem jaypipes: ok hon
03:25:58 mriedem jaypipes: it's gonna have to be tomorrow at this point
03:26:06 mriedem but i did get your libvirt get_inventory patch reviewed today
03:26:08 mriedem with gusto!
03:27:37 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for limits endpoint instead of stevedore https://review.openstack.org/460813
04:03:31 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for os-services endpoint instead of stevedore https://review.openstack.org/460818
04:06:33 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for limits endpoint instead of stevedore https://review.openstack.org/460813
04:16:00 openstackgerrit jichenjc proposed openstack/nova master: e plain routes list for os-simple-tenant-usage https://review.openstack.org/460821
04:23:50 openstackgerrit jichenjc proposed openstack/nova master: use plain routes list for os-simple-tenant-usage https://review.openstack.org/460821
04:26:28 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for os-certificates endpoint instead of stevedore https://review.openstack.org/460813
05:15:37 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for os-services endpoint instead of stevedore https://review.openstack.org/460818
06:04:01 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for os-certificates endpoint instead of stevedore https://review.openstack.org/460813
06:08:20 openstackgerrit jichenjc proposed openstack/nova master: Use plain routes list for os-certificates endpoint instead of stevedore https://review.openstack.org/460813
06:11:42 openstackgerrit jichenjc proposed openstack/nova master: use plain routes list for os-simple-tenant-usage https://review.openstack.org/460821
06:47:48 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Amend the "list-show-all-server-migration-types" spec https://review.openstack.org/460847
07:40:43 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix unnecessary description in servers-admin-action https://review.openstack.org/460866
07:54:50 openstackgerrit Takashi NATSUME proposed openstack/nova master: WIP: Specify keymap on server boot (1/2) https://review.openstack.org/458752
08:49:46 slaweq_ hello, I have a question about old Juno release of Nova
08:50:12 slaweq_ I'm investigating why pci_device connected to instance is "duplicated" during migration
08:50:37 slaweq_ I found that it's probably because of lazy loading of pci_devices attribute in instance object
08:51:04 slaweq_ do You maybe know if there was such bug in Juno, maybe it's already fixed in newer releases? :)
09:05:35 openstackgerrit jichenjc proposed openstack/nova master: Trivial fix: fix broken links https://review.openstack.org/460883
09:06:55 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
10:36:13 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'log' commands https://review.openstack.org/453176
10:36:14 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'host' commands https://review.openstack.org/453175
10:36:14 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'project', 'account' commands https://review.openstack.org/453174
10:36:15 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'agent' commands https://review.openstack.org/450795
10:37:20 sfinucan sdague: Could you take a look at those (and the other two patches in the series) at some point, please? ^
10:40:07 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Attach and detach encryptors during swap_volume https://review.openstack.org/460244
10:49:48 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove flavor-rxtx policy https://review.openstack.org/444036
10:50:56 sfinucan alex_xu: Could you take a look at https://review.openstack.org/#/c/444036, please? Would like to know if there's any reason to keep a separate policy for an API extension around (I don't think so)
10:53:40 johnthetubaguy sfinucan: I would rather we deprecate that kind of thing, people could be using that
10:54:09 sfinucan johnthetubaguy: The policy or the API endpoint?
10:55:02 sfinucan If the former, have we a procedure for deprecating API policies? I was following the precedent of https://review.openstack.org/#/c/459608 and just removing them
10:56:00 johnthetubaguy sfinucan: thats different to one that actually has a really effect
10:56:05 johnthetubaguy real
10:56:52 sfinucan I'm still trying to parse how that extension works, but I don't think it has
10:57:22 johnthetubaguy sfinucan: it leaks information to the user about the rxtx limits on a given flavor
10:57:27 sfinucan All the API samples for flavors-detail etc. show a 'rxtx_factor' parameter
10:57:33 sfinucan e.g. doc/api_samples/flavors/flavors-detail-resp.json
10:57:36 johnthetubaguy sfinucan: or rather you can hide that limit from the user
10:57:48 johnthetubaguy sfinucan: correct, the default policy allows you to see that
10:58:09 sfinucan ah, so if we defaulted to false those API samples would need to be updated
10:58:39 johnthetubaguy sfinucan: I believe so
10:59:23 johnthetubaguy sfinucan: its in the change: https://review.openstack.org/#/c/444036/13/nova/api/openstack/compute/flavor_rxtx.py
11:00:54 johnthetubaguy sfinucan: now there is a slightly stupid thing, there is nothing in tree that actually uses that property, AFAIK
11:02:36 sfinucan I had it in my head that a controller == a URL/endpoint though. I was expecting to see something FlavorRxtxController map to '/flavors/{flavorID}/rxtx' or something
11:03:21 johnthetubaguy sfinucan: thats an extension to an existing controller
11:03:48 sfinucan yup - literally *just* saw the @wsgi.extends decorator :) This is starting to make sense
11:03:59 johnthetubaguy its mid evolution in there, it used to be clearer
11:04:13 johnthetubaguy they are all getting rolled into the main API code this cycle
11:04:22 johnthetubaguy to stop the... crazy
11:06:13 openstackgerrit Artom Lifshitz proposed openstack/nova master: Pass APIVersionRequest to extensions https://review.openstack.org/425876
11:06:56 openstackgerrit Takashi NATSUME proposed openstack/nova master: WIP: Specify keymap on server boot (2/2) https://review.openstack.org/460987
11:09:23 sfinucan johnthetubaguy: Hmm, looking at the various '_flavor_get_from_db' functions in 'nova/objects/flavor.py', I don't see anything that would prevent the 'rxtx_factor' being populated in the standard response
11:09:34 sfinucan This stuff is slightly confusing indeed :)
11:09:53 sfinucan ...after lunch
11:10:19 johnthetubaguy sfinucan: it could be broken, I bet there are no tests for all that
11:17:22 openstackgerrit John Garbutt proposed openstack/nova master: Add description to policies in migrations.py https://review.openstack.org/459727
11:30:54 zioproto I have a nova bug, I am not sure if I explained by self correctly: https://bugs.launchpad.net/nova/+bug/1686999
11:30:54 openstack Launchpad bug 1686999 in OpenStack Compute (nova) "Instance with two attached volumes fails to start with error: Duplicate ID 'drive-ide0-0-0' for drive" [Undecided,New]
12:13:40 sfinucan johnthetubaguy: I _think_ that controller did nothing
12:13:58 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'os-flavor-rxtx' policy https://review.openstack.org/444036
12:13:58 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'FlavorRxtxController' extension https://review.openstack.org/461010
12:13:59 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove useless API tests https://review.openstack.org/461011
12:14:42 sfinucan So I've written those ^^^ to test as much (in the gate - weird "no db" issues if I try running 'tox -e api-samples' locally. If so, we can delete the policy as also doing nothing, IMO
12:44:31 mriedem o/
12:45:42 gibi mriedem: hi! I think I figured out what happens with flavor-delete in https://review.openstack.org/#/c/460377 , I left comments in the patch
12:46:37 mriedem gibi: ah thanks
12:46:46 mriedem i must have been looking at the wrong main db api method
12:48:02 mriedem gibi: i'm not sure about changing it from [] to None
12:48:14 mriedem i'd have to change one of the notification samples at least
12:48:51 gibi mriedem: yes, but I think it is bugfix on that sample
12:49:08 gibi mriedem: before your patch flavor-delete always reported projects=[]
12:49:20 gibi mriedem: even if the flavor had some projects associated before the delete
12:49:26 gibi mriedem: and that is missleading
12:49:40 gibi mriedem: we use None (null) value in the notification to signal if data is missing
12:49:42 mriedem yeah
12:49:44 mriedem ok,
12:49:55 mriedem then i think i'll probably pull that part of the change out and do that first
12:50:07 gibi mriedem: sure, that would be cleaner
13:01:03 johnthetubaguy mriedem: was Nova v1.0 really a thing, I thought only the EC2 API was used back then?
13:01:55 mriedem johnthetubaguy: i found it in the 2010.1 release code in launchpad
13:02:14 mriedem https://launchpad.net/nova/austin/2010.1
13:02:16 johnthetubaguy mriedem: curious
13:02:44 mriedem nova/api/__init__.py
13:03:01 mriedem http://paste.openstack.org/show/608325/
13:06:03 mriedem gibi: oh, it can't be None
13:06:07 mriedem flavor.projects is not a nullable field

Earlier   Later