Wednesday, 2019-03-13

*** betherly has joined #openstack-ironic00:02
*** dsneddon has joined #openstack-ironic00:05
*** yajushiSri has quit IRC00:07
*** betherly has quit IRC00:07
larsksAm 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_08181400:21
larskss/checking/failing/00:22
*** betherly has joined #openstack-ironic00:23
*** betherly has quit IRC00:28
*** gyee has quit IRC00:30
*** betherly has joined #openstack-ironic00:44
*** betherly has quit IRC00:49
*** whoami-rajat has joined #openstack-ironic00:51
openstackgerritGhanshyam Mann proposed openstack/ironic master: Migrate ironic-grenade-dsvm-multinode-multitenant job to Ubuntu Bionic  https://review.openstack.org/64231500:58
dustinclarsks: seems to me like maybe it is failing due to the missing license header...not completely sure, though01:03
larsksdustinc: oh, hah, I think you're right.  I didn't even see that after all of those warnings.01:05
larsksThanks for looking :)01:05
*** betherly has joined #openstack-ironic01:05
openstackgerritMerged openstack/ironic-python-agent master: Correct clean_build command according to Makefile  https://review.openstack.org/64251501:06
*** betherly has quit IRC01:10
openstackgerritQianBiao Ng proposed openstack/ironic master: Add Huawei iBMC driver support  https://review.openstack.org/63928801:21
*** betherly has joined #openstack-ironic01:26
*** betherly has quit IRC01:31
openstackgerritKaifeng Wang proposed openstack/ironic master: Follow up to available node protection  https://review.openstack.org/64293501:51
openstackgerritLars Kellogg-Stedman proposed openstack/ironic master: prevent serial console lockout  https://review.openstack.org/64273402:04
*** betherly has joined #openstack-ironic02:08
*** betherly has quit IRC02:12
*** betherly has joined #openstack-ironic02:29
*** mcarden has joined #openstack-ironic02:30
*** betherly has quit IRC02:34
*** betherly has joined #openstack-ironic02:50
*** betherly has quit IRC02:54
*** Qianbiao2 has joined #openstack-ironic03:08
*** betherly has joined #openstack-ironic03:11
*** betherly has quit IRC03:15
*** w14161_1 has quit IRC03:24
*** MattMan_ has quit IRC03:24
*** MattMan_ has joined #openstack-ironic03:25
*** w14161_1 has joined #openstack-ironic03:25
*** betherly has joined #openstack-ironic03:31
*** betherly has quit IRC03:36
*** dsneddon has quit IRC03:43
*** stendulker has joined #openstack-ironic03:45
*** dsneddon has joined #openstack-ironic04:08
*** dsneddon has quit IRC04:12
*** rh-jelabarre has quit IRC04:29
*** andrein has quit IRC04:35
openstackgerritMerged openstack/ironic master: Fix missing print format error  https://review.openstack.org/64234604:39
openstackgerritMerged openstack/ironic master: Document building configdrive on the server side  https://review.openstack.org/64244104:39
*** dsneddon has joined #openstack-ironic04:40
openstackgerritMerged openstack/ironic master: Fix formatting issue in doc  https://review.openstack.org/64271904:43
*** betherly has joined #openstack-ironic04:44
*** dsneddon has quit IRC04:45
*** betherly has quit IRC04:49
*** betherly has joined #openstack-ironic05:05
*** betherly has quit IRC05:09
*** dsneddon has joined #openstack-ironic05:11
*** dsneddon has quit IRC05:15
*** dsneddon has joined #openstack-ironic05:26
*** dsneddon has quit IRC05:31
*** hwoarang has quit IRC05:53
*** betherly has joined #openstack-ironic05:57
*** hwoarang has joined #openstack-ironic06:00
*** betherly has quit IRC06:01
*** dsneddon has joined #openstack-ironic06:05
*** dsneddon has quit IRC06:10
*** verma-varsha has joined #openstack-ironic06:18
*** pcaruana has joined #openstack-ironic06:19
*** dsneddon has joined #openstack-ironic06:20
*** dsneddon has quit IRC06:25
*** yolanda has quit IRC06:36
*** stendulker has quit IRC06:36
*** verma-varsha has quit IRC06:38
*** e0ne has joined #openstack-ironic06:40
*** diga has joined #openstack-ironic06:58
*** dsneddon has joined #openstack-ironic06:58
*** hwoarang has quit IRC07:07
*** hwoarang has joined #openstack-ironic07:08
*** yolanda has joined #openstack-ironic07:09
*** diga has quit IRC07:09
*** dsneddon has quit IRC07:11
*** monica_pardhi has joined #openstack-ironic07:12
*** diga has joined #openstack-ironic07:13
*** rachit7 has joined #openstack-ironic07:16
*** tonyb is now known as tonyb_gone07:20
*** tonyb_gone is now known as tonyb07:21
digamonica_pardhi: rachit7: Welcome to Ironic group :)07:21
*** PradipK has joined #openstack-ironic07:21
rachit7Thank you @diga07:21
digaPradipK: welcome to group:)07:21
PradipKThanks :)07:22
arne_wiebalckgood morning, ironic!07:27
*** e0ne has quit IRC07:31
*** dsneddon has joined #openstack-ironic07:45
*** gkadam has joined #openstack-ironic07:48
*** dsneddon has quit IRC07:50
*** e0ne has joined #openstack-ironic07:51
*** monica_pardhi has quit IRC07:51
*** dsneddon has joined #openstack-ironic07:54
*** tssurya has joined #openstack-ironic08:02
*** PradipK has quit IRC08:05
*** lennyb has joined #openstack-ironic08:05
*** rachit7 has quit IRC08:07
rpittau|afkgood morning ironic! o/08:09
*** rcernin has quit IRC08:09
*** rpittau|afk is now known as rpittau08:09
rpittauhappy hump day :)08:09
*** diga has quit IRC08:18
*** e0ne has quit IRC08:36
*** hwoarang has quit IRC08:40
*** hwoarang has joined #openstack-ironic08:42
*** iurygregory has joined #openstack-ironic08:46
iurygregorygood morning o/08:46
*** andrein has joined #openstack-ironic09:03
*** sburette has joined #openstack-ironic09:03
*** dtantsur|afk is now known as dtantsur09:14
dtantsurmorning ironic09:14
jiapeiMorning dtantsur09:15
jiapeiFor 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
iurygregorymorning dtantsur o/09:18
dtantsurjiapei: 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 useful09:19
rpittauhi dtantsur :)09:19
rpittauhey iurygregory :)09:19
iurygregoryrpittau, o/09:20
jiapeidtantsur: the log is here http://electronicjungle.net:8082/lxca_ironic_logs/02/642402/5/check/lenovo-xclarity-ironic/0c8938c/logs/job-output.json09:22
jiapeiwondering why the job-output.txt has lots of "waiting on logger"...09:22
*** FlorianFa has joined #openstack-ironic09:23
*** hwoarang has quit IRC09:23
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add docs on UEFI boot libvirt-based emulation  https://review.openstack.org/64283009:27
iurygregoryrpittau, I think I've found the issue for the env .-.09:28
iurygregorycli_dir = os.path.join(os.path.abspath('.'), '.tox/functional/bin')09:28
rpittauiurygregory, oh great!09:30
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Rename `drivers` to `systems`  https://review.openstack.org/63849209:30
rpittauwhere was that ? tempest-ironic ?09:30
*** hwoarang has joined #openstack-ironic09:32
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add Managers resource support  https://review.openstack.org/63868209:32
dtantsurjiapei: please look into fixing it, reading enormous JSON is pretty inconvenient :)09:32
iurygregoryrpittau, nope in base.py ironicclient/tests/functional09:33
* iurygregory not sure if tox allows in the cmd a way to set the python version09:33
*** derekh has joined #openstack-ironic09:35
*** diga has joined #openstack-ironic09:36
jiapeidtantsur: OK, we'll look into the txt issue :)09:37
iurygregorydtantsur, 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#L4009:42
iurygregoryany problem if we switch for the approach that glance did: http://git.openstack.org/cgit/openstack/python-glanceclient/tree/glanceclient/tests/functional/base.py#n5109:43
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add read-only Virtual Media support  https://review.openstack.org/63878709:50
openstackgerritMark Goddard proposed openstack/ironic master: Return 405 for old versions in allocation and deploy template APIs  https://review.openstack.org/64276209:51
*** Emine has joined #openstack-ironic09:53
dtantsurlarsks: hi, you around?09:54
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add writable Virtual Media resource mocks  https://review.openstack.org/63933809:54
dtantsuriurygregory: no problem at all09:54
iurygregorydtantsur, ack09:54
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add Managers<->Systems linkage in sushy-emulator  https://review.openstack.org/64030409:58
openstackgerritDmitry Tantsur proposed openstack/ironic master: honor ipmi_port in serial console drivers  https://review.openstack.org/64093010:00
dtantsurlarsks: added a release note ^^10:00
openstackgerritMark Goddard proposed openstack/ironic master: Return 405 for old versions in allocation and deploy template APIs  https://review.openstack.org/64276210:05
openstackgerritIury Gregory Melo Ferreira proposed openstack/python-ironicclient master: Run jobs under python2 and python3  https://review.openstack.org/64247410:09
openstackgerritRiccardo Pittau proposed openstack/ironic master: Adding ansible python interpreter as driver_info  https://review.openstack.org/64240210:15
mgoddarddtantsur: "I've seen _route in the node controller returning "None" in some case. Will it have the same effect?"10:24
mgoddarddtantsur: I can't see any examples of it returning None10:24
rpittauTheJulia, dtantsur, wdyt about switching postgres test to ipxe ?10:28
*** hwoarang has quit IRC10:28
*** hwoarang has joined #openstack-ironic10:29
rpittauit's kind of weird what's happening with that test, the virtual node tries to load ipxe anyway even if it shouldn't10:30
rpittauactually if anyone has any idea on why this is happening please don't restrain yourself :)10:31
dtantsurmgoddard: ah, it was lookup https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/node.py#L160710:37
mgoddarddtantsur: yeah, just replied10:37
dtantsurrpittau: we need some coverage for pxe (much more than for postgres)10:37
dtantsurrpittau: maybe we forgot to disable the "ipxe" boot interface or switch the default to "pxe"?10:37
dtantsurpreviously it was a configuration option "ipxe_enabled"10:37
rpittaudtantsur, the option ipxe_enabled is set to False :/10:38
dtantsurmgoddard: yeah, maybe. I don't care either way10:38
dtantsurrpittau: right, but it does not matter much if the "ipxe" boot interface is used10:38
dtantsurit only affects the "pxe" boot interface10:38
rpittaudtantsur, the boot_interfaces enabled are fake and pxe10:39
dtantsurhmm10:39
dtantsurrpittau: do you have an example job link?10:39
rpittauI reproduces it on a devstack machine and I' mvery very very puzzled10:39
rpittaudtantsur, https://review.openstack.org/64243010:40
patchbotpatch 642430 - ironic - [DNM] testing postgres job - 5 patch sets10:40
dtantsurrpittau: can you check that the node actually ended up with boot_interface=pxe?10:40
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add libvirt-based virtual media emulation  https://review.openstack.org/64045210:42
dtantsurthe CI job looks correct10:43
etingofjust 10 patches and VM has been booted through virtual media, first time ever in my life ^10:44
*** dsneddon has quit IRC10:44
dtantsur\o/10:44
dtantsurrpittau: I can only think about something firewall-related, tbh10:44
dtantsurDHCP options in Neutron look about right10:45
*** priteau has joined #openstack-ironic10:45
rpittaudtantsur, ok, thanks, I'll dig a little bit more10:47
*** w14161_1 has quit IRC10:51
openstackgerritMerged openstack/ironic master: [docs] IPv6 support for iLO  https://review.openstack.org/63859010:55
*** w14161_1 has joined #openstack-ironic10:57
openstackgerritVarsha Verma proposed openstack/ironic master: Deprecates `hash_distribution_replicas` config option  https://review.openstack.org/64300210:57
*** diga has quit IRC11:03
*** dsneddon has joined #openstack-ironic11:16
*** dsneddon has quit IRC11:21
Qianbiao2Hello dtantsur11:22
Qianbiao2Is our CI log ok like this http://112.93.129.99/builds/Post-build-task-test/5/11:23
*** gkadam_ has joined #openstack-ironic11:24
*** gkadam has quit IRC11:25
dtantsurQianbiao2: hi! it's better, but can we also see the overall job output, including devstack and tempest?11:26
Qianbiao2we did not use tempest ;(11:26
Qianbiao2I try tempest, it's impossible to download everything thought network in our VM11:26
Qianbiao2Now, i am using a standalone ENV with everything ready.11:27
Qianbiao2and everytime, git download patch, then reinstall ironic.11:27
*** verma-varsha has joined #openstack-ironic11:27
Qianbiao2and deploy baremetal using an ansible playbook11:27
Qianbiao2before CI run, will cleanup logs. and after CI build, will copy logs to a nginx server root.11:28
Qianbiao2logs will contains: jenkins build log, ironic-api log, ironic conductor log, dnsmasq log when CI is ready.11:29
dtantsurQianbiao2: fine with me, can we have the output of that playbook?11:31
dtantsurright11:31
* dtantsur should read the whole message before responding11:31
Qianbiao2sure, it will contains in jenkins build log later11:32
Qianbiao2we are using playbook https://github.com/IamFive/ansible-ironic-standalone11:33
*** yaawang has quit IRC11:33
*** e0ne has joined #openstack-ironic11:36
openstackgerritMark Goddard proposed openstack/ironic master: WIP: Add iDRAC RAID deploy steps  https://review.openstack.org/64173111:38
openstackgerritMark Goddard proposed openstack/ironic master: Allow methods to be both deploy and clean steps  https://review.openstack.org/64301211:39
*** yaawang has joined #openstack-ironic11:42
*** verma-varsha has quit IRC11:46
*** dsneddon has joined #openstack-ironic11:50
*** rh-jelabarre has joined #openstack-ironic11:53
*** rh-jelabarre has quit IRC11:53
*** rh-jelabarre has joined #openstack-ironic11:53
*** dsneddon has quit IRC11:55
*** hwoarang has quit IRC11:56
*** hwoarang has joined #openstack-ironic11:58
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add docs on libvirt virtual media emulation  https://review.openstack.org/64302712:04
openstackgerritVarsha Verma proposed openstack/ironic master: Deprecates `hash_distribution_replicas` config option  https://review.openstack.org/64300212:10
*** dsneddon has joined #openstack-ironic12:23
*** hwoarang has quit IRC12:24
*** hwoarang has joined #openstack-ironic12:28
*** dsneddon has quit IRC12:29
*** dsneddon has joined #openstack-ironic12:55
iurygregoryAnyone 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_26833412:56
dtantsuriurygregory: I think the message has changed on the ironic side12:58
iurygregorydtantsur, so we need to updtade the tests only?12:58
dtantsuryep12:58
iurygregoryack12:59
iurygregorygoing to look at the patchs in ironic and update the client12:59
iurygregoryor someone have a patch already?12:59
* dtantsur does not12:59
*** diga has joined #openstack-ironic12:59
*** dsneddon has quit IRC13:00
*** dsneddon has joined #openstack-ironic13:01
*** e0ne has quit IRC13:05
openstackgerritMerged openstack/ironic master: Follow up to available node protection  https://review.openstack.org/64293513:07
openstackgerritMerged openstack/ironic master: Fixing a bash test in devstack ironic lib  https://review.openstack.org/64280913:07
openstackgerritMerged openstack/ironic-inspector master: Revise api-ref docs  https://review.openstack.org/61351513:07
*** dsneddon has quit IRC13:10
*** mbeierl has joined #openstack-ironic13:19
*** mbeierl has quit IRC13:23
*** priteau has quit IRC13:27
*** pandy has joined #openstack-ironic13:32
*** mjturek has joined #openstack-ironic13:33
TheJuliagood morning13:35
dtantsurmorning TheJulia13:36
TheJuliarpittau: 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 machines13:37
*** mbeierl has joined #openstack-ironic13:37
rpittauTheJulia, hi, yeah, I did some more digging to understand how that worked and I see that it's a remap to tftp13:39
rpittaualthough it doesn't seem to work13:40
rpittauprobably something on the internal routing13:41
openstackgerritRachit Kapadia proposed openstack/ironic master: Set boot_mode in node properties during OOB Introspection  https://review.openstack.org/63969813:42
*** dsneddon has joined #openstack-ironic13:42
*** bfournie has joined #openstack-ironic13:44
*** dsneddon has quit IRC13:47
TheJuliathe 405 error patch has been approved13:47
openstackgerritMerged openstack/ironic master: honor ipmi_port in serial console drivers  https://review.openstack.org/64093013:48
*** dsneddon has joined #openstack-ironic13:55
*** baha has joined #openstack-ironic13:56
*** Emine has quit IRC13:59
*** hwoarang has quit IRC14:01
*** dsneddon has quit IRC14:02
*** gkadam_ has quit IRC14:02
*** dsneddon has joined #openstack-ironic14:04
*** hwoarang has joined #openstack-ironic14:08
*** Emine has joined #openstack-ironic14:08
*** dsneddon has quit IRC14:09
*** irclogbot_1 has quit IRC14:09
*** e0ne has joined #openstack-ironic14:10
*** irclogbot_1 has joined #openstack-ironic14:12
*** Qianbiao2 has quit IRC14:12
*** hwoarang has quit IRC14:17
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add docs on libvirt virtual media emulation  https://review.openstack.org/64302714:18
*** hwoarang has joined #openstack-ironic14:19
*** bnemec has quit IRC14:22
*** Emine has quit IRC14:24
*** Emine has joined #openstack-ironic14:24
*** irclogbot_1 has quit IRC14:25
*** bnemec has joined #openstack-ironic14:27
*** irclogbot_1 has joined #openstack-ironic14:27
digadtantsur: TheJulia: Hi14:29
digadtantsur: CI is up today ?14:29
digahttps://review.openstack.org/#/c/641683/14:29
patchbotpatch 641683 - ironic (stable/ocata) - Fix OOB introspection to use pxe_enabled flag in i... - 4 patch sets14:29
dtantsurdiga: well, that problem has just been solved. I'm not sure how many problems you'll see in ocata though14:29
*** dougsz has joined #openstack-ironic14:29
digadtantsur: okay14:29
digadtantsur: let me recheck14:30
openstackgerritDigambar proposed openstack/ironic stable/ocata: Fix OOB introspection to use pxe_enabled flag in idrac driver  https://review.openstack.org/64168314:31
*** jtomasek has quit IRC14:38
*** dsneddon has joined #openstack-ironic14:39
*** dsneddon has quit IRC14:45
*** baha has quit IRC14:47
*** dsneddon has joined #openstack-ironic14:54
*** baha has joined #openstack-ironic15:02
arne_wiebalckdtantsur: 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
dtantsurarne_wiebalck: yep15:04
arne_wiebalckcool, thx15:04
* arne_wiebalck was wondering if that would look like sneaking in changes after the spec was approved :)15:04
dtantsurarne_wiebalck: we've been doing it pretty regularly :) they're still reviewed after all15:11
arne_wiebalckdtantsur: 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 version15:12
dtantsuryes, it's fine. I recall one of my specs changing 3 or 4 times :)15:12
arne_wiebalckdtantsur: "when specs meet reality" :-D15:13
dtantsurI prefer to think we're agile ;)15:14
arne_wiebalcklol15:14
*** andrein has quit IRC15:30
*** hwoarang has quit IRC15:32
*** hwoarang has joined #openstack-ironic15:33
*** diga has quit IRC15:35
*** irclogbot_1 has quit IRC15:36
*** irclogbot_1 has joined #openstack-ironic15:39
*** andrein has joined #openstack-ironic15:46
*** irclogbot_1 has quit IRC15:49
TheJulia"When specs meet reality" is a good way of putting it15:50
*** irclogbot_1 has joined #openstack-ironic15:51
*** irclogbot_1 has quit IRC15:52
*** irclogbot_1 has joined #openstack-ironic15:56
NobodyCamGood Morning Ironic'ers15:56
iurygregorymorning NobodyCam15:56
NobodyCamGood Morning iurygregory :) \o15:57
dtantsurmorning NobodyCam15:58
iurygregory=)15:58
NobodyCammorning dtantsur :) happy hump day16:01
rpittauhi NobodyCam :)16:03
NobodyCamhey hey rpittau :) \o16:03
*** dsneddon has quit IRC16:04
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add libvirt-based virtual media emulation  https://review.openstack.org/64045216:09
*** diga has joined #openstack-ironic16:09
digadtantsur: Hi16:09
dtantsurhi diga16:10
digadtantsur: Cherry-pick patch is passing all the zull check :) - /msg nickserv identify Diga091516:10
digasorry16:10
digahttps://review.openstack.org/#/c/641683/16:10
patchbotpatch 641683 - ironic (stable/ocata) - Fix OOB introspection to use pxe_enabled flag in i... - 5 patch sets16:10
dtantsurdiga: you may have just exposed your password :)16:10
digadtantsur: yes, by mistake16:10
digaNeed to change now :)16:10
dtantsurdiga: better change it, this channel is logged16:10
digayes16:10
dtantsurdiga: ocata has python-dracclient>=0.1.0 in its driver-requirements. can you confirm this patch works with dracclient==0.1.016:12
*** diga has quit IRC16:15
*** jtomasek has joined #openstack-ironic16:16
JayFTheJulia: arne_wiebalck: That sounds like a title to a summit presentation if I've ever heard one :D16:16
*** diga has joined #openstack-ironic16:18
*** trown is now known as trown|lunch16:18
digadtantsur: Finally reset the password :)16:18
digadtantsur: Please review this patch - https://review.openstack.org/#/c/641683/16:18
patchbotpatch 641683 - ironic (stable/ocata) - Fix OOB introspection to use pxe_enabled flag in i... - 5 patch sets16:18
*** dsneddon has joined #openstack-ironic16:19
*** Emine has quit IRC16:20
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Render System->Managers linkage  https://review.openstack.org/64309416:20
dtantsurdiga: 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.016:22
*** dsneddon has quit IRC16:23
*** priteau has joined #openstack-ironic16:24
digaLet me check16:24
digaone min16:24
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add docs on libvirt virtual media emulation  https://review.openstack.org/64302716:24
*** irclogbot_1 has quit IRC16:24
mgoddardTheJulia: 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
patchbotpatch 643012 - ironic - Allow methods to be both deploy and clean steps - 1 patch set16:25
*** irclogbot_1 has joined #openstack-ironic16:25
*** gyee has joined #openstack-ironic16:25
dtantsurmgoddard: I'd rebase if it does not really need the previous patches16:26
digadtantsur: This patch needs python-dracclient>=1.5.0 version16:26
mgoddarddtantsur: ack16:26
dtantsurdiga: then we cannot backport it.16:27
dtantsurwell, at least we need to make sure it doesn't break completely with 0.1.016:28
*** dsneddon has joined #openstack-ironic16:28
digadtantsur: okay16:28
digarpioso: you there ?16:29
openstackgerritMark Goddard proposed openstack/ironic master: Allow methods to be both deploy and clean steps  https://review.openstack.org/64301216:29
dtantsurdiga: it's bad enough that we backported it to Pike, which has >=1.3.0...16:30
digadtantsur: yes, we need version 1.5.0 for this patch16:30
digadtantsur: should I abandon this one ?16:31
dtantsurdiga: I guess so16:31
digaokay16:31
*** dsneddon has quit IRC16:34
TheJuliaJayF: You raise a very good point! Get a China business visa and present it in the fall?16:34
*** dsneddon has joined #openstack-ironic16:35
dtantsur++16:36
TheJuliaIt occurs to me that that whole thing with visas is going to push all of the summit processes up...16:36
digadtantsur: Thanks for your quick review :)16:36
digaNeed to sign off for the day!16:36
dtantsurTheJulia: welcome to my world, I have cookies :)16:36
TheJuliaIt was a huge pain to get one, at least it is multiple entry for me16:36
TheJuliaMmmmm! Cookies!16:36
* dtantsur has recently experienced the doubtful pleasure of a face-to-face interview in the US embassy in Frankfurt16:37
iurygregoryChina Visa requires that you have some vaccines if i do remember16:37
TheJuliaI did not have to submit medical records, but requirements vary by country of citizenship and origin16:37
iurygregoryyeah16:40
iurygregorysince Brazil is in south america i need yellow fever D:16:40
dtantsurwow16:40
TheJuliaUS State department recommends travelers to china have influenza, typhoid, measles, Hepatitis A and B, and tuberculosis vaccines16:40
*** diga has quit IRC16:40
dtantsurI don't see it in the requirements list. But it may be a good idea.16:40
iurygregorybut like we dont have this for many years ...16:40
iurygregorywell in the middle of the year time for vaccines yay16:41
*** FlorianFa has quit IRC16:41
TheJuliaIt reminds me, I need to get another TDAP soon...16:42
openstackgerritArne Wiebalck proposed openstack/ironic-specs master: Add details on target_raid_config for Software RAID support  https://review.openstack.org/64310016:43
dtantsurI don't even remember which ones I have..16:43
iurygregoryTDAP? .-.16:43
TheJuliaThere 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 vaccine16:43
arne_wiebalckJayF: TheJulia: ^^ "spec met reality"  :)16:43
iurygregoryoh gotcha =)16:44
etingofthat reminds me an awesome book on all these vaccines and their discoveries -- https://www.amazon.com/Microbe-Hunters-Paul-Kruif/dp/B002TO3ZUW16:44
TheJuliaiurygregory: TDAP is Tetanus, Diptheria, Pertussis. In the states it is one of those "every 10 years or as needed vaccines"16:45
iurygregorywoa =O16:45
iurygregoryat least is every 10 yrs16:45
iurygregoryXD16:45
arne_wiebalckJayF:  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 years16:45
TheJuliaarne_wiebalck: interesting!16:46
JayFarne_wiebalck: I barely remember that, but I'm glad my loud voice continues to echo through the halls of openstack :P16:46
arne_wiebalckseems they found out vaccinations last much longer than expected16:46
dustincphew because I was trying to think of th elast time I got TDAP and couldn't remember16:46
TheJuliadustinc: likely a good idea to maybe... get another?16:46
* TheJulia ducks16:46
TheJuliaIf you can't remember.... ;)16:46
JayFI use a dual pronged approach to avoiding tetanus: vaccinations and never leaving my house ;)16:47
dtantsurlol16:47
TheJuliaheh16:47
dtantsurTheJulia: have they announced the city officially? I keep forgetting.16:48
TheJuliadtantsur: They did on twitter I think and I kind of got a little upset that they were already starting to advertise16:49
dtantsurTheJulia: so, which one is it? :)16:49
* TheJulia looks since she doesn't want to miss-speak16:49
TheJuliaWebsite says Shanghai16:49
TheJuliaBeijing 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 week16:52
dtantsurheh16:52
TheJuliaOne was an AWS conference too...16:53
TheJulia#awkward16:53
*** FlorianFa has joined #openstack-ironic16:54
*** e0ne has quit IRC17:01
dtantsurindeed17:03
arne_wiebalckbye, everyone o/17:05
*** sburette has quit IRC17:05
*** trown|lunch is now known as trown17:07
mgoddardrpioso: hi17:15
*** andrein has quit IRC17:15
dtantsur\o17:15
*** dtantsur is now known as dtantsur|afk17:15
rpittaugood night! o/17:16
*** rpittau is now known as rpittau|afk17:16
*** gyee has quit IRC17:20
*** gyee has joined #openstack-ironic17:20
*** tssurya has quit IRC17:26
*** pandy has quit IRC17:48
*** ianychoi has quit IRC17:55
*** dougsz has quit IRC17:58
*** derekh has quit IRC18:00
rpiosomgoddard: Hello18:05
* rpioso has been busy downstream18:06
gyeeTheJulia, 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 glance18:16
gyeethe error occurred here https://github.com/openstack/ironic/blob/stable/rocky/ironic/drivers/modules/pxe.py#L291 because pxe_config_path doesn't exist18:17
gyeebut it should've been created here https://github.com/openstack/ironic/blob/stable/rocky/ironic/drivers/modules/pxe.py#L289 when provision state is active18:17
*** priteau has quit IRC18:18
gyeenot sure if this is a known bug as pxe.py look very different between rocky and master18:19
*** pcaruana has quit IRC18:29
*** Emine has joined #openstack-ironic18:33
gyeelog seem to indicate that provision state was set to 'deploying' instead of 'active' so that file was never created in the first place18:34
*** priteau has joined #openstack-ironic18:40
*** priteau has quit IRC18:44
*** jtomasek has quit IRC18:48
*** gmann is now known as gmann_afk18:48
TheJuliagyee: it will create the subfolder, but not the /tftpboot folder. That must exist and ironic must have privilges to write to it18:56
TheJuliagyee: major differences between rocky and master is I retooled some of the locations so we could break ipxe into its own interface18:57
gyeeTheJulia, the folder exists and have the correct perm18:58
*** bfournie has quit IRC18:59
gyeebut according to the code, the subfolder will only be created if the provision state is 'active'18:59
gyeethough I don't understand the difference between 'active' and 'deploying'18:59
gyeeit'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#L28919:01
*** e0ne has joined #openstack-ironic19:01
*** zzzeek has quit IRC19:04
*** michchap has quit IRC19:05
*** zzzeek has joined #openstack-ironic19:07
TheJuliagyee: it should be created in deploying. It needs to be created in deploying as well19:08
TheJuliafeels like a thing I hit yesterday actually19:08
gyeethat's weird cause I don't see any other errors prior to that one19:08
TheJuliaoh19:09
TheJuliaso ipxe shouldn't be writing config to /tftpboot19:09
TheJuliait would be writing it to /httpboot19:09
gyeeI don't see /httpboot19:10
TheJuliait may be different based upon your ironic config19:10
gyeeyes, I have this19:10
gyee[deploy]19:10
gyeehttp_root = /tftpboot19:10
gyeeso it should be created in /tftpboot/19:11
gyeeTheJulia, 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#L28419:12
gyeemaybe some other code path will create that file as well?19:12
*** emine__ has joined #openstack-ironic19:12
TheJuliayeah19:12
*** Emine has quit IRC19:13
TheJuliathe problem is the config is attempting to be switched to, but there is no config to switch19:13
TheJuliagyee: 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
patchbotpatch 635996 - ironic - fast tracked deployment support - 22 patch sets19:13
TheJuliabut I'm not sure how your getting there to be honest19:14
gyeeI hit this without setting boot_option:'local'19:14
gyeedefault boot_option is 'netboot' I think19:15
gyee'local' works fine, only 'netboot' is giving me this problem19:15
*** EmilienM has joined #openstack-ironic19:17
EmilienMhello there19:17
openstackgerritMark Goddard proposed openstack/ironic master: Fix idrac Job.state renamed to Job.status  https://review.openstack.org/64314219:18
mgoddardrpioso: ^^^19:18
EmilienMI'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 hypervisor19:18
EmilienMhttp://paste.openstack.org/show/Bz6gl5oFokoah9Ug3VZc/19:19
EmilienMI'm wondering if this kind of error rings a bell to someone :)19:19
EmilienM(also asked on #openstack-nova channel)19:21
rpiosomgoddard: Ack19:23
TheJuliagyee: in standalone testing I take it?19:24
TheJuliagyee: and your using a partition image I suspect?19:24
TheJuliaEmilienM: is ironic-api even up and running?19:25
EmilienMTheJulia: I think it's https://bugzilla.redhat.com/show_bug.cgi?id=168681719:25
openstackbugzilla.redhat.com bug 1686817 in openstack-nova "[osp15] nova is broken on the undercloud with ssl" [Urgent,Assigned] - Assigned to mbooth19:25
TheJuliaEmilienM: my _guess_ would be that it has not yet connected to the api at all because it is invoking version negotiation19:25
EmilienMTheJulia: Ironic is fine and reports zero error19:26
TheJuliaOnce instantiated, it doesn't need to call for version negotiation19:26
TheJuliawell, instantiated and has the first connection19:26
EmilienMTheJulia: I think I just found the reason, I need to disable SSL on my undercloud (for nova at least) until the bug is resolved19:26
TheJuliaso ssl failure, yeah... that kind of makes sense becaues it wouldn't have negotiated yet19:27
EmilienMit'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 now19:30
*** andrein has joined #openstack-ironic19:30
gyeeTheJulia, why partition image make a difference?19:32
TheJuliagyee: whole disk images generally implies a local boot option19:45
TheJuliayou've basically stubled upon a valid bug, and I encourage you to submit a patch19:46
gyeeTheJulia, 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-ironic19:53
openstackgerritMerged openstack/ironic master: Allow methods to be both deploy and clean steps  https://review.openstack.org/64301219:55
*** andrein has quit IRC20:16
openstackgerritLin Yang proposed openstack/sushy master: Fix wrong connector attribute in processor  https://review.openstack.org/64315220:16
openstackgerritMerged openstack/ironic master: Add python3 unit test with drivers installed  https://review.openstack.org/64198420:17
openstackgerritMerged openstack/ironic master: Return 405 for old versions in allocation and deploy template APIs  https://review.openstack.org/64276220:17
*** andrein has joined #openstack-ironic20:28
*** e0ne has quit IRC20:38
*** gmann_afk is now known as gmann20:41
*** e0ne has joined #openstack-ironic20:41
*** e0ne has quit IRC20:45
*** mjturek has quit IRC20:55
*** trown is now known as trown|outtypewww21:10
*** whoami-rajat has quit IRC21:11
*** bfournie has quit IRC21:21
*** baha has quit IRC21:28
*** rcernin has joined #openstack-ironic21:57
*** rh-jelabarre has quit IRC22:02
*** lifeless has quit IRC22:27
*** lifeless has joined #openstack-ironic22:27
*** bfournie has joined #openstack-ironic22:29
*** ianychoi has joined #openstack-ironic23:06
*** mjturek has joined #openstack-ironic23:18
openstackgerritJulia Kreger proposed openstack/ironic master: fast tracked deployment support  https://review.openstack.org/63599623:27
openstackgerritJulia Kreger proposed openstack/ironic master: fast tracked deployment support  https://review.openstack.org/63599623:34
openstackgerritJulia Kreger proposed openstack/ironic-tempest-plugin master: Add fast-track testing capability  https://review.openstack.org/63703923:43
openstackgerritJulia Kreger proposed openstack/ironic master: Add fast-track testing  https://review.openstack.org/64110423:43
*** mjturek has quit IRC23:43
*** johnhaan has joined #openstack-ironic23:56

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!