*** saripurigopi has joined #openstack-ironic | 00:01 | |
*** naohirot has joined #openstack-ironic | 00:05 | |
*** Sukhdev has quit IRC | 00:13 | |
*** Sukhdev has joined #openstack-ironic | 00:13 | |
*** amotoki has joined #openstack-ironic | 00:17 | |
*** karimb has joined #openstack-ironic | 00:17 | |
*** garthb has quit IRC | 00:21 | |
*** amotoki has quit IRC | 00:22 | |
*** shadower has quit IRC | 00:23 | |
*** sdake has quit IRC | 00:23 | |
*** shadower has joined #openstack-ironic | 00:23 | |
*** tsekiyama has quit IRC | 00:26 | |
*** enikanorov has joined #openstack-ironic | 00:27 | |
*** enikanorov_ has quit IRC | 00:28 | |
*** mtanino has quit IRC | 00:29 | |
openstackgerrit | Shuquan Huang proposed openstack/ironic: Fix ubuntu install command in install guide https://review.openstack.org/229333 | 00:33 |
---|---|---|
openstackgerrit | Merged openstack/ironic: Use standard locale when executing 'dd' command https://review.openstack.org/216478 | 00:39 |
*** itsuugo has quit IRC | 00:40 | |
*** alexpilotti has joined #openstack-ironic | 00:44 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Move unit tests to correct directory https://review.openstack.org/229196 | 00:47 |
jroll | devananda: backport of that locale bug breaking deploys https://review.openstack.org/#/c/229692/ | 00:47 |
*** alexpilotti has quit IRC | 00:48 | |
jlvillal | jroll, The patch is breaking deploys? Or need to do a backport patch, to fix breaking deploys? | 00:54 |
jroll | jlvillal: the bug is 'deploys are broken in japanese locale' | 00:54 |
jroll | that fixes it | 00:54 |
jlvillal | jroll, Ah, okay. I first read it as you had back-ported it and then it broke deploys. | 00:55 |
jroll | nooooo. | 00:55 |
jlvillal | Thus I asked my question :) | 00:55 |
*** Sukhdev has quit IRC | 00:57 | |
*** enikanorov_ has joined #openstack-ironic | 01:04 | |
*** enikanorov has quit IRC | 01:06 | |
*** amotoki has joined #openstack-ironic | 01:11 | |
*** lazy_prince has joined #openstack-ironic | 01:13 | |
*** amotoki has quit IRC | 01:15 | |
*** karimb has quit IRC | 01:28 | |
*** saripurigopi has quit IRC | 01:28 | |
*** killer_prince has joined #openstack-ironic | 01:39 | |
*** alexpilotti has joined #openstack-ironic | 01:39 | |
*** lazy_prince has quit IRC | 01:42 | |
*** ijw has quit IRC | 01:46 | |
*** alexpilotti has quit IRC | 01:48 | |
killer_prince | Not sure if its right place to ask but does any one know how to enable colors for openstack log files in vi..? | 01:52 |
*** amotoki has joined #openstack-ironic | 02:05 | |
jlvillal | killer_prince, I would assume you need to 'teach' vim about them. So I think you need a vim syntax file for it. | 02:08 |
* jlvillal suggests lots of googling | 02:08 | |
*** jerrygb has joined #openstack-ironic | 02:09 | |
killer_prince | yea.. a quik google search point me to some syntax file for syslog etc.. but nothing for openstack logs.. probably I cam use them and extend them for openstack... | 02:09 |
*** amotoki has quit IRC | 02:09 | |
*** sambetts has quit IRC | 02:11 | |
*** saripurigopi has joined #openstack-ironic | 02:16 | |
*** sambetts has joined #openstack-ironic | 02:17 | |
jlvillal | killer_prince, sounds like a plan :) | 02:25 |
*** amotoki has joined #openstack-ironic | 02:28 | |
*** dims__ has quit IRC | 02:29 | |
*** dims_ has joined #openstack-ironic | 02:29 | |
openstackgerrit | Haomeng,Wang proposed openstack/ironic: Fix ubuntu install command in install guide https://review.openstack.org/229333 | 02:29 |
*** lazy_prince has joined #openstack-ironic | 02:32 | |
*** killer_prince has quit IRC | 02:32 | |
*** saripurigopi has quit IRC | 02:34 | |
*** itsuugo has joined #openstack-ironic | 02:37 | |
*** amotoki has quit IRC | 02:49 | |
*** enikanorov has joined #openstack-ironic | 02:57 | |
openstackgerrit | Merged openstack/bifrost: gcc can also be missing on deb distros https://review.openstack.org/227047 | 02:59 |
*** enikanorov_ has quit IRC | 03:00 | |
*** amotoki has joined #openstack-ironic | 03:02 | |
*** amotoki_ has joined #openstack-ironic | 03:04 | |
*** willsama has joined #openstack-ironic | 03:07 | |
*** amotoki has quit IRC | 03:08 | |
openstackgerrit | Merged openstack/python-ironicclient: Replace six.iteritems() with .items() https://review.openstack.org/228572 | 03:08 |
*** itsuugo has quit IRC | 03:09 | |
*** amotoki_ has quit IRC | 03:10 | |
openstackgerrit | Merged openstack/ironic: Fix ubuntu install command in install guide https://review.openstack.org/229333 | 03:11 |
*** harshs has quit IRC | 03:11 | |
*** itsuugo has joined #openstack-ironic | 03:13 | |
*** dims__ has joined #openstack-ironic | 03:15 | |
*** amotoki has joined #openstack-ironic | 03:15 | |
*** dims___ has joined #openstack-ironic | 03:16 | |
*** itsuugo has quit IRC | 03:18 | |
*** dims____ has joined #openstack-ironic | 03:18 | |
*** dims_ has quit IRC | 03:18 | |
*** rloo has quit IRC | 03:19 | |
*** lazy_prince has quit IRC | 03:19 | |
*** dims__ has quit IRC | 03:20 | |
*** dims___ has quit IRC | 03:21 | |
*** puranamr has joined #openstack-ironic | 03:23 | |
*** Nisha has joined #openstack-ironic | 03:25 | |
*** r-daneel has quit IRC | 03:26 | |
*** alexpilotti has joined #openstack-ironic | 03:27 | |
*** alexpilotti has quit IRC | 03:32 | |
*** enikanorov_ has joined #openstack-ironic | 03:32 | |
*** enikanorov has quit IRC | 03:34 | |
*** puranamr has quit IRC | 03:36 | |
*** dims____ has quit IRC | 03:37 | |
*** ijw has joined #openstack-ironic | 03:39 | |
*** amotoki has quit IRC | 03:40 | |
*** lazy_prince has joined #openstack-ironic | 03:51 | |
*** amotoki has joined #openstack-ironic | 03:52 | |
*** links has joined #openstack-ironic | 03:55 | |
*** puranamr has joined #openstack-ironic | 03:56 | |
*** jerrygb has quit IRC | 03:57 | |
*** jerrygb has joined #openstack-ironic | 03:58 | |
*** boris-42 has quit IRC | 03:59 | |
*** sdake has joined #openstack-ironic | 04:01 | |
*** Nisha has quit IRC | 04:01 | |
*** puranamr has quit IRC | 04:06 | |
*** rameshg87 has joined #openstack-ironic | 04:08 | |
jlvillal | rameshg87, I heard a rumor you are working on a patch to start using ironic-lib. Is that true? | 04:09 |
rameshg87 | jlvillal: ah yeah, it's true | 04:10 |
jlvillal | Great :) | 04:10 |
jlvillal | rameshg87, Is it already posted? | 04:10 |
rameshg87 | jlvillal: faizan was doing the work, he is trying to continue that work | 04:10 |
rameshg87 | jlvillal: he stopped working on this ironic, but he might just continue this stuff | 04:10 |
rameshg87 | jlvillal: if he doesn't get time to complete it, I will pick it up this week | 04:11 |
jlvillal | rameshg87, Okay. Just interested in reviewing it when it the patch is uploaded. | 04:11 |
rameshg87 | jlvillal: sure, will add you | 04:11 |
jlvillal | rameshg87, Thanks! | 04:11 |
rameshg87 | jlvillal: there is WIP patch btw | 04:11 |
* rameshg87 finds it out | 04:11 | |
rameshg87 | jlvillal: https://review.openstack.org/#/c/184443/ | 04:12 |
jlvillal | rameshg87, Thank you. | 04:12 |
rameshg87 | jlvillal: this needs to be polished and tested | 04:12 |
rameshg87 | jlvillal: and he has patch to refactor agent to use ironic-lib as well: https://review.openstack.org/#/q/owner:faizan.barmawer%2540gmail.com+status:open,n,z | 04:13 |
jlvillal | rameshg87, I added myself as a reviewer so I can find it. Thanks. | 04:13 |
rameshg87 | if you are interested in seeing it | 04:13 |
jlvillal | rameshg87, I am. Thanks again. | 04:13 |
jlvillal | rameshg87, strange. Using firefox get an error with that link. But chrome works. Too lazy to try to figure it out and file a bug on firefox :) | 04:15 |
*** Sukhdev has joined #openstack-ironic | 04:21 | |
rameshg87 | jlvillal: :) | 04:22 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Refactor IRMCVirtualMediaIscsiDeploy by applying new BootInterface https://review.openstack.org/221371 | 04:22 |
*** willsama has quit IRC | 04:23 | |
jlvillal | rameshg87, As a note. Quite a few ironic-lib patches were uploaded today. | 04:25 |
jlvillal | Trying to get it ready to use. | 04:25 |
*** sdake_ has joined #openstack-ironic | 04:25 | |
* jlvillal prepares to call it a night. | 04:25 | |
rameshg87 | jlvillal: thanks. I will have a look at them before you come back tomorrow again :) | 04:25 |
*** Nisha has joined #openstack-ironic | 04:25 | |
jlvillal | Thanks. Good day | 04:25 |
rameshg87 | goodnight | 04:25 |
*** sdake has quit IRC | 04:28 | |
*** harshs has joined #openstack-ironic | 04:28 | |
*** VikasC has joined #openstack-ironic | 04:46 | |
*** Marga_ has quit IRC | 04:47 | |
*** harshs has quit IRC | 04:59 | |
openstackgerrit | Merged openstack/ironic-lib: Update link for where to report bugs https://review.openstack.org/229620 | 05:08 |
*** alexpilotti has joined #openstack-ironic | 05:15 | |
*** alexpilotti has quit IRC | 05:19 | |
*** sdake_ is now known as sdake | 05:21 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Refactor IRMCVirtualMediaAgentDeploy by applying new BootInterface https://review.openstack.org/221577 | 05:23 |
openstackgerrit | Merged openstack/ironic-lib: Remove README.md and update README.rst https://review.openstack.org/229622 | 05:42 |
openstackgerrit | Merged openstack/ironic-lib: Remove execute permission from exception.py https://review.openstack.org/229619 | 05:44 |
*** Sukhdev has quit IRC | 05:56 | |
*** Sukhdev has joined #openstack-ironic | 05:56 | |
*** hblixt has joined #openstack-ironic | 05:57 | |
*** Sukhdev has quit IRC | 05:59 | |
*** ijw has quit IRC | 06:00 | |
openstackgerrit | Merged openstack/ironic-lib: Move tests/ under ironic_lib/, use ostestr https://review.openstack.org/229483 | 06:04 |
openstackgerrit | Merged openstack/ironic-lib: Better optimize test runs https://review.openstack.org/229484 | 06:04 |
openstackgerrit | Merged openstack/ironic-lib: Add a cover target to tox https://review.openstack.org/229485 | 06:04 |
*** lsmola has joined #openstack-ironic | 06:06 | |
openstackgerrit | Ramakrishnan G proposed openstack/ironic-lib: Put py34 first in the env order of tox https://review.openstack.org/229616 | 06:07 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Imported Translations from Zanata https://review.openstack.org/229258 | 06:28 |
*** Marga_ has joined #openstack-ironic | 06:39 | |
*** Marga_ has quit IRC | 06:47 | |
*** aarefiev has joined #openstack-ironic | 06:54 | |
*** olaph has quit IRC | 06:55 | |
*** vdrok has joined #openstack-ironic | 06:55 | |
vdrok | morning ironic | 06:58 |
*** alexpilotti has joined #openstack-ironic | 07:03 | |
*** karimb has joined #openstack-ironic | 07:06 | |
*** boris-42 has joined #openstack-ironic | 07:07 | |
*** alexpilotti has quit IRC | 07:08 | |
*** penick has joined #openstack-ironic | 07:17 | |
*** yuikotakada has joined #openstack-ironic | 07:20 | |
*** penick has quit IRC | 07:24 | |
*** ifarkas has joined #openstack-ironic | 07:25 | |
sambetts | Morning all o/ | 07:31 |
yuikotakada | sambetts, o/ | 07:34 |
sambetts | Hey yuikotakada ! | 07:36 |
yuikotakada | sambetts, hi, what's the weather like in your country? I will go to Dublin in the next week :) | 07:37 |
sambetts | yuikotakada: Its not bad at the moment, pretty mild, between 10C and 15C | 07:39 |
yuikotakada | sambetts, thanks, sounds a little bit colder than Japan. should I bring some sweaters? | 07:41 |
sambetts | yuikotakada: I would, the wind can have a bit of a bite to it | 07:42 |
*** Marga_ has joined #openstack-ironic | 07:44 | |
yuikotakada | sambetts, :) | 07:44 |
*** openstackgerrit has quit IRC | 07:46 | |
sambetts | yuikotakada: what the weather like in Tokyo, I got a weather warning through on my emails but that was for Okinawa region | 07:47 |
*** e0ne has joined #openstack-ironic | 07:47 | |
*** openstackgerrit has joined #openstack-ironic | 07:47 | |
*** Marga_ has quit IRC | 07:48 | |
*** e0ne has quit IRC | 07:49 | |
yuikotakada | sambetts, it's windy and will be raining today, but almost calm recently :) | 07:52 |
*** e0ne has joined #openstack-ironic | 07:54 | |
sambetts | yuikotakada: Thats good news! I'm really looking forward to exploring Tokyo, want to go shopping in Akiba :) | 07:55 |
*** e0ne has quit IRC | 07:56 | |
yuikotakada | sambetts, hehe, you will be able to enjoy there, no doubt. | 07:57 |
sambetts | yuikotakada: my partner wants to visit the big pokecentre too :-P | 07:59 |
*** priteau has joined #openstack-ironic | 08:01 | |
yuikotakada | sambetts, oh, will you come with your girlfriend? :D haha, does she like pokemon? | 08:02 |
*** Nisha has quit IRC | 08:06 | |
*** mgoddard has quit IRC | 08:06 | |
sambetts | yuikotakada: Yeah, we've both wanted to visit tokyo for ages so I'm bringing her with me, she's a huge fan of pokemon :-P | 08:06 |
*** priteau has quit IRC | 08:08 | |
*** derekh has joined #openstack-ironic | 08:08 | |
*** itsuugo has joined #openstack-ironic | 08:11 | |
*** romainh has joined #openstack-ironic | 08:13 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Enable radosgw support in ironic https://review.openstack.org/229389 | 08:13 |
openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Support proxy in HTTPClient https://review.openstack.org/229286 | 08:13 |
yuikotakada | sambetts, is it in Ikebukuro? Both Akiba and Ikebukuro are near by Summit place, enjoy! | 08:15 |
*** alexpilotti has joined #openstack-ironic | 08:18 | |
*** jistr has joined #openstack-ironic | 08:18 | |
*** mgoddard has joined #openstack-ironic | 08:20 | |
sambetts | yuikotakada: Google reckons its about 40 mins on the Yamanote to Ikebukuro, does that sound about right? | 08:23 |
*** e0ne has joined #openstack-ironic | 08:26 | |
yuikotakada | sambetts, does it mean from the hotel at Shinagawa? Or just station to station? | 08:26 |
sambetts | from the Hotel | 08:31 |
sambetts | yuikotakada: It reckons 30 mins from station to station | 08:32 |
*** ndipanov has joined #openstack-ironic | 08:34 | |
yuikotakada | sambetts, yeah, it seems true. Prince Hotel is a little bit far from the station | 08:34 |
*** e0ne has quit IRC | 08:36 | |
*** killer_prince has joined #openstack-ironic | 08:37 | |
*** e0ne has joined #openstack-ironic | 08:38 | |
*** MattMan has left #openstack-ironic | 08:38 | |
*** yuriyz has joined #openstack-ironic | 08:38 | |
*** lazy_prince has quit IRC | 08:41 | |
sambetts | yuikotakada: :) its not to far | 08:42 |
*** Marga_ has joined #openstack-ironic | 08:45 | |
yuikotakada | sambetts, :) | 08:46 |
*** mgoddard has left #openstack-ironic | 08:48 | |
*** Marga_ has quit IRC | 08:50 | |
*** killer_prince has quit IRC | 08:56 | |
*** lazy_prince has joined #openstack-ironic | 08:56 | |
*** mgoddard has joined #openstack-ironic | 09:06 | |
openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Support proxy in HTTPClient https://review.openstack.org/229286 | 09:11 |
*** alexpilotti has quit IRC | 09:13 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Enable radosgw support in ironic https://review.openstack.org/229389 | 09:13 |
*** foexle has joined #openstack-ironic | 09:14 | |
*** x3k` has joined #openstack-ironic | 09:18 | |
*** Nisha has joined #openstack-ironic | 09:21 | |
*** praneshp has quit IRC | 09:21 | |
*** mgoddard_ has joined #openstack-ironic | 09:25 | |
*** mgoddard has left #openstack-ironic | 09:28 | |
*** saripurigopi has joined #openstack-ironic | 09:30 | |
*** marzif has joined #openstack-ironic | 09:32 | |
*** x3k` is now known as x3k | 09:35 | |
*** sdake_ has joined #openstack-ironic | 09:38 | |
*** romcheg has joined #openstack-ironic | 09:39 | |
*** mgoddard_ has quit IRC | 09:40 | |
*** mgoddard_ has joined #openstack-ironic | 09:40 | |
*** pelix has joined #openstack-ironic | 09:41 | |
*** sdake has quit IRC | 09:42 | |
*** Marga_ has joined #openstack-ironic | 09:46 | |
*** digimonde has joined #openstack-ironic | 09:47 | |
*** MattMan has joined #openstack-ironic | 09:47 | |
*** boris-42 has quit IRC | 09:49 | |
*** Marga_ has quit IRC | 09:51 | |
*** e0ne has quit IRC | 09:53 | |
*** dims has joined #openstack-ironic | 09:55 | |
*** e0ne has joined #openstack-ironic | 09:56 | |
*** naohirot has quit IRC | 09:58 | |
*** alexpilotti has joined #openstack-ironic | 09:58 | |
lazy_prince | Hi.. | 10:03 |
sambetts | Hey lazy_prince | 10:05 |
lazy_prince | So I have a doubt.. The network ports in neutron for baremetal ironic instance is deleted by nova or ironic..? | 10:05 |
sambetts | lazy_prince: Nova I believe | 10:06 |
lazy_prince | aha.. k.. and its done as part or clean_up call in the driver..? | 10:07 |
sambetts | lazy_prince: the neutron ports are created and deleted outside of the virt driver as far i can tell, the virt driver is just called to plug the hypervisor side of the port, so plug vifs and unplug vifs | 10:10 |
sambetts | are ironic specific but the rest isn't | 10:10 |
lazy_prince | okay.. | 10:10 |
*** amotoki_ has joined #openstack-ironic | 10:11 | |
*** romainh has quit IRC | 10:19 | |
*** amotoki_ has quit IRC | 10:19 | |
*** alexpilotti has quit IRC | 10:35 | |
*** alexpilotti has joined #openstack-ironic | 10:37 | |
*** Nisha_away has joined #openstack-ironic | 10:40 | |
*** Nisha has quit IRC | 10:40 | |
*** Nisha_away has quit IRC | 10:44 | |
*** Nisha_away has joined #openstack-ironic | 10:45 | |
*** amotoki_ has joined #openstack-ironic | 10:46 | |
*** Marga_ has joined #openstack-ironic | 10:46 | |
*** Marga_ has quit IRC | 10:51 | |
*** Nisha_away has quit IRC | 10:57 | |
*** trown|outttypeww is now known as trown | 11:00 | |
*** marzif has quit IRC | 11:04 | |
*** marzif has joined #openstack-ironic | 11:05 | |
*** itsuugo has quit IRC | 11:06 | |
*** itsuugo has joined #openstack-ironic | 11:13 | |
*** marzif has quit IRC | 11:14 | |
*** marzif has joined #openstack-ironic | 11:15 | |
*** marzif has quit IRC | 11:16 | |
*** marzif has joined #openstack-ironic | 11:16 | |
* rameshg87 goes home | 11:19 | |
*** rameshg87 has quit IRC | 11:19 | |
*** marzif has quit IRC | 11:26 | |
*** marzif has joined #openstack-ironic | 11:26 | |
*** itsuugo has quit IRC | 11:31 | |
*** baoli has joined #openstack-ironic | 11:31 | |
*** marzif has quit IRC | 11:36 | |
*** marzif has joined #openstack-ironic | 11:37 | |
*** agordeev has joined #openstack-ironic | 11:38 | |
*** marzif has quit IRC | 11:46 | |
*** dtantsur|afk is now known as dtantsur | 11:47 | |
*** Marga_ has joined #openstack-ironic | 11:47 | |
dtantsur | morning/afternoon ironic, hi yuikotakada, sambetts, vdrok | 11:47 |
* dtantsur was on the meeting | 11:47 | |
*** marzif has joined #openstack-ironic | 11:49 | |
*** Nisha has joined #openstack-ironic | 11:50 | |
*** itsuugo has joined #openstack-ironic | 11:51 | |
TheJulia | good morning everyone | 11:51 |
dtantsur | morning TheJulia | 11:51 |
*** baoli has quit IRC | 11:52 | |
*** Marga_ has quit IRC | 11:52 | |
*** amotoki has quit IRC | 11:52 | |
*** amotoki_ is now known as amotoki | 11:52 | |
TheJulia | dtantsur: I'm looking at inspector and I've got two questions, What is the recommended database config? Basically hitting errors about tables not existing, its not pretty. Additionally, I'm wondering why it only seems to be for the non agent variations, is there a reason behind that? | 11:53 |
dtantsur | TheJulia, well, we've just switched to alembic for migrations, so things might be a bit messy. Did you run the 'ironic-inspector-dbsync --config-file ... upgrade"? | 11:54 |
dtantsur | TheJulia, second, "for non-agent variations" you mean ironic inspection interface? | 11:55 |
TheJulia | dtantsur: I did run the dbsync upgrade, it exits cleanly but the service tosses exceptions about tables missing | 11:56 |
TheJulia | dtantsur: yeah, like agent_ssh | 11:57 |
TheJulia | pxe_ssh has it wired in, agent_ssh does not, just curious why not | 11:57 |
trown | oh ... the inspect interface in ironic | 11:57 |
dtantsur | I don't know, probably we should just fix it... maybe there was some objection from J* | 11:57 |
dtantsur | sorry, I've not payed too much attention to this stuff recently | 11:57 |
trown | I think that is not actually working in the pxe_ssh either | 11:57 |
trown | there are at least downstream reports that doing inspection that way hangs, I have not tried to reproduce it | 11:58 |
dtantsur | TheJulia, as to upgrade, did you supply the same configuration file to both dbsync tool and to inspector itself? | 11:58 |
TheJulia | seems simple enough to wire through fwiw, I did it locally in about 10 minutes | 11:58 |
dtantsur | yeah | 11:58 |
dtantsur | trown, hmm, it works on devstack :) | 11:59 |
dtantsur | TheJulia, this is our devstack plugin, it might help you: https://github.com/openstack/ironic-inspector/blob/master/devstack/plugin.sh | 11:59 |
trown | dtantsur: ya, who knows... I am not convinced it is not user error | 12:00 |
dtantsur | trown, that's what we run in gate: https://github.com/openstack/ironic-inspector/blob/master/devstack/exercise.sh#L87-L89 | 12:00 |
trown | ah cool | 12:00 |
dtantsur | TheJulia, will you submit a patch to wire it in for agent drivers, or should I? | 12:00 |
TheJulia | dtantsur: I'll put it on my list to do later today | 12:02 |
dtantsur | thanks a lot! | 12:02 |
TheJulia | no problem! :) | 12:02 |
dtantsur | yeah, previously we didn't support IPA as an inspection ramdisk, maybe that's why I didn't do it... | 12:02 |
TheJulia | Hmm.. sqlite, I guess that works, I was kind of hoping to get it running with mysql since I already had it running | 12:03 |
dtantsur | TheJulia, it is supposed to work, not sure why it could fail... | 12:04 |
*** aarefiev has quit IRC | 12:04 | |
dtantsur | TheJulia, did you ever run inspector before with the same database? | 12:05 |
*** marzif has quit IRC | 12:06 | |
*** marzif has joined #openstack-ironic | 12:07 | |
TheJulia | dtantsur: well, some of the schema fails to load in mysql because on of hte fields lacks a max length | 12:08 |
TheJulia | dtantsur: just trying to get it stood up for the first time really | 12:09 |
*** romainh has joined #openstack-ironic | 12:09 | |
*** alexpilo_ has joined #openstack-ironic | 12:11 | |
*** aarefiev has joined #openstack-ironic | 12:12 | |
sambetts | TheJulia: Which field? I wonder if we need to do some sqlite -> mysql compat stuff, I thought Alchemy would handle that though :/ | 12:15 |
*** alexpilo_ has quit IRC | 12:15 | |
dtantsur | oh :( | 12:16 |
dtantsur | TheJulia, sounds like a bug | 12:17 |
*** marzif has quit IRC | 12:17 | |
*** jerrygb has quit IRC | 12:17 | |
*** marzif has joined #openstack-ironic | 12:17 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Removed integrated ZUUL reference patcher https://review.openstack.org/228242 | 12:20 |
TheJulia | sambetts: as far as I can tell, it does not handle it for you :( where would you guys like bugs? | 12:21 |
sambetts | https://bugs.launchpad.net/ironic-inspector | 12:22 |
TheJulia | is it normal to see "OperationalError: (pysqlite2.dbapi2.OperationalError) attempt to write a readonly database" in debug mode when using sqlite? | 12:25 |
*** VikasC has quit IRC | 12:27 | |
dtantsur | TheJulia, something with access rights on the database? | 12:28 |
dtantsur | maybe you first ran it from root, then from a user? | 12:28 |
*** baoli has joined #openstack-ironic | 12:28 | |
*** Haomeng has quit IRC | 12:30 | |
*** Haomeng has joined #openstack-ironic | 12:31 | |
*** mkovacik has joined #openstack-ironic | 12:32 | |
TheJulia | dtantsur: looks like, interestingly enough | 12:36 |
*** nicodemos has joined #openstack-ironic | 12:43 | |
dtantsur | sambetts, we'll have to make one more liberty release due to this bug | 12:48 |
*** Marga_ has joined #openstack-ironic | 12:48 | |
*** puranamr has joined #openstack-ironic | 12:48 | |
sambetts | dtantsur: yeah :/ | 12:50 |
*** puranamr has quit IRC | 12:52 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: WIP: Add agent inspection support for IPMI and SSH drivers https://review.openstack.org/229862 | 12:53 |
*** Marga_ has quit IRC | 12:53 | |
*** rloo has joined #openstack-ironic | 12:55 | |
*** jaypipes has joined #openstack-ironic | 12:58 | |
openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - RPC https://review.openstack.org/206243 | 12:58 |
openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/206244 | 12:58 |
openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - net https://review.openstack.org/206245 | 12:58 |
openstackgerrit | Om Kumar proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 12:58 |
openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - DB https://review.openstack.org/206232 | 12:58 |
openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - objs https://review.openstack.org/206238 | 12:58 |
openstackgerrit | Om Kumar proposed openstack/ironic: Add network provider interface and implementations https://review.openstack.org/139687 | 12:58 |
dtantsur | sambetts, ok, it's REALLY bad :( we probably should not use TEXT in primary keys, which means we need a weird migration here... | 12:59 |
*** cdearborn has joined #openstack-ironic | 12:59 | |
TheJulia | Looks like I'm getting closer, so I guess I just need to make the ipa ramdisk be the default thing that boots when pxe occurs... | 13:00 |
dtantsur | TheJulia, is https://github.com/openstack/ironic-inspector#using-ipa what you need? | 13:01 |
TheJulia | dtantsur: exactly what I needed, thank you for the link | 13:02 |
sambetts | dtantsur: oh dear :'( | 13:05 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-lib: Move to oslo.i18n https://review.openstack.org/229486 | 13:05 |
dtantsur | sambetts, yeah, it means a disastrous upgrade path for people with sqlite: drop you database and rerun :( | 13:06 |
dtantsur | actually, lemme check sqlite docs | 13:06 |
jroll | :| | 13:07 |
jroll | morning everyone | 13:07 |
dtantsur | morning jroll, yeah, we got a terrible bug in our db schema | 13:07 |
dtantsur | (our = inspector) | 13:07 |
sambetts | dtantsur: Really? I think the migration code should be able to do a rebuild like that without losing anything, I admit it might be a complicated migration but I think its doable | 13:08 |
jroll | yeah, I see that :( | 13:08 |
dtantsur | sambetts, we're saved. sqlite does not distinguish between TEXT and VARCHAR. | 13:08 |
rloo | morning jroll, TheJulia. hi dtantsur, sambetts | 13:09 |
dtantsur | morning rloo | 13:09 |
jroll | morning rloo :) | 13:09 |
dtantsur | sambetts, but we must switch our gate to mysql, I guess | 13:10 |
sambetts | dtantsur: nice! we'll need to fix the existing migrations then from TEXT to VARCHAR | 13:11 |
dtantsur | yeah, I'm on it | 13:11 |
*** athomas has quit IRC | 13:11 | |
sambetts | dtantsur: Good idea! I know there are things that mysql supports that sqlite doesn't though which might be something we need to be wary of | 13:12 |
sambetts | though | 13:12 |
dtantsur | anyway, serious people usually use mysql or postgres, so we'd better be closer to them | 13:13 |
TheJulia | good morning rloo jroll | 13:14 |
sambetts | dtantsur: Yup | 13:14 |
jroll | morning TheJulia :) | 13:16 |
*** athomas has joined #openstack-ironic | 13:17 | |
sambetts | dtantsur: Thanks for working on that! :) | 13:17 |
*** lazy_prince has quit IRC | 13:18 | |
*** marzif has quit IRC | 13:20 | |
*** marzif has joined #openstack-ironic | 13:20 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Fix database schema for mysql and switch the gate to testing it https://review.openstack.org/229875 | 13:22 |
dtantsur | sambetts, something like that ^^ | 13:22 |
dtantsur | if it passes the gate, then we can assume mysql is working :) | 13:22 |
*** marzif has quit IRC | 13:25 | |
*** Nisha has quit IRC | 13:25 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Fix database schema for mysql and switch the gate to testing it https://review.openstack.org/229875 | 13:25 |
dtantsur | had to fix README ^^ | 13:25 |
*** trown is now known as trown|brb | 13:35 | |
openstackgerrit | Merged openstack/ironic-lib: Put py34 first in the env order of tox https://review.openstack.org/229616 | 13:42 |
openstackgerrit | Merged openstack/ironic-lib: Remove TODO https://review.openstack.org/229454 | 13:42 |
jroll | jlvillal: you mentioned a broken gerrit link on firefox yesterday, is this also broken for you? http://perm.ly/ironic-review-dashboard | 13:45 |
jroll | jlvillal: I get "line 1:28 no viable alternative at character '%'" | 13:46 |
*** trown|brb is now known as trown | 13:47 | |
*** Marga_ has joined #openstack-ironic | 13:49 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Fix database schema for mysql and switch the gate to testing it https://review.openstack.org/229875 | 13:53 |
*** Marga_ has quit IRC | 13:53 | |
*** links has quit IRC | 13:57 | |
jlvillal | jroll, That link worked for me. But the error message you mention is what I saw yesterday. On a different computer but same OS | 14:03 |
dtantsur | jroll, I get the same error as you | 14:03 |
dtantsur | fortunately enough I use gertty most of the time :) | 14:03 |
jlvillal | I am running Firefox 40.0.3 | 14:04 |
TheJulia | wow, I think I have inspection working | 14:04 |
jlvillal | About to update | 14:04 |
dtantsur | TheJulia, congrats :) | 14:05 |
NobodyCam | good morning jlvillal TheJulia dtantsur sambetts jroll rloo and eveyone else | 14:05 |
jlvillal | dtantsur, jroll: Updating to 41.0.1 broke it for me | 14:05 |
dtantsur | yeah, I have 41 | 14:06 |
jlvillal | So I think bug in firefox | 14:06 |
dtantsur | morning NobodyCam | 14:06 |
jlvillal | NobodyCam, Morning! | 14:06 |
NobodyCam | good morning :) | 14:06 |
*** willsama has joined #openstack-ironic | 14:07 | |
TheJulia | dtantsur: I'm really not sure what kind of testing needs to be wrapped around with https://review.openstack.org/#/c/229862/, but It worked like a champ with agent_ssh as my driver | 14:08 |
*** dims has quit IRC | 14:10 | |
*** david-lyle has quit IRC | 14:11 | |
*** david-lyle has joined #openstack-ironic | 14:13 | |
rloo | good morning NobodyCam! | 14:14 |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Allow empty password for ipmitool console https://review.openstack.org/229904 | 14:15 |
jroll | jlvillal: dtantsur urgh | 14:16 |
jroll | dtantsur: I need to work out getting gertty running. my terminal colors make it black on black :( | 14:17 |
jroll | morning NobodyCam :) | 14:17 |
NobodyCam | :) | 14:17 |
* sambetts also needs to setup gertty | 14:17 | |
openstackgerrit | Ruby Loo proposed openstack/ironic-python-agent: Use doc8 style checker https://review.openstack.org/229683 | 14:22 |
jroll | rloo: dtantsur others: are y'all good with an oct 9 deadline for submitting summit session proposals, and we can talk through them in the oct 12 meeting? | 14:22 |
dtantsur | I'm fine with it | 14:23 |
rloo | jroll: hmm. that doesn't leave much time to go through them before the meeting, but yeah, unless we are expected to read specs before talking about them :) | 14:23 |
*** romcheg has quit IRC | 14:24 | |
rloo | jroll: thinking about it, yeah, we don't have to decide everything at the meeting so it is a good first pass anyway. | 14:24 |
*** romcheg has joined #openstack-ironic | 14:24 | |
jroll | rloo: yeah, I figure we talk through them at the meeting and make a final decision later that week | 14:24 |
*** romcheg has quit IRC | 14:25 | |
jroll | we have to have our list by oct 15 | 14:25 |
* dtantsur sets a reminder to update the driver composition spec before the deadline | 14:25 | |
* jroll emails | 14:25 | |
rloo | thx jroll | 14:25 |
jroll | np | 14:25 |
*** mtanino has joined #openstack-ironic | 14:27 | |
rloo | that reminds me. dtantsur: what's the status with http://specs.openstack.org/openstack/ironic-specs/specs/approved/futurist.html? | 14:27 |
dtantsur | rloo, I didn't have time for it in Liberty, planning to do in the first half of Mitaka | 14:28 |
rloo | dtantsur: great! | 14:28 |
*** notq has quit IRC | 14:29 | |
openstackgerrit | Merged openstack/python-ironicclient: Use doc8 style checker https://review.openstack.org/229676 | 14:30 |
jlvillal | jroll, How long until we can add ironic-lib to the requirements.txt of ironic? | 14:31 |
jlvillal | Or did we get blocked by feature freeze to g-r? | 14:31 |
jroll | jlvillal: it should be added with the first commit that uses it | 14:31 |
jroll | it's in g-r | 14:31 |
jroll | which reminds me, we should land all the things and release that | 14:32 |
jlvillal | jroll, Excellent. Thanks. | 14:32 |
jroll | np | 14:32 |
dtantsur | jlvillal, I think g-r is unfrozen already | 14:32 |
dtantsur | IIRC they branches stable/liberty on Mon | 14:32 |
jlvillal | dtantsur, Sort of. I was trying to move the mimic patch forward. And they said they were still working on releasing things. So be patient. Even though they had removed the -2 | 14:33 |
dtantsur | I guess they just don't have much time these days | 14:33 |
*** saripurigopi has quit IRC | 14:33 | |
jlvillal | Yeah | 14:33 |
jroll | yeah, it's a time thing, not a freeze thing :P | 14:34 |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Allow empty password for ipmitool console https://review.openstack.org/229904 | 14:36 |
*** digimonde has quit IRC | 14:37 | |
*** saripurigopi has joined #openstack-ironic | 14:39 | |
jroll | rloo: sending an email with specs process reminders; anything besides new specs go to approved/ and old specs don't need re-approval? | 14:39 |
rloo | jroll: don't think so. Just the caveat maybe that old specs don't need re-approval but we reserve the right to unapprove... | 14:40 |
jroll | ya | 14:40 |
jroll | rloo: thanks | 14:40 |
*** notq has joined #openstack-ironic | 14:43 | |
openstackgerrit | Merged openstack/ironic: Follow up for: Prepare for functional testing patch https://review.openstack.org/229033 | 14:43 |
jvmario | good morning ironic | 14:46 |
dtantsur | morning jvmario | 14:47 |
*** Marga_ has joined #openstack-ironic | 14:50 | |
BadCub | morning ironicville | 14:50 |
dtantsur | TheJulia, https://review.openstack.org/#/c/229875/ passed the gate, could you please check if it fixes the mysql issue for you? | 14:51 |
dtantsur | sambetts, seems to work ^^ | 14:51 |
sambetts | dtantsur: woop :) | 14:51 |
dtantsur | ifarkas, hi! please review https://review.openstack.org/#/c/229875 it's an urgent fix | 14:52 |
ifarkas | dtantsur, alright! | 14:52 |
NobodyCam | morning jvmario | 14:52 |
*** rameshg87 has joined #openstack-ironic | 14:53 | |
rameshg87 | jroll: hi | 14:53 |
dtantsur | and morning BadCub | 14:53 |
jroll | hiya rameshg87 :) | 14:53 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Initial support for installation of ironic-inspector https://review.openstack.org/229930 | 14:53 |
rameshg87 | oh JoshNang is still here | 14:53 |
TheJulia | dtantsur: checking | 14:53 |
rameshg87 | jroll: JoshNang: any updates on zapping work ? is JoshNang going to continue that ? if not, i would like to take it up | 14:54 |
BadCub | hiya dtantsur | 14:54 |
jroll | rameshg87: please do take it, josh won't have time :) | 14:54 |
*** Marga_ has quit IRC | 14:54 | |
rameshg87 | jroll: okay | 14:54 |
jroll | rameshg87: as long as you aren't taking on too much work :) | 14:54 |
rloo | jroll, rameshg87: I was going to redo the zapping. | 14:54 |
rameshg87 | rloo: oh, you are welcome. | 14:54 |
rloo | jroll, rameshg87: wrt changing from 'zapping' to 'manual clean' vs 'automated clean'. | 14:55 |
jroll | oh \o/ | 14:55 |
rameshg87 | rloo: zapping is important for raid, i wanted to get it in for next feature release | 14:55 |
rameshg87 | :) | 14:55 |
rloo | jroll: only cuz the discussions is still fresh in my mind. | 14:55 |
rloo | rameshg87: yeah, i want to get it in too. i spent so much time reviewing all that stuff... | 14:55 |
rameshg87 | yes | 14:56 |
jroll | rloo: cool, do a spec update first to commit your memory to persistent storage :) | 14:56 |
rloo | rameshg87: the trick is how to reuse the code/state machine. i have to think about it. | 14:56 |
rloo | jroll: yeah, the spec update is the easy part. it was the coding that stumped me. | 14:56 |
rameshg87 | rloo: i still have to go through your comments after i last reviewed JoshNang code | 14:56 |
rameshg87 | on zapping | 14:56 |
rameshg87 | rloo: jroll: meanwhile i would like to propose some changes to cleaning framework that stopped raid from enabling cleaning | 14:57 |
jroll | rloo: oh right, the state machine. sigh. | 14:57 |
* rcarrillocruz is intrigued by ironic-inspect, goes check what is about | 14:57 | |
rloo | rameshg87: well, not sure it is worth it. might want to wait. up to you. we also need to get the boot interface patches reviewed too if folks have time... | 14:57 |
rameshg87 | rloo: yeah, i reviewed the ilo and irmc boot interface patches yesterday | 14:57 |
rameshg87 | rloo: i will go through that again tomorrow my time again | 14:58 |
rloo | rameshg87: and oneview driver would be good to get in. i don't have the energy to do big reviews. need a break. | 14:58 |
jroll | rameshg87: how are those going, are they close? | 14:58 |
jroll | oneview++ | 14:58 |
JoshNang | rameshg87: yeah, unfortunately I won't be writing more Ironic code. Feel free to pick it up! I think rloo's comments sum up the direction we were considering | 14:58 |
rameshg87 | rloo: i reviewed that too once, in my list for tomorrow :) | 14:58 |
*** harshs has joined #openstack-ironic | 14:58 | |
rameshg87 | JoshNang: sure, thanks :) | 14:58 |
*** tsekiyama has joined #openstack-ironic | 14:58 | |
*** aojea_ has joined #openstack-ironic | 14:59 | |
TheJulia | dtantsur: looks good, no errors on startup | 14:59 |
rloo | thank you rameshg87! | 14:59 |
TheJulia | dtantsur: or db creation | 14:59 |
rameshg87 | jroll: 1) make cleaning steps executed predictable 2) make a facility to opt-out clean steps on per node basis. these are problems i would like to address | 14:59 |
rameshg87 | jroll: anyway i will propose a spec for that tomorrow | 14:59 |
dtantsur | TheJulia, awesome | 14:59 |
*** jaypipes has quit IRC | 14:59 | |
TheJulia | dtantsur: short of re-installing a bunch of stuff right now, I can't run the workflow at the moment | 15:00 |
jroll | rameshg87: I have concerns with part 2, I think that's odd, but I'd love to read your spec | 15:00 |
rameshg87 | jroll: sure :) | 15:00 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add retries to ssh._get_hosts_name_for_node https://review.openstack.org/224828 | 15:00 |
* rameshg87 believes jroll is hard to convince on things related to cleaning | 15:00 | |
*** itsuugo has quit IRC | 15:00 | |
jroll | rameshg87: nah, I'm hard to convince on things related to "special" nodes | 15:01 |
rameshg87 | pets i suppose :) | 15:01 |
jroll | yep | 15:01 |
jroll | I do understand it's a useful thing to people | 15:01 |
jroll | just need to be careful with it | 15:02 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add retries to ssh._get_hosts_name_for_node https://review.openstack.org/224828 | 15:02 |
rloo | jroll, rameshg87: my guess is wrt eg RAID. where some nodes might want to opt out. | 15:03 |
rameshg87 | rloo: yeah | 15:03 |
rloo | jroll, rameshg87. there was that discussion with lucas about a diff architecture for cleaning. | 15:03 |
rloo | jroll, rameshg87: i don't know if we want to go there? | 15:04 |
rameshg87 | rloo: yeah, i was going along that lines too. but what lucas explained had some minor issues | 15:04 |
jroll | rloo: right... I'm just not sure "allow a node to skip a clean step" is the right way to do that | 15:04 |
rameshg87 | rloo: for example, he suggested to make clean_step configurable in a CONF file so that it is unique across all nodes | 15:04 |
jroll | rloo: I also worry that we're constantly re-architecting all the things :) | 15:04 |
rameshg87 | rloo: but clean steps executed in ironic (not in agent) is not predicatable as different drivers may have different clean steps | 15:04 |
rloo | jroll, rameshg87: OK, then I will use existing architecture. | 15:05 |
rloo | rameshg87: it is predictable per driver. | 15:05 |
rloo | rameshg87: well, per driver & IPA | 15:05 |
rameshg87 | rloo: yeah, but with driver composition reform | 15:05 |
rameshg87 | rloo: i don't know what's gonna happen. people can pull in interfaces they like onto different nodes | 15:06 |
rameshg87 | theoretically | 15:06 |
*** mjturek1 has quit IRC | 15:06 | |
rloo | rameshg87: it should still be predictable. just maybe not obvious. The API should return the clean steps. | 15:06 |
*** sdake has joined #openstack-ironic | 15:06 | |
rameshg87 | rloo: yeah until we have that API :) | 15:06 |
jroll | rloo: I'm not saying we shouldn't refactor/rearchitect things, I just need a more coherent proposal than a few IRC vonersations :) | 15:07 |
rloo | rameshg87: don't worry. we'll get that API sometime ;) | 15:07 |
rameshg87 | rloo: thanks ;-) | 15:07 |
*** rameshg87 is now known as rameshg87-afk | 15:07 | |
rloo | jroll: yeah, gotcha. and given everything on our plate, i'm not sure that any rearchitect is a high priority. | 15:08 |
jroll | rloo: right | 15:08 |
*** mkovacik has quit IRC | 15:09 | |
jroll | rloo: effort:value needs to be proven :P | 15:09 |
*** mkovacik has joined #openstack-ironic | 15:09 | |
*** sdake_ has quit IRC | 15:10 | |
*** mjturek1 has joined #openstack-ironic | 15:12 | |
*** Marga_ has joined #openstack-ironic | 15:14 | |
*** jerrygb has joined #openstack-ironic | 15:19 | |
*** jerrygb has quit IRC | 15:19 | |
*** jerrygb has joined #openstack-ironic | 15:19 | |
dtantsur | folks, please welcome mkovacik, he is starting to work on ironic, inspector and related stuff with us | 15:21 |
NobodyCam | welcome to ironic-landa mkovacik | 15:22 |
betherly | hi mkovacik :) | 15:22 |
rloo | welcome mkovacik! If you have any questions, ask dtantsur :) | 15:23 |
trown | o/ mkovacik | 15:23 |
mkovacik | hello guys :) | 15:24 |
TheJulia | welcome | 15:24 |
jroll | mkovacik: welcome \o/ | 15:24 |
sambetts | o/ mkovacik | 15:25 |
*** jaypipes has joined #openstack-ironic | 15:30 | |
*** dims has joined #openstack-ironic | 15:30 | |
* mkovacik is away: I'm busy | 15:32 | |
jvmario | sambetts: thanks for the review on 226058! i tried to address your comments | 15:32 |
sambetts | jvmario: :) just reading now | 15:34 |
*** saripurigopi1 has joined #openstack-ironic | 15:35 | |
*** saripurigopi has quit IRC | 15:35 | |
jroll | jlvillal: cool with my thoughts here? https://review.openstack.org/#/c/229447/ | 15:36 |
*** persia has quit IRC | 15:38 | |
jlvillal | jroll, look in a few | 15:40 |
jroll | yeah no rush | 15:41 |
*** aojea_ has quit IRC | 15:43 | |
*** rameshg87-afk is now known as rameshg87 | 15:43 | |
* rameshg87 is done for day | 15:43 | |
* jroll wonders out loud if we should do stable branches for IPA | 15:43 | |
jroll | \o night rameshg87 | 15:43 |
rameshg87 | have a good day everyone | 15:43 |
*** rameshg87 has left #openstack-ironic | 15:43 | |
jroll | JayF: trown devananda any thoughts on IPA stable branches/releases? | 15:43 |
dtantsur | oh, moar stable branches :) | 15:44 |
rloo | jroll: if IPA is going to be a big part of Ironic, I think there should be stable branch/releases. | 15:44 |
jroll | dtantsur: idk why I thought you were out for the day | 15:44 |
jroll | thoughts? | 15:44 |
trown | jroll: I would commit to helping maintain them if that helps in the decision | 15:44 |
jroll | rloo: yeah, that's my thinking | 15:44 |
*** Marga_ has quit IRC | 15:44 | |
dtantsur | jroll, Oo | 15:45 |
trown | s/them/it/ | 15:45 |
*** Marga_ has joined #openstack-ironic | 15:45 | |
jroll | dtantsur: ? | 15:45 |
*** mgoddard__ has joined #openstack-ironic | 15:45 | |
dtantsur | jroll, so, as to stable branches, I'd prefer we have them, as otherwise in a year we'll have to somehow ensure that IPA works with K, L, M... | 15:46 |
dtantsur | I mean, IPA master | 15:46 |
dtantsur | that's is fine, but limits ability to deprecate things | 15:46 |
dtantsur | s/is// | 15:46 |
jroll | dtantsur: indeed | 15:47 |
*** itsuugo has joined #openstack-ironic | 15:47 | |
*** mgoddard1 has joined #openstack-ironic | 15:47 | |
*** mgoddard1 has left #openstack-ironic | 15:47 | |
dtantsur | If we plan on stable/liberty for IPA, I would really appreciate if https://review.openstack.org/#/c/225092/ gets in | 15:48 |
jroll | dtantsur: I'm thinking mitaka forward, kind of late for liberty stable | 15:48 |
*** mgoddard_ has quit IRC | 15:48 | |
jroll | but I could check with the release team, if liberty is useful to y'all | 15:48 |
trown | jroll: why late for liberty stable? | 15:48 |
dtantsur | jroll, tripleo is still discussing stable branches for liberty :) | 15:48 |
jroll | ok, maybe it's not late | 15:48 |
dtantsur | inspector got it on Monday | 15:48 |
*** romainh has left #openstack-ironic | 15:48 | |
jroll | dtantsur: and yeah, I plan to release 1.0.0 after that lands | 15:49 |
dtantsur | super | 15:49 |
trown | awesome | 15:49 |
trown | I am in favor of making liberty/stable = 1.0.0 | 15:49 |
jroll | yeah | 15:49 |
jroll | here's the first step: https://review.openstack.org/#/c/229983/ | 15:50 |
jroll | making launchpad and such now | 15:50 |
dtantsur | the only problem (which can be solved post-liberty IMO) is that we need to tell our gates to actually use stable/liberty IPA | 15:51 |
dtantsur | I guess now we just download the tarball, right? | 15:51 |
*** harshs_ has joined #openstack-ironic | 15:51 | |
jroll | tell which gates? | 15:52 |
dtantsur | e.g. ironic one, and everything that uses IPA in its gate | 15:52 |
jroll | oh, use the release instead of master | 15:52 |
jroll | that's a good point | 15:52 |
dtantsur | yep | 15:52 |
JayF | jroll: you know how I feel about IPA releases :/ | 15:52 |
jroll | I tend to think we should test with both | 15:52 |
JayF | jroll: although with the agent getting more complex I can see some of the need | 15:52 |
jroll | JayF: yeah, and it makes it easy for packagers etc, if we're moving to IPA by default we should really do this stuff | 15:53 |
*** harshs has quit IRC | 15:53 | |
*** harshs_ is now known as harshs | 15:53 | |
*** romcheg has joined #openstack-ironic | 15:53 | |
trown | ya if there is not an upstream stable release then each downstream package will duplicate effort maintaining it | 15:53 |
*** lazy_prince has joined #openstack-ironic | 15:53 | |
*** e0ne has quit IRC | 15:53 | |
sambetts | jvmario: replied :) | 15:55 |
jvmario | ty | 15:55 |
jroll | JayF: would you mind reviewing 225092? | 15:55 |
jroll | (totally cool if you're busy) | 15:55 |
JayF | I actually have some time | 15:56 |
JayF | was going to upstream review today a little | 15:56 |
jroll | <3 | 15:56 |
jroll | dtantsur: one of us should do a bug scrub to move things to https://bugs.launchpad.net/ironic-python-agent | 15:59 |
jroll | or both | 15:59 |
jroll | maybe I could tackle that this weekend | 15:59 |
JayF | IPA getting it's own launchpad? | 16:00 |
JayF | moving up in the world! | 16:00 |
jroll | ikr | 16:01 |
jroll | JayF: so to be a separate "deliverable", meaning not released in lockstep, it needs it's own launchpad | 16:01 |
JayF | aha | 16:01 |
dtantsur | oh yeah | 16:01 |
openstackgerrit | Merged openstack/ironic-inspector: Fix database schema for mysql and switch the gate to testing it https://review.openstack.org/229875 | 16:01 |
*** mgoddard has joined #openstack-ironic | 16:02 | |
dtantsur | sambetts, https://review.openstack.org/#/c/229988/ is a cherry-pick of ^^ to liberty | 16:02 |
*** mgoddard__ has quit IRC | 16:03 | |
lazy_prince | jroll: can you have a look at https://review.openstack.org/139687 and see if it is in line with what you had in mind... | 16:03 |
dtantsur | jroll, https://bugs.launchpad.net/ironic/+bugs?field.tag=agent is a good start | 16:03 |
jroll | dtantsur: yeah, that's where I was going to start | 16:03 |
sambetts | dtantsur: +2 | 16:04 |
dtantsur | jroll, I'll add the new project to the bug dashboard | 16:04 |
jroll | thanks! | 16:05 |
jroll | lazy_prince: I'll try to look in a bit | 16:05 |
lazy_prince | jroll: take your time... no hurry... | 16:05 |
dtantsur | JayF, thanks for review, I've answered in gerrit | 16:07 |
JayF | dtantsur: qq that will basically impact my response to your comments on the review | 16:08 |
JayF | dtantsur: can inspection be enabled in the default ramdisk now via external means? i.e. kernel command line? | 16:08 |
JayF | dtantsur: if so, then I think we reduce the value of the prebuilt ramdisk a bunch by not including all needed requirements | 16:09 |
JayF | dtantsur: even for an optional plugin of an optional feature :) | 16:09 |
dtantsur | JayF, kernel command line is the only way to enable it actually :) also, this is just one of inspection plugins, inspection would work without it | 16:09 |
dtantsur | JayF, I don't think people appreciate installing numpy and pandas because of " optional plugin of an optional feature" :) | 16:10 |
JayF | I don't know how large those libraries are so hard for me to make sense of that :P | 16:10 |
jroll | heh, ++ for not installing numpy by default | 16:10 |
jroll | JayF: numpy is a massive C library | 16:10 |
*** thiagop has joined #openstack-ironic | 16:10 | |
thiagop | Good morning Ironic | 16:10 |
jroll | it's terrible to install | 16:10 |
thiagop | BadCub: are you around? | 16:10 |
jroll | morning thiagop | 16:10 |
JayF | dtantsur: re: the tarball stuff, I still feel like we shuld have an upper limit there | 16:10 |
thiagop | jroll: / | 16:10 |
*** killer_prince has joined #openstack-ironic | 16:10 | |
BadCub | I am always lurking thiagop LOL | 16:11 |
JayF | dtantsur: even if you do something like take the last 5000 lines or something that'll be a limit never likely to be hit | 16:11 |
dtantsur | then why do we need it? :) | 16:11 |
JayF | dtantsur: because I don't want to be the operator troubleshooting why my agent blew up when I was trying to troubleshoot my agent blowing up :) | 16:12 |
JayF | dtantsur: especially in error cases, there could easily be an explosion of logs; think multiple tracebacks, flapping services in the ramdisk, etc etc | 16:12 |
BadCub | hiya JayF | 16:13 |
*** r-daneel has joined #openstack-ironic | 16:13 | |
JayF | hey BadCub | 16:13 |
dtantsur | JayF, well, I dunno. In this situation we can easily overflow memory with logs themselves, even before IPA is started | 16:13 |
*** lazy_prince has quit IRC | 16:14 | |
*** romcheg1 has joined #openstack-ironic | 16:14 | |
JayF | dtantsur: I would think the http transfer limit is much less than available ram in many cases | 16:14 |
JayF | definately in all of my cases | 16:14 |
*** trown is now known as trown|lunch | 16:15 | |
*** romcheg has quit IRC | 16:15 | |
dtantsur | JayF, so you suggest doing `journalctl | tail -n1000` there instead? | 16:16 |
jroll | 'http transfer limit'? | 16:16 |
dtantsur | or maybe it has builtin tail... | 16:16 |
JayF | dtantsur: yeah, something like that | 16:16 |
JayF | dtantsur: and I bet it does support limiting it's own outpur | 16:16 |
JayF | jroll: I'm just saying, I have a machine that could have 500mb of logs | 16:17 |
JayF | jroll: passing that back up to Ironic API seems painful/impossible | 16:17 |
jroll | ya, I get it, just a weird phrase | 16:17 |
jroll | ironic-inspector api* | 16:17 |
jroll | :D | 16:17 |
dtantsur | well, it's a long-running machine | 16:17 |
dtantsur | but ok, if you folks agree, I'll put something like that.. I wonder what sane number of lines is | 16:18 |
JayF | dtantsur: yeah, I get that it's an uncommon/edge case, I just think with an easy fix to avoid it we should jfdi now | 16:18 |
dtantsur | jroll, JayF, suggestions on precise -n XXX? | 16:18 |
JayF | dtantsur: let me do some science on some logs real quick | 16:18 |
dtantsur | FWIW, last journal I got while testing this patch is of 757 lines | 16:18 |
*** jaypipes has quit IRC | 16:18 | |
jroll | dtantsur: idk, 5000? | 16:18 |
dtantsur | sounds like a safe border, yes | 16:19 |
* dtantsur updates the patch | 16:19 | |
JayF | dtantsur: doing some science | 16:19 |
JayF | 10k would still be under a meg | 16:19 |
JayF | and give people lots and lots of lines | 16:19 |
dtantsur | JayF, did you agree with not installing 'hardware' by default? | 16:19 |
JayF | dtantsur: yeah, I'm fine with that. Just wanted to have the discussion | 16:20 |
JayF | dtantsur: Just with IPA, I think driver-requirements pattern is less good b/c of the fact we prebuild them for people, but given it's size and difficulty to install, I think that counterbalances my concern | 16:20 |
dtantsur | ack | 16:21 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent: Add "logs" and "extra-hardware" inspection collectors https://review.openstack.org/225092 | 16:26 |
dtantsur | jroll, JayF, ^^ | 16:26 |
jlvillal | jroll, I updated my vote to +1. I guess I thought we supported Python 3 :) | 16:26 |
*** karimb has quit IRC | 16:26 | |
jroll | jlvillal: I don't feel comfortable saying it's supported until the gate actually runs against py3 | 16:27 |
dtantsur | we run tests on it, but it's a bit early to call it "support" :) | 16:27 |
jlvillal | Okay! Thanks :) | 16:27 |
jroll | "if it isn't tested it's broken" | 16:27 |
jroll | so like basically all of ironic is broken :( | 16:27 |
dtantsur | lol | 16:27 |
*** garthb has joined #openstack-ironic | 16:28 | |
dtantsur | at least a big number of API versions, which are only unit tested | 16:28 |
* jroll needs to convince everyone to do third party CI | 16:28 | |
jroll | dtantsur: did that tempest patch still not merge? | 16:28 |
dtantsur | IIRC no | 16:28 |
jroll | urgh | 16:28 |
*** jistr has quit IRC | 16:29 | |
dtantsur | https://review.openstack.org/#/c/166386/ | 16:29 |
dtantsur | running now, see you tomorrow | 16:29 |
jroll | dtantsur: +A | 16:29 |
*** dtantsur is now known as dtantsur|afk | 16:29 | |
jroll | hopefully 1.0.0 later today then | 16:29 |
dtantsur|afk | jroll, JayF, thanks! | 16:29 |
*** alexpilo_ has joined #openstack-ironic | 16:29 | |
JayF | np! | 16:29 |
*** lazy_prince has joined #openstack-ironic | 16:31 | |
*** Sukhdev has joined #openstack-ironic | 16:32 | |
*** lazy_prince has quit IRC | 16:32 | |
krtaylor | jroll, on third party, do we want to target drivers to test by the end of mitaka or is that pushing it? maybe summit discussion... | 16:32 |
jroll | krtaylor: I'm not sure, I think the only way to ensure it is to kick drivers out of tree, but I don't love that :( | 16:33 |
*** killer_prince has quit IRC | 16:33 | |
*** alexpilo_ has quit IRC | 16:33 | |
*** ndipanov has quit IRC | 16:34 | |
krtaylor | jroll, I think we'd have to have a period of transition like neutron/cinder did | 16:34 |
*** lazy_prince has joined #openstack-ironic | 16:34 | |
jroll | krtaylor: of course, I'm just not sure that's what we want to do. need to think/discuss it more | 16:34 |
krtaylor | understood, I'll put it as a topic for discussion on the test/qa etherpad | 16:36 |
sambetts | aarefiev: ping | 16:37 |
openstackgerrit | Merged openstack/ironic-lib: Update setup.cfg https://review.openstack.org/229447 | 16:38 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-lib: Move to oslo.i18n https://review.openstack.org/229486 | 16:38 |
jroll | rloo: dtantsur|afk ^ fixed a thing | 16:38 |
jroll | I'd like to release an ironic-lib when that lands | 16:38 |
*** amit213 has joined #openstack-ironic | 16:43 | |
rloo | NobodyCam: if you have a few minutes and want to do a review: ^^ https://review.openstack.org/#/c/229486/ | 16:44 |
rloo | JayF: if you've got time, it would be great to get your eyes on the specs | 16:45 |
JayF | rloo: I haven't been reviewing specs as much b/c I can't commit the time to follow through on the design | 16:45 |
JayF | rloo: I hope that will change in the new couple of months, but I don't want to -1 with comments on someone if I don't know I'll have the time to followup with them on my concerns | 16:46 |
rloo | JayF: oh. you mean once someone reviews, they have to follow up? | 16:46 |
*** garthb has quit IRC | 16:46 | |
JayF | I mean, I feel that responsibility when I review specs | 16:46 |
rloo | JayF: I'd rather have your -1 + comments and no follow up, than nothing. | 16:46 |
JayF | As long as others are OK with that, wfm | 16:46 |
JayF | I always before tried to pick a few and review them through to the finish line | 16:47 |
rloo | JayF: I am. I don't always follow through and don't think I should have to. Others can choose to ignore or not. | 16:47 |
jroll | as long as the comments are clear, and clearly fixed, I don't feel bad not waiting for a -1'er to follow up | 16:47 |
jroll | if it's unclear I ping said -1'er | 16:47 |
rloo | jroll: ++ | 16:48 |
jroll | rloo: I should probably review some specs too, eh | 16:48 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Python 3.4 unit tests fail with LANG=C https://review.openstack.org/229520 | 16:53 |
rloo | jroll: well, we should all review specs etc. there's only so many hours in a day... | 16:53 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add agent inspection support for IPMI and SSH drivers https://review.openstack.org/229862 | 16:54 |
*** e0ne has joined #openstack-ironic | 16:54 | |
*** e0ne has quit IRC | 16:55 | |
*** trown|lunch is now known as trown | 16:56 | |
*** persia has joined #openstack-ironic | 16:57 | |
jroll | rloo: right :) | 16:58 |
*** harshs has quit IRC | 16:59 | |
*** puranamr has joined #openstack-ironic | 16:59 | |
*** baoli has quit IRC | 17:00 | |
jroll | devananda: NobodyCam: I'd like to release ironic-lib today, pls to review https://review.openstack.org/#/c/229486/ | 17:00 |
*** puranamr has quit IRC | 17:01 | |
*** derekh has quit IRC | 17:01 | |
*** baoli has joined #openstack-ironic | 17:01 | |
*** sdake_ has joined #openstack-ironic | 17:02 | |
*** dims_ has joined #openstack-ironic | 17:03 | |
*** sdake_ has quit IRC | 17:04 | |
*** jerrygb has quit IRC | 17:04 | |
*** dims has quit IRC | 17:04 | |
JayF | rloo: jroll: qq about https://review.openstack.org/#/c/229625/2 -- do we even want to continue referencing specs against a "development cycle" vs the actual Ironic release they went out in? | 17:05 |
JayF | I find the mixing of openstack keywords and actual ironic releases a little confusing, and I actually know what's going on. What hope would our users have? | 17:05 |
*** sdake has quit IRC | 17:05 | |
rloo | JayF: that's a good question. I didn't know, and I think there's still value with the dev cycle. | 17:05 |
jroll | JayF: it's a good question, I'm not sure - for now we've decided to communicate the dev cycle | 17:06 |
JayF | jinx? | 17:06 |
jroll | release notes communicate what features went with the release | 17:06 |
JayF | lol | 17:06 |
rloo | JayF: I was thinking that we could, under each dev cycle, put the actual release too | 17:06 |
JayF | rloo: I think that'd actually be a great improvement | 17:06 |
JayF | rloo: like Liberty (4.1-4.2) or something like that | 17:06 |
sambetts | rloo, JayF, jroll: When I went through that patch this morning, I was confused about the backlog, what the difference between approved and backlog? | 17:07 |
rloo | JayF: Liberty (4.1-4.2) followed by subheading 4.1? or is the (..) enough? | 17:07 |
JayF | sambetts: backlog doesn't have implementation details | 17:07 |
*** Marga_ has quit IRC | 17:07 | |
rloo | sambetts: if you are confused, then it needs a better description? | 17:07 |
JayF | rloo: either would work; I was trying to save the work of actually mapping the specs to which release | 17:07 |
jroll | rloo++ | 17:07 |
*** Marga_ has joined #openstack-ironic | 17:08 | |
* jroll steps away for a while | 17:08 | |
rloo | JayF: ok, I can update. in a meeting now, so ... some time soon :) | 17:10 |
JayF | rloo: I think I'm going to land it as-is for now? It's 10x better than what was there before | 17:10 |
JayF | rloo: we can add the release numbers later | 17:10 |
rloo | JayF: later is fine with me too. | 17:11 |
*** tsekiyama has quit IRC | 17:11 | |
*** Marga_ has quit IRC | 17:13 | |
sambetts | JayF: hmm, so the backlog folder is where specs can land without their BP getting approved? | 17:13 |
openstackgerrit | Merged openstack/ironic-specs: Reorganize the specs on main page https://review.openstack.org/229625 | 17:14 |
JayF | sambetts: I think a good example is the hardware capabilities spec | 17:14 |
JayF | sambetts: we lay out what we'd like to do | 17:14 |
JayF | sambetts: but don't talk about implementation because everyone agreed while it was a good idea, we didn't need it enough to do it now | 17:14 |
JayF | sambetts: and writing most of the implementation stuff in the spec is like 40% of the discovery work to writing the feature | 17:14 |
*** praneshp has joined #openstack-ironic | 17:15 | |
sambetts | JayF: so the approved folder should be named more like in-progress then ? | 17:16 |
JayF | sambetts: IDK; honestly | 17:16 |
JayF | sambetts: I agree it's a little confusing | 17:16 |
sambetts | because if a specs has landed at all I would say thats an approved spec | 17:16 |
*** praneshp_ has joined #openstack-ironic | 17:17 | |
sambetts | if we had backlog, inprogress, implemented that would be more obvious from my perspective | 17:18 |
rloo | sambetts: inprogress instead of approved? | 17:19 |
sambetts | yh | 17:19 |
rloo | sambetts: so we don't know that it is in progress. we just know that the spec was approved. | 17:19 |
*** praneshp has quit IRC | 17:19 | |
*** praneshp_ is now known as praneshp | 17:19 | |
sambetts | rloo: then it should be in the backlog | 17:20 |
rloo | sambetts: backlog specs weren't approved. | 17:20 |
sambetts | but we +2'd it in the specs repo anyway? | 17:20 |
JayF | backlog specs are "this is a good idea, someone should come along, add details, and get it approved" | 17:21 |
rloo | sambetts: or were they? I haven't looked into it. We +2 to add it to backlog. not that it is approved. | 17:21 |
sambetts | JayF: isn't saying this is a good idea someone should do this approving it? | 17:22 |
rloo | sambetts: maybe we should put 'incomplete, needs owner, community agrees good idea'? | 17:22 |
JayF | sambetts: approving a spec is much more about the implementation in the spec than the idea | 17:22 |
rloo | sambetts: no, cuz the spec is incomplete | 17:22 |
rloo | sambetts: maybe we shouldn't have a backlog, or not have it there. not sure, since it is clear that it isn't clear why it is there. | 17:23 |
sambetts | maybe have folders: incomplete, backlog, inprogress, implmented ? | 17:23 |
sambetts | :/ this is a hard one | 17:24 |
rloo | sambetts: question for you. when you go to that page, what are you looking for? | 17:24 |
NobodyCam | rloo: looking | 17:24 |
rloo | sambetts: personally, i just want to see what specs have been implemented, and which ones have been approved but not-yet-implemented. | 17:24 |
rloo | thx NobodyCam! | 17:25 |
jroll | sambetts: what jay said: backlog is "we think this is useful, but don't know how to do it yet"; approved is "we think this is useful and we've approved the implementation" | 17:27 |
sambetts | I've never actually used that page... I've either looked at the repo or looked at the BP page :/ I didn't even know we had a backlog until I saw this | 17:27 |
rloo | sambetts: ha ha. we have to market better :) | 17:27 |
NobodyCam | rloo: jroll +2/+A | 17:28 |
rloo | NobodyCam: thx! | 17:28 |
*** alexpilo_ has joined #openstack-ironic | 17:28 | |
jroll | NobodyCam: thanks \o/ | 17:29 |
jroll | sambetts: if you're looking at the BP page, I'm so sorry :P | 17:29 |
rloo | jroll: he isn't. he's looking at the spec page. | 17:29 |
sambetts | I am now :-P | 17:29 |
openstackgerrit | Merged openstack/ironic-lib: Move to oslo.i18n https://review.openstack.org/229486 | 17:29 |
*** romcheg1 has quit IRC | 17:30 | |
rloo | sambetts: oh, me confused. | 17:30 |
sambetts | :-P | 17:30 |
*** romcheg has joined #openstack-ironic | 17:30 | |
jroll | ruh roh. | 17:32 |
jroll | http://logs.openstack.org/92/225092/4/check/gate-tempest-dsvm-ironic-agent_ssh-src/77f4da3/logs/devstacklog.txt.gz | 17:32 |
jroll | :( | 17:32 |
*** athomas has quit IRC | 17:32 | |
*** alexpilo_ has quit IRC | 17:33 | |
sambetts | :( | 17:33 |
jroll | zigo: what's the chance that's something you could help with? | 17:34 |
*** romcheg has quit IRC | 17:34 | |
sambetts | Its getting late where I am so I'm calling it a night, see everyone tomorrow o/ | 17:35 |
*** Nisha_away has joined #openstack-ironic | 17:35 | |
rloo | night sambetts | 17:36 |
sambetts | o/ rloo | 17:36 |
*** Marga_ has joined #openstack-ironic | 17:36 | |
jroll | night sam | 17:37 |
jroll | hm, working locally, maybe just flaked out for a moment | 17:37 |
*** Sukhdev has quit IRC | 17:39 | |
devananda | gah. too many things to do offline rightnow :-/ | 17:41 |
devananda | g'morning all! | 17:41 |
JayF | heya deva | 17:41 |
jroll | fyi this releases ironic-lib 0.2.0 https://review.openstack.org/#/c/230036/ | 17:42 |
jroll | morning devananda | 17:42 |
jlvillal | devananda: Good morning | 17:43 |
jlvillal | jroll: FYI: https://review.openstack.org/#/c/207689/ I will backport it to ironic-lib | 17:43 |
jlvillal | Or is that cross-port | 17:43 |
jroll | jlvillal: urgh, thanks | 17:44 |
jroll | did that actually land after the last -lib sync? | 17:44 |
jroll | these are the hard ones | 17:45 |
jroll | because that breaks the library's api | 17:45 |
jlvillal | jroll: Yes | 17:45 |
jlvillal | Well I think so | 17:45 |
jroll | it isn't a problem now, but things like that will become problematic | 17:45 |
jroll | yeah, looks like it was missed | 17:45 |
jlvillal | It is not in ironic-lib now | 17:45 |
jroll | sigh | 17:45 |
jlvillal | jroll: Well if we switch to using ironic-lib it won't happen :) | 17:46 |
jroll | right | 17:46 |
*** garthb has joined #openstack-ironic | 17:47 | |
*** pelix has quit IRC | 17:51 | |
*** itsuugo has quit IRC | 17:55 | |
*** itsuugo has joined #openstack-ironic | 17:56 | |
*** lazy_prince has quit IRC | 18:02 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic-lib: Add 'node_uuid' in debug logs to assist debugging https://review.openstack.org/230058 | 18:10 |
jlvillal | jroll: ^^^ My attempt at the cross-port | 18:10 |
*** tsekiyama has joined #openstack-ironic | 18:10 | |
jroll | thanks | 18:10 |
jlvillal | Unit testing is very different | 18:10 |
*** amotoki has quit IRC | 18:10 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic-lib: Add 'node_uuid' in debug logs to assist debugging https://review.openstack.org/230058 | 18:11 |
* jlvillal fixed extra blank line aka PEP8 error | 18:11 | |
*** Sukhdev has joined #openstack-ironic | 18:12 | |
TheJulia | :) | 18:13 |
openstackgerrit | Merged openstack/ironic: Python 3.4 unit tests fail with LANG=C https://review.openstack.org/229520 | 18:18 |
jlvillal | jroll: Any idea where this code came from? https://github.com/openstack/ironic-lib/blob/master/ironic_lib/disk_utils.py#L157-L160 | 18:19 |
jroll | probably not | 18:19 |
* jroll click | 18:19 | |
jlvillal | It says it was a cross-port from Ironic, but I don't see it in Ironic | 18:19 |
jroll | jlvillal: whole disk images maybe? | 18:19 |
jroll | hm | 18:19 |
jlvillal | ironic has the function in ironic/drivers/modules/deploy_utils.py | 18:20 |
jlvillal | jroll: Could be my mess-up. One sec | 18:21 |
jlvillal | jroll: No, I was on correct tip of ironic | 18:21 |
* jlvillal was worried he had gone back to patch from months ago. | 18:21 | |
jroll | I'll look | 18:22 |
jlvillal | jroll: Thanks. Just slight worrying to me if functions are different | 18:22 |
jlvillal | Okay maybe more than slightly... | 18:23 |
jroll | jlvillal: urgh, yeah, not seeing it | 18:25 |
jlvillal | jroll: Glad it isn't just me | 18:25 |
jroll | at least in the referenced commits | 18:25 |
*** penick has joined #openstack-ironic | 18:25 | |
*** alexpilo_ has joined #openstack-ironic | 18:27 | |
*** marzif has joined #openstack-ironic | 18:27 | |
jroll | jlvillal: grepping through 'git log -p' also gives me nothing | 18:27 |
jroll | wtf | 18:27 |
jlvillal | jroll: same... | 18:28 |
*** tsekiyama has quit IRC | 18:28 | |
openstackgerrit | Merged openstack/ironic-python-agent: Use doc8 style checker https://review.openstack.org/229683 | 18:28 |
jroll | jlvillal: more interesting, is_iscsi_device() never existed in ironic | 18:29 |
jlvillal | jroll: Right!! | 18:29 |
jlvillal | I'm like where the hell did that come from??? | 18:29 |
jlvillal | And I don't think syed works on Ironic anymore. | 18:30 |
jroll | :/ | 18:30 |
jlvillal | jroll: I'll do a patch to sync that function with what is in ironic | 18:30 |
jlvillal | Probably after lunch though | 18:31 |
*** openstackgerrit has quit IRC | 18:31 | |
jroll | jlvillal: ok, thanks | 18:31 |
*** alexpilo_ has quit IRC | 18:31 | |
*** openstackgerrit has joined #openstack-ironic | 18:32 | |
jlvillal | jroll: For that function is like deleting those five lines and replace with one line. Want me to add to my current patch? | 18:35 |
jroll | jlvillal: no, separate patch; it's a different change | 18:36 |
jlvillal | jroll: Will do | 18:36 |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Ensure all methods in util.py have docstrings. https://review.openstack.org/220313 | 18:40 |
openstackgerrit | John L. Villalovos proposed openstack/ironic-lib: Sync the make_partitions function with ironic https://review.openstack.org/230071 | 18:45 |
*** tsekiyama has joined #openstack-ironic | 18:45 | |
*** marzif has quit IRC | 18:45 | |
*** Nisha_away has quit IRC | 18:46 | |
*** harshs has joined #openstack-ironic | 18:49 | |
TheJulia | cinerama: Any issue if I address nits for 217374 in a follow-up rev? | 19:02 |
cinerama | i think of those things, the one i'd most like to see is adding the comment to the generated file, especially since we already do it for one of those files | 19:04 |
*** itsuugo has quit IRC | 19:04 | |
TheJulia | cinerama: you mean the dnsmasq specific file? | 19:13 |
TheJulia | if so, I'll have to look up and see if we can even do that, not familuar with the file format | 19:14 |
cinerama | ah ok. if we're not sure we can do that perhaps it is better to hold off on it then | 19:16 |
TheJulia | I do agree its a good idea, I'll go dig into the file format once I've had some time to shift gears back to that | 19:18 |
*** tsekiyama has quit IRC | 19:18 | |
*** david-lyle has quit IRC | 19:18 | |
*** alexpilo_ has joined #openstack-ironic | 19:21 | |
*** mgoddard has quit IRC | 19:25 | |
*** mgoddard has joined #openstack-ironic | 19:25 | |
*** alexpilo_ has quit IRC | 19:26 | |
*** david-lyle has joined #openstack-ironic | 19:27 | |
*** mgoddard has quit IRC | 19:32 | |
*** puranamr has joined #openstack-ironic | 19:33 | |
*** mgoddard has joined #openstack-ironic | 19:37 | |
*** itsuugo has joined #openstack-ironic | 19:40 | |
zigo | jroll: Sorry, help with what? | 19:41 |
zigo | The "Packages Hash Sum mismatch" ? | 19:41 |
zigo | Nothing I can help with, it's the mirror which is currently getting in sync. | 19:42 |
zigo | So just wait for a bit, and get the gate to recheck. | 19:42 |
jroll | zigo: yeah, I think it was just a blip, looks ok now | 19:42 |
jroll | yep, thanks | 19:42 |
zigo | jroll: Unfortunately, mirror sync in Debian are still not atomic... :( | 19:42 |
zigo | Someone one day will have to fix this ... :/ | 19:42 |
*** itsuugo has quit IRC | 19:44 | |
jroll | zigo: yeah, it wasn't clear to me if it was a sync failure thing or something bigger when I pinged :) | 19:44 |
zigo | This annoyed me more than once. | 19:44 |
*** puranamr has quit IRC | 19:45 | |
*** itsuugo has joined #openstack-ironic | 19:46 | |
*** sambetts has quit IRC | 19:47 | |
*** logan2 has quit IRC | 19:47 | |
*** sambetts has joined #openstack-ironic | 19:48 | |
*** mgoddard has quit IRC | 19:52 | |
*** ifarkas has quit IRC | 19:55 | |
*** logan2 has joined #openstack-ironic | 19:55 | |
*** mgoddard has joined #openstack-ironic | 19:56 | |
*** Haomeng|2 has joined #openstack-ironic | 20:01 | |
*** mgoddard has quit IRC | 20:02 | |
*** mgoddard has joined #openstack-ironic | 20:02 | |
*** Haomeng has quit IRC | 20:03 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic-lib: Sync the make_partitions function with ironic https://review.openstack.org/230071 | 20:06 |
*** mgoddard has quit IRC | 20:08 | |
*** mgoddard has joined #openstack-ironic | 20:11 | |
*** mgoddard has quit IRC | 20:17 | |
openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: Implement rescue mode https://review.openstack.org/171878 | 20:18 |
*** tsekiyama has joined #openstack-ironic | 20:18 | |
*** alexpilo_ has joined #openstack-ironic | 20:20 | |
*** nicodemos has quit IRC | 20:23 | |
*** tsekiyama has quit IRC | 20:23 | |
*** alexpilo_ has quit IRC | 20:24 | |
*** mgoddard has joined #openstack-ironic | 20:25 | |
*** derekh has joined #openstack-ironic | 20:46 | |
*** tsekiyama has joined #openstack-ironic | 20:46 | |
*** ekarlso has quit IRC | 20:55 | |
*** ijw has joined #openstack-ironic | 20:56 | |
*** ekarlso has joined #openstack-ironic | 20:58 | |
*** tsekiyama has quit IRC | 20:58 | |
*** mgoddard has quit IRC | 20:59 | |
*** derekh has quit IRC | 21:00 | |
*** mgoddard has joined #openstack-ironic | 21:07 | |
*** mgoddard has quit IRC | 21:12 | |
*** ijw has quit IRC | 21:13 | |
*** tsekiyama has joined #openstack-ironic | 21:14 | |
*** ijw has joined #openstack-ironic | 21:16 | |
*** mgoddard has joined #openstack-ironic | 21:17 | |
*** alexpilo_ has joined #openstack-ironic | 21:19 | |
*** alexpilo_ has quit IRC | 21:24 | |
openstackgerrit | Merged openstack/ironic-webclient: Expanded clean command. https://review.openstack.org/227404 | 21:28 |
*** Marga_ has quit IRC | 21:28 | |
openstackgerrit | Merged openstack/ironic-webclient: Added repository field. https://review.openstack.org/227979 | 21:29 |
*** trown is now known as trown|outttypeww | 21:30 | |
*** karimb has joined #openstack-ironic | 21:30 | |
*** Marga_ has joined #openstack-ironic | 21:30 | |
*** mgoddard has quit IRC | 21:32 | |
*** mgoddard has joined #openstack-ironic | 21:33 | |
*** e0ne has joined #openstack-ironic | 21:34 | |
*** boris-42 has joined #openstack-ironic | 21:38 | |
*** david-lyle has quit IRC | 21:43 | |
*** Marga_ has quit IRC | 21:46 | |
*** Marga_ has joined #openstack-ironic | 21:47 | |
openstackgerrit | Merged openstack/ironic-webclient: Added phantomjs as dependency https://review.openstack.org/227405 | 21:51 |
*** saripurigopi1 has quit IRC | 21:51 | |
openstackgerrit | Merged openstack/ironic-webclient: Upgraded bower dependencies https://review.openstack.org/227406 | 21:51 |
openstackgerrit | Merged openstack/ironic-webclient: Added "gulp package" to pretest https://review.openstack.org/227407 | 21:51 |
openstackgerrit | Merged openstack/ironic-webclient: Split storage components into individual files https://review.openstack.org/227408 | 21:51 |
*** mgoddard has quit IRC | 21:53 | |
NobodyCam | krotscheck: betherly: wow ^^^^^^ \o/ | 21:54 |
betherly | Night all! | 21:55 |
NobodyCam | good night betherly | 21:55 |
*** willsama has quit IRC | 21:56 | |
*** david-lyle has joined #openstack-ironic | 21:56 | |
*** Marga_ has quit IRC | 21:59 | |
*** Marga_ has joined #openstack-ironic | 21:59 | |
*** Marga_ has quit IRC | 22:00 | |
*** Marga_ has joined #openstack-ironic | 22:00 | |
*** foexle has quit IRC | 22:01 | |
*** mgoddard has joined #openstack-ironic | 22:01 | |
*** harshs has quit IRC | 22:02 | |
*** harshs has joined #openstack-ironic | 22:03 | |
*** sambetts has quit IRC | 22:04 | |
*** e0ne has quit IRC | 22:04 | |
*** ijw has quit IRC | 22:06 | |
*** sambetts has joined #openstack-ironic | 22:06 | |
*** mjturek1 has quit IRC | 22:07 | |
krotscheck | NobodyCam: We've been busy | 22:12 |
openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Added $$resourceCache service. https://review.openstack.org/227444 | 22:13 |
openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Include fontawesome fonts https://review.openstack.org/228570 | 22:13 |
openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: [WIP] Split resources into individual files https://review.openstack.org/228571 | 22:13 |
*** cdearborn has quit IRC | 22:16 | |
*** mgoddard has quit IRC | 22:18 | |
jroll | wtf is up with IPA gate today :( | 22:18 |
krotscheck | jroll: The keg skunked | 22:21 |
* krotscheck may have an odd definition of IPA | 22:21 | |
jroll | hehe | 22:21 |
*** mgoddard has joined #openstack-ironic | 22:22 | |
jroll | at least the gate isn't too busy | 22:22 |
*** achanda has joined #openstack-ironic | 22:23 | |
*** itsuugo has quit IRC | 22:26 | |
*** mgoddard has quit IRC | 22:34 | |
*** mgoddard has joined #openstack-ironic | 22:38 | |
*** mgoddard has quit IRC | 22:44 | |
*** sambetts has quit IRC | 22:45 | |
*** sambetts has joined #openstack-ironic | 22:48 | |
*** mgoddard has joined #openstack-ironic | 22:48 | |
*** mgoddard has quit IRC | 22:53 | |
*** mgoddard has joined #openstack-ironic | 22:57 | |
*** mgoddard has quit IRC | 23:03 | |
*** harshs has quit IRC | 23:04 | |
*** mgoddard has joined #openstack-ironic | 23:07 | |
*** chlong has quit IRC | 23:07 | |
*** harshs has joined #openstack-ironic | 23:08 | |
*** ijw has joined #openstack-ironic | 23:09 | |
*** karimb has quit IRC | 23:11 | |
*** mgoddard has quit IRC | 23:13 | |
*** ijw has quit IRC | 23:13 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic-lib: Add more info level logs to disk_utils.work_on_disk() method https://review.openstack.org/230201 | 23:14 |
jlvillal | jroll: ^^^ Is 2nd patch to sync things up between ironic and ironic-lib | 23:14 |
openstackgerrit | John L. Villalovos proposed openstack/ironic-lib: Add more info level logs to disk_utils.work_on_disk() method https://review.openstack.org/230201 | 23:16 |
*** alexpilo_ has joined #openstack-ironic | 23:16 | |
*** mgoddard has joined #openstack-ironic | 23:16 | |
*** alexpilo_ has quit IRC | 23:21 | |
*** mgoddard has quit IRC | 23:28 | |
*** yuikotakada has quit IRC | 23:35 | |
*** mgoddard has joined #openstack-ironic | 23:36 | |
jroll | devananda: let's land https://review.openstack.org/#/c/229740/ and https://review.openstack.org/#/c/229692/ and release a 4.2.1 | 23:36 |
* jroll drops for a bit | 23:37 | |
*** baoli has quit IRC | 23:39 | |
*** mgoddard has quit IRC | 23:42 | |
*** mgoddard has joined #openstack-ironic | 23:43 | |
devananda | holy sh*t - we have a completed translation AND ITS JAPANESE? | 23:44 |
devananda | so JA is at 46%, which is apparently enough for inclusion | 23:45 |
*** lqu has quit IRC | 23:45 | |
devananda | ooh - it has some "approved". the the rlanguages don't have any approved | 23:46 |
devananda | we have *9* more languages above 33% that just need approval | 23:46 |
devananda | jroll: ^ | 23:46 |
devananda | https://translate.openstack.org/iteration/view/ironic/master/languages | 23:46 |
*** achanda has quit IRC | 23:47 | |
*** mgoddard has quit IRC | 23:49 | |
devananda | both approved | 23:49 |
devananda | I'd love it if we can get more of those languages in the next release | 23:49 |
openstackgerrit | Shinn'ya Hoshino proposed openstack/ironic: Fix a bug error by passwords includes just numbers https://review.openstack.org/220385 | 23:49 |
*** ijw has joined #openstack-ironic | 23:52 | |
openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: [WIP] Configuration Resource https://review.openstack.org/229055 | 23:52 |
openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Removed $$defaultConfiguration, renamed 'api' to 'apiBase' https://review.openstack.org/230207 | 23:52 |
*** mgoddard has joined #openstack-ironic | 23:53 | |
krotscheck | In case someone wants to get started on what's turning out to be a rather large commit, that WIP patch has most of the functional bits in it now. It's just not well covered yet. | 23:53 |
*** achanda has joined #openstack-ironic | 23:54 | |
*** dims has joined #openstack-ironic | 23:56 | |
*** ijw has quit IRC | 23:57 | |
*** bnemec has joined #openstack-ironic | 23:57 | |
*** mgoddard has quit IRC | 23:58 | |
*** dims has quit IRC | 23:58 | |
*** dims_ has quit IRC | 23:58 | |
jroll | devananda: nice! thanks for the +As | 23:59 |
*** dims has joined #openstack-ironic | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!