Thursday, 2017-04-06

*** Hanchen has joined #openstack-meeting-501:31
*** iceyao has joined #openstack-meeting-501:34
*** Hanchen has quit IRC02:15
*** Hanchen has joined #openstack-meeting-502:20
*** Qiming has left #openstack-meeting-502:46
*** Hanchen has quit IRC02:50
*** Hanchen has joined #openstack-meeting-502:53
*** ramishra has left #openstack-meeting-503:15
*** Hanchen has quit IRC04:02
*** ricolin has joined #openstack-meeting-504:03
*** zaneb has quit IRC04:10
*** zaneb has joined #openstack-meeting-504:18
*** Hanchen has joined #openstack-meeting-504:38
*** ksumit has quit IRC05:43
*** makowals has joined #openstack-meeting-506:09
*** matrohon has joined #openstack-meeting-506:56
*** rarcea has joined #openstack-meeting-507:15
*** aarefiev_afk is now known as aarefiev07:17
*** ralonsoh has joined #openstack-meeting-508:06
*** ralonsoh_ has joined #openstack-meeting-508:06
*** ralonsoh has quit IRC08:10
*** ralonsoh_ is now known as ralonsoh08:12
*** Hanchen has quit IRC08:14
*** markvoelker has quit IRC08:26
*** iceyao has quit IRC09:50
*** iceyao has joined #openstack-meeting-509:51
*** markvoelker has joined #openstack-meeting-510:27
*** markvoelker has quit IRC10:31
*** markvoelker has joined #openstack-meeting-512:28
*** markvoelker has quit IRC12:32
*** ralonsoh has quit IRC13:05
*** ralonsoh has joined #openstack-meeting-513:06
*** iceyao has quit IRC13:46
*** izaakk has quit IRC14:13
*** MarkAtwood has quit IRC14:14
*** izaakk has joined #openstack-meeting-514:15
*** MarkAtwood has joined #openstack-meeting-514:16
*** markvoelker has joined #openstack-meeting-514:29
*** markvoelker has quit IRC14:35
*** markvoelker has joined #openstack-meeting-514:59
*** matrohon has quit IRC15:03
*** zaneb has quit IRC15:32
*** mjturek has joined #openstack-meeting-515:34
*** zaneb has joined #openstack-meeting-515:34
*** hshiina has joined #openstack-meeting-515:52
*** dtantsur has joined #openstack-meeting-516:00
TheJuliao/16:00
TheJulia#startmeeting ironic_bfv16:00
openstackMeeting started Thu Apr  6 16:00:38 2017 UTC and is due to finish in 60 minutes.  The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot.16:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
*** openstack changes topic to " (Meeting topic: ironic_bfv)"16:00
*** phuongnh has joined #openstack-meeting-516:00
dtantsuro/16:00
openstackThe meeting name has been set to 'ironic_bfv'16:00
joannao/16:00
hshiinao/16:00
TheJuliaOur agenda today, as always can be found on the wiki: https://wiki.openstack.org/wiki/Meetings/Ironic-BFV16:01
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic-BFV16:01
TheJulia#topic Announcements/Reminders16:01
*** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic_bfv)"16:01
TheJuliaI have no announcements today.  Does anyone have any announcements or reminders today?16:03
hshiinai have nothing16:03
TheJuliaWell, actually, I do have one announcement, the common/cinder.py code landed \o/16:03
joannaah I thought it's the current status part :D16:03
joannaanyway \o/16:03
TheJuliaTrue, but it feels like there should be a party now :)16:04
TheJuliaAnyway, moving on if nobody has anything else for this part of the meeting.16:04
joannathat's so true :)16:04
TheJulia#topic Current Status16:04
*** openstack changes topic to "Current Status (Meeting topic: ironic_bfv)"16:04
*** aarefiev is now known as aarefiev_afk16:05
joannathere's a new patch: https://review.openstack.org/#/c/453839 that is a follow up to newly landed common cinder interface16:05
TheJuliaI updated the etherpad and added the follow-up to the common code, and a WIP that I committed to begin hammering out last week regarding skipping the deployment if BFV makes sense16:05
TheJulia#link https://etherpad.openstack.org/p/Ironic-BFV16:05
TheJulia#link https://review.openstack.org/#/c/45383916:06
TheJuliaFolks, please review ^^^16:06
TheJuliaAlso, please take a quick look at the WIP change I posted, https://review.openstack.org/#/c/454243/ and leave any feedback if pertinent.  Naturally I still need to write tests, which I suspect will be quite a bit of code.16:07
TheJulia#link https://review.openstack.org/#/c/454243/16:07
hshiinaTheJulia, sure. thank you for your work16:08
TheJuliahshiina: I skimmed over the the client revisions and marked them -1 since we should have a depends-on tag in the commit message to prevent premature merging16:08
hshiinaTheJulia, i will add the tag16:09
TheJuliaThank you!16:09
TheJuliaAside from that, does anyone have anything else status wise to report?16:09
joannaI'll update the driver, wire, ipxe so they have a proper base, today or early next week16:10
TheJuliaAwesome, I was going to ask about that16:11
joanna:)16:11
TheJulia#info joanna is going to update the driver, storage call wire-in, and ipxe template updates to have the proper base in the next few days.16:11
TheJuliaSo I guess onward we go, if there are no objections.16:12
TheJulia#topic Planning/Priorities16:12
*** openstack changes topic to "Planning/Priorities (Meeting topic: ironic_bfv)"16:12
TheJuliaFrom a priorities standpoint, I'd like to see reviews on  https://review.openstack.org/#/c/366197/ and https://review.openstack.org/#/c/406290 this next week.16:13
joannaI think since the common patch was merged, we can choose a new one for next week priority list16:14
TheJuliaI don't think it will take much to get joanna's follow-up rev landed.16:14
TheJuliaAgreed, I'd like to get a little more velocity personally, but I suspect if we get one patch a week for the main ironic code, we're doing really good.16:15
TheJuliahshiina: do you feel that there needs to be any specific attention paid to the nova or client revisions at this time, or do you think we should just try to get some of the changes in the conductor landed for now?16:17
hshiinai agree to review conductor patches following dependency16:18
joannahshiina: If you need any reviews, please let me know, I'll be happy to do it :)16:18
hshiinajoanna, thank you16:19
TheJuliaOkay, seems like we're good to move on to discussion?  Any objections?16:19
joanna+16:19
hshiinanothing16:19
TheJulia#topic Discussion16:20
*** openstack changes topic to "Discussion (Meeting topic: ironic_bfv)"16:20
TheJuliaI see we have one item today that hshiina has raised regarding API path.16:20
TheJuliahshiina you have the floor16:20
joannalooks like my fault... :)16:21
hshiinajoanna gave a comment to the API patch regarding API path16:21
dtantsurcommented on the patch already, but I'd prefer structured approach, i.e. /volume/{connectors,targets}. easier for API discovery, and otherwise nicer.16:21
TheJuliadtantsur: I saw, thank you!16:22
joannaI like the approach where api paths are resource based16:22
joannalogical grouping is not resource-based, if there are no objects representing the group16:22
dtantsurneither is root URL16:23
joannabut that's a specific case16:23
dtantsur"object" can be a collection of links, like /v1 links to /v1/nodes, /v1/ports, etc16:23
joannawhile here we'd introduce a mixup16:23
joannayes, but there's not GET for /v1/../volume16:24
*** ralonsoh has quit IRC16:24
dtantsurif somebody wants to traverse Ironic API to know its capabilities, they'll see a group /v1/volume there, jump into it, and see /v1/volume/...16:24
dtantsurthere is, just like there is get for /v116:24
joannabut /v1 is our root for available APIs16:25
TheJuliaFor some reason I also think /v1 is optional, but I might be confusing that part of the api code with something else16:25
joannain other cases path are resource based, and grouping is resource-wise16:25
dtantsurjoanna, right, and /v1/volume is a root for volume API16:25
* dtantsur wishes devananda was here, he's a big fan of traverseable API16:26
joannabut there's not "volume" object returned by the API, right?16:26
TheJuliaThere is not16:27
dtantsuryes, and I don't see it as a problem16:27
TheJuliaa get could return what is available, I think we do something similar for provision/state16:27
joannathere is no other case where we use a resource that does not represent actual resource just to group stuff16:28
joannaor is there? (not sure)16:28
dtantsurjoanna, / and /v1 as I already noticed16:28
dtantsurat least, dunno about something else16:28
joannabut /v1 and / are something that may be compared to new index.html :) I don't want to introduce something non-standard that will be misleading16:29
dtantsurI don't agree with "non-standard"16:30
*** zaneb has quit IRC16:30
joannaand having all the resources represent something, having one just for grouping would make me surprised (and probably I'd think that I am doing something wrong as I cannot retrieve volume object from volume resource)16:30
dtantsurthe / and /v1 are not special, we are just used to them16:30
dtantsurjoanna, heh, you did not work with Redfish protocol, it's built completely on this principle :)16:31
dtantsuranyway, I'm not -2 on volume-connectors, kinda -0.5 rather16:31
TheJuliaPersonally, I'd prefer what was agreed to in the spec.16:32
joannadtantsur: I didn't :(16:32
TheJuliasince otherwise, we need to go rev the spec, and all of the patches to change it.16:32
dtantsurjoanna, there is a whole idea of making HTTP API traverseable in this fashion, so that you start at / and walk you way up to the resource you need. but this is kinda off-topic now16:33
joannaTheJulia, dtantsur: if that will create additional work, I'm fine with /volume/volume*16:34
joannadtantsur: is there a RFE/bug/spec for that? It looks interesting :)16:34
dtantsurjoanna, kinda like https://en.wikipedia.org/wiki/HATEOAS16:36
*** ksumit has joined #openstack-meeting-516:36
TheJuliaI don't remember a bug off-hand.  There was hope of making a v2 api with consistency in that regard, but that hasn't really gone anywhere.16:36
TheJuliaAnyway, I think we are getting off-topic16:36
joannadtantsur: so based on: https://tools.ietf.org/html/rfc5988 ?16:36
joannaTheJulia: sorry :)16:37
TheJuliajoanna: no, it is good to discuss!  Just, we shouldn't spend the entire meeting on it :)16:37
jlvillalDrive by link: https://lwn.net/SubscriberLink/718801/1b8957dbeaa0d0ca/16:38
jlvillalLWN article about booting from remote storage16:38
joannasuming up - we keep /volume/ for grouping, right? :)16:39
TheJuliaI think so yes16:40
joannajlvillal: thank you :)16:40
TheJuliajlvillal: oh the fun of booting systems from remote storage! \o/ :016:41
* TheJulia hears crickets and thinks that is agreement16:41
hshiinashould 'get /v1/volume' return links to connectors and targets?16:41
dtantsurhshiina, yes, it must have a meaningful response16:41
hshiinadtantsur, sure, thanks. i will fix the patch16:42
TheJulia#agreed Keep /volume/ for grouping in the api.16:42
TheJuliaSo if there is nothing else, time for Open Discussion?16:42
TheJulia#topic Open Discussion16:43
*** openstack changes topic to "Open Discussion (Meeting topic: ironic_bfv)"16:43
TheJuliaSo I forgot one thing in announcements.  I will at the community leadership training next week, and on vacation the following week. joanna has volunteered to run the meeting next week.16:44
TheJulia#info joanna to run the next meeting.16:44
*** zaneb has joined #openstack-meeting-516:45
TheJulia#info TheJulia will be moderately unavailable next week, and on vacation the following week. Returning the 24th.16:45
TheJuliaAnything random for discussion today?16:45
TheJuliaI guess we're done for the day then...16:47
*** phuongnh has quit IRC16:47
joannathank you TheJulia, dtantsur, hshiina, jlvillal :)16:47
TheJuliaThank you everyone!  Have a wonderful day and talk to you, at least some, next week!16:48
hshiinathank you everyone!16:48
dtantsurthanks!16:48
TheJulia#endmeeting16:48
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings"16:48
openstackMeeting ended Thu Apr  6 16:48:49 2017 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:48
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-04-06-16.00.html16:48
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-04-06-16.00.txt16:48
*** dtantsur has left #openstack-meeting-516:48
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-04-06-16.00.log.html16:48
*** phuongnh has joined #openstack-meeting-516:50
*** phuongnh has quit IRC16:53
*** zaneb has quit IRC16:56
*** zaneb has joined #openstack-meeting-516:57
*** zaneb has quit IRC16:57
*** hshiina has quit IRC17:14
*** zaneb has joined #openstack-meeting-517:22
*** ksumit has quit IRC17:33
*** ksumit has joined #openstack-meeting-517:50
*** ksumit has left #openstack-meeting-517:50
*** Rockyg has joined #openstack-meeting-518:47
*** Rockyg has quit IRC19:19
*** openstack has joined #openstack-meeting-519:47
*** ChanServ sets mode: +o openstack19:47
*** ksumit has joined #openstack-meeting-520:34
*** ksumit has quit IRC21:25
*** bzhao_ has joined #openstack-meeting-521:43
*** openstack has joined #openstack-meeting-522:03
*** ChanServ sets mode: +o openstack22:03
*** rarcea has quit IRC22:24
*** markvoelker has quit IRC23:37

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