| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Adds local_gb discovery https://review.openstack.org/483179 | 00:26 |
|---|---|---|
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Adds local_gb discovery https://review.openstack.org/483179 | 00:36 |
| *** zhangfei has joined #openstack-ironic | 01:02 | |
| *** wong has joined #openstack-ironic | 01:06 | |
| wong | morning | 01:06 |
| *** hw_wutianwei has joined #openstack-ironic | 01:07 | |
| *** r-daneel has joined #openstack-ironic | 01:18 | |
| *** phuongnh has joined #openstack-ironic | 01:21 | |
| *** yufei has joined #openstack-ironic | 01:23 | |
| *** mtanino has joined #openstack-ironic | 01:55 | |
| *** mtanino has quit IRC | 01:55 | |
| *** r-daneel has quit IRC | 02:06 | |
| *** r-daneel has joined #openstack-ironic | 02:23 | |
| wong | hi, may i ask a question? | 02:36 |
| *** syqian has joined #openstack-ironic | 02:42 | |
| wong | my deploy image is ubuntu,and my user's image canbe centos? | 02:42 |
| wong | when I use nova boot to create an instance, everything seems OK, suddenly,the system of deply image was closed,then nova throw exception...I have few log to fix it | 02:59 |
| *** zhangfei has quit IRC | 03:06 | |
| *** vishwanathj has quit IRC | 03:08 | |
| *** vishwanathj has joined #openstack-ironic | 03:08 | |
| yufei | you may find related logs in ironic-conductor | 03:10 |
| *** mtreinish has quit IRC | 03:14 | |
| wong | yeah, I get the path from the troubleshooting, but when I use 'tar' to decompression it, it failed... | 03:17 |
| *** zhangfei has joined #openstack-ironic | 03:19 | |
| *** mtreinish has joined #openstack-ironic | 03:20 | |
| *** syqian has quit IRC | 03:38 | |
| *** fragatina has quit IRC | 03:38 | |
| *** syqian has joined #openstack-ironic | 03:39 | |
| *** fragatina has joined #openstack-ironic | 03:39 | |
| *** udesale has joined #openstack-ironic | 03:42 | |
| *** links has joined #openstack-ironic | 03:46 | |
| openstackgerrit | paresh sao proposed openstack/ironic master: Added a condition for 'ilo' hardware type https://review.openstack.org/475342 | 03:52 |
| *** anish has quit IRC | 04:09 | |
| *** jjung has joined #openstack-ironic | 04:17 | |
| *** adisky__ has joined #openstack-ironic | 04:28 | |
| *** fragatina has quit IRC | 04:31 | |
| *** fragatina has joined #openstack-ironic | 04:32 | |
| *** rama_y has joined #openstack-ironic | 04:36 | |
| *** rbartal has joined #openstack-ironic | 04:45 | |
| *** dyasny__ has quit IRC | 04:49 | |
| *** dyasny__ has joined #openstack-ironic | 04:49 | |
| *** Dinesh_Bhor has joined #openstack-ironic | 04:56 | |
| openstackgerrit | Aparna proposed openstack/proliantutils master: Support to get hardware raid support https://review.openstack.org/422575 | 05:30 |
| *** boris-42____ has joined #openstack-ironic | 05:30 | |
| *** mvpnitesh has joined #openstack-ironic | 05:40 | |
| wong | how to fix this 'Could not create NetworkPortal in configFS' which I find in deploy log | 05:41 |
| *** syqian_ has joined #openstack-ironic | 05:41 | |
| *** mjura has joined #openstack-ironic | 05:42 | |
| *** syqian has quit IRC | 05:45 | |
| openstackgerrit | Merged openstack/ironic master: Allow updating interfaces on a node in available state https://review.openstack.org/482448 | 05:49 |
| *** syqian has joined #openstack-ironic | 06:01 | |
| *** syqian_ has quit IRC | 06:03 | |
| *** udesale__ has joined #openstack-ironic | 06:07 | |
| *** rama_y has quit IRC | 06:08 | |
| *** udesale has quit IRC | 06:10 | |
| *** marios has joined #openstack-ironic | 06:12 | |
| *** skazi has joined #openstack-ironic | 06:20 | |
| wong | hi, my deploy image throw this, http://paste.openstack.org/show/616266/ how can I fix it? | 06:21 |
| *** rcernin has joined #openstack-ironic | 06:23 | |
| *** makowals has joined #openstack-ironic | 06:23 | |
| *** syqian has quit IRC | 06:24 | |
| *** syqian has joined #openstack-ironic | 06:25 | |
| *** tuanluong has joined #openstack-ironic | 06:32 | |
| *** yufei has quit IRC | 06:33 | |
| openstackgerrit | Ankit Kumar proposed openstack/proliantutils master: Add cpu virtualisation capability https://review.openstack.org/486500 | 06:42 |
| *** jaganathan has joined #openstack-ironic | 06:46 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add supported_power_states field to node APIs https://review.openstack.org/382194 | 06:55 |
| *** ChubYann has quit IRC | 07:00 | |
| *** udesale has joined #openstack-ironic | 07:02 | |
| *** ccamacho has joined #openstack-ironic | 07:02 | |
| *** udesale__ has quit IRC | 07:04 | |
| *** yolanda_ has joined #openstack-ironic | 07:05 | |
| *** yolanda_ has quit IRC | 07:06 | |
| *** 7ITABD5MB has joined #openstack-ironic | 07:06 | |
| *** 07IAALFJ9 has joined #openstack-ironic | 07:06 | |
| *** 07IAALFJ9 has quit IRC | 07:07 | |
| *** 7ITABD5MB has quit IRC | 07:08 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add supported_power_states field to node APIs https://review.openstack.org/382194 | 07:08 |
| *** yolanda_ has joined #openstack-ironic | 07:08 | |
| *** zhangfei has quit IRC | 07:10 | |
| *** zhangfei has joined #openstack-ironic | 07:10 | |
| *** yolanda_ is now known as yolanda | 07:11 | |
| *** ralonsoh has joined #openstack-ironic | 07:13 | |
| *** mvpnitesh has quit IRC | 07:13 | |
| *** mvpnitesh has joined #openstack-ironic | 07:14 | |
| *** zhangfei has quit IRC | 07:15 | |
| *** lennyb has quit IRC | 07:19 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Adds local_gb discovery https://review.openstack.org/483179 | 07:21 |
| *** ralonsoh_ has joined #openstack-ironic | 07:22 | |
| *** ralonsoh has quit IRC | 07:22 | |
| *** zhangfei has joined #openstack-ironic | 07:28 | |
| *** lennyb has joined #openstack-ironic | 07:32 | |
| *** samodid has joined #openstack-ironic | 07:52 | |
| *** athomas has joined #openstack-ironic | 08:09 | |
| openstackgerrit | Mateusz Kowalski proposed openstack/ironic-python-agent master: BMC can be configured using different lan channel https://review.openstack.org/480679 | 08:12 |
| *** syqian_ has joined #openstack-ironic | 08:14 | |
| *** syqian has quit IRC | 08:16 | |
| *** syqian__ has joined #openstack-ironic | 08:16 | |
| wong | OK, I know the reason,it's a bug | 08:18 |
| *** syqian_ has quit IRC | 08:20 | |
| *** derekh has joined #openstack-ironic | 08:23 | |
| *** aparnav has joined #openstack-ironic | 08:26 | |
| *** lucas-afk is now known as lucasagomes | 08:26 | |
| *** e0ne has joined #openstack-ironic | 08:27 | |
| *** mvpnitesh has quit IRC | 08:29 | |
| *** jjung has quit IRC | 08:29 | |
| *** milan has joined #openstack-ironic | 08:31 | |
| milan | morning Ironic! | 08:32 |
| milan | :) | 08:32 |
| wong | good afternoon :-) | 08:32 |
| lucasagomes | milan, wong morning :D | 08:33 |
| *** mvpnitesh has joined #openstack-ironic | 08:33 | |
| *** dtantsur|afk is now known as dtantsur | 08:34 | |
| dtantsur | morning ironic | 08:34 |
| milan | morning lucasagomes, ugt morning wong! ;) | 08:35 |
| milan | morning dtantsur! :) | 08:35 |
| lucasagomes | dtantsur, o/ | 08:35 |
| dtantsur | o/ | 08:35 |
| *** milan has quit IRC | 08:37 | |
| *** milan has joined #openstack-ironic | 08:38 | |
| wong | do you think this is a BUG? http://paste.openstack.org/show/616285/ | 08:39 |
| *** milan has quit IRC | 08:43 | |
| *** milan has joined #openstack-ironic | 08:45 | |
| *** _milan_ has joined #openstack-ironic | 08:50 | |
| *** milan has quit IRC | 08:53 | |
| *** dsneddon_ has joined #openstack-ironic | 08:56 | |
| dtantsur | wong: it's hard to tell from just a short description, but it may be an issue in your environment too. I don't think OpenStack cares much about the port you use, maybe something is blocking or using it? | 08:56 |
| *** litao__ has joined #openstack-ironic | 08:58 | |
| *** etingof has quit IRC | 09:02 | |
| *** jaosorior has joined #openstack-ironic | 09:02 | |
| *** etingof has joined #openstack-ironic | 09:10 | |
| wong | but it's easy to happened in ubuntu, I take a few hours to fixed at last,maybe we can write code like this: catch the exception when the default port was taken, then we delete the target port, and retry to call rtslib_fb.NetworkPortal | 09:13 |
| *** sambetts|afk is now known as sambetts | 09:16 | |
| sambetts | Morning all | 09:16 |
| *** boris-42____ has quit IRC | 09:19 | |
| vdrok | morning ! | 09:32 |
| _milan_ | morning sambetts, vdrok! :) | 09:34 |
| vdrok | morning _milan_ wong dtantsur lucasagomes and sambetts ! :) | 09:35 |
| lucasagomes | vdrok, o/ morning | 09:35 |
| lucasagomes | sambetts, morning too | 09:35 |
| *** _milan_ is now known as milan | 09:35 | |
| sambetts | o/ lucasagomes vdrok milan | 09:35 |
| * milan may experience random network (and brain) outages today O:-) | 09:36 | |
| *** etingof has quit IRC | 09:49 | |
| *** etingof has joined #openstack-ironic | 09:50 | |
| *** jjung has joined #openstack-ironic | 09:50 | |
| *** vmud213 has joined #openstack-ironic | 09:59 | |
| *** tuanluong has quit IRC | 10:02 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Adds local_gb discovery https://review.openstack.org/483179 | 10:03 |
| *** udesale__ has joined #openstack-ironic | 10:04 | |
| *** udesale has quit IRC | 10:07 | |
| openstackgerrit | Merged openstack/proliantutils master: Add cpu virtualisation capability https://review.openstack.org/486500 | 10:10 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add supported_power_states field to node APIs https://review.openstack.org/382194 | 10:12 |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Adds macs discovery https://review.openstack.org/483127 | 10:15 |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: Redfish: Adds local_gb discovery https://review.openstack.org/483179 | 10:16 |
| *** MattMan_1 has quit IRC | 10:19 | |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector master: POC for what the docs main page could look like https://review.openstack.org/486555 | 10:19 |
| *** MattMan_1 has joined #openstack-ironic | 10:19 | |
| openstackgerrit | Debayan Ray proposed openstack/proliantutils master: Add supported boot modes to server capabilities https://review.openstack.org/275053 | 10:20 |
| *** racedo has joined #openstack-ironic | 10:38 | |
| wong | morning/evening | 10:41 |
| wong | :-) | 10:41 |
| *** udesale has joined #openstack-ironic | 10:42 | |
| *** udesale__ has quit IRC | 10:43 | |
| *** jkilpatr has quit IRC | 10:45 | |
| *** phuongnh has quit IRC | 10:50 | |
| *** lucasagomes is now known as lucas-hungry | 10:52 | |
| *** udesale has quit IRC | 10:55 | |
| *** r-daneel has quit IRC | 10:55 | |
| *** vmud213 has quit IRC | 10:57 | |
| openstackgerrit | Debayan Ray proposed openstack/proliantutils master: Add supported boot modes to server capabilities https://review.openstack.org/275053 | 10:59 |
| *** jkilpatr has joined #openstack-ironic | 11:02 | |
| *** jjung has quit IRC | 11:02 | |
| *** vsaienk0 has quit IRC | 11:05 | |
| *** stendulker has joined #openstack-ironic | 11:06 | |
| openstackgerrit | Debayan Ray proposed openstack/proliantutils master: Add test cases parity for RIS init https://review.openstack.org/486570 | 11:06 |
| *** Dinesh_Bhor has quit IRC | 11:07 | |
| *** zhangfei has quit IRC | 11:10 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/486380 | 11:14 |
| *** stendulker has quit IRC | 11:22 | |
| dtantsur | sambetts: wanna take a look at https://review.openstack.org/#/c/446763/ before it gets approved? | 11:28 |
| patchbot | patch 446763 - ironic - Start passing portgroup information to Neutron | 11:28 |
| dtantsur | also morning :) | 11:28 |
| sambetts | o/ hey, I'll take a look please | 11:28 |
| openstackgerrit | Aparna proposed openstack/proliantutils master: Redfish: Adds server capabilities https://review.openstack.org/484360 | 11:30 |
| *** jjung has joined #openstack-ironic | 11:35 | |
| *** aparnav has quit IRC | 11:38 | |
| *** xavierr|brb is now known as xavierr | 11:39 | |
| *** amarao has joined #openstack-ironic | 11:54 | |
| *** litao__ has quit IRC | 11:58 | |
| xavierr | morning ironic? | 11:58 |
| xavierr | s/?/!/ :) | 11:59 |
| *** fellypefca has joined #openstack-ironic | 12:01 | |
| *** lucas-hungry is now known as lucasagomes | 12:03 | |
| *** milan has quit IRC | 12:04 | |
| *** mrtenio|afk is now known as mrtenio | 12:08 | |
| dtantsur | morning xavierr :) | 12:08 |
| TheJulia | good morning everyone | 12:17 |
| * TheJulia walks in looking like a zombie | 12:18 | |
| sambetts | o/ TheJulia /me hands over a large coffee | 12:18 |
| xavierr | coffee += 1 | 12:19 |
| openstackgerrit | Merged openstack/ironic master: DevStack: Add configuration for boot-from-volume https://review.openstack.org/466333 | 12:19 |
| TheJulia | Thanks sambetts! | 12:19 |
| TheJulia | \o/ | 12:19 |
| TheJulia | dtantsur: thank you for kicking that again in the gate earlier today | 12:19 |
| dtantsur | morning TheJulia! | 12:20 |
| dtantsur | np. big thanks goes to jlvillal for kicking the gate on various patches through the whole weekend | 12:20 |
| dtantsur | TheJulia: now we need to press the nova folks into merging their part in the coming days :) | 12:21 |
| TheJulia | Yeah, I saw that too. :) | 12:21 |
| TheJulia | I worked so much last week, totally focused on stuff that I was sick all weekend | 12:21 |
| dtantsur | TheJulia: ugh :( don't push yourself too hard, we need you healthy more than we need BFV patches in Pike ;) | 12:22 |
| dtantsur | unrelated: as we release ironicclient this week, can I get some eyes on https://review.openstack.org/#/c/484866/ please? | 12:22 |
| patchbot | patch 484866 - python-ironicclient - Log warning when API version is not specified for ... | 12:22 |
| dtantsur | also please see the ML discussion that vdrok started | 12:23 |
| vdrok | good morning TheJulia ! | 12:23 |
| *** asettle has joined #openstack-ironic | 12:23 | |
| *** baoli has joined #openstack-ironic | 12:24 | |
| fellypefca | Morning everyone :) | 12:24 |
| *** jjung has quit IRC | 12:24 | |
| asettle | sambetts: got another question for you, if you have a mo. I'm looking around for some info on some work that was meant to be underway for ironic. Something something adding portgroups? | 12:24 |
| dtantsur | asettle: hi, https://docs.openstack.org/ironic/latest/admin/portgroups.html may be of use | 12:25 |
| asettle | AHUH | 12:26 |
| asettle | dtantsur: points to you | 12:26 |
| asettle | THank you :D | 12:26 |
| dtantsur | np :) | 12:26 |
| asettle | After that doc PTL went around and told everyone to move everything, things are not as easy to find ;) | 12:26 |
| TheJulia | dtantsur: yes :) | 12:26 |
| sambetts | asettle: gah what an annoy person ;) | 12:27 |
| sambetts | annoying* | 12:27 |
| asettle | I knowwwwww. | 12:27 |
| asettle | We should form a coup! | 12:27 |
| sambetts | hah ;) viva la revolution | 12:28 |
| TheJulia | dtantsur: I was planning on looking at that path and saw the ml thread. I have mixed feelings on a major bump right at the end for releasing the client library. If it was the beginning of the cycle, and upper constraints were already locked down for pike... that is another matter then. | 12:28 |
| openstackgerrit | Wang KaiFeng proposed openstack/ironic-specs master: Add spec for specific faults support https://review.openstack.org/334113 | 12:28 |
| asettle | sambetts: I should leave now before they work out there's an imposter in your midst :P | 12:29 |
| TheJulia | s/thread/post/, I think it was a lone email when I saw it | 12:29 |
| sambetts | asettle: Ironic has had portgroups for at least one release if not two but they have been statically configured, but we're in the process of accepted a patch to allow | 12:29 |
| openstackgerrit | Merged openstack/ironic-python-agent master: Collect processor, memory and BIOS output of dmidecode https://review.openstack.org/435573 | 12:29 |
| asettle | sambetts: okay neat :) thank you | 12:29 |
| *** jjung has joined #openstack-ironic | 12:29 | |
| sambetts | accepting a patch to allow ironic to inform the switches of the portgroup config it needs | 12:29 |
| asettle | sambetts: what releases are those? NEwton and Ocata? Or Ocata and Pike? | 12:30 |
| *** mvpnitesh has quit IRC | 12:30 | |
| sambetts | newton and ocata | 12:30 |
| *** mvpnitesh has joined #openstack-ironic | 12:31 | |
| *** jcoufal has joined #openstack-ironic | 12:31 | |
| asettle | Have you got a link to that patch, by any chance? | 12:31 |
| sambetts | the patch that added portgroups? | 12:32 |
| sambetts | there were several adding the DB model, internal APIs and REST API peices | 12:33 |
| sambetts | or do you mean the new patch? | 12:33 |
| asettle | new patch :) | 12:33 |
| *** baoli has quit IRC | 12:34 | |
| sambetts | asettle: https://review.openstack.org/#/c/446763/ | 12:34 |
| patchbot | patch 446763 - ironic - Start passing portgroup information to Neutron | 12:34 |
| asettle | Thank you :D | 12:34 |
| *** syqian_ has joined #openstack-ironic | 12:34 | |
| *** baoli has joined #openstack-ironic | 12:34 | |
| asettle | I also think this was something jroll was working on (perhaps I am wrong), something about fixing the issue with the single nova-compute to Ironic? | 12:35 |
| openstackgerrit | Arundhati Surpur proposed openstack/bifrost master: Update .gitignore https://review.openstack.org/486593 | 12:35 |
| asettle | Do you know anything about that? | 12:35 |
| jroll | that was done in newton :) | 12:35 |
| * jroll didn't read up | 12:35 | |
| sambetts | yup ^ | 12:35 |
| asettle | jroll: was there docs for that? | 12:36 |
| asettle | Also, morning! | 12:36 |
| jroll | asettle: I don't think so | 12:36 |
| asettle | Awesomeeeeeee | 12:36 |
| jroll | the only thing you need to do to enable it | 12:36 |
| sambetts | :-P | 12:36 |
| jroll | is turn on more than one compute | 12:36 |
| asettle | jroll: got the config for that? | 12:36 |
| jroll | no config :) | 12:37 |
| asettle | omfg | 12:37 |
| TheJulia | Yeah, i think it was done as a release note | 12:37 |
| asettle | Hahahahhaa | 12:37 |
| jroll | best feature ever right | 12:37 |
| *** syqian__ has quit IRC | 12:37 | |
| asettle | Love it | 12:37 |
| dtantsur | wow wiwm a real jroll! | 12:37 |
| asettle | So, we have a feature in Ironic, where you can now enable more than one nova-compute and that's not quite written anywhere? :p | 12:37 |
| dtantsur | s/wiwm/wow | 12:37 |
| jroll | asettle: it was in release notes | 12:37 |
| * asettle goes hunting for release notes | 12:38 | |
| dtantsur | asettle: one nice guy, called Jim, promised us a reference architecture. then he left ;) | 12:38 |
| jroll | I'm not sure if our docs say that you can/should run one or more than one | 12:38 |
| asettle | dtantsur: that bastard! | 12:38 |
| jroll | heh | 12:38 |
| openstackgerrit | Arundhati Surpur proposed openstack/bifrost master: Removes unnecessary utf-8 encoding https://review.openstack.org/486594 | 12:38 |
| dtantsur | now another nice guy, called Dmitry, promises the same. and he may even deliver! | 12:38 |
| *** trown|outtypewww is now known as trown | 12:38 | |
| asettle | Oh he sounds much nicer | 12:38 |
| asettle | jroll: maybe those docs should say that thing... | 12:38 |
| asettle | Would that be release notes for Newton or Ocata? | 12:38 |
| asettle | Probs Newton, cause it was enabled in Newton | 12:38 |
| asettle | Yes, thanks Alex | 12:38 |
| asettle | You're welcome Alex | 12:38 |
| * TheJulia steps away for a little bit | 12:38 | |
| sambetts | and an evil guy called Sam will change the architechture as soon as its written down ;) | 12:39 |
| asettle | omfg the release notes are in a fucking tarball | 12:39 |
| asettle | *head desk* | 12:39 |
| asettle | That's it, I don't care anymore. | 12:39 |
| jroll | O_o | 12:39 |
| sambetts | asettle: https://docs.openstack.org/releasenotes/ironic/ ? | 12:40 |
| jroll | https://docs.openstack.org/releasenotes/nova/newton.html | 12:40 |
| asettle | Sokay, 10 seconds after I threw my tantrum I ended up .. yeah, there | 12:40 |
| dtantsur | asettle: somewhere here it lives https://docs.openstack.org/releasenotes/nova/newton.html#id10 | 12:40 |
| dtantsur | Ctrl+F "ironic" actually shows it | 12:40 |
| sambetts | ahahahaha instant stampead of links to the same page | 12:40 |
| asettle | You guys <3 | 12:41 |
| dtantsur | :) | 12:41 |
| jroll | asettle: btw, we don't say one way or the other about it here: https://docs.openstack.org/ironic/latest/install/configure-compute.html | 12:41 |
| * dtantsur makes a note to fix it | 12:42 | |
| asettle | jroll: so, maybe we should? | 12:42 |
| asettle | I mean, while I'm here, I"m happy to | 12:42 |
| jroll | probably | 12:42 |
| jroll | :) | 12:42 |
| asettle | dtantsur: you want it? I'm happy to update | 12:42 |
| dtantsur | asettle: you're very welcome to fix it, if you don't mind. otherwise I'll get to it one day | 12:42 |
| asettle | Sure, why not. I'm the one badgering you guys for this info :p | 12:43 |
| dtantsur | thanks! :) | 12:43 |
| asettle | Ah, guess who's never cloned the ironic repo before... | 12:43 |
| asettle | whose* | 12:43 |
| asettle | No that was right hte first time | 12:43 |
| dtantsur | it's not as long as cloning nova for the first time, I guess.. or even worse project-config :) | 12:43 |
| asettle | *gasp* | 12:43 |
| asettle | I would never | 12:44 |
| asettle | Actually, manuals is definitely worse than nova | 12:44 |
| dtantsur | hmm, maybe. I remember cloning it for some reason one day.. | 12:44 |
| asettle | RIP your laptop/desktop | 12:45 |
| dtantsur | I no longer have it cloned, but I also have a different laptop nowadays. Maybe it's related? ;) | 12:45 |
| asettle | "There is no configuration to do to enable this feature; it is always enabled." | 12:46 |
| asettle | No way to disable it either? | 12:46 |
| asettle | If so, I'll just include a note at the beginning of the doc | 12:46 |
| dtantsur | yep, no way to disable it (why would you?) | 12:47 |
| *** syqian_ has quit IRC | 12:47 | |
| dtantsur | now a phylosophical question: should a reference architecture guide go to install or admin docs? | 12:48 |
| * dtantsur is never good with the difference | 12:48 | |
| asettle | dtantsur: Good question. | 12:48 |
| asettle | Hmm | 12:48 |
| asettle | Depends on the purpose. If you're including it in an overview for a new consumer, I'd go install. | 12:49 |
| asettle | If it's a high level overview, I'd go admin. | 12:49 |
| asettle | But since it *is* a reference architecture, I would also recommend it goes in the new 'reference' section. | 12:49 |
| *** amarao1 has joined #openstack-ironic | 12:49 | |
| sambetts | I think its useful from a dev perspective too | 12:50 |
| dtantsur | yeah, but the primary audience IMO is the deployers planning to roll out an ironic deployment | 12:51 |
| dtantsur | which pulls me towards the install guide more | 12:51 |
| sambetts | yeah if its recomomended deploy architechtures definatyl | 12:51 |
| asettle | dtantsur: in that case, that would make sense. | 12:51 |
| asettle | Besides, once its in one document, you can always reference it from other docs | 12:51 |
| dtantsur | true | 12:52 |
| dtantsur | especially now that it's a matter of :doc:`..` | 12:52 |
| asettle | So, you're not blocked in | 12:52 |
| asettle | Basically, ya | 12:52 |
| *** milan has joined #openstack-ironic | 12:52 | |
| *** chlong has joined #openstack-ironic | 12:53 | |
| *** rbartal has quit IRC | 12:53 | |
| *** ralonsoh_ is now known as ralonsoh | 12:54 | |
| openstackgerrit | Alexandra Settle proposed openstack/ironic master: Adding note for ironic virt driver nova-compute changes https://review.openstack.org/486601 | 12:54 |
| asettle | Okay, dtantsur ^ just included a small note. Nothing crazy. If you think it needs more, lemme know. But I figured short and sweet makes sense. | 12:55 |
| openstackgerrit | Alexandra Settle proposed openstack/ironic master: Adding note for ironic virt driver nova-compute changes https://review.openstack.org/486601 | 12:56 |
| * dtantsur clicks | 12:56 | |
| asettle | It's a real page turner | 12:58 |
| *** sasha2 has joined #openstack-ironic | 13:00 | |
| *** jroll has quit IRC | 13:03 | |
| dtantsur | asettle: some comments there, I think we should write a bit longer text, mentioning the caveat with the active nodes | 13:03 |
| *** jaypipes has joined #openstack-ironic | 13:04 | |
| *** jroll has joined #openstack-ironic | 13:04 | |
| asettle | Makes sense :) happy to iterate. I'm just the chump coming along to add it in | 13:04 |
| asettle | I'll update! | 13:04 |
| dtantsur | cheers | 13:04 |
| asettle | Oh, ftr, important renders the same, but in a different colour so it stands out | 13:04 |
| asettle | Crazy stuff | 13:04 |
| dtantsur | cool, ok | 13:04 |
| asettle | Your additions look sound :) i'll update when I get a mo. | 13:05 |
| asettle | What's the deal with not including code names? Any particular reason? | 13:05 |
| dtantsur | I remember there was some push from someone, I thought it was the docs team ;) | 13:06 |
| dtantsur | like, users make more sense out of "the Compute service" than out of "Nova" | 13:06 |
| dtantsur | which I agree FWIW | 13:06 |
| openstackgerrit | Merged openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/486380 | 13:08 |
| asettle | Oh yeah, we definitely pushed for Compute service rather than Nova, but when it comes to using code names (like something that is literal) we would use "nova-compute" | 13:08 |
| *** rloo has joined #openstack-ironic | 13:08 | |
| dtantsur | yeah, I don't have big problems with either | 13:08 |
| *** r-daneel has joined #openstack-ironic | 13:09 | |
| *** mjturek has joined #openstack-ironic | 13:12 | |
| *** r-daneel has quit IRC | 13:13 | |
| *** mjturek has quit IRC | 13:17 | |
| *** links has quit IRC | 13:20 | |
| *** zhangfei has joined #openstack-ironic | 13:30 | |
| *** mjturek has joined #openstack-ironic | 13:30 | |
| mjturek | TheJulia: sorry for not being responsive on Friday, I had to head out early. Goal is to get an update out for the boot script patch before IRC meeting | 13:31 |
| dtantsur | morning mjturek | 13:31 |
| mjturek | morning dtantsur! | 13:31 |
| TheJulia | mjturek: Good morning! no worries, I needed to just step away from things on friday. | 13:31 |
| TheJulia | mjturek: so in other words, you did me a favor | 13:32 |
| TheJulia | :) | 13:32 |
| mjturek | :) | 13:32 |
| mjturek | hope you all had a nice weekend :) | 13:32 |
| *** cdearborn has joined #openstack-ironic | 13:33 | |
| dtantsur | not bad at all. and you? | 13:35 |
| openstackgerrit | M V P Nitesh proposed openstack/python-ironicclient master: Fixes multiple chassis delete https://review.openstack.org/455662 | 13:37 |
| mjturek | coool - had a great weekend | 13:37 |
| *** Dinesh_Bhor has joined #openstack-ironic | 13:40 | |
| openstackgerrit | Merged openstack/ironic master: Start passing portgroup information to Neutron https://review.openstack.org/446763 | 13:42 |
| dtantsur | nice! ^^^ | 13:43 |
| * dtantsur wonders how far we are from this feature to be complete | 13:43 | |
| dtantsur | vdrok: ^^^? | 13:43 |
| sambetts | there is a component which is the ml2 drivers adopting the new fields, but from the ironic side I think thats it | 13:44 |
| dtantsur | awesome. that's what I thought | 13:44 |
| sambetts | there isn't a way to request port groups or not port groups from nova which I think is another goal | 13:45 |
| vdrok | ++ sambetts is right. Maybe some docs only | 13:45 |
| jlvillal | Good morning Ironic | 13:46 |
| jlvillal | If anyone has friends in Neutron, it would be great to get https://review.openstack.org/485365 merged. | 13:47 |
| patchbot | patch 485365 - neutron - Ignore cast exceptions in AgentNotifierApi | 13:47 |
| jlvillal | That fixes our grenade gate issue. | 13:47 |
| dtantsur | jlvillal: friends in neutron? lucasagomes! | 13:49 |
| * TheJulia sadly has no friends in the land of neutron | 13:49 | |
| dtantsur | :D | 13:49 |
| dtantsur | morning jlvillal | 13:49 |
| TheJulia | oh wait | 13:49 |
| jlvillal | :) | 13:49 |
| sambetts | heh I'll go and see if I can poke anyone | 13:49 |
| lucasagomes | jlvillal, dtantsur hah hey | 13:49 |
| lucasagomes | jlvillal, lemme see if I can get some eyes on it | 13:50 |
| *** Goneri has joined #openstack-ironic | 13:50 | |
| dtantsur | thanks Lucas :) | 13:50 |
| jlvillal | lucasagomes, Thanks lucasagomes :) | 13:50 |
| openstackgerrit | Merged openstack/python-ironicclient master: Updated from global requirements https://review.openstack.org/485959 | 13:51 |
| *** trown is now known as trown|brb | 13:52 | |
| *** crushil has joined #openstack-ironic | 13:56 | |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector master: POC of what the user guide index could look like https://review.openstack.org/486620 | 13:57 |
| *** trown|brb is now known as trown | 13:57 | |
| sambetts | dtantsur: ^ I've put together this to POC an idea for what the user guide might look like | 13:57 |
| sambetts | without the includes | 13:57 |
| dtantsur | cool | 13:59 |
| dtantsur | sambetts: I suggest you sync with milan and merge it into his patches | 14:00 |
| dtantsur | to avoid changing all the links twice | 14:00 |
| dtantsur | I need to wait for Jenkins, but I like your idea | 14:00 |
| * milan checks | 14:01 | |
| sambetts | spoke to the docs team quite a bit this morning about what they are expecting from the projects regarding how the pages look, and right now thats still being worked on, but the main thing I found out was that there are going to be direct links to the user guide / admin / install locations from the docs.o.o website | 14:02 |
| sambetts | for example https://docs.openstack.org/pike/admin/index.html | 14:03 |
| sambetts | provides direct links to the project admin guides | 14:03 |
| *** rama_y has joined #openstack-ironic | 14:05 | |
| xavierr | vdrok: may you add https://review.openstack.org/#/c/482209/ to your list? :) | 14:05 |
| patchbot | patch 482209 - ironic - Removes agent mixin from oneview drivers | 14:05 |
| dtantsur | sambetts: nice! | 14:06 |
| *** namnh has joined #openstack-ironic | 14:06 | |
| *** crushil has quit IRC | 14:06 | |
| milan | dtantsur, sambetts makes sense w/ that POC | 14:07 |
| milan | sambetts, feel free to squash | 14:09 |
| namnh | rloo: hi ruby, are you here? I've just left my comment on your patch after testing :) | 14:09 |
| rloo | namnh: hi, yes, I'm here. Going through my emails still. Thx for testing! I'll take a look in the next hour. | 14:09 |
| rloo | namnh: i hope :) | 14:09 |
| namnh | rloo: i will make the detail of workflow testing and send you later. the result seems OK :) | 14:10 |
| sambetts | milan: what do you think of this version of the index page too: https://review.openstack.org/#/c/486555/ | 14:10 |
| patchbot | patch 486555 - ironic-inspector - POC for what the docs main page could look like | 14:10 |
| sambetts | milan: it was based on a few of the other projects landing pages | 14:10 |
| sambetts | I can squash that in too | 14:10 |
| sambetts | to your conclusion patch | 14:10 |
| milan | sambetts, makes sense, let's do that | 14:11 |
| rloo | namnh: thank you so much, that is great news! | 14:11 |
| sambetts | milan: I'll do that and get them pushed awesome :) will do | 14:11 |
| milan | :D sambetts thanks for the help here! | 14:11 |
| sambetts | NP :) just want to get them landed | 14:11 |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi master: WIP: Apply firmware update through XCC https://review.openstack.org/485353 | 14:12 |
| *** crushil has joined #openstack-ironic | 14:12 | |
| dtantsur | first +2, nice | 14:12 |
| openstackgerrit | Alexandra Settle proposed openstack/ironic master: Adding note for ironic virt driver nova-compute changes https://review.openstack.org/486601 | 14:13 |
| namnh | rloo: According your commit message, the patch don't need to be merged so it means there is only one remaining patch set (https://review.openstack.org/#/c/408556/63) need to be merged to finish this feature in Pike | 14:13 |
| patchbot | patch 408556 - ironic - Add new dbsync command with first online data migr... | 14:13 |
| *** crushil has quit IRC | 14:13 | |
| namnh | rloo: is that right? | 14:13 |
| *** crushil has joined #openstack-ironic | 14:13 | |
| rloo | namnh: yes, that is right. It is dummy code to use for testing, not a real feature/change :) | 14:14 |
| sambetts | milan: I'll do the same thing to the install guide too if you don't mind | 14:14 |
| rloo | namnh: and yes, there is one more code patch, the dbsync command. (then a few doc-related patches) | 14:14 |
| milan | sambetts, I don't mind at all! Thanks! | 14:15 |
| *** mvpnitesh has quit IRC | 14:15 | |
| namnh | rloo: is this two path need to be updated and merged: https://review.openstack.org/#/c/419439/, https://review.openstack.org/#/c/416565/ to finish rolling ? | 14:19 |
| patchbot | patch 419439 - ironic - Rolling upgrades related dev documentation | 14:19 |
| patchbot | patch 416565 - ironic - Rolling upgrade procedure documentation | 14:19 |
| rloo | namnh: yes, not worth reviewing because they may change quite a bit. i decided to wait until all the code patches merged first, before updating them. | 14:19 |
| *** zhangfei has quit IRC | 14:19 | |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi master: WIP: Apply firmware update through XCC https://review.openstack.org/485353 | 14:21 |
| vdrok | morning jlvillal xavierr and rloo ! | 14:24 |
| dtantsur | pas-ha: hi! do you still plan on proposing the ansible driver to ironic? this can be a good topic for the PTG, and I guess a spec would help too | 14:25 |
| dtantsur | I wonder if we should start moving around now | 14:25 |
| namnh | rloo: i understood, btw, i've built a topology with two ir-api (act-act), two ir-cond (act-act) to test the patch. however, i am considering one thing about TFTP-server, can you config two TFTP-server (act-act) by mounting the two TFTP-server with a same folder that stores data? | 14:25 |
| namnh | s/you/we/g | 14:25 |
| sambetts | namnh: you can run a tftp server per conductor | 14:26 |
| dtantsur | +W on the neutron change hopefully fixing us | 14:26 |
| sambetts | namnh: then you don't need to mount a store because each conductor only deals with the data they care about | 14:27 |
| *** jcoufal has quit IRC | 14:27 | |
| jlvillal | Thanks sambetts lucasagomes. Looks like the Neutron patch was just approved :) Let's hope it merges easily. | 14:27 |
| lucasagomes | jlvillal, w00t :D glad to help as I can man | 14:27 |
| sambetts | \o/ | 14:27 |
| *** jcoufal has joined #openstack-ironic | 14:27 | |
| *** jaganathan has quit IRC | 14:29 | |
| xavierr | morning vdrok o/ | 14:29 |
| vdrok | oh, cool! | 14:30 |
| vdrok | xavierr: added :) | 14:30 |
| xavierr | vdrok: nice! | 14:30 |
| namnh | sambetts: thanks for your reply. it means when the TFTP-server has a problem on a ir-cond then we can not create BM on the conductor, is that right? | 14:30 |
| *** baoli has quit IRC | 14:31 | |
| *** crushil has quit IRC | 14:31 | |
| *** crushil has joined #openstack-ironic | 14:32 | |
| sambetts | right so a conductor might have failures but other conductors won't be affected | 14:32 |
| *** zhangfei has joined #openstack-ironic | 14:32 | |
| *** yufei has joined #openstack-ironic | 14:36 | |
| *** yufei has quit IRC | 14:36 | |
| *** yufei has joined #openstack-ironic | 14:36 | |
| *** athomas has quit IRC | 14:37 | |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector master: Introducing the user content https://review.openstack.org/480250 | 14:38 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector master: Docs migration conclusion https://review.openstack.org/480251 | 14:38 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector master: Introducing the installation content https://review.openstack.org/480249 | 14:38 |
| sambetts | milan, dtantsur ^ | 14:38 |
| *** mtanino has joined #openstack-ironic | 14:38 | |
| dtantsur | is anybody able to build ironic docs locally? I get soooo many warning (treated as errors ofc) | 14:39 |
| sambetts | dtantsur: might be that you have some of the old docs/python module referecne stuff left around | 14:39 |
| dtantsur | Warning, treated as error: | 14:40 |
| dtantsur | /home/dtantsur/Projects/ironic/ironic/api/app.py:docstring of ironic.api.app.IronicCORS:0: (ERROR/3) Error in "wsme:service" directive: | 14:40 |
| dtantsur | unknown option: "module". | 14:40 |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Add hardware type for HPE OneView https://review.openstack.org/485798 | 14:40 |
| dtantsur | does not seem so | 14:40 |
| sambetts | dtantsur: hmm I'll give it a go | 14:40 |
| *** athomas has joined #openstack-ironic | 14:41 | |
| dtantsur | ehmm, and do we even have the new install guide linked from somewhere? Oo | 14:42 |
| sambetts | dtantsur: yeah its linked out of the admin guide | 14:42 |
| dtantsur | ah, ok. I would also link it from the landing page | 14:43 |
| sambetts | dtantsur: but we need to reorganise the whole index page, /me is looking at that right now | 14:43 |
| dtantsur | cool | 14:43 |
| rloo | hi vdrok! | 14:44 |
| sambetts | docs built ok for me, my guess is that you might have doc/source/api populated with the old python module ref | 14:44 |
| sambetts | unfortunatly git won't kill that dir for you | 14:45 |
| sambetts | because it was git ignored before | 14:45 |
| *** syqian has joined #openstack-ironic | 14:45 | |
| rloo | and a ProductiveMondayMorning to one and all ironic'ers, sambetts, dtantsur, TheJulia, namnh, jlvillal, xavierr, lucasagomes, milan, mjturek | 14:45 |
| sambetts | o/ rloo | 14:45 |
| dtantsur | morning rloo :) | 14:45 |
| milan | morning rloo! :) | 14:45 |
| rloo | dtantsur: i was able to build docs locally last week, will try again today | 14:46 |
| dtantsur | sambetts: I killed it already | 14:46 |
| mjturek | gm rloo! | 14:46 |
| milan | sambetts, ack, will check w/ the CI | 14:46 |
| sambetts | just built it with latest master, and it all worked ok so :/ | 14:48 |
| sambetts | not sure what causing that error | 14:48 |
| TheJulia | good morning rloo | 14:48 |
| rloo | dtantsur: if you hadn't built docs for awhile and your local workspace has some generated docs (like in the old api or something directory) that could cause a problem. | 14:49 |
| dtantsur | yeah, but I've killed what I knew about.. | 14:49 |
| dtantsur | I guess my only option is to kill the "doc" dir, and then restore it from git | 14:49 |
| *** rpioso has joined #openstack-ironic | 14:50 | |
| *** makowals has quit IRC | 14:51 | |
| *** baoli has joined #openstack-ironic | 14:52 | |
| namnh | rloo: good morning rloo :) | 14:52 |
| *** makowals has joined #openstack-ironic | 14:54 | |
| namnh | rloo: oh, I see there is only short time for Pike, do you think the feature can be finished on this cycle? | 14:54 |
| rloo | namnh: yup, i think it will. but there's always the possibility that it won't. | 14:54 |
| rloo | namnh: it is a high/essential priority for ironic | 14:55 |
| rloo | namnh: oh, also forgot to mention, that is a nice test setup | 14:55 |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Remove check if enabled for OneViewInspect https://review.openstack.org/486192 | 14:55 |
| openstackgerrit | Fellype Cavalcante proposed openstack/ironic master: Add a flag to always peform persistent boot on PXE interface https://review.openstack.org/483430 | 14:56 |
| namnh | rloo: thanks, in this test, I have changed the step by step as your email and update the command to update node :) i will send the result ( video if necessary) later | 14:58 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: [doc] update ironic's landing page https://review.openstack.org/485820 | 14:58 |
| *** baoli has quit IRC | 14:59 | |
| *** zhangfei has quit IRC | 14:59 | |
| sambetts | rloo: I didn't realise you were working on ^ :/ | 15:02 |
| rloo | sambetts: rolling upgrades? yeah, i took over when xek couldn't work on it any more. | 15:03 |
| sambetts | rloo: no the doc update | 15:03 |
| rloo | sambetts: oh. i didn't want to work on it. i didn't see any patches related to it and it was driving me crazy | 15:03 |
| rloo | sambetts: was that on your plate to do? | 15:03 |
| sambetts | rloo: I didn't see any patches relating to it either so I starting looking into it this morning XD | 15:04 |
| rloo | sambetts: ha ha | 15:04 |
| rloo | sambetts: great minds and all ;) | 15:04 |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: bifrost-create-vm-nodes: Do not use KVM VMs by default https://review.openstack.org/486645 | 15:04 |
| rloo | sambetts: i think there's still more work to do but I didn't want to spend too much time on it. | 15:05 |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Remove check if enabled for OneViewInspect https://review.openstack.org/486192 | 15:07 |
| *** crushil has quit IRC | 15:07 | |
| *** crushil has joined #openstack-ironic | 15:08 | |
| *** yufei has quit IRC | 15:08 | |
| sambetts | rloo: I spoke to the docs team a little bit this morning about what they want in terms of design/look from the index/landing page from projects, there are about 3 different ways that projects are doing it, yours is inline with projects like neutron where there is a heading for each sub-guide, however there are some other projects that have simplified the landing page a bit and I kind of prefer | 15:09 |
| sambetts | that | 15:09 |
| rloo | sambetts: how about 1. we land my patch so people can see *that* way, then you submit a separate patch to do it the way you prefer. so we can compare. | 15:10 |
| *** baoli has joined #openstack-ironic | 15:10 | |
| rloo | sambetts: hard to compare with what we have now :-( | 15:10 |
| *** rbartal has joined #openstack-ironic | 15:10 | |
| rloo | sambetts: i think i know what you mean; i went with the heading for each sub-guide cuz that was more in line with what we originally had before the doc migration. but i like the streamlined approach too. | 15:11 |
| lucasagomes | rloo, good morning :D (sorry i was in a call) | 15:11 |
| dtantsur | rloo, sambetts: ok, I removed 'doc" and restored it from git, I've rebuilt the tox env and I've removed all pyc files. still fails. my luck? :( | 15:11 |
| sambetts | ;/ | 15:11 |
| rloo | sambetts: the streamlined approach seemed to 'need' more bikeshedding, opinions. my goal was just to clean up the page so it made sense. | 15:11 |
| rloo | dtantsur: what is failing? | 15:11 |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Clean up and simplify doc/source/index https://review.openstack.org/486649 | 15:12 |
| sambetts | rloo: ^ that is my version | 15:12 |
| rloo | lucasagomes: no worries, I debating about being nice to you anyway cuz you know, you're only part ironic now ;) | 15:12 |
| sambetts | dtantsur: super weird... | 15:12 |
| rloo | sambetts: no jenkins yet... | 15:12 |
| dtantsur | rloo: a lot of sphinx warnings, similar to one I pasted above | 15:12 |
| lucasagomes | rloo, lol I still think ironic is the best project in OS btw (-: | 15:13 |
| rloo | dtantsur: you got the latest sphinx and whatever packages too? | 15:13 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Fix small issues in the installation documentation https://review.openstack.org/486651 | 15:13 |
| dtantsur | rloo, TheJulia: I owed you some nits ^^^ | 15:13 |
| rloo | lucasagomes: in that case, i'll continue to be nice to you :D | 15:13 |
| sambetts | yeah maybe rm -rf .tox/docs? | 15:13 |
| lucasagomes | \o/ | 15:13 |
| TheJulia | dtantsur: <3 | 15:13 |
| dtantsur | I cannot verify this patch, because see above :( | 15:13 |
| dtantsur | sambetts: isn't it what 'tox -edocs -r' does? | 15:13 |
| TheJulia | dtantsur: acknowledged | 15:14 |
| *** rama_y has quit IRC | 15:14 | |
| sambetts | ah your using -r, yeah that'd do the same | 15:14 |
| sambetts | your git tree clean? | 15:14 |
| rloo | dtantsur: i don't know much about the docs, looked at your error and it doesn't ring a bell. | 15:15 |
| rloo | dtantsur: wrt ansible driver, i believe there is already a spec for it | 15:15 |
| dtantsur | yep... | 15:15 |
| dtantsur | rloo: ah, right. I guess it needs reviving | 15:15 |
| rloo | dtantsur: we may not need to discuss at ptg. depends on state of spec. and would need to recall what the issues were with it. | 15:17 |
| dtantsur | right. /me adds the topic | 15:17 |
| rloo | dtantsur: i wish at the ptg, we could get commitment from companies wrt bodies to be there for the cycle. | 15:18 |
| dtantsur | weeeellll | 15:18 |
| dtantsur | yeah, I wish so too :) | 15:18 |
| rloo | dtantsur: just a wish. we have no control there! | 15:18 |
| * TheJulia steps away to have a real lunch | 15:21 | |
| *** chlong_ has joined #openstack-ironic | 15:21 | |
| *** chlong has quit IRC | 15:24 | |
| openstackgerrit | Merged openstack/ironic master: Updated from global requirements https://review.openstack.org/486319 | 15:25 |
| *** yufei has joined #openstack-ironic | 15:26 | |
| dtantsur | rloo: if you have a minute, mind checking https://review.openstack.org/#/c/484866/ please? | 15:27 |
| patchbot | patch 484866 - python-ironicclient - Log warning when API version is not specified for ... | 15:27 |
| rloo | dtantsur: ah, forgot to look at that. (maybe hoping that it would have merged by now, heh.) ok, will look in a few min. | 15:28 |
| *** dsneddon_ is now known as dsneddon | 15:33 | |
| *** rcernin has quit IRC | 15:34 | |
| dtantsur | sambetts: do you think you'll be able to push the IPA API versioning in this cycle? | 15:37 |
| *** lihi has quit IRC | 15:37 | |
| *** lihi has joined #openstack-ironic | 15:38 | |
| *** ccamacho has quit IRC | 15:41 | |
| sambetts | dtantsur: I would like too, I need to rebase and ensure its up to date, because there have been a number of patches touch the API/RPC API recently which has merged conflicted | 15:42 |
| *** namnh has quit IRC | 15:42 | |
| dtantsur | ack | 15:44 |
| *** vmud213 has joined #openstack-ironic | 15:47 | |
| *** crushil has quit IRC | 15:47 | |
| rloo | dtantsur: it is just missing a period in one of the strings. can you edit quickly? | 15:48 |
| *** skazi has quit IRC | 15:49 | |
| dtantsur | lemme see | 15:50 |
| dtantsur | rloo: maybe a follow-up since it was approved? | 15:51 |
| *** rama_y has joined #openstack-ironic | 15:51 | |
| vdrok | oh, was just going to look | 15:52 |
| rloo | dtantsur: grr, that jlvillal guy. | 15:52 |
| dtantsur | rloo: I'll also expand the docs of the params, so I guess it's worth a follow-up anyway | 15:52 |
| rloo | dtantsur: ok thx | 15:53 |
| *** vmud213 has quit IRC | 15:57 | |
| *** vmud213 has joined #openstack-ironic | 15:57 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Refactoring the firewall https://review.openstack.org/471831 | 15:58 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Introducing a dnsmasq PXE filter driver https://review.openstack.org/466448 | 15:58 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: TEST check dnsmasq PXE filter in devstack https://review.openstack.org/468712 | 15:58 |
| *** rcernin has joined #openstack-ironic | 15:58 | |
| *** jjung_ has joined #openstack-ironic | 15:59 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Follow up to the API version warning patches https://review.openstack.org/486677 | 15:59 |
| dtantsur | rloo: ^^^ | 15:59 |
| *** marios has quit IRC | 16:01 | |
| jlvillal | Sweet! Neutron patch merged, Ironic grenade should be much more happy now :) | 16:01 |
| *** baoli has quit IRC | 16:02 | |
| dtantsur | RECHECK ALL DE THINGZZ | 16:02 |
| *** baoli has joined #openstack-ironic | 16:02 | |
| sambetts | \o/ | 16:03 |
| sambetts | *DDOS's jenkins* | 16:03 |
| *** mjura has quit IRC | 16:03 | |
| *** lucasagomes is now known as lucas-afk | 16:04 | |
| *** ijw has joined #openstack-ironic | 16:05 | |
| sambetts | milan: main reason for adding the toctree in the admin index was because I expect we'll need to put other things in there just not sure what yet :/ | 16:05 |
| *** vmud213 has quit IRC | 16:05 | |
| milan | sambetts, ack +2 then | 16:06 |
| sambetts | \o/ | 16:06 |
| *** vmud213 has joined #openstack-ironic | 16:06 | |
| openstackgerrit | Sam Betts proposed openstack/ironic master: Receive and store agent version on heartbeat https://review.openstack.org/364861 | 16:06 |
| *** baoli has quit IRC | 16:07 | |
| *** syqian_ has joined #openstack-ironic | 16:09 | |
| *** ijw has quit IRC | 16:10 | |
| *** jjung_ has quit IRC | 16:10 | |
| *** mguz_ has quit IRC | 16:10 | |
| *** syqian has quit IRC | 16:11 | |
| *** chlong__ has joined #openstack-ironic | 16:13 | |
| *** chlong_ has quit IRC | 16:13 | |
| openstackgerrit | Merged openstack/python-ironicclient master: Log warning when API version is not specified for the ironic tool https://review.openstack.org/484866 | 16:15 |
| dtantsur | \o/ | 16:17 |
| * dtantsur builds python-netmiko for Fedora, because of networking-generic-switch :) | 16:23 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: PGSQL: create Enum before using https://review.openstack.org/472725 | 16:23 |
| *** nicodemos has joined #openstack-ironic | 16:25 | |
| * sambetts voms | 16:26 | |
| sambetts | ;) | 16:26 |
| openstackgerrit | Ilya Etingof proposed openstack/virtualbmc master: Fixes cli parser when run under Python 3.3+ https://review.openstack.org/486686 | 16:26 |
| openstackgerrit | Sam Betts proposed openstack/ironic-python-agent master: Include IPA Version during heartbeat https://review.openstack.org/364834 | 16:26 |
| *** samodid has quit IRC | 16:28 | |
| *** yufei has quit IRC | 16:29 | |
| jlvillal | Anyone have an idea on how to get rid of the warning messages: RuntimeWarning: You have iterated over the result of pkg_resources.parse_version. when doing a tox run? | 16:30 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector master: Introducing the user content https://review.openstack.org/480250 | 16:31 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector master: Docs migration conclusion https://review.openstack.org/480251 | 16:31 |
| jlvillal | I think it is caused by this line (not 100% sure on that). https://github.com/openstack/ironic/blob/11e79b921457a9b239bf0d771716a089c0683b69/ironic/tests/unit/conductor/mgr_utils.py#L45 | 16:31 |
| *** e0ne has quit IRC | 16:31 | |
| *** baoli has joined #openstack-ironic | 16:33 | |
| *** srart has joined #openstack-ironic | 16:34 | |
| *** ricardoas has quit IRC | 16:36 | |
| sambetts | jlvillal: trying that out locally in the python command line doesn't seem to yeild the same warning :/ | 16:38 |
| jlvillal | sambetts: When I do my py35 tox run I see a ton of warning messages | 16:38 |
| jlvillal | RuntimeWarning: You have iterated over the result of pkg_resources.parse_version. This is a legacy behavior which is inconsistent with the new version class introduced in setuptools 8.0. In most cases, conversion to a tuple is unnecessary. For comparison of versions, sort the Version instances directly. If you have another use case requiring the tuple, please file a bug with the setuptools project describing that need.' | 16:38 |
| sambetts | yeah I see the warnings in the ironic logs, but not if I do it in the python command line (although maybe I need to setup logging) | 16:39 |
| *** chlong_ has joined #openstack-ironic | 16:40 | |
| *** crushil has joined #openstack-ironic | 16:40 | |
| *** chlong__ has quit IRC | 16:41 | |
| vdrok | jlvillal: it might also be connected with usage of inspect | 16:42 |
| vdrok | not sure how to fix this. as the libs we use may also be causing this | 16:42 |
| sambetts | this is what I did in the python interpretor http://paste.openstack.org/show/616336/ | 16:42 |
| jlvillal | vdrok: I saw some warnings about inspect also, but different. I think different, not sure. | 16:42 |
| sambetts | yeah its hard to tell if these logs are coming from ironic use of stuff or libs use of stuff | 16:43 |
| jlvillal | vdrok: Adding a traceback to the pkg_resources file | 16:43 |
| jlvillal | Here is a pastebin: http://paste.openstack.org/show/616337/ | 16:44 |
| * jlvillal looks at it | 16:44 | |
| sambetts | hmmm looks like versionutils is the cause | 16:44 |
| sambetts | from oslo utils | 16:45 |
| jlvillal | sambetts: +1 | 16:45 |
| jlvillal | dansmith: Have you noticed any warning like: http://paste.openstack.org/show/616337/ ? | 16:45 |
| sambetts | jlvillal: that traceback thing is a super cool trick! I've not seen that before | 16:46 |
| jlvillal | dansmith: Asking you as the the versioned objects god :) | 16:46 |
| jlvillal | sambetts: heh. Yeah, I just added it to the code. Not sure why they didn't do it in the first place. | 16:46 |
| jlvillal | sambetts: Kind of hard to figure out where the heck the error is being generated from without it. | 16:46 |
| *** jjung has quit IRC | 16:47 | |
| sambetts | +1000 so many deprecation warning are like that... | 16:47 |
| sambetts | its like yes I can see its deprecated but where is it being used!? | 16:47 |
| *** stendulker has joined #openstack-ironic | 16:47 | |
| sambetts | then thats always followed by "oh look its not in my code its in X library" XD | 16:47 |
| jlvillal | heh :) | 16:48 |
| anupn | Hi milan: wrt dnsmasq filter driver, do you think resetting iptables be done? As in my case there was a rule in iptables under inspector chain which was blocking packets getting received on dhcp | 16:48 |
| sambetts | jlvillal: there are a whole bunch of deprecation warnings in ironics test output, the sqlalchemy ones I tried to tackle at one point but didn't get much help from oslo.db what the new way to do it was | 16:49 |
| vdrok | sambetts: I remember only this one https://review.openstack.org/407625, are there more? | 16:52 |
| patchbot | patch 407625 - ironic - Switch to new location for oslo.db test cases (MERGED) | 16:52 |
| * vdrok checks too | 16:52 | |
| *** syqian_ has quit IRC | 16:52 | |
| *** kaifeng has joined #openstack-ironic | 16:52 | |
| sambetts | vdrok: there is stuff to do with the fixtures | 16:53 |
| *** syqian_ has joined #openstack-ironic | 16:53 | |
| sambetts | I'm not sure I ever finished the patch / pushed one because I couldn't every work out how to make it work | 16:53 |
| sambetts | there are these ones now /home/sam/Work/ironic/.tox/py27/local/lib/python2.7/site-packages/oslo_db/tests/sqlalchemy/__init__.py:18: DeprecationWarning: Function 'oslo_db.sqlalchemy.test_base.optimize_db_test_loader()' has moved to 'oslo_db.sqlalchemy.test_fixtures.optimize_package_test_loader()' | 16:55 |
| sambetts | load_tests = test_base.optimize_db_test_loader(__file__) | 16:55 |
| sambetts | 16:55 | |
| openstackgerrit | Michael Turek proposed openstack/ironic master: Generate iPXE boot script on start up https://review.openstack.org/484032 | 16:55 |
| fellypefca | dtantsur Hey, I was looking your patch (https://review.openstack.org/#/c/416232/) and based on it I was implementing our patch of Driver Composition. | 16:55 |
| patchbot | patch 416232 - ironic - Do not fail in Inspector.__init__ if [inspector]en... (MERGED) | 16:55 |
| sambetts | this was the one that caused me most pain and couldn't fix /home/sam/Work/ironic/.tox/py27/local/lib/python2.7/site-packages/oslo_db/sqlalchemy/enginefacade.py:340: OsloDBDeprecationWarning: EngineFacade is deprecated; please use oslo_db.sqlalchemy.enginefacade | 16:55 |
| fellypefca | dtantsur But I got a review saying that self.inspect = inspect.OneViewInspect() in classic drivers will break on our Agent Driver. What you think about that? https://review.openstack.org/#/c/486192 | 16:56 |
| patchbot | patch 486192 - ironic - Remove check if enabled for OneViewInspect | 16:56 |
| *** derekh has quit IRC | 16:56 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Guide on upgrading to hardware types https://review.openstack.org/486696 | 16:57 |
| dansmith | jlvillal: the parse_version warning? | 16:57 |
| dtantsur | fellypefca: I'll put it on my queue (nearly meeting time now) | 16:58 |
| mjturek | TheJulia: waaait - https://review.openstack.org/#/c/484032/4/ironic/drivers/modules/agent.py@455 won't this line still cause the problem with cleaning? | 16:58 |
| patchbot | patch 484032 - ironic - Generate iPXE boot script on start up | 16:58 |
| fellypefca | dtantsur ok! thanks | 16:58 |
| jlvillal | dansmith: Yeah. | 16:58 |
| jlvillal | dansmith: RuntimeWarning: You have iterated over the result of pkg_resources.parse_version. | 16:59 |
| dansmith | jlvillal: I dunno what that means.. I guess it's just because we use that version util? | 16:59 |
| milan | anupn, I need to doublecheck what's in the iptables filter but the dnsmasq filter should better not touch iptables at all | 17:00 |
| jlvillal | dansmith: I'm not sure either. We just see a bunch of deprecation warnings in our tox runs. Seems to be from versioned objects | 17:00 |
| milan | * what's going on in the iptables driver | 17:00 |
| jlvillal | dansmith: Think of it as a heads up | 17:00 |
| dtantsur | meeeeeeting tiiiiime | 17:00 |
| anupn | milan, cool. | 17:00 |
| milan | dtantsur, ack ;) | 17:00 |
| TheJulia | mjturek: I think it will derail it | 17:01 |
| sambetts | dansmith, jlvillal, I think its because the pkg_resource lib now supports comparitors directly instead of having to break apart the version objects it returns like version[0] >= otherversion[0] | 17:01 |
| sambetts | here: https://github.com/openstack/oslo.utils/blob/master/oslo_utils/versionutils.py#L45 | 17:02 |
| *** ralonsoh has quit IRC | 17:02 | |
| mjturek | TheJulia: so we should still have the check in should_write_image that you added in https://review.openstack.org/#/c/485812/6/ironic/drivers/modules/storage/cinder.py@365 | 17:02 |
| patchbot | patch 485812 - ironic - Correct BFV PXE logic and debugging | 17:02 |
| TheJulia | mjturek: no, I think it is that we don't need the check on 455 and 456 because the check in the if we are deploying where we do need to have the logic | 17:03 |
| *** sasha2 has quit IRC | 17:05 | |
| *** sasha2 has joined #openstack-ironic | 17:05 | |
| *** fragatina has quit IRC | 17:06 | |
| *** rbartal has quit IRC | 17:06 | |
| *** chlong_ has quit IRC | 17:08 | |
| *** ricardoas has joined #openstack-ironic | 17:08 | |
| mjturek | TheJulia: if we remove that check though, we'll hit prepare_ramdisk during boot_from_volume. Maybe I'm misunderstanding but in the bfv case, I thought we only wanted prepare_ramdisk when cleaning | 17:09 |
| TheJulia | we shouldn't though... | 17:09 |
| *** harlowja has joined #openstack-ironic | 17:09 | |
| TheJulia | oh | 17:09 |
| TheJulia | you mean a deployed active instance if deploy.prepare() is called again during takeover? | 17:09 |
| *** chlong_ has joined #openstack-ironic | 17:10 | |
| dtantsur | easy thing to approve (I think): https://review.openstack.org/#/c/473913/ | 17:12 |
| patchbot | patch 473913 - ironic - py3.5:Workaround fix for forcing virtualbmc instal... | 17:12 |
| TheJulia | mjturek: line 445 should take care of during deploy, boot.prepare_instance will be called regardless due to active state | 17:13 |
| *** ricardoas has quit IRC | 17:14 | |
| *** ricardoas has joined #openstack-ironic | 17:14 | |
| TheJulia | mjturek: totally happy to go high bandwidth on a call and sort through this after the ironic meeting | 17:14 |
| mjturek | ahhhhh I think I see now | 17:15 |
| mjturek | but yeah that might be useful | 17:15 |
| * TheJulia moves laptop to the espresso machine to begin it's start-up sequence | 17:16 | |
| openstackgerrit | Ilya Etingof proposed openstack/virtualbmc master: Fixes cli parser when run under Python 3.3+ https://review.openstack.org/486686 | 17:17 |
| *** stendulker has quit IRC | 17:17 | |
| *** stendulker has joined #openstack-ironic | 17:18 | |
| *** ChubYann has joined #openstack-ironic | 17:18 | |
| *** trown is now known as trown|lunch | 17:18 | |
| edleafe | General question: how often would an operator typically restart the Nova compute service that handles Ironic? | 17:19 |
| *** baoli has quit IRC | 17:20 | |
| *** baoli has joined #openstack-ironic | 17:23 | |
| *** samodid has joined #openstack-ironic | 17:25 | |
| dtantsur | on upgrades/reconfiguration? | 17:26 |
| edleafe | dtantsur: Sure, and also during normal operation | 17:28 |
| sambetts | rloo: btw my ironic index patch has render now http://docs-draft.openstack.org/49/486649/1/check/gate-ironic-docs-ubuntu-xenial/aaeeea1//doc/build/html/ | 17:29 |
| *** mguz has joined #openstack-ironic | 17:30 | |
| *** sambetts_ has joined #openstack-ironic | 17:31 | |
| *** stendulker has quit IRC | 17:32 | |
| *** stendulker has joined #openstack-ironic | 17:32 | |
| * sambetts --> going mobile | 17:32 | |
| *** sambetts is now known as sambetts|afk | 17:33 | |
| *** fragatin_ has joined #openstack-ironic | 17:34 | |
| *** fragatin_ has quit IRC | 17:34 | |
| *** fragatin_ has joined #openstack-ironic | 17:34 | |
| *** fragatina has joined #openstack-ironic | 17:37 | |
| *** fragatina has quit IRC | 17:37 | |
| *** baoli has quit IRC | 17:37 | |
| *** fragatin_ has quit IRC | 17:37 | |
| *** fragatina has joined #openstack-ironic | 17:38 | |
| jroll | edleafe: when I ran it in prod, basically only for upgrades, unless troubleshooting | 17:38 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Updated from global requirements https://review.openstack.org/486335 | 17:42 |
| *** baoli has joined #openstack-ironic | 17:45 | |
| *** edleafe_ has joined #openstack-ironic | 17:46 | |
| *** amarao1 has quit IRC | 17:46 | |
| *** amarao1 has joined #openstack-ironic | 17:46 | |
| edleafe | jroll: thanks | 17:47 |
| jroll | np | 17:49 |
| *** baoli has quit IRC | 17:50 | |
| *** amarao1 has quit IRC | 17:51 | |
| *** edleafe- has joined #openstack-ironic | 17:53 | |
| *** edleafe has quit IRC | 17:55 | |
| *** edleafe_ has quit IRC | 17:56 | |
| *** edleafe- is now known as edleafe | 17:56 | |
| TheJulia | mjturek: meeting over! | 18:00 |
| TheJulia | preferred communication method? | 18:00 |
| *** stendulker has quit IRC | 18:00 | |
| mjturek | I'll shoot you my conference #, wanna use that? | 18:01 |
| vdrok | thanks all, /me goes home | 18:01 |
| vdrok | good night! | 18:01 |
| mjturek | good night vdrok! | 18:01 |
| TheJulia | mjturek: that or hangouts. Whatever works for you | 18:01 |
| TheJulia | just... not zoom. It does not work well for me | 18:02 |
| *** milan has quit IRC | 18:02 | |
| mjturek | lolol fair enough :) | 18:02 |
| rloo | good night vdrok! | 18:03 |
| *** e0ne has joined #openstack-ironic | 18:04 | |
| openstackgerrit | Xavier proposed openstack/ironic master: Removes agent mixin from oneview drivers https://review.openstack.org/482209 | 18:04 |
| dtantsur | good night! | 18:05 |
| *** dtantsur is now known as dtantsur|afk | 18:05 | |
| *** vmud213 has quit IRC | 18:05 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: [doc] update ironic's landing page https://review.openstack.org/485820 | 18:06 |
| kaifeng | TheJulia: hi julia, i have a question just now :) it's business hours? | 18:06 |
| openstackgerrit | Merged openstack/ironic master: Adding note for ironic virt driver nova-compute changes https://review.openstack.org/486601 | 18:08 |
| kaifeng | TheJulia: according to the spec, we don't support multipath, it's a constraints or just haven't implemented in tis intial | 18:08 |
| kaifeng | this initial patches | 18:08 |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Migrate node-details controller tests to new framework https://review.openstack.org/486711 | 18:11 |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi master: WIP: Apply firmware update through XCC https://review.openstack.org/485353 | 18:11 |
| *** ricardoas has left #openstack-ironic | 18:11 | |
| *** skazi has joined #openstack-ironic | 18:12 | |
| * kaifeng wonders if hangout, wait.. | 18:13 | |
| *** rcernin has quit IRC | 18:16 | |
| *** skazi has quit IRC | 18:17 | |
| *** trown|lunch is now known as trown | 18:17 | |
| TheJulia | kaifeng: sorry, was on the phone with mjturek | 18:20 |
| *** sauloaislan has joined #openstack-ironic | 18:21 | |
| TheJulia | kaifeng: so we can't explicitly support multiparty so we state it as a contextual constraint. We can pass data, but we can make NO promises to cinder that the guest OS knows how to do multipathing correctly | 18:21 |
| sauloaislan | morning ironic | 18:22 |
| TheJulia | kaifeng: btw, I'm in UTC-4 until late August when I'll be in denver for the PTG. Pretty much anytime I'm not asleep or on a plane, is okay to ping me with questions. :) | 18:23 |
| TheJulia | good morning sauloaislan | 18:24 |
| *** skazi has joined #openstack-ironic | 18:26 | |
| sauloaislan | TheJulia: mjturek: I'm trying to develop a PoC for a custom implementation of storage interface. but when deploying I'm getting an error when calling prepare ramdisk here https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent.py#L467 | 18:31 |
| sauloaislan | I suspect of this https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent.py#L466 | 18:31 |
| openstackgerrit | Julia Kreger proposed openstack/python-ironicclient master: Follow up to the API version warning patches https://review.openstack.org/486677 | 18:31 |
| *** baoli has joined #openstack-ironic | 18:32 | |
| TheJulia | rloo: ^^^ I updated dtantsur|afk's revision to address your nit, since I agree with it. | 18:32 |
| sauloaislan | I think this patch may introduced an error https://github.com/openstack/ironic/commit/2c18f22f9976870b71bfd1e471afa18fd50a7a25 | 18:33 |
| sauloaislan | TheJulia: mjturek: could you take a look? :) | 18:33 |
| rloo | TheJulia: ok, thx. I'll review. | 18:33 |
| TheJulia | sauloaislan: looking and pondering | 18:33 |
| mjturek | sauloaislan: this is actually getting reverted | 18:33 |
| sauloaislan | mjturek: because of this? | 18:34 |
| TheJulia | sauloaislan: similar issue, specifically with the transition from ACTIVE -> DELETED -> CLEANING | 18:34 |
| mjturek | sauloaislan: not specifically, the idea there was that if you're hitting prepare_ramdisk when booting from volume you'd return from the method before any deploy_opts were used | 18:35 |
| mjturek | but yeah - definitely related | 18:36 |
| *** baoli has quit IRC | 18:37 | |
| sauloaislan | mjturek: that's makes sense. I need to keep track of this, internal stuff.. :) | 18:37 |
| sauloaislan | ty TheJulia and mjturek | 18:37 |
| kaifeng | TheJulia: Thank you, does that mean, it's because we can't support configdrive now, and guest os have no means to do post operations? | 18:37 |
| mjturek | np sauloaislan :) | 18:37 |
| TheJulia | kaifeng: partly. In all honesty, multiparty support needs to best started prior to a configuration drive being read. One thing we've done is if we have multiple connector records, we treat it as supporting multipath and tell cinder that, its just we can't make any commitments of actual guest OS having multipathd or the windows equivalent running | 18:39 |
| TheJulia | ugh | 18:40 |
| TheJulia | autocorrect, why do you hate me | 18:40 |
| sauloaislan | mjturek: do you have the link for the revert on gerrit? | 18:40 |
| TheJulia | s/multiparty/multipath/ | 18:40 |
| mjturek | https://review.openstack.org/#/c/484032/ | 18:40 |
| TheJulia | sauloaislan: mjturek is working on that patch right now | 18:40 |
| patchbot | patch 484032 - ironic - Generate iPXE boot script on start up | 18:40 |
| sauloaislan | great o/ | 18:41 |
| mjturek | TheJulia: I hate to say it but the change we discussed on the patch (adding node.provision_state == DEPLOYING), taking a closer look seems like it wouldn't do anything. If the state is deploying, it would enter the first part of the if block. | 18:43 |
| mjturek | doh | 18:43 |
| TheJulia | mjturek: yes, the conditional check could be moved up as well :) | 18:43 |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Add property-collection-editor directive https://review.openstack.org/481152 | 18:43 |
| TheJulia | the check above starts with if, if memory serves | 18:43 |
| kaifeng | TheJulia: I heard that ipxe supports multipath, so if we generate correct template, it will be passed to guest os, is that correct? | 18:44 |
| TheJulia | the one before that is a separate if | 18:44 |
| TheJulia | kaifeng: so, that support is relatively new. In theory yes. In the mean time, what we do is we record all targets we receive, attach them as sandisks and then sanboot the very first one. | 18:44 |
| TheJulia | kaifeng: Doing ^^^ means that we basically leave it up to the guest to sort out, in theory | 18:45 |
| *** ricardoas has joined #openstack-ironic | 18:46 | |
| TheJulia | kaifeng: I think we would be more than happy to accept patches for updated ipxe behavior, although we would likely want to retain older logic from prior to april-2017 (I think that is when they added that to ipxe) | 18:46 |
| TheJulia | and even then, the statement in the spec will still stand, because we have no control of what is running inside the guest OS | 18:46 |
| kaifeng | so it's a can do in theory, needs time to shape it out :) | 18:47 |
| TheJulia | kaifeng: oh, if the conditions are right, absolutely can be done, it is just ironic as a service can't make promises of unknown disk images | 18:47 |
| TheJulia | It would be something that we have to explicitly state as "Here is how this works, if you need to make use of this, consider x, y, and z" | 18:48 |
| *** etingof has quit IRC | 18:51 | |
| *** fellypefca has quit IRC | 18:51 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/486320 | 18:51 |
| kaifeng | Glad to know that, thanks for your kind answers :) | 18:51 |
| TheJulia | kaifeng: anytime :) | 18:51 |
| kaifeng | TheJulia: I have to say good bye now, wishing you a nice day :) | 18:54 |
| TheJulia | Have a wonderful day! | 18:54 |
| *** kaifeng has quit IRC | 18:54 | |
| openstackgerrit | Michael Turek proposed openstack/ironic master: Generate iPXE boot script on start up https://review.openstack.org/484032 | 18:57 |
| rloo | jlvillal: i hope that neutron bug fixed the gate; I've updated our whiteboard :) | 19:00 |
| *** skazi has quit IRC | 19:12 | |
| *** racedo has quit IRC | 19:18 | |
| *** racedo has joined #openstack-ironic | 19:18 | |
| *** racedo has quit IRC | 19:19 | |
| *** e0ne has quit IRC | 19:21 | |
| *** baoli has joined #openstack-ironic | 19:22 | |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi master: Apply firmware update through XCC https://review.openstack.org/485353 | 19:28 |
| rloo | since pyghmi isn't under ironic umbrella, anyone know how to stop those ^^ | 19:31 |
| mjturek | TheJulia: in case you missed it I did upload a rev on 484032 | 19:35 |
| jlvillal | rloo: I think it did. I had created 3 test patches to test it and ran each one twice. So a total of six runs and zero grenade failures. | 19:36 |
| * jlvillal is reminded to abandon those test patches | 19:36 | |
| jlvillal | I was almost tempted to create a bot that would just keep doing recheck if it saw a grenade failure :) | 19:37 |
| rloo | jlvillal: i haven't seen any of those failures since so i think it is good! | 19:37 |
| rloo | jlvillal: heh | 19:37 |
| jlvillal | rloo: Awesome! | 19:37 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP/DNM: Try to test boot-from-volume https://review.openstack.org/485349 | 19:41 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Tempest scenario test for boot-from-volume https://review.openstack.org/472740 | 19:41 |
| TheJulia | rloo: I think I remember how to fix that | 19:42 |
| TheJulia | mjturek: replied, and ^^^ | 19:42 |
| TheJulia | for testing | 19:42 |
| TheJulia | rloo: jlvillal: reviews would be appreciated on 484032 :) | 19:43 |
| *** samodid has quit IRC | 19:43 | |
| jlvillal | TheJulia: Will do. Need to finish an internal task here, but after that | 19:47 |
| TheJulia | 10-4 | 19:47 |
| * jlvillal is very glad the gate is working. After having the one patch take 12 rechecks to get merged :( | 19:47 | |
| TheJulia | rloo: found it, I'll submit a change shortly | 19:49 |
| rloo | TheJulia: i look forward to it :) | 19:50 |
| *** jcoufal has quit IRC | 19:54 | |
| *** etingof has joined #openstack-ironic | 19:56 | |
| sauloaislan | mjturek: Hey, I just tested your last patch-set of this https://review.openstack.org/#/c/484032/ and worked in my scenario. | 19:57 |
| patchbot | patch 484032 - ironic - Generate iPXE boot script on start up | 19:57 |
| TheJulia | \o/ | 19:59 |
| TheJulia | sauloaislan: awesome to hear, would you comment on the patch? I setup the tempest test addition as well as our test patch to follow it | 19:59 |
| TheJulia | rloo: https://review.openstack.org/486741 | 20:01 |
| patchbot | patch 486741 - openstack-infra/project-config - Remove non-ironic governed gerrit annoucements | 20:01 |
| *** e0ne has joined #openstack-ironic | 20:01 | |
| *** chihhsin has quit IRC | 20:05 | |
| *** patriciadomin has quit IRC | 20:05 | |
| *** Sukhdev has joined #openstack-ironic | 20:05 | |
| *** crushil has quit IRC | 20:08 | |
| *** patriciadomin has joined #openstack-ironic | 20:09 | |
| mjturek | awesome! thanks sauloaislan | 20:12 |
| TheJulia | And good news too http://logs.openstack.org/49/485349/3/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-ubuntu-xenial/5a66f6f/logs/ironic-bm-logs/node-0_no_ansi_2017-07-24-20:05:52.txt.gz | 20:12 |
| TheJulia | It passed the tempest job | 20:12 |
| mjturek | \o/ | 20:12 |
| * TheJulia slides a beer over to mjturek | 20:12 | |
| TheJulia | I'm super tempted to address the nit on the tempest job now | 20:13 |
| mjturek | :) | 20:13 |
| *** rpioso has quit IRC | 20:21 | |
| TheJulia | Well, once the current CI runs are done | 20:26 |
| openstackgerrit | Merged openstack/ironic master: Refactor VIFPortIDMixin: rename https://review.openstack.org/486124 | 20:30 |
| *** baoli has quit IRC | 20:31 | |
| *** chihhsin has joined #openstack-ironic | 20:32 | |
| *** rpioso has joined #openstack-ironic | 20:35 | |
| openstackgerrit | Merged openstack/ironic master: py3.5:Workaround fix for forcing virtualbmc installation with pip2 https://review.openstack.org/473913 | 20:36 |
| *** baoli has joined #openstack-ironic | 20:44 | |
| *** baoli has quit IRC | 20:45 | |
| *** baoli has joined #openstack-ironic | 20:45 | |
| *** crushil has joined #openstack-ironic | 20:48 | |
| openstackgerrit | Merged openstack/pyghmi master: Apply firmware update through XCC https://review.openstack.org/485353 | 20:49 |
| *** e0ne has quit IRC | 20:50 | |
| *** Goneri has quit IRC | 20:52 | |
| *** trown is now known as trown|outtypewww | 20:54 | |
| *** jkilpatr has quit IRC | 21:04 | |
| *** crushil has quit IRC | 21:05 | |
| *** crushil has joined #openstack-ironic | 21:08 | |
| openstackgerrit | Julia Kreger proposed openstack/python-ironicclient master: Follow up to the API version warning patches https://review.openstack.org/486677 | 21:09 |
| *** crushil has quit IRC | 21:10 | |
| *** crushil has joined #openstack-ironic | 21:10 | |
| *** pc_m has quit IRC | 21:11 | |
| jlvillal | TheJulia: mjturek: I did a review of: https://review.openstack.org/#/c/484032/ | 21:13 |
| patchbot | patch 484032 - ironic - Generate iPXE boot script on start up | 21:13 |
| jlvillal | Mainly issues in the unit tests | 21:13 |
| * TheJulia sighs | 21:14 | |
| *** baoli has quit IRC | 21:15 | |
| TheJulia | mjturek: you around to update the rev tonight? | 21:16 |
| *** crushil has quit IRC | 21:16 | |
| *** baoli has joined #openstack-ironic | 21:16 | |
| *** crushil has joined #openstack-ironic | 21:18 | |
| TheJulia | jlvillal: if it does pass everything, it might just be good to follow-up so we can get it landed, because it not landing tonight will likely cost us another day before nova might consider it as able to be landed in nova. I can always toss a follow-up to the unit tests later tonight. | 21:20 |
| *** baoli has quit IRC | 21:20 | |
| *** crushil has quit IRC | 21:21 | |
| *** crushil has joined #openstack-ironic | 21:26 | |
| *** crushil has quit IRC | 21:28 | |
| *** baoli has joined #openstack-ironic | 21:28 | |
| jlvillal | TheJulia: I would be okay with it as a follow-up | 21:29 |
| rloo | TheJulia: sorry, I forgot to look at 484032. Did you want me to look at it still, or is it ok with jlvillal's followup? | 21:31 |
| TheJulia | rloo: I think it is okay, largely some some test stuff. Up to you. jlvillal: I can post a follow-up in a couple hours, I have to go into town and get some dinner first | 21:35 |
| rloo | TheJulia: ack. jlvillal: you OK +A'ing that? I'm guessing that's what TheJulia wants? | 21:35 |
| rpioso | Does changing the default value of an optional function parameter constitute a breaking, non-backwards compatible change to an API? | 21:35 |
| jlvillal | TheJulia: Can you let me know when it passes the tests? | 21:35 |
| jlvillal | rloo: Yeah, I would be okay with +A once we know it is working. | 21:36 |
| jlvillal | As long as a follow-up is done. | 21:36 |
| rloo | jlvillal: you sure we need the tests to pass? :) | 21:36 |
| jlvillal | rloo: Yes!!! :P | 21:36 |
| rloo | jlvillal: add a comment saying that TheJulia promises there will be a follow up :) | 21:36 |
| jlvillal | heh | 21:37 |
| rloo | phew, /me got out of that one easily... | 21:37 |
| *** crushil has joined #openstack-ironic | 21:38 | |
| *** crushil has quit IRC | 21:40 | |
| *** pc_m has joined #openstack-ironic | 21:40 | |
| rpioso | rloo: ^^^ :) | 21:40 |
| rloo | rpioso: oh, missed that. yes. | 21:41 |
| rpioso | rloo: :-( | 21:41 |
| rpioso | rloo: ty | 21:41 |
| *** baoli has quit IRC | 21:41 | |
| rloo | rpioso: am trying to think about how we dealt with it in the past. deprecate/add a new one? | 21:41 |
| *** crushil has joined #openstack-ironic | 21:41 | |
| rloo | rpioso: is this an internal API? I'm guessing not otherwise you wouldn't ask. | 21:42 |
| rpioso | rloo: It's in python-dracclient. | 21:42 |
| rloo | rpioso: oh. i don't pay much attention to those. if you change, it means a new release of dracclient, right? | 21:43 |
| *** crushil has quit IRC | 21:43 | |
| rpioso | rloo: Yes, a new release is planned -- https://review.openstack.org/#/c/482371/2/dracclient/client.py@674 | 21:44 |
| patchbot | patch 482371 - python-dracclient - Parameterize iDRAC is ready retries at class level | 21:44 |
| rloo | rpioso: i guess if you up the min release of dracclient that can be used with ironic? but that would break older versions of ironic+dracclient. | 21:44 |
| *** crushil has joined #openstack-ironic | 21:44 | |
| rloo | rpioso: my personal opinion is this is a hw thing, if you want to face the wrath of your users, but i think we are also somewhat responsible or something so... | 21:44 |
| rpioso | rloo: Upping the min release of dracclient is planned, too -- https://review.openstack.org/#/c/481318 | 21:46 |
| patchbot | patch 481318 - ironic - Revert "Wait until iDRAC is ready before out-of-ba... | 21:46 |
| rloo | rpioso: how do we deal with older releases of ironic. do you up those min releases too? | 21:47 |
| rloo | rpioso: or does this only apply to master/pike ironic? | 21:47 |
| *** crushil has quit IRC | 21:48 | |
| rpioso | rloo: We haven't determined that, yet. The python-dracclient is a bug fix, tho. | 21:48 |
| rloo | rpioso: so lets say i am running ocata ironic with old(er) version of dracclient. presumably that will work. what if i upgraded dracclient to this new release. will things still work? | 21:48 |
| rloo | rpioso: so if it is a bug and you are fixing it, then presumably it was broken before so you are unbreaking a broken thing? | 21:48 |
| *** crushil has joined #openstack-ironic | 21:49 | |
| rpioso | rloo: Exactly. | 21:49 |
| rloo | rpioso: in that case, i suspect it is fine to make the change you are proposing. | 21:50 |
| *** crushil has quit IRC | 21:50 | |
| rpioso | rloo: https://www.youtube.com/watch?v=7Ag3ZKuUoy0 | 21:50 |
| rpioso | rloo: :) | 21:51 |
| rloo | rpioso: :) | 21:51 |
| *** Goneri has joined #openstack-ironic | 21:53 | |
| rpioso | rloo: The issue is a bit more subtle. It's been suggested that an existing API function's default params be changed so that that it ends up leveraging new __init__ default param values. The new __init__ default param values are the same as the existing API function's. | 21:54 |
| rpioso | rloo: https://review.openstack.org/#/c/482371/2/dracclient/client.py@674 | 21:54 |
| patchbot | patch 482371 - python-dracclient - Parameterize iDRAC is ready retries at class level | 21:54 |
| rloo | rpioso: i'm in the middle of something else; don't have the brain power to dive into your issue right now. | 21:55 |
| rpioso | rloo: Fair enuf. | 21:56 |
| rpioso | rloo: My concern is that it too closely ties the API function to the __init__ routine. | 21:57 |
| rpioso | rloo: I'll pursue it further in Gerrit. | 21:57 |
| rpioso | rloo: ty | 21:58 |
| TheJulia | jlvillal: sure, I'll check again in a few minutes, before I head into town since it was on just the grenade jobs when I last looked like 20 minutes ago | 21:59 |
| jlvillal | TheJulia: What is the test patch? | 21:59 |
| *** sambetts_ has quit IRC | 22:01 | |
| jlvillal | TheJulia: If it is: https://review.openstack.org/#/c/485349/ Then both grenade jobs failed. | 22:03 |
| patchbot | patch 485349 - ironic - WIP/DNM: Try to test boot-from-volume | 22:03 |
| TheJulia | http://logs.openstack.org/49/485349/3/check/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial/dd628fb/logs/new/screen-n-cpu.txt.gz#_2017-07-24_21_14_31_226 fun! | 22:13 |
| *** fragatin_ has joined #openstack-ironic | 22:14 | |
| TheJulia | jlvillal: does grenade upgrade nova before ironic? | 22:16 |
| jlvillal | TheJulia: yes. http://logs.openstack.org/49/485349/3/check/gate-grenade-dsvm-ironic-ubuntu-xenial/e372904/logs/grenade.sh.summary.txt.gz | 22:16 |
| TheJulia | fuck | 22:16 |
| TheJulia | it should be ironic then nova :( | 22:17 |
| *** fragatina has quit IRC | 22:18 | |
| TheJulia | ugh, and why did that one pass last week | 22:19 |
| jlvillal | TheJulia: Maybe it is an intermittent failure? Not sure. | 22:19 |
| jlvillal | TheJulia: boot-from-volume is not enabled in the grenade job. | 22:20 |
| *** jkilpatr has joined #openstack-ironic | 22:21 | |
| *** cdearborn has quit IRC | 22:21 | |
| TheJulia | Oh yeah, the previous success it started n-cpu like 20-25 seconds after ir-api | 22:22 |
| TheJulia | so init race \o/ | 22:22 |
| TheJulia | We should likely fix that | 22:22 |
| TheJulia | on the multi-node job, just looks like one of the two n-cpu processes didn't start... but there are not logs :\ | 22:28 |
| TheJulia | jlvillal: thanks for rechecking | 22:30 |
| jlvillal | TheJulia: You're welcome | 22:30 |
| *** jaypipes has quit IRC | 22:32 | |
| TheJulia | I just commented on mjturek's change. I'm going to go into town now. I'll likely also look at fixing the init order in grenade and see if it is an easy fix. | 22:32 |
| *** fragatin_ has quit IRC | 22:42 | |
| *** fragatina has joined #openstack-ironic | 22:43 | |
| *** rama_y_ has joined #openstack-ironic | 22:49 | |
| *** rama_y has quit IRC | 22:49 | |
| *** Goneri has quit IRC | 22:50 | |
| *** rama_y_ has quit IRC | 22:53 | |
| *** crushil has joined #openstack-ironic | 22:57 | |
| *** mjturek has quit IRC | 23:00 | |
| *** crushil has quit IRC | 23:01 | |
| *** Sukhdev has quit IRC | 23:05 | |
| *** rpioso has quit IRC | 23:28 | |
| JayF | TheJulia: so yeah, bad news too: at the last summit I was at, I remember a conversation between jroll and someone else about grenade ordering, and they asserted "plugins last" was the One True Way(tm) and I think it sorta spun wheels there | 23:34 |
| JayF | TheJulia: This is all super lossy though, just letting you know it might be a conversation vs just a pr | 23:34 |
| *** chlong__ has joined #openstack-ironic | 23:36 | |
| *** chlong_ has quit IRC | 23:37 | |
| TheJulia | Ugh, I remember that conversation now. If that is the case, we can't land the Nova driver this cycle. | 23:51 |
| TheJulia | Which... Just... Yeah | 23:52 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!