*** Hanchen has joined #openstack-meeting-5 | 01:31 | |
*** iceyao has joined #openstack-meeting-5 | 01:34 | |
*** Hanchen has quit IRC | 02:15 | |
*** Hanchen has joined #openstack-meeting-5 | 02:20 | |
*** Qiming has left #openstack-meeting-5 | 02:46 | |
*** Hanchen has quit IRC | 02:50 | |
*** Hanchen has joined #openstack-meeting-5 | 02:53 | |
*** ramishra has left #openstack-meeting-5 | 03:15 | |
*** Hanchen has quit IRC | 04:02 | |
*** ricolin has joined #openstack-meeting-5 | 04:03 | |
*** zaneb has quit IRC | 04:10 | |
*** zaneb has joined #openstack-meeting-5 | 04:18 | |
*** Hanchen has joined #openstack-meeting-5 | 04:38 | |
*** ksumit has quit IRC | 05:43 | |
*** makowals has joined #openstack-meeting-5 | 06:09 | |
*** matrohon has joined #openstack-meeting-5 | 06:56 | |
*** rarcea has joined #openstack-meeting-5 | 07:15 | |
*** aarefiev_afk is now known as aarefiev | 07:17 | |
*** ralonsoh has joined #openstack-meeting-5 | 08:06 | |
*** ralonsoh_ has joined #openstack-meeting-5 | 08:06 | |
*** ralonsoh has quit IRC | 08:10 | |
*** ralonsoh_ is now known as ralonsoh | 08:12 | |
*** Hanchen has quit IRC | 08:14 | |
*** markvoelker has quit IRC | 08:26 | |
*** iceyao has quit IRC | 09:50 | |
*** iceyao has joined #openstack-meeting-5 | 09:51 | |
*** markvoelker has joined #openstack-meeting-5 | 10:27 | |
*** markvoelker has quit IRC | 10:31 | |
*** markvoelker has joined #openstack-meeting-5 | 12:28 | |
*** markvoelker has quit IRC | 12:32 | |
*** ralonsoh has quit IRC | 13:05 | |
*** ralonsoh has joined #openstack-meeting-5 | 13:06 | |
*** iceyao has quit IRC | 13:46 | |
*** izaakk has quit IRC | 14:13 | |
*** MarkAtwood has quit IRC | 14:14 | |
*** izaakk has joined #openstack-meeting-5 | 14:15 | |
*** MarkAtwood has joined #openstack-meeting-5 | 14:16 | |
*** markvoelker has joined #openstack-meeting-5 | 14:29 | |
*** markvoelker has quit IRC | 14:35 | |
*** markvoelker has joined #openstack-meeting-5 | 14:59 | |
*** matrohon has quit IRC | 15:03 | |
*** zaneb has quit IRC | 15:32 | |
*** mjturek has joined #openstack-meeting-5 | 15:34 | |
*** zaneb has joined #openstack-meeting-5 | 15:34 | |
*** hshiina has joined #openstack-meeting-5 | 15:52 | |
*** dtantsur has joined #openstack-meeting-5 | 16:00 | |
TheJulia | o/ | 16:00 |
---|---|---|
TheJulia | #startmeeting ironic_bfv | 16:00 |
openstack | Meeting 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 |
openstack | Useful 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-5 | 16:00 | |
dtantsur | o/ | 16:00 |
openstack | The meeting name has been set to 'ironic_bfv' | 16:00 |
joanna | o/ | 16:00 |
hshiina | o/ | 16:00 |
TheJulia | Our agenda today, as always can be found on the wiki: https://wiki.openstack.org/wiki/Meetings/Ironic-BFV | 16:01 |
TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic-BFV | 16:01 |
TheJulia | #topic Announcements/Reminders | 16:01 |
*** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic_bfv)" | 16:01 | |
TheJulia | I have no announcements today. Does anyone have any announcements or reminders today? | 16:03 |
hshiina | i have nothing | 16:03 |
TheJulia | Well, actually, I do have one announcement, the common/cinder.py code landed \o/ | 16:03 |
joanna | ah I thought it's the current status part :D | 16:03 |
joanna | anyway \o/ | 16:03 |
TheJulia | True, but it feels like there should be a party now :) | 16:04 |
TheJulia | Anyway, moving on if nobody has anything else for this part of the meeting. | 16:04 |
joanna | that's so true :) | 16:04 |
TheJulia | #topic Current Status | 16:04 |
*** openstack changes topic to "Current Status (Meeting topic: ironic_bfv)" | 16:04 | |
*** aarefiev is now known as aarefiev_afk | 16:05 | |
joanna | there's a new patch: https://review.openstack.org/#/c/453839 that is a follow up to newly landed common cinder interface | 16:05 |
TheJulia | I 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 sense | 16:05 |
TheJulia | #link https://etherpad.openstack.org/p/Ironic-BFV | 16:05 |
TheJulia | #link https://review.openstack.org/#/c/453839 | 16:06 |
TheJulia | Folks, please review ^^^ | 16:06 |
TheJulia | Also, 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 |
hshiina | TheJulia, sure. thank you for your work | 16:08 |
TheJulia | hshiina: 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 merging | 16:08 |
hshiina | TheJulia, i will add the tag | 16:09 |
TheJulia | Thank you! | 16:09 |
TheJulia | Aside from that, does anyone have anything else status wise to report? | 16:09 |
joanna | I'll update the driver, wire, ipxe so they have a proper base, today or early next week | 16:10 |
TheJulia | Awesome, I was going to ask about that | 16: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 |
TheJulia | So I guess onward we go, if there are no objections. | 16:12 |
TheJulia | #topic Planning/Priorities | 16:12 |
*** openstack changes topic to "Planning/Priorities (Meeting topic: ironic_bfv)" | 16:12 | |
TheJulia | From 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 |
joanna | I think since the common patch was merged, we can choose a new one for next week priority list | 16:14 |
TheJulia | I don't think it will take much to get joanna's follow-up rev landed. | 16:14 |
TheJulia | Agreed, 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 |
TheJulia | hshiina: 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 |
hshiina | i agree to review conductor patches following dependency | 16:18 |
joanna | hshiina: If you need any reviews, please let me know, I'll be happy to do it :) | 16:18 |
hshiina | joanna, thank you | 16:19 |
TheJulia | Okay, seems like we're good to move on to discussion? Any objections? | 16:19 |
joanna | + | 16:19 |
hshiina | nothing | 16:19 |
TheJulia | #topic Discussion | 16:20 |
*** openstack changes topic to "Discussion (Meeting topic: ironic_bfv)" | 16:20 | |
TheJulia | I see we have one item today that hshiina has raised regarding API path. | 16:20 |
TheJulia | hshiina you have the floor | 16:20 |
joanna | looks like my fault... :) | 16:21 |
hshiina | joanna gave a comment to the API patch regarding API path | 16:21 |
dtantsur | commented on the patch already, but I'd prefer structured approach, i.e. /volume/{connectors,targets}. easier for API discovery, and otherwise nicer. | 16:21 |
TheJulia | dtantsur: I saw, thank you! | 16:22 |
joanna | I like the approach where api paths are resource based | 16:22 |
joanna | logical grouping is not resource-based, if there are no objects representing the group | 16:22 |
dtantsur | neither is root URL | 16:23 |
joanna | but that's a specific case | 16:23 |
dtantsur | "object" can be a collection of links, like /v1 links to /v1/nodes, /v1/ports, etc | 16:23 |
joanna | while here we'd introduce a mixup | 16:23 |
joanna | yes, but there's not GET for /v1/../volume | 16:24 |
*** ralonsoh has quit IRC | 16:24 | |
dtantsur | if 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 |
dtantsur | there is, just like there is get for /v1 | 16:24 |
joanna | but /v1 is our root for available APIs | 16:25 |
TheJulia | For some reason I also think /v1 is optional, but I might be confusing that part of the api code with something else | 16:25 |
joanna | in other cases path are resource based, and grouping is resource-wise | 16:25 |
dtantsur | joanna, right, and /v1/volume is a root for volume API | 16:25 |
* dtantsur wishes devananda was here, he's a big fan of traverseable API | 16:26 | |
joanna | but there's not "volume" object returned by the API, right? | 16:26 |
TheJulia | There is not | 16:27 |
dtantsur | yes, and I don't see it as a problem | 16:27 |
TheJulia | a get could return what is available, I think we do something similar for provision/state | 16:27 |
joanna | there is no other case where we use a resource that does not represent actual resource just to group stuff | 16:28 |
joanna | or is there? (not sure) | 16:28 |
dtantsur | joanna, / and /v1 as I already noticed | 16:28 |
dtantsur | at least, dunno about something else | 16:28 |
joanna | but /v1 and / are something that may be compared to new index.html :) I don't want to introduce something non-standard that will be misleading | 16:29 |
dtantsur | I don't agree with "non-standard" | 16:30 |
*** zaneb has quit IRC | 16:30 | |
joanna | and 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 |
dtantsur | the / and /v1 are not special, we are just used to them | 16:30 |
dtantsur | joanna, heh, you did not work with Redfish protocol, it's built completely on this principle :) | 16:31 |
dtantsur | anyway, I'm not -2 on volume-connectors, kinda -0.5 rather | 16:31 |
TheJulia | Personally, I'd prefer what was agreed to in the spec. | 16:32 |
joanna | dtantsur: I didn't :( | 16:32 |
TheJulia | since otherwise, we need to go rev the spec, and all of the patches to change it. | 16:32 |
dtantsur | joanna, 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 now | 16:33 |
joanna | TheJulia, dtantsur: if that will create additional work, I'm fine with /volume/volume* | 16:34 |
joanna | dtantsur: is there a RFE/bug/spec for that? It looks interesting :) | 16:34 |
dtantsur | joanna, kinda like https://en.wikipedia.org/wiki/HATEOAS | 16:36 |
*** ksumit has joined #openstack-meeting-5 | 16:36 | |
TheJulia | I 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 |
TheJulia | Anyway, I think we are getting off-topic | 16:36 |
joanna | dtantsur: so based on: https://tools.ietf.org/html/rfc5988 ? | 16:36 |
joanna | TheJulia: sorry :) | 16:37 |
TheJulia | joanna: no, it is good to discuss! Just, we shouldn't spend the entire meeting on it :) | 16:37 |
jlvillal | Drive by link: https://lwn.net/SubscriberLink/718801/1b8957dbeaa0d0ca/ | 16:38 |
jlvillal | LWN article about booting from remote storage | 16:38 |
joanna | suming up - we keep /volume/ for grouping, right? :) | 16:39 |
TheJulia | I think so yes | 16:40 |
joanna | jlvillal: thank you :) | 16:40 |
TheJulia | jlvillal: oh the fun of booting systems from remote storage! \o/ :0 | 16:41 |
* TheJulia hears crickets and thinks that is agreement | 16:41 | |
hshiina | should 'get /v1/volume' return links to connectors and targets? | 16:41 |
dtantsur | hshiina, yes, it must have a meaningful response | 16:41 |
hshiina | dtantsur, sure, thanks. i will fix the patch | 16:42 |
TheJulia | #agreed Keep /volume/ for grouping in the api. | 16:42 |
TheJulia | So if there is nothing else, time for Open Discussion? | 16:42 |
TheJulia | #topic Open Discussion | 16:43 |
*** openstack changes topic to "Open Discussion (Meeting topic: ironic_bfv)" | 16:43 | |
TheJulia | So 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-5 | 16:45 | |
TheJulia | #info TheJulia will be moderately unavailable next week, and on vacation the following week. Returning the 24th. | 16:45 |
TheJulia | Anything random for discussion today? | 16:45 |
TheJulia | I guess we're done for the day then... | 16:47 |
*** phuongnh has quit IRC | 16:47 | |
joanna | thank you TheJulia, dtantsur, hshiina, jlvillal :) | 16:47 |
TheJulia | Thank you everyone! Have a wonderful day and talk to you, at least some, next week! | 16:48 |
hshiina | thank you everyone! | 16:48 |
dtantsur | thanks! | 16:48 |
TheJulia | #endmeeting | 16:48 |
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings" | 16:48 | |
openstack | Meeting ended Thu Apr 6 16:48:49 2017 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:48 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-04-06-16.00.html | 16:48 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-04-06-16.00.txt | 16:48 |
*** dtantsur has left #openstack-meeting-5 | 16:48 | |
openstack | Log: http://eavesdrop.openstack.org/meetings/ironic_bfv/2017/ironic_bfv.2017-04-06-16.00.log.html | 16:48 |
*** phuongnh has joined #openstack-meeting-5 | 16:50 | |
*** phuongnh has quit IRC | 16:53 | |
*** zaneb has quit IRC | 16:56 | |
*** zaneb has joined #openstack-meeting-5 | 16:57 | |
*** zaneb has quit IRC | 16:57 | |
*** hshiina has quit IRC | 17:14 | |
*** zaneb has joined #openstack-meeting-5 | 17:22 | |
*** ksumit has quit IRC | 17:33 | |
*** ksumit has joined #openstack-meeting-5 | 17:50 | |
*** ksumit has left #openstack-meeting-5 | 17:50 | |
*** Rockyg has joined #openstack-meeting-5 | 18:47 | |
*** Rockyg has quit IRC | 19:19 | |
*** openstack has joined #openstack-meeting-5 | 19:47 | |
*** ChanServ sets mode: +o openstack | 19:47 | |
*** ksumit has joined #openstack-meeting-5 | 20:34 | |
*** ksumit has quit IRC | 21:25 | |
*** bzhao_ has joined #openstack-meeting-5 | 21:43 | |
*** openstack has joined #openstack-meeting-5 | 22:03 | |
*** ChanServ sets mode: +o openstack | 22:03 | |
*** rarcea has quit IRC | 22:24 | |
*** markvoelker has quit IRC | 23:37 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!