*** rbudden has quit IRC | 00:00 | |
*** piet has quit IRC | 00:02 | |
*** admin0 has quit IRC | 00:07 | |
*** sdake has quit IRC | 00:07 | |
*** piet has joined #openstack-ironic | 00:08 | |
*** coolsvap|away has quit IRC | 00:12 | |
*** sdake has joined #openstack-ironic | 00:13 | |
*** coolsvap|away has joined #openstack-ironic | 00:14 | |
*** mbound_ has quit IRC | 00:29 | |
*** naohirot has joined #openstack-ironic | 00:32 | |
*** dims has quit IRC | 00:42 | |
*** metsuke has quit IRC | 00:45 | |
*** alexpilotti has quit IRC | 00:47 | |
*** thrash|biab is now known as thrash|g0ne | 00:59 | |
*** alexpilotti has joined #openstack-ironic | 01:07 | |
*** sdake has quit IRC | 01:10 | |
*** baoli has joined #openstack-ironic | 01:11 | |
*** alexpilotti has quit IRC | 01:13 | |
*** baoli has quit IRC | 01:16 | |
*** dims has joined #openstack-ironic | 01:16 | |
*** zhenguo has joined #openstack-ironic | 01:17 | |
*** alexpilotti has joined #openstack-ironic | 01:24 | |
*** alexpilotti has quit IRC | 01:28 | |
*** mbound_ has joined #openstack-ironic | 01:29 | |
*** mbound_ has quit IRC | 01:34 | |
*** dims has quit IRC | 01:40 | |
*** Haomeng has quit IRC | 01:43 | |
*** praneshp has joined #openstack-ironic | 01:46 | |
*** dims has joined #openstack-ironic | 01:50 | |
*** Haomeng has joined #openstack-ironic | 01:53 | |
*** dims has quit IRC | 02:06 | |
*** Pespa has quit IRC | 02:13 | |
*** sdake has joined #openstack-ironic | 02:17 | |
*** changzhi has joined #openstack-ironic | 02:21 | |
*** alexpilotti has joined #openstack-ironic | 02:23 | |
*** hoangcx has joined #openstack-ironic | 02:26 | |
*** alexpilotti has quit IRC | 02:27 | |
*** dims has joined #openstack-ironic | 02:29 | |
*** piet has quit IRC | 02:30 | |
*** sdake has quit IRC | 02:34 | |
*** sdake_ has joined #openstack-ironic | 02:35 | |
*** lazy_prince has joined #openstack-ironic | 02:39 | |
*** hoangcx has left #openstack-ironic | 02:40 | |
*** dims has quit IRC | 02:45 | |
*** dims has joined #openstack-ironic | 02:47 | |
*** rloo has quit IRC | 02:49 | |
*** baoli has joined #openstack-ironic | 03:05 | |
*** rvasilets___ has quit IRC | 03:05 | |
*** links has joined #openstack-ironic | 03:06 | |
*** baoli has quit IRC | 03:08 | |
*** dims has quit IRC | 03:08 | |
*** MattMan has quit IRC | 03:09 | |
*** MattMan has joined #openstack-ironic | 03:09 | |
*** rbudden has joined #openstack-ironic | 03:10 | |
*** praneshp has quit IRC | 03:11 | |
*** piet has joined #openstack-ironic | 03:13 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add console feature to ssh driver https://review.openstack.org/241453 | 03:16 |
---|---|---|
*** greghaynes has quit IRC | 03:16 | |
*** greghaynes has joined #openstack-ironic | 03:17 | |
*** coolsvap|away is now known as coolsvap | 03:21 | |
*** baoli has joined #openstack-ironic | 03:27 | |
*** baoli has quit IRC | 03:28 | |
*** piet has quit IRC | 03:28 | |
*** praneshp has joined #openstack-ironic | 03:30 | |
*** smoriya_ has quit IRC | 03:34 | |
*** baoli has joined #openstack-ironic | 03:34 | |
*** baoli has quit IRC | 03:35 | |
*** praneshp_ has joined #openstack-ironic | 03:36 | |
*** praneshp has quit IRC | 03:37 | |
*** praneshp_ is now known as praneshp | 03:37 | |
*** deray has joined #openstack-ironic | 03:54 | |
deray | hi ironicers! good morning | 04:02 |
* deray trying to figure whats the issue with gate jobs's failure in https://review.openstack.org/#/c/259921/ | 04:04 | |
*** alexpilotti has joined #openstack-ironic | 04:24 | |
Haomeng | deray: morning:) | 04:25 |
Haomeng | deray: I think it depens on a tox file patch | 04:26 |
Haomeng | deray: this one - https://review.openstack.org/#/c/259445/, fixed by lucas already | 04:27 |
Haomeng | deray: so we can have a recheck after a while | 04:27 |
Haomeng | deray: and try to rebase | 04:28 |
*** alexpilotti has quit IRC | 04:29 | |
*** deray has quit IRC | 04:41 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Add support for providing custom Swift endpoint URLs https://review.openstack.org/251277 | 04:44 |
*** deray has joined #openstack-ironic | 04:46 | |
*** alexpilotti has joined #openstack-ironic | 04:47 | |
*** rbudden has quit IRC | 04:47 | |
deray | Haomeng: gm | 04:50 |
deray | Haomeng: thx for your suggestion, will do :) | 04:50 |
*** piet has joined #openstack-ironic | 04:51 | |
*** alexpilotti has quit IRC | 04:52 | |
*** praneshp has quit IRC | 04:57 | |
*** praneshp has joined #openstack-ironic | 04:58 | |
*** kalpase has joined #openstack-ironic | 05:02 | |
*** bkumar has joined #openstack-ironic | 05:05 | |
*** piet has quit IRC | 05:05 | |
*** piet has joined #openstack-ironic | 05:13 | |
Haomeng | deray: wel:) | 05:14 |
*** sdake_ has quit IRC | 05:23 | |
*** gcb has joined #openstack-ironic | 05:24 | |
*** alexpilotti has joined #openstack-ironic | 05:24 | |
*** alexpilotti has quit IRC | 05:29 | |
*** alexpilotti has joined #openstack-ironic | 05:45 | |
*** baoli has joined #openstack-ironic | 05:47 | |
*** lazy_prince has quit IRC | 05:47 | |
*** alexpilotti has quit IRC | 05:50 | |
*** baoli has quit IRC | 05:51 | |
*** xiaomaoke has joined #openstack-ironic | 05:52 | |
*** alexpilotti has joined #openstack-ironic | 06:06 | |
openstackgerrit | Merged openstack/ironic: Add extensions to the scripts at devstack/tools/ironic/scripts https://review.openstack.org/259139 | 06:10 |
*** alexpilotti has quit IRC | 06:11 | |
*** alexpilotti has joined #openstack-ironic | 06:23 | |
*** Marga_ has joined #openstack-ironic | 06:25 | |
*** Marga__ has joined #openstack-ironic | 06:26 | |
*** vdrok has quit IRC | 06:27 | |
*** alexpilotti has quit IRC | 06:28 | |
*** Marga_ has quit IRC | 06:30 | |
*** changzhi has quit IRC | 06:39 | |
*** ChubYann has quit IRC | 06:40 | |
*** tangchen has quit IRC | 06:44 | |
*** alexpilotti has joined #openstack-ironic | 06:44 | |
*** alexpilotti has quit IRC | 06:49 | |
*** piet has quit IRC | 06:57 | |
*** sdake has joined #openstack-ironic | 06:58 | |
*** coolsvap is now known as coolsvap|away | 06:58 | |
*** coolsvap|away is now known as coolsvap | 07:04 | |
openstackgerrit | Debayan Ray proposed openstack/ironic: Activate iLO license as part of manual cleaning process https://review.openstack.org/259921 | 07:04 |
*** alexpilotti has joined #openstack-ironic | 07:05 | |
*** alexpilotti has quit IRC | 07:09 | |
deray | hi Haomeng, will you able to have a look at this patch, https://review.openstack.org/#/c/211019/9, when you are a little free. thx in advance. | 07:09 |
Haomeng | deray: sure | 07:10 |
deray | Haomeng: let me know if you need further context. thank you :) | 07:11 |
Haomeng | deray: but I am not familiar with ilo:) | 07:12 |
Haomeng | deray: ok | 07:12 |
deray | Haomeng: its just that there is a new method "update_firmware" which has been added to the ilo/management interface, as a manual clean step, and this will update the firmware of iLO, CPLD, Power PIC and ROM of the baremetal in the OOB way. | 07:15 |
deray | The actual firmware update will be done by the ``proliantutils`` library. | 07:15 |
Haomeng | deray: ok, will try to get understanding and review:) | 07:15 |
*** sdake has quit IRC | 07:15 | |
Haomeng | deray: ok, got:) | 07:16 |
openstackgerrit | Yuiko Takada proposed openstack/ironic: Migrate Tempest tests into Ironic tree https://review.openstack.org/253982 | 07:16 |
openstackgerrit | Yuiko Takada proposed openstack/ironic: Add support API microversions for Tempest tests https://review.openstack.org/260358 | 07:16 |
*** vinm213 has joined #openstack-ironic | 07:19 | |
*** praneshp_ has joined #openstack-ironic | 07:22 | |
*** praneshp has quit IRC | 07:23 | |
*** praneshp_ is now known as praneshp | 07:23 | |
*** alexpilotti has joined #openstack-ironic | 07:24 | |
deray | Haomeng: thx.. this code also takes care of the extraction of the firmware image file (actual extraction will still happen on the proliantutils side). | 07:24 |
Haomeng | deray: wel:) | 07:25 |
Haomeng | deray: ok | 07:25 |
*** alexpilotti has quit IRC | 07:29 | |
*** Marga__ has quit IRC | 07:30 | |
*** vinm213 has quit IRC | 07:33 | |
*** vinm213 has joined #openstack-ironic | 07:35 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Enable ironic node pty console by default https://review.openstack.org/256272 | 07:36 |
openstackgerrit | Tan Lin proposed openstack/ironic: Add choices option to several options https://review.openstack.org/251732 | 07:38 |
openstackgerrit | Tan Lin proposed openstack/ironic: Add possible values config info https://review.openstack.org/250132 | 07:38 |
openstackgerrit | Tan Lin proposed openstack/ironic: Remove duplicate doc on CONF https://review.openstack.org/260365 | 07:38 |
*** ukalifon1 has joined #openstack-ironic | 07:38 | |
*** Marga_ has joined #openstack-ironic | 07:44 | |
openstackgerrit | Yuiko Takada proposed openstack/ironic: Migrate Tempest tests into Ironic tree https://review.openstack.org/253982 | 07:56 |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic-lib: Use imageutils from oslo.utils https://review.openstack.org/257248 | 07:59 |
*** tangchen has joined #openstack-ironic | 08:02 | |
*** kalpase has quit IRC | 08:03 | |
*** kalpase has joined #openstack-ironic | 08:05 | |
*** ifarkas has joined #openstack-ironic | 08:05 | |
*** mkovacik_ has joined #openstack-ironic | 08:08 | |
*** e0ne has joined #openstack-ironic | 08:11 | |
*** bkumar has quit IRC | 08:16 | |
*** coolsvap is now known as coolsvap|away | 08:19 | |
openstackgerrit | Yuiko Takada proposed openstack/ironic: [WIP]Add APIs for State Machine https://review.openstack.org/258764 | 08:21 |
*** coolsvap|away is now known as coolsvap | 08:21 | |
openstackgerrit | Tan Lin proposed openstack/ironic: Remove duplicate doc in config options https://review.openstack.org/260365 | 08:22 |
*** alexpilotti has joined #openstack-ironic | 08:24 | |
*** e0ne has quit IRC | 08:28 | |
*** alexpilotti has quit IRC | 08:29 | |
*** ukalifon2 has joined #openstack-ironic | 08:30 | |
openstackgerrit | Yuiko Takada proposed openstack/ironic: [WIP]Add APIs for State Machine https://review.openstack.org/258764 | 08:30 |
*** ukalifon1 has quit IRC | 08:31 | |
*** e0ne has joined #openstack-ironic | 08:33 | |
*** teju has joined #openstack-ironic | 08:38 | |
*** alexpilotti has joined #openstack-ironic | 08:45 | |
*** alexpilotti has quit IRC | 08:50 | |
*** admin0 has joined #openstack-ironic | 09:04 | |
*** alexpilotti has joined #openstack-ironic | 09:08 | |
*** mbound_ has joined #openstack-ironic | 09:09 | |
*** alexpilotti has quit IRC | 09:12 | |
*** e0ne has quit IRC | 09:16 | |
*** e0ne has joined #openstack-ironic | 09:20 | |
chenke | is Inspection necessary part in Kilo? | 09:21 |
chenke | while i validate a node, it print blow info: | 09:22 |
chenke | inspect | None | not supported | 09:22 |
vsaienko | good morning Ironic team! | 09:23 |
vsaienko | does anyone know why https://review.openstack.org/#/c/255898/ is not merged? | 09:24 |
*** alexpilotti has joined #openstack-ironic | 09:24 | |
*** e0ne has quit IRC | 09:24 | |
aarefiev | chenke: inspect isn't core interface, so it may be None if unsupported by a driver. | 09:24 |
aarefiev | vsaienko: morning | 09:25 |
chenke | aarefiev: tku, how can i turn it off? | 09:25 |
chenke | aarefiev: you see it block my validate, so i want to turn it off. | 09:26 |
*** alexpilotti has quit IRC | 09:28 | |
*** xiaomaoke has quit IRC | 09:30 | |
aarefiev | chenke: what driver are you using? take a look on it's InspectInterface | 09:32 |
chenke | i'm using pxe_ipmitool | 09:34 |
*** derekh has joined #openstack-ironic | 09:35 | |
*** deray has quit IRC | 09:36 | |
aarefiev | chenke: enabled config option in [inspector] group | 09:37 |
*** alexpilotti has joined #openstack-ironic | 09:44 | |
*** alexpilotti has quit IRC | 09:49 | |
*** vinm213 has quit IRC | 09:49 | |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector-specs: High Availability for Ironic Inspector https://review.openstack.org/253675 | 09:52 |
*** naohirot has quit IRC | 09:55 | |
*** zhenguo has quit IRC | 10:01 | |
*** zhenguo has joined #openstack-ironic | 10:01 | |
*** mgould has joined #openstack-ironic | 10:04 | |
mgould | morning Ironic | 10:04 |
aarefiev | mgould: morning | 10:05 |
*** alexpilotti has joined #openstack-ironic | 10:05 | |
mgould | aarefiev, morning! | 10:06 |
*** alexpilotti has quit IRC | 10:10 | |
*** mgould has quit IRC | 10:13 | |
*** alexpilotti has joined #openstack-ironic | 10:23 | |
*** e0ne has joined #openstack-ironic | 10:26 | |
*** alexpilotti has quit IRC | 10:27 | |
*** daemontool has joined #openstack-ironic | 10:31 | |
*** Pespa has joined #openstack-ironic | 10:37 | |
*** rvasilets___ has joined #openstack-ironic | 10:49 | |
*** mgould has joined #openstack-ironic | 10:52 | |
*** ukalifon2 has quit IRC | 11:06 | |
*** 64MAALEJU has joined #openstack-ironic | 11:06 | |
*** vsaienko has quit IRC | 11:06 | |
*** gcb has quit IRC | 11:08 | |
*** dtantsur|afk is now known as dtantsur | 11:08 | |
*** vinm213 has joined #openstack-ironic | 11:08 | |
dtantsur | Morning Ironic, says man in the empty office :) | 11:08 |
mgould | morning dtantstur, me too :-) | 11:08 |
mgould | *dtantsur | 11:09 |
*** gcb has joined #openstack-ironic | 11:12 | |
*** alexpilotti has joined #openstack-ironic | 11:12 | |
Haomeng | deray: I am not familiar with iLO and FW updating, so just left some comments inline which are common code concerns. | 11:15 |
*** lucas-dinner is now known as lucasagomes | 11:15 | |
aarefiev | dtantsur: morning | 11:17 |
Haomeng | dtantsur: morning, still on duty, cool man:) | 11:18 |
dtantsur | :) | 11:18 |
*** deray has joined #openstack-ironic | 11:20 | |
*** pas-ha has joined #openstack-ironic | 11:21 | |
*** gcb has quit IRC | 11:23 | |
lucasagomes | :-) morning all | 11:26 |
dtantsur | o/ | 11:26 |
lucasagomes | gate still inconsistent :-/ do we have the tempest patch to remove rebuild proposed already ? | 11:29 |
dtantsur | yes | 11:30 |
dtantsur | lucasagomes, https://review.openstack.org/#/c/260121/ | 11:30 |
dtantsur | you can try pocking people on #openstack-qa | 11:30 |
lucasagomes | dtantsur, nice, yeah | 11:30 |
lucasagomes | I was thikning about adding a dummy patch in ironic with depends-on to see if it helps | 11:30 |
lucasagomes | compare the time it takes to run with other jobs | 11:31 |
dtantsur | lucasagomes, at least it will make gate time somewhat shorter.. | 11:31 |
lucasagomes | yeah | 11:31 |
lucasagomes | that's the plan | 11:31 |
dtantsur | and we won't take so much infra resources, even if we still fail occasionally | 11:31 |
dtantsur | but I suspect it might help | 11:31 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Dummy patch, testing gate https://review.openstack.org/260450 | 11:36 |
Haomeng | lucasagomes: morning:) | 11:46 |
lucasagomes | Haomeng, hi there! | 11:46 |
Haomeng | lucasagomes: will go home, today is late:) | 11:46 |
lucasagomes | Haomeng, oh have a great evening over there | 11:47 |
Haomeng | lucasagomes: nice day:) | 11:47 |
lucasagomes | Haomeng, enjoy the rest of the day | 11:47 |
Haomeng | lucasagomes: thanks lucas:) | 11:47 |
mgould | Haomeng, good night! | 11:48 |
*** tester567345 has joined #openstack-ironic | 11:51 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector-specs: CMDB integration https://review.openstack.org/253054 | 11:52 |
*** tester567345 has quit IRC | 11:55 | |
*** smoriya has quit IRC | 11:56 | |
*** smoriya_afk has joined #openstack-ironic | 11:58 | |
*** smoriya_afk is now known as smoriya | 11:58 | |
*** jaypipes has joined #openstack-ironic | 12:09 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-specs: Implement libvirt-based driver for virtual HW https://review.openstack.org/254421 | 12:12 |
*** daemontool has quit IRC | 12:19 | |
*** daemontool has joined #openstack-ironic | 12:21 | |
*** alexpilotti has quit IRC | 12:22 | |
*** dprince has joined #openstack-ironic | 12:34 | |
*** kalpase has quit IRC | 12:35 | |
*** yuriyz has quit IRC | 12:36 | |
*** ndipanov has joined #openstack-ironic | 12:36 | |
*** Pespa has left #openstack-ironic | 12:41 | |
*** kalpase has joined #openstack-ironic | 12:43 | |
*** yuriyz has joined #openstack-ironic | 12:47 | |
TheJulia | Good morning | 12:50 |
lucasagomes | TheJulia, hello there, good morning | 12:51 |
openstackgerrit | janonymous proposed openstack/ironic-inspector: Put py34 first in the env order of tox https://review.openstack.org/260487 | 12:53 |
*** sdake has joined #openstack-ironic | 12:54 | |
ifarkas | lucasagomes, dtantsur, hi, could you please review https://review.openstack.org/#/c/249256/ ? | 12:56 |
*** baoli has joined #openstack-ironic | 12:56 | |
lucasagomes | ifarkas, hi there! ++ sure | 12:56 |
ifarkas | thanks! | 12:56 |
openstackgerrit | janonymous proposed openstack/python-ironic-inspector-client: Put py34 first in the env order of tox https://review.openstack.org/260496 | 12:59 |
*** alexpilotti has joined #openstack-ironic | 12:59 | |
*** praneshp has quit IRC | 13:00 | |
*** dims has joined #openstack-ironic | 13:01 | |
*** alexpilotti has quit IRC | 13:09 | |
lucasagomes | ifarkas, +2, I left some suggestions inline but they are cosmetics | 13:10 |
ifarkas | lucasagomes, thanks, I will fix them anyway | 13:12 |
lucasagomes | ifarkas, no worries, unless you need to submit a new patch set | 13:12 |
lucasagomes | commented just in case | 13:12 |
ifarkas | lucasagomes, ah, ok. makes sense | 13:12 |
lucasagomes | it's comestic, to make some conditionals a bit more pythonic | 13:12 |
lucasagomes | so not a big deal | 13:13 |
openstackgerrit | janonymous proposed openstack/pyghmi: Put py34 first in the env order of tox https://review.openstack.org/260517 | 13:15 |
*** alexpilotti has joined #openstack-ironic | 13:16 | |
*** alexpilotti has quit IRC | 13:16 | |
*** sdake has quit IRC | 13:19 | |
jroll | morning y'all | 13:19 |
jroll | lucasagomes: dtantsur wdyt https://review.openstack.org/#/c/260235/ | 13:20 |
lucasagomes | jroll, morning | 13:20 |
* lucasagomes looks | 13:20 | |
jroll | kind of a step backwards but I think that's a large part of this | 13:20 |
jroll | zer0c00l_away pinged me last night though, is working on disabling some services | 13:21 |
lucasagomes | jroll, right, yeah def a step back... I'm waiting for the tests to run with the tempest removing rebuild to see if it helps | 13:21 |
jroll | yeah ok | 13:21 |
lucasagomes | but if using a older version is quicker to boot it's something to consider yes | 13:22 |
jroll | fwiw | 13:22 |
jroll | [ 130.874783] systemd[1]: Startup finished in 12.785s (kernel) + 31.947s (initrd) + 1min 25.692s (userspace) = 2min 10.425s. | 13:22 |
jroll | way faster | 13:22 |
jroll | and another: [ 105.378007] systemd[1]: Startup finished in 11.022s (kernel) + 24.276s (initrd) + 1min 9.837s (userspace) = 1min 45.136s. | 13:22 |
lucasagomes | yeah | 13:23 |
lucasagomes | jroll, we can also identify some of the services that are taking some time and disable it | 13:23 |
dtantsur | jroll, +2 | 13:23 |
jroll | lucasagomes: yeah, zer0c00l_away is looking at that | 13:23 |
jroll | I tried a bit yesterday, but couldn't find much that didn't seem necessary | 13:24 |
jroll | also, afaik cloud-config is the way to do that, which runs after most of the services | 13:24 |
lucasagomes | jroll, yeah, there's a couple of things I think we can do to make it faster in the service itself | 13:25 |
jroll | dtantsur: heh, so quick, thanks | 13:25 |
lucasagomes | e.g | 13:25 |
lucasagomes | jroll, http://logs.openstack.org/50/260450/1/check/gate-tempest-dsvm-ironic-agent_ssh/0971134/logs/devstacklog.txt.gz#_2015-12-22_12_57_19_262 | 13:26 |
lucasagomes | if we could get it from some local server | 13:26 |
lucasagomes | http://logs.openstack.org/50/260450/1/check/gate-tempest-dsvm-ironic-agent_ssh/0971134/logs/devstacklog.txt.gz#_2015-12-22_12_57_50_359 | 13:26 |
lucasagomes | look how long it takes to download that ramdisk | 13:26 |
jroll | I mean | 13:26 |
jroll | 30 seconds isn't bad | 13:26 |
jroll | but devstack does have a way to cache that | 13:27 |
lucasagomes | numpy takes a lot of time to build too | 13:27 |
jroll | anyway, I think we should land that revert until we figure out how to make the ramdisk lighter | 13:27 |
lucasagomes | but idk if we can do sth about it | 13:27 |
jroll | and keep the rebuild/cleaning tests around | 13:27 |
jroll | we don't lose a whole lot by using older coreos, afaik | 13:27 |
lucasagomes | ack | 13:27 |
lucasagomes | yeah that's alright | 13:27 |
lucasagomes | the rebuild tests passed as well btw | 13:28 |
jroll | yeah | 13:28 |
lucasagomes | https://review.openstack.org/#/c/260450/ | 13:28 |
lucasagomes | +a the revert | 13:28 |
lucasagomes | let's give it a go | 13:28 |
jroll | http://logs.openstack.org/50/260450/1/check/gate-tempest-dsvm-ironic-agent_ssh/0971134/console.html#_2015-12-22_13_22_08_061 | 13:28 |
jroll | that's running very close to timeout | 13:29 |
lucasagomes | yeah | 13:29 |
lucasagomes | 1h + | 13:29 |
dtantsur | I still think running rebuild in all gates isn't the way to go | 13:29 |
jroll | dtantsur: I agree, I hope someone will look at how to deal with that | 13:29 |
lucasagomes | dtantsur, it's a valid test, but i wish we could have a personalized test for that too | 13:29 |
*** kalpase has quit IRC | 13:29 | |
jroll | then again, it feels wasteful to use an entire devstack build for rebuild :P | 13:29 |
lucasagomes | as we talked in the meeting, starting with an active enode and so on | 13:30 |
*** mbound_ has quit IRC | 13:30 | |
dtantsur | last time I checked, the spec for enrolling active nodes was banning rebuild, but I hope it will change | 13:30 |
lucasagomes | yeah | 13:31 |
lucasagomes | it may be tricky | 13:31 |
dtantsur | my take on that spec is that any node we enroll should be full-features, i.e. pass all validations and be able to rebuild | 13:32 |
TheJulia | dtantsur: I was basically going to re-write it to allow it to occur, but bottom line is its less of a streamlined process and the operator will still be responsible for supplying all required details. | 13:32 |
TheJulia | dtantsur: which basically meets your want of validations to be able to pass | 13:33 |
dtantsur | awesome | 13:33 |
TheJulia | Just, brain got completely derailed by covering for badcub and nobodycam last month | 13:34 |
*** athomas has joined #openstack-ironic | 13:36 | |
*** alexpilotti has joined #openstack-ironic | 13:39 | |
*** alexpilotti has quit IRC | 13:44 | |
*** piet has joined #openstack-ironic | 13:44 | |
*** mbound_ has joined #openstack-ironic | 13:46 | |
*** vdrok has joined #openstack-ironic | 13:46 | |
*** admin0 has left #openstack-ironic | 13:52 | |
*** alexpilotti has joined #openstack-ironic | 13:53 | |
*** mbound_ has quit IRC | 13:54 | |
*** alexpilotti has quit IRC | 13:55 | |
*** links has quit IRC | 13:59 | |
*** piet has quit IRC | 13:59 | |
vdrok | good morning everyone | 14:01 |
vdrok | is it only me, or gerrit is slow today? | 14:01 |
dtantsur | didn't notice particular slowness | 14:02 |
dtantsur | and morning vdrok :) | 14:02 |
vdrok | morning dtantsur :) | 14:02 |
lucasagomes | vdrok, morning! | 14:07 |
lucasagomes | yeah it was a bit slow here, but I thought it was my internet | 14:07 |
lucasagomes | maybe not | 14:07 |
vdrok | morning lucasagomes | 14:07 |
vdrok | now it started giving me 502 sometimes | 14:07 |
*** Marga_ has quit IRC | 14:07 | |
mgould | vdrok, lucasagomes: morning! | 14:07 |
vdrok | morning mgould | 14:07 |
lucasagomes | mgould, hey there! morning | 14:07 |
*** Marga_ has joined #openstack-ironic | 14:08 | |
lucasagomes | vdrok, ++ 502 Proxy Error here too | 14:09 |
*** sdake has joined #openstack-ironic | 14:09 | |
vdrok | crap, holidays are coming indeed :) | 14:09 |
dtantsur | :D | 14:10 |
*** alexpilotti has joined #openstack-ironic | 14:10 | |
lucasagomes | lol | 14:10 |
lucasagomes | yeah, today's my last work day of the year | 14:10 |
*** rbudden has joined #openstack-ironic | 14:11 | |
*** Marga_ has quit IRC | 14:13 | |
*** thrash|g0ne is now known as thrash | 14:16 | |
*** athomas has quit IRC | 14:20 | |
lucasagomes | yeah gerrit seems down now :-/ | 14:20 |
* lucasagomes thinks he lost his replies | 14:20 | |
*** athomas has joined #openstack-ironic | 14:20 | |
*** admin0 has joined #openstack-ironic | 14:24 | |
*** teju has left #openstack-ironic | 14:25 | |
*** piet has joined #openstack-ironic | 14:31 | |
*** mgould has quit IRC | 14:34 | |
deray | thx lucasagomes for your post. | 14:40 |
* deray was wondering why review.openstack.org not accessible | 14:41 | |
vdrok | deray, people in infra are looking into it already | 14:41 |
deray | oh okay .. was in the middle of addressing some comments. | 14:42 |
deray | vdrok, lucasagomes, if you have tiem today, can you have a look at this patch https://review.openstack.org/#/c/211019 raised by me? | 14:44 |
deray | this patch was raised quite a while back. but I have started workign on it freshly. thx in advance. | 14:45 |
*** alexpilotti has quit IRC | 14:45 | |
*** alexpilotti has joined #openstack-ironic | 14:45 | |
openstackgerrit | William Stevenson proposed openstack/ironic-python-agent: Adding lldp functionality to IPA https://review.openstack.org/260567 | 14:48 |
*** rloo has joined #openstack-ironic | 14:48 | |
*** mgould has joined #openstack-ironic | 14:48 | |
dtantsur | jroll, revert has failed in gate, hmm | 14:48 |
jroll | lol | 14:48 |
jroll | of course it did :x | 14:49 |
*** alexpilotti has quit IRC | 14:49 | |
*** alexpilotti has joined #openstack-ironic | 14:49 | |
jroll | dtantsur: http://logs.openstack.org/35/260235/2/gate/gate-tempest-dsvm-ironic-agent_ssh-src/a52f8f5/console.html#_2015-12-22_14_08_52_014 | 14:49 |
dtantsur | yeah.. | 14:50 |
*** vinm213 has quit IRC | 14:54 | |
*** links has joined #openstack-ironic | 14:54 | |
lucasagomes | deray, hi there, sorry i was in a call | 14:58 |
lucasagomes | I will look at the patch, I will just grab some quick lunch | 14:59 |
lucasagomes | and look at it right after | 14:59 |
*** lucasagomes is now known as lucas-hungry | 14:59 | |
*** deray has quit IRC | 15:01 | |
rloo | hi, morning dtantsur, jroll, lucas-hungry. Gate still isn't happy :-( | 15:02 |
dtantsur | morning rloo, that's true.. | 15:04 |
*** dtantsur is now known as dtantsur|brb | 15:04 | |
jroll | hi rloo | 15:04 |
rloo | dtantsur, jroll: things are failing in stable/kilo too. do you know if it is for the same reasons? | 15:05 |
jroll | rloo: no clue, honestly, I haven't looked | 15:05 |
* dtantsur|brb is not sure either | 15:05 | |
rloo | jroll: ok, if it is for the same reasons then there are other things that changed besides our code. | 15:05 |
rloo | jroll: unless stable/kilo is using master ipa. hmm. | 15:06 |
*** alexpilotti has quit IRC | 15:06 | |
*** alexpilotti has joined #openstack-ironic | 15:06 | |
dtantsur|brb | rloo, everything uses master IPA iirc | 15:06 |
rloo | dtantsur|brb: sigh. | 15:07 |
*** ndipanov has quit IRC | 15:08 | |
*** sdake has quit IRC | 15:10 | |
rloo | jroll, dtantsur|brb, etc, did you see this about nova - containers: https://review.openstack.org/#/c/256440/ | 15:12 |
jroll | rloo: yep, been following it | 15:16 |
rloo | jroll: it is very interesting... | 15:17 |
jroll | it is | 15:18 |
jroll | I don't think it affects us today, but it may open up some new ideas in the future | 15:18 |
*** sinh has quit IRC | 15:19 | |
*** zigo_ is now known as zigo | 15:19 | |
*** sinh has joined #openstack-ironic | 15:20 | |
NobodyCam | good morning Ironicers | 15:31 |
BadCub | morning folks | 15:31 |
NobodyCam | happy almost holiday | 15:31 |
rloo | morning NobodyCam, BadCub, and happy almost holiday to you too! | 15:33 |
BadCub | and to you too rloo :) | 15:34 |
NobodyCam | morning rloo :) | 15:35 |
NobodyCam | is yahoo doing a holiday shutdown this year? | 15:35 |
*** alexpilotti has quit IRC | 15:35 | |
*** alexpilotti has joined #openstack-ironic | 15:37 | |
openstackgerrit | Ruby Loo proposed openstack/ironic: Add console feature to ssh driver https://review.openstack.org/241453 | 15:38 |
rloo | NobodyCam: they stopped the holiday shutdown a while ago. During Mayer's reign. (But regardless, I shutdown anyway!) | 15:40 |
NobodyCam | lol ++++ | 15:41 |
*** alexpilotti has quit IRC | 15:42 | |
*** sinh has quit IRC | 15:47 | |
*** sinh has joined #openstack-ironic | 15:54 | |
*** baoli has quit IRC | 15:55 | |
*** baoli has joined #openstack-ironic | 15:56 | |
*** davideagnello has quit IRC | 15:58 | |
*** teju has joined #openstack-ironic | 15:59 | |
*** davideagnello has joined #openstack-ironic | 15:59 | |
*** awiddersheim has quit IRC | 15:59 | |
*** lucas-hungry is now known as lucasagomes | 16:04 | |
lucasagomes | rloo, good morning | 16:04 |
jroll | lucasagomes: this cockpit thing in fedora is nice | 16:05 |
rloo | lucasagomes: :) | 16:05 |
NobodyCam | morning lucasagomes | 16:06 |
lucasagomes | jroll, yeah I read a bit a bout it but haven't used yet | 16:06 |
lucasagomes | that's the sys mgmt thing right? | 16:06 |
lucasagomes | NobodyCam, morning | 16:06 |
jroll | yeah | 16:06 |
NobodyCam | :) | 16:06 |
jroll | I mean, it's not that useful to me | 16:06 |
NobodyCam | morning jroll | 16:06 |
jroll | but pretty graphs and stuff | 16:06 |
jroll | hey, morning NobodyCam | 16:06 |
openstackgerrit | Merged openstack/bifrost: Syntax changes for use of pip_install https://review.openstack.org/260219 | 16:06 |
lucasagomes | heh yeah, may worth taking a look at what they used to created it | 16:07 |
*** Marga_ has joined #openstack-ironic | 16:09 | |
*** ifarkas has quit IRC | 16:09 | |
lucasagomes | jroll, btw re indexable value field question in #253605 I don't actually know the performance answer, probably devananda can talk about it cause he understands db's way better than I do | 16:10 |
lucasagomes | but AFAIK, we shouldn't index a TEXT field | 16:10 |
NobodyCam | :) | 16:10 |
jroll | lucasagomes: yeah, I just don't know how this is going to improve filter speed then | 16:10 |
lucasagomes | cause it can contain a lot of text (unlimited) | 16:10 |
jroll | because "WHERE key=cpus and value=2" won't be performant, if we aren't indexing | 16:11 |
*** mgould has quit IRC | 16:11 | |
lucasagomes | jroll, yeah, I'm counting with the fact that it's now query-able... Where before it lived in a blob | 16:12 |
lucasagomes | so we couldn't even write a query to it | 16:12 |
jroll | right | 16:12 |
lucasagomes | jroll, yeah, let's wait for devananda or someone's else that knows better | 16:12 |
* lucasagomes googles too | 16:12 | |
*** Marga_ has quit IRC | 16:13 | |
*** mgould has joined #openstack-ironic | 16:15 | |
*** dtantsur|brb is now known as dtantsur | 16:15 | |
dtantsur | morning NobodyCam, BadCub | 16:15 |
rloo | hey, ipa is versioned/packaged now, right? (or is it?) | 16:16 |
NobodyCam | hey hey dtantsur | 16:16 |
BadCub | morning dtantsur :) | 16:17 |
dtantsur | rloo, it is | 16:17 |
rloo | if ipa is versioned, shouldn't we have it in our requirements? | 16:17 |
*** daemontool has quit IRC | 16:17 | |
jroll | rloo: no, we don't import it | 16:17 |
jroll | there's a couple things we need to do: | 16:17 |
jroll | 1) stable branch tarballs of IPA | 16:18 |
jroll | 2) use those in stable branch ironic tests | 16:18 |
mariojv | rloo: it is versioned though: https://github.com/openstack/ironic-python-agent/blob/master/setup.cfg#L3 | 16:18 |
rloo | jroll: do we have bugs/wishlist for that stuff? | 16:19 |
jroll | rloo: dunno | 16:19 |
rloo | jroll, mariojv: we don't even mention ipa in our driver-requirements.txt. I guess our assumption has always been to use master ipa. | 16:20 |
jroll | it has, we did a liberty release for red hat and never followed up on the rest :( | 16:21 |
dtantsur | rloo, it's useless to mention IPA in any *-requirements.txt, it's not installed like that | 16:21 |
rloo | jroll, mariojv: i was wondering what we do wrt ironic + ipa and backwards compat (or not) of older ipa. | 16:21 |
dtantsur | using master IPA is fine, but it requires a bit stronger promises about backward compatibility | 16:21 |
dtantsur | and probably gating IPA master against liberty and kilo openstack | 16:22 |
rloo | dtantsur: oh, right. well, not totally useless, but we need to document somewhere wrt backwards compat (or not). | 16:22 |
dtantsur | rloo, I just don't want people to pip install ipa on their conductors :D | 16:22 |
mariojv | i think what dtantsur is saying is correct, IPA is just installed on the nodes ironic manages right? not at all in ironic itself | 16:22 |
rloo | dtantsur: right! heh. | 16:22 |
jroll | rloo: we always said we remain compat with every ipa commit, but now that we release IPA we can probably change it | 16:23 |
*** teju has quit IRC | 16:23 | |
lucasagomes | mariojv, yes | 16:23 |
jroll | rloo: I would like to say, for instance, liberty IPA is compatible with mitaka ironic and vice versa, and maybe even go two versions each way, idk | 16:24 |
*** baoli has quit IRC | 16:24 | |
dtantsur | jroll, the revert has failed the 2nd time.. I'm starting to think that switch to newer coreos is not the cause | 16:25 |
rloo | jroll etc: i was just reading this spec and wondering about backwards compat (or not): https://review.openstack.org/#/c/248968/ | 16:25 |
jroll | dtantsur: yeah, maybe, feel free to -A it | 16:25 |
*** baoli has joined #openstack-ironic | 16:26 | |
*** coolsvap is now known as coolsvap|away | 16:26 | |
rloo | dtantsur: i was wondering if there is a multiple of causes, and/or if it is the hw/setup in *some* test environments. | 16:26 |
dtantsur | likely yes... I still want to drop rebuild and see if it fixes anything | 16:26 |
*** athomas has quit IRC | 16:27 | |
jroll | rloo: which piece of it? I guess we should fail gracefully if IPA doesn't support that | 16:27 |
*** alexpilotti has joined #openstack-ironic | 16:27 | |
lucasagomes | yeah and it failed at in the -src job | 16:27 |
lucasagomes | :-/ | 16:27 |
* lucasagomes looks if it was timeout | 16:28 | |
lucasagomes | http://logs.openstack.org/35/260235/2/check/gate-tempest-dsvm-ironic-agent_ssh-src/1c6cf96/logs/devstacklog.txt.gz#_2015-12-22_16_01_37_761 | 16:28 |
lucasagomes | it wasn't | 16:28 |
jroll | oh | 16:28 |
rloo | jroll: it'll need changes to IPA to do that right? so what happens if an older IPA is running. | 16:28 |
jroll | 2015-12-22 16:01:37.651 | 1mEm1m: m1mFailed to fetch http://httpredir.debian.org/debian/pool/main/g/gnutls28/libgnutls-deb0-28_3.3.8-6+deb8u3_amd64.deb Error reading from server. Remote end closed connection [IP: 5.153.231.35 80] | 16:28 |
jroll | rloo: right, so like I said we should fail gracefully, or somehow determine that IPA supports that ahead of time (which is really hard since you can't do that without booting the ramdisk) | 16:29 |
rloo | lucasagomes: sigh. is that a new failure?! | 16:29 |
lucasagomes | probably just a momentum one | 16:30 |
jroll | that's just a random failure | 16:30 |
dtantsur | mmm, recheck once more? | 16:30 |
* lucasagomes tries build it locally | 16:30 | |
jroll | it's what happens when you depend on the internet | 16:30 |
lucasagomes | yeah the link seems to wrok | 16:30 |
lucasagomes | work* | 16:30 |
lucasagomes | dtantsur, wait a little before recheck | 16:31 |
rloo | jroll: right. i'm guessing we're going to need to deal with that cuz there may be more future features like this. | 16:31 |
dtantsur | lucasagomes, I'll leave it up to someone else | 16:31 |
lucasagomes | ack | 16:31 |
lucasagomes | yeah I'm just looking at the logs from the first and second ran | 16:31 |
jroll | rloo: yeah, I feel like we may have used capabilities before | 16:32 |
*** alexpilotti has quit IRC | 16:32 | |
rloo | jroll: I'm just going to comment about it and punt it for someone else to think about :) | 16:33 |
jroll | heh, ok | 16:33 |
lucasagomes | the first run it built, but timed out on "wait call-back" | 16:34 |
lucasagomes | http://logs.openstack.org/35/260235/2/gate/gate-tempest-dsvm-ironic-agent_ssh-src/a52f8f5/logs/screen-ir-cond.txt.gz#_2015-12-22_14_23_29_526 | 16:34 |
lucasagomes | at deploy time (not even rebuild), no lookup | 16:34 |
lucasagomes | jroll, http://logs.openstack.org/35/260235/2/gate/gate-tempest-dsvm-ironic-agent_ssh-src/a52f8f5/logs/ironic-bm-logs/baremetalbrbm_0_console_22-12-2015-14:23:35.txt.gz | 16:34 |
lucasagomes | the console show some dependencies errors when booting IPA | 16:34 |
lucasagomes | probably just the revert is not enough | 16:34 |
lucasagomes | Failed to add dependency on opt-ironic\x2dpython\x2dagent-run.moun, ignoring: Invalid argument | 16:35 |
*** dprince has quit IRC | 16:35 | |
lucasagomes | there's a missing .mount there | 16:35 |
jroll | O_o | 16:35 |
jroll | notice .moun | 16:35 |
jroll | which was a bug we fixed already | 16:36 |
lucasagomes | tho it seems that it tried to start... not sure it failed later (we don't have the "systemctl status" output there) | 16:36 |
jroll | ugh | 16:38 |
jroll | https://review.openstack.org/#/c/257521/ | 16:38 |
lucasagomes | jroll, I will try to build it locally see what I can find... removed the +a for now | 16:38 |
lucasagomes | jroll, oh x.x | 16:39 |
*** keekz has joined #openstack-ironic | 16:41 | |
*** piet has quit IRC | 16:41 | |
*** dims has quit IRC | 16:42 | |
*** vishwana_ has joined #openstack-ironic | 16:42 | |
*** piet has joined #openstack-ironic | 16:44 | |
*** vishwanathj has quit IRC | 16:45 | |
lucasagomes | rloo, yeah I'm not found of the new gerrit interface either :-) | 16:46 |
lucasagomes | fond* | 16:46 |
rloo | lucasagomes: I have no idea how I did that! :-( Thx for asking about it. | 16:46 |
lucasagomes | happens, no bothers | 16:47 |
*** mgould has quit IRC | 16:47 | |
*** admin0 has quit IRC | 16:50 | |
*** mgould has joined #openstack-ironic | 16:52 | |
*** praneshp has joined #openstack-ironic | 16:53 | |
*** Marga_ has joined #openstack-ironic | 17:10 | |
*** dims has joined #openstack-ironic | 17:12 | |
*** daemontool has joined #openstack-ironic | 17:13 | |
*** boris-42 has quit IRC | 17:13 | |
*** Marga_ has quit IRC | 17:15 | |
*** piet has quit IRC | 17:15 | |
*** davideagnello has quit IRC | 17:15 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: [PROOF OF CONCEPT] Ansible deploy driver https://review.openstack.org/238183 | 17:17 |
*** alexpilotti has joined #openstack-ironic | 17:21 | |
*** rloo has quit IRC | 17:22 | |
*** rbudden has quit IRC | 17:23 | |
*** davideagnello has joined #openstack-ironic | 17:23 | |
*** alexpilotti has quit IRC | 17:26 | |
*** derekh has quit IRC | 17:29 | |
dtantsur | see you tomorrow folks (happy holidays to those not working tomorrow) | 17:30 |
*** devananda has quit IRC | 17:30 | |
lucasagomes | dtantsur, I won't be here tomorrow so happy holidays! | 17:35 |
dtantsur | :) | 17:35 |
*** dtantsur is now known as dtantsur|afk | 17:35 | |
*** alexpilotti has joined #openstack-ironic | 17:37 | |
*** piet has joined #openstack-ironic | 17:38 | |
mgould | dtantsur|afk, happy holidays! | 17:39 |
*** dprince has joined #openstack-ironic | 17:39 | |
*** devananda has joined #openstack-ironic | 17:40 | |
mgould | lucasagomes, I'm trying to help d0ugal debug an ironic problem | 17:40 |
lucasagomes | mgould, hey there... which problem is it? | 17:40 |
*** eggmaster has quit IRC | 17:40 | |
d0ugal | lucasagomes: https://www.dropbox.com/s/3aiy2rmjr1ga3z4/Screenshot_baremetalbrbm_3_2015-12-22_17%3A07%3A54.png?dl=0 | 17:41 |
mgould | the error message includes "inc: command not found", so I'm wondering if it's related to your Peano-arithmetic patch | 17:41 |
lucasagomes | mgould, d0ugal yes it's | 17:41 |
lucasagomes | d0ugal, are you running o CentOS or RHEL right? | 17:41 |
d0ugal | lucasagomes: CentOS, yup. This is a tripleo virt setup | 17:42 |
lucasagomes | that's because the iPXE ROM on those OSs are too old and the inc command is not available on that version of iPXE | 17:42 |
lucasagomes | d0ugal, is it a virt setup or baremetal? | 17:42 |
lucasagomes | if virt: https://review.openstack.org/#/c/250745/ | 17:42 |
d0ugal | lucasagomes: virt | 17:42 |
lucasagomes | if baremetal you can just get a new iPXE ROM pre-built and drop in the /tftpboot dir | 17:42 |
lucasagomes | you can get it from http://boot.ipxe.org/ | 17:43 |
lucasagomes | d0ugal, ok yeah so you can apply that #250745 to the Ironic conductor | 17:43 |
lucasagomes | it will workaround the inc not found command | 17:43 |
d0ugal | lucasagomes: awesome. | 17:43 |
d0ugal | lucasagomes: How would you apply it? | 17:43 |
lucasagomes | d0ugal, just apply it in the ironic-conductor and restart the service | 17:43 |
mgould | the workaround involves doing unary arithmetic using string concatenation :-) | 17:44 |
lucasagomes | or you can just copy that iboot.ipxe template from the patch | 17:44 |
d0ugal | mgould: No idea what that means. :) | 17:44 |
lucasagomes | write to a file somewhere in the ironic-conductor and change the ironic.conf pointing to that file | 17:44 |
lucasagomes | ipxe_boot_script=<path to the new script> | 17:44 |
d0ugal | neat | 17:44 |
d0ugal | lucasagomes: after I've patched, can I just restart the deploy or do I need to do anything else? | 17:45 |
lucasagomes | d0ugal, just restart the ironic-conductor service and restart the deloy yes | 17:45 |
lucasagomes | deploy* | 17:45 |
d0ugal | lucasagomes: Great, thanks | 17:45 |
lucasagomes | no problem | 17:45 |
mgould | d0ugal, it represents numbers as strings "", "x", "xx" etc and increments them by adding another "x" | 17:45 |
d0ugal | I am running out of time today but I'll try and give it a go quickly | 17:45 |
lucasagomes | mgould, the patch I pointed him to just fallback to the old behavior if "inc" is not found | 17:46 |
d0ugal | mgould: oh... :) | 17:46 |
mgould | lucasagomes, ah, right | 17:46 |
lucasagomes | we thought that the arithmetic one I did was too ugly to be upstream | 17:46 |
d0ugal | lol | 17:46 |
lucasagomes | cause, in fairness, CentOS/RHEL should update their packages | 17:46 |
lucasagomes | that inc command exist since begging of 2013 | 17:46 |
lucasagomes | so it's not fair to push that ugly thing upstream due that | 17:46 |
mgould | lucasagomes, to quote the Jargon File, "it awes with its fiendish brilliance, even as its crockishness appalls" :-) | 17:47 |
lucasagomes | lol | 17:47 |
lucasagomes | d0ugal, mgould and FYI the owner of the ipxe patch was already communicated and he's rebasing it for 7.3 | 17:47 |
*** e0ne has quit IRC | 17:47 | |
mgould | lucasagomes, excellent! | 17:48 |
lucasagomes | ipxe packaged* | 17:48 |
lucasagomes | package* urghh | 17:48 |
lucasagomes | there's an RFE somewhere in bugzilla, I gotta dig my email to find it | 17:48 |
mgould | I was especially amused because I have friends in the Haskell/dependent-types community who use unary arithmetic for type-level programming, and say "of course this is far too slow for production use" :-) | 17:48 |
lucasagomes | hah | 17:49 |
*** piet has quit IRC | 17:50 | |
mgould | well, mostly they say "production? what's that?", but the effect is much the same :-) | 17:50 |
*** admin0 has joined #openstack-ironic | 17:54 | |
*** praneshp has quit IRC | 17:55 | |
*** openstackgerrit has quit IRC | 18:02 | |
*** Marga_ has joined #openstack-ironic | 18:03 | |
*** openstackgerrit has joined #openstack-ironic | 18:03 | |
lucasagomes | folks if you guys have some time, mind taking a look at https://review.openstack.org/#/c/257497/ ? | 18:05 |
*** rvasilets___ has quit IRC | 18:05 | |
* mgould takes a look | 18:06 | |
jroll | lucasagomes: feels like that should be one commit, because the first will break without the second, right? | 18:07 |
lucasagomes | jroll, yeah it won't work w/o the template changes | 18:08 |
lucasagomes | jroll, I don't mind squashing it in one commit | 18:08 |
lucasagomes | I separated because it would be easier to review | 18:08 |
jroll | lucasagomes: that would be cool | 18:08 |
jroll | meh, still only a couple hundred lines | 18:08 |
lucasagomes | and added docs + release note in the last | 18:08 |
lucasagomes | jroll, ack... lemme squash it real quick | 18:09 |
jroll | thanks! | 18:09 |
*** rbudden has joined #openstack-ironic | 18:10 | |
*** piet has joined #openstack-ironic | 18:14 | |
*** links has quit IRC | 18:15 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add UEFI support for iPXE https://review.openstack.org/257497 | 18:16 |
lucasagomes | jroll, mgould ^ | 18:16 |
jroll | ty | 18:16 |
lucasagomes | np | 18:16 |
jroll | +2 | 18:22 |
*** rloo has joined #openstack-ironic | 18:23 | |
lucasagomes | jroll, thanks! | 18:27 |
lucasagomes | FYI that revert patch worked locally for me | 18:27 |
jroll | cool, so just intermittent gate things | 18:28 |
lucasagomes | so odd, why it didn't call lookup on that patch... I feel like recheck'ing it again | 18:28 |
lucasagomes | yeah | 18:28 |
jroll | well, it didn't finish building | 18:28 |
* lucasagomes trigger recheck | 18:28 | |
jroll | unless you mean the other one, in which case I think it hit the boot timeout | 18:28 |
lucasagomes | jroll, I was looking at the previous run | 18:28 |
lucasagomes | yeah the one that timed out | 18:29 |
jroll | yeah, so tempest has a few timeouts | 18:29 |
*** piet has quit IRC | 18:29 | |
*** baoli has quit IRC | 18:29 | |
jroll | one is deploy timeout, which is the time from 'nova boot' to active | 18:29 |
lucasagomes | if you look at the ir-cond, there's no lookup there, only power on and then you can see deleted being called for the node in "wait call-back" | 18:29 |
lucasagomes | yeah | 18:29 |
*** ChubYann has joined #openstack-ironic | 18:29 | |
lucasagomes | but the ramdisk booted if you look at the virt console | 18:29 |
lucasagomes | let's run it again to see | 18:29 |
jroll | OH | 18:30 |
jroll | I missed that, huh. | 18:30 |
jroll | maybe it depends on /run | 18:30 |
lucasagomes | yeah, that's where I saw that /run thing | 18:30 |
lucasagomes | yeah but it worked locally and has been working in gate too | 18:30 |
jroll | I can rebase on top of the fix for that | 18:30 |
lucasagomes | oddly enough | 18:30 |
jroll | or we can wait for one more run | 18:30 |
lucasagomes | jroll, yeah, I just triggered so... if you want to add that as a dependency | 18:31 |
lucasagomes | np, gate didn't pick the patch yet | 18:31 |
lucasagomes | maybe that's needed, because locally I tested with pxe_* drivers, and that is an agent_* check | 18:32 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: Revert "Use latest CoreOS stable when building" https://review.openstack.org/260235 | 18:32 |
jroll | done | 18:32 |
lucasagomes | so maybe when deploying via agent_* method it needs /run there | 18:32 |
* lucasagomes guess | 18:32 | |
jroll | I doubt it | 18:32 |
jroll | but /shrug | 18:32 |
lucasagomes | yeah | 18:32 |
* lucasagomes is not trying to mask some systemd services to speed up the coreos image boot | 18:32 | |
lucasagomes | is now* | 18:32 |
*** harlowja has quit IRC | 18:36 | |
*** harlowja has joined #openstack-ironic | 18:36 | |
*** e0ne has joined #openstack-ironic | 18:37 | |
*** piet has joined #openstack-ironic | 18:37 | |
*** piet has quit IRC | 18:41 | |
*** piet has joined #openstack-ironic | 18:51 | |
* mgould -> home; happy holidays, everybody! | 18:59 | |
mgould | see you all next year :-) | 18:59 |
*** mgould has quit IRC | 19:00 | |
*** intr1nsic has quit IRC | 19:01 | |
*** intr1nsic has joined #openstack-ironic | 19:02 | |
rloo | hey lucasagomes, were you going to push another revision for https://review.openstack.org/#/c/253605? cuz you say 'done' but I don't see any changes :) | 19:05 |
lucasagomes | rloo, yes, but I want to ask devananda first about jim's comment on performance of the value field | 19:06 |
rloo | lucasagomes: ok. no worries. | 19:06 |
devananda | lucasagomes: ohhai | 19:07 |
lucasagomes | devananda, hi there! | 19:07 |
lucasagomes | devananda, heh if you can take a look at https://review.openstack.org/#/c/253605 line number 67 | 19:08 |
*** piet has quit IRC | 19:08 | |
*** awiddersheim has joined #openstack-ironic | 19:08 | |
*** praneshp has joined #openstack-ironic | 19:09 | |
devananda | lucasagomes: your sample query is missing the FROM <table> clause :p | 19:09 |
lucasagomes | oh ++ | 19:10 |
* lucasagomes comment on the patch to not forget | 19:10 | |
devananda | lucasagomes: hmm, so if all `values` fields are JsonEncodedType, then we can not index them in mysql | 19:12 |
devananda | lucasagomes: whereas a prefix index on that field would facilitate faster searching | 19:12 |
devananda | eg, INDEX (property_id, value(15)) | 19:12 |
*** penick has joined #openstack-ironic | 19:12 | |
devananda | the first XX characters of the value will provide a reasonable amount of cardinality to the index without being overly large -- I'm only guessing at the number "15" | 19:13 |
lucasagomes | so we just index part of the value, that seems fair | 19:13 |
devananda | in production the cardinality of different prefix lengths would vary between deployments (based on the data, after all) | 19:14 |
devananda | but that doesn't make as much sense for a JSON field | 19:15 |
lucasagomes | right... not sure how we can workaround that tho | 19:15 |
lucasagomes | that json type is needed to keep the backward compatibility with the current API | 19:15 |
devananda | if the value is simply a string, what will the JSON repr look like? | 19:15 |
lucasagomes | that is a json type | 19:16 |
lucasagomes | devananda, a quoted string | 19:16 |
devananda | {'string value'} | 19:16 |
devananda | right? | 19:16 |
lucasagomes | nop, just ' " string " ' | 19:16 |
lucasagomes | (without the spaces) | 19:16 |
lucasagomes | In [2]: json.dumps("ssss") | 19:16 |
lucasagomes | Out[2]: '"ssss"' | 19:16 |
devananda | huh | 19:16 |
*** piet has joined #openstack-ironic | 19:17 | |
devananda | >>> x=u'foo "bar"' | 19:17 |
devananda | >>> json.dumps(x) | 19:17 |
devananda | '"foo \\"bar\\""' | 19:18 |
lucasagomes | yeah maybe using a native JSON fields is the way to go for our filter API, but not sure how we can work that around the ORM thing | 19:18 |
lucasagomes | using the* | 19:18 |
devananda | native JSON support is a recent addition to MySQL | 19:18 |
lucasagomes | yeah | 19:18 |
lucasagomes | and postgres supports it for a long time | 19:19 |
devananda | hmm | 19:19 |
devananda | might be an option | 19:19 |
lucasagomes | not sure how sqlalchemy actually deals with that, I think it does not... | 19:19 |
lucasagomes | devananda, yeah | 19:19 |
devananda | lucasagomes: https://dev.mysql.com/doc/refman/5.7/en/json.html | 19:20 |
devananda | we'd need to do some additional db work to index it | 19:20 |
* lucasagomes reads | 19:20 | |
devananda | https://dev.mysql.com/doc/refman/5.7/en/create-table.html#create-table-secondary-indexes-virtual-columns | 19:20 |
*** baoli has joined #openstack-ironic | 19:21 | |
* devananda reads too | 19:21 | |
lucasagomes | http://docs.sqlalchemy.org/en/latest/dialects/postgresql.html#json-types | 19:21 |
lucasagomes | but that's postgres only for sqlalchemy | 19:22 |
*** baoli_ has joined #openstack-ironic | 19:22 | |
*** admin0 has left #openstack-ironic | 19:22 | |
devananda | lemme poke at this json support in mysql 5.7 | 19:22 |
*** baoli has quit IRC | 19:26 | |
devananda | lucasagomes: it looks like mysql 5.7 isn't packaged for ubuntu yet at all | 19:27 |
lucasagomes | *sigh* | 19:27 |
lucasagomes | probably mariadb with JSON support is | 19:27 |
lucasagomes | devananda, but we can't rely on it right? Because of the ORM | 19:28 |
devananda | according to docs, mariadb supports json type as of 10.0.16, and ubuntu is packaging 10.0.22 | 19:29 |
devananda | i'm disinclined to pin ironic to require mariadb, and not support mysql, though | 19:29 |
lucasagomes | yeah | 19:30 |
devananda | (or mysql ahead of distro packaging) | 19:30 |
lucasagomes | it's a ball of wax :-/ | 19:30 |
* lucasagomes steps away for 5 min... | 19:30 | |
devananda | oooh -- correction, mariadb supports json _table_type_ in 10.0.16, not column type | 19:31 |
devananda | and only through the "CONNECT" engine, ie, not in InnoDB | 19:32 |
devananda | so that's not actually what we want | 19:32 |
*** piet has quit IRC | 19:33 | |
*** penick has quit IRC | 19:41 | |
*** dims has quit IRC | 19:45 | |
devananda | lucasagomes: left a few comments on the spec | 19:46 |
*** Marga_ has quit IRC | 19:51 | |
*** Marga_ has joined #openstack-ironic | 19:51 | |
lucasagomes | devananda, thanks a lot! | 19:51 |
lucasagomes | I will update that spec and matching patch next year | 19:54 |
lucasagomes | I'm calling it a day and I will be off for the next days due xmas and new year | 19:54 |
lucasagomes | so happy holidays all, enjoy it ! | 19:54 |
jroll | alright, thanks lucasagomes, see you next year :) | 19:54 |
jroll | enjoy the time off | 19:54 |
lucasagomes | jroll, thanks a lot! Enjoy it there too | 19:55 |
* jroll throws lucas a pint | 19:55 | |
lucasagomes | see you all next year | 19:55 |
* lucasagomes grabs and drink it! | 19:55 | |
lucasagomes | o/ | 19:55 |
rloo | happy holidays lucasagomes! | 19:55 |
*** lucasagomes is now known as lucas-pto | 19:55 | |
*** Marga_ has quit IRC | 19:55 | |
*** dims has joined #openstack-ironic | 20:04 | |
*** dims_ has joined #openstack-ironic | 20:11 | |
*** dims has quit IRC | 20:14 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Follow-up to PXE substrate support https://review.openstack.org/260201 | 20:14 |
*** 64MAALEJU has quit IRC | 20:15 | |
*** boris-42 has joined #openstack-ironic | 20:20 | |
*** baoli_ has quit IRC | 20:26 | |
*** e0ne has quit IRC | 20:28 | |
*** Marga_ has joined #openstack-ironic | 20:28 | |
*** e0ne has joined #openstack-ironic | 20:32 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Update CONTRIBUTING.rst to cover conditionals https://review.openstack.org/260705 | 20:33 |
*** baoli has joined #openstack-ironic | 20:49 | |
*** baoli has quit IRC | 20:54 | |
*** baoli has joined #openstack-ironic | 20:55 | |
*** Marga_ has quit IRC | 21:05 | |
*** Marga_ has joined #openstack-ironic | 21:06 | |
*** Marga_ has quit IRC | 21:15 | |
*** penick has joined #openstack-ironic | 21:19 | |
*** davidlenwell has quit IRC | 21:24 | |
*** penick has quit IRC | 21:26 | |
*** thingee has quit IRC | 21:27 | |
*** adam_g has quit IRC | 21:27 | |
*** baoli has quit IRC | 21:27 | |
*** trown is now known as trown|outttypeww | 21:28 | |
*** baoli has joined #openstack-ironic | 21:28 | |
*** mrda_vacation is now known as mrda_internet_br | 21:30 | |
*** mrda_internet_br is now known as mrda_no_internet | 21:30 | |
*** thingee has joined #openstack-ironic | 21:31 | |
*** adam_g has joined #openstack-ironic | 21:33 | |
*** adam_g has quit IRC | 21:34 | |
*** adam_g has joined #openstack-ironic | 21:34 | |
*** davidlenwell has joined #openstack-ironic | 21:36 | |
openstackgerrit | Ruby Loo proposed openstack/ironic-lib: Use imageutils from oslo.utils https://review.openstack.org/257248 | 21:36 |
*** e0ne has quit IRC | 21:40 | |
*** Marga_ has joined #openstack-ironic | 21:43 | |
*** davidlenwell has quit IRC | 21:43 | |
*** e0ne has joined #openstack-ironic | 21:45 | |
*** Marga_ has quit IRC | 21:48 | |
*** adam_g has quit IRC | 21:52 | |
*** tmckay has joined #openstack-ironic | 21:54 | |
*** davidlenwell has joined #openstack-ironic | 21:55 | |
tmckay | Hi ironic folks. So I'm trying to launch a Fedora 22 baremetal image. It gets to the final boot (both nova and ironic think the instance/node is "active") and it drops into the dracut shell | 21:55 |
tmckay | because none of the /dev/disk symlinks are there | 21:55 |
*** adam_g has joined #openstack-ironic | 21:56 | |
tmckay | which iiuc are created by udev | 21:56 |
*** adam_g has quit IRC | 21:56 | |
*** adam_g has joined #openstack-ironic | 21:56 | |
tmckay | so, does this sound familiar to anyone here? udev not working correctly with F22 images? | 21:56 |
* tmckay this is a little outside of my normal day-to-day | 21:56 | |
*** e0ne has quit IRC | 21:59 | |
*** e0ne has joined #openstack-ironic | 21:59 | |
*** piet has joined #openstack-ironic | 22:00 | |
*** davidlenwell has quit IRC | 22:02 | |
*** mbound_ has joined #openstack-ironic | 22:05 | |
*** alexpilotti has quit IRC | 22:06 | |
*** adam_g has quit IRC | 22:06 | |
*** tmckay has left #openstack-ironic | 22:07 | |
*** mbound_ has quit IRC | 22:07 | |
*** alexpilotti has joined #openstack-ironic | 22:10 | |
*** adam_g has joined #openstack-ironic | 22:10 | |
*** adam_g has quit IRC | 22:11 | |
*** adam_g has joined #openstack-ironic | 22:11 | |
*** dprince has quit IRC | 22:12 | |
*** davidlenwell has joined #openstack-ironic | 22:12 | |
*** piet has quit IRC | 22:14 | |
*** e0ne has quit IRC | 22:16 | |
*** david-lyle has joined #openstack-ironic | 22:17 | |
*** ndipanov has joined #openstack-ironic | 22:24 | |
openstackgerrit | Merged openstack/ironic-lib: Use imageutils from oslo.utils https://review.openstack.org/257248 | 22:26 |
*** thingee has quit IRC | 22:27 | |
*** thrash is now known as thrash|g0ne | 22:30 | |
*** thingee has joined #openstack-ironic | 22:35 | |
*** Marga_ has joined #openstack-ironic | 22:44 | |
*** Marga_ has quit IRC | 22:49 | |
*** bradjones has quit IRC | 23:10 | |
*** bradjones has joined #openstack-ironic | 23:13 | |
*** bradjones has quit IRC | 23:13 | |
*** bradjones has joined #openstack-ironic | 23:13 | |
*** dims has joined #openstack-ironic | 23:18 | |
*** dims_ has quit IRC | 23:19 | |
openstackgerrit | Merged openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/260264 | 23:19 |
*** harlowja has quit IRC | 23:25 | |
*** harlowja has joined #openstack-ironic | 23:25 | |
openstackgerrit | Merged openstack/python-ironicclient: Add reno for release notes management https://review.openstack.org/259380 | 23:32 |
*** Marga_ has joined #openstack-ironic | 23:45 | |
*** ChuckC has joined #openstack-ironic | 23:49 | |
*** Marga_ has quit IRC | 23:50 | |
*** alexpilotti has quit IRC | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!