Monday, 2016-12-19

*** yamamoto has joined #openstack-meeting-300:00
*** yamamoto has quit IRC00:01
*** yamamoto has joined #openstack-meeting-300:05
*** baoli has quit IRC00:09
*** yamamoto has quit IRC00:11
*** yamamoto has joined #openstack-meeting-300:12
*** dimtruck is now known as zz_dimtruck00:15
*** galstrom is now known as galstrom_zzz00:20
*** kenji-i has quit IRC00:23
*** hurgleburgler has quit IRC00:47
*** hoangcx has joined #openstack-meeting-300:59
*** baoli has joined #openstack-meeting-301:00
*** baoli has quit IRC01:05
*** baoli has joined #openstack-meeting-301:07
*** matjazp has quit IRC01:13
*** tuanluong has joined #openstack-meeting-301:46
*** zz_dimtruck is now known as dimtruck02:08
*** huanxuan has joined #openstack-meeting-302:08
*** dimtruck is now known as zz_dimtruck02:18
*** sankarshan_away is now known as sankarshan02:48
*** baoli has quit IRC02:55
*** gouthamr has joined #openstack-meeting-302:56
*** baoli has joined #openstack-meeting-302:57
*** amotoki has joined #openstack-meeting-302:57
*** amotoki has quit IRC02:59
*** amotoki has joined #openstack-meeting-302:59
*** zz_dimtruck has quit IRC03:05
*** zz_dimtruck has joined #openstack-meeting-303:05
*** zz_dimtruck is now known as dimtruck03:06
*** garloff has quit IRC03:08
*** garloff has joined #openstack-meeting-303:09
*** ediardo has quit IRC03:09
*** gouthamr has quit IRC03:11
*** ediardo has joined #openstack-meeting-303:11
*** noama has quit IRC03:12
*** noama has joined #openstack-meeting-303:12
*** irenab_ has joined #openstack-meeting-303:15
*** irenab has quit IRC03:17
*** irenab_ is now known as irenab03:17
*** Ace___ has joined #openstack-meeting-303:25
*** alanmeadows has quit IRC03:25
*** alanmeadows has joined #openstack-meeting-303:26
*** baoli has quit IRC03:30
*** baoli has joined #openstack-meeting-303:30
*** HenryG has quit IRC03:30
*** baoli has quit IRC03:30
*** HenryG has joined #openstack-meeting-303:33
*** Ace____ has joined #openstack-meeting-303:40
*** Ace___ has quit IRC03:43
*** sankarshan is now known as sankarshan_away03:48
*** psachin has joined #openstack-meeting-303:53
*** sdake has joined #openstack-meeting-303:56
*** coolsvap has joined #openstack-meeting-303:59
*** marst has quit IRC04:11
*** dimtruck is now known as zz_dimtruck04:18
*** sdake has quit IRC04:28
*** baoli has joined #openstack-meeting-304:32
*** donghao has quit IRC04:40
*** donghao has joined #openstack-meeting-304:41
*** zz_dimtruck is now known as dimtruck04:46
*** prateek has joined #openstack-meeting-304:49
*** baoli has quit IRC04:53
*** lpetrut has joined #openstack-meeting-305:04
*** bks has joined #openstack-meeting-305:24
*** lpetrut has quit IRC05:26
*** amotoki has quit IRC05:32
*** yamamoto has quit IRC05:37
*** amotoki has joined #openstack-meeting-305:55
*** amotoki has quit IRC05:58
*** amotoki has joined #openstack-meeting-306:18
*** shu-mutou-AWAY has quit IRC06:20
*** enginy has joined #openstack-meeting-306:32
*** enginy has quit IRC06:34
*** enginy has joined #openstack-meeting-306:35
*** enginy has quit IRC06:36
*** qwertyco has joined #openstack-meeting-306:36
*** dimtruck is now known as zz_dimtruck06:37
*** yamamoto has joined #openstack-meeting-306:38
*** yamamoto has quit IRC06:38
*** yamamoto has joined #openstack-meeting-306:38
*** anilvenkata has joined #openstack-meeting-306:47
*** lpetrut has joined #openstack-meeting-306:49
*** amotoki has quit IRC07:10
*** matjazp has joined #openstack-meeting-307:11
*** matjazp has quit IRC07:16
*** matjazp has joined #openstack-meeting-307:19
*** baoli has joined #openstack-meeting-307:21
*** baoli has quit IRC07:26
*** amotoki has joined #openstack-meeting-307:32
*** pcaruana has joined #openstack-meeting-307:33
*** andreas_s has joined #openstack-meeting-307:41
*** rossella_s has joined #openstack-meeting-307:46
*** pgadiya has joined #openstack-meeting-307:48
*** lpetrut has quit IRC07:50
*** claudiub has joined #openstack-meeting-307:59
*** tuanluong has quit IRC08:00
*** matjazp has quit IRC08:03
*** amotoki has quit IRC08:06
*** matjazp has joined #openstack-meeting-308:10
*** egallen has joined #openstack-meeting-308:12
*** yamamoto has quit IRC08:13
*** matjazp has quit IRC08:13
*** matjazp_ has joined #openstack-meeting-308:18
*** baoli has joined #openstack-meeting-308:22
*** qwertyco has quit IRC08:24
*** amotoki has joined #openstack-meeting-308:25
*** MarkBaker has quit IRC08:30
*** qwertyco has joined #openstack-meeting-308:36
*** amotoki has quit IRC08:40
*** mrunge has quit IRC08:41
*** baoli has quit IRC08:45
*** mrunge has joined #openstack-meeting-308:46
*** noama has quit IRC08:48
*** jtomasek has joined #openstack-meeting-308:57
*** mickeys has quit IRC08:59
*** tuanluong has joined #openstack-meeting-309:01
*** mickeys has joined #openstack-meeting-309:03
*** yamahata has quit IRC09:04
*** mickeys has quit IRC09:04
*** qwertyco has quit IRC09:06
*** tuanluong_ has joined #openstack-meeting-309:11
*** MarkBaker has joined #openstack-meeting-309:13
*** yamamoto has joined #openstack-meeting-309:14
*** jtomasek has quit IRC09:19
*** pgadiya has quit IRC09:19
*** qwebirc2641 has joined #openstack-meeting-309:20
*** qwebirc2641 is now known as itamaro09:21
*** yamamoto has quit IRC09:28
*** pgadiya has joined #openstack-meeting-309:31
*** matjazp_ has quit IRC09:39
*** ibmko has joined #openstack-meeting-309:51
*** sankarshan_away is now known as sankarshan09:52
*** Adri2000 has quit IRC09:54
*** matjazp has joined #openstack-meeting-309:55
*** amotoki has joined #openstack-meeting-309:56
*** lpetrut has joined #openstack-meeting-310:03
*** mickeys has joined #openstack-meeting-310:05
*** MarkBaker has quit IRC10:05
*** Adri2000 has joined #openstack-meeting-310:06
*** amotoki has quit IRC10:06
*** matjazp has quit IRC10:06
*** MarkBaker has joined #openstack-meeting-310:07
*** amotoki has joined #openstack-meeting-310:09
*** mickeys has quit IRC10:11
*** rossella_s has quit IRC10:11
*** rossella_s has joined #openstack-meeting-310:12
*** tuanluong has quit IRC10:16
*** MarkBaker has quit IRC10:17
*** amotoki has quit IRC10:23
*** sankarshan is now known as sankarshan_away10:37
*** hoangcx has quit IRC10:38
*** amotoki has joined #openstack-meeting-310:38
*** bks has quit IRC10:43
*** amotoki has quit IRC10:45
*** tuanluong_ has quit IRC10:55
*** ociuhandu has joined #openstack-meeting-311:10
*** yamamoto has joined #openstack-meeting-311:11
*** MarkBaker has joined #openstack-meeting-311:16
*** egallen has quit IRC11:18
*** yamamoto has quit IRC11:22
*** hecliunyx has joined #openstack-meeting-311:31
*** sdague has joined #openstack-meeting-311:33
*** itamaro has quit IRC11:37
*** baoli has joined #openstack-meeting-311:44
*** baoli has quit IRC11:48
*** ociuhandu has quit IRC11:57
*** yamamoto has joined #openstack-meeting-312:02
*** yamamoto has quit IRC12:02
*** mickeys has joined #openstack-meeting-312:08
*** yamamoto has joined #openstack-meeting-312:09
*** matjazp has joined #openstack-meeting-312:10
*** mickeys has quit IRC12:13
*** sdake has joined #openstack-meeting-312:18
*** sdake_ has joined #openstack-meeting-312:22
*** sdake has quit IRC12:23
*** prateek has quit IRC12:24
*** ibmko has quit IRC12:25
*** rtheis has joined #openstack-meeting-312:25
*** sdake has joined #openstack-meeting-312:29
*** iurygregory has joined #openstack-meeting-312:29
*** sdake_ has quit IRC12:32
*** pgadiya has quit IRC12:38
*** egallen has joined #openstack-meeting-312:44
*** yamamoto has quit IRC12:44
*** baoli has joined #openstack-meeting-312:44
*** huanxuan has quit IRC12:46
*** yamamoto has joined #openstack-meeting-312:49
*** baoli has quit IRC12:49
*** pgadiya has joined #openstack-meeting-312:53
*** jtomasek has joined #openstack-meeting-312:56
*** ociuhandu has joined #openstack-meeting-312:58
*** mickeys has joined #openstack-meeting-313:09
*** tuan_luong has joined #openstack-meeting-313:09
*** mickeys has quit IRC13:13
*** yamamoto has quit IRC13:15
*** huanxuan has joined #openstack-meeting-313:16
*** gouthamr has joined #openstack-meeting-313:26
*** watanabe_isao has joined #openstack-meeting-313:27
*** prateek has joined #openstack-meeting-313:31
*** alexismonville has joined #openstack-meeting-313:31
*** noama has joined #openstack-meeting-313:37
*** ibmko has joined #openstack-meeting-313:38
*** noama has left #openstack-meeting-313:39
*** yamamoto has joined #openstack-meeting-313:44
*** baoli has joined #openstack-meeting-313:45
*** pgadiya has quit IRC13:46
*** tuan_luong has quit IRC13:49
*** baoli has quit IRC13:50
*** ibmko has quit IRC13:55
*** mickeys has joined #openstack-meeting-314:10
*** mickeys has quit IRC14:14
*** cleong has joined #openstack-meeting-314:14
*** lblanchard has joined #openstack-meeting-314:15
*** baoli has joined #openstack-meeting-314:19
*** sdake_ has joined #openstack-meeting-314:20
*** sdague_ has joined #openstack-meeting-314:21
*** sdake has quit IRC14:23
*** sdague has quit IRC14:24
*** baoli has quit IRC14:25
*** sdague_ is now known as sdague14:27
*** baoli has joined #openstack-meeting-314:29
*** watanabe_isao has quit IRC14:34
*** psachin has quit IRC14:37
*** VW has joined #openstack-meeting-314:41
*** singlethink has joined #openstack-meeting-314:44
*** prateek has quit IRC14:46
*** jtomasek has quit IRC14:50
*** prateek has joined #openstack-meeting-314:51
*** jtomasek has joined #openstack-meeting-314:59
*** matjazp has quit IRC15:01
*** zz_dimtruck is now known as dimtruck15:10
*** mickeys has joined #openstack-meeting-315:10
*** jtomasek has quit IRC15:13
*** mickeys has quit IRC15:15
*** reedip_ has joined #openstack-meeting-315:23
*** huanxuan has quit IRC15:30
*** marst has joined #openstack-meeting-315:34
*** yamamoto has quit IRC15:38
*** aarefiev is now known as aarefiev_afk15:41
*** baoli has quit IRC15:44
*** prateek has quit IRC15:44
*** baoli has joined #openstack-meeting-315:45
*** ocartier has joined #openstack-meeting-315:48
*** mtanino has joined #openstack-meeting-315:50
*** Sukhdev has joined #openstack-meeting-315:51
*** Sukhdev has quit IRC15:53
*** yamahata has joined #openstack-meeting-315:54
*** iyamahat has joined #openstack-meeting-315:55
*** yamahata has quit IRC15:56
*** yamahata has joined #openstack-meeting-315:57
*** dimtruck is now known as zz_dimtruck15:59
*** VW_ has joined #openstack-meeting-316:01
*** VW_ has quit IRC16:02
*** VW_ has joined #openstack-meeting-316:02
*** Sukhdev_ has joined #openstack-meeting-316:04
*** cgayan has quit IRC16:04
*** cgayan has joined #openstack-meeting-316:04
*** VW has quit IRC16:04
*** sdague has quit IRC16:06
*** VW_ has quit IRC16:07
*** VW has joined #openstack-meeting-316:08
*** zz_dimtruck is now known as dimtruck16:10
*** mickeys has joined #openstack-meeting-316:11
*** jtomasek has joined #openstack-meeting-316:12
*** mhayden has quit IRC16:13
*** pcaruana has quit IRC16:15
*** spotz_zzz has quit IRC16:15
*** mickeys has quit IRC16:15
*** jtomasek has quit IRC16:16
*** spotz_zzz has joined #openstack-meeting-316:16
*** galyna has joined #openstack-meeting-316:27
*** Sukhdev_ has quit IRC16:39
*** yamamoto has joined #openstack-meeting-316:39
*** mhayden has joined #openstack-meeting-316:41
*** matjazp has joined #openstack-meeting-316:42
*** cinerama has joined #openstack-meeting-316:43
*** cinerama has quit IRC16:44
*** cinerama has joined #openstack-meeting-316:44
*** VW has quit IRC16:44
*** VW has joined #openstack-meeting-316:45
*** mariojv has joined #openstack-meeting-316:46
*** rloo has joined #openstack-meeting-316:46
*** yamamoto has quit IRC16:49
*** VW has quit IRC16:49
*** dtroyer has left #openstack-meeting-316:52
*** cdearborn has joined #openstack-meeting-316:58
*** VW has joined #openstack-meeting-316:59
jroll#startmeeting ironic17:00
openstackMeeting started Mon Dec 19 17:00:00 2016 UTC and is due to finish in 60 minutes.  The chair is jroll. Information about MeetBot at http://wiki.debian.org/MeetBot.17:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.17:00
*** openstack changes topic to " (Meeting topic: ironic)"17:00
openstackThe meeting name has been set to 'ironic'17:00
TheJuliao/17:00
vdroko/17:00
xeko/17:00
mariojvo/17:00
zhenguoo/17:00
jrollhey folks17:00
joanna o/17:00
galynao/17:00
jlvillalo/17:00
*** bfournie has joined #openstack-meeting-317:00
jrollas always, the agenda:17:00
jroll#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting17:00
rlooo/17:00
jroll#topic announcements and reminders17:00
*** openstack changes topic to "announcements and reminders (Meeting topic: ironic)"17:00
jrollyou may be surprised to learn it's the holiday season17:00
jrollso expect it to be pretty quiet the next couple weeks17:01
aslezilo/17:01
jrollwe agreed already to not meet december 26, and to meet january 2 for those that are working17:01
jrollTheJulia has a couple announcements as well17:01
TheJuliaIf there are no objections, I believe it is time for us to go ahead and remove ironic-webclient.  With regards to doing so, I’ve already obtained the blessing of krotscheck who has indicated he does not intend to continue working on it.  This should not be confused with ironic-ui.17:01
TheJuliaFor context: Ironic-webclient was created as a conceptual stand-alone UI interface to help drive CORS adoption.  The original developer(s) have since changed employers and moved on to other things.  It has been approximately six month since a change has landed in the repository, and there have been no releases to date.17:02
jrollalso note most people at the summit were in favor of this17:02
* jroll is +117:02
bfournieo/17:02
cdearborno/17:02
jlvillal+1 on removal17:02
TheJuliaAdditionally, unrelated to the web client, the ironic-ui has had bugs added for functionality deficiencies where ironic supports something that the UI does not.17:03
TheJuliajroll: Thank you17:03
jrolloh, nice17:03
jroll\o/17:03
*** VW has quit IRC17:03
jrollanyone have anything else?17:03
rlooTheJulia: so we have folks maintaining ironic-ui?17:04
jlvillalWe are now fully a tempest plugin?17:04
TheJuliarloo: We do17:04
*** pc_m has joined #openstack-meeting-317:04
jrollwe've been fully a tempest plugin, but good point - the tempest patch merged to remove ironic code from tempest's tree \o/17:04
rlooTheJulia: I'll ping you later, will add a subteam for that17:04
jrollI managed to break inspector, but there's code up to fix that17:04
TheJuliarloo: Excellent!17:05
rlooyay for tempest plugin, boo for jroll breaking inspector :)17:05
jlvillal:)17:05
jrollindeed!17:05
xavierro/17:05
jrollok, let's move on17:06
jroll#topic subteam status updates17:06
*** openstack changes topic to "subteam status updates (Meeting topic: ironic)"17:06
jroll#link https://etherpad.openstack.org/p/IronicWhiteBoard17:06
jrollbeginning at line 7617:06
jrollmade some good progress on portgroups, driver composition, some CI/QA stuff this week17:07
jrollwe should probably focus a bit harder on BFV this week, so TheJulia has some work to do while everyone else is taking time off :P17:07
rloowrt portgroups support -- it mentions after the client stuff merges, needs nova patches. can the nova patches be done before attach/detach API?17:08
TheJuliaLess rebasing \o/17:08
jrollrloo: we agreed after attach/detach because it will be far simpler17:08
jrollactually, it will mean no changes in nova for portgroups iirc17:08
*** lucasagomes has joined #openstack-meeting-317:08
jroll(since it's all handled in attach/detach api)17:08
rloojroll: that's what i thought. am going to update that subteam report for portgroups support17:08
jrollrloo: yeah, thanks, that was my report, oops17:09
rloojroll: clarified :)17:09
*** VW has joined #openstack-meeting-317:09
jrollthanks17:09
rloovsaienk0: wrt attach/detach, the client patches are up-to-date?17:10
*** VW has quit IRC17:10
rlooor whoever updated the attach/detach status.. ?17:10
jlvillalvdrok: ^^^ ?17:10
vdrokrloo: I'm not sure about that17:10
jrollthat might be from last week17:10
*** VW has joined #openstack-meeting-317:10
rloovdrok: i just looked, the client patch was updated this morning17:11
* jroll updates it17:11
vdrokhave not looked at the client yet17:11
*** mickeys has joined #openstack-meeting-317:12
rlooyay for inspector supporting upgrades and following standard deprecations!17:13
* jroll \o/17:13
rlooi saw that iBoot, WoL and AMT drivers have been removed; I'm going to remove AMT from the Drivers section of the subteam reports17:14
lucasagomes++17:14
lucasagomesthat said, all these drivers are available at ironic-staging-drivers already17:14
jroll+117:14
lucasagomesfor those that are still relying on them17:14
*** sdake_ is now known as sdake17:15
*** cleong has quit IRC17:15
*** cleong has joined #openstack-meeting-317:15
rloojroll: wrt priorities, i would also like to get discussion/approval of rolling upgrades dbsync command17:16
jroll++17:16
jrollwe need to do that asap17:16
*** bobmel_ has quit IRC17:16
*** mickeys has quit IRC17:16
rloojroll: was going to put it as #4. what are you thinking? #2?17:16
jrollI'm fine with #217:16
rloojroll: will do17:16
jrollor we could just get it done in channel today and not worry17:17
rloojroll: that's my plan :)17:17
jrollcool17:18
jrollany other questions/comments here?17:18
TheJuliaNone here17:18
jroll#topic Granting +1/-1 rights to OPNFV CI for Bifrost Patches17:19
*** openstack changes topic to "Granting +1/-1 rights to OPNFV CI for Bifrost Patches (Meeting topic: ironic)"17:19
jrollTheJulia: this is you17:19
TheJuliaFdegir couldn't make it to the meeting today, so I'll be the stand-in.17:19
TheJuliaFdegir is with OPNFV, and they have been running a fairly stable third party CI for Bifrost for a while now. OPNVF is interested in obtaining CI voting rights on bifrost in order to help gate any breaking changes since they are utilizing bifrost heavily.17:19
TheJuliaI have no objections with this, but wanted to run it past the larger community.17:19
jrollis there data on the stability?17:20
*** nicodemos has joined #openstack-meeting-317:20
TheJuliaMy anecdotal observations have been that it appears to be about as stable as openstack ci it's self, however I've not hunted down any statistics17:20
jrollalso can we clarify what "voting" means? can a patch still merge with it voting -1? (I'm fairly certain that third party CI isn't allowed to actually block)17:20
*** reedip_ has quit IRC17:21
jrolliirc voting third party CI just means their name is next to jenkins at the top in the vote list, or something17:21
TheJuliaVoting would be that the test would vote as a CI system.  It is not forbidden from what I understand, it is just discouraged because of potential headaches17:21
lucasagomesTheJulia, you can create a graph at http://graphite.openstack.org to check the failure rate (after the meeting idk)17:21
jrollTheJulia: so a -1 from this CI means a patch cannot merge, correct?17:22
TheJulialucasagomes: Yeah, I should do that.17:22
TheJuliajroll: As I understand it, yes17:22
* jroll feels like he needs to dig up clarity17:22
jrolllast question for now, is there a patch up for this already?17:23
TheJuliajroll: no, it is a permission grant in garrit.  The group already exists, their CI is just not a member to give it the ability to vote yet17:23
*** corey_ has joined #openstack-meeting-317:23
*** cleong has quit IRC17:23
TheJuliagerrit17:23
*** milan has joined #openstack-meeting-317:23
*** corey_ is now known as Guest162217:24
milano/17:24
jrollTheJulia: ah17:24
jrollI think I need to learn a bit more about this, honestly, I wish I had read up first17:24
*** aNuposic has joined #openstack-meeting-317:24
TheJuliaWell, I'm fine with revisiting it a little later then17:24
jrollI also see the process involves an email with data: http://docs.openstack.org/infra/system-config/third_party.html#permissions-on-your-third-party-system17:24
TheJuliaHmm, that seems a little different from the last time I read that, but that was months ago.17:25
*** bobmel has joined #openstack-meeting-317:25
TheJuliaSo, defer, collect more information, and go from there.17:26
jrollyeah, I think so17:26
TheJuliaSounds good to me17:26
jrollor we could kick off the ML thread now (or maybe in january) and gather info there17:26
jrollany last comments?17:26
jrollanyone, not just julia :P17:27
TheJuliaI suspect January would be best considering how quiet it will be the next few weeks.  I'll check-in with fdegir first.17:27
jrollcool17:27
*** alexismonville has quit IRC17:27
jrollthanks TheJulia :)17:27
TheJuliaMy pleasure :)17:28
rloothx TheJulia! (I'm good with it but i don't use bifrost so my vote doesn't really count)17:28
jroll#topic open discussion17:28
*** openstack changes topic to "open discussion (Meeting topic: ironic)"17:28
jrollwho's got a thing :)17:28
* TheJulia hears crickets17:28
jrollgreat, I'm hungry :P17:28
joannaI have17:29
joanna:)17:29
*** pas-ha has joined #openstack-meeting-317:29
jrolljoanna: go ahead17:29
joannaso... I was working on a bug: https://bugs.launchpad.net/ironic/+bug/146114017:29
openstackLaunchpad bug 1461140 in Ironic "conflict (HTTP 409) incorrect for some cases" [Medium,In progress] - Assigned to Joanna Taryma (jtaryma)17:29
joannaI read the discussion posted in comments, and in my latest comment you can read what I got17:29
* lucasagomes reads17:30
joannakeeping it short - 409 cannot be replaced with 423 locked for node locked situation cause it's not a valid http response code17:30
rloojoanna: did we decide to have 423 for locked?17:30
jlvillaljoanna: According to what is it not a valid code?17:30
jlvillaljoanna: I mean, what complains about that code?17:30
jrolljlvillal: six, so it seems17:31
jrollwhich uses stdlib17:31
joannawsmeext.pecan in response validation check python http client's responses dict17:31
*** alexismonville has joined #openstack-meeting-317:31
joannaand 423 is not here17:31
jlvillalAh17:31
joannaI think this is because it's WebDAV17:31
lucasagomesyeah I think pecan won't accept WebDAV17:32
lucasagomesjoanna, what about the Retry-After: header ?17:32
joanna(standard for distributed document edition)17:32
vdrokalso, I think here is an rfc on http status codes https://tools.ietf.org/html/rfc7231.html17:32
vdrokno 423 here17:32
lucasagomeswe could have the Retry-After set for the exceptions that are retryable17:32
lucasagomesif non-retryable we emit the header17:32
jrollthis is about changing the response codes for things like "node with duplicate name" or "delete node with instance"17:32
joannaI got stuck on the code, lucadagomes17:32
jrollfwiw, not about changing node locked exceptions17:33
rloojroll: what?17:33
vdrokrloo: anything that can be retried successfully17:33
rloojroll: shouldn't it just be 400 then?17:33
lucasagomesjroll, yeah, I think so17:33
jrollrloo: reading the bug, it seems we want to change it for those cases, not for locked cases17:33
joannanope, exactly the opposite17:33
lucasagomesrloo, actually 409 (conflict) is correct for such errors17:33
lucasagomesafaict17:33
joanna409 is perfect for duplicate names17:33
joanna(and ids and addresses and other stuff)17:34
joannait was created for such cases17:34
jrollthen I'm confused, I guess, or didn't read the discussion that goes the opposite way of the bug description17:34
lucasagomesit's just that we abuse of 409 for things such node locked17:34
joannayet, I agree that node locked is not 40917:34
jroll(note I didn't say I agree with the bug description, just pointed out that's what it says)17:34
*** SergeyLukjanov__ has quit IRC17:34
rloojroll: yeah, i wrote that bug description early on. it needs to be updated to reflect the discussion we had a long time ago.17:35
joannajroll, imho this IRC discussion gives this bug a direction :)17:35
*** SergeyLukjanov has joined #openstack-meeting-317:35
vdrokso If I understand correctly, client always retries on 409, while we should do it only in cases when we actually can retry and get a success17:35
joannarloo - I can rewrite it following the discussion17:35
jrollrloo: yes, updating the description would be helpful to be able to talk about the bug17:35
lucasagomesvdrok, that's my understanding as well17:35
rloojoanna: we already had a discussion awhile ago -- what was decided in that earlier discussion?17:35
*** alexismonville has quit IRC17:35
joannait shouldn't retry on 409 imho - because if a name/id/sth is in use it will fail anyway17:36
joannaon node locked it should becuase it might have been freed up in the meantime17:36
vdrokwhen eg we set standalone_ports_supported on a portgroup that has pxe_enabled ports in it, we throw conflict, but the client should not retry, it should be resolved first17:36
*** bobmel_ has joined #openstack-meeting-317:37
jrollreading that irc discussion, we agreed there is a problem, and lucasagomes and dtantsur were going to put a plan together to fix it. part of the problem was too many locks, part was too many retries.17:37
jlvillaljoanna: Can you also give an example of the error pecan is throwing using 423?17:37
jrolland we would discuss the status code to replace 409 in the spec.17:37
lucasagomesthe problem of changing 409 is that old clients will not retry on the new return code17:38
joannait ended up with 'we have to do a research'17:38
jrolljlvillal: 423 is not a valid status code, per the RFE, so I'm kind of against it anyway :) https://tools.ietf.org/html/rfc7231.html17:38
lucasagomesjoanna, that's why I was thinking whether continue to use 409 + a header would be better17:38
*** bobmel has quit IRC17:38
joannawe might use the prerequisite not ready + retry17:38
lucasagomesat least with the backward compat POV in it, we could have a header that the client checks to know whether it should retry or not when a 409 is raised17:38
rloojroll: so i think we need a spec and we can move this discussion to that spec17:39
joannawhen the node is locked, and 409 for conflicts + no retry17:39
vdrokI like the header idea too17:39
*** rloo has quit IRC17:39
jlvillaljroll: Well it is valid in this other RFC...  https://tools.ietf.org/html/rfc2518   But point taken.17:39
*** rloo has joined #openstack-meeting-317:39
*** andreas_s has quit IRC17:39
*** rossella_s has quit IRC17:39
joannalucasagomes: then all conflicts that will fail by definition would create unnecessary traffic17:40
jrolljlvillal: which is an http extension for distributed authoring, dunno if it applies here17:40
joannaalso...17:40
joannaI don't think that using 204 code as a POST response is valid17:40
joannawould you all be fine with me preparing a spec about tidying up the responses?17:41
lucasagomesjoanna, that would be another problem, kinda off topic17:41
lucasagomesjoanna, but I'm good with the spec yeah17:42
vdrokhm ,why? 204 just means that there is no content in response17:42
jrollyeah, I'd prefer to solve the locking thing17:42
jrollnot really interested in debating semantics of 200 vs 204 for post requests17:42
joannait's sort of off topic, but the spec could be more generic the17:42
joannathen*17:42
joanna204 is not a response for POST17:42
joanna201 created should be sent17:42
*** mickeys has joined #openstack-meeting-317:43
lucasagomesjoanna, well, by experience. This type of spec will take way more time to get merged than if you focus on one problem at time17:43
lucasagomesyou can fix the return codes in parallel, I think we should have a separated spec for the locking17:43
lucasagomesbut it's up to you17:43
rloofixing/changing the return codes will require, at minimum, and RFE17:43
milanlucasagomes, +1 less bikeshedding17:43
rloos/and/an/17:44
joannalucasagomes, thanks it is a good advice! :)17:44
joannarloo - can I treat that bug as an RFE?17:44
joanna(rename the name of the bug?)17:44
rlooup to you, as long as it is clear what it is.17:45
*** bobmel has joined #openstack-meeting-317:45
rloojoanna: or just create a new rfe and link/point the existing bug to it.17:45
rloojoanna: i don't think we care that much. as long as it describes the problem, etc.17:45
joannaOK, as it already needs rewriting, I think I'll create a new one and a link17:46
joannathanks, rloo17:46
jrollcool, so we're good here?17:46
*** bobmel_ has quit IRC17:46
jrollanything else for open discussion?17:46
joannayes, thanks all17:46
lucasagomesjoanna, thank YOU for fixing it :-)17:46
joannai haven't fixed it... yet ;)17:47
lucasagomesfor trying :D17:47
jrollheh17:47
jrollobligatory gl;hf :P17:47
jrollthanks for a good meeting all :) see you back in channel17:47
vdrokthanks!17:47
jroll#endmeeting17:47
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings"17:47
openstackMeeting ended Mon Dec 19 17:47:54 2016 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)17:47
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2016/ironic.2016-12-19-17.00.html17:47
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2016/ironic.2016-12-19-17.00.txt17:47
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2016/ironic.2016-12-19-17.00.log.html17:47
joannahave a great evening, all!17:47
*** lucasagomes has left #openstack-meeting-317:48
*** rloo has left #openstack-meeting-317:48
milano/17:48
*** milan has left #openstack-meeting-317:48
*** aNuposic has left #openstack-meeting-317:48
*** bfournie has left #openstack-meeting-317:48
*** lpetrut has quit IRC17:48
*** sdague has joined #openstack-meeting-317:48
*** ociuhandu has quit IRC17:49
*** mariojv has left #openstack-meeting-317:50
*** pc_m has left #openstack-meeting-317:52
*** Swami has joined #openstack-meeting-317:52
*** tellesnobrega has joined #openstack-meeting-317:53
*** tenobreg has quit IRC17:56
*** cdearborn has left #openstack-meeting-317:56
*** bobmel_ has joined #openstack-meeting-317:57
*** bobmel has quit IRC17:59
*** iyamahat has quit IRC18:00
*** yamahata has quit IRC18:01
*** MarkBaker has quit IRC18:14
*** nicodemos has left #openstack-meeting-318:15
*** ocartier has quit IRC18:15
*** ociuhandu has joined #openstack-meeting-318:20
*** iyamahat has joined #openstack-meeting-318:22
*** armax has joined #openstack-meeting-318:25
*** pas-ha has left #openstack-meeting-318:28
*** matrohon has joined #openstack-meeting-318:36
*** gugl has joined #openstack-meeting-318:38
*** yamahata has joined #openstack-meeting-318:40
*** lpetrut has joined #openstack-meeting-318:40
*** baoli has quit IRC18:59
*** baoli has joined #openstack-meeting-319:00
*** emagana has joined #openstack-meeting-319:00
*** s3wong has joined #openstack-meeting-319:01
*** egallen has quit IRC19:04
*** anilvenkata has quit IRC19:06
*** david-lyle_ has joined #openstack-meeting-319:13
*** tellesnobrega has quit IRC19:13
*** david-lyle has quit IRC19:13
*** openstack has joined #openstack-meeting-319:16
*** ChanServ sets mode: +o openstack19:16
*** matrohon has quit IRC19:20
*** armax has quit IRC19:21
*** egallen has joined #openstack-meeting-319:29
*** egallen has quit IRC19:30
*** armax has joined #openstack-meeting-319:31
*** baoli has quit IRC19:32
*** baoli has joined #openstack-meeting-319:32
*** hurgleburgler has joined #openstack-meeting-319:54
*** lpetrut has quit IRC20:00
*** bobmel has joined #openstack-meeting-320:02
*** bobmel_ has quit IRC20:02
*** breton_ is now known as breton20:04
*** tellesnobrega has joined #openstack-meeting-320:19
*** mickeys has quit IRC20:24
*** mickeys has joined #openstack-meeting-320:25
*** ccamacho has quit IRC20:30
*** Sukhdev has quit IRC20:33
*** reedip_ has joined #openstack-meeting-320:39
*** iurygregory has quit IRC20:42
*** ccamacho has joined #openstack-meeting-320:46
*** baoli has quit IRC20:48
*** gugl has quit IRC20:53
*** gugl has joined #openstack-meeting-320:53
*** gugl has quit IRC20:59
*** rockyg has joined #openstack-meeting-321:02
*** galstrom_zzz is now known as galstrom21:04
*** baoli has joined #openstack-meeting-321:04
*** rtheis has quit IRC21:13
*** ccamacho has quit IRC21:15
*** gouthamr has quit IRC21:17
*** Guest1622 has quit IRC21:18
*** emagana has quit IRC21:21
*** VW_ has joined #openstack-meeting-321:21
*** emagana has joined #openstack-meeting-321:24
*** VW has quit IRC21:24
*** VW_ has quit IRC21:26
*** garloff has quit IRC21:31
*** galstrom is now known as galstrom_zzz21:35
*** galstrom_zzz is now known as galstrom21:48
*** VW has joined #openstack-meeting-322:05
*** VW has quit IRC22:07
*** VW has joined #openstack-meeting-322:07
*** VW has quit IRC22:08
*** VW has joined #openstack-meeting-322:09
*** lblanchard has quit IRC22:10
*** Sukhdev has joined #openstack-meeting-322:10
*** gugl has joined #openstack-meeting-322:25
*** baoli has quit IRC22:29
*** garloff has joined #openstack-meeting-322:37
*** galstrom is now known as galstrom_zzz22:46
*** reedip_ has quit IRC22:55
*** Swami has quit IRC22:56
*** baoli has joined #openstack-meeting-322:56
*** baoli has quit IRC23:01
*** sdake has quit IRC23:02
*** rockyg has quit IRC23:05
*** singlethink has quit IRC23:25
*** neiljerram has quit IRC23:27
*** neiljerram has joined #openstack-meeting-323:27
*** Sukhdev has quit IRC23:34
*** matjazp has quit IRC23:43
*** sdague has quit IRC23:44
*** gouthamr has joined #openstack-meeting-323:58
*** Sukhdev has joined #openstack-meeting-323:59

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