Thursday, 2018-06-14

openstackgerritjiang wei proposed openstack/nova master: Add action initiator attribute to the payload  https://review.openstack.org/53624300:38
openstackgerritTakashi NATSUME proposed openstack/nova master: Update admin/flavors document  https://review.openstack.org/57306301:08
openstackgerritzhangdaolong proposed openstack/nova master: Fix error log output information parameters.  https://review.openstack.org/57527701:11
openstackgerritZhenyu Zheng proposed openstack/nova master: Use ThreadPoolExecutor for max_concurrent_live_migrations  https://review.openstack.org/56350501:16
openstackgerritZhenyu Zheng proposed openstack/nova master: Compute: add support to abort queued live migration  https://review.openstack.org/56854201:42
openstackgerritCorey Bryant proposed openstack/nova master: Fix execute mock for test_convert_image_with_errors  https://review.openstack.org/57530502:00
openstackgerritTakashi NATSUME proposed openstack/nova master: Subject: Remove mox in unit/network/test_neutronv2.py (8)  https://review.openstack.org/57531102:34
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (8)  https://review.openstack.org/57531102:34
openstackgerritjiang wei proposed openstack/nova master: Add action initiator attribute to the payload  https://review.openstack.org/53624303:03
openstackgerritTakashi NATSUME proposed openstack/nova master: Add the guideline to write API reference  https://review.openstack.org/56905803:07
*** bhagyashris has quit IRC03:35
openstackgerritZhenyu Zheng proposed openstack/nova master: Add full traceback to ExceptionPayload in versioned notifications  https://review.openstack.org/56409203:41
*** bhagyashris has joined #openstack-placement03:46
*** takashin has left #openstack-placement04:31
*** e0ne has joined #openstack-placement04:53
*** e0ne has quit IRC05:00
openstackgerritjiang wei proposed openstack/nova master: Add action initiator attribute to the payload  https://review.openstack.org/53624305:36
openstackgerrityatin proposed openstack/nova stable/queens: [Stable Only] Initialise failed_builds in IronicNodeState  https://review.openstack.org/57499705:41
openstackgerritZhenyu Zheng proposed openstack/nova master: Compute: add support to abort queued live migration  https://review.openstack.org/56854206:14
openstackgerritZhenyu Zheng proposed openstack/nova master: Add full traceback to ExceptionPayload in versioned notifications  https://review.openstack.org/56409206:17
openstackgerritShuo Liu proposed openstack/nova-specs master: fix spelling mistake  https://review.openstack.org/57347906:36
*** tssurya has joined #openstack-placement06:41
*** ttsiouts has joined #openstack-placement07:14
*** efried has quit IRC07:26
*** efried has joined #openstack-placement07:27
*** ttsiouts has quit IRC07:27
*** tssurya_ has joined #openstack-placement07:27
*** ttsiouts has joined #openstack-placement07:31
*** tssurya_ has quit IRC07:31
*** ttsiouts has quit IRC07:33
openstackgerritjiang wei proposed openstack/nova master: Add action initiator attribute to the update payload  https://review.openstack.org/53624307:47
*** tetsuro has quit IRC07:49
*** ttsiouts has joined #openstack-placement07:50
*** ttsiouts has quit IRC07:56
*** ttsiouts has joined #openstack-placement07:57
*** takashin has joined #openstack-placement09:22
*** e0ne has joined #openstack-placement09:28
openstackgerritAlexandre arents proposed openstack/nova master: Preserve images_type of instance during live migration  https://review.openstack.org/57052809:43
*** nicolasbock has joined #openstack-placement10:37
*** ttsiouts has quit IRC10:47
openstackgerritzhangdaolong proposed openstack/nova master: Fix error log output information parameters.  https://review.openstack.org/57527710:50
*** cdent has joined #openstack-placement11:20
openstackgerritBalazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler  https://review.openstack.org/56726811:25
openstackgerritBalazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding  https://review.openstack.org/56945911:25
openstackgerritBalazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request  https://review.openstack.org/57331711:25
*** nicolasbock has quit IRC11:42
*** ttsiouts has joined #openstack-placement11:46
openstackgerritAlexandre arents proposed openstack/nova master: Preserve images_type of instance during live migration  https://review.openstack.org/57052812:00
openstackgerritMerged openstack/nova master: Add trusted_image_certificates to REST API  https://review.openstack.org/48620412:01
openstackgerritZhenyu Zheng proposed openstack/nova master: Use ThreadPoolExecutor for max_concurrent_live_migrations  https://review.openstack.org/56350512:03
openstackgerritZhenyu Zheng proposed openstack/nova master: Compute: add support to abort queued live migration  https://review.openstack.org/56854212:07
*** nicolasbock has joined #openstack-placement12:11
*** cdent has quit IRC12:29
openstackgerritCorey Bryant proposed openstack/nova master: Fix execute mock for test_convert_image_with_errors  https://review.openstack.org/57530512:33
*** edmondsw has joined #openstack-placement12:54
*** mriedem has joined #openstack-placement13:03
openstackgerritMerged openstack/nova stable/queens: [Stable Only] Initialise failed_builds in IronicNodeState  https://review.openstack.org/57499713:11
efriedjaypipes: You gonna hit https://review.openstack.org/#/c/559480/ today?  Should be a pretty easy re-approve (you were +A at PS21).13:39
jaypipesefried: ack13:48
*** ttsiouts has quit IRC13:55
*** tetsuro has joined #openstack-placement14:00
jaypipesefried: done.14:02
efriedcool mon14:02
*** ttsiouts has joined #openstack-placement14:03
*** cdent has joined #openstack-placement14:14
openstackgerritBalazs Gibizer proposed openstack/nova master: Add notification support for trusted_certs  https://review.openstack.org/56326914:17
openstackgerritBalazs Gibizer proposed openstack/nova master: Add certificate validation docs  https://review.openstack.org/56015814:17
openstackgerritBalazs Gibizer proposed openstack/nova master: WIP: Add trusted certs to feature support matrix docs  https://review.openstack.org/57489014:17
openstackgerritBalazs Gibizer proposed openstack/nova master: Remove max_size parameter from fake_libvirt_utils.fetch_*image methods  https://review.openstack.org/57491114:17
cdentjaypipes++ on the rant14:19
* efried notes that jaypipes just got incremented in place14:20
cdentefried: are you familiar with irc karma trackers? I hope we never have one, but it is hard habit to break14:22
efriedcdent: No, never heard of it.14:22
cdentbasically any time you ++ a nick that person's karma is incremented14:24
cdentand you can list it14:24
efriedwhoah14:27
efriedSounds like a popularity contest, despite anyone's best efforts to the contrary.14:27
efriedDo karma points age out, or is it also a measure of how long you've been around?14:27
cdentit only works in small groups, which is why I think it should never happen here14:28
cdentof course, I tend to think that chat of any form only works in small groups14:28
cdentso...14:28
cdentre aging, it depends on the bot14:28
cdentit's not a universal thing14:28
*** ttsiouts has quit IRC14:29
*** ttsiouts has joined #openstack-placement14:31
openstackgerritChris Dent proposed openstack/nova master: [placement] Add status and links fields to version document at /  https://review.openstack.org/57511714:32
cdentefried: an opportunistic rebase on ^ to try to overcome the cellsv1 thing (has the fix merged yet?). can you put your imprimatur back on it please?14:33
efriedcdent: yes, the cellsv1 fix merged (see email).  Looking...14:33
efriedcdent: You're planning to compose a placement email tomorrow?14:34
cdentyes14:34
efriedcdent: If you remember, would you please update the link at https://wiki.openstack.org/w/index.php?title=Meetings/NovaScheduler&action=edit&section=4 ?14:34
cdentsure14:34
efriedthx14:34
cdentIt will be a bit harder to write the update email, because I usually do that with the laptop, in a coffee place, but I think I can overcome that limitation14:35
efriedhehehe14:35
cdentbut it may have a different tone as I'll be standing14:35
efriedfor sure14:35
openstackgerritDan Smith proposed openstack/nova stable/pike: Change consecutive build failure limit to a weigher  https://review.openstack.org/57324814:35
*** tetsuro has quit IRC14:46
cdentefried: on placement meeting chair, I'm meh about being on the hook for that since I'm already on the hook (self imposed) for the weekly update14:46
cdentboth is ... tedious14:46
cdentso while no one else wants it, I'd prefer to be interim14:46
*** takashin has left #openstack-placement15:01
*** e0ne has quit IRC15:03
openstackgerritMerged openstack/nova master: XenAPI: define a new image handler to use vdi streaming  https://review.openstack.org/48647515:08
openstackgerritMerged openstack/nova master: XenAPI: update the document related to vdi streaming  https://review.openstack.org/56844415:08
*** e0ne has joined #openstack-placement15:40
*** ttsiouts has quit IRC15:45
openstackgerritMerged openstack/nova master: Add enhanced KVM storage QoS quotas  https://review.openstack.org/55853015:47
openstackgerritMerged openstack/nova master: placement: Allocation.consumer field  https://review.openstack.org/56540516:10
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add NUMACell.network_info  https://review.openstack.org/56443916:23
openstackgerritStephen Finucane proposed openstack/nova master: network: Unchain _get_phynet_info from _get_port_vnic_info  https://review.openstack.org/56444316:23
openstackgerritStephen Finucane proposed openstack/nova master: network: Add 'create_resource_requests' to network API  https://review.openstack.org/56444416:23
openstackgerritStephen Finucane proposed openstack/nova master: network: Retrieve tunneled status in '_get_network_info'  https://review.openstack.org/56444516:23
openstackgerritStephen Finucane proposed openstack/nova master: objects: Remove NUMATopologyLimits.obj_from_db_obj  https://review.openstack.org/53741216:23
openstackgerritStephen Finucane proposed openstack/nova master: objects: Remove legacy '_to_dict' functions  https://review.openstack.org/53741316:23
openstackgerritStephen Finucane proposed openstack/nova master: hardware: Start accounting for networks in NUMA placement  https://review.openstack.org/56444816:23
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add RequestSpec.numa_networks  https://review.openstack.org/56444216:23
openstackgerritStephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.numa_networks  https://review.openstack.org/56445216:23
openstackgerritStephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options  https://review.openstack.org/56444016:23
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_info field  https://review.openstack.org/56444116:23
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add NUMATopologyLimits.networks  https://review.openstack.org/57548616:23
openstackgerritStephen Finucane proposed openstack/nova master: DNM: objects: Store RequestSpec.numa_networks  https://review.openstack.org/57548716:23
openstackgerritStephen Finucane proposed openstack/nova master: network: Remove 'create_pci_requests_for_sriov_ports'  https://review.openstack.org/57548816:23
openstackgerritDan Smith proposed openstack/nova master: Make check_can_live_migrate_destination use long_rpc_timeout  https://review.openstack.org/57549316:47
*** e0ne has quit IRC16:48
efriedjaypipes: Well, we have support for nested providers in alloc cands in microversion 1.27.16:52
efriedjaypipes: Can't say exactly where it crept in, but it's there in current master.16:52
efriedcdent, edleafe: How do we handle this?16:53
efriedDo we need to put up code to revert the behavior in 1.27?16:53
cdenti would hope/think not16:53
cdentwhat we have, whenever it leaked in, is a microversion that has incomplete support for nested16:54
cdentor even wrong16:54
cdentand what 1.28 gives us is correct16:54
cdentyou should be able to repeat your gabbi trick, downgrading the microversion until it blows up?16:55
efriedI.e. "don't worry about the fact that you get different behaviors in 1.27 depending which commit you're working against, because you shouldn't be trying to do this against nrp in the first place, you dolt" ?16:55
cdentno, more like: CD has risks, it's okay16:56
efriedcdent: It's not even a 1.27 thing.  The leak is about which code you're running.16:56
cdentokay my statement still stands: sometimes master won't be right16:57
cdentrarely16:57
efriedThe test passes in 1.1316:57
cdentwhich test are your running?16:57
efriedhere, I'll put up a DNM to show you what I've done...16:59
cdentk, i'll be slow to respond, going on a call...16:59
openstackgerritEric Fried proposed openstack/nova master: DNM: Leaked nrp-in-alloc-cands behavior  https://review.openstack.org/57549716:59
efriedcdent: ^16:59
efriedWhat I think I hear you saying is that we should git r merged before we tag the next thingy, and not worry about it.17:00
*** e0ne has joined #openstack-placement17:00
efriedoff to lunch17:00
cdentefried: pretty much17:01
cdentefried: if you can provide a bit more detail on what's wrong and what right would look like that would be great. I'm sure I can figure it out, but with the number of calls and shit going on, don't make me think17:02
*** e0ne has quit IRC17:10
*** e0ne has joined #openstack-placement17:13
openstackgerritElod Illes proposed openstack/nova master: Reject interface attach with QoS aware port  https://review.openstack.org/57007817:16
openstackgerritElod Illes proposed openstack/nova master: Reject networks with QoS policy  https://review.openstack.org/57007917:16
*** e0ne has quit IRC17:19
openstackgerritMatt Riedemann proposed openstack/nova master: Add troubleshooting item about ignored microversions  https://review.openstack.org/57550617:23
*** e0ne has joined #openstack-placement17:24
*** e0ne has quit IRC17:27
efriedcdent: All I know at the moment is that, at some point (maybe multiple points) in the nrp-in-alloc-candidates series, we made behavior changes without microversions.17:45
*** e0ne has joined #openstack-placement17:45
efriedPrior to that series, if you asked for allocation candidates against a setup with trees, you would only get candidates that involved a single non-sharing provider.17:45
cdentefried: when the series is fully merged, will we have something like the right  behavior (at the proper microversions)?17:46
cdentIf so, I'd say: let's just make that happen, and try not to worry17:46
efriedcdent: Well, certainly the idea is for 1.28 (or more likely 1.29, since consumer gens will likely beat this) to behave properly for provider trees.  The question is whether the behavior changes prior to that microversion are acceptable.17:47
efriedcdent: So there's some commit before which you could only get back an allocation_request with multiple providers in it if N-1 of them were sharing.  But after that commit, you could get back allocation_requests with multiple non-sharing providers from the same tree.17:49
cdenti'm asking: after extant code merges is <= 1.27 doing what we expect and is 1.28 doing what we expect. If so, let's just ship code.17:50
cdentIf something is out there there running off a commit that is doing the wrong thing, we can just tell them to use newer code17:50
efriedcdent: I guess that's what I'm getting at.  If someone's using Queens today, and they have provider trees for whatever reason, and they're using GET /allocation_candidates (at an old microversion), and they're relying on the fact that, when run against those trees, they only get results that involve one non-sharing provider...17:52
efriedcdent: And then they upgrade to rocky and change nothing else...17:52
efriedcdent: Then they'll start getting different behavior (more results back).17:52
efriedSo is that acceptable on the reasoning that "Provider trees are unsupported/undefined/unpredictable prior to microversion 1.2{8|9}, so anything you do against them, including GET /allocation_candidates, is at your own risk" ?17:53
cdentI'm not sure we are communicating each other's comments/questions to each other well. /me thinks17:53
efriedcdent: I think you're asking whether, after this series merges, 1.27 will go back to behaving properly17:54
*** tssurya has quit IRC17:54
efriedcdent: If it does, then we're okay, because we were only "broken" for a few commits in the middle of an active development cycle.17:56
cdenti had the impression from what you were saying that during rocky we've merged code that changed how things behave (at a variety of microversions)17:56
cdentbut sometimes it seems like you are saying: queens is also broken17:56
efriedcdent: I'm saying that first thing - we've merged code during Rocky that changes behavior.17:57
efriedI don't think q is broken.17:57
cdentmy statement is: whatever the changes are between the beginning of rocky and the end of rocky, as long as at the end of rocky the diff between rocky and queens is as expected, we are okay17:57
cdentbecause running any unreleased version of rocky is an acceptance of risk17:57
edleafethat sounds like an argument for one microversion per release :)17:58
openstackgerritChris Dent proposed openstack/nova master: Optional separate database for placement API  https://review.openstack.org/36276617:58
openstackgerritChris Dent proposed openstack/nova master: Isolate placement database config  https://review.openstack.org/54143517:58
openstackgerritChris Dent proposed openstack/nova master: Ensure that os-traits sync is attempted only at start of process  https://review.openstack.org/55385717:58
cdenthush your mouth edleafe :)17:58
cdentstriving to be CD is good. failing sometimes is okay too.17:59
efriedcdent: Gotcha.17:59
openstackgerritChris Dent proposed openstack/nova master: Add PLACEMENT_DB_ENABLED=True to the nova-next job  https://review.openstack.org/56406717:59
cdentefried: if you disagree, that's cool. I'm trying to see a path to keep us moving well, accepting a bit of slack.18:00
cdentit sounds like in this case we probably could have had some better tests to insure that behaviors were observable rather than you discovering them by chance18:01
cdentsuch as: when nested exist, acs shoud looks like this at microversion x, y, z18:01
cdentanyway, I'm being called away to dinner18:01
efriedright, I think that's what we don't have, agree we should add it.18:01
efriedThanks for the talk.18:01
*** e0ne has quit IRC18:27
openstackgerritMatt Riedemann proposed openstack/nova master: Fix nits from trusted certs notification change  https://review.openstack.org/57552118:30
*** tssurya has joined #openstack-placement18:33
*** tssurya has quit IRC18:35
*** tssurya has joined #openstack-placement18:35
*** cdent has quit IRC18:55
*** e0ne has joined #openstack-placement19:15
jaypipesefried: sorry, been out all day. what's happened?19:33
jaypipesefried: something about a microversion problem?19:33
efriedjaypipes: Yeah, the nrp-in-alloc-cands series has already enabled nrp-in-alloc-cands in current master.19:34
jaypipesefried: nothing *uses* the returned values from n-r-p in alloc cands, though.19:35
jaypipesefried: or am I missing something?19:35
jaypipesefried: as in, is there anything that is yet asking for granular request groups?19:36
efriedNothing to do with granular.19:36
jaypipesefried: because granular request groups are required for n-r-p to even be useful to the scheduler.19:36
efriedno19:36
efriedthey're not19:36
jaypipesefried: and there isn't anything yet actually creating nested providers (on the compute side) so...19:37
efriedjaypipes: A potentially busted scenario.  In Queens, we support creating nrps, but if you GET /allocation_candidates against a nrp, the only candidates you get back are ones that are satisfied by a single RP (anywhere in the tree).  In Nova, that'll always be the root RP, because that's always where the VCPU and MEMORY_MB live.  But even in Nova, there wouldn't be anything stopping you from putting other resources in n19:37
efriedNow you upgrade to Rocky, but change nothing else, including the microversion you were using to GET /allocation_candidates.  But now instead of getting back only candidates satisfied by a single non-sharing provider, you start getting back candidates satisfied by multiple non-sharing providers.19:38
efriedBam, behavior change, potential for bustage.19:38
efriedIt won't be a problem in Nova.19:38
efriedBut in theoretical other-consumer-of-placement-land, it would be.19:38
jaypipesefried: I'm failing to see how it would be a problem...19:39
efriedWhat Chris asserted was that, as long as by the end of this series, we're back to pre-1.28 behaving like it used to with respect to nrps, we're fine.19:39
openstackgerritMatt Riedemann proposed openstack/nova master: Add certificate validation docs  https://review.openstack.org/56015820:13
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add trusted certs to feature support matrix docs  https://review.openstack.org/57489020:13
openstackgerritMatt Riedemann proposed openstack/nova master: Remove max_size parameter from fake_libvirt_utils.fetch_*image methods  https://review.openstack.org/57491120:13
openstackgerritMatt Riedemann proposed openstack/nova master: Fix nits from trusted certs notification change  https://review.openstack.org/57552120:13
efriedjaypipes: Sorry, multiple context switches20:14
efriedjaypipes: But what we need is some test cases that show what the behavior of pre-1.28 GET /a-c looked like against nrps so we can be relatively sure we've matched it once this series is fully landed.20:15
efriedjaypipes: It's a problem if a consumer has come to rely on GET /a-c at e.g. microversion 1.13 behaving a certain way despite the presence of child providers, which have been available to create since Queens.20:16
*** ttsiouts has joined #openstack-placement20:19
*** e0ne_ has joined #openstack-placement20:26
*** ttsiouts has quit IRC20:28
*** ttsiouts has joined #openstack-placement20:28
*** e0ne has quit IRC20:29
jaypipesefried: I guess I'm just thick. I don't see the specific problem that is being worried about. perhaps you could elaborate? if a client calls GET /a-c with 1.13 and gets a certain result and then calls GET /a-c with 1.27 and gets a different result, what's wrong with that? I thought 1.27 was all about changing the result of GET /a-c to include all resource class inventories in provider_summaries instead of only the inventories for resources20:34
jaypipesthat were requested, right? so the client is already expecting that behaviour change, no? or are you saying that before 1.27 we were not returning multiple providers in *a single provider tree* a single allocation request, but after 1.27 we *are* returning multiple providers in a single provider tree? because we have been returning multiple providers (sharing and non-sharing) in a single allocation request for a long time now.20:34
jaypipesefried: I'm not trying to be obtuse, honest. I'm just struggling to understand what the core of the issue is.20:35
efriedjaypipes: It's that second thing I'm saying - except the distinction is that we've only been returning an alloc_req that spans N providers if N-1 of them are sharing.20:36
efriedjaypipes: Whereas now, in this pre-1.28 but post-beginning-of-this-series master version of 1.13, we'll return an alloc_req that spans N providers even if >1 of them are non-sharing.20:37
jaypipesefried: ok, understood.20:37
efriedi.e. a superset of the allocation_request results we used to return20:37
jaypipesefried: agree with cdent then... not sure it's a huge issue if we "close the loop" by the end of this cycle.20:37
efriedjaypipes: right, I agree with that, so the action to take is to make tests that prove that.20:38
jaypipesefried: ack20:38
efriedWhich is going to be a little tricky to do because we're broken now.20:38
jaypipesefried: you keep using this word "broken".. :)20:38
efriedSorry, "altered"20:38
jaypipesefried: are you saying there is no way to tell which behaviour we're currently displaying?20:38
efriedwhich is the word you use for a neutered pet.20:38
jaypipeslol20:38
efriedjaypipes: Well, I can show you the altered behavior in the current master...20:39
efriedjaypipes: https://review.openstack.org/57549720:39
efriedthis copies the fixture update (to make the fixture nested) and one of the gabbi tests from the top of the series20:39
efriedjaypipes: but sets the microversion to 1.13 and proves we still get the nested results.20:40
jaypipesefried: k, good example, thanks.20:40
jaypipesthat gabbit clears it up for me, thanks20:40
jaypipes(that's why I like gabbits, btw...)20:40
jaypipesdeclarative HTTP diffs FTW.20:41
efriedjaypipes: So the thing to do will probably be some extension of this, but composed against queens, and then carried along until it is no longer broken in rocky.20:41
efriedsorry, altered20:41
efriedexcept in that case we'll actually write the case so it's working in queens, but broken in rocky.20:41
openstackgerritMatt Riedemann proposed openstack/nova master: Add trusted certs to feature support matrix docs  https://review.openstack.org/57489020:43
openstackgerritMatt Riedemann proposed openstack/nova master: Remove max_size parameter from fake_libvirt_utils.fetch_*image methods  https://review.openstack.org/57491120:43
openstackgerritMatt Riedemann proposed openstack/nova master: Fix nits from trusted certs notification change  https://review.openstack.org/57552120:43
openstackgerritMatt Riedemann proposed openstack/nova master: Time how long pre_live_migration() takes  https://review.openstack.org/57448120:47
openstackgerritEric Fried proposed openstack/nova-specs master: WIP: Nova-Cyborg workflow  https://review.openstack.org/57554520:53
mriedemefried: likely better to put that in here https://docs.openstack.org/nova/latest/reference/index.html#forward-looking-plans21:22
mriedemthe specs repo will complain about following the template21:22
efriedmriedem: I took the whole template out.  I'm just using this as a dumping ground for this thing as a strawman, not really expecting to build a real spec around it.21:22
mriedemefried: right, but it could live in the forward looking plans section as a strawman21:23
efriedmriedem: Do things in there not have to follow the spec template?21:23
mriedemthere have also been a couple of nova/cyborg workflow specs in cyborg's specs repo21:23
mriedemefried: correct21:23
efriedmriedem: Not in their specs repo, which is confusing.  But yeah, I've seen those, and they're in parts wrong and mostly too high-level.21:23
mriedemyeah... https://specs.openstack.org/openstack/cyborg-specs/21:24
mriedemadmin guide for cyborg specs...21:24
mriedemsudo cat cyborg-specs21:24
efriedmriedem: https://review.openstack.org/#/q/project:openstack/cyborg-specs21:24
efriedI did start there, but there was a bunch of stuff I would have had to bootstrap about the specs repo, and I wanted to get this out quick without a lot of paperwork :)21:25
mriedemwas thinking of https://review.openstack.org/#/c/448228/21:26
mriedemwhich was then https://review.openstack.org/#/c/508572/21:26
mriedemyeah really high level21:27
*** tssurya has quit IRC21:30
openstackgerritMerged openstack/nova master: [placement] Add status and links fields to version document at /  https://review.openstack.org/57511721:53
*** edmondsw has quit IRC21:54
openstackgerritMerged openstack/nova master: Use oslo.messaging per-call monitoring  https://review.openstack.org/56669621:55
openstackgerritMatt Riedemann proposed openstack/nova master: Handle broken instance mapping in InstanceLister  https://review.openstack.org/57555622:01
openstackgerritMerged openstack/nova master: Add notification support for trusted_certs  https://review.openstack.org/56326922:03
*** e0ne_ has quit IRC22:24
*** nicolasbock has quit IRC22:34
*** ttsiouts has quit IRC23:17
*** takashin has joined #openstack-placement23:58

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