*** betherly has joined #openstack-ironic | 00:02 | |
*** dsneddon has joined #openstack-ironic | 00:05 | |
*** yajushiSri has quit IRC | 00:07 | |
*** betherly has quit IRC | 00:07 | |
larsks | Am I reading this correctly? The pep8 is checking because the pep8 module isn't passing the pep8 check? http://logs.openstack.org/34/642734/3/check/openstack-tox-pep8/e128299/job-output.txt.gz#_2019-03-12_14_31_32_081814 | 00:21 |
---|---|---|
larsks | s/checking/failing/ | 00:22 |
*** betherly has joined #openstack-ironic | 00:23 | |
*** betherly has quit IRC | 00:28 | |
*** gyee has quit IRC | 00:30 | |
*** betherly has joined #openstack-ironic | 00:44 | |
*** betherly has quit IRC | 00:49 | |
*** whoami-rajat has joined #openstack-ironic | 00:51 | |
openstackgerrit | Ghanshyam Mann proposed openstack/ironic master: Migrate ironic-grenade-dsvm-multinode-multitenant job to Ubuntu Bionic https://review.openstack.org/642315 | 00:58 |
dustinc | larsks: seems to me like maybe it is failing due to the missing license header...not completely sure, though | 01:03 |
larsks | dustinc: oh, hah, I think you're right. I didn't even see that after all of those warnings. | 01:05 |
larsks | Thanks for looking :) | 01:05 |
*** betherly has joined #openstack-ironic | 01:05 | |
openstackgerrit | Merged openstack/ironic-python-agent master: Correct clean_build command according to Makefile https://review.openstack.org/642515 | 01:06 |
*** betherly has quit IRC | 01:10 | |
openstackgerrit | QianBiao Ng proposed openstack/ironic master: Add Huawei iBMC driver support https://review.openstack.org/639288 | 01:21 |
*** betherly has joined #openstack-ironic | 01:26 | |
*** betherly has quit IRC | 01:31 | |
openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Follow up to available node protection https://review.openstack.org/642935 | 01:51 |
openstackgerrit | Lars Kellogg-Stedman proposed openstack/ironic master: prevent serial console lockout https://review.openstack.org/642734 | 02:04 |
*** betherly has joined #openstack-ironic | 02:08 | |
*** betherly has quit IRC | 02:12 | |
*** betherly has joined #openstack-ironic | 02:29 | |
*** mcarden has joined #openstack-ironic | 02:30 | |
*** betherly has quit IRC | 02:34 | |
*** betherly has joined #openstack-ironic | 02:50 | |
*** betherly has quit IRC | 02:54 | |
*** Qianbiao2 has joined #openstack-ironic | 03:08 | |
*** betherly has joined #openstack-ironic | 03:11 | |
*** betherly has quit IRC | 03:15 | |
*** w14161_1 has quit IRC | 03:24 | |
*** MattMan_ has quit IRC | 03:24 | |
*** MattMan_ has joined #openstack-ironic | 03:25 | |
*** w14161_1 has joined #openstack-ironic | 03:25 | |
*** betherly has joined #openstack-ironic | 03:31 | |
*** betherly has quit IRC | 03:36 | |
*** dsneddon has quit IRC | 03:43 | |
*** stendulker has joined #openstack-ironic | 03:45 | |
*** dsneddon has joined #openstack-ironic | 04:08 | |
*** dsneddon has quit IRC | 04:12 | |
*** rh-jelabarre has quit IRC | 04:29 | |
*** andrein has quit IRC | 04:35 | |
openstackgerrit | Merged openstack/ironic master: Fix missing print format error https://review.openstack.org/642346 | 04:39 |
openstackgerrit | Merged openstack/ironic master: Document building configdrive on the server side https://review.openstack.org/642441 | 04:39 |
*** dsneddon has joined #openstack-ironic | 04:40 | |
openstackgerrit | Merged openstack/ironic master: Fix formatting issue in doc https://review.openstack.org/642719 | 04:43 |
*** betherly has joined #openstack-ironic | 04:44 | |
*** dsneddon has quit IRC | 04:45 | |
*** betherly has quit IRC | 04:49 | |
*** betherly has joined #openstack-ironic | 05:05 | |
*** betherly has quit IRC | 05:09 | |
*** dsneddon has joined #openstack-ironic | 05:11 | |
*** dsneddon has quit IRC | 05:15 | |
*** dsneddon has joined #openstack-ironic | 05:26 | |
*** dsneddon has quit IRC | 05:31 | |
*** hwoarang has quit IRC | 05:53 | |
*** betherly has joined #openstack-ironic | 05:57 | |
*** hwoarang has joined #openstack-ironic | 06:00 | |
*** betherly has quit IRC | 06:01 | |
*** dsneddon has joined #openstack-ironic | 06:05 | |
*** dsneddon has quit IRC | 06:10 | |
*** verma-varsha has joined #openstack-ironic | 06:18 | |
*** pcaruana has joined #openstack-ironic | 06:19 | |
*** dsneddon has joined #openstack-ironic | 06:20 | |
*** dsneddon has quit IRC | 06:25 | |
*** yolanda has quit IRC | 06:36 | |
*** stendulker has quit IRC | 06:36 | |
*** verma-varsha has quit IRC | 06:38 | |
*** e0ne has joined #openstack-ironic | 06:40 | |
*** diga has joined #openstack-ironic | 06:58 | |
*** dsneddon has joined #openstack-ironic | 06:58 | |
*** hwoarang has quit IRC | 07:07 | |
*** hwoarang has joined #openstack-ironic | 07:08 | |
*** yolanda has joined #openstack-ironic | 07:09 | |
*** diga has quit IRC | 07:09 | |
*** dsneddon has quit IRC | 07:11 | |
*** monica_pardhi has joined #openstack-ironic | 07:12 | |
*** diga has joined #openstack-ironic | 07:13 | |
*** rachit7 has joined #openstack-ironic | 07:16 | |
*** tonyb is now known as tonyb_gone | 07:20 | |
*** tonyb_gone is now known as tonyb | 07:21 | |
diga | monica_pardhi: rachit7: Welcome to Ironic group :) | 07:21 |
*** PradipK has joined #openstack-ironic | 07:21 | |
rachit7 | Thank you @diga | 07:21 |
diga | PradipK: welcome to group:) | 07:21 |
PradipK | Thanks :) | 07:22 |
arne_wiebalck | good morning, ironic! | 07:27 |
*** e0ne has quit IRC | 07:31 | |
*** dsneddon has joined #openstack-ironic | 07:45 | |
*** gkadam has joined #openstack-ironic | 07:48 | |
*** dsneddon has quit IRC | 07:50 | |
*** e0ne has joined #openstack-ironic | 07:51 | |
*** monica_pardhi has quit IRC | 07:51 | |
*** dsneddon has joined #openstack-ironic | 07:54 | |
*** tssurya has joined #openstack-ironic | 08:02 | |
*** PradipK has quit IRC | 08:05 | |
*** lennyb has joined #openstack-ironic | 08:05 | |
*** rachit7 has quit IRC | 08:07 | |
rpittau|afk | good morning ironic! o/ | 08:09 |
*** rcernin has quit IRC | 08:09 | |
*** rpittau|afk is now known as rpittau | 08:09 | |
rpittau | happy hump day :) | 08:09 |
*** diga has quit IRC | 08:18 | |
*** e0ne has quit IRC | 08:36 | |
*** hwoarang has quit IRC | 08:40 | |
*** hwoarang has joined #openstack-ironic | 08:42 | |
*** iurygregory has joined #openstack-ironic | 08:46 | |
iurygregory | good morning o/ | 08:46 |
*** andrein has joined #openstack-ironic | 09:03 | |
*** sburette has joined #openstack-ironic | 09:03 | |
*** dtantsur|afk is now known as dtantsur | 09:14 | |
dtantsur | morning ironic | 09:14 |
jiapei | Morning dtantsur | 09:15 |
jiapei | For XClarity CI, we finally managed to add the deploy test in the CI jobs, a log is at http://lxca-logs.electronicjungle.net/02/642402/5/check/lenovo-xclarity-ironic/0c8938c/ | 09:17 |
iurygregory | morning dtantsur o/ | 09:18 |
dtantsur | jiapei: could you correct the collected files? there are no service logs and http://electronicjungle.net:8082/lxca_ironic_logs/02/642402/5/check/lenovo-xclarity-ironic/0c8938c/logs/job-output.txt doesn't look useful | 09:19 |
rpittau | hi dtantsur :) | 09:19 |
rpittau | hey iurygregory :) | 09:19 |
iurygregory | rpittau, o/ | 09:20 |
jiapei | dtantsur: the log is here http://electronicjungle.net:8082/lxca_ironic_logs/02/642402/5/check/lenovo-xclarity-ironic/0c8938c/logs/job-output.json | 09:22 |
jiapei | wondering why the job-output.txt has lots of "waiting on logger"... | 09:22 |
*** FlorianFa has joined #openstack-ironic | 09:23 | |
*** hwoarang has quit IRC | 09:23 | |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Add docs on UEFI boot libvirt-based emulation https://review.openstack.org/642830 | 09:27 |
iurygregory | rpittau, I think I've found the issue for the env .-. | 09:28 |
iurygregory | cli_dir = os.path.join(os.path.abspath('.'), '.tox/functional/bin') | 09:28 |
rpittau | iurygregory, oh great! | 09:30 |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Rename `drivers` to `systems` https://review.openstack.org/638492 | 09:30 |
rpittau | where was that ? tempest-ironic ? | 09:30 |
*** hwoarang has joined #openstack-ironic | 09:32 | |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Add Managers resource support https://review.openstack.org/638682 | 09:32 |
dtantsur | jiapei: please look into fixing it, reading enormous JSON is pretty inconvenient :) | 09:32 |
iurygregory | rpittau, nope in base.py ironicclient/tests/functional | 09:33 |
* iurygregory not sure if tox allows in the cmd a way to set the python version | 09:33 | |
*** derekh has joined #openstack-ironic | 09:35 | |
*** diga has joined #openstack-ironic | 09:36 | |
jiapei | dtantsur: OK, we'll look into the txt issue :) | 09:37 |
iurygregory | dtantsur, we are having an issue to set a new env to run the functional tests under python3 (I've created a new env in tox.ini) but since we have a hardcode path to run the functional tests it will fail https://github.com/openstack/python-ironicclient/blob/master/ironicclient/tests/functional/base.py#L40 | 09:42 |
iurygregory | any problem if we switch for the approach that glance did: http://git.openstack.org/cgit/openstack/python-glanceclient/tree/glanceclient/tests/functional/base.py#n51 | 09:43 |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Add read-only Virtual Media support https://review.openstack.org/638787 | 09:50 |
openstackgerrit | Mark Goddard proposed openstack/ironic master: Return 405 for old versions in allocation and deploy template APIs https://review.openstack.org/642762 | 09:51 |
*** Emine has joined #openstack-ironic | 09:53 | |
dtantsur | larsks: hi, you around? | 09:54 |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Add writable Virtual Media resource mocks https://review.openstack.org/639338 | 09:54 |
dtantsur | iurygregory: no problem at all | 09:54 |
iurygregory | dtantsur, ack | 09:54 |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Add Managers<->Systems linkage in sushy-emulator https://review.openstack.org/640304 | 09:58 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: honor ipmi_port in serial console drivers https://review.openstack.org/640930 | 10:00 |
dtantsur | larsks: added a release note ^^ | 10:00 |
openstackgerrit | Mark Goddard proposed openstack/ironic master: Return 405 for old versions in allocation and deploy template APIs https://review.openstack.org/642762 | 10:05 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/python-ironicclient master: Run jobs under python2 and python3 https://review.openstack.org/642474 | 10:09 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Adding ansible python interpreter as driver_info https://review.openstack.org/642402 | 10:15 |
mgoddard | dtantsur: "I've seen _route in the node controller returning "None" in some case. Will it have the same effect?" | 10:24 |
mgoddard | dtantsur: I can't see any examples of it returning None | 10:24 |
rpittau | TheJulia, dtantsur, wdyt about switching postgres test to ipxe ? | 10:28 |
*** hwoarang has quit IRC | 10:28 | |
*** hwoarang has joined #openstack-ironic | 10:29 | |
rpittau | it's kind of weird what's happening with that test, the virtual node tries to load ipxe anyway even if it shouldn't | 10:30 |
rpittau | actually if anyone has any idea on why this is happening please don't restrain yourself :) | 10:31 |
dtantsur | mgoddard: ah, it was lookup https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/node.py#L1607 | 10:37 |
mgoddard | dtantsur: yeah, just replied | 10:37 |
dtantsur | rpittau: we need some coverage for pxe (much more than for postgres) | 10:37 |
dtantsur | rpittau: maybe we forgot to disable the "ipxe" boot interface or switch the default to "pxe"? | 10:37 |
dtantsur | previously it was a configuration option "ipxe_enabled" | 10:37 |
rpittau | dtantsur, the option ipxe_enabled is set to False :/ | 10:38 |
dtantsur | mgoddard: yeah, maybe. I don't care either way | 10:38 |
dtantsur | rpittau: right, but it does not matter much if the "ipxe" boot interface is used | 10:38 |
dtantsur | it only affects the "pxe" boot interface | 10:38 |
rpittau | dtantsur, the boot_interfaces enabled are fake and pxe | 10:39 |
dtantsur | hmm | 10:39 |
dtantsur | rpittau: do you have an example job link? | 10:39 |
rpittau | I reproduces it on a devstack machine and I' mvery very very puzzled | 10:39 |
rpittau | dtantsur, https://review.openstack.org/642430 | 10:40 |
patchbot | patch 642430 - ironic - [DNM] testing postgres job - 5 patch sets | 10:40 |
dtantsur | rpittau: can you check that the node actually ended up with boot_interface=pxe? | 10:40 |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Add libvirt-based virtual media emulation https://review.openstack.org/640452 | 10:42 |
dtantsur | the CI job looks correct | 10:43 |
etingof | just 10 patches and VM has been booted through virtual media, first time ever in my life ^ | 10:44 |
*** dsneddon has quit IRC | 10:44 | |
dtantsur | \o/ | 10:44 |
dtantsur | rpittau: I can only think about something firewall-related, tbh | 10:44 |
dtantsur | DHCP options in Neutron look about right | 10:45 |
*** priteau has joined #openstack-ironic | 10:45 | |
rpittau | dtantsur, ok, thanks, I'll dig a little bit more | 10:47 |
*** w14161_1 has quit IRC | 10:51 | |
openstackgerrit | Merged openstack/ironic master: [docs] IPv6 support for iLO https://review.openstack.org/638590 | 10:55 |
*** w14161_1 has joined #openstack-ironic | 10:57 | |
openstackgerrit | Varsha Verma proposed openstack/ironic master: Deprecates `hash_distribution_replicas` config option https://review.openstack.org/643002 | 10:57 |
*** diga has quit IRC | 11:03 | |
*** dsneddon has joined #openstack-ironic | 11:16 | |
*** dsneddon has quit IRC | 11:21 | |
Qianbiao2 | Hello dtantsur | 11:22 |
Qianbiao2 | Is our CI log ok like this http://112.93.129.99/builds/Post-build-task-test/5/ | 11:23 |
*** gkadam_ has joined #openstack-ironic | 11:24 | |
*** gkadam has quit IRC | 11:25 | |
dtantsur | Qianbiao2: hi! it's better, but can we also see the overall job output, including devstack and tempest? | 11:26 |
Qianbiao2 | we did not use tempest ;( | 11:26 |
Qianbiao2 | I try tempest, it's impossible to download everything thought network in our VM | 11:26 |
Qianbiao2 | Now, i am using a standalone ENV with everything ready. | 11:27 |
Qianbiao2 | and everytime, git download patch, then reinstall ironic. | 11:27 |
*** verma-varsha has joined #openstack-ironic | 11:27 | |
Qianbiao2 | and deploy baremetal using an ansible playbook | 11:27 |
Qianbiao2 | before CI run, will cleanup logs. and after CI build, will copy logs to a nginx server root. | 11:28 |
Qianbiao2 | logs will contains: jenkins build log, ironic-api log, ironic conductor log, dnsmasq log when CI is ready. | 11:29 |
dtantsur | Qianbiao2: fine with me, can we have the output of that playbook? | 11:31 |
dtantsur | right | 11:31 |
* dtantsur should read the whole message before responding | 11:31 | |
Qianbiao2 | sure, it will contains in jenkins build log later | 11:32 |
Qianbiao2 | we are using playbook https://github.com/IamFive/ansible-ironic-standalone | 11:33 |
*** yaawang has quit IRC | 11:33 | |
*** e0ne has joined #openstack-ironic | 11:36 | |
openstackgerrit | Mark Goddard proposed openstack/ironic master: WIP: Add iDRAC RAID deploy steps https://review.openstack.org/641731 | 11:38 |
openstackgerrit | Mark Goddard proposed openstack/ironic master: Allow methods to be both deploy and clean steps https://review.openstack.org/643012 | 11:39 |
*** yaawang has joined #openstack-ironic | 11:42 | |
*** verma-varsha has quit IRC | 11:46 | |
*** dsneddon has joined #openstack-ironic | 11:50 | |
*** rh-jelabarre has joined #openstack-ironic | 11:53 | |
*** rh-jelabarre has quit IRC | 11:53 | |
*** rh-jelabarre has joined #openstack-ironic | 11:53 | |
*** dsneddon has quit IRC | 11:55 | |
*** hwoarang has quit IRC | 11:56 | |
*** hwoarang has joined #openstack-ironic | 11:58 | |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Add docs on libvirt virtual media emulation https://review.openstack.org/643027 | 12:04 |
openstackgerrit | Varsha Verma proposed openstack/ironic master: Deprecates `hash_distribution_replicas` config option https://review.openstack.org/643002 | 12:10 |
*** dsneddon has joined #openstack-ironic | 12:23 | |
*** hwoarang has quit IRC | 12:24 | |
*** hwoarang has joined #openstack-ironic | 12:28 | |
*** dsneddon has quit IRC | 12:29 | |
*** dsneddon has joined #openstack-ironic | 12:55 | |
iurygregory | Anyone have ideas about http://logs.openstack.org/74/642474/10/check/ironicclient-functional-python2/8fa7cc0/job-output.txt.gz#_2019-03-13_10_35_43_467692 http://logs.openstack.org/74/642474/10/check/ironicclient-functional/3a3bc51/job-output.txt.gz#_2019-03-13_10_37_56_268334 | 12:56 |
dtantsur | iurygregory: I think the message has changed on the ironic side | 12:58 |
iurygregory | dtantsur, so we need to updtade the tests only? | 12:58 |
dtantsur | yep | 12:58 |
iurygregory | ack | 12:59 |
iurygregory | going to look at the patchs in ironic and update the client | 12:59 |
iurygregory | or someone have a patch already? | 12:59 |
* dtantsur does not | 12:59 | |
*** diga has joined #openstack-ironic | 12:59 | |
*** dsneddon has quit IRC | 13:00 | |
*** dsneddon has joined #openstack-ironic | 13:01 | |
*** e0ne has quit IRC | 13:05 | |
openstackgerrit | Merged openstack/ironic master: Follow up to available node protection https://review.openstack.org/642935 | 13:07 |
openstackgerrit | Merged openstack/ironic master: Fixing a bash test in devstack ironic lib https://review.openstack.org/642809 | 13:07 |
openstackgerrit | Merged openstack/ironic-inspector master: Revise api-ref docs https://review.openstack.org/613515 | 13:07 |
*** dsneddon has quit IRC | 13:10 | |
*** mbeierl has joined #openstack-ironic | 13:19 | |
*** mbeierl has quit IRC | 13:23 | |
*** priteau has quit IRC | 13:27 | |
*** pandy has joined #openstack-ironic | 13:32 | |
*** mjturek has joined #openstack-ironic | 13:33 | |
TheJulia | good morning | 13:35 |
dtantsur | morning TheJulia | 13:36 |
TheJulia | rpittau: I'm fine with switching postgres, but all nodes basically try to ipxe in our virutal environments because of hte network card roms that get loaded... except uefi machines | 13:37 |
*** mbeierl has joined #openstack-ironic | 13:37 | |
rpittau | TheJulia, hi, yeah, I did some more digging to understand how that worked and I see that it's a remap to tftp | 13:39 |
rpittau | although it doesn't seem to work | 13:40 |
rpittau | probably something on the internal routing | 13:41 |
openstackgerrit | Rachit Kapadia proposed openstack/ironic master: Set boot_mode in node properties during OOB Introspection https://review.openstack.org/639698 | 13:42 |
*** dsneddon has joined #openstack-ironic | 13:42 | |
*** bfournie has joined #openstack-ironic | 13:44 | |
*** dsneddon has quit IRC | 13:47 | |
TheJulia | the 405 error patch has been approved | 13:47 |
openstackgerrit | Merged openstack/ironic master: honor ipmi_port in serial console drivers https://review.openstack.org/640930 | 13:48 |
*** dsneddon has joined #openstack-ironic | 13:55 | |
*** baha has joined #openstack-ironic | 13:56 | |
*** Emine has quit IRC | 13:59 | |
*** hwoarang has quit IRC | 14:01 | |
*** dsneddon has quit IRC | 14:02 | |
*** gkadam_ has quit IRC | 14:02 | |
*** dsneddon has joined #openstack-ironic | 14:04 | |
*** hwoarang has joined #openstack-ironic | 14:08 | |
*** Emine has joined #openstack-ironic | 14:08 | |
*** dsneddon has quit IRC | 14:09 | |
*** irclogbot_1 has quit IRC | 14:09 | |
*** e0ne has joined #openstack-ironic | 14:10 | |
*** irclogbot_1 has joined #openstack-ironic | 14:12 | |
*** Qianbiao2 has quit IRC | 14:12 | |
*** hwoarang has quit IRC | 14:17 | |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Add docs on libvirt virtual media emulation https://review.openstack.org/643027 | 14:18 |
*** hwoarang has joined #openstack-ironic | 14:19 | |
*** bnemec has quit IRC | 14:22 | |
*** Emine has quit IRC | 14:24 | |
*** Emine has joined #openstack-ironic | 14:24 | |
*** irclogbot_1 has quit IRC | 14:25 | |
*** bnemec has joined #openstack-ironic | 14:27 | |
*** irclogbot_1 has joined #openstack-ironic | 14:27 | |
diga | dtantsur: TheJulia: Hi | 14:29 |
diga | dtantsur: CI is up today ? | 14:29 |
diga | https://review.openstack.org/#/c/641683/ | 14:29 |
patchbot | patch 641683 - ironic (stable/ocata) - Fix OOB introspection to use pxe_enabled flag in i... - 4 patch sets | 14:29 |
dtantsur | diga: well, that problem has just been solved. I'm not sure how many problems you'll see in ocata though | 14:29 |
*** dougsz has joined #openstack-ironic | 14:29 | |
diga | dtantsur: okay | 14:29 |
diga | dtantsur: let me recheck | 14:30 |
openstackgerrit | Digambar proposed openstack/ironic stable/ocata: Fix OOB introspection to use pxe_enabled flag in idrac driver https://review.openstack.org/641683 | 14:31 |
*** jtomasek has quit IRC | 14:38 | |
*** dsneddon has joined #openstack-ironic | 14:39 | |
*** dsneddon has quit IRC | 14:45 | |
*** baha has quit IRC | 14:47 | |
*** dsneddon has joined #openstack-ironic | 14:54 | |
*** baha has joined #openstack-ironic | 15:02 | |
arne_wiebalck | dtantsur: For the spec update we discussed during the weekly meeting (i.e. to be more specific about the target_raid_layout for software RAIDs) I submit a patch for the approved spec? | 15:03 |
dtantsur | arne_wiebalck: yep | 15:04 |
arne_wiebalck | cool, thx | 15:04 |
* arne_wiebalck was wondering if that would look like sneaking in changes after the spec was approved :) | 15:04 | |
dtantsur | arne_wiebalck: we've been doing it pretty regularly :) they're still reviewed after all | 15:11 |
arne_wiebalck | dtantsur: sounds good to me, just wanted to make sure this is indeed the process; in the end, it is not changing anything, it is just being more specific on some parts left open in the current version | 15:12 |
dtantsur | yes, it's fine. I recall one of my specs changing 3 or 4 times :) | 15:12 |
arne_wiebalck | dtantsur: "when specs meet reality" :-D | 15:13 |
dtantsur | I prefer to think we're agile ;) | 15:14 |
arne_wiebalck | lol | 15:14 |
*** andrein has quit IRC | 15:30 | |
*** hwoarang has quit IRC | 15:32 | |
*** hwoarang has joined #openstack-ironic | 15:33 | |
*** diga has quit IRC | 15:35 | |
*** irclogbot_1 has quit IRC | 15:36 | |
*** irclogbot_1 has joined #openstack-ironic | 15:39 | |
*** andrein has joined #openstack-ironic | 15:46 | |
*** irclogbot_1 has quit IRC | 15:49 | |
TheJulia | "When specs meet reality" is a good way of putting it | 15:50 |
*** irclogbot_1 has joined #openstack-ironic | 15:51 | |
*** irclogbot_1 has quit IRC | 15:52 | |
*** irclogbot_1 has joined #openstack-ironic | 15:56 | |
NobodyCam | Good Morning Ironic'ers | 15:56 |
iurygregory | morning NobodyCam | 15:56 |
NobodyCam | Good Morning iurygregory :) \o | 15:57 |
dtantsur | morning NobodyCam | 15:58 |
iurygregory | =) | 15:58 |
NobodyCam | morning dtantsur :) happy hump day | 16:01 |
rpittau | hi NobodyCam :) | 16:03 |
NobodyCam | hey hey rpittau :) \o | 16:03 |
*** dsneddon has quit IRC | 16:04 | |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Add libvirt-based virtual media emulation https://review.openstack.org/640452 | 16:09 |
*** diga has joined #openstack-ironic | 16:09 | |
diga | dtantsur: Hi | 16:09 |
dtantsur | hi diga | 16:10 |
diga | dtantsur: Cherry-pick patch is passing all the zull check :) - /msg nickserv identify Diga0915 | 16:10 |
diga | sorry | 16:10 |
diga | https://review.openstack.org/#/c/641683/ | 16:10 |
patchbot | patch 641683 - ironic (stable/ocata) - Fix OOB introspection to use pxe_enabled flag in i... - 5 patch sets | 16:10 |
dtantsur | diga: you may have just exposed your password :) | 16:10 |
diga | dtantsur: yes, by mistake | 16:10 |
diga | Need to change now :) | 16:10 |
dtantsur | diga: better change it, this channel is logged | 16:10 |
diga | yes | 16:10 |
dtantsur | diga: ocata has python-dracclient>=0.1.0 in its driver-requirements. can you confirm this patch works with dracclient==0.1.0 | 16:12 |
*** diga has quit IRC | 16:15 | |
*** jtomasek has joined #openstack-ironic | 16:16 | |
JayF | TheJulia: arne_wiebalck: That sounds like a title to a summit presentation if I've ever heard one :D | 16:16 |
*** diga has joined #openstack-ironic | 16:18 | |
*** trown is now known as trown|lunch | 16:18 | |
diga | dtantsur: Finally reset the password :) | 16:18 |
diga | dtantsur: Please review this patch - https://review.openstack.org/#/c/641683/ | 16:18 |
patchbot | patch 641683 - ironic (stable/ocata) - Fix OOB introspection to use pxe_enabled flag in i... - 5 patch sets | 16:18 |
*** dsneddon has joined #openstack-ironic | 16:19 | |
*** Emine has quit IRC | 16:20 | |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Render System->Managers linkage https://review.openstack.org/643094 | 16:20 |
dtantsur | diga: in case you missed my question: ocata has python-dracclient>=0.1.0 in its driver-requirements. can you confirm this patch works with dracclient==0.1.0 | 16:22 |
*** dsneddon has quit IRC | 16:23 | |
*** priteau has joined #openstack-ironic | 16:24 | |
diga | Let me check | 16:24 |
diga | one min | 16:24 |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Add docs on libvirt virtual media emulation https://review.openstack.org/643027 | 16:24 |
*** irclogbot_1 has quit IRC | 16:24 | |
mgoddard | TheJulia: dtantsur: https://review.openstack.org/#/c/643012 is stacked on top of two other patches. Should I rebase it onto master, as it has +A, or wait for reviews of the other two? | 16:25 |
patchbot | patch 643012 - ironic - Allow methods to be both deploy and clean steps - 1 patch set | 16:25 |
*** irclogbot_1 has joined #openstack-ironic | 16:25 | |
*** gyee has joined #openstack-ironic | 16:25 | |
dtantsur | mgoddard: I'd rebase if it does not really need the previous patches | 16:26 |
diga | dtantsur: This patch needs python-dracclient>=1.5.0 version | 16:26 |
mgoddard | dtantsur: ack | 16:26 |
dtantsur | diga: then we cannot backport it. | 16:27 |
dtantsur | well, at least we need to make sure it doesn't break completely with 0.1.0 | 16:28 |
*** dsneddon has joined #openstack-ironic | 16:28 | |
diga | dtantsur: okay | 16:28 |
diga | rpioso: you there ? | 16:29 |
openstackgerrit | Mark Goddard proposed openstack/ironic master: Allow methods to be both deploy and clean steps https://review.openstack.org/643012 | 16:29 |
dtantsur | diga: it's bad enough that we backported it to Pike, which has >=1.3.0... | 16:30 |
diga | dtantsur: yes, we need version 1.5.0 for this patch | 16:30 |
diga | dtantsur: should I abandon this one ? | 16:31 |
dtantsur | diga: I guess so | 16:31 |
diga | okay | 16:31 |
*** dsneddon has quit IRC | 16:34 | |
TheJulia | JayF: You raise a very good point! Get a China business visa and present it in the fall? | 16:34 |
*** dsneddon has joined #openstack-ironic | 16:35 | |
dtantsur | ++ | 16:36 |
TheJulia | It occurs to me that that whole thing with visas is going to push all of the summit processes up... | 16:36 |
diga | dtantsur: Thanks for your quick review :) | 16:36 |
diga | Need to sign off for the day! | 16:36 |
dtantsur | TheJulia: welcome to my world, I have cookies :) | 16:36 |
TheJulia | It was a huge pain to get one, at least it is multiple entry for me | 16:36 |
TheJulia | Mmmmm! Cookies! | 16:36 |
* dtantsur has recently experienced the doubtful pleasure of a face-to-face interview in the US embassy in Frankfurt | 16:37 | |
iurygregory | China Visa requires that you have some vaccines if i do remember | 16:37 |
TheJulia | I did not have to submit medical records, but requirements vary by country of citizenship and origin | 16:37 |
iurygregory | yeah | 16:40 |
iurygregory | since Brazil is in south america i need yellow fever D: | 16:40 |
dtantsur | wow | 16:40 |
TheJulia | US State department recommends travelers to china have influenza, typhoid, measles, Hepatitis A and B, and tuberculosis vaccines | 16:40 |
*** diga has quit IRC | 16:40 | |
dtantsur | I don't see it in the requirements list. But it may be a good idea. | 16:40 |
iurygregory | but like we dont have this for many years ... | 16:40 |
iurygregory | well in the middle of the year time for vaccines yay | 16:41 |
*** FlorianFa has quit IRC | 16:41 | |
TheJulia | It reminds me, I need to get another TDAP soon... | 16:42 |
openstackgerrit | Arne Wiebalck proposed openstack/ironic-specs master: Add details on target_raid_config for Software RAID support https://review.openstack.org/643100 | 16:43 |
dtantsur | I don't even remember which ones I have.. | 16:43 |
iurygregory | TDAP? .-. | 16:43 |
TheJulia | There are some immunity tests that can be run by doctors. Like I have insane immunity to Hep A and B now because they can actually measure it thanks to the vaccine | 16:43 |
arne_wiebalck | JayF: TheJulia: ^^ "spec met reality" :) | 16:43 |
iurygregory | oh gotcha =) | 16:44 |
etingof | that reminds me an awesome book on all these vaccines and their discoveries -- https://www.amazon.com/Microbe-Hunters-Paul-Kruif/dp/B002TO3ZUW | 16:44 |
TheJulia | iurygregory: TDAP is Tetanus, Diptheria, Pertussis. In the states it is one of those "every 10 years or as needed vaccines" | 16:45 |
iurygregory | woa =O | 16:45 |
iurygregory | at least is every 10 yrs | 16:45 |
iurygregory | XD | 16:45 |
arne_wiebalck | JayF: Incidentally, this is also meant to address the concern you raised a while ago about "accidental" s/w RAID creation. | 16:45 |
* arne_wiebalck just learnt the other day at the doctor it is now 20 years | 16:45 | |
TheJulia | arne_wiebalck: interesting! | 16:46 |
JayF | arne_wiebalck: I barely remember that, but I'm glad my loud voice continues to echo through the halls of openstack :P | 16:46 |
arne_wiebalck | seems they found out vaccinations last much longer than expected | 16:46 |
dustinc | phew because I was trying to think of th elast time I got TDAP and couldn't remember | 16:46 |
TheJulia | dustinc: likely a good idea to maybe... get another? | 16:46 |
* TheJulia ducks | 16:46 | |
TheJulia | If you can't remember.... ;) | 16:46 |
JayF | I use a dual pronged approach to avoiding tetanus: vaccinations and never leaving my house ;) | 16:47 |
dtantsur | lol | 16:47 |
TheJulia | heh | 16:47 |
dtantsur | TheJulia: have they announced the city officially? I keep forgetting. | 16:48 |
TheJulia | dtantsur: They did on twitter I think and I kind of got a little upset that they were already starting to advertise | 16:49 |
dtantsur | TheJulia: so, which one is it? :) | 16:49 |
* TheJulia looks since she doesn't want to miss-speak | 16:49 | |
TheJulia | Website says Shanghai | 16:49 |
TheJulia | Beijing would have essentially resulted in us being in the China National Convention Center, which is a nice building, but when I was there for Open Infra days, there was 2-3 other conferences occurring in the same building and construction on some levels for a conference the following week | 16:52 |
dtantsur | heh | 16:52 |
TheJulia | One was an AWS conference too... | 16:53 |
TheJulia | #awkward | 16:53 |
*** FlorianFa has joined #openstack-ironic | 16:54 | |
*** e0ne has quit IRC | 17:01 | |
dtantsur | indeed | 17:03 |
arne_wiebalck | bye, everyone o/ | 17:05 |
*** sburette has quit IRC | 17:05 | |
*** trown|lunch is now known as trown | 17:07 | |
mgoddard | rpioso: hi | 17:15 |
*** andrein has quit IRC | 17:15 | |
dtantsur | \o | 17:15 |
*** dtantsur is now known as dtantsur|afk | 17:15 | |
rpittau | good night! o/ | 17:16 |
*** rpittau is now known as rpittau|afk | 17:16 | |
*** gyee has quit IRC | 17:20 | |
*** gyee has joined #openstack-ironic | 17:20 | |
*** tssurya has quit IRC | 17:26 | |
*** pandy has quit IRC | 17:48 | |
*** ianychoi has quit IRC | 17:55 | |
*** dougsz has quit IRC | 17:58 | |
*** derekh has quit IRC | 18:00 | |
rpioso | mgoddard: Hello | 18:05 |
* rpioso has been busy downstream | 18:06 | |
gyee | TheJulia, I ran into this error in stable/rocky when using ipxe, 'No such file or directory: u'/tftpboot/720fd149-a142-4503-9bac-09926b828beb/config' and the code doesn't seem to make sense at the first glance | 18:16 |
gyee | the error occurred here https://github.com/openstack/ironic/blob/stable/rocky/ironic/drivers/modules/pxe.py#L291 because pxe_config_path doesn't exist | 18:17 |
gyee | but it should've been created here https://github.com/openstack/ironic/blob/stable/rocky/ironic/drivers/modules/pxe.py#L289 when provision state is active | 18:17 |
*** priteau has quit IRC | 18:18 | |
gyee | not sure if this is a known bug as pxe.py look very different between rocky and master | 18:19 |
*** pcaruana has quit IRC | 18:29 | |
*** Emine has joined #openstack-ironic | 18:33 | |
gyee | log seem to indicate that provision state was set to 'deploying' instead of 'active' so that file was never created in the first place | 18:34 |
*** priteau has joined #openstack-ironic | 18:40 | |
*** priteau has quit IRC | 18:44 | |
*** jtomasek has quit IRC | 18:48 | |
*** gmann is now known as gmann_afk | 18:48 | |
TheJulia | gyee: it will create the subfolder, but not the /tftpboot folder. That must exist and ironic must have privilges to write to it | 18:56 |
TheJulia | gyee: major differences between rocky and master is I retooled some of the locations so we could break ipxe into its own interface | 18:57 |
gyee | TheJulia, the folder exists and have the correct perm | 18:58 |
*** bfournie has quit IRC | 18:59 | |
gyee | but according to the code, the subfolder will only be created if the provision state is 'active' | 18:59 |
gyee | though I don't understand the difference between 'active' and 'deploying' | 18:59 |
gyee | it's supposed to be created here if the provision state is 'active' https://github.com/openstack/ironic/blob/stable/rocky/ironic/drivers/modules/pxe.py#L289 | 19:01 |
*** e0ne has joined #openstack-ironic | 19:01 | |
*** zzzeek has quit IRC | 19:04 | |
*** michchap has quit IRC | 19:05 | |
*** zzzeek has joined #openstack-ironic | 19:07 | |
TheJulia | gyee: it should be created in deploying. It needs to be created in deploying as well | 19:08 |
TheJulia | feels like a thing I hit yesterday actually | 19:08 |
gyee | that's weird cause I don't see any other errors prior to that one | 19:08 |
TheJulia | oh | 19:09 |
TheJulia | so ipxe shouldn't be writing config to /tftpboot | 19:09 |
TheJulia | it would be writing it to /httpboot | 19:09 |
gyee | I don't see /httpboot | 19:10 |
TheJulia | it may be different based upon your ironic config | 19:10 |
gyee | yes, I have this | 19:10 |
gyee | [deploy] | 19:10 |
gyee | http_root = /tftpboot | 19:10 |
gyee | so it should be created in /tftpboot/ | 19:11 |
gyee | TheJulia, according to the code, the file will only get created if provision state is 'active' or 'unrescue' https://github.com/openstack/ironic/blob/stable/rocky/ironic/drivers/modules/pxe.py#L284 | 19:12 |
gyee | maybe some other code path will create that file as well? | 19:12 |
*** emine__ has joined #openstack-ironic | 19:12 | |
TheJulia | yeah | 19:12 |
*** Emine has quit IRC | 19:13 | |
TheJulia | the problem is the config is attempting to be switched to, but there is no config to switch | 19:13 |
TheJulia | gyee: maybe see if something like the change in https://review.openstack.org/#/c/635996/21..22/ironic/common/pxe_utils.py fixes your issue? | 19:13 |
patchbot | patch 635996 - ironic - fast tracked deployment support - 22 patch sets | 19:13 |
TheJulia | but I'm not sure how your getting there to be honest | 19:14 |
gyee | I hit this without setting boot_option:'local' | 19:14 |
gyee | default boot_option is 'netboot' I think | 19:15 |
gyee | 'local' works fine, only 'netboot' is giving me this problem | 19:15 |
*** EmilienM has joined #openstack-ironic | 19:17 | |
EmilienM | hello there | 19:17 |
openstackgerrit | Mark Goddard proposed openstack/ironic master: Fix idrac Job.state renamed to Job.status https://review.openstack.org/643142 | 19:18 |
mgoddard | rpioso: ^^^ | 19:18 |
EmilienM | I've been playing with ironic/nova on stein in tripleo environment (don't be scared :D) && I'm having an issue to register the nova compute as an hypervisor | 19:18 |
EmilienM | http://paste.openstack.org/show/Bz6gl5oFokoah9Ug3VZc/ | 19:19 |
EmilienM | I'm wondering if this kind of error rings a bell to someone :) | 19:19 |
EmilienM | (also asked on #openstack-nova channel) | 19:21 |
rpioso | mgoddard: Ack | 19:23 |
TheJulia | gyee: in standalone testing I take it? | 19:24 |
TheJulia | gyee: and your using a partition image I suspect? | 19:24 |
TheJulia | EmilienM: is ironic-api even up and running? | 19:25 |
EmilienM | TheJulia: I think it's https://bugzilla.redhat.com/show_bug.cgi?id=1686817 | 19:25 |
openstack | bugzilla.redhat.com bug 1686817 in openstack-nova "[osp15] nova is broken on the undercloud with ssl" [Urgent,Assigned] - Assigned to mbooth | 19:25 |
TheJulia | EmilienM: my _guess_ would be that it has not yet connected to the api at all because it is invoking version negotiation | 19:25 |
EmilienM | TheJulia: Ironic is fine and reports zero error | 19:26 |
TheJulia | Once instantiated, it doesn't need to call for version negotiation | 19:26 |
TheJulia | well, instantiated and has the first connection | 19:26 |
EmilienM | TheJulia: I think I just found the reason, I need to disable SSL on my undercloud (for nova at least) until the bug is resolved | 19:26 |
TheJulia | so ssl failure, yeah... that kind of makes sense becaues it wouldn't have negotiated yet | 19:27 |
EmilienM | it's on nova side. Thanks for great support - I'll report back if I have an update on that one, but workaround is disabling ssl now | 19:30 |
*** andrein has joined #openstack-ironic | 19:30 | |
gyee | TheJulia, why partition image make a difference? | 19:32 |
TheJulia | gyee: whole disk images generally implies a local boot option | 19:45 |
TheJulia | you've basically stubled upon a valid bug, and I encourage you to submit a patch | 19:46 |
gyee | TheJulia, let me play around with the code a bit more. Still trying to figure out various state transitions at this point. | 19:48 |
*** bfournie has joined #openstack-ironic | 19:53 | |
openstackgerrit | Merged openstack/ironic master: Allow methods to be both deploy and clean steps https://review.openstack.org/643012 | 19:55 |
*** andrein has quit IRC | 20:16 | |
openstackgerrit | Lin Yang proposed openstack/sushy master: Fix wrong connector attribute in processor https://review.openstack.org/643152 | 20:16 |
openstackgerrit | Merged openstack/ironic master: Add python3 unit test with drivers installed https://review.openstack.org/641984 | 20:17 |
openstackgerrit | Merged openstack/ironic master: Return 405 for old versions in allocation and deploy template APIs https://review.openstack.org/642762 | 20:17 |
*** andrein has joined #openstack-ironic | 20:28 | |
*** e0ne has quit IRC | 20:38 | |
*** gmann_afk is now known as gmann | 20:41 | |
*** e0ne has joined #openstack-ironic | 20:41 | |
*** e0ne has quit IRC | 20:45 | |
*** mjturek has quit IRC | 20:55 | |
*** trown is now known as trown|outtypewww | 21:10 | |
*** whoami-rajat has quit IRC | 21:11 | |
*** bfournie has quit IRC | 21:21 | |
*** baha has quit IRC | 21:28 | |
*** rcernin has joined #openstack-ironic | 21:57 | |
*** rh-jelabarre has quit IRC | 22:02 | |
*** lifeless has quit IRC | 22:27 | |
*** lifeless has joined #openstack-ironic | 22:27 | |
*** bfournie has joined #openstack-ironic | 22:29 | |
*** ianychoi has joined #openstack-ironic | 23:06 | |
*** mjturek has joined #openstack-ironic | 23:18 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: fast tracked deployment support https://review.openstack.org/635996 | 23:27 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: fast tracked deployment support https://review.openstack.org/635996 | 23:34 |
openstackgerrit | Julia Kreger proposed openstack/ironic-tempest-plugin master: Add fast-track testing capability https://review.openstack.org/637039 | 23:43 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Add fast-track testing https://review.openstack.org/641104 | 23:43 |
*** mjturek has quit IRC | 23:43 | |
*** johnhaan has joined #openstack-ironic | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!