| *** crushil has joined #openstack-ironic | 00:00 | |
| *** crushil has quit IRC | 00:03 | |
| *** crushil has joined #openstack-ironic | 00:03 | |
| *** crushil has quit IRC | 00:06 | |
| *** crushil has joined #openstack-ironic | 00:07 | |
| *** crushil has quit IRC | 00:10 | |
| *** crushil has joined #openstack-ironic | 00:10 | |
| *** crushil has quit IRC | 00:13 | |
| *** crushil has joined #openstack-ironic | 00:13 | |
| *** crushil has quit IRC | 00:17 | |
| *** crushil has joined #openstack-ironic | 00:17 | |
| *** baoli has quit IRC | 00:18 | |
| *** crushil has quit IRC | 00:20 | |
| *** crushil has joined #openstack-ironic | 00:20 | |
| *** baoli has joined #openstack-ironic | 00:22 | |
| *** crushil has quit IRC | 00:23 | |
| *** crushil has joined #openstack-ironic | 00:24 | |
| *** crushil has quit IRC | 00:27 | |
| *** crushil has joined #openstack-ironic | 00:27 | |
| *** crushil has quit IRC | 00:30 | |
| *** crushil has joined #openstack-ironic | 00:31 | |
| *** crushil has quit IRC | 00:34 | |
| *** crushil has joined #openstack-ironic | 00:34 | |
| *** crushil has quit IRC | 00:37 | |
| *** crushil has joined #openstack-ironic | 00:38 | |
| *** crushil has quit IRC | 00:41 | |
| *** crushil has joined #openstack-ironic | 00:41 | |
| *** crushil has quit IRC | 00:44 | |
| *** crushil has joined #openstack-ironic | 00:44 | |
| *** hw_wutianwei has joined #openstack-ironic | 00:49 | |
| *** crushil has quit IRC | 00:51 | |
| *** gcb has joined #openstack-ironic | 00:52 | |
| *** phuongnh has joined #openstack-ironic | 01:02 | |
| *** baoli has quit IRC | 01:04 | |
| *** gcb has quit IRC | 01:07 | |
| *** gcb has joined #openstack-ironic | 01:08 | |
| *** baoli has joined #openstack-ironic | 01:30 | |
| *** pmannidi has quit IRC | 01:44 | |
| *** rbudden has joined #openstack-ironic | 01:45 | |
| *** pmannidi has joined #openstack-ironic | 01:49 | |
| *** mat128 has joined #openstack-ironic | 02:04 | |
| *** mat128 has quit IRC | 02:14 | |
| *** mat128 has joined #openstack-ironic | 02:21 | |
| *** mat128 has quit IRC | 02:25 | |
| *** fragatina has joined #openstack-ironic | 02:26 | |
| *** fragatina has quit IRC | 02:33 | |
| *** rama_y_ has joined #openstack-ironic | 02:39 | |
| *** baoli has quit IRC | 03:14 | |
| *** zhurong has joined #openstack-ironic | 03:25 | |
| *** rbudden has quit IRC | 03:38 | |
| *** aparnav has joined #openstack-ironic | 03:58 | |
| *** udesale has joined #openstack-ironic | 04:16 | |
| *** zhurong has quit IRC | 04:16 | |
| *** links has joined #openstack-ironic | 04:20 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Support `update_firmware` to hpsum based firmware update https://review.openstack.org/422572 | 04:38 |
|---|---|---|
| *** SerenaFeng has joined #openstack-ironic | 04:43 | |
| *** saop_ has joined #openstack-ironic | 04:51 | |
| *** zhurong has joined #openstack-ironic | 04:59 | |
| saop_ | vsaienk0, Hi | 05:12 |
| saop_ | vsaienk0, In our CI nova boot is not happening, it directly goes for ironic node-active, and our test cases are failing with that | 05:13 |
| saop_ | vsaienk0, There is any variable added in local.conf of anywhere else to separate ironic standalone test and simple ironic test cases | 05:13 |
| *** zhurong has quit IRC | 05:14 | |
| *** nmathew has joined #openstack-ironic | 05:32 | |
| *** vikrant has joined #openstack-ironic | 05:34 | |
| *** dsneddon has joined #openstack-ironic | 05:36 | |
| *** ijw has quit IRC | 05:39 | |
| *** ijw has joined #openstack-ironic | 05:40 | |
| openstackgerrit | Chris Smart proposed openstack/ironic master: WIP: test buildroot IPA image https://review.openstack.org/445763 | 05:41 |
| *** Nisha_Agarwal has joined #openstack-ironic | 06:04 | |
| *** jaosorior has joined #openstack-ironic | 06:04 | |
| *** bfournie has joined #openstack-ironic | 06:07 | |
| *** Sukhdev has joined #openstack-ironic | 06:07 | |
| *** Nisha_ has joined #openstack-ironic | 06:08 | |
| *** bfournie has quit IRC | 06:08 | |
| *** bfournie has joined #openstack-ironic | 06:09 | |
| *** Nisha_Agarwal has quit IRC | 06:12 | |
| *** rama_y_ has quit IRC | 06:30 | |
| *** moshele has joined #openstack-ironic | 06:30 | |
| *** moshele has quit IRC | 06:36 | |
| *** Sukhdev has quit IRC | 06:48 | |
| *** jaosorior has quit IRC | 06:53 | |
| *** jaosorior has joined #openstack-ironic | 06:55 | |
| *** fxpester has joined #openstack-ironic | 06:56 | |
| *** karthiks_afk has joined #openstack-ironic | 06:57 | |
| *** ChubYann has quit IRC | 07:02 | |
| *** yogi has joined #openstack-ironic | 07:03 | |
| *** jaosorior has quit IRC | 07:14 | |
| *** jaosorior has joined #openstack-ironic | 07:15 | |
| *** jaosorior has quit IRC | 07:16 | |
| *** jaosorior has joined #openstack-ironic | 07:19 | |
| pas-ha | morning ironic :) | 07:25 |
| *** moshele has joined #openstack-ironic | 07:29 | |
| openstackgerrit | Chris Smart proposed openstack/ironic master: WIP: test buildroot IPA image https://review.openstack.org/445763 | 07:35 |
| *** makowals has joined #openstack-ironic | 07:37 | |
| *** erolg has joined #openstack-ironic | 07:37 | |
| *** yonglihe has quit IRC | 07:39 | |
| *** karthiks_afk is now known as karthiks | 07:43 | |
| *** moshele has quit IRC | 07:45 | |
| *** bcornec has quit IRC | 07:45 | |
| *** ralonsoh has joined #openstack-ironic | 07:51 | |
| *** zhugaoxiao has quit IRC | 07:57 | |
| *** zhugaoxiao has joined #openstack-ironic | 07:58 | |
| *** tesseract has joined #openstack-ironic | 07:58 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| *** mjura has joined #openstack-ironic | 08:10 | |
| *** bcornec has joined #openstack-ironic | 08:12 | |
| *** pcaruana has joined #openstack-ironic | 08:14 | |
| *** Serlex has joined #openstack-ironic | 08:15 | |
| *** Nisha_ has quit IRC | 08:21 | |
| *** amarao has joined #openstack-ironic | 08:41 | |
| *** dtantsur|afk is now known as dtantsur | 08:47 | |
| dtantsur | morning ironic | 08:47 |
| *** racedo has joined #openstack-ironic | 08:47 | |
| *** pmannidi has quit IRC | 08:49 | |
| *** stendulker has joined #openstack-ironic | 08:56 | |
| *** jtomasek|afk is now known as jtomasek | 08:57 | |
| *** athomas has joined #openstack-ironic | 08:57 | |
| *** openstackgerrit has quit IRC | 09:02 | |
| *** bcornec has quit IRC | 09:03 | |
| aparnav | morning dtantsur, ironic :) | 09:04 |
| *** skazi has quit IRC | 09:05 | |
| *** skazi has joined #openstack-ironic | 09:05 | |
| *** stendulker has quit IRC | 09:06 | |
| *** Nisha_ has joined #openstack-ironic | 09:07 | |
| *** skazi has quit IRC | 09:08 | |
| *** skazi has joined #openstack-ironic | 09:09 | |
| *** skazi has quit IRC | 09:09 | |
| *** skazi has joined #openstack-ironic | 09:09 | |
| *** vills has joined #openstack-ironic | 09:09 | |
| *** vills has quit IRC | 09:10 | |
| *** Nisha_ has quit IRC | 09:14 | |
| *** e0ne has joined #openstack-ironic | 09:18 | |
| *** stendulker has joined #openstack-ironic | 09:22 | |
| *** alineb has joined #openstack-ironic | 09:23 | |
| *** openstackgerrit has joined #openstack-ironic | 09:23 | |
| openstackgerrit | Chris Smart proposed openstack/ironic master: WIP: test buildroot IPA image https://review.openstack.org/445763 | 09:23 |
| *** e0ne has quit IRC | 09:24 | |
| *** e0ne has joined #openstack-ironic | 09:24 | |
| *** lucas-afk is now known as lucasagomes | 09:28 | |
| lucasagomes | morning all | 09:28 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic master: Remove unneeded exception handling from agent driver https://review.openstack.org/447423 | 09:28 |
| lucasagomes | thomas_oneill, I believe everyone that works on bifrost is in this channel | 09:33 |
| *** tuan_ has joined #openstack-ironic | 09:41 | |
| *** SerenaFeng has quit IRC | 09:44 | |
| thomas_oneill | lucasagomes: haha yeah but this is the ironic channel :D | 09:45 |
| lucasagomes | folks, I've added ironic-{core, release} groups to the sushy-{core, release} groups respectively now that we decided that it should go under the ironic umbrella! Happy hacking :D | 09:45 |
| *** derekh has joined #openstack-ironic | 09:45 | |
| dtantsur | \o/ | 09:46 |
| lucasagomes | thomas_oneill, right, bifrost is part of the ironic governance btw | 09:47 |
| lucasagomes | so it kinda makes sense :-) | 09:47 |
| lucasagomes | (I think it's up to the bifrost devs too, if there's a need to have a new channel I don't see why not) | 09:47 |
| tuan_ | Hi Ironic folks, | 09:50 |
| tuan_ | i have a question that hopefully you guys can give some advices | 09:51 |
| tuan_ | do you know in the production of using ironic for provisioning both of baremetal and vm | 09:51 |
| *** mgoddard has joined #openstack-ironic | 09:53 | |
| openstackgerrit | Merged openstack/python-ironicclient master: Add negative tests for baremetal node commands https://review.openstack.org/337719 | 10:01 |
| *** athomas has quit IRC | 10:02 | |
| dtantsur | tuan_, I know, but using ironic for provisioning vms is a terrible idea | 10:04 |
| dtantsur | or did you mean using nova+vm and nova+ironic in the same cloud? | 10:05 |
| *** athomas has joined #openstack-ironic | 10:09 | |
| *** MattMan has quit IRC | 10:10 | |
| *** MattMan has joined #openstack-ironic | 10:10 | |
| tuan_ | dtantsur: yeap | 10:11 |
| tuan_ | it is my consideration | 10:11 |
| dtantsur | tuan_, the latter? jroll was looking into it. I was able to do PoC with host aggregates as well. | 10:12 |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: roles: bifrost-create-vm-nodes: Improve host matching when deleting from vbmc https://review.openstack.org/447435 | 10:12 |
| tuan_ | dtantsur: what about the problem of nova-scheduler in this case | 10:12 |
| dtantsur | what's the problem? | 10:16 |
| *** kromanenko has joined #openstack-ironic | 10:17 | |
| tuan_ | AFAIK nova-scheduler has no method to specify an exact node for ironic to provision | 10:17 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Add functional API tests for portgroups https://review.openstack.org/436878 | 10:19 |
| dtantsur | tuan_, you can make it do it, if you want to. but why do you need it, and how is it related? | 10:22 |
| *** gcb has quit IRC | 10:23 | |
| tuan_ | well, i just wanna know the behavior of Ironic to compare to vm :) | 10:24 |
| tuan_ | so now with the baremetal and vm in one cloud | 10:24 |
| tuan_ | are we going to have each of them in an aggregation | 10:25 |
| *** ijw has quit IRC | 10:25 | |
| tuan_ | dtantsur: otherwise, in the scenario of both baremetal and vm, when we configure the virt driver for nova on controller node | 10:26 |
| tuan_ | how do we satisfy with drivers that are needed for both baremetal and vms | 10:26 |
| *** ijw has joined #openstack-ironic | 10:27 | |
| dtantsur | not sure if I get the question, but you need separate nova-compute instances for bm and vm | 10:28 |
| tuan_ | dtantsur:yeap, and each of them is configured based on the purpose to be used for ironicdriver or virtdriver | 10:30 |
| *** phuongnh has quit IRC | 10:30 | |
| dtantsur | yep | 10:30 |
| tuan_ | and we also use different flavor for each of Ironic and normal vm | 10:31 |
| tuan_ | with the help of host aggregate? | 10:31 |
| dtantsur | tuan_, I don't know what the best way is, to be honest. I got some success with host aggregates, jroll may have better ideas | 10:35 |
| tuan_ | dtantsur: hmm, since we also have PoC for cell or domain in this case | 10:36 |
| dtantsur | yeah, there are options | 10:36 |
| tuan_ | but i myself do not like to use cell just for using both of Ironic and normal VM | 10:36 |
| tuan_ | may i ask you about your experiences in this scenario | 10:37 |
| tuan_ | as you mentioned from beginning? | 10:37 |
| tuan_ | how terrible is it | 10:37 |
| dtantsur | tuan_, here's what I was doing: http://tripleo.org/advanced_deployment/baremetal_overcloud.html#assigning-host-aggregates | 10:41 |
| *** jkilpatr has quit IRC | 10:41 | |
| tuan_ | dtantsur: Thanks, i also did the same host aggregation as you were doing | 10:44 |
| tuan_ | however, i do not have iformation in real production as you said that | 10:44 |
| tuan_ | it is terrible | 10:44 |
| tuan_ | how terrible is it | 10:45 |
| dtantsur | yep, dunno | 10:45 |
| dtantsur | let's wait for jroll | 10:45 |
| tuan_ | okay, thanks again | 10:45 |
| openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: roles: bifrost-ironic-install: Retry failed pip installation https://review.openstack.org/447444 | 10:47 |
| vdrok | good morning ironic! | 10:51 |
| vdrok | and morning dtantsur lucasagomes tuan_ | 10:51 |
| dtantsur | o/ | 10:51 |
| tuan_ | vdrok: actually, i am going to afternoon | 10:52 |
| tuan_ | :D | 10:52 |
| vdrok | tuan_: http://www.total-knowledge.com/~ilya/mips/ugt.html :) | 10:52 |
| tuan_ | vdrok: great :D | 10:53 |
| lucasagomes | vdrok, morning o/ | 10:54 |
| openstackgerrit | Chris Smart proposed openstack/ironic master: WIP: test buildroot IPA image https://review.openstack.org/445763 | 10:54 |
| *** udesale__ has joined #openstack-ironic | 10:56 | |
| *** udesale has quit IRC | 10:57 | |
| *** jkilpatr has joined #openstack-ironic | 10:58 | |
| *** udesale has joined #openstack-ironic | 10:59 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: flake8: Specify 'ironic_python_agent' as name of app https://review.openstack.org/446529 | 11:00 |
| *** udesale__ has quit IRC | 11:00 | |
| jroll | tuan_: dtantsur: yeah, host aggregates is the way for now, after pike it shouldn't be needed. see also https://cloudnull.io/2016/11/multi-hypervisor-openstack-cloud/ | 11:01 |
| jroll | morning | 11:01 |
| dtantsur | o/ | 11:05 |
| *** nmathew has quit IRC | 11:05 | |
| *** stendulker has quit IRC | 11:07 | |
| vdrok | morning jroll | 11:08 |
| Serlex | Morning | 11:08 |
| xavierr | good morning Ironic | 11:08 |
| tuan_ | jroll: Thanks jroll, i do not know that the commits mentioned inside the post will solve the case of host aggregate | 11:08 |
| tuan_ | :) | 11:08 |
| tuan_ | great | 11:08 |
| jroll | tuan_: you're welcome. I plan to write docs about it this cycle upstream | 11:09 |
| tuan_ | jroll: Yeah, that should be awesome:) | 11:10 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Add new transaction starting -> error on timeout https://review.openstack.org/437377 | 11:14 |
| *** aarefiev_afk is now known as aarefiev | 11:15 | |
| aarefiev | morning all! | 11:15 |
| dtantsur | o/ | 11:16 |
| *** zhurong has joined #openstack-ironic | 11:17 | |
| *** udesale has quit IRC | 11:19 | |
| *** udesale has joined #openstack-ironic | 11:19 | |
| openstackgerrit | Merged openstack/ironic master: Remove old oslo.messaging transport aliases https://review.openstack.org/447295 | 11:30 |
| openstackgerrit | Merged openstack/ironic master: devstack: Don't modprobe inside containers https://review.openstack.org/446232 | 11:33 |
| TheJulia | Good morning | 11:37 |
| *** zhurong has quit IRC | 11:37 | |
| dtantsur | morning TheJulia | 11:39 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Add negative functional API tests for portgroups https://review.openstack.org/437535 | 11:39 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Extend tests and checks for node VIFs https://review.openstack.org/436003 | 11:40 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Add negative tests for VIF attach/detach operations https://review.openstack.org/435952 | 11:40 |
| TheJulia | dtantsur: I don't know if I'll be online during the meeting today due to a flight, updated the whiteboard. | 11:46 |
| *** udesale has quit IRC | 11:47 | |
| dtantsur | thanks! | 11:47 |
| dtantsur | oh, you fly a lot recently | 11:48 |
| openstackgerrit | Merged openstack/ironic-lib master: flake8: Specify 'ironic_lib' as name of app https://review.openstack.org/446605 | 11:54 |
| TheJulia | dtantsur: yeah :( | 12:02 |
| *** snecklifter has joined #openstack-ironic | 12:02 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: [WIP] Ensure dying SOL console on conductor death https://review.openstack.org/446571 | 12:02 |
| *** vsaienko has joined #openstack-ironic | 12:02 | |
| vdrok | morning TheJulia ! | 12:03 |
| tuan_ | jroll: Hope you are still there | 12:06 |
| jroll | tuan_: I'm here all day :) | 12:06 |
| *** aparnav has quit IRC | 12:06 | |
| tuan_ | could you share some idea about how terrible when we use both of baremetal and vm in a cloud | 12:07 |
| *** mat128 has joined #openstack-ironic | 12:07 | |
| mat128 | morning | 12:07 |
| jroll | tuan_: some things are a bit weird, we're trying to collect them here: https://etherpad.openstack.org/p/ironic-and-virt-mixed-limitations . part of the docs writing will be verifying some of this | 12:09 |
| *** dprince has joined #openstack-ironic | 12:10 | |
| *** tuan_ has quit IRC | 12:11 | |
| *** tuan_ has joined #openstack-ironic | 12:16 | |
| tuan_ | jroll: When you have free time, i would appreciate if you can share some experiences about it | 12:17 |
| tuan_ | thanks in advanced | 12:17 |
| jroll | tuan_: some things are a bit weird, we're trying to collect them here: https://etherpad.openstack.org/p/ironic-and-virt-mixed-limitations . part of the docs writing will be verifying some of this | 12:17 |
| tuan_ | perfect | 12:17 |
| tuan_ | i am going to read it | 12:17 |
| jroll | tuan_: my goal this cycle is: deploy this, test it, write down what is terrible, make a plan to fix it, write docs | 12:18 |
| tuan_ | jroll: perfect :D | 12:18 |
| jroll | :) | 12:19 |
| *** bfournie has quit IRC | 12:21 | |
| dtantsur | jroll, this resource_class thingy, when should we start making noise about it? | 12:26 |
| * dtantsur wonders if he has to switch the whole tripleo... | 12:26 | |
| jroll | dtantsur: literally writing that plan right now | 12:26 |
| dtantsur | awesome | 12:27 |
| *** snecklifter has quit IRC | 12:27 | |
| jroll | dtantsur: assuming people are happy with it, it will be possible to switch before pike deploy, and required before queens deploy | 12:27 |
| dtantsur | aha, so I should plan on Pike, but nothing will break without it before Queens final, right? | 12:27 |
| jroll | I should have something to share in a few hours after nova scheduler meeting | 12:27 |
| jroll | yeah, that's the plan | 12:27 |
| jroll | we don't have warning logs in ocata | 12:27 |
| dtantsur | okie.. I wonder what the upgrade path for existing nodes is | 12:28 |
| dtantsur | I hope you're writing it as well :) | 12:28 |
| jroll | I would say mid-queens, to be fair, but it's on the nova side | 12:28 |
| jroll | yep | 12:28 |
| jroll | upgrade plan: ironic node-update add resource_class=foo; nova flavor-key set extra_spec resources:CUSTOM_FOO=1 | 12:29 |
| jroll | (roughly) | 12:29 |
| dtantsur | aha. should we drop everything node.properties at the same time? will capabilities still work? | 12:29 |
| jroll | we don't need to drop node.properties, that will be handled nova side. we'll be able to set for example, extra_spec resources:VCPU=0 to not schedule by CPU count, but still show CPU in nova flavor-list | 12:30 |
| dtantsur | nice, nice | 12:31 |
| jroll | the ComputeCapabilitiesFilter will still exist, afaik, and we'll also be working to use placement "traits" there during queens | 12:31 |
| dtantsur | I wonder what is right in case of tripleo.. I'm inclined to stop scheduling on properties, but maybe that's only me. | 12:31 |
| jroll | (so that filter can go away) | 12:31 |
| dtantsur | much upgrade not wow :( | 12:31 |
| jroll | which part? heh | 12:32 |
| dtantsur | all of them. on the bright side, this replaces a lof of ugly scheduling hacks in tripleo | 12:32 |
| jroll | the resource tracker will stop tracking node properties in queens, btw, so you need to | 12:32 |
| jroll | yeah | 12:32 |
| dtantsur | "a lof" = something between "a lot" and "a loaf" | 12:32 |
| dtantsur | :D | 12:32 |
| jroll | lol | 12:32 |
| dtantsur | we essentially have resource classes, for example. we call them "profiles" and use capabilities for them. | 12:33 |
| jroll | oh, perfect, so it will only change how you do that | 12:33 |
| nicodemos | good morning, ironic. | 12:33 |
| jroll | hey nicodemos :) | 12:33 |
| nicodemos | jroll :) | 12:34 |
| *** ralonsoh_ has joined #openstack-ironic | 12:34 | |
| dtantsur | yeah, this is a certain improvement for us. but requires an upgrade. | 12:35 |
| jroll | dtantsur: how do tripleo upgrades work, are there release-specific scripts? | 12:35 |
| dtantsur | jroll, well, kinda.. that's what we had this time: https://github.com/openstack/instack-undercloud/commit/ad47a3c40fbf13427457e4b0b9fb7a684de6377a | 12:36 |
| dtantsur | it seems like I can write some puppet code as well | 12:36 |
| *** jcoufal has joined #openstack-ironic | 12:37 | |
| dtantsur | this is undercloud though. for overcloud we have ansible within heat templates (brrrrr) | 12:37 |
| jroll | :| | 12:37 |
| jroll | I guess you need this on both sides | 12:37 |
| dtantsur | I guess so too.. chatting with folks on #tripleo now, I think I'll manage it :) | 12:37 |
| * dtantsur brb | 12:38 | |
| jroll | I'd almost prefer a python script, shouldn't be too hard | 12:38 |
| jroll | I'm going to try to get this done pretty early in pike, hopefully that leaves tripleo time to write the upgrade scripts in pike and not worry about it in queens | 12:38 |
| dtantsur | s/leaves tripleo time/leaves dtantsur time/ :( | 12:39 |
| dtantsur | but thanks :) | 12:39 |
| jroll | make dansmith do it :P | 12:39 |
| dtantsur | haha, this is an option | 12:39 |
| *** hw_wutianwei has quit IRC | 12:40 | |
| *** snecklifter has joined #openstack-ironic | 12:41 | |
| *** gcb has joined #openstack-ironic | 12:49 | |
| *** mvpnitesh has joined #openstack-ironic | 12:51 | |
| *** bfournie has joined #openstack-ironic | 12:53 | |
| *** trown|outtypewww is now known as trown | 12:53 | |
| *** rbudden has joined #openstack-ironic | 12:56 | |
| mvpnitesh | Hi All , i'm trying to set the provision state of an node to active , i'm getting the following error : "Deploy failed for instance None. Error: Unexpected error while running command. Command: sudo ironic-rootwrap /etc/ironic/rootwrap.conf iscsiadm -m discovery -t st -p 10.10.0.150:3260 Exit code: 96 Stdout: u'' Stderr: u'/usr/bin/ironic-rootwrap: Executable not found: iscsiadm (filter match = iscsiadm)'" | 12:57 |
| TheJulia | mvpnitesh: you need to install the iscsi tools package for your distribution | 12:58 |
| mvpnitesh | should i install iscsi-initiator-utils-6.2.0.873-32.el7.i686.rpm | 13:00 |
| *** ppiela has joined #openstack-ironic | 13:00 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-lib master: Replace udevadm settle with test -e https://review.openstack.org/446950 | 13:01 |
| * jroll brb | 13:01 | |
| TheJulia | mvpnitesh: does it prove iscsiadm? | 13:02 |
| *** sasha2 has joined #openstack-ironic | 13:04 | |
| *** sasha2 has quit IRC | 13:05 | |
| mvpnitesh | TheJulia: I don't know :( , can you please tell me how to check about iscsiadm? or could you please tell me which package i've to install for getting iscsi tools | 13:06 |
| mat128 | mvpnitesh: yum install iscsi-initiator-utils.x86_64 | 13:06 |
| mat128 | from http://www.tecmint.com/iscsi-initiator-client-setup/ | 13:06 |
| *** sasha2 has joined #openstack-ironic | 13:06 | |
| TheJulia | mat128: Thank you :) | 13:07 |
| mat128 | :) | 13:07 |
| mvpnitesh | thanks Mat128 , post that should i do any configuration change ?? | 13:07 |
| mat128 | I'm sure theres an equivalent to packages.ubuntu.com and the "path search" for RHEL/CentOS but couldn't find it | 13:07 |
| mat128 | mvpnitesh: Installing the package should be enough | 13:07 |
| openstackgerrit | Merged openstack/ironic-ui master: Fix oslo_debug_helper not running https://review.openstack.org/436898 | 13:08 |
| *** rloo has joined #openstack-ironic | 13:08 | |
| mat128 | mvpnitesh: and retry in Ironic (move node back to active) | 13:08 |
| *** jaypipes has joined #openstack-ironic | 13:08 | |
| mvpnitesh | <mat128> thanks , sure i'll do that | 13:08 |
| *** links has quit IRC | 13:11 | |
| *** snecklifter has quit IRC | 13:13 | |
| dtantsur | morning mat128, mvpnitesh | 13:13 |
| * dtantsur brb finally for real | 13:13 | |
| *** dtantsur is now known as dtantsur|brb | 13:13 | |
| *** snecklifter has joined #openstack-ironic | 13:15 | |
| TheJulia | Oh they joy of review.o.o from a plane.... | 13:16 |
| mat128 | TheJulia: it seems you're spending more time on a plane than anywhere else | 13:16 |
| mat128 | TheJulia: I wonder what a geolocation would reveal on your public IP address ;) | 13:17 |
| TheJulia | mat128: irccloud... and my at&t mifi tunnels back to the states. | 13:17 |
| mat128 | TheJulia: so you have cell coverage in the air? | 13:18 |
| TheJulia | mat128: Well, the aircraft does. I'm on the in-flight wifi at the moment. | 13:18 |
| mat128 | ahh | 13:18 |
| *** bcornec has joined #openstack-ironic | 13:19 | |
| *** siva has joined #openstack-ironic | 13:19 | |
| *** siva has quit IRC | 13:19 | |
| mat128 | gotta run, ttyl | 13:20 |
| TheJulia | enjoy | 13:20 |
| *** Goneri has joined #openstack-ironic | 13:21 | |
| *** gcb has quit IRC | 13:22 | |
| *** gcb has joined #openstack-ironic | 13:23 | |
| *** Sukhdev has joined #openstack-ironic | 13:26 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Mention state machine diagram generation in contrib guide https://review.openstack.org/447494 | 13:27 |
| Sukhdev | Ironic experts, ever since I have upgraded my setup to newest code, I have not been able to get successful devstack install. placement-api fails with this failure - http://paste.openstack.org/show/603373/ | 13:28 |
| Sukhdev | any insight to fix? | 13:29 |
| jroll | Sukhdev: have you completely reinstalled? (e.g. new VM) and also can you post your local.conf? | 13:29 |
| jroll | that looks like nova-compute though :) | 13:29 |
| TheJulia | NobodyCam: cinerama: I went through the outstanding bifrost reviews, a number should be in-flight to land, but a few need second reviews. If you have time. | 13:29 |
| mariojv | good morning ironic o/ it's good to be back | 13:30 |
| mariojv | i will probably not grep through 20k lines of irc for mentions, so please ping me here again if you did so in the last week when i was on break | 13:31 |
| openstackgerrit | Merged openstack/bifrost stable/ocata: Workaround for network start race condition https://review.openstack.org/446350 | 13:31 |
| openstackgerrit | Merged openstack/bifrost master: roles: bifrost-ironic-install: Add PyMySQL dependency for ironic-dbsync https://review.openstack.org/444197 | 13:31 |
| Sukhdev | jroll : yes, this is not on a VM, but, a BM host - but, yes, full cleanup and upgrade of all packages, along with all the code - here is local.conf - http://paste.openstack.org/show/603414/ | 13:32 |
| TheJulia | betherly: I know your likely not up to it today, but if your bored, there are a number of small ironic-ui revisions that could use a second core reviewer. :) | 13:32 |
| jroll | Sukhdev: does "cleanup" include reinstall the operating system? | 13:34 |
| jroll | Sukhdev: are you using the ocata branch of devstack? | 13:34 |
| Sukhdev | jroll : no | 13:34 |
| Sukhdev | jroll : yes, pulled the latest devstack as well | 13:34 |
| jroll | Sukhdev: not latest devstack, ocata devstack :P | 13:35 |
| jroll | your local.conf looks sane, though | 13:35 |
| Sukhdev | jroll : you think that may be the issue? devstack is master branch | 13:35 |
| openstackgerrit | Merged openstack/bifrost stable/ocata: scripts: test-bifrost.sh: Initialize the ZUUL_BRANCH variable https://review.openstack.org/441041 | 13:35 |
| jroll | Sukhdev: yes, use the ocata branch | 13:36 |
| Sukhdev | jroll : I was actually trying to run everything with latest master - was having the same issue, so, tried to go back to stable/ocata - but, did not change the devstack | 13:36 |
| jroll | ah | 13:36 |
| jroll | master everything should work, stable/ocata everything should work | 13:36 |
| jroll | CI proves it :P | 13:37 |
| Sukhdev | :-) | 13:37 |
| *** snecklifter has quit IRC | 13:37 | |
| *** baoli has joined #openstack-ironic | 13:37 | |
| jroll | Sukhdev: it looks like ironic isn't running in your logs, you might also check for errors in either ir-api or ir-cond | 13:37 |
| *** dprince has quit IRC | 13:37 | |
| betherly | TheJulia: I'm not really meant to be looking at my phone as it's still sore and I'm not allowed to look at tv or laptop screens at all until near the end of the week so work be able to review patches until Friday most likely I'm sorry :( | 13:38 |
| TheJulia | betherly: next week then, go rest! | 13:38 |
| TheJulia | betherly: sorry for bothering you | 13:38 |
| Sukhdev | jroll : right - when it fails, I do not see Ironic running at all - do not know why. Hence, I thought may be some local.conf config knob changed perhaps | 13:38 |
| betherly | TheJulia: will do them as soon as I can for sure. Bored to death hence on phone at all | 13:39 |
| TheJulia | betherly: :) | 13:39 |
| *** Sukhdev has quit IRC | 13:39 | |
| *** dprince has joined #openstack-ironic | 13:39 | |
| *** Sukhdev has joined #openstack-ironic | 13:39 | |
| *** ralonsoh_ has quit IRC | 13:40 | |
| jroll | Sukhdev: ah, doh, you need enable_service ir-api ir-cond | 13:41 |
| Sukhdev | jroll : yikes!! | 13:42 |
| openstackgerrit | Merged openstack/bifrost master: Change dib element to enable serial console https://review.openstack.org/445831 | 13:42 |
| *** clenimar has quit IRC | 13:43 | |
| Sukhdev | jroll : while fiddling with things, I think I accidently delete it - I have looked at that conf file gazzion time and did not catch it - how stupid of me :-):-) | 13:43 |
| jroll | Sukhdev: heh, no worries | 13:44 |
| openstackgerrit | Miles Gould proposed openstack/ironic-ui master: Show boot device in Node Details/Configuration https://review.openstack.org/447085 | 13:45 |
| Sukhdev | jroll : thanks - hopefully, that will resolve it | 13:45 |
| jroll | np :) | 13:45 |
| *** mgould|afk is now known as mgould | 13:49 | |
| *** mat128 has quit IRC | 13:51 | |
| *** snecklifter has joined #openstack-ironic | 13:52 | |
| *** crushil has joined #openstack-ironic | 13:52 | |
| *** lucasagomes is now known as lucas-hungry | 13:53 | |
| *** [1]cdearborn has joined #openstack-ironic | 13:57 | |
| *** gcb has quit IRC | 13:57 | |
| *** gcb has joined #openstack-ironic | 13:58 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Add missing 'autospec' argument to mock.patch https://review.openstack.org/447306 | 13:59 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Ensure dying SOL console on conductor death https://review.openstack.org/446571 | 13:59 |
| *** gcb_ has joined #openstack-ironic | 14:01 | |
| *** amarao has quit IRC | 14:01 | |
| *** nicodemos has quit IRC | 14:01 | |
| *** amarao has joined #openstack-ironic | 14:01 | |
| *** dprince has quit IRC | 14:01 | |
| *** gcb has quit IRC | 14:02 | |
| *** saop_ has quit IRC | 14:03 | |
| *** dprince has joined #openstack-ironic | 14:03 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic-specs master: API Evolution - ETAG identifiers https://review.openstack.org/381991 | 14:04 |
| *** dprince has quit IRC | 14:12 | |
| *** udesale has joined #openstack-ironic | 14:12 | |
| *** dprince has joined #openstack-ironic | 14:17 | |
| *** bcornec has quit IRC | 14:19 | |
| *** mvpnitesh has quit IRC | 14:19 | |
| *** causten has joined #openstack-ironic | 14:22 | |
| *** tuan_ has quit IRC | 14:22 | |
| *** ijw has quit IRC | 14:25 | |
| *** ijw has joined #openstack-ironic | 14:28 | |
| *** vikrant has quit IRC | 14:30 | |
| *** udesale has quit IRC | 14:39 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Ensure dying SOL console on conductor death https://review.openstack.org/446571 | 14:39 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-lib master: Replace udevadm settle with test -e https://review.openstack.org/446950 | 14:40 |
| *** rama_y has quit IRC | 14:41 | |
| *** aNuposic has quit IRC | 14:41 | |
| Sukhdev | jroll : No luck | 14:42 |
| jroll | Sukhdev: surely progress, though? | 14:43 |
| Sukhdev | jroll : exact same error - what is beyond me is that Ir-api, ir-cond does not run | 14:43 |
| jroll | Sukhdev: is there a screen session for them? | 14:43 |
| *** dtantsur|brb is now known as dtantsur | 14:44 | |
| Sukhdev | jroll : no | 14:44 |
| *** Goneri has quit IRC | 14:44 | |
| jroll | O_o | 14:44 |
| Sukhdev | jroll : last screen session is placement-api, which shows it did not start - when I look at n-cpu logs, it shows failing because it can not connect to ironic nodes | 14:45 |
| Sukhdev | jroll : it shows this failure - http://paste.openstack.org/show/603373/ | 14:45 |
| jroll | Sukhdev: right, makes sense, just odd that ironic services aren't running | 14:46 |
| jroll | can you paste your new local.conf? | 14:46 |
| *** dyasny has joined #openstack-ironic | 14:47 | |
| Sukhdev | jroll : http://paste.openstack.org/show/603434/ | 14:47 |
| Sukhdev | jroll : I went back to master branch - just to make sure devstack and rest of the stuff is all from master branch | 14:48 |
| jroll | Sukhdev: cool | 14:48 |
| jroll | Sukhdev: local.conf looks good, something weird is happening, I think I would need full devstack logs to help here | 14:49 |
| Sukhdev | jroll : you mean logs from stack.sh, right? | 14:50 |
| jroll | Sukhdev: yes | 14:50 |
| jroll | Sukhdev: should be on the filesystem as devstacklog.txt btw | 14:50 |
| Sukhdev | jroll : let me put them in paste | 14:51 |
| jroll | cool, I see no reason this shouldn't be either working or killing the devstack install | 14:51 |
| *** lucas-hungry is now known as lucasagomes | 14:56 | |
| *** dtantsur is now known as dtantsur|mtg | 14:57 | |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Fix Javascript unassignable errors https://review.openstack.org/447532 | 14:58 |
| mariojv | does anyone have that link to the summit brainstorm etherpad handy? | 15:01 |
| mariojv | i'd like an ops session if no one already proposed one | 15:02 |
| rloo | mariojv: i think it is mentioned in last week's meeting. (and welcome back). TheJulia would know. | 15:03 |
| Sukhdev | jroll : odd - the devstack.log file is empty - only the summary file. | 15:03 |
| rloo | hi and Happy Spring Equinox everyone, jroll, Sukhdev, mariojv, TheJulia | 15:04 |
| Sukhdev | rloo : thanks - same to you | 15:05 |
| jroll | Sukhdev: huh, weird | 15:06 |
| jroll | mariojv: https://etherpad.openstack.org/p/BOS-ironic-brainstorming | 15:07 |
| Sukhdev | jroll : ever since I decided to upgrade to latest code, all kind of weirdness has crept in :-):-) | 15:07 |
| jroll | Sukhdev: this is why I recommend paving everything and reinstalling the OS | 15:08 |
| rloo | mariojv, jroll: I added that link to our etherpad | 15:09 |
| *** amarao has quit IRC | 15:09 | |
| *** amarao has joined #openstack-ironic | 15:09 | |
| *** hw_wutianwei has joined #openstack-ironic | 15:10 | |
| *** dyasny has quit IRC | 15:12 | |
| soliosg | jroll, Sukhdev: I wonder if 'enable_service ir-api ir-cond' is needed in a normal devstack+ironic install | 15:13 |
| *** amarao has quit IRC | 15:13 | |
| mariojv | thanks and hi rloo | 15:13 |
| mariojv | is it spring? i didn't notice :) | 15:13 |
| jroll | soliosg: I believe it is (though it would be nice to just happen if you enable the plugin) | 15:13 |
| soliosg | jroll, Sukhdev: because ironic's dev-quickstart guide doesn't mention it | 15:13 |
| jroll | ah | 15:14 |
| jroll | maybe not then! | 15:14 |
| soliosg | jroll: yeah, I believe just the enable_plugin ironic ... should do the trick | 15:14 |
| jroll | soliosg: indeed, you're right | 15:15 |
| Sukhdev | soliosg jroll : ah that explains why I did not have it in my local.conf - I thought I accidently delete it - and, when I put it back in, it did not make any difference | 15:15 |
| Sukhdev | soliosg : thanks for pointing this out - makes sense | 15:16 |
| mgould | mariojv: allegedly | 15:16 |
| NobodyCam | Good Morning Ironic'ers | 15:17 |
| kromanenko | Hello! | 15:17 |
| NobodyCam | morning kromanenko :) | 15:17 |
| *** srart has joined #openstack-ironic | 15:17 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Add negative functional API tests for portgroups https://review.openstack.org/437535 | 15:18 |
| *** amarao has joined #openstack-ironic | 15:18 | |
| mariojv | i updated https://etherpad.openstack.org/p/ironic-review-party with a new review party time for anyone curious | 15:21 |
| mariojv | mainly just to reflect daylight saving time, since most previous attendees would have been affected by daylight saving time | 15:21 |
| mariojv | now 20:00 UTC Thursdays, lmk if there are any objections | 15:21 |
| mariojv | thanks | 15:21 |
| mariojv | emailed mailing list about this just now, too | 15:21 |
| *** crushil has quit IRC | 15:21 | |
| mjturek | mariojv: effective this week? | 15:22 |
| Sukhdev | jroll : in the local.conf that I pasted did you see anything which will prevent from stack.sh logs - for some odd reason, I see devstack logs empty | 15:22 |
| mariojv | mjturek: yes | 15:22 |
| mjturek | cool cool | 15:22 |
| mariojv | i'll probably hang out for 5 mins after the scheduled time in case anyone doesn't get the message on time though | 15:23 |
| *** ijw has quit IRC | 15:23 | |
| mjturek | yeah good idea! | 15:23 |
| jroll | Sukhdev: LOGFILE=HOME/devstack.log | 15:23 |
| jroll | Sukhdev: is not the same as $HOME/devstack.log | 15:23 |
| jroll | so see if there's a directory called HOME somewhere I guess :P | 15:24 |
| Sukhdev | jroll : ha ha - I saw it as soon as you pointed it out :-) | 15:24 |
| jroll | maybe ~/devstack/HOME/... | 15:24 |
| jroll | :) | 15:24 |
| *** ijw has joined #openstack-ironic | 15:24 | |
| *** yogi has quit IRC | 15:24 | |
| soliosg | Sukhdev: looks like swift is enabled in your local.conf | 15:24 |
| Sukhdev | jroll : right - I see the logs there - damn - cut-n-pate | 15:24 |
| *** dyasny has joined #openstack-ironic | 15:25 | |
| Sukhdev | soliosg : is that bad thing? should it not be enabled? | 15:25 |
| soliosg | Sukhdev: but the stable/ocata for swift is not defined | 15:25 |
| soliosg | Sukhdev: just wondering, I don't have any idea what could be wrong ;) | 15:25 |
| Sukhdev | soliosg : I switched to everything to master branch | 15:26 |
| soliosg | Sukhdev: swift can be enabled, that shouldn't be a problem | 15:26 |
| Sukhdev | soliosg : so, hopefully no issue related to stable/ocata | 15:26 |
| *** jkilpatr has quit IRC | 15:27 | |
| jroll | yeah, that shouldn't be a problem | 15:27 |
| soliosg | Sukhdev: LOGFILE=HOME/devstack.log has missing the $ | 15:28 |
| *** baoli has quit IRC | 15:28 | |
| soliosg | Sukhdev: that should explain the devstack.log being empty | 15:28 |
| *** jkilpatr has joined #openstack-ironic | 15:28 | |
| Sukhdev | soliosg : right - that was mistake - hence, the logs are going under devstack/HOME :-) | 15:28 |
| TheJulia | mariojv: I created it, but sadly didn't get any brainstorming done last week. I basically spent the entire week under the weather :( | 15:29 |
| TheJulia | mariojv: looks like you've got the link though :) | 15:30 |
| mariojv | TheJulia: i'll find it in last week's meeting notes. thanks | 15:30 |
| *** baoli has joined #openstack-ironic | 15:30 | |
| TheJulia | oh, one moment | 15:30 |
| TheJulia | https://etherpad.openstack.org/p/BOS-ironic-brainstorming | 15:31 |
| mariojv | tyvm! | 15:31 |
| *** beekneemech is now known as bnemec | 15:32 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Extend tests and checks for node VIFs https://review.openstack.org/436003 | 15:32 |
| *** baha has joined #openstack-ironic | 15:32 | |
| *** chlong has joined #openstack-ironic | 15:33 | |
| *** baoli has quit IRC | 15:36 | |
| *** fxpester has quit IRC | 15:37 | |
| *** baoli has joined #openstack-ironic | 15:38 | |
| TheJulia | mariojv: I did get a chance to watch the ops mid cycle session etherpad notes appear last week. I do feel like there should be some time set aside to possibly discuss some of those things, as well as communications and perceptions with the operator portion of the community | 15:38 |
| * jroll steps away until the meeting | 15:38 | |
| mariojv | sounds good TheJulia | 15:38 |
| openstackgerrit | Chris Krelle proposed openstack/bifrost master: Follow up to 447435 https://review.openstack.org/447555 | 15:39 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Add negative tests for VIF attach/detach operations https://review.openstack.org/435952 | 15:39 |
| *** rpioso has joined #openstack-ironic | 15:40 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Add negative tests for VIF attach/detach operations https://review.openstack.org/435952 | 15:40 |
| openstackgerrit | Merged openstack/bifrost master: roles: bifrost-create-vm-nodes: Improve host matching when deleting from vbmc https://review.openstack.org/447435 | 15:42 |
| vdrok | jroll: could you merge https://review.openstack.org/#/c/443800/3 ? | 15:43 |
| patchbot | patch 443800 - ironic (stable/ocata) - Updated from global requirements | 15:43 |
| vdrok | and good morning mariojv NobodyCam rlo | 15:43 |
| vdrok | rloo: :) | 15:43 |
| mariojv | morning vdrok | 15:43 |
| rloo | hi vdrok, NobodyCam! | 15:44 |
| openstackgerrit | Merged openstack/ironic master: flake8: Specify 'ironic' as name of app https://review.openstack.org/446562 | 15:44 |
| *** jaosorior has quit IRC | 15:45 | |
| NobodyCam | Morning vdrok & roll :) | 15:45 |
| NobodyCam | rloo: even :p | 15:47 |
| rloo | NobodyCam: :D | 15:47 |
| NobodyCam | lol hehehe :) | 15:47 |
| *** Nisha_Agarwal has joined #openstack-ironic | 15:47 | |
| *** Goneri has joined #openstack-ironic | 15:48 | |
| Serlex | hi, how can I further my investigation on trying to boot an ironic instance resulting in "No valid host was found" in nova_conductor | 15:50 |
| Serlex | most likely mis-configuration in flavor attributes? | 15:50 |
| *** links has joined #openstack-ironic | 15:51 | |
| rloo | Serlex: did you take a look at https://docs.openstack.org/developer/ironic/deploy/troubleshooting.html | 15:51 |
| *** hw_wutianwei has quit IRC | 15:51 | |
| Serlex | nope thanks | 15:51 |
| *** rbudden has quit IRC | 15:52 | |
| *** amarao has quit IRC | 15:53 | |
| *** sayantani01 has joined #openstack-ironic | 15:55 | |
| *** links has quit IRC | 15:55 | |
| jroll | vdrok: done | 15:56 |
| * jroll disappears again | 15:56 | |
| *** links has joined #openstack-ironic | 15:56 | |
| vdrok | jroll: thx! | 15:56 |
| joanna | morning rloo, vdrok, NobodyCam, mariojv, jroll, Serlex :) | 15:58 |
| mariojv | morning joanna | 15:58 |
| NobodyCam | Good morning joanna :) | 15:58 |
| NobodyCam | morning mariojv | 15:58 |
| vdrok | morning joanna and Serlex ! | 15:58 |
| NobodyCam | :) | 15:58 |
| mariojv | morning NobodyCam | 15:58 |
| rloo | hi vdrok, wrt 413113, i still see deprecation warnings. did you mean to handle them all or only some of them? | 15:59 |
| *** jaosorior has joined #openstack-ironic | 15:59 | |
| rloo | morning joanna! | 15:59 |
| vdrok | rloo: if you mean the policy ones, we can't fix them all I think | 15:59 |
| rloo | vdrok: the .set_latent() ones | 16:00 |
| vdrok | rloo: ah, lemme check | 16:00 |
| *** trown is now known as trown|outtypewww | 16:00 | |
| rloo | vdrok: i commented. copy/pasted form the console | 16:00 |
| vdrok | thanks | 16:00 |
| Serlex | Morning where you based | 16:00 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/ironic master: Remove config option verbose from ironic.conf.sample https://review.openstack.org/433466 | 16:00 |
| Serlex | im about to head out | 16:01 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Fix docstrings for creating methods in baremetal api tests https://review.openstack.org/447565 | 16:01 |
| vdrok | rloo: ah, so that's inside oslo I suppose | 16:02 |
| vdrok | at least from what I see in the message | 16:02 |
| *** Syed__ has joined #openstack-ironic | 16:02 | |
| vdrok | oslo_middleware itself might not have switched yet :) | 16:02 |
| rloo | vdrok: oh, i only grep'd, didn't look into them. ok, i'll assume the rest are like that too. | 16:03 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Fix docstrings for creating methods in baremetal api tests https://review.openstack.org/447565 | 16:05 |
| *** links has quit IRC | 16:05 | |
| *** crushil has joined #openstack-ironic | 16:10 | |
| *** dtantsur|mtg is now known as dtantsur | 16:11 | |
| dtantsur | morning rloo, NobodyCam, mariojv | 16:11 |
| NobodyCam | Morning dtantsur :) | 16:11 |
| jlvillal | vsaienk0: Very cool https://review.openstack.org/447525 :) | 16:12 |
| patchbot | patch 447525 - openstack-infra/project-config - Increase concurrency for ironic jobs to 4 | 16:12 |
| jlvillal | Some big time savings on the grenade and multi-node jobs. | 16:13 |
| *** Nisha_Agarwal has quit IRC | 16:13 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 16:15 | |
| *** tesseract has quit IRC | 16:15 | |
| *** Sukhdev has quit IRC | 16:16 | |
| *** Sukhdev_ has joined #openstack-ironic | 16:16 | |
| *** jkilpatr has quit IRC | 16:16 | |
| vsaienk0 | jlvillal: yup, increasing concurrency saves us 1h on multinode grenade job, but I would like to collect more statistic | 16:17 |
| jlvillal | vsaienk0: Thanks for working on it! | 16:19 |
| mariojv | morning dtantsur | 16:21 |
| vdrok | morning jlvillal | 16:22 |
| jlvillal | Hi vdrok mariojv vsaienk0 NobodyCam dtantsur rloo jroll TheJulia JayF and everyone else :) | 16:23 |
| mariojv | hi jlvillal | 16:23 |
| TheJulia | o/ jlvillal from someplace over north america | 16:23 |
| NobodyCam | Morning jlvillal :) | 16:23 |
| rloo | hi jlvillal, dtantsur | 16:23 |
| jlvillal | :) | 16:24 |
| jlvillal | TheJulia: Fancy buying that WiFi access ;) | 16:24 |
| TheJulia | Day passes kind of make sense though :) | 16:24 |
| jlvillal | True | 16:25 |
| TheJulia | Although... monthly might make sense soon | 16:25 |
| *** pcaruana has quit IRC | 16:27 | |
| lucasagomes | folks, quick q re driver composition: With old driver style we had a conditional to verify if the library(ies) the driver is dependent on was installed at load time: https://github.com/openstack/ironic/blob/master/ironic/drivers/irmc.py#L47-L50 | 16:29 |
| *** Sukhdev has joined #openstack-ironic | 16:29 | |
| lucasagomes | but, for the new hardware types, we don't have anything were we can do that, right ? | 16:30 |
| *** jkilpatr has joined #openstack-ironic | 16:30 | |
| lucasagomes | jroll, dtantsur ^ ? | 16:30 |
| lucasagomes | I wonder if it's something we can do at the constructor of the hardware type class, e.g: https://review.openstack.org/#/c/438982/11/ironic/drivers/redfish.py | 16:32 |
| patchbot | patch 438982 - ironic - Add redfish driver | 16:32 |
| TheJulia | Hmm... | 16:34 |
| dtantsur | lucasagomes, we can probably do it, when e.g. all power methods depend on the given library | 16:35 |
| aarefiev | jroll, dtantsur: could you w+1 on https://review.openstack.org/#/c/443878/1 please | 16:35 |
| patchbot | patch 443878 - python-ironic-inspector-client (stable/ocata) - Updated from global requirements | 16:35 |
| rloo | lucasagomes: it seems like we should move/add those checks to the *-interfaces. | 16:35 |
| rloo | dtantsur, lucasagomes: then we have to decide if we want to check that when a conductor starts up (ie do we assume if those interfaces are enabled even if not used, that dependencies must exist) | 16:36 |
| rloo | dtantsur, lucasagomes: or do that check when a driver is actually created. | 16:36 |
| lucasagomes | dtantsur, rloo right... yeah adding the checks to the interfaces constructors makes sense to me | 16:37 |
| lucasagomes | rloo, which will probably fail at conductor start up | 16:37 |
| lucasagomes | (if enabled) | 16:37 |
| rloo | lucasagomes: ok then, i can't recall now, when we first access those driver interfaces :) | 16:38 |
| lucasagomes | rloo, yeah me neither, I need to test it out | 16:38 |
| lucasagomes | cause before it was at load time and would fail at conductor startup if we couldn't load the driver | 16:38 |
| lucasagomes | so I think we should keep the bahaviour, if possible | 16:38 |
| rloo | lucasagomes: that makes sense to me | 16:39 |
| rloo | lucasagomes: it's a bug, not an rfe, right? | 16:39 |
| lucasagomes | rloo, IMHO, bug | 16:39 |
| lucasagomes | maybe could be appended to the driver composition work so it would be part of it | 16:40 |
| openstackgerrit | Annie Lezil proposed openstack/ironic-python-agent master: Collect NIC name given by BIOS https://review.openstack.org/396406 | 16:40 |
| lucasagomes | but, whatever way seems good to me | 16:40 |
| rloo | lucasagomes: yeah, i was just thinking that. would be good to open a bug still, and then add a link to that bug in the etherpad subteam thing. | 16:40 |
| *** causten has quit IRC | 16:40 | |
| lucasagomes | rloo, I will open a bug | 16:40 |
| lucasagomes | 1 sec | 16:40 |
| rloo | lucasagomes: cuz the main driver comp bug was closed i think | 16:40 |
| jroll | lucasagomes: I think we init all of the interfaces at startup, so you could do it there... which as I read scrollback I think you're getting to | 16:41 |
| *** causten has joined #openstack-ironic | 16:41 | |
| lucasagomes | jroll, yeah that's what I was planning to test | 16:41 |
| *** stendulker has joined #openstack-ironic | 16:41 | |
| lucasagomes | so probably it's more about documenting how to in the developer documentation (if it works) | 16:41 |
| jroll | lucasagomes: yeah +1 | 16:42 |
| lucasagomes | anyway, lemme try it out (/me needs to build a devstack env) | 16:42 |
| *** rama_ has joined #openstack-ironic | 16:43 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic-python-agent master: Collect NIC name given by BIOS https://review.openstack.org/396406 | 16:43 |
| *** rbudden has joined #openstack-ironic | 16:44 | |
| *** rama_ is now known as Guest53956 | 16:44 | |
| *** Guest53956 is now known as rama_y | 16:45 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: devstack: When Python 3 enabled, use Python 3 https://review.openstack.org/445636 | 16:45 |
| *** causten has quit IRC | 16:45 | |
| *** erolg has quit IRC | 16:47 | |
| *** jaosorior has quit IRC | 16:50 | |
| dtantsur | reminder: please provide status updates for the meeting! | 16:51 |
| *** jaosorior has joined #openstack-ironic | 16:52 | |
| *** nicodemos has joined #openstack-ironic | 16:54 | |
| *** Serlex has quit IRC | 16:55 | |
| *** vmud213 has joined #openstack-ironic | 16:56 | |
| mgould | TheJulia, ppiela, robcresswell: how do I run JS unit tests for ironic-ui? | 16:57 |
| mgould | I've tried creating one that ought to fail and running `tox -e selenium`, but it's not failing | 16:58 |
| jlvillal | Meeting in about two minutes over in #openstack-meeting-3 | 16:58 |
| * jlvillal read selenium and for a moment thought he was in the wrong channel | 16:58 | |
| dtantsur | lol | 16:58 |
| jlvillal | I did, I double checked :) | 16:59 |
| *** Sukhdev has quit IRC | 17:01 | |
| *** Sukhdev_ is now known as Sukhdev | 17:01 | |
| *** vmud213 has quit IRC | 17:01 | |
| *** vmud213 has joined #openstack-ironic | 17:01 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic-python-agent master: Collect NIC name given by BIOS https://review.openstack.org/396406 | 17:01 |
| ppiela | mgould: one way is http://<horizon-host>:8000/jasmine/?spec=horizon.dashboard.admin.ironic | 17:01 |
| mgould | ppiela: thanks! | 17:02 |
| jlvillal | lucasagomes: How do you decide on who is in "ironic-staging-drivers-core" ? | 17:03 |
| mgould | ppiela: now all the node-details specs are failing :-) | 17:04 |
| lucasagomes | jlvillal, there's no methodology really... I just added people that were interested and I trusted (all ironic cores so far) | 17:05 |
| lucasagomes | jlvillal, maybe one day we can have staging-drivers under ironic umbrella as well :-) | 17:05 |
| jlvillal | lucasagomes: Cool. Thanks :) | 17:05 |
| *** fragatina has joined #openstack-ironic | 17:05 | |
| * lucasagomes thinks it would be good | 17:06 | |
| * jlvillal agrees | 17:06 | |
| *** stendulker has quit IRC | 17:06 | |
| *** fragatin_ has joined #openstack-ironic | 17:06 | |
| mgould | ppiela: still fails after I checked out the master branch :-( | 17:07 |
| mgould | nodeUuid is undefined in http://localhost:8000/static/dashboard/admin/ironic/node-details/node-details.controller.spec.js | 17:07 |
| mgould | line 27 | 17:07 |
| ppiela | mgould: I will check | 17:08 |
| mgould | there's a declaration/assignment of that variable on line 22 :-( | 17:09 |
| *** vmud213 has quit IRC | 17:09 | |
| *** vmud213 has joined #openstack-ironic | 17:10 | |
| *** fragatina has quit IRC | 17:10 | |
| *** vmud213 has quit IRC | 17:11 | |
| *** vmud213 has joined #openstack-ironic | 17:11 | |
| *** mat128 has joined #openstack-ironic | 17:13 | |
| ppiela | mgould: Fails for me also, I will fix | 17:13 |
| mgould | ppiela: thanks! | 17:14 |
| mgould | so I guess those tests aren't run in CI? | 17:14 |
| *** aNuposic has joined #openstack-ironic | 17:14 | |
| ppiela | mgould: I have wondered about that, but would assume not | 17:15 |
| *** vmud213 has quit IRC | 17:17 | |
| vdrok | lucasagomes: mind approving a tiny patch? https://review.openstack.org/447560 | 17:19 |
| patchbot | patch 447560 - ironic-staging-drivers - Specify ironic_staging_drivers as name of the app | 17:19 |
| vdrok | stolen from jlvillal :) | 17:19 |
| lucasagomes | vdrok, hi there, /me looks | 17:19 |
| jlvillal | vdrok: :) | 17:19 |
| lucasagomes | vdrok, done! | 17:19 |
| vdrok | thx! | 17:19 |
| *** dyasny has quit IRC | 17:25 | |
| *** mat128 has quit IRC | 17:29 | |
| *** fragatin_ has quit IRC | 17:30 | |
| *** kromanenko has quit IRC | 17:30 | |
| *** mat128 has joined #openstack-ironic | 17:34 | |
| *** mat128 has quit IRC | 17:34 | |
| *** jaosorior has quit IRC | 17:35 | |
| mariojv | jkilpatr: on https://bugs.launchpad.net/ironic/+bug/1672457 - is this reproducible in ocata? i'm also curious if the config option for # periodic task workers is the default | 17:38 |
| openstack | Launchpad bug 1672457 in Ironic "Ironic Conductor performance trends down with uptime" [Medium,Confirmed] | 17:38 |
| *** dyasny has joined #openstack-ironic | 17:38 | |
| jkilpatr | mariojv, no one's got a sizable ocata cloud up to test yet. I have working builds on a small scale, I could adjust their refresh rate and then leave them for a day or two to find out. | 17:39 |
| mariojv | cool, was just curious | 17:39 |
| vdrok | jkilpatr: also, could you add memory graphs too? | 17:39 |
| jkilpatr | mariojv, it's really obvious on bigger clouds since the slope is determined by node count, rook had a couple hundred node cloud that slowed significantly over a weekend. | 17:39 |
| jkilpatr | vdrok, gimme 5 minutes | 17:40 |
| vdrok | sure, thanks :) | 17:40 |
| mariojv | jkilpatr: how many nodes/conductor? | 17:40 |
| mariojv | i found that 200 nodes/conductor scales well | 17:40 |
| *** baoli has quit IRC | 17:40 | |
| mariojv | but that's on an old release | 17:40 |
| jkilpatr | mariojv, no tuning, so whatever tripleo deploys with, which is I think one conductor on the undercloud? (someone correct me please) | 17:41 |
| mariojv | that'd likely be an issue | 17:41 |
| jroll | wanyen: I don't know what you're talking about with host aggregates. do you have a reference for "The nova os_traits said that it addresses the problem for host aggregate"? and yes, ironic works fine with host aggregates | 17:41 |
| *** mat128 has joined #openstack-ironic | 17:41 | |
| mariojv | perhaps jroll will discuss # of conductors in the reference architecture he's making | 17:41 |
| jroll | mariojv: I don't have any sort of lab to be able to test that to make a good statement on it | 17:41 |
| dtantsur | jkilpatr, yes, all-in-one undercloud | 17:41 |
| mariojv | fair jroll | 17:42 |
| *** chlong has quit IRC | 17:42 | |
| *** baoli has joined #openstack-ironic | 17:42 | |
| *** baoli has quit IRC | 17:42 | |
| jkilpatr | vdrok, https://snapshot.raintank.io/dashboard/snapshot/WULvdWAGoqvwc5OOIlm8TH4kDAa3Uq2s?panelId=72&fullscreen | 17:43 |
| jkilpatr | click on various ironic processes to see their memory usage too, going to add that to the ticket | 17:43 |
| vdrok | thank you | 17:44 |
| jkilpatr | vdrok, let me know if you need any other data. | 17:44 |
| *** baoli has joined #openstack-ironic | 17:46 | |
| *** alineb has quit IRC | 17:48 | |
| jlvillal | jkilpatr: Kind of surprised the API service takes so much memory. That is only surprised from a gut feeling and not exploring what it allocated compared to the conductor. | 17:48 |
| *** mat128 has quit IRC | 17:48 | |
| jkilpatr | jlvillal, yeha less than a gig but nontrival | 17:50 |
| jlvillal | jkilpatr: Oh, I was reading that as 3 GB. | 17:51 |
| jlvillal | jkilpatr: ironic-api Virtual | 17:51 |
| jkilpatr | jlvillal, virtual includes any memory it could possibly be said to be using, so libraries that more than one process uses, swap it may be using etc | 17:52 |
| jkilpatr | rss is actual physical ram usage for that specific process (aka what you would get back by killing the process) | 17:53 |
| openstackgerrit | Miles Gould proposed openstack/ironic-ui master: Document how to run JavaScript unit tests https://review.openstack.org/447624 | 17:55 |
| *** jcoufal_ has joined #openstack-ironic | 18:00 | |
| * jlvillal intentionally picks the smallest of all the ironic thingees :) | 18:00 | |
| *** rbudden has quit IRC | 18:00 | |
| *** kaifeng has joined #openstack-ironic | 18:00 | |
| mjturek | dtantsur: I can handle the ironic/common code for translation cleanup | 18:00 |
| jlvillal | dtantsur: Or I will see if one of my co-workers wants to do ironic-lib. If not, then I will take it. | 18:01 |
| rloo | jlvillal: just assign yourself or whomever on the bug | 18:01 |
| dtantsur | mjturek, joanna, jlvillal, let's create an etherpad, and spread the work there please | 18:01 |
| mjturek | sounds good | 18:01 |
| jlvillal | rloo: Is there a bug? Can I have the link? | 18:01 |
| rloo | jlvillal: in the etherpad, i have it noted there | 18:02 |
| lucasagomes | calling it a day, have a great evening all! | 18:02 |
| jlvillal | rloo: Thanks | 18:02 |
| *** mgould is now known as mgould|afk | 18:02 | |
| *** derekh has quit IRC | 18:02 | |
| *** lucasagomes is now known as lucas-afk | 18:02 | |
| *** ralonsoh has quit IRC | 18:02 | |
| *** jcoufal has quit IRC | 18:03 | |
| mjturek | dtantsur, joanna, jlvillal: https://etherpad.openstack.org/p/ironic-translation-cleanup something like this? | 18:04 |
| joanna | mjturek: great! | 18:05 |
| vdrok | good night everyone! | 18:05 |
| mjturek | night vdrok! | 18:06 |
| *** mat128 has joined #openstack-ironic | 18:06 | |
| *** aarefiev is now known as aarefiev_pto | 18:06 | |
| *** mgoddard has quit IRC | 18:07 | |
| joanna | vdrok: gnight! | 18:07 |
| *** yogi has joined #openstack-ironic | 18:07 | |
| jlvillal | rloo: So if I have a LOG.exception(_LE("foo")) | 18:08 |
| jlvillal | Should that become: LOG.exception(_("foo")) ? | 18:09 |
| rloo | jlvillal: to be honest, i don't know the details. i am just the messenger :) | 18:09 |
| jlvillal | Or: LOG.exception("foor") | 18:09 |
| jlvillal | s/foor/foo/ | 18:09 |
| *** athomas has quit IRC | 18:09 | |
| rloo | jlvillal: that is an exception so doesn't need any | 18:09 |
| * soliosg suggests to have a common gerrit topic, so patches are quickly spotted | 18:10 | |
| rloo | jlvillal: i mean, that is a log | 18:10 |
| jlvillal | rloo: Okay. Thanks. | 18:10 |
| rloo | soliosg: please use the bug number. that's what the bug is for :) | 18:10 |
| *** baoli has quit IRC | 18:10 | |
| *** mtanino has joined #openstack-ironic | 18:11 | |
| *** e0ne has quit IRC | 18:11 | |
| rloo | jlvillal: vs the raise Exception(msg) which should be translated. unless someone changes that too, but we're only talking about logs now. | 18:11 |
| *** mat128 has quit IRC | 18:11 | |
| soliosg | rloo: oops, forgot there's a bug ;), yes let's use the bug number | 18:11 |
| jlvillal | rloo: makes sense | 18:11 |
| *** baoli has joined #openstack-ironic | 18:11 | |
| aNuposic | i can take up for ironic drivers if that helps. Are we doing sub-project wise? | 18:11 |
| dtantsur | jlvillal, LOG.exception loses _LE | 18:11 |
| dtantsur | it's just for logging the traceback | 18:12 |
| jlvillal | dtantsur: Thanks | 18:12 |
| mjturek | jlvillal: sample nova patch https://review.openstack.org/#/c/447466/ | 18:12 |
| patchbot | patch 447466 - nova - remove log translation from nova.api.metadata | 18:12 |
| dtantsur | g'night, see you tomorrow | 18:12 |
| *** dtantsur is now known as dtantsur|afk | 18:12 | |
| aNuposic | rloo, dtantsur I can take up to remove markers from ironic drivers | 18:12 |
| dtantsur|afk | aNuposic, please use https://etherpad.openstack.org/p/ironic-translation-cleanup to sync. thansk! | 18:13 |
| *** causten has joined #openstack-ironic | 18:13 | |
| aNuposic | dtantsur|afk, cool | 18:13 |
| soliosg | jlvillal: from what i recall reading the log wiki, we should use _() if we propagating the message outside the context (raising an exception) | 18:14 |
| rama_y | mjturek, thanks! | 18:14 |
| *** ChubYann has joined #openstack-ironic | 18:15 | |
| openstackgerrit | Xavier proposed openstack/ironic master: Introduces onetime boot flag in oneview drivers https://review.openstack.org/436469 | 18:16 |
| *** baoli has quit IRC | 18:16 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-lib master: Remove translation of log messages https://review.openstack.org/447633 | 18:17 |
| jlvillal | ironic-lib was kind of easy :) | 18:17 |
| openstackgerrit | Merged openstack/ironic master: Fix usage of various deprecated methods https://review.openstack.org/413113 | 18:18 |
| *** bcornec has joined #openstack-ironic | 18:18 | |
| *** nicodemos has quit IRC | 18:20 | |
| jlvillal | rloo: https://review.openstack.org/447633 :) | 18:20 |
| patchbot | patch 447633 - ironic-lib - Remove translation of log messages | 18:20 |
| rloo | jlvillal: what is tt.py.save? | 18:21 |
| jlvillal | rloo: Dang it! | 18:21 |
| soliosg | jlvillal: in the ironic-lib patch ^^, you left an extra pair of parentheses, are they needed? | 18:21 |
| rloo | jlvillal: ha ha | 18:21 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-lib master: Remove translation of log messages https://review.openstack.org/447633 | 18:22 |
| aNuposic | mariojv, are you planning to rebase https://review.openstack.org/#/c/400437/ today? | 18:22 |
| patchbot | patch 400437 - ironic - Agent driver rescue implementation | 18:22 |
| *** causten has quit IRC | 18:22 | |
| mariojv | aNuposic: tomorrow morning | 18:22 |
| jlvillal | soliosg: What do you mean? Where? | 18:22 |
| mariojv | aNuposic: i have a lot of comments to respond to on the previous patches, as well | 18:22 |
| aNuposic | mariojv, okay cool | 18:23 |
| *** fragatina has joined #openstack-ironic | 18:23 | |
| jlvillal | soliosg: I tried to remove the unneeded ones | 18:23 |
| soliosg | jlvillal: example, LOG.warning(('warning string here'), {dict}) | 18:23 |
| jlvillal | soliosg: link? | 18:24 |
| *** mat128 has joined #openstack-ironic | 18:24 | |
| aNuposic | mariojv, yup, also Jay was saying is there a need to create a rescue user? I mean we can do all the operations with root | 18:24 |
| jlvillal | soliosg: Or leave a review on patch set 2, with a minus 1 for that | 18:24 |
| soliosg | jlvillal: https://review.openstack.org/#/c/447633/1/ironic_lib/disk_partitioner.py@134 | 18:24 |
| patchbot | patch 447633 - ironic-lib - Remove translation of log messages | 18:24 |
| mariojv | aNuposic: i don't want to give the user a privileged user by default | 18:25 |
| jlvillal | soliosg: You need the parenthesis there. The string spans multiple lines. | 18:25 |
| mariojv | s/privileged user/root user | 18:25 |
| *** causten has joined #openstack-ironic | 18:25 | |
| soliosg | jlvillal: my bad | 18:25 |
| jlvillal | soliosg: Well I think it is better to have it. For the single line ones I deleted the parenthesis. Multi-line ones I kept them. | 18:25 |
| *** crushil has quit IRC | 18:26 | |
| jlvillal | soliosg: It might not be completely necessary, but I think it helps. I know in some situations you need it and others you don't. So for consistency I use them on multi-line strings. | 18:26 |
| openstackgerrit | Xavier proposed openstack/ironic master: Introduces onetime boot flag in oneview drivers https://review.openstack.org/436469 | 18:26 |
| *** baoli has joined #openstack-ironic | 18:28 | |
| aNuposic | mariojv, I see. That makes sense, so if you are gonna keep the rescue user i can change my patch accordingly. | 18:28 |
| mariojv | ok | 18:28 |
| *** mat128 has quit IRC | 18:29 | |
| *** vsaienko has quit IRC | 18:29 | |
| *** mat128 has joined #openstack-ironic | 18:31 | |
| *** kaifeng has quit IRC | 18:31 | |
| *** mjura has quit IRC | 18:31 | |
| *** baoli has quit IRC | 18:32 | |
| *** rbudden has joined #openstack-ironic | 18:32 | |
| *** crushil has joined #openstack-ironic | 18:34 | |
| openstackgerrit | Merged openstack/ironic-ui master: Fix typo in base-node.service.js https://review.openstack.org/446821 | 18:35 |
| openstackgerrit | Merged openstack/ironic-ui master: Fix docstrings for node validation https://review.openstack.org/443760 | 18:35 |
| openstackgerrit | Merged openstack/ironic-ui master: s/UUID/UUID or name/ in docstrings https://review.openstack.org/444954 | 18:35 |
| openstackgerrit | Merged openstack/ironic-ui master: Correct Release Notes theme https://review.openstack.org/439144 | 18:35 |
| *** baoli has joined #openstack-ironic | 18:38 | |
| *** bcornec has quit IRC | 18:40 | |
| rloo | jroll: https://etherpad.openstack.org/p/BOS-TC-vm-baremetal-platform | 18:40 |
| rloo | jroll: for ironic one, i guess we want to do things separate from that | 18:41 |
| rloo | TheJulia, mariojv, dtantsur|afk: I also added the BOS-ironic-brainstorming link to https://wiki.openstack.org/wiki/Forum/Boston2017 | 18:42 |
| *** mgoddard has joined #openstack-ironic | 18:43 | |
| jroll | rloo: can you rephrase your question? :) | 18:44 |
| rloo | mariojv, TheJulia: I haven't been paying attention, but here are deadlines, today is end of brainstorming phase... http://lists.openstack.org/pipermail/openstack-dev/2017-March/114002.html | 18:44 |
| *** baoli has quit IRC | 18:44 | |
| rloo | jroll: heh, just wanted to be clear, whatever brainstorming in ironic forum thingy, shouldn't overlap with stuff in that other thing | 18:44 |
| rloo | jroll: not that there is any ironic brainstorming yet so maybe doesn't matter. | 18:45 |
| jroll | rloo: I don't mind if it overlaps a bit, but yeah, is separate | 18:45 |
| mariojv | great | 18:46 |
| openstackgerrit | Varun Gadiraju proposed openstack/ironic master: Remove translation of log messages from ironic/dhcp directory https://review.openstack.org/447642 | 18:46 |
| rloo | jroll: ok | 18:46 |
| openstackgerrit | Merged openstack/ironic-lib master: Add missing 'autospec' statements to unit test mocks https://review.openstack.org/447248 | 18:47 |
| *** crushil has quit IRC | 18:50 | |
| *** bcornec has joined #openstack-ironic | 18:50 | |
| rpioso | aNuposic: I'd like to help with the translation cleanup in drivers. | 18:54 |
| *** causten has quit IRC | 18:55 | |
| rpioso | aNuposic: How 'bout if I take drivers/modules/drac? | 18:55 |
| aNuposic | rpioso, sure | 18:55 |
| aNuposic | rpioso, go ahead | 18:56 |
| rpioso | aNuposic: I'll note that on the etherpad. | 18:56 |
| rpioso | aNuposic: ty | 18:56 |
| aNuposic | rpioso, yes please. anytime :) | 18:56 |
| openstackgerrit | Varun Gadiraju proposed openstack/ironic master: Remove translation of log messages from ironic/dhcp and ironic/cmd https://review.openstack.org/447642 | 18:57 |
| *** Nisha_Agarwal has quit IRC | 18:57 | |
| *** sbezverk has joined #openstack-ironic | 18:59 | |
| sbezverk | hello, my baremetal node gets incorrect tftp address to download pxelinux.0 I cannot figure out where doesw it taking from? any hints? | 19:00 |
| mariojv | jlvillal: where is the enable_python3_package function defined here ? https://review.openstack.org/#/c/445636/5/devstack/plugin.sh | 19:03 |
| patchbot | patch 445636 - ironic - devstack: When Python 3 enabled, use Python 3 | 19:03 |
| mariojv | jlvillal: i did not find it here | 19:03 |
| *** cdearborn has joined #openstack-ironic | 19:03 | |
| mariojv | https://github.com/openstack/ironic/blob/master/devstack/lib/ironic | 19:03 |
| mariojv | presumably it's somewhere in the actual devstack repo? but i'm having trouble untangling the dependency graph | 19:04 |
| jroll | mariojv: yeah, that's a devstack thing | 19:04 |
| *** racedo has quit IRC | 19:04 | |
| * jroll looks | 19:04 | |
| *** crushil has joined #openstack-ironic | 19:05 | |
| jroll | inc/python line 170 | 19:05 |
| mariojv | jroll: https://github.com/openstack-dev/devstack/blob/6523d6e09789076d0a430bf7b608511fcb1c406a/inc/python#L170 | 19:05 |
| mariojv | yup | 19:05 |
| jroll | ya | 19:05 |
| mariojv | thx | 19:05 |
| jroll | np | 19:05 |
| mariojv | jlvillal: what about python-openstackclient? | 19:05 |
| *** [1]cdearborn has quit IRC | 19:05 | |
| * mariojv will just leave that as a comment | 19:05 | |
| mariojv | ah, there's a todo | 19:05 |
| mariojv | nvm | 19:05 |
| sbezverk | mgoddard: ping | 19:06 |
| mgoddard | hi sbezverk | 19:06 |
| *** baoli has joined #openstack-ironic | 19:07 | |
| jroll | I'm not feeling well, I'm gonna bounce for the day, see y'all tomorrow | 19:08 |
| aNuposic | jlvillal, wdyt about raising NotAcceptable or Invalid when one of the params are missing while initializing a connection with radosgw, do you think of other exception suitable? | 19:09 |
| mariojv | hope you feel better tomorrow jim | 19:11 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-lib master: Remove translation of log messages https://review.openstack.org/447633 | 19:11 |
| jlvillal | aNuposic, Not sure off the top of my head. Maybe see if there are other instances of code doing something similar | 19:12 |
| jroll | thanks | 19:12 |
| jlvillal | mariojv, python-openstackclient isn't our project. | 19:13 |
| jlvillal | mariojv, Or are you talking about the Python 3 stuff. | 19:13 |
| jlvillal | mariojv, Sorry at lunch and sorta reading messages. I'll reply when I am back :) | 19:13 |
| sbezverk | mgoddard: qq do you know how to configure tftp server ip for pool used by baremetal nodes? | 19:13 |
| sbezverk | mgoddard: I am getting my baremetal vm up but neutron gives wrong tftp so it cannot load pxelinux.0 | 19:14 |
| *** baoli has quit IRC | 19:14 | |
| mgoddard | sbezverk: are you using devstack? | 19:15 |
| sbezverk | mgoddard: no I am kolla-kubernetes and it is almost working :) | 19:16 |
| mgoddard | sbezverk: ah, ok. it's in ironic.conf: [pxe] tftp_server | 19:17 |
| sbezverk | mgoddard: tftp_server = X.X.X.X where ip address of pxe listening interface rirght? | 19:18 |
| sbezverk | mgoddard: what is confusing neutron's dhcp gives that address | 19:19 |
| sbezverk | so I am not sure at what point pxe's tftp ip will be injected into neutron dhcp address reply | 19:19 |
| openstackgerrit | Varun Gadiraju proposed openstack/ironic master: Remove translation of log messages from ironic/dhcp and ironic/cmd https://review.openstack.org/447642 | 19:25 |
| *** baoli has joined #openstack-ironic | 19:25 | |
| TheJulia | rloo: Well, that is totally awesome :( | 19:25 |
| mariojv | jlvillal: oh, i thought devstack tests might be using it | 19:25 |
| mariojv | nevermind hten | 19:26 |
| mariojv | *then | 19:26 |
| *** vsaienko has joined #openstack-ironic | 19:26 | |
| mariojv | easy review for any stabel/ocata cores https://review.openstack.org/#/c/443804/ | 19:29 |
| patchbot | patch 443804 - ironic-python-agent (stable/ocata) - Updated from global requirements | 19:29 |
| mgoddard | sbezverk: yes that's right | 19:30 |
| TheJulia | rloo: mariojv: relistically, the tc/vm/baremetal session covers a good chunk of what I'd want to talk about. I guess really we don't have anything unless mariojv does? | 19:30 |
| mgoddard | sbezverk: you should be able to see it as config on the neutron port | 19:30 |
| *** nicodemos has joined #openstack-ironic | 19:31 | |
| mariojv | TheJulia: i still need to look over the session list | 19:31 |
| mariojv | TheJulia: but an operator session focused specifically on ironic would be good imho | 19:31 |
| mariojv | wondering if there are any standalone users that wouldn't be in that meeting, for example | 19:32 |
| mariojv | TheJulia: I will certainly attend that one, though, looks like it'll be a productive session | 19:32 |
| TheJulia | I _really_ wish I had enough foresight/lead time to be at the ops mid cycle because I think it would have been revealing | 19:33 |
| mariojv | for sure | 19:33 |
| sbezverk | mgoddard: thank you | 19:35 |
| mariojv | TheJulia: BTW, it looks like this is attempting a strategy to kill consoles when a conductor goes down that goes against what you were proposing with subprocessing: https://review.openstack.org/#/c/446571 | 19:35 |
| patchbot | patch 446571 - ironic - Ensure dying SOL console on conductor death | 19:35 |
| mariojv | i like the one you were exploring better, since that'd cover more cases (i think) | 19:35 |
| TheJulia | I just feel like even though we have a good number of operators that directly interact with this community, that there is either context lost or disconnect that I'd love to see bridged (not that it truly can be) | 19:36 |
| *** crushil has quit IRC | 19:37 | |
| *** [1]cdearborn has joined #openstack-ironic | 19:37 | |
| *** cdearborn has quit IRC | 19:39 | |
| mariojv | TheJulia: i'd be good with a mixed session where we both hear about things we can make better for operators (code and docs we can write), and have part of the session for more "how do we get better at community" wrt operators | 19:39 |
| mariojv | i personally feel like we're pretty responsive in irc, have good docs, etc., but i was on a team that had operators engaged with upstream dev, so kind of biased | 19:40 |
| mariojv | how does one go about proposing a summit session like this ^ ? | 19:40 |
| *** crushil has joined #openstack-ironic | 19:41 | |
| mariojv | for some reason filtering isn't working for me at https://www.openstack.org/summit/boston-2017/summit-schedule/ | 19:41 |
| mariojv | i guess maybe the selection already happened? | 19:42 |
| TheJulia | mariojv: I kind of like the patch as a one size fits all kind of thing. Wax it all if it is the namespace and move on. Processes exiting on their own doesn't always work, especially with udp sockets and ipmitool being involved :( | 19:43 |
| *** krtaylor has quit IRC | 19:43 | |
| mariojv | OK - i'll update with that irc comment. still awaiting reno but that's a different thing | 19:43 |
| TheJulia | mariojv: thank you :) | 19:43 |
| TheJulia | I _think_ the default kernel timeout for a udp socket is something like 12 hours too.... | 19:44 |
| TheJulia | mariojv: the schedule seems to load for me, at least in safari | 19:45 |
| mariojv | the schedule itself will load, but not the filters | 19:45 |
| mariojv | unless i click on a tag from a presentation (not typing them in manually) | 19:45 |
| mariojv | it's fine. i presume that whatever we put in the brainstorm etherpad will be taken into consideration :) | 19:46 |
| TheJulia | mariojv: works for me.... | 19:46 |
| mariojv | "The formal submission tool will be announced as open after a few weeks of brainstorming | 19:47 |
| patchbot | Error: No closing quotation | 19:47 |
| mariojv | " | 19:47 |
| patchbot | Error: No closing quotation | 19:47 |
| mariojv | from https://wiki.openstack.org/wiki/Forum | 19:47 |
| * TheJulia just stops and thinks for a moment... | 19:48 | |
| *** mat128 has quit IRC | 19:54 | |
| anish | dtantsur|afk: still waiting on you for https://review.openstack.org/#/c/401415/ | 19:55 |
| patchbot | patch 401415 - python-dracclient - Fetch system attributes | 19:55 |
| TheJulia | [1]cdearborn: ^^^ may be of interest to you | 19:57 |
| *** e0ne has joined #openstack-ironic | 19:58 | |
| mariojv | soliosg: is the added import here OK, with all the tempest manager.py shenanigans going on? https://review.openstack.org/#/c/435952/12/ironic_tempest_plugin/tests/api/admin/test_nodes.py | 19:59 |
| patchbot | patch 435952 - ironic - Add negative tests for VIF attach/detach operations | 19:59 |
| mariojv | i'd presume so but wanted to check with you first | 19:59 |
| *** baoli has quit IRC | 20:01 | |
| *** baoli has joined #openstack-ironic | 20:01 | |
| *** baoli has quit IRC | 20:02 | |
| jlvillal | mariojv: the Python 3 job passed the tests. | 20:04 |
| mariojv | \o/ | 20:04 |
| TheJulia | woot | 20:04 |
| jlvillal | No not those tests ;) | 20:04 |
| mariojv | jlvillal: approved | 20:04 |
| jlvillal | Just the gate tests. Still fails for Python 3. | 20:04 |
| jlvillal | mariojv: Thanks :) | 20:04 |
| mariojv | jlvillal: right, that's what i meant by "the tests" | 20:04 |
| mariojv | :P | 20:04 |
| jlvillal | I'm thinking that making jobs pass with Python 3 is going to be harder than I thought. | 20:05 |
| *** dprince has quit IRC | 20:05 | |
| jlvillal | Sadly :( | 20:05 |
| openstackgerrit | Merged openstack/ironic master: Validation before perform node deallocation https://review.openstack.org/386014 | 20:05 |
| *** linggao has joined #openstack-ironic | 20:07 | |
| soliosg | mariojv: checking about that 'from tempest import test' | 20:08 |
| *** baoli has joined #openstack-ironic | 20:12 | |
| mariojv | thanks soliosg | 20:12 |
| mariojv | soliosg: i -1'ed for other reasons, but if it's an issue feel free to leave that as a comment on the patch itself | 20:13 |
| *** kencjohnston_ has joined #openstack-ironic | 20:15 | |
| *** francoblanco_ has joined #openstack-ironic | 20:15 | |
| *** Sukhdev_ has joined #openstack-ironic | 20:17 | |
| *** zhenguo_ has joined #openstack-ironic | 20:18 | |
| *** makowals_ has joined #openstack-ironic | 20:18 | |
| *** lucas-afk has quit IRC | 20:18 | |
| Sukhdev_ | jroll : After cleaning things a bit, I was able to get everything going. When I launch "nova boot" it returns no hosts found. All the standard stuff looks good. See it here - http://paste.openstack.org/show/603493/ | 20:18 |
| *** ijw has quit IRC | 20:18 | |
| *** rbudden has quit IRC | 20:19 | |
| *** galyna1 has joined #openstack-ironic | 20:19 | |
| *** sai has quit IRC | 20:20 | |
| *** rook has quit IRC | 20:20 | |
| *** vsaienk01 has joined #openstack-ironic | 20:20 | |
| *** bl0m1_ has joined #openstack-ironic | 20:21 | |
| *** bcornec2 has joined #openstack-ironic | 20:21 | |
| vsaienko | jlvillal, shouldn't ironic running under py3.5 with https://review.openstack.org/#/c/445636/ on py3 experimental job? | 20:21 |
| patchbot | patch 445636 - ironic - devstack: When Python 3 enabled, use Python 3 | 20:21 |
| *** rook has joined #openstack-ironic | 20:21 | |
| *** lucasagomes has joined #openstack-ironic | 20:21 | |
| *** krtaylor has joined #openstack-ironic | 20:21 | |
| *** rook is now known as Guest45025 | 20:21 | |
| *** andyjones2001 has joined #openstack-ironic | 20:22 | |
| *** sai has joined #openstack-ironic | 20:22 | |
| jlvillal | vsaienko: That is the goal... | 20:22 |
| *** jmccrory_ has joined #openstack-ironic | 20:22 | |
| *** e0ne has quit IRC | 20:22 | |
| jlvillal | vsaienko: i think we can still land that though. Yes Python 3 job doesn't work yet. But no reason to stop that patch. | 20:23 |
| vsaienko | jlvillal, do you know why it is still launched under python2 http://logs.openstack.org/36/445636/4/experimental/gate-tempest-dsvm-ironic-ipa-partition-pxe_ipmitool-tinyipa-python3-ubuntu-xenial-nv/96fb7f4/logs/devstacklog.txt.gz#_2017-03-16_18_00_53_662 | 20:23 |
| jlvillal | vsaienko: I said in the commit message it doesn't work. | 20:23 |
| jlvillal | vsaienko: But I see it installing it with Python 3. | 20:23 |
| *** baoli has quit IRC | 20:23 | |
| vsaienko | jlvillal, where it installing ironic under python3? I see in the logs that it is installed under python2 | 20:24 |
| *** sdake_ has joined #openstack-ironic | 20:24 | |
| *** sdake_ has quit IRC | 20:24 | |
| *** sdake_ has joined #openstack-ironic | 20:24 | |
| vsaienko | jlvillal, please have look http://logs.openstack.org/36/445636/4/experimental/gate-tempest-dsvm-ironic-ipa-partition-pxe_ipmitool-tinyipa-python3-ubuntu-xenial-nv/96fb7f4/logs/devstacklog.txt.gz#_2017-03-16_18_00_53_662 | 20:24 |
| *** Sukhdev__ has joined #openstack-ironic | 20:25 | |
| *** jcoufal has joined #openstack-ironic | 20:26 | |
| *** xhku_ has joined #openstack-ironic | 20:26 | |
| *** Guest45025 is now known as rook-ugh | 20:27 | |
| *** bcornec has quit IRC | 20:29 | |
| *** jcoufal_ has quit IRC | 20:29 | |
| *** jkilpatr has quit IRC | 20:29 | |
| *** Sukhdev has quit IRC | 20:29 | |
| *** makowals has quit IRC | 20:29 | |
| *** francoblanco has quit IRC | 20:29 | |
| *** soliosg has quit IRC | 20:29 | |
| *** vsaienk0 has quit IRC | 20:29 | |
| *** galyna has quit IRC | 20:29 | |
| *** aolwas has quit IRC | 20:29 | |
| *** sdake has quit IRC | 20:29 | |
| *** zhenguo has quit IRC | 20:29 | |
| *** andyjone1 has quit IRC | 20:29 | |
| *** jmccrory has quit IRC | 20:29 | |
| *** bl0m1 has quit IRC | 20:29 | |
| *** fbouliane has quit IRC | 20:29 | |
| *** kencjohnston has quit IRC | 20:29 | |
| *** marios has quit IRC | 20:29 | |
| *** soliosg1 is now known as soliosg | 20:29 | |
| *** jmccrory_ is now known as jmccrory | 20:29 | |
| jlvillal | vsaienko: Still looking... | 20:29 |
| *** Sukhdev__ is now known as Sukhdev | 20:29 | |
| *** francoblanco_ is now known as francoblanco | 20:29 | |
| *** zhenguo_ is now known as zhenguo | 20:30 | |
| *** marios has joined #openstack-ironic | 20:31 | |
| jlvillal | vsaienko: Let's see what the experimental job result is | 20:32 |
| *** jkilpatr has joined #openstack-ironic | 20:33 | |
| vsaienko | jlvillal, ok | 20:33 |
| jlvillal | vsaienko: I had a bug before :( https://review.openstack.org/#/c/445636/4/devstack/plugin.sh@15 | 20:33 |
| patchbot | patch 445636 - ironic - devstack: When Python 3 enabled, use Python 3 | 20:33 |
| jlvillal | vsaienko: No '$' | 20:33 |
| jlvillal | But new version doesn't use the loop. | 20:33 |
| jlvillal | vsaienko: So the error was like: http://logs.openstack.org/36/445636/2/experimental/gate-tempest-dsvm-ironic-ipa-partition-pxe_ipmitool-tinyipa-python3-ubuntu-xenial-nv/3baeb6d/logs/devstacklog.txt.gz#_2017-03-14_22_54_08_546 | 20:34 |
| openstackgerrit | Anup Navare proposed openstack/ironic master: Config drive support for Ceph https://review.openstack.org/407639 | 20:34 |
| jlvillal | aNuposic: I looked at ^^^ I don't see how it is dying early. | 20:36 |
| jlvillal | aNuposic: Does it die early for you in your testing? You try to start the conductor and it fails within a few seconds? | 20:37 |
| *** baoli has joined #openstack-ironic | 20:37 | |
| *** soliosg1 has joined #openstack-ironic | 20:37 | |
| aNuposic | jlvillal, i have raised an exception | 20:38 |
| jlvillal | aNuposic: That isn't what I am asking for. I left a review comment. | 20:38 |
| aNuposic | jlvillal, so you want to make conductor die, okay will do that instead of raising an exception. | 20:39 |
| jlvillal | aNuposic: I think so. But maybe others will disagree. | 20:39 |
| aNuposic | jlvillal, do you expect to do that inside swift.py? | 20:40 |
| jlvillal | mariojv: what do you think? If your config is wrong do you want the conductor to die right away. Or just log an exception at some future point and not deploy? | 20:40 |
| mariojv | depends on the config option | 20:40 |
| mariojv | if it's something like you giving an impossible API URL to a conductor, fail right away, because everything will fail anyway | 20:41 |
| aNuposic | mariojv, it is authurl, username or password for radosgw connection | 20:41 |
| mariojv | if it's some less important config option, log an exception later | 20:41 |
| mariojv | maybe log a warning at startup | 20:41 |
| mariojv | hmm | 20:41 |
| aNuposic | mariojv, swift makes conductor die at later stage | 20:41 |
| jlvillal | mariojv: So in this case, they are saying to use radosgw for storing the config drive | 20:41 |
| mariojv | what do we do elsewhere? | 20:41 |
| mariojv | i'd say yeah preserve existing behavior with swift | 20:41 |
| aNuposic | but if you don't want to wait till swift try then exception can be raised | 20:42 |
| mariojv | from ironic's perspective, they're exactly the same except auth | 20:42 |
| aNuposic | at start | 20:42 |
| vsaienko | jlvillal, just want to make sure it works as expected, since I don't understand what is use_python3_for in the comment https://github.com/openstack-dev/devstack/blob/6523d6e09789076d0a430bf7b608511fcb1c406a/stackrc#L110 | 20:42 |
| jlvillal | mariojv: Bought if the other options aren't set it will not work. | 20:42 |
| jlvillal | s/Bought/but/ | 20:42 |
| mariojv | i feel like i haven't looked at this patch long enough to have good context | 20:42 |
| mariojv | can you leave this in a comment on the patch, and i'll look later when fully re-reviewing? | 20:42 |
| jlvillal | mariojv: No worries. I'm voting for die early. Rather than having to find out an hour or so later that it doesn't work. | 20:43 |
| mariojv | log early is another alternative | 20:43 |
| jlvillal | But I might be out-voted. | 20:43 |
| *** pmannidi has joined #openstack-ironic | 20:43 | |
| mariojv | might be better to have some degraded service compared to full control plane outage during the new deploy | 20:43 |
| jlvillal | mariojv: But then you still don't figure it out until an hour or so later when you try to deploy. Most likely. | 20:43 |
| jlvillal | When you are wondering, why didn't it deploy? | 20:43 |
| aNuposic | mariojv, so with current patch https://review.openstack.org/#/c/407639/20/ironic/common/swift.py@49 it will raise exception early | 20:44 |
| patchbot | patch 407639 - ironic - Config drive support for Ceph | 20:44 |
| jlvillal | aNuposic: What is your definition of "early"? | 20:44 |
| mariojv | like i said, i'll review later if you can summarize this in a comment (if it's not there already) :) | 20:44 |
| aNuposic | jlvillal, it will not be that late, basically it will happen when swift tries to create a container | 20:44 |
| aNuposic | jlvillal, mariojv that is when swift will throw an error | 20:44 |
| mariojv | i like matching existing behavior for other failed deploys based on config options in ironic | 20:44 |
| aNuposic | basically make conductor die | 20:45 |
| mariojv | or matching existing behavior for config options that need each other to work properly | 20:45 |
| jlvillal | aNuposic: So not until the deploy is attempted will the exception be raised. Correct? | 20:45 |
| vsaienko | JayF, please have look once have a time https://review.openstack.org/#/c/446950/ jroll told that you had experience with udevadm settle | 20:46 |
| patchbot | patch 446950 - ironic-lib - Replace udevadm settle with test -e | 20:46 |
| aNuposic | jlvillal, yes. Do you expect to check for the config options when conductor starts? | 20:47 |
| jlvillal | aNuposic: That is what I would prefer. | 20:47 |
| jlvillal | aNuposic: We will see what others think. I can be out-voted :) | 20:48 |
| mariojv | vsaienko: he's out until the 23rd of march | 20:49 |
| aNuposic | jlvillal, Okay let me wait then until mariojv and other folks take a look at it | 20:50 |
| *** alineb has joined #openstack-ironic | 20:52 | |
| *** kjw3 has joined #openstack-ironic | 20:52 | |
| mrda | Morning Ironic | 20:52 |
| joanna | morning mrda | 20:53 |
| mrda | hi joanna | 20:53 |
| jlvillal | Hi mrda ! | 20:54 |
| *** ijw has joined #openstack-ironic | 20:55 | |
| *** ijw has quit IRC | 20:56 | |
| jlvillal | vsaienko: http://logs.openstack.org/36/445636/5/experimental/gate-tempest-dsvm-ironic-ipa-partition-pxe_ipmitool-tinyipa-python3-ubuntu-xenial-nv/b4e9c4a/logs/devstacklog.txt.gz#_2017-03-20_20_41_07_730 | 20:56 |
| jlvillal | :) | 20:56 |
| *** ijw has joined #openstack-ironic | 20:56 | |
| *** bfournie has quit IRC | 21:03 | |
| vsaienko | jlvillal, thanks, I see it now :) | 21:04 |
| jlvillal | vsaienko: Thank you. Still not working. But one tiny bit fixed. Probably many more things to fix... | 21:05 |
| * jlvillal is reminded of grenade work... | 21:05 | |
| *** baoli has quit IRC | 21:09 | |
| *** baoli has joined #openstack-ironic | 21:11 | |
| *** baoli has quit IRC | 21:11 | |
| *** baoli has joined #openstack-ironic | 21:11 | |
| *** Goneri has quit IRC | 21:12 | |
| *** crushil has quit IRC | 21:12 | |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Add support for soft power transitions https://review.openstack.org/444887 | 21:13 |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Add support for starting/stopping/accessing the serial console https://review.openstack.org/437419 | 21:13 |
| openstackgerrit | Chris Smart proposed openstack/ironic master: WIP: test buildroot IPA image https://review.openstack.org/445763 | 21:15 |
| *** zackf has joined #openstack-ironic | 21:17 | |
| vsaienko | jlvillal, can't we do enable_python3_package in local.conf from project-config? | 21:18 |
| *** zackf has quit IRC | 21:20 | |
| jlvillal | vsaienko: Probably. | 21:20 |
| jlvillal | vsaienko: Just easier for us (ironic) to make changes to our devstack file than to project-config | 21:20 |
| vsaienko | jlvillal, it potentially may affect all current jobs right? | 21:21 |
| vsaienko | I mean in future, it may break us | 21:22 |
| jlvillal | vsaienko: Well the 'enable_python3_package' only updates the environment variable that is only used in Python 3 jobs. | 21:22 |
| jlvillal | vsaienko: ENABLED_PYTHON3_PACKAGES | 21:23 |
| vsaienko | jlvillal, I prefer to do it via project-config to avoid breaking stable jobs | 21:23 |
| *** amotoki has joined #openstack-ironic | 21:23 | |
| jlvillal | vsaienko: I don't see how it will break stable jobs. I'm confused. | 21:24 |
| *** baoli has quit IRC | 21:24 | |
| jlvillal | vsaienko: If a job is running under Python 2, nothing changes. | 21:24 |
| jlvillal | vsaienko: Maybe I'm not understanding what you mean. | 21:24 |
| Sukhdev_ | Dear Experts, can you take a look at this paste and see what could be missing which causes "No host available" error - http://paste.openstack.org/show/603493/ | 21:29 |
| jlvillal | vsaienko: It has no impact on stable jobs: https://github.com/openstack-dev/devstack/blob/master/inc/python#L259-L304 | 21:30 |
| *** baoli has joined #openstack-ironic | 21:32 | |
| vsaienko | jlvillal, it has no impact now, but in future this behavior may be changed without our notice. What this patch gives for us we still can test py3 job | 21:33 |
| *** baoli has quit IRC | 21:33 | |
| *** amarao has joined #openstack-ironic | 21:35 | |
| *** ijw has quit IRC | 21:35 | |
| *** bfournie has joined #openstack-ironic | 21:39 | |
| *** rbudden has joined #openstack-ironic | 21:45 | |
| openstackgerrit | Merged openstack/ironic master: devstack: When Python 3 enabled, use Python 3 https://review.openstack.org/445636 | 21:49 |
| *** zackf has joined #openstack-ironic | 21:53 | |
| *** [1]rpioso has joined #openstack-ironic | 21:53 | |
| *** mat128 has joined #openstack-ironic | 21:55 | |
| *** rpioso has quit IRC | 21:55 | |
| *** zackf has quit IRC | 21:58 | |
| *** mnaser has joined #openstack-ironic | 21:58 | |
| mnaser | hi bifrost folks, kolla is failing to build containers because of this: {{ ansible_env.SUDO_USER }}: 'dict object' has no attribute 'SUDO_USER' | 21:58 |
| mnaser | ive narrowed/tracked it down to this - https://github.com/openstack/bifrost/blob/36e32ecdc26035ecd853f830b30d42827662a68a/playbooks/install.yaml#L11 | 21:58 |
| *** mat128 has quit IRC | 22:01 | |
| *** baoli has joined #openstack-ironic | 22:01 | |
| *** rpioso has joined #openstack-ironic | 22:02 | |
| *** jkilpatr has quit IRC | 22:03 | |
| *** [1]rpioso has quit IRC | 22:03 | |
| joanna | WRT translations: we sometimes have sth like msg=_(<>), LOG.error(msg), raise Exception(msg) | 22:05 |
| joanna | this will result in having some log entries translated | 22:05 |
| joanna | since msg is often parametrized, it's also impossible to do msg=<>, LOG.error(msg), raise Exception(_(msg)) | 22:06 |
| joanna | any ideas? | 22:06 |
| *** baoli has quit IRC | 22:06 | |
| *** ijw has joined #openstack-ironic | 22:06 | |
| *** sdake_ is now known as sdake | 22:07 | |
| jlvillal | joanna: I think leave it as is. I think only change _L[ECWI]() functions. | 22:15 |
| jlvillal | That is my gut feeling. | 22:15 |
| *** mgoddard has quit IRC | 22:15 | |
| *** [1]cdearborn has quit IRC | 22:16 | |
| jlvillal | Think it will get ugly trying to have a non-translated and translated version of the same string. | 22:16 |
| joanna | jlvillal: I have mixed feelings, I think I may have recurring nightmares with multi-language logs if we do so... | 22:16 |
| *** marcus__ has joined #openstack-ironic | 22:16 | |
| openstackgerrit | Mohammed Naser proposed openstack/bifrost master: Resolve evaluation of bifrost-keystone-client-config https://review.openstack.org/447713 | 22:19 |
| *** jcoufal has quit IRC | 22:19 | |
| mnaser | hey folks, our gate is broken on kolla, can any core members give a bit of urgency to that ^ | 22:19 |
| openstackgerrit | Mohammed Naser proposed openstack/bifrost master: Resolve evaluation of bifrost-keystone-client-config https://review.openstack.org/447713 | 22:20 |
| jlvillal | TheJulia: NobodyCam:^^^ | 22:21 |
| mnaser | added a related-bug | 22:21 |
| * NobodyCam takes a look | 22:22 | |
| mnaser | i could alternatively add a | default to owner (which could be another way of resolving it) | 22:24 |
| NobodyCam | mnaser: SUDO_USER is also used here: https://github.com/openstack/bifrost/blob/master/playbooks/roles/bifrost-create-vm-nodes/tasks/main.yml#L118-L120 | 22:25 |
| mnaser | NobodyCam except when "ansible_env.SUDO_USER is defined" avoids this issue | 22:25 |
| mnaser | where as in the other one it has no choice | 22:25 |
| mnaser | i think pretty much it was breaking because of no |default on the owner | 22:25 |
| mnaser | but i figure its cleaner to take this other route | 22:26 |
| *** ijw has quit IRC | 22:26 | |
| *** linggao has quit IRC | 22:27 | |
| NobodyCam | yea I saw that, would you kill me if I thought adding the default would be a better solution? it seems more consistent with existing code | 22:27 |
| mnaser | not at all NobodyCam | 22:27 |
| mnaser | whatever makes merging easier, give me a sec | 22:27 |
| *** sayantani01 has quit IRC | 22:28 | |
| openstackgerrit | Mohammed Naser proposed openstack/bifrost master: Resolve evaluation of bifrost-keystone-client-config https://review.openstack.org/447713 | 22:28 |
| NobodyCam | awesome. :) this is a blocker for the kolla gate? | 22:28 |
| mnaser | NobodyCam voila ^ | 22:28 |
| mnaser | yeah | 22:28 |
| mnaser | all image builds are failing because of this and therefore nothing is merging (related bug has a bit of info) | 22:28 |
| NobodyCam | ack. we'll hit the button as soon as tests are done! | 22:29 |
| mnaser | let me rewrite the commit message | 22:29 |
| NobodyCam | :) | 22:29 |
| openstackgerrit | Annie Lezil proposed openstack/ironic-inspector master: Inspector rules API does not return all attributes. https://review.openstack.org/442876 | 22:29 |
| openstackgerrit | Mohammed Naser proposed openstack/bifrost master: Resolve evaluation of bifrost-keystone-client-config https://review.openstack.org/447713 | 22:30 |
| mnaser | aaand that should be it ^ | 22:30 |
| *** rook-ugh is now known as rook | 22:30 | |
| *** rook is now known as Guest8383 | 22:31 | |
| mnaser | thanks in advance NobodyCam ! | 22:31 |
| NobodyCam | :) happy to assist :) | 22:31 |
| NobodyCam | thank you for the patch :) | 22:32 |
| *** amotoki has quit IRC | 22:41 | |
| *** jkilpatr has joined #openstack-ironic | 22:42 | |
| *** krtaylor has quit IRC | 22:45 | |
| *** marcus__ is now known as Zombine | 22:45 | |
| *** Guest8383 is now known as rook-ugh | 22:47 | |
| NobodyCam | manner: tests failed? I am blocked from checking why | 22:48 |
| *** zackf has joined #openstack-ironic | 22:49 | |
| NobodyCam | mnaser: looks like some of the tests failed. I am blocked from seeing why | 22:51 |
| *** mtanino has quit IRC | 22:53 | |
| *** zackf has quit IRC | 22:55 | |
| *** baha has quit IRC | 22:55 | |
| *** linggao has joined #openstack-ironic | 22:58 | |
| openstackgerrit | Merged openstack/bifrost master: Resolve evaluation of bifrost-keystone-client-config https://review.openstack.org/447713 | 23:17 |
| *** vsaienko has quit IRC | 23:23 | |
| *** Sukhdev has quit IRC | 23:26 | |
| *** rpioso has quit IRC | 23:34 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic master: Remove translation of log messages from ironic/conductor https://review.openstack.org/447727 | 23:37 |
| *** dsneddon has quit IRC | 23:38 | |
| openstackgerrit | Joanna Taryma proposed openstack/ironic master: Add base cinder common interface https://review.openstack.org/355625 | 23:39 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!