| *** igordcard has quit IRC | 00:14 | |
| *** matsuhashi has joined #openstack-ironic | 00:27 | |
| *** nosnos has joined #openstack-ironic | 00:43 | |
| *** eghobo has joined #openstack-ironic | 02:44 | |
| *** sseago has quit IRC | 03:08 | |
| *** sseago has joined #openstack-ironic | 03:26 | |
| *** takadayuiko has joined #openstack-ironic | 03:29 | |
| *** nosnos has quit IRC | 03:34 | |
| *** harlowja_ is now known as harlowja_away | 03:47 | |
| *** lazy_prince has joined #openstack-ironic | 03:54 | |
| *** k4n0 has joined #openstack-ironic | 03:59 | |
| *** sysexit has joined #openstack-ironic | 04:19 | |
| *** nosnos has joined #openstack-ironic | 04:26 | |
| *** lokesh184 has joined #openstack-ironic | 04:43 | |
| *** vkdrao has joined #openstack-ironic | 05:08 | |
| *** pradipta_away is now known as pradipta | 05:10 | |
| *** eghobo has quit IRC | 05:27 | |
| *** eghobo has joined #openstack-ironic | 05:27 | |
| openstackgerrit | Sirushti Murugesan proposed a change to openstack/ironic-specs: Whole-Disk-Image Spec https://review.openstack.org/97150 | 06:00 |
|---|---|---|
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/96063 | 06:03 |
| openstackgerrit | Sirushti Murugesan proposed a change to openstack/ironic-specs: Whole-Disk-Image Spec https://review.openstack.org/97150 | 06:13 |
| *** lazy_prince has quit IRC | 06:30 | |
| *** sysexit has quit IRC | 06:30 | |
| *** lazy_prince has joined #openstack-ironic | 06:33 | |
| *** jcoufal has joined #openstack-ironic | 06:39 | |
| *** dtantsur|afk is now known as dtantsur | 07:04 | |
| dtantsur | Good monday, Ironic :) | 07:05 |
| mrda | Hi dtantsur! | 07:06 |
| dtantsur | mrda, hi | 07:06 |
| mrda | My Monday is almost finished, but I'll see you in 12 hours at the meeting :) | 07:07 |
| *** radsy has quit IRC | 07:09 | |
| *** eguz has joined #openstack-ironic | 07:09 | |
| dtantsur | mrda, yeah see you | 07:11 |
| *** eghobo has quit IRC | 07:13 | |
| dtantsur | quick walk, brb | 07:18 |
| *** ifarkas has joined #openstack-ironic | 07:25 | |
| *** lsmola has joined #openstack-ironic | 07:25 | |
| *** eguz has quit IRC | 07:26 | |
| *** Mikhail_D_wk has joined #openstack-ironic | 07:33 | |
| Mikhail_D_wk | Good morning Ironic! :) | 07:37 |
| *** romcheg has joined #openstack-ironic | 07:48 | |
| *** jistr has joined #openstack-ironic | 07:54 | |
| *** max_lobur has joined #openstack-ironic | 08:13 | |
| *** max_lobur has quit IRC | 08:15 | |
| *** max_lobur has joined #openstack-ironic | 08:19 | |
| *** sysexit has joined #openstack-ironic | 08:24 | |
| *** Mikhail_D_wk has left #openstack-ironic | 08:27 | |
| *** Mikhail_D_wk has joined #openstack-ironic | 08:30 | |
| *** Mikhail_D_ltp has joined #openstack-ironic | 08:30 | |
| -openstackstatus- NOTICE: setuptools upstream has broken the world. it's a known issue. we're hoping that a solution materializes soon | 08:31 | |
| *** ChanServ changes topic to "setuptools upstream has broken the world. it's a known issue. we're hoping that a solution materializes soon" | 08:31 | |
| *** max_lobur has quit IRC | 08:33 | |
| *** foexle has joined #openstack-ironic | 08:58 | |
| *** ndipanov has joined #openstack-ironic | 09:05 | |
| *** martyntaylor has joined #openstack-ironic | 09:11 | |
| *** rameshg87 has joined #openstack-ironic | 09:20 | |
| *** mrda is now known as mrda-away | 09:21 | |
| *** igordcard has joined #openstack-ironic | 09:21 | |
| *** pradipta is now known as pradipta_away | 09:46 | |
| *** ramineni has joined #openstack-ironic | 10:02 | |
| *** max_lobur has joined #openstack-ironic | 10:26 | |
| *** rameshg87 has left #openstack-ironic | 11:10 | |
| *** dtantsur is now known as dtantsur|lunch | 11:11 | |
| *** ramineni has quit IRC | 11:12 | |
| romcheg | Morning Ironic! | 11:21 |
| openstackgerrit | Vladimir Kozhukalov proposed a change to openstack/ironic-python-agent: Flow extension uses extension manager from agent https://review.openstack.org/92080 | 11:21 |
| *** sysexit has quit IRC | 11:22 | |
| *** Poornima has joined #openstack-ironic | 11:26 | |
| *** lokesh184 has quit IRC | 11:43 | |
| NobodyCam | Good Morning Ironic | 11:53 |
| *** radsy has joined #openstack-ironic | 11:59 | |
| *** radsy has joined #openstack-ironic | 11:59 | |
| *** Poornima has quit IRC | 12:11 | |
| *** Poornima|mtg has joined #openstack-ironic | 12:11 | |
| *** Poornima|mtg is now known as Poornima | 12:11 | |
| *** radsy has quit IRC | 12:15 | |
| romcheg | Morning NobodyCam~ | 12:21 |
| romcheg | ! | 12:21 |
| *** sysexit has joined #openstack-ironic | 12:22 | |
| *** matsuhashi has quit IRC | 12:22 | |
| yuriyz | morning all | 12:23 |
| NobodyCam | good morning romcheg & yuriyz | 12:24 |
| NobodyCam | 230 meg for a printer driver :-p | 12:26 |
| *** jdob has joined #openstack-ironic | 12:27 | |
| Mikhail_D_ltp | Good morning folks! :) | 12:29 |
| *** dtantsur|lunch is now known as dtantsur | 12:32 | |
| dtantsur | morning romcheg, NobodyCam, Mikhail_D_ltp, yuriyz! | 12:32 |
| NobodyCam | morning Mikhail_D_ltp and dtantsur :) | 12:33 |
| *** praefect has joined #openstack-ironic | 12:33 | |
| dtantsur | folks, anyone works on kind of review dashboard for Ironic? I'm impressed by dashboard for QA (see ML) and willing to create ours :) | 12:34 |
| *** praefect has quit IRC | 12:35 | |
| romcheg | dtantsur: I haven't heard about that | 12:36 |
| *** Poornima has quit IRC | 12:37 | |
| dtantsur | ok, will play with it :) | 12:41 |
| *** k4n0 has quit IRC | 12:41 | |
| *** linggao has joined #openstack-ironic | 12:50 | |
| *** rloo has joined #openstack-ironic | 12:52 | |
| dtantsur | Folks, how do you like this dashboard: http://goo.gl/Fiuhvz ? | 13:01 |
| NobodyCam | dtantsur: The page you requested was not found, or you do not have permission to view this page. | 13:03 |
| NobodyCam | :-p | 13:03 |
| dtantsur | NobodyCam, very strange. Are you logged in? | 13:04 |
| NobodyCam | I was not, it logged me in refresh fixed | 13:04 |
| NobodyCam | nice, is it possiable to add a reviews tagged with critical bugs to that list | 13:05 |
| dtantsur | NobodyCam, it requires Gerrit to know about Launchpad :) Can we add any kind of flags to reviews? Maybe use something in topic branch? | 13:07 |
| NobodyCam | we can chat about that in the meeting | 13:07 |
| NobodyCam | :) | 13:07 |
| rloo | dtantsur: looks good. did we want to ignore work-in-progress ones? | 13:07 |
| *** matty_dubs|gone is now known as matty_dubs | 13:08 | |
| dtantsur | rloo, to me WIP means the author does not want review or will ping someone directly to review it | 13:08 |
| NobodyCam | rloo: good morning, I ok with not having that in this dashboard | 13:08 |
| dtantsur | rloo, and good morning :) | 13:08 |
| rloo | dtantsur: not bad if there are only 5 that haven't gotten feedback within the past 5 days! | 13:08 |
| dtantsur | rloo, seems like that :) | 13:09 |
| rloo | morning NobodyCam. Hi dtantsur. | 13:09 |
| NobodyCam | as folks will ask for help / reviews in WIP reviews | 13:09 |
| dtantsur | note that everything in this list has +1 from Jenkins | 13:09 |
| dtantsur | to me it makes sense, what about you? | 13:09 |
| rloo | dtantsur: ah. a bit concerned about +1 from jenkins. Don't we sometimes get +2s, and jenkins hiccups? | 13:09 |
| NobodyCam | I have not seen that | 13:10 |
| rloo | I'm fine w/o WIP here. Just wanted to check. Once you use it, you get used to it. | 13:10 |
| rloo | I've seen quite a few cases, where we +1, and jenkins -1. So.. if there are no cases where we +1/2 and jenkins -1, then it doesn't hurt to show those, cuz there aren't any. | 13:11 |
| rloo | but if there are cases where jenkins -1, is it worth seeing? | 13:11 |
| dtantsur | rloo, well... I don't know, we have too many patches that got -1, it will distract | 13:11 |
| rloo | dtantsur: can you show +1, +2 regardless of jenkins? | 13:12 |
| *** lazy_prince has quit IRC | 13:12 | |
| rloo | how do people know to do a eg 'recheck no bug'? | 13:13 |
| dtantsur | rloo, it's possible, I'm just not sure, I like it... Let us try | 13:13 |
| dtantsur | rloo, well, we also get email notifications :) | 13:13 |
| rloo | dtantsur: but we get email notifications of the status of all these too ;) | 13:13 |
| dtantsur | rloo, I see this as an instrument for scheduling reviews. And btw it does not show your own patches at all now | 13:14 |
| dtantsur | (this is what I copy-pasted from Sean's work) | 13:14 |
| dtantsur | rloo, do you want something like "My patches requiring attention"? :) | 13:15 |
| rloo | dtantsur: hmm, if it is only for scheduling reviews then it is probably fine. but i guess i'd like to see the entire list in one place. which is why i mentioned the WIP stuff. otherwise, it means I need to go to two different views. | 13:16 |
| rloo | dtantsur: i guess i don't like not seeing everything. it makes me a bit nervous. | 13:16 |
| dtantsur | ack, let me think a bit... | 13:16 |
| rloo | dtantsur: my opinion only though :-) | 13:17 |
| dtantsur | rloo, the more people like it the better :) I don't feel like creating dashboard only for myself | 13:17 |
| rloo | dtantsur: ha ha. Yes, that's true. I suspect this will be useful for other projects too. | 13:18 |
| *** nosnos has quit IRC | 13:18 | |
| dtantsur | at least I'll send a PR to Sean to include this dashboard into his work. So I want to be as good as possible :) | 13:19 |
| dtantsur | NobodyCam, rloo, ok, how do you like it now? http://goo.gl/IywkWK | 13:24 |
| dtantsur | it includes your patches that have any kind of -1 | 13:24 |
| dtantsur | (I can't check it myself, as I don't have -1 right now) | 13:24 |
| *** vkdrao has quit IRC | 13:25 | |
| rloo | dtantsur: i can give one of your patches a -1 if you want to test it ;) | 13:26 |
| dtantsur | rloo, no, thanks))) I tested it via temporary removing condition on author | 13:26 |
| rloo | dtantsur: for the Specs, is there any ordering to that or do you show them all? | 13:26 |
| dtantsur | rloo, all non-WIP | 13:26 |
| dtantsur | I have no clear idea here :) | 13:27 |
| rloo | dtantsur: hopefully we don't get to the same point with specs that we need them broken down into 'needs approval', etc. | 13:27 |
| dtantsur | (I also do not include own WIP patches, as I assume you're working on them right now) | 13:27 |
| dtantsur | rloo, me too. Nova has tens of specs, they may need it | 13:28 |
| *** pradipta_away is now known as pradipta | 13:28 | |
| rloo | dtantsur: is it worth putting some sort of header after the Ironic Specs, to indicate that the rest are ironic reviews? | 13:29 |
| dtantsur | rloo, I think I can't. I'm limited by what Gerrit provides... | 13:29 |
| dtantsur | (and I'm not a Gerrit guru as well) | 13:29 |
| rloo | dtantsur: it is fine for now. just wondering what happens if ironic specs needs to be subdivided. | 13:30 |
| rloo | dtantsur: at this rate, you might become a gerrit guru ;) | 13:30 |
| *** jcoufal has quit IRC | 13:31 | |
| dtantsur | oh, that's possible :D | 13:35 |
| *** spearson has joined #openstack-ironic | 13:56 | |
| openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Wipe any metadata from a nodes disk https://review.openstack.org/93133 | 13:57 |
| jroll | morning ironic :) | 13:59 |
| jroll | dtantsur: that dashboard is awesome | 13:59 |
| NobodyCam | good morning jroll | 13:59 |
| *** jcoufal has joined #openstack-ironic | 14:00 | |
| dtantsur | morning jroll, thanks :) | 14:00 |
| matty_dubs | Mornin' y'all | 14:00 |
| dtantsur | matty_dubs, morning | 14:00 |
| NobodyCam | morning matty_dubs | 14:01 |
| rloo | morning matty_dubs, jroll! | 14:03 |
| *** pradipta is now known as pradipta_away | 14:06 | |
| max_lobur | morning all :) | 14:09 |
| max_lobur | dtantsur: the dashboard looks impressive | 14:10 |
| dtantsur | morning, max_lobur, thanks :) | 14:10 |
| GheRivero | dtantsur: great dashboard! | 14:13 |
| dtantsur | thanks :) hope you folks will find it useful day-to-day indeed | 14:13 |
| NobodyCam | gah | 14:14 |
| *** ChanServ changes topic to "OpenStack Bare Metal Provisioning | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic | Status: https://etherpad.openstack.org/p/IronicWhiteBoard" | 14:15 | |
| -openstackstatus- NOTICE: setuptools issue was fixed in upstream in 3.7.1 and 4.0.1, please, recheck on bug 1325514 | 14:16 | |
| *** mgagne has joined #openstack-ironic | 14:17 | |
| *** blamar has joined #openstack-ironic | 14:19 | |
| *** pradipta_away is now known as pradipta | 14:19 | |
| openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Wipe any metadata from a nodes disk https://review.openstack.org/93133 | 14:21 |
| NobodyCam | brb... quick walkies | 14:21 |
| openstackgerrit | Sirushti Murugesan proposed a change to openstack/ironic-specs: Whole Disk Image Support https://review.openstack.org/97150 | 14:27 |
| *** jgrimm has joined #openstack-ironic | 14:27 | |
| Shrews | dtantsur: nice. but i wonder why 96498 doesn't show up in any of those dashboard categories? | 14:29 |
| Shrews | does it not show your own reviews? | 14:29 |
| *** romcheg1 has joined #openstack-ironic | 14:31 | |
| NobodyCam | we may end up bugging out later on today to avoid bad weather comming this way, keeping an eye on hte weather channel, atm | 14:34 |
| Shrews | NobodyCam: where 'bouts? | 14:35 |
| NobodyCam | we about 20 miles out side omaha right now | 14:36 |
| NobodyCam | plan was to leave tomorrow, however there is a large storm due in then :-p | 14:36 |
| rloo | NobodyCam: be safe. | 14:37 |
| Shrews | eh, ol' "Tornado Alley" never gets bad weather | 14:37 |
| NobodyCam | oh ya... | 14:37 |
| NobodyCam | :-p | 14:37 |
| NobodyCam | heheheh | 14:37 |
| matty_dubs | NobodyCam: Do you keep a blog or something of the places you visit? Seems like it could be pretty neat. | 14:37 |
| NobodyCam | its that whole tornado vs RV thing we don't like | 14:37 |
| Shrews | just tie the rv down with bungie cords, you'll be fine | 14:37 |
| NobodyCam | matty_dubs: there a facebook page that John keeps mostly upto date | 14:38 |
| NobodyCam | I never log in | 14:38 |
| NobodyCam | Shrews: LOL | 14:38 |
| dtantsur | Shrews, it show only your changes that require your attention, i.e. once with any negative feedback. 96498 does not have negative feedback | 14:39 |
| Shrews | dtantsur: i would expect it under "Needs Approval", no? | 14:39 |
| matty_dubs | dtantsur: I just clicked the link to this. Pretty slick! | 14:39 |
| dtantsur | Shrews, other see it, but not you :) you can't approve it | 14:40 |
| NobodyCam | ya looks really good | 14:40 |
| dtantsur | (specially checked, I'm seeing it) | 14:40 |
| Shrews | dtantsur: ok, that's what i was wondering | 14:40 |
| Shrews | would be super awesome if someone put that link in the topic | 14:41 |
| rloo | dtantsur: I don't think it is true that you can't approve your own change. Just that you *shouldn't*. | 14:42 |
| openstackgerrit | Dan Prince proposed a change to openstack/ironic: Port iBoot PDU driver from Nova https://review.openstack.org/50977 | 14:42 |
| dtantsur | rloo, sure :) I didn't account for exceptional cases in this dashboard | 14:42 |
| rloo | dtantsur: I thought that only the 'my patches...' part would be different for individuals. Not sure how I feel about the rest being possibly different. If eg this is used for review day, not everyone will see the same thing :-( | 14:45 |
| *** pradipta is now known as pradipta_away | 14:46 | |
| dtantsur | rloo, it feels quite native to me that you don't see you own patches in a review queue. It's ok when it's 1-3, but according to stats Lucas has 18 patches on review | 14:47 |
| dtantsur | imagine his review queue, if he sees his own patches :) | 14:47 |
| rloo | dtantsur: everyone is different. It doesn't seem native to me ;). Maybe a preference? | 14:48 |
| Shrews | then he would be burdened just like everyone else :) | 14:48 |
| dtantsur | :D | 14:48 |
| dtantsur | it's just a generated link, it does not have any preferences... | 14:49 |
| rloo | dtantsur: for me, it all goes back to me wanting one place that shows me 'everything'. if some things aren't shown, it is disconcerting to me. | 14:49 |
| rloo | dtantsur: I acknowledge that I wouldn't consider myself the 'norm' though ;) | 14:50 |
| dtantsur | rloo, this link is specially for you http://goo.gl/C1b5LF :) | 14:50 |
| Shrews | rloo: i tend to feel the same | 14:51 |
| * Shrews bookmarks rloo's special link | 14:51 | |
| rloo | dtantsur: thx! Now if you would give me one that shows the -1 from jenkins AND the last section being 'WorkInProgress', I'd +2 it! | 14:52 |
| rloo | dtantsur: one other thing. the 'Other' section are those that have at least one negative feedback? Or are there other cases besides that? | 14:53 |
| dtantsur | rloo, yeah, any -1 (not -2) | 14:53 |
| dtantsur | rloo, let my clarify what you want: to show -1 from jenkins everywhere? + one more section that shows everything WIP? | 14:54 |
| rloo | dtantsur: I would suggest then, instead of 'Other', 'Have Negative Feedback' so it is clearer. | 14:54 |
| rloo | dtantsur: yes, -1 from jenkins everywhere, and a last section with WIP. (I am hoping...) | 14:55 |
| dtantsur | rloo, Shrews: how do you like http://goo.gl/5O3SWt ? | 15:00 |
| dtantsur | I maintain two versions in my source code :) | 15:00 |
| rloo | dtansur: /me drools ;) | 15:00 |
| *** Mikhail_D_ltp has quit IRC | 15:00 | |
| rloo | dtantsur: take a look at the Needs Approval list. There's one that has a -1 from jenkins. Useful to know... | 15:00 |
| dtantsur | rloo, :) any more ideas? | 15:01 |
| *** romcheg1 has quit IRC | 15:01 | |
| rloo | dtantsur: I'm in heaven now. This is GREAT! | 15:02 |
| rloo | dtantsur: I don't have much to say about the specs part since I haven't done any reviews there. I did wonder about the ironicclient but that list is usually so small I don't think it is worth adding it here. | 15:03 |
| dtantsur | rloo, specs for ironicclient? do we have them? | 15:04 |
| dtantsur | or do you mean reviews for ironicclient? they are there :) | 15:04 |
| rloo | dtantsur: I meant ironicclient reviews. Are they there? | 15:04 |
| dtantsur | they are, I see a few | 15:04 |
| dtantsur | and IPA as well | 15:04 |
| rloo | dtantsur: oh yeah, I see them now. and the IPA. I didn't actually look before; just assumed you only put ironic stuff. Yay! | 15:05 |
| dtantsur | So folks, we have two dashboards now: http://goo.gl/hqRrRw (mine preferred) http://goo.gl/5O3SWt (rloo and Shrews) | 15:06 |
| Shrews | woot! | 15:06 |
| dtantsur | I don't feel like choosing The One Blessed, use whatever you want :) | 15:06 |
| dtantsur | source code for generating here: https://github.com/Divius/gerrit-dash-creator | 15:06 |
| rloo | thx dtantsur! | 15:07 |
| Shrews | dtantsur: yes, thanks | 15:08 |
| *** romcheg has quit IRC | 15:09 | |
| *** romcheg has joined #openstack-ironic | 15:16 | |
| *** martyntaylor has left #openstack-ironic | 15:17 | |
| NobodyCam | romcheg: on 93133 what are you looking for there, completlly removing the try block or change raise err to just raise? | 15:19 |
| romcheg | NobodyCam: just raise should pass the error but save the stack. That's what I mean | 15:20 |
| NobodyCam | :) | 15:20 |
| NobodyCam | I can do that | 15:20 |
| NobodyCam | hehehe | 15:20 |
| romcheg | I think we experienced the problem like that in Neutron | 15:20 |
| romcheg | It was very hard to debug certain things | 15:21 |
| openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Wipe any metadata from a nodes disk https://review.openstack.org/93133 | 15:22 |
| NobodyCam | :) | 15:22 |
| Shrews | is that what the save_and_reraise_exception class is for? or does that solve a different problem? | 15:25 |
| dtantsur | folks, can you do me a favor before today's meeting? I see we have a lot of bugs with assignee, some does not look like being worked on | 15:27 |
| dtantsur | can everyone have a look a t bugs assigned to him/her and check if you actually work on them? | 15:28 |
| dtantsur | and unassign and set status appropriately, if not | 15:28 |
| * dtantsur does not want to ping everyone via bug report :( | 15:28 | |
| Shrews | NobodyCam: If you get to 50 patch sets, I'm going to throw you a party :) | 15:29 |
| NobodyCam | lol getting close :) | 15:29 |
| NobodyCam | ok heres the real question... for patch #2 (of 2) should we just replace parted? | 15:30 |
| NobodyCam | no lucas | 15:30 |
| NobodyCam | I can bring that up in the meeting | 15:31 |
| romcheg | NobodyCam: If we use sfdisk/sgdisk then I'd vote for replacing parted even though it looks to be better. | 15:32 |
| agordeev | good morning everyone! | 15:33 |
| romcheg | NobodyCam: having more than one dependency for a single action seems to be an overkill | 15:33 |
| agordeev | JayF: morning! just pinging you in case of https://review.openstack.org/#/c/92847 :) | 15:34 |
| NobodyCam | romcheg: yes totally agree, but if we want to do more then just preserve-ephemeral... Ie remove swap and root partitions then I think we'll need to replace parted | 15:35 |
| NobodyCam | morning agordeev | 15:35 |
| dtantsur | morning, agordeev | 15:35 |
| agordeev | NobodyCam: dtantsur morning :) | 15:36 |
| NobodyCam | if we're good with not re-partitionsioning then we can keep parted I think | 15:36 |
| romcheg | NobodyCam: Since everything is encapsulated in DiscPartitioner it should be quite easy to replace parted | 15:37 |
| NobodyCam | oh ya, Just getting a feel for the water befor I jump in :) | 15:37 |
| linggao | Hi Ironic, test cases failed with ImportError: No module named sendfile | 15:37 |
| linggao | Does anyone know what is missing in my venv? | 15:38 |
| NobodyCam | morning linggao .. thats a new on for me | 15:38 |
| NobodyCam | have you rebuilt your venv? | 15:38 |
| linggao | I pulled master, then did tox -r | 15:38 |
| NobodyCam | humm | 15:39 |
| dtantsur | linggao, pysendfile | 15:39 |
| dtantsur | that was recently introduced | 15:39 |
| linggao | file "ironic/common/glance_service/base_image_service.py" | 15:39 |
| linggao | it has import sendfile. | 15:39 |
| dtantsur | linggao, https://review.openstack.org/#/c/92560/ | 15:39 |
| *** foexle has quit IRC | 15:40 | |
| NobodyCam | brb | 15:40 |
| dtantsur | relocating home for the meeting, see you then | 15:41 |
| *** dtantsur is now known as dtantsur|afk | 15:41 | |
| linggao | dtantsur|afk, thanks | 15:42 |
| linggao | :q | 15:42 |
| linggao | :q | 15:42 |
| linggao | woops, wrong window | 15:43 |
| NobodyCam | :-p | 15:43 |
| mgagne | is there a page where I can find the terminology used in Ironic? Chassis, node, etc. | 15:57 |
| *** matty_dubs is now known as matty_dubs|lunch | 15:58 | |
| NobodyCam | mgagne: https://wiki.openstack.org/wiki/Ironic would be a good place to start | 15:58 |
| NobodyCam | if you have a question I'm happy to try and help answer | 15:59 |
| *** ifarkas has quit IRC | 15:59 | |
| mgagne | NobodyCam: couldn't find the word "chassis" on this page =) | 15:59 |
| mgagne | NobodyCam: I googled already and couldn't find a page explaining what a chassis is | 16:00 |
| NobodyCam | http://docs.openstack.org/developer/ironic/webapi/v1.html?highlight=chassis#Chassis | 16:00 |
| mgagne | NobodyCam: yep, I read the page and couldn't come up with a clear definition of what is a chassis. | 16:00 |
| mgagne | NobodyCam: I know nodes are attached to a chassis but my team here do not understand why you would want a chassis. (me neither) | 16:01 |
| NobodyCam | chassis could be thought of as a grouping of nodes. consider a blade type system where you may have many nodes but one bmc | 16:02 |
| rloo | mgagne: a chassis is optional, so you don't need to use/specify it. | 16:02 |
| NobodyCam | ironic is not making full use of the chassis at this point in time | 16:02 |
| NobodyCam | later it may do things like hold like values for nodes in it. | 16:03 |
| NobodyCam | ie. ipmi creds | 16:03 |
| mgagne | cool thanks for the clarification | 16:04 |
| JayF | agordeev: ot | 16:05 |
| JayF | agordeev: it's open in a tab. I'll try to take a good look today. | 16:06 |
| *** dwalleck has joined #openstack-ironic | 16:06 | |
| *** ewindisch has quit IRC | 16:09 | |
| NobodyCam | ok any one know why nova bm does this: https://github.com/openstack/nova/blob/master/nova/cmd/baremetal_deploy_helper.py#L105 | 16:14 |
| NobodyCam | do we need to attempt to partition a disk 3 times | 16:14 |
| NobodyCam | is sfdisk that unstable? | 16:15 |
| *** dwalleck has quit IRC | 16:15 | |
| *** igordcard has quit IRC | 16:16 | |
| *** ellenh has joined #openstack-ironic | 16:17 | |
| *** shakamunyi has joined #openstack-ironic | 16:37 | |
| openstackgerrit | Ruby Loo proposed a change to openstack/python-ironicclient: Add set_provision_state command https://review.openstack.org/89301 | 16:39 |
| Shrews | NobodyCam: see commit 703cdd82f773e57daef56128ca103eb810be9644 | 16:40 |
| *** dwalleck has joined #openstack-ironic | 16:41 | |
| Shrews | NobodyCam: https://bugs.launchpad.net/nova/+bug/1191084 | 16:42 |
| NobodyCam | Shrews: thank you ... great catch | 16:42 |
| Shrews | i'm sometimes useful | 16:43 |
| NobodyCam | s/sometimes/always/ | 16:43 |
| NobodyCam | ;) | 16:44 |
| *** rwsu has joined #openstack-ironic | 16:46 | |
| *** blamar has quit IRC | 16:50 | |
| *** jistr has quit IRC | 16:52 | |
| *** zdiN0bot has joined #openstack-ironic | 16:52 | |
| *** harlowja_away is now known as harlowja_ | 16:54 | |
| *** matty_dubs|lunch is now known as matty_dubs | 16:56 | |
| openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Enforce a minimum time between all IPMI commands https://review.openstack.org/96558 | 16:57 |
| jroll | I feel like force_power_state_during_sync should default to false. thoughts? | 17:09 |
| *** spearson has quit IRC | 17:13 | |
| *** max_lobur has quit IRC | 17:13 | |
| *** ewindisch has joined #openstack-ironic | 17:15 | |
| *** dwalleck has quit IRC | 17:16 | |
| NobodyCam | jroll: hummmm | 17:16 |
| jroll | reasoning: software swooping in and changing the state of a server without telling you, should not be the default. :) | 17:17 |
| NobodyCam | ie the p-task | 17:18 |
| NobodyCam | ? | 17:18 |
| NobodyCam | brb | 17:20 |
| jroll | NobodyCam: yeah, the power task will power on/off nodes to match the state in the db, if force_power_state_during_sync is true | 17:21 |
| jroll | seems like a bad default to me | 17:21 |
| *** sysexit has quit IRC | 17:21 | |
| *** ndipanov has quit IRC | 17:38 | |
| openstackgerrit | Chris Behrens proposed a change to openstack/ironic: Virt driver logging improvements https://review.openstack.org/97047 | 17:41 |
| openstackgerrit | Chris Behrens proposed a change to openstack/ironic: Allow overriding the log level for ironicclient https://review.openstack.org/97048 | 17:41 |
| NobodyCam | I can see both sides, and it really depends on what action I'm trying to take, migrate nodes from novaBM to ironic to not changes power state, testing in the gate heck ya power down the node | 17:41 |
| NobodyCam | so which should be default, the action taken most often I would say | 17:42 |
| *** martyntaylor has joined #openstack-ironic | 17:43 | |
| jroll | I could argue against powering off in the gate | 17:44 |
| jroll | but defaults should be good for users | 17:44 |
| jroll | not for the gate | 17:44 |
| jroll | we have computers to set options in the gate | 17:44 |
| NobodyCam | this is true, | 17:44 |
| NobodyCam | I wont fight to hard as I can see both sides :) so I would be good with either, as long as there is an option to set | 17:45 |
| jroll | ok | 17:45 |
| * jroll sends a patch to find out what others think | 17:45 | |
| *** Mikhail_D_ltp has joined #openstack-ironic | 17:46 | |
| NobodyCam | brb | 17:46 |
| openstackgerrit | Dan Prince proposed a change to openstack/ironic: Port iBoot PDU driver from Nova https://review.openstack.org/50977 | 17:50 |
| devananda | morning, all | 17:51 |
| jroll | \o deva | 17:52 |
| matty_dubs | Howdy devananda | 17:52 |
| *** dwalleck has joined #openstack-ironic | 17:54 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Let ipmitool natively retry commands https://review.openstack.org/96902 | 17:58 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Enforce a minimum time between all IPMI commands https://review.openstack.org/96558 | 18:00 |
| *** ifarkas has joined #openstack-ironic | 18:01 | |
| *** blamar has joined #openstack-ironic | 18:02 | |
| NobodyCam | good morning devananda :) | 18:03 |
| openstackgerrit | A change was merged to stackforge/pyghmi: Added password file support https://review.openstack.org/96569 | 18:03 |
| *** dtantsur|afk is now known as dtantsur | 18:05 | |
| *** zul has quit IRC | 18:08 | |
| *** zul has joined #openstack-ironic | 18:13 | |
| *** zdiN0bot has quit IRC | 18:13 | |
| *** zdiN0bot has joined #openstack-ironic | 18:13 | |
| dtantsur | Folks, how much time ago have you updated your devstack? I'm really curious, whether someone hit the same issue as me: https://review.openstack.org/#/c/97309/ | 18:25 |
| devananda | dtantsur: been a bit .. i should do that today | 18:27 |
| NobodyCam | stepping afk for a few minutes b4 meeting | 18:28 |
| dtantsur | wow, I've seen tempest run timed out after 50 seconds Oo Seems like it can't be easily reproduced, though... | 18:34 |
| openstackgerrit | Russell Haering proposed a change to openstack/ironic: Log exceptions from deploy and tear_down https://review.openstack.org/97318 | 18:35 |
| Shrews | Heh, Apple's new programming language is called Swift. | 18:48 |
| *** zdiN0bot has quit IRC | 18:50 | |
| *** openstackstatus has quit IRC | 18:50 | |
| * dtantsur missed Apple's new programming language Oo | 18:50 | |
| *** openstackstatus has joined #openstack-ironic | 18:51 | |
| *** ChanServ sets mode: +v openstackstatus | 18:51 | |
| NobodyCam | Shrews: oh cool so with apple buying Beats we could see a "Swift Beats" program | 18:53 |
| *** lucasagomes has joined #openstack-ironic | 18:54 | |
| matty_dubs | Heh, the Gizmodo liveblog says "The only thing more boring than watching someone code is reading someone typing about watching someone code, so instead, here are a few other Swifts you might enjoy." | 18:55 |
| matty_dubs | And then includes photos of what I presume are Taylor Swift and Jonathan Swift | 18:55 |
| NobodyCam | lol | 18:56 |
| matty_dubs | (Not that I would know what Jonathan Swift looks like, but I figured it wasn't Bach.) | 18:56 |
| dtantsur | Mikhail_D_ltp, are you around? I just wanted to ask you to rebase https://review.openstack.org/#/c/86536/ on top of master (without depending on part 2) and fix a few things - this one is for higher priority | 18:56 |
| *** sysexit has joined #openstack-ironic | 18:58 | |
| Mikhail_D_ltp | dtantsur: Hi! :) Yeah, I am rebasing this now :) | 18:58 |
| *** mrda-away is now known as mrda | 18:59 | |
| dtantsur | cool! we may try to give it a go this evening - next morning | 18:59 |
| mrda | Morning Ironic | 18:59 |
| dtantsur | morning, mrda :) | 18:59 |
| NobodyCam | good morning mrda | 18:59 |
| dtantsur | oh, regarding morning: 9pm, time of meeting :) | 19:00 |
| devananda | meeting time! | 19:00 |
| NobodyCam | lol 2 pm here | 19:00 |
| mrda | 4:30am here :) | 19:00 |
| devananda | mrda: eek! | 19:00 |
| dtantsur | World is huge :) | 19:00 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements https://review.openstack.org/96228 | 19:00 |
| *** igordcard has joined #openstack-ironic | 19:12 | |
| *** ellenh has quit IRC | 19:18 | |
| *** sysexit has quit IRC | 19:25 | |
| *** zdiN0bot has joined #openstack-ironic | 19:35 | |
| *** zdiN0bot has quit IRC | 19:35 | |
| matty_dubs | rloo: Oops, I just triaged https://bugs.launchpad.net/ironic/+bug/1324169 not realizing you had already done so. And I did it differently... I debated Low vs. Wishlist, but the "Not a bug. It's an enhancement/new feature." description won me over. | 19:38 |
| matty_dubs | Am I wrong, or is it sort of ambiguously in between? | 19:38 |
| matty_dubs | (Note that I don't have a strong opinion, I just want to tune my triage sense.) | 19:38 |
| rloo | matty_dubs: i don't really know. it probably depends on the strings that are being updated and how bad they are :-) | 19:39 |
| matty_dubs | Ha, okay. So my selection wasn't egregiously wrong. | 19:39 |
| *** zdiN0bot has joined #openstack-ironic | 19:39 | |
| *** martyntaylor has quit IRC | 19:46 | |
| *** ellenh has joined #openstack-ironic | 19:52 | |
| openstackgerrit | Ellen Hui proposed a change to openstack/ironic-python-agent: Tries to advertise valid default IP https://review.openstack.org/96980 | 19:52 |
| openstackgerrit | Ellen Hui proposed a change to openstack/ironic-python-agent: Tries to advertise valid default IP https://review.openstack.org/96980 | 19:53 |
| *** zdiN0bot has quit IRC | 19:56 | |
| *** openstackgerrit has quit IRC | 19:58 | |
| *** sysexit has joined #openstack-ironic | 19:58 | |
| *** zdiN0bot has joined #openstack-ironic | 19:59 | |
| lifeless | NobodyCam: (cont) so whats the other patch for ? | 20:00 |
| NobodyCam | the next one will remove the call to make partitions on rebuilds | 20:00 |
| *** yuriyz has quit IRC | 20:01 | |
| *** max_lobur has joined #openstack-ironic | 20:01 | |
| NobodyCam | there where many comments that we need to account for swap and root partitions changing sizes on rebiiilds | 20:01 |
| lucasagomes | yeah, is that possible? | 20:01 |
| NobodyCam | lucasagomes: seem like a missuse of rebuild to me | 20:02 |
| lucasagomes | off-topic: startmeeting milk... milk!? | 20:02 |
| lucasagomes | NobodyCam, right, the nova command doesn't offer the option of changing sizes? | 20:02 |
| dtantsur | yeah, that was suddenly | 20:02 |
| matty_dubs | lucasagomes: LOL, I was JUST wondering that | 20:02 |
| matty_dubs | https://wiki.openstack.org/wiki/Milk | 20:02 |
| matty_dubs | I still don't quite get it | 20:03 |
| NobodyCam | lucasagomes: but you can change flavor | 20:03 |
| lucasagomes | NobodyCam, ahh right | 20:03 |
| lucasagomes | hmm so that's possible | 20:03 |
| lucasagomes | to change the sizes I mean | 20:03 |
| NobodyCam | I don't personally think you should...but you can | 20:03 |
| JayF | You can change flavor *and* preserve ephemeral at the same time? | 20:04 |
| romcheg | Ok guys, it's late here. | 20:04 |
| NobodyCam | night romcheg | 20:04 |
| romcheg | See you tomorrow | 20:04 |
| romcheg | Have a good evening/day! | 20:04 |
| jroll | that doesn't sound feasible in ironic | 20:04 |
| dtantsur | I'm going as well, g'night | 20:04 |
| lucasagomes | same question as JayF | 20:04 |
| jroll | unless you're going to rsync the ephemeral to a different machine | 20:04 |
| lucasagomes | romcheg, dtantsur night | 20:04 |
| jroll | or something like that | 20:04 |
| *** jbjohnso has joined #openstack-ironic | 20:05 | |
| mrda | and I'm off for an early morning run :) See you in 90 mins :) | 20:05 |
| devananda | dtantsur: which of the review dashboards you linked do you want sdague to merge? | 20:06 |
| devananda | dtantsur: now'd be the time to let him know -- he just pinged me about it | 20:06 |
| NobodyCam | well maybe you cann't change flavor on rebuild | 20:06 |
| dtantsur | devananda, both, I'd say. We didn't come to agreement and one size does not fit all | 20:07 |
| devananda | sorry, i was distracted guys -- re: rebuild, I didn' tthink you could change flavor. really? | 20:07 |
| NobodyCam | I'll you could edit it. | 20:07 |
| dtantsur | devananda, I like the first, rloo and Shrews - the second | 20:07 |
| devananda | NobodyCam: ^ | 20:07 |
| jroll | I mean, it's not going to be possible to change flavor on rebuild in ironic | 20:07 |
| NobodyCam | ya was just reviewing | 20:07 |
| devananda | right | 20:07 |
| jroll | except *maybe* if you're booting from cinder or something | 20:07 |
| *** dtantsur is now known as dtantsur|afk | 20:08 | |
| devananda | so romcheg's comment seemed to be regarding non-Nova clients issuing a rebuild | 20:08 |
| devananda | who could presumably have simply edited node.instance_info directly | 20:08 |
| devananda | my answer is, if we're going to support rebuild (which we clearly are) then we need to have some reasonable safeguards | 20:08 |
| devananda | like blocking certain changes to instance_info while an instance is deployed | 20:08 |
| devananda | er | 20:09 |
| devananda | yea. that. :) | 20:09 |
| NobodyCam | devananda: +1 | 20:09 |
| JayF | Side question: Does Ironic /support/ being used by non-nova clients? | 20:09 |
| lucasagomes | hmm :/ yeah we def have to policies around it | 20:09 |
| devananda | JayF: technically anyone could implement a client that speaks to our REST API | 20:10 |
| NobodyCam | JayF: https://review.openstack.org/#/c/89301/ | 20:10 |
| JayF | NobodyCam: what's up? | 20:10 |
| jroll | devananda: technically, yes, but do we actively support things doing so | 20:10 |
| jbjohnso | the graph of check jobs being an ever increasing value is nice and all | 20:10 |
| devananda | JayF: practically speaking, there's a lot of support for making ironic usable as a stand-alone service | 20:10 |
| jroll | oh, I guess so. | 20:10 |
| JayF | OH | 20:10 |
| JayF | it's a shell client | 20:11 |
| NobodyCam | :) | 20:11 |
| *** openstackgerrit has joined #openstack-ironic | 20:11 | |
| lucasagomes | heh we need to approve that patch | 20:11 |
| lucasagomes | 5 +2! | 20:11 |
| rloo | lucasagomes: the problem is jenkins... i'm not sure why it isn't +1'ing. | 20:11 |
| NobodyCam | lol | 20:11 |
| * devananda approves | 20:12 | |
| rloo | hey, thought you had to wait for jenkins first before approving? | 20:12 |
| jbjohnso | nope | 20:12 |
| devananda | rloo: jenkins is having problems | 20:12 |
| devananda | rloo: you can approve any time. jenkins won't merge teh approval until it passes gate | 20:12 |
| lucasagomes | well if jenkins is working properly I would wait, if it's current having problems and the patch is "safe" | 20:12 |
| lucasagomes | I would approve it anyway | 20:12 |
| rloo | ah, ok. for whatever reason, i thought we weren't suppposed to approve until jenkins liked it. good to know! | 20:13 |
| lucasagomes | alright ppl, I'm going eat something | 20:13 |
| lucasagomes | it's late here | 20:13 |
| rloo | lucasagomes: there's one of yours that jenkins doesn't like either. | 20:13 |
| jbjohnso | I've had more downloads of confluent than I would have expected... | 20:13 |
| lucasagomes | and holiday as well :D | 20:13 |
| devananda | lucasagomes: cheers o/ | 20:13 |
| lucasagomes | rloo, heh yeah, I think a couple of them >.<' | 20:14 |
| NobodyCam | night lucasagomes | 20:14 |
| lucasagomes | have a good night everybody, see ye tomorrow! | 20:14 |
| *** zdiN0bot has quit IRC | 20:14 | |
| *** lucasagomes is now known as lucas-dinner | 20:14 | |
| rloo | night lucasagomes! maybe i'll approve... | 20:14 |
| devananda | fwiw ppl, i tlooks like jenkins/nodepool is essentially down right now. -infra team is working on fixing it | 20:14 |
| NobodyCam | so going back to the 93133 split we'll keep parted and rework the logic to not call make partitions on rebuilds with PE set | 20:15 |
| devananda | NobodyCam: ++ | 20:16 |
| devananda | walking home and getting lunch ... bbiab | 20:16 |
| *** ifarkas has quit IRC | 20:17 | |
| *** ellenh has quit IRC | 20:18 | |
| *** zdiN0bot has joined #openstack-ironic | 20:21 | |
| *** dwalleck has quit IRC | 20:25 | |
| *** eghobo has joined #openstack-ironic | 20:26 | |
| *** aboutGod has joined #openstack-ironic | 20:31 | |
| *** zdiN0bot has quit IRC | 20:33 | |
| *** aboutGod has left #openstack-ironic | 20:36 | |
| *** lucas-dinner has quit IRC | 20:40 | |
| openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Cleanup mock patch without `with` part 3 https://review.openstack.org/86536 | 20:45 |
| *** linggao has quit IRC | 20:48 | |
| *** zdiN0bot has joined #openstack-ironic | 20:53 | |
| *** ellenh has joined #openstack-ironic | 20:53 | |
| *** Mikhail_D_ltp has quit IRC | 20:54 | |
| *** jbjohnso has quit IRC | 21:01 | |
| devananda | jroll: take a look at https://review.openstack.org/#/c/87825/ | 21:01 |
| devananda | jroll: neutron-external-ports. i haven't read it yet | 21:01 |
| jroll | devananda: yeah, I saw that a couple weeks ago or something | 21:02 |
| *** jdob has quit IRC | 21:03 | |
| NobodyCam | brb | 21:04 |
| *** matty_dubs is now known as matty_dubs|gone | 21:10 | |
| mrda | . | 21:19 |
| *** jcoufal has quit IRC | 21:25 | |
| *** zdiN0bot has quit IRC | 21:39 | |
| *** Guest70585 has joined #openstack-ironic | 21:44 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/python-ironicclient: Remove non-ASCII chars from source files https://review.openstack.org/95985 | 21:44 |
| *** max_lobur has quit IRC | 21:50 | |
| *** zdiN0bot has joined #openstack-ironic | 21:51 | |
| *** sysexit has quit IRC | 21:52 | |
| mrda | devananda: just fyi, mikal is online now | 21:52 |
| *** anteaya has quit IRC | 21:54 | |
| *** Guest70585 is now known as anteaya | 21:54 | |
| devananda | mrda: thanks! | 21:54 |
| openstackgerrit | Ramakrishnan G proposed a change to openstack/ironic: Add IloDriver and its IloPower module https://review.openstack.org/89500 | 22:08 |
| *** zdiN0bot has quit IRC | 22:09 | |
| *** zdiN0bot has joined #openstack-ironic | 22:15 | |
| *** zdiN0bot has quit IRC | 22:20 | |
| jroll | can someone point me to how to set config options for gate testing etc? is that devtest or? | 22:23 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Default force_power_state_during_sync to False https://review.openstack.org/97361 | 22:23 |
| jroll | related: ^ | 22:23 |
| russell_h | devananda: on my logging patch, is it reasonable to also TRACE log the exception in a separate message? | 22:23 |
| *** BadCub_ has quit IRC | 22:26 | |
| *** NobodyCam has quit IRC | 22:26 | |
| *** jgrimm has quit IRC | 22:28 | |
| *** jrist is now known as jrist-afk | 22:29 | |
| *** zdiN0bot has joined #openstack-ironic | 22:32 | |
| *** zdiN0bot has quit IRC | 22:37 | |
| openstackgerrit | Russell Haering proposed a change to openstack/ironic: Log exceptions from deploy and tear_down https://review.openstack.org/97318 | 22:39 |
| devananda | russell_h: TRACE is really serious. that drops a whole python traceback | 22:43 |
| russell_h | yeah | 22:43 |
| devananda | russell_h: basically means "the app failed in a way that you'll need a stack dump to figure out" | 22:43 |
| devananda | russell_h: which is not going to help in this case, AFAICT | 22:44 |
| russell_h | if a driver is throwing exceptions thats probably true | 22:44 |
| devananda | no ... | 22:44 |
| devananda | russell_h: well, rather than say no, let me ask, how | 22:44 |
| devananda | :) | 22:44 |
| russell_h | I spent half a day trying to figure out where "Port %s could not be found" came from | 22:44 |
| devananda | grep? | 22:44 |
| russell_h | try it, let me know how it goes :) | 22:44 |
| russell_h | here's how I look at it: if you're running with TRACE level logging enabled, you probably want lots of info | 22:45 |
| devananda | that's not how openstack's log levels work | 22:46 |
| russell_h | I'm assuming that in weird python world TRACE is a really high level | 22:46 |
| russell_h | oh | 22:46 |
| devananda | trace is higher than warn | 22:46 |
| devananda | debug -> info -> warn -> trace -> error | 22:46 |
| devananda | IIRC | 22:46 |
| devananda | *debug -> (none) -> info ... | 22:46 |
| russell_h | in that case, you're right, I retract everything :) | 22:47 |
| devananda | :) | 22:47 |
| *** zdiN0bot has joined #openstack-ironic | 22:48 | |
| *** hemna is now known as hemnafk | 22:49 | |
| russell_h | is trace even a real log level | 22:49 |
| devananda | russell_h: it's not listed here https://docs.python.org/2/library/logging.html#levels | 22:50 |
| *** BadCub has joined #openstack-ironic | 22:51 | |
| *** NobodyCam has joined #openstack-ironic | 22:52 | |
| devananda | russell_h: ah - i am thinking of Logging.exception | 22:58 |
| devananda | russell_h: it logs at error level and includes a traceback | 22:59 |
| devananda | https://docs.python.org/2/library/logging.html#logger-objects | 22:59 |
| *** zdiN0bot has quit IRC | 22:59 | |
| *** harlowja_ has quit IRC | 23:04 | |
| *** harlowja has joined #openstack-ironic | 23:04 | |
| *** zdiN0bot has joined #openstack-ironic | 23:10 | |
| *** igordcard has quit IRC | 23:23 | |
| *** romcheg has quit IRC | 23:23 | |
| *** radsy has joined #openstack-ironic | 23:25 | |
| *** radsy has joined #openstack-ironic | 23:25 | |
| *** zdiN0bot has quit IRC | 23:29 | |
| *** zdiN0bot has joined #openstack-ironic | 23:31 | |
| mrda | Just to report back on my action from the meeting today: jhesketh is hoping to look into ironic 3rd party ci things today or tomorrow. | 23:37 |
| mrda | I guess that's for devananda ot NobodyCam ^^^ | 23:37 |
| *** zdiN0bot has quit IRC | 23:38 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Remove 'fake' and 'ssh' drivers from default enabled list https://review.openstack.org/88711 | 23:42 |
| *** aweeks has joined #openstack-ironic | 23:45 | |
| *** zdiN0bot has joined #openstack-ironic | 23:46 | |
| *** jogo has joined #openstack-ironic | 23:49 | |
| jogo | would like to get an ironic +1 on https://review.openstack.org/#/c/94043/ | 23:49 |
| jogo | (nova patch) | 23:49 |
| jroll | jogo: at a glance, I don't believe that touches ironic at all, but I'll leave that to someone else to verify | 23:50 |
| jogo | jroll: correct it doesn't touch ironic but it touches nova baremetal | 23:51 |
| jogo | so feature parity etc | 23:51 |
| jogo | it removes unused code | 23:51 |
| jogo | but want to double check | 23:51 |
| jroll | right, and it looks like there's a baremetal test failing | 23:51 |
| jroll | I know nothing about baremetal so I'll defer :) | 23:51 |
| jogo | heh thanks | 23:52 |
| *** zdiN0bot has quit IRC | 23:52 | |
| devananda | jogo: /me looks | 23:54 |
| devananda | jogo: are you familiar with the underlying issue there? | 23:55 |
| jogo | devananda: yeah | 23:57 |
| devananda | jogo: i looked into it briefly a few months back but didn't quite wrap my head around what it /should/ be doing | 23:57 |
| jogo | capabilities were a think that compute nodes broadcast to the scheduler and were never stored in DB | 23:58 |
| jogo | and nothing but baremetal was using it | 23:58 |
| jogo | so there is a bit of backlog in #tripleo that gives some context | 23:58 |
| *** matsuhashi has joined #openstack-ironic | 23:59 | |
| devananda | jogo: right -- so what's the correct new behavior, eg. for ironic? | 23:59 |
| jogo | in short we just want to confirm that ironic doesn't use 'host capabilities' as thy don'y work | 23:59 |
| devananda | we don't use it any more than nova-bm used it :) | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!