Friday, 2018-06-15

openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (9)  https://review.openstack.org/57558101:16
openstackgerritjiang wei proposed openstack/nova master: Add action initiator attribute to the update payload  https://review.openstack.org/53624301:27
openstackgerritZhenyu Zheng proposed openstack/nova master: Add full traceback to ExceptionPayload in versioned notifications  https://review.openstack.org/56409201:39
*** mriedem has quit IRC02:07
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver  https://review.openstack.org/52338702:18
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver  https://review.openstack.org/52765802:18
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add snapshot function  https://review.openstack.org/53424002:18
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add power actions  https://review.openstack.org/54334002:18
openstackgerritjichenjc proposed openstack/nova master: z/VM Driver: add get console output  https://review.openstack.org/54334402:18
openstackgerritZhenyu Zheng proposed openstack/nova master: Use ThreadPoolExecutor for max_concurrent_live_migrations  https://review.openstack.org/56350502:46
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc  https://review.openstack.org/52952002:57
openstackgerritChen proposed openstack/nova master: Add supplementary info for simple_cell_setup cmd  https://review.openstack.org/57559903:26
openstackgerritMerged openstack/nova master: Add troubleshooting item about ignored microversions  https://review.openstack.org/57550603:33
*** tetsuro has joined #openstack-placement04:05
openstackgerritTony Breeds proposed openstack/nova stable/pike: [Stable Only] Initialise failed_builds in IronicNodeState  https://review.openstack.org/57561204:26
openstackgerritTony Breeds proposed openstack/nova stable/pike: [Stable Only] Initialise failed_builds in IronicNodeState  https://review.openstack.org/57561204:26
openstackgerritMerged openstack/nova master: Add certificate validation docs  https://review.openstack.org/56015804:33
openstackgerritMerged openstack/nova master: rework allocation handler _allocations_dict()  https://review.openstack.org/56540704:49
openstackgerritMerged openstack/nova master: Provide a direct interface to placement  https://review.openstack.org/57257604:52
openstackgerritTetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree  https://review.openstack.org/55948005:01
openstackgerritTetsuro Nakamura proposed openstack/nova master: Add microversion for nested allocation candidate  https://review.openstack.org/56548705:01
openstackgerritTetsuro Nakamura proposed openstack/nova master: Fix nits in placement-return-all-resources series  https://review.openstack.org/57360405:01
openstackgerritTakashi NATSUME proposed openstack/nova master: Update admin/flavors document  https://review.openstack.org/57306305:09
*** nicolasbock has joined #openstack-placement05:14
*** nicolasbock has quit IRC05:18
*** tetsuro has quit IRC05:45
*** tetsuro has joined #openstack-placement05:51
openstackgerritZhenyu Zheng proposed openstack/nova master: Compute: add support to abort queued live migration  https://review.openstack.org/56854206:24
*** tssurya has joined #openstack-placement06:40
openstackgerritLee Yarwood proposed openstack/nova stable/queens: ironic: stop lying to the RT when ironic is down  https://review.openstack.org/57562806:53
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (9)  https://review.openstack.org/57558107:09
*** e0ne has joined #openstack-placement07:11
openstackgerritLei Zhang proposed openstack/nova master: Add method to get cpu traits  https://review.openstack.org/56031707:11
*** ttsiouts has joined #openstack-placement07:17
openstackgerritjichenjc proposed openstack/nova master: remove ec2 object definitions  https://review.openstack.org/55715007:22
openstackgerritjichenjc proposed openstack/nova master: remove ec2 db functions  https://review.openstack.org/55757207:22
*** ttsiouts has quit IRC07:26
*** ttsiouts has joined #openstack-placement07:26
*** takashin has left #openstack-placement07:32
openstackgerritZhenyu Zheng proposed openstack/nova master: API: add support to abort queued live migration in microversion 2.63  https://review.openstack.org/57313607:41
*** ttsiouts has quit IRC07:42
*** ttsiouts has joined #openstack-placement07:46
openstackgerritZhenyu Zheng proposed openstack/nova master: Use ThreadPoolExecutor for max_concurrent_live_migrations  https://review.openstack.org/56350507:53
*** ttsiouts has quit IRC08:03
*** ttsiouts has joined #openstack-placement08:03
openstackgerritkarim proposed openstack/nova master: Handle rebuild of instances with image traits  https://review.openstack.org/56949808:10
*** ttsiouts has quit IRC08:13
*** ttsiouts has joined #openstack-placement08:19
openstackgerritZhenyu Zheng proposed openstack/nova master: Add full traceback to ExceptionPayload in versioned notifications  https://review.openstack.org/56409208:20
openstackgerritMerged openstack/nova master: Re-using the code of os brick cinder  https://review.openstack.org/56570308:28
*** bauzas is now known as PapaOurs08:30
*** ttsiouts has quit IRC08:32
*** ttsiouts has joined #openstack-placement08:32
openstackgerritZhenyu Zheng proposed openstack/nova master: Add full traceback to ExceptionPayload in versioned notifications  https://review.openstack.org/56409208:37
*** tetsuro has quit IRC09:06
openstackgerritMerged openstack/nova master: Make check_can_live_migrate_destination use long_rpc_timeout  https://review.openstack.org/57549309:55
*** cdent has joined #openstack-placement10:09
*** jaypipes is now known as leakypipes10:24
*** ttsiouts has quit IRC10:44
openstackgerritMerged openstack/nova master: Update nova-status & docs: require placement 1.25  https://review.openstack.org/57514011:12
*** yikun has quit IRC11:43
*** ttsiouts has joined #openstack-placement11:43
*** edmondsw has joined #openstack-placement12:01
*** e0ne has quit IRC12:08
*** e0ne has joined #openstack-placement12:08
cdentp!spy12:17
*** gibi is now known as giblet12:19
gibletcdent: thanks for making me remember that it is Friday :)12:19
cdentmy pleasure, although I have no idea how I did that12:21
* cdent needs food, biab12:24
*** e0ne has quit IRC12:26
openstackgerritChris Dent proposed openstack/nova master: Optional separate database for placement API  https://review.openstack.org/36276612:26
openstackgerritChris Dent proposed openstack/nova master: Isolate placement database config  https://review.openstack.org/54143512:26
openstackgerritChris Dent proposed openstack/nova master: Ensure that os-traits sync is attempted only at start of process  https://review.openstack.org/55385712:26
openstackgerritChris Dent proposed openstack/nova master: Add PLACEMENT_DB_ENABLED=True to the nova-next job  https://review.openstack.org/56406712:27
openstackgerritJulia Kreger proposed openstack/nova master: virt/ironic: Implement rescue and unrescue  https://review.openstack.org/41648712:30
*** stephenfin is now known as finucannot12:32
openstackgerritMatt Riedemann proposed openstack/nova master: Remove max_size parameter from fake_libvirt_utils.fetch_*image methods  https://review.openstack.org/57491112:38
openstackgerritMatt Riedemann proposed openstack/nova master: Fix nits from trusted certs notification change  https://review.openstack.org/57552112:38
openstackgerritMatt Riedemann proposed openstack/nova master: Add trusted certs to feature support matrix docs  https://review.openstack.org/57489012:38
*** efried is now known as fried_rice12:40
fried_ricewoot, placement direct merged12:41
*** ttsiouts has quit IRC12:41
*** ttsiouts has joined #openstack-placement12:42
fried_ricecdent: We maybe want to let cinder know12:42
smcginnisfried_rice: Can you give me the tl;dr on what the means?12:44
fried_ricesmcginnis: I was just about to poke you in -cinder :)12:44
smcginnisfried_rice: Oh great. Yeah, there would probably be better so other folks see.12:45
fried_ricesmcginnis: I was just trying to lay hands on the etherpad from YVR.  Do you have it at your fingertips?12:45
smcginnisLooking...12:46
smcginnishttps://etherpad.openstack.org/p/YVR-cinder-placement12:46
fried_ricesmcginnis: got it.12:46
fried_ricethanks12:46
*** edleafe is now known as figleaf12:48
*** rubasov has quit IRC13:01
*** tssurya is now known as sususuryashines13:02
openstackgerritSurya Seetharaman proposed openstack/nova master: Make nova list ignore down cells  https://review.openstack.org/57573413:35
openstackgerritGeorg Hoesch proposed openstack/nova master: handle logfiles for tcp-based consoles  https://review.openstack.org/57573513:36
*** mriedem has joined #openstack-placement13:45
openstackgerritDan Smith proposed openstack/nova master: Update the disk_cachemodes to mention an rbd detail  https://review.openstack.org/57574013:45
*** dansmith is now known as superdan13:47
cdentfried_rice, smcginnis: I reckon it would be bad news for cinder to start from placement direct. Better would be for cinder to figure out how to use placement "normally", and then for the standalone special case, consider placement direct or something like it.13:52
cdentplacement direct, as currently written, is really for special cases only, for use in extremis only13:52
fried_ricecdent: I hope I expressed that caveat sufficiently in my email - if not, please reply with stronger wording.13:53
smcginnisOK, that makes sense.13:53
cdentI haven't read any email yet since coming back from lunch13:53
smcginnisJust read it - makes sense.13:56
openstackgerritBalazs Gibizer proposed openstack/nova master: Add full traceback to ExceptionPayload in versioned notifications  https://review.openstack.org/56409214:01
cdentI _really_ do not want to see use of placement without a server as anything close to normal14:02
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_info field  https://review.openstack.org/56444114:08
*** rubasov has joined #openstack-placement14:22
openstackgerritEric Fried proposed openstack/nova master: Enforce placement minimum in nova.cmd.status  https://review.openstack.org/56896514:37
fried_ricemriedem, melwitt: Thanks for the review on ^  -- Manual rebase takes PlacementDirect (https://review.openstack.org/#/c/572576/) into account.14:37
mriedemfried_rice: +214:39
fried_ricemriedem: thanks14:39
openstackgerritMerged openstack/nova stable/queens: mock utils.execute() in qemu-img unit test  https://review.openstack.org/57425514:42
*** ttsiouts has quit IRC14:43
cdentgiblet, fried_rice : https://review.openstack.org/#/c/362766/ is a similar "take placement direct into account rebase" you were +A on that before14:44
fried_riceadmit it cdent, you're just going for the record.14:45
fried_riceI think you have like 7 patch sets to go.14:45
cdenthuh, I figured I had much longer than that14:45
*** ttsiouts has joined #openstack-placement14:46
fried_riceWell, we don't have an official score, but someone asked in -nova the other day and we found a couple in the hundred-teens.14:46
cdentfried_rice: you can squelch my quest by approving that code now. my heart will break in twain14:46
fried_ricecdent: If I approve yours, then mine will bounce.14:47
cdenteven better14:47
cdentbiab14:47
fried_ricegiblet: Manual rebase on ^ is trivial, you want a crack at it or should I fast-approve?14:48
*** ttsiouts has quit IRC14:48
gibletfried_rice: go ahead, I'm in the trusted_cert business now14:48
*** ttsiouts has joined #openstack-placement14:49
fried_riceack14:49
fried_ricecdent: Reapproved the ones what needed it.14:50
*** mriedem has quit IRC14:54
*** mriedem has joined #openstack-placement14:54
*** rubasov has quit IRC15:03
*** e0ne has joined #openstack-placement15:06
cdentthanks fried_rice . placement update finally done. so long. especially when standing.15:09
fried_ricecdent: Yup, reading it now, so far so good.15:09
openstackgerritZack Cornelius proposed openstack/nova master: DNM: Libvirt file backed memory  https://review.openstack.org/57576615:16
openstackgerritBalazs Gibizer proposed openstack/nova master: Add full traceback to ExceptionPayload in versioned notifications  https://review.openstack.org/56409215:24
openstackgerritMatt Riedemann proposed openstack/nova master: Add policy rule to block image-backed servers with 0 root disk flavor  https://review.openstack.org/56128415:25
*** ttsiouts has quit IRC15:28
cdentfried_rice: https://www.youtube.com/watch?v=B436avtEXzs15:42
fried_ricecdent: What is the term for a REST verb + path?  E.g. "GET /resource_providers/{uuid}".  Is that called a "route"?15:48
fried_ricefigleaf: ^15:51
openstackgerritMerged openstack/nova master: Remove max_size parameter from fake_libvirt_utils.fetch_*image methods  https://review.openstack.org/57491115:52
fried_riceAccording to https://developer.wordpress.org/rest-api/extending-the-rest-api/routes-and-endpoints/ the /resource_providers/{uuid} bit is the "route" and the verb+route is called an endpoint.  Which is backwards from what I thought.  <== finucannot re https://review.openstack.org/#/c/477478/3/nova/api/openstack/placement/rest_api_version_history.rst15:55
finucannotfried_rice: I had totally forgotten about that patch15:56
fried_ricefinucannot: Thanks to cdent's placement email for that :)15:56
cdenthuh. I never would have called method + URI an endpoint15:57
cdentprobably because I'm one of those idealists who cares only for URIs and whatever methods those URIs is based on what works, and not prior knowledge15:57
cdenthttp://blog.whatfettle.com/2008/10/06/the-uri-is-the-thing/15:58
cdentbigger version of the image at https://www.flickr.com/photos/psd/2918889380/15:58
cdent(with annotations)15:58
openstackgerritMatt Riedemann proposed openstack/nova master: Add policy rule to block image-backed servers with 0 root disk flavor  https://review.openstack.org/56128415:59
cdentnever much cared for "route" either, but it has proven too useful in code to avoid15:59
fried_ricecdent: Well, that was just one reference I found; there doesn't seem to be a consensus.15:59
fried_ricehttp://apiglossary.com/#Endpoint16:00
fried_ricehttps://stackoverflow.com/questions/18588318/rest-whats-the-name-for-the-http-verb-and-endpoint  (tl;dr, method+endpoint = "operation")16:01
cdentming chan's answer is not very resource oriented at all16:03
cdentsince it is operation oriented16:03
cdentbut it doesn't really matter (to me)16:03
* cdent finds a seat16:03
fried_riceI like "operation" - it's descriptive and pretty unambiguous.16:04
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add RequestSpec.numa_networks  https://review.openstack.org/56444216:20
openstackgerritStephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.numa_networks  https://review.openstack.org/56445216:20
openstackgerritStephen Finucane proposed openstack/nova master: network: Remove 'create_pci_requests_for_sriov_ports'  https://review.openstack.org/57548816:20
openstackgerritStephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options  https://review.openstack.org/56444016:20
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_info field  https://review.openstack.org/56444116:20
* figleaf catches up from meeting-land16:31
figleafI wasn't aware that there was any such official term16:31
figleaf'resource action' and 'resource operation' seem equally descriptive, but unsatisfying16:32
*** fried_rice is now known as fried_rolls16:32
figleafThat wp page seems like internal jargon, and not anything official16:33
openstackgerritMerged openstack/nova master: Optional separate database for placement API  https://review.openstack.org/36276616:37
figleaf^^^ WHAT???16:38
openstackgerritSurya Seetharaman proposed openstack/nova master: Make nova list ignore down cells  https://review.openstack.org/57573416:47
openstackgerritMerged openstack/nova master: Add PLACEMENT_DB_ENABLED=True to the nova-next job  https://review.openstack.org/56406716:51
openstackgerritMerged openstack/nova master: Fix nits from trusted certs notification change  https://review.openstack.org/57552116:58
*** e0ne has quit IRC17:08
openstackgerritMathieu Gagné proposed openstack/nova master: Fix rebuild of baremetal instance when vm_state is ERROR  https://review.openstack.org/52355917:34
*** sususuryashines has quit IRC17:39
*** e0ne has joined #openstack-placement18:00
*** tssurya has joined #openstack-placement19:13
*** fried_rolls is now known as fried_rice19:18
fried_riceGuess that means I need another rebase.  But congratulations cdent.19:18
openstackgerritEric Fried proposed openstack/nova master: Handle agg generation conflict in report client  https://review.openstack.org/55666919:37
fried_ricecdent: Rebased ^ (previously had your +1).19:37
openstackgerritEric Fried proposed openstack/nova master: Test alloc_cands with indirectly sharing RPs  https://review.openstack.org/51960120:06
openstackgerritMaciej Kucia proposed openstack/nova-specs master: Fix Non-unique network names in Servers IPs API response  https://review.openstack.org/55812520:40
openstackgerritEric Fried proposed openstack/nova master: PCPU: Define numa dedicated CPU resource class  https://review.openstack.org/56177020:46
openstackgerritEric Fried proposed openstack/nova master: Nix unused raise_if_custom_resource_class_pre_v1_1  https://review.openstack.org/57584720:49
*** e0ne has quit IRC20:49
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Fix Non-unique network names in Servers IPs API response  https://review.openstack.org/55812520:56
*** markguz has quit IRC21:00
openstackgerritEric Fried proposed openstack/nova master: Enforce placement minimum in nova.cmd.status  https://review.openstack.org/56896521:04
openstackgerritEric Fried proposed openstack/nova master: Use placement.inventory.inuse in report client  https://review.openstack.org/56863921:10
openstackgerritMatt Riedemann proposed openstack/nova master: Add policy rule to block image-backed servers with 0 root disk flavor  https://review.openstack.org/56128421:10
* fried_rice heartily validates cdent's placement email21:14
*** edmondsw has quit IRC21:29
openstackgerritMatt Riedemann proposed openstack/nova master: unquiesce instance after quiesce failure  https://review.openstack.org/55086521:35
*** figleaf is now known as edleafe21:57
*** cdent has quit IRC22:38
openstackgerritMerged openstack/nova master: Fix execute mock for test_convert_image_with_errors  https://review.openstack.org/57530522:45
openstackgerritMerged openstack/nova master: Ignore UserWarning for scope checks during test runs  https://review.openstack.org/57523722:46
openstackgerritZack Cornelius proposed openstack/nova master: Implement file backed memory for instances in libvirt  https://review.openstack.org/56787622:49
openstackgerritZack Cornelius proposed openstack/nova master: DNM: Libvirt file backed memory  https://review.openstack.org/57576622:49
*** edmondsw has joined #openstack-placement22:50
*** edmondsw has quit IRC22:55
openstackgerritEric Fried proposed openstack/nova master: DNM: try to fix lower-constraints  https://review.openstack.org/57587223:10
openstackgerritEric Fried proposed openstack/nova master: DNM: try to fix lower-constraints  https://review.openstack.org/57587223:11
openstackgerritEric Fried proposed openstack/nova master: Handle agg generation conflict in report client  https://review.openstack.org/55666923:49
*** fried_rice is now known as efried23:53

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!