| *** mrunge has quit IRC | 00:02 | |
| *** mrunge has joined #openstack-ironic | 00:05 | |
| *** patriciadomin has quit IRC | 00:06 | |
| *** patriciadomin has joined #openstack-ironic | 00:09 | |
| *** athomas has quit IRC | 00:18 | |
| *** patriciadomin has quit IRC | 00:51 | |
| *** patriciadomin has joined #openstack-ironic | 00:52 | |
| *** namnh has joined #openstack-ironic | 01:00 | |
| *** patriciadomin has quit IRC | 01:03 | |
| *** tiendc has joined #openstack-ironic | 01:04 | |
| *** patriciadomin has joined #openstack-ironic | 01:04 | |
| *** gcb has joined #openstack-ironic | 01:05 | |
| *** patriciadomin has quit IRC | 01:09 | |
| *** patriciadomin has joined #openstack-ironic | 01:09 | |
| *** phuongnh has joined #openstack-ironic | 01:16 | |
| *** patriciadomin has quit IRC | 01:17 | |
| *** patriciadomin has joined #openstack-ironic | 01:18 | |
| *** gcb has quit IRC | 01:18 | |
| *** yangyapeng has quit IRC | 01:25 | |
| *** yangyapeng has joined #openstack-ironic | 01:26 | |
| openstackgerrit | brandon zhao proposed openstack/ironic-inspector master: Fix to use . to source script files https://review.openstack.org/543274 | 01:31 |
|---|---|---|
| *** yangyapeng has quit IRC | 01:31 | |
| *** yangyapeng has joined #openstack-ironic | 01:32 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Update docstring to agent client related codes https://review.openstack.org/543207 | 01:33 |
| *** yangyapeng has quit IRC | 01:36 | |
| *** trungnv has joined #openstack-ironic | 01:43 | |
| *** patriciadomin has quit IRC | 01:56 | |
| *** patriciadomin has joined #openstack-ironic | 01:56 | |
| *** yangyapeng has joined #openstack-ironic | 02:02 | |
| *** d0ugal_ has quit IRC | 02:03 | |
| *** yangyapeng has quit IRC | 02:05 | |
| *** yangyapeng has joined #openstack-ironic | 02:06 | |
| *** d0ugal_ has joined #openstack-ironic | 02:13 | |
| *** gcb has joined #openstack-ironic | 02:30 | |
| *** d0ugal_ has quit IRC | 03:02 | |
| *** gcb has quit IRC | 03:07 | |
| *** patriciadomin has quit IRC | 03:11 | |
| *** patriciadomin has joined #openstack-ironic | 03:11 | |
| *** d0ugal_ has joined #openstack-ironic | 03:12 | |
| *** ijw has quit IRC | 03:15 | |
| *** trungnv has quit IRC | 03:18 | |
| *** gcb has joined #openstack-ironic | 03:20 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic-tempest-plugin master: Rescue mode tempest tests https://review.openstack.org/528699 | 03:22 |
| *** patriciadomin has quit IRC | 03:24 | |
| *** patriciadomin has joined #openstack-ironic | 03:25 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: devstack: add support for rescue mode https://review.openstack.org/524118 | 03:28 |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: Update a CI job for rescue mode https://review.openstack.org/528704 | 03:35 |
| *** trungnv has joined #openstack-ironic | 03:40 | |
| *** links has joined #openstack-ironic | 03:44 | |
| *** patriciadomin has quit IRC | 03:48 | |
| *** patriciadomin has joined #openstack-ironic | 03:50 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic-tempest-plugin master: Rescue mode standalone tests https://review.openstack.org/538119 | 03:51 |
| *** stendulker has joined #openstack-ironic | 03:54 | |
| *** patriciadomin has quit IRC | 04:05 | |
| *** patriciadomin has joined #openstack-ironic | 04:05 | |
| *** d0ugal_ has quit IRC | 04:06 | |
| *** d0ugal_ has joined #openstack-ironic | 04:14 | |
| *** ijw has joined #openstack-ironic | 04:15 | |
| *** patriciadomin has quit IRC | 04:42 | |
| *** patriciadomin has joined #openstack-ironic | 04:43 | |
| *** patriciadomin has quit IRC | 05:19 | |
| *** patriciadomin has joined #openstack-ironic | 05:20 | |
| *** moshele has joined #openstack-ironic | 05:24 | |
| *** moshele has quit IRC | 05:29 | |
| *** stendulker_ has joined #openstack-ironic | 05:32 | |
| *** stendulker has quit IRC | 05:35 | |
| *** fragatina has joined #openstack-ironic | 05:38 | |
| *** etingof has quit IRC | 05:43 | |
| *** fragatina has quit IRC | 05:45 | |
| *** patriciadomin has quit IRC | 05:46 | |
| *** patriciadomin has joined #openstack-ironic | 05:47 | |
| *** olivierbourdon38 has joined #openstack-ironic | 05:54 | |
| *** patriciadomin has quit IRC | 05:59 | |
| *** dpawar has joined #openstack-ironic | 05:59 | |
| *** patriciadomin has joined #openstack-ironic | 06:00 | |
| *** etingof has joined #openstack-ironic | 06:00 | |
| *** links has quit IRC | 06:07 | |
| *** olivierbourdon38 has quit IRC | 06:08 | |
| *** moshele has joined #openstack-ironic | 06:14 | |
| *** gcb has quit IRC | 06:41 | |
| *** rbartal has joined #openstack-ironic | 06:43 | |
| *** mjura has joined #openstack-ironic | 06:49 | |
| *** mjura has quit IRC | 06:49 | |
| *** mjura_ has joined #openstack-ironic | 06:49 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic master: Update "standalone" job for supporting rescue mode https://review.openstack.org/537821 | 06:51 |
| *** links has joined #openstack-ironic | 06:51 | |
| *** gcb has joined #openstack-ironic | 06:56 | |
| *** moshele has quit IRC | 06:57 | |
| *** moshele has joined #openstack-ironic | 06:59 | |
| *** gcb has quit IRC | 07:02 | |
| *** marios has joined #openstack-ironic | 07:07 | |
| *** dpawar has quit IRC | 07:10 | |
| *** parlos has joined #openstack-ironic | 07:11 | |
| *** ijw_ has joined #openstack-ironic | 07:17 | |
| *** ijw has quit IRC | 07:21 | |
| pas-ha | morning ironic :) | 07:27 |
| *** olivierbourdon38 has joined #openstack-ironic | 07:27 | |
| *** charcol has quit IRC | 07:31 | |
| *** ijw_ has quit IRC | 07:35 | |
| *** stendulker has joined #openstack-ironic | 07:36 | |
| *** stendulker_ has quit IRC | 07:39 | |
| *** stendulker has quit IRC | 07:40 | |
| *** jtomasek has joined #openstack-ironic | 07:43 | |
| *** slaweq has joined #openstack-ironic | 07:45 | |
| *** dpawar has joined #openstack-ironic | 07:48 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 07:51 | |
| *** dpawar has quit IRC | 07:52 | |
| *** MattMan has quit IRC | 08:01 | |
| *** MattMan has joined #openstack-ironic | 08:01 | |
| *** pcaruana has joined #openstack-ironic | 08:07 | |
| *** dsneddon has quit IRC | 08:07 | |
| *** dsneddon has joined #openstack-ironic | 08:09 | |
| *** tesseract has joined #openstack-ironic | 08:18 | |
| *** ameeda has joined #openstack-ironic | 08:33 | |
| ameeda | Hi, when I try to introspect I got this message "no configuration methods succeeded http://ipxe.org/040ee119" | 08:33 |
| *** dpawar has joined #openstack-ironic | 08:33 | |
| *** dtantsur|afk is now known as dtantsur | 08:33 | |
| dtantsur | morning ironic | 08:34 |
| ameeda | Morning dtantsur :) | 08:34 |
| hjensas | morning o/ | 08:34 |
| ameeda | regarding of my question, is that something related of PXE or switch configration "STP or fastport" | 08:35 |
| dtantsur | ameeda: essentially, DHCP failed. it may be due to STP indeed. | 08:35 |
| ameeda | btw, there are 16 nodes introspected successfully, all nodes are connected to same network switches | 08:36 |
| etingof | morning everyone having atomic number 26! ;) | 08:36 |
| *** ijw has joined #openstack-ironic | 08:36 | |
| dtantsur | Oo | 08:36 |
| dtantsur | morning etingof :D | 08:36 |
| dtantsur | ameeda: so 16 are ok and one is not? | 08:36 |
| ameeda | actually 5 are not | 08:36 |
| ameeda | I read about the error, they said that I have to update PXE, | 08:37 |
| *** stendulker has joined #openstack-ironic | 08:37 | |
| ameeda | this what I see on node console https://pasteboard.co/H7gQFO6.png | 08:37 |
| ameeda | after introspection finished with 16 nodes. it raise error with 5 nodes uuid timed out | 08:38 |
| *** rcernin has quit IRC | 08:39 | |
| dtantsur | ameeda: just a quick guess: check that your DHCP pool for introspection is bigger than 16 | 08:40 |
| ameeda | dtantsur: you mean that I have to check dhcp start/end range ? | 08:41 |
| *** vsaienk0 has joined #openstack-ironic | 08:43 | |
| *** vsaienk0 has quit IRC | 08:53 | |
| *** jaganathan has joined #openstack-ironic | 08:55 | |
| dtantsur | ameeda: in case of tripleo, this option: https://github.com/openstack/instack-undercloud/blob/master/undercloud.conf.sample#L385 | 08:55 |
| ameeda | dtantsur: what about this options https://github.com/openstack/instack-undercloud/blob/master/undercloud.conf.sample#L373-L378 | 08:57 |
| dtantsur | ameeda: these are for provisioning network on the overcloud nodes. it should not overlap with inspection iprange, but should be from the same subnet | 08:57 |
| ameeda | dtantsur: when I run introspection I watch tcpdump on undercloud, I didn't see any DHCP request from the node. it make since that the issue caused by the range ? | 08:58 |
| dtantsur | ameeda: then probably not, but rather something wrong with the network topology or boot order on the machine | 08:59 |
| *** parlos has quit IRC | 08:59 | |
| ameeda | dtantsur: Thanks of the help, I will check the configurations also DHCP range. | 09:00 |
| ameeda | just note: when I search on this error "no configuration methods succeeded http://ipxe.org/040ee119" . they said that something with pxe, what do you think ? it make since ? | 09:01 |
| dtantsur | usually it's about DHCP anyway | 09:02 |
| *** d0ugal_ has quit IRC | 09:03 | |
| *** d0ugal has joined #openstack-ironic | 09:03 | |
| *** d0ugal has quit IRC | 09:03 | |
| *** d0ugal has joined #openstack-ironic | 09:03 | |
| ameeda | dtantsur: thank you | 09:04 |
| *** racedo has joined #openstack-ironic | 09:09 | |
| *** xsjfrgwq has joined #openstack-ironic | 09:10 | |
| xsjfrgwq | ┬┬─┐┌─┐ ┌─┐┬ ┬┌─┐┌─┐┬─┐┌┐┌┌─┐┌┬┐┌─┐ ┌─┐┬─┐┌─┐ | 09:11 |
| xsjfrgwq | ┬┬─┐┌─┐ ┌─┐┬ ┬┌─┐┌─┐┬─┐┌┐┌┌─┐┌┬┐┌─┐ ┌─┐┬─┐┌─┐ | 09:11 |
| xsjfrgwq | ┬┬─┐┌─┐ ┌─┐┬ ┬┌─┐┌─┐┬─┐┌┐┌┌─┐┌┬┐┌─┐ ┌─┐┬─┐┌─┐ | 09:11 |
| xsjfrgwq | │├┬┘│ └─┐│ │├─┘├┤ ├┬┘│││├┤ │ └─┐ │ │├┬┘│ ┬ | 09:11 |
| xsjfrgwq | │├┬┘│ └─┐│ │├─┘├┤ ├┬┘│││├┤ │ └─┐ │ │├┬┘│ ┬ | 09:11 |
| xsjfrgwq | │├┬┘│ └─┐│ │├─┘├┤ ├┬┘│││├┤ │ └─┐ │ │├┬┘│ ┬ | 09:11 |
| xsjfrgwq | ┴┴└─└─┘o└─┘└─┘┴ └─┘┴└─┘└┘└─┘ ┴ └─┘o└─┘┴└─└─┘ | 09:11 |
| xsjfrgwq | ┴┴└─└─┘o└─┘└─┘┴ └─┘┴└─┘└┘└─┘ ┴ └─┘o└─┘┴└─└─┘ | 09:11 |
| xsjfrgwq | ┴┴└─└─┘o└─┘└─┘┴ └─┘┴└─┘└┘└─┘ ┴ └─┘o└─┘┴└─└─┘ | 09:11 |
| *** xsjfrgwq has quit IRC | 09:11 | |
| *** parlos has joined #openstack-ironic | 09:13 | |
| *** dirk has quit IRC | 09:33 | |
| *** dirk has joined #openstack-ironic | 09:34 | |
| *** BadCub has quit IRC | 09:41 | |
| *** BadCub has joined #openstack-ironic | 09:41 | |
| *** ijw has quit IRC | 09:46 | |
| *** serlex has joined #openstack-ironic | 09:47 | |
| *** phuongnh has quit IRC | 09:48 | |
| *** serlex1 has joined #openstack-ironic | 09:54 | |
| *** serlex1 has left #openstack-ironic | 09:54 | |
| *** shekar has quit IRC | 09:54 | |
| *** shekar has joined #openstack-ironic | 09:55 | |
| *** e0ne has joined #openstack-ironic | 09:55 | |
| openstackgerrit | Ankit Kumar proposed openstack/ironic master: Adding condition for setting variable https://review.openstack.org/535651 | 09:55 |
| *** serlex has quit IRC | 09:57 | |
| *** zshi has quit IRC | 10:01 | |
| *** namnh has quit IRC | 10:08 | |
| *** zshi has joined #openstack-ironic | 10:14 | |
| *** tiendc has quit IRC | 10:17 | |
| *** derekh has joined #openstack-ironic | 10:17 | |
| *** olivierbourdon38 has quit IRC | 10:31 | |
| *** lucas-afk is now known as lucasagomes | 10:44 | |
| *** ijw has joined #openstack-ironic | 10:46 | |
| *** vsaienk0 has joined #openstack-ironic | 10:47 | |
| *** vsaienk0 has quit IRC | 10:57 | |
| *** parlos has quit IRC | 10:58 | |
| *** stendulker has quit IRC | 11:05 | |
| dtantsur | pas-ha: hi, is https://review.openstack.org/542206 ready for approval? | 11:05 |
| patchbot | patch 542206 - ironic-inspector - ironic_inspector: ironic: Fix 'auth_type' when 'au... | 11:05 |
| *** sambetts|afk is now known as sambetts | 11:06 | |
| sambetts | Morning all | 11:06 |
| pas-ha | still trying to test it locally, no strict conformation yet but at least inspector starts w/o mentioned errors... | 11:06 |
| dtantsur | morning sambetts | 11:07 |
| openstackgerrit | Sam Betts proposed openstack/ironic-tempest-plugin master: Add [baremetal] available_nodes configuration option https://review.openstack.org/541383 | 11:11 |
| *** mrunge_ has joined #openstack-ironic | 11:23 | |
| *** mrunge has quit IRC | 11:25 | |
| *** parlos has joined #openstack-ironic | 11:33 | |
| dtantsur | "Your visa application was approved." \o/ | 11:55 |
| sambetts | dtantsur: \o/ | 12:01 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Update iLO documentation for deprecating classical drivers https://review.openstack.org/542593 | 12:02 |
| pas-ha | dtantsur: heh, we are still waiting, hopefully tomorrow we'll get the answer | 12:11 |
| dtantsur | good luck :) | 12:11 |
| *** yangyapeng has quit IRC | 12:14 | |
| *** vsaienk0 has joined #openstack-ironic | 12:26 | |
| *** bfournie has quit IRC | 12:27 | |
| *** bfournie has joined #openstack-ironic | 12:27 | |
| openstackgerrit | Olivier Bourdon proposed openstack/bifrost master: Fix for proper usage when keystone enabled https://review.openstack.org/543441 | 12:29 |
| *** yangyapeng has joined #openstack-ironic | 12:30 | |
| *** bfournie has quit IRC | 12:31 | |
| *** olivierbourdon38 has joined #openstack-ironic | 12:34 | |
| *** yangyapeng has quit IRC | 12:35 | |
| *** dpawar has quit IRC | 12:35 | |
| *** vsaienk0 has quit IRC | 12:36 | |
| openstackgerrit | Olivier Bourdon proposed openstack/bifrost master: Fix for proper usage when keystone enabled https://review.openstack.org/543441 | 12:36 |
| lucasagomes | TheJulia, dtantsur hi there, have we got the dates sorted for the ironic party in dublin already ? If so, let's book the venue | 12:38 |
| lucasagomes | and morning all :-) | 12:39 |
| dtantsur | lucasagomes: judging by https://doodle.com/poll/d4ff6m9hxg887n9q it's going to be Tuesday, but let's wait for TheJulia | 12:39 |
| *** jlabarre has joined #openstack-ironic | 12:40 | |
| *** hw_wutianwei has joined #openstack-ironic | 12:41 | |
| lucasagomes | ack | 12:42 |
| *** milan_ has joined #openstack-ironic | 12:44 | |
| jroll | morning everyone | 12:46 |
| *** yangyapeng has joined #openstack-ironic | 12:47 | |
| dtantsur | hey, happy Monday jroll | 12:52 |
| *** r-daneel has joined #openstack-ironic | 12:56 | |
| *** bfournie has joined #openstack-ironic | 13:01 | |
| *** parlos has quit IRC | 13:02 | |
| *** vsaienk0 has joined #openstack-ironic | 13:05 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Clean up CI playbooks https://review.openstack.org/535896 | 13:07 |
| *** weshay|ruck is now known as weshay | 13:10 | |
| olivierbourdon38 | Hello everyone | 13:15 |
| *** jcoufal has joined #openstack-ironic | 13:15 | |
| *** vsaienk0 has quit IRC | 13:15 | |
| olivierbourdon38 | I have a question concerning bifrost usage in an env where the bifrost master server is launched within a KVM machine as well as slaves machines | 13:15 |
| olivierbourdon38 | whenever the slaves boots on the network it perfectly grabs the IPA image and register itself with ironic (note that I also have keystone enabled) | 13:16 |
| olivierbourdon38 | when I do openstack baremetal node list I can see the new node | 13:16 |
| olivierbourdon38 | however as I have virtualbmc running on the KVM host machine and working correctly (ipmitool ... chassis power on/off/status just works great with proper slave vm port) | 13:18 |
| olivierbourdon38 | I do not understand what part of the deployment/playbooks/utilities/... would be responsible to do the complement of enrollment infos as the vmbc host/IP + port + credentials should be filled in somehow ? | 13:20 |
| olivierbourdon38 | thanks for any pointer/hint/help | 13:23 |
| *** dprince has joined #openstack-ironic | 13:24 | |
| ameeda | dtantsur: I check undercloud.conf introspection_range -> inspection_iprange = 172.31.255.203,172.31.255.254 | 13:26 |
| openstackgerrit | Olivier Bourdon proposed openstack/bifrost master: Fix for proper usage when keystone enabled https://review.openstack.org/543441 | 13:32 |
| *** vsaienk0 has joined #openstack-ironic | 13:45 | |
| *** serlex has joined #openstack-ironic | 13:48 | |
| *** dtantsur is now known as dtantsur|brb | 13:52 | |
| *** rloo has joined #openstack-ironic | 13:53 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Validate instance_info.traits against node traits https://review.openstack.org/543461 | 13:55 |
| *** vsaienk0 has quit IRC | 13:55 | |
| *** zshi has quit IRC | 13:59 | |
| *** slaweq has left #openstack-ironic | 14:01 | |
| *** baha has joined #openstack-ironic | 14:02 | |
| *** mjturek has joined #openstack-ironic | 14:03 | |
| *** trown|outtypewww is now known as trown | 14:07 | |
| openstackgerrit | Merged openstack/bifrost stable/ocata: Zuul: Remove project name https://review.openstack.org/542787 | 14:09 |
| TheJulia | lucasagomes: dtantsur|brb: Yeah, Tuesday seems to be the best date. | 14:12 |
| *** zshi has joined #openstack-ironic | 14:12 | |
| *** rajinir has quit IRC | 14:12 | |
| *** rajinir has joined #openstack-ironic | 14:12 | |
| TheJulia | olivierbourdon38: so... IPA has no way of knowing about that virtualbmc is controlling your power, so no details about power control are sent to ironic. You would have to have to add them yourself after the fact, although the nice thing is you could use the os_ironic ansible module to set the parameters and go from there | 14:15 |
| *** mjturek_ has joined #openstack-ironic | 14:16 | |
| *** patriciadomin_ has joined #openstack-ironic | 14:16 | |
| olivierbourdon38 | thanks <TheJulia> got it | 14:16 |
| TheJulia | Just TheJulia :) Your IRC client is adding <> | 14:17 |
| * TheJulia sips coffee in an attempt to wake up | 14:18 | |
| sambetts | TheJulia, olivierbourdon38: if your using inspector, we provide the ironic inspector rules feature to fill in this gap, basically it allows you to write a rule that matches on some information that was inspected and then set additional parameters on the inspected node in ironic | 14:19 |
| *** patriciadomin has quit IRC | 14:19 | |
| *** mjturek has quit IRC | 14:19 | |
| sambetts | https://docs.openstack.org/ironic-inspector/latest/user/usage.html#introspection-rules | 14:20 |
| jroll | trying to add a config option from ironic, to a group that's defined by an oslo library, has anyone done anything like that before? not obvious to me how I would do that. context: https://review.openstack.org/#/c/542992/ | 14:20 |
| patchbot | patch 542992 - ironic - Add optional healthcheck middleware | 14:20 |
| * jroll wonders if he could do some foo with importing the group from oslo or something | 14:21 | |
| TheJulia | sambetts: that is a good point | 14:22 |
| sambetts | jroll: the register opts function for cfg.CONF just takes a string, so you can just do CONF.register_opt(<your new opt>, group="the group from oslo") | 14:22 |
| jroll | sambetts: like this? https://review.openstack.org/#/c/542992/2/ironic/conf/healthcheck.py :) | 14:22 |
| patchbot | patch 542992 - ironic - Add optional healthcheck middleware | 14:22 |
| olivierbourdon38 | TheJulia: I was adding them manually, hopefully IRC client allows me to be even more lazy, not like slack which requires you to type # or @ ;-) | 14:22 |
| TheJulia | olivierbourdon38: most clients support tab autocomplete.... ;) | 14:23 |
| jroll | olivierbourdon38: if you want to be lazy, try typing "the" and press tab :) | 14:23 |
| jroll | :P | 14:23 |
| olivierbourdon38 | sambetts TheJulia: thanks will have a look at this | 14:23 |
| sambetts | jroll: yup, so that should work in code, but the sample config generator is a bit dumb and doesn't use the groups defined by the code, you have to define it here to make it discoverable https://github.com/openstack/ironic/blob/master/ironic/conf/opts.py#L64 | 14:25 |
| sambetts | jroll: so you need to add a ('healthcheck', ironic.conf.healthcheck.opts) line | 14:26 |
| sambetts | to that file | 14:26 |
| jroll | oh jeez | 14:26 |
| jroll | thanks | 14:26 |
| * jroll thought adding to ironic/conf/__init__ would be enough :( | 14:26 | |
| sambetts | yeah... its really a bit dumb | 14:27 |
| *** vsaienk0 has joined #openstack-ironic | 14:27 | |
| jroll | indeed | 14:29 |
| jroll | thanks again sambetts | 14:29 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Add optional healthcheck middleware https://review.openstack.org/542992 | 14:29 |
| rloo | Good Monday Morning everyone including jroll, sambetts, olivierbourdon38, TheJulia, dtantsur|brb, lucasagomes | 14:35 |
| lucasagomes | rloo, morning! | 14:36 |
| rloo | lucasagomes, dtantsur|brb, TheJulia: when I sent out email about a PTG dinner, I had mentioned Feb 16 as deadline. If we want to decide sooner, eg, today, i can send out email to say today (tomorrow morning, don't make me do utc times) | 14:36 |
| olivierbourdon38 | good morning rloo | 14:37 |
| jroll | morning rloo :) | 14:37 |
| lucasagomes | rloo, oh if it says Feb 16 it's fine, we can wait. I was just wondering about the date/numer of people cause I thought the deadline was last friday | 14:37 |
| lucasagomes | and I would go ahead and book it | 14:37 |
| rloo | lucasagomes: do you think feb 16 is too late? i picked that date cuz it is the same date as proposing ptg topics :) | 14:38 |
| rloo | lucasagomes: you back in dublin now? | 14:38 |
| lucasagomes | rloo, I think Feb 16 is fine | 14:38 |
| openstackgerrit | Merged openstack/python-ironicclient stable/ocata: Zuul: Remove project name https://review.openstack.org/542981 | 14:39 |
| lucasagomes | rloo, not yet, I will be back on the 21st... But I will book from brazil | 14:39 |
| lucasagomes | or ask derekh if he can do it too :D | 14:39 |
| rloo | lucasagomes: ahh, ok, will ping you and/or derekh. thx! | 14:39 |
| lucasagomes | rloo, thanks a lot! | 14:39 |
| rloo | lucasagomes: btw, it is never too late to come back to ironic ;) | 14:40 |
| lucasagomes | rloo, hah I still think it's the best project in openstack o/! | 14:41 |
| lucasagomes | I see jroll is back too | 14:41 |
| lucasagomes | rloo, maybe in the future, who knows :D | 14:41 |
| rloo | lucasagomes: :D | 14:41 |
| derekh | rloo: I'm happy to help out, I don't live in dublin so lucasagomes might have a better idea of a suitable place but I do at least live in the same country | 14:41 |
| lucasagomes | that said, maybe soon we will start to look at having OVN servicing DHCP for a provider network (for the baremetal case) | 14:42 |
| *** mjturek_ has quit IRC | 14:42 | |
| lucasagomes | so I hope to help with that and be closer to ironic again :D | 14:42 |
| jroll | lucasagomes: :D | 14:42 |
| lucasagomes | derekh, ++ we were thinking about Against the Grain | 14:42 |
| rloo | derekh: oh, i thought you lived in dublin too! Thx for helping out, I'm sure between you and lucasagomes, we'll have a great meeting place. Looking forward to seeing you there! | 14:42 |
| lucasagomes | derekh, we had a xmas party for red hat before there ~2 years ago ? | 14:42 |
| olivierbourdon38 | folks, any idea when the CI failures will be fixed ? (Unable to establish connection to http://127.0.0.1:5050) | 14:44 |
| *** vsaienk0 has quit IRC | 14:44 | |
| *** Goneri has joined #openstack-ironic | 14:44 | |
| derekh | lucasagomes: yup, against the grain is a good spot assuming we can get there | 14:44 |
| *** r-daneel has quit IRC | 14:45 | |
| derekh | lucasagomes: looks like it would be bus or taxi but should be fine | 14:45 |
| lucasagomes | derekh, yeah | 14:45 |
| rloo | derekh, lucasagomes: or walk? | 14:45 |
| lucasagomes | we can even walk I guess, if people feel like doing it (and it's not too cold) | 14:45 |
| rloo | olivierbourdon38: what CI failure? | 14:46 |
| lucasagomes | derekh, the only decent pub I know close to the croke park is fagans | 14:47 |
| *** mjturek has joined #openstack-ironic | 14:47 | |
| lucasagomes | derekh, http://faganspub.ie/ | 14:47 |
| * lucasagomes will get some food and brb quickly, before another meeting | 14:48 | |
| *** lucasagomes is now known as lucas-hungry | 14:48 | |
| mjturek | gm rloo :) | 14:48 |
| derekh | lucas-hungry: looks good too, I don't think I've been there | 14:48 |
| rloo | morning mjturek! am hoping you will be at ptg and our note taker :) | 14:48 |
| mjturek | rloo: found out on Thursday that I will be attending! | 14:49 |
| mjturek | so happy to :) | 14:49 |
| TheJulia | rloo: I guess it would be good to send a follow-up email today, but the 16th is fine. I just think most people have already started to solidifying their schedules since so many different groups are trying to align plans. | 14:49 |
| rloo | mjturek: awesome! | 14:50 |
| olivierbourdon38 | rloo: on the bifrost submitted patches all 3 OPNFV test fails whatever the current review currently as well as some Zuul ones | 14:50 |
| olivierbourdon38 | even patchsets which deal with docs/.gitreview/... | 14:50 |
| rloo | TheJulia: let's mention in meeting today and i'll send follow up. It isn't a problem to change the date to tomorrow. We may want to beat the crowd and pick/book sooner rather than later? | 14:51 |
| olivierbourdon38 | https://review.openstack.org/542795 | 14:51 |
| patchbot | patch 542795 - bifrost - Collect ironic-inspector logs | 14:51 |
| rloo | olivierbourdon38: oh :-( | 14:51 |
| TheJulia | rloo: yeah | 14:51 |
| olivierbourdon38 | https://review.openstack.org/542172 | 14:51 |
| patchbot | patch 542172 - bifrost (stable/queens) - Update .gitreview for stable/queens | 14:51 |
| TheJulia | olivierbourdon38: Yeah, we broke inspector, I think there is a patch up for it, I've not looked yet this morning, trying to take care of an expense thing before I shift gears | 14:51 |
| rloo | TheJulia, olivierbourdon38: yolanda indicates that it is due to https://review.openstack.org/#/c/542206/ | 14:52 |
| patchbot | patch 542206 - ironic-inspector - ironic_inspector: ironic: Fix 'auth_type' when 'au... | 14:52 |
| * rloo looks... | 14:52 | |
| yolanda | hi yep, Markos started reporting the failures on bifrost, and he continued with that finding | 14:52 |
| rloo | yolanda: thx. i see that pas-ha isn't convinced that this will fix it though. | 14:53 |
| TheJulia | It _might_, then again we have a ton of custom code in python-ironicclient | 15:03 |
| TheJulia | If we can get another review on https://review.openstack.org/542172, I'll feel less worried | 15:04 |
| patchbot | patch 542172 - bifrost (stable/queens) - Update .gitreview for stable/queens | 15:04 |
| sambetts | TheJulia: +A | 15:05 |
| * rloo glad that TheJulia is less worried now :) | 15:06 | |
| TheJulia | If it works on master, we will need to backport it to queens as well | 15:07 |
| rloo | TheJulia: did you want to +A it? | 15:07 |
| TheJulia | oh shoot | 15:08 |
| TheJulia | I pasted the wrong link | 15:08 |
| * TheJulia needs more coffee | 15:08 | |
| TheJulia | I meant https://review.openstack.org/#/c/542206/ | 15:08 |
| patchbot | patch 542206 - ironic-inspector - ironic_inspector: ironic: Fix 'auth_type' when 'au... | 15:08 |
| rloo | TheJulia: ah, that explains it :) | 15:08 |
| rloo | TheJulia: it seems fine to me, the change, but I didn't test and don't know inspector that well. i'd feel better if pas-ha weighed in with his +1. | 15:09 |
| TheJulia | rloo: will there be any RFEs to discuss today? | 15:09 |
| TheJulia | actually, there should be a way to test | 15:09 |
| rloo | TheJulia: i am so tired. i hadn't planned on that. but i could. is there stuff about ptg that we want to talk about though? wondering if we should start with the topics. | 15:10 |
| TheJulia | rloo: I was thinking of breaking the topics up into individual etherpads and getting all of that sorted this afternoon | 15:10 |
| rloo | TheJulia: wrt 542206, pas-ha mentioned in patch that he has a bifrost patch dependent on this patch. | 15:10 |
| rloo | TheJulia: ok, so not ready to discuss the topics. | 15:11 |
| rloo | TheJulia: i can look/see if any rfe's waiting for a quick decision on spec or not. | 15:11 |
| sambetts | TheJulia: I'm +2 on that auth patch too | 15:12 |
| rloo | TheJulia: i have a good bike-shedding one. well, all bikeshedding is good. not an rfe though :) | 15:12 |
| TheJulia | re: ptg, it would be good for people to put feedback on if they feel they need to discuss the topic at the ptg, so we can prioritize/sort/schedule | 15:12 |
| TheJulia | rloo: Does the bike shed end up purple? | 15:13 |
| rloo | TheJulia: good idea.do you want the feedback on the ptg etherpad or in the separate ones you're going to create? | 15:13 |
| rloo | TheJulia: can't predict how the bike shed ends up; that's for the group to decide :D | 15:13 |
| *** crushil has quit IRC | 15:14 | |
| TheJulia | rloo: ptg etherpad so it is all in one place and we can sort through it/see thoughts | 15:14 |
| TheJulia | Also, it means less context switching for me when I finally make the other etherpads | 15:15 |
| rloo | TheJulia: ok, let's mention in today's meeting. | 15:15 |
| rloo | TheJulia: i think deadline for proposing topics is this friday. so people should vote this week but close the votes next Monday at the meeting maybe? | 15:16 |
| *** moshele has quit IRC | 15:17 | |
| TheJulia | re the patch for inspector, yeah... it doesn't look like it will do it | 15:17 |
| rloo | TheJulia: next Mon (19th) is a holiday in most? of canada. in the US, is it? Is it presidents Day? | 15:17 |
| TheJulia | rloo: yes, As long as people stat putting feedback it is good, I can always hold off on schedule creation/etherpad creation until Friday afternoon. It is not an issue for me. I was also just thinking from last week there was the ask to begin splitting the pads apart for things we will definitely be discussing | 15:18 |
| rloo | TheJulia: i think it makes sense to start creating pads for topics; there are some that will be clear folks are interested in | 15:19 |
| rloo | TheJulia: pas-ha's patch failed: https://review.openstack.org/#/c/542795/. i'll add link to the other patch. | 15:19 |
| patchbot | patch 542795 - bifrost - Collect ironic-inspector logs | 15:19 |
| *** lucas-hungry is now known as lucasagomes | 15:19 | |
| lucasagomes | derekh, yeah it's not bad | 15:19 |
| lucasagomes | derekh, we can go for it if people think that against the grain (or other pub in/near city centre) is too far | 15:20 |
| TheJulia | rloo: so it failed because of the way the job works | 15:20 |
| * TheJulia goes to see if she can fix the job to be better behaved | 15:21 | |
| rloo | TheJulia: thx for looking into it, i'm updating our whiteboard wrt the bifrost failure | 15:22 |
| TheJulia | Did anyone figure out anything regarding grenade on thursday/Friday? | 15:22 |
| *** sergek_ has quit IRC | 15:22 | |
| *** sergek_ has joined #openstack-ironic | 15:23 | |
| *** dprince has quit IRC | 15:23 | |
| *** hjensas has quit IRC | 15:24 | |
| rloo | TheJulia: i don't think anything happened on thurs wrt grenade. don't know about fri. | 15:26 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Change ironic-inspector to be source installed https://review.openstack.org/543491 | 15:26 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Change ironic-inspector to be source installed https://review.openstack.org/543491 | 15:28 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: Collect ironic-inspector logs https://review.openstack.org/542795 | 15:28 |
| TheJulia | That _should_ do it | 15:28 |
| *** milan_ has quit IRC | 15:28 | |
| derekh | lucasagomes: yup | 15:31 |
| TheJulia | based on the job results this morning, I'm guessing no progress | 15:32 |
| *** crushil has joined #openstack-ironic | 15:32 | |
| TheJulia | (re grenade) | 15:32 |
| * TheJulia proposes updated meeting agenda for today | 15:39 | |
| *** links has quit IRC | 15:39 | |
| *** ijw_ has joined #openstack-ironic | 15:42 | |
| TheJulia | rloo: bifrost jobs are passing with the job fix I put in | 15:43 |
| TheJulia | and pas-ha's change | 15:43 |
| *** ijw has quit IRC | 15:46 | |
| *** dprince has joined #openstack-ironic | 15:48 | |
| rloo | TheJulia: great. looking... | 15:52 |
| *** olivierbourdon38 has quit IRC | 15:53 | |
| *** vsaienk0 has joined #openstack-ironic | 15:53 | |
| *** mjura_ has quit IRC | 16:00 | |
| *** cdearborn has joined #openstack-ironic | 16:00 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy-tools master: Support EthernetInterface resource in the emulator https://review.openstack.org/538338 | 16:01 |
| *** Nisha_Agarwal has joined #openstack-ironic | 16:01 | |
| *** pcaruana has quit IRC | 16:04 | |
| *** jlabarre has quit IRC | 16:04 | |
| *** vsaienk0 has quit IRC | 16:06 | |
| *** jlabarre has joined #openstack-ironic | 16:09 | |
| *** krtaylor has quit IRC | 16:10 | |
| rloo | sambetts: just to verify, this is finished, right? https://bugs.launchpad.net/ironic/+bug/1602265 | 16:10 |
| openstack | Launchpad bug 1602265 in ironic-python-agent "[RFE] IPA API Versioning" [Wishlist,In progress] - Assigned to Sam Betts (sambetts) | 16:10 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: tox: fix pep8 to not fail if no pot files are found https://review.openstack.org/543523 | 16:11 |
| *** moshele has joined #openstack-ironic | 16:12 | |
| cdearborn | good morning Ironic! | 16:12 |
| sambetts | rloo: I believe so | 16:12 |
| cdearborn | I have a know issue related to the iDRAC driver that I would like to put in the docs somewhere. Was wondering if anyone has thoughts on where something like that should be documented? | 16:13 |
| sambetts | cdearborn: https://github.com/openstack/ironic/blob/master/doc/source/admin/drivers/idrac.rst | 16:13 |
| sambetts | I expect that would be the best place ^ | 16:14 |
| rloo | sambetts: thx, i'll quickly review the spec then, just-to-be-sure :) | 16:14 |
| sambetts | rloo: thanks | 16:14 |
| cdearborn | sambetts: thx! that was my thought as well | 16:15 |
| openstackgerrit | Merged openstack/ironic-python-agent master: Update reno for stable/queens https://review.openstack.org/542157 | 16:18 |
| *** hjensas has joined #openstack-ironic | 16:21 | |
| *** hjensas has quit IRC | 16:21 | |
| *** hjensas has joined #openstack-ironic | 16:21 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: tox: fix pep8 to not fail if no pot files are found https://review.openstack.org/543523 | 16:22 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Imported Translations from Zanata https://review.openstack.org/542609 | 16:22 |
| *** dtantsur|brb is now known as dtantsur | 16:24 | |
| dtantsur | morning TheJulia, rloo, cdearborn | 16:26 |
| * dtantsur forgot to return to IRC | 16:26 | |
| TheJulia | o/ dtantsur | 16:26 |
| TheJulia | dtantsur: you've been busy this morning :) | 16:26 |
| rloo | dtantsur: :) | 16:27 |
| cdearborn | good morning dtantsur! | 16:27 |
| dtantsur | relatively busy :) | 16:27 |
| dtantsur | TheJulia, sambetts, I've found a nasty problem, badly affecting ironic-inspector: https://bugs.launchpad.net/ironic-inspector/+bug/1748893 | 16:28 |
| openstack | Launchpad bug 1748893 in Ironic Inspector "Port list returns HTTP 400 if a node is removed while listing ports" [Critical,Triaged] | 16:28 |
| dtantsur | specifically, it was seen in our discovery job | 16:28 |
| TheJulia | sweet | 16:30 |
| sambetts | dtantsur: oh interesting... so its a race between us listing the ports and someone else deleting the node from ironic? | 16:31 |
| *** r-daneel has joined #openstack-ironic | 16:32 | |
| dtantsur | yep | 16:32 |
| dtantsur | ditto for port groups btw | 16:32 |
| sambetts | dtantsur: its a bug in ironic though by the looks of it | 16:34 |
| sambetts | not in ironic inspector | 16:34 |
| dtantsur | yep | 16:34 |
| dtantsur | node listing API should not return 400 in this case | 16:34 |
| dtantsur | s/node/port/ but you get the idea | 16:34 |
| sambetts | \o/ internal races... | 16:35 |
| *** ijw_ has quit IRC | 16:35 | |
| *** baha has quit IRC | 16:36 | |
| sambetts | dtantsur: so I think the error is here: https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/port.py#L87 | 16:40 |
| dtantsur | sambetts: well, this code is for creating a node, and for creating a node it is correct | 16:41 |
| sambetts | dtantsur: this is in the port controller | 16:41 |
| dtantsur | s/node/port/ | 16:41 |
| dtantsur | the problem is that it also runs on listing/getting ports | 16:41 |
| sambetts | and for creating the port its correct yes | 16:41 |
| * dtantsur has stared at this code is despair already | 16:41 | |
| sambetts | the problem is knowing if its a get or a post | 16:41 |
| sambetts | i.e. it should be a 400 if its a post request (create) but ignored if its a get | 16:42 |
| dtantsur | I was thinking of catching BadRequest while iterating on ports in PortCollection.convert_with_links | 16:42 |
| dtantsur | and first try to remove portgroup_uuid, if it still fails just skip the port | 16:42 |
| * dtantsur does not like his solution | 16:42 | |
| jroll | how is a node being removed without removing the ports for that node? O_o | 16:43 |
| *** moshele has quit IRC | 16:43 | |
| dtantsur | jroll: the ports are removed | 16:43 |
| sambetts | I think its a race | 16:43 |
| * jroll thought all ports were meant to have a node | 16:43 | |
| jroll | ah | 16:43 |
| dtantsur | the timeline is 1. we fetch ports, 2. the node is removed, 3. we try to fetch associated nodes for ports | 16:43 |
| jroll | yep, makes sense | 16:44 |
| jroll | we should be joining the node uuid when we do the db query, imo | 16:44 |
| jroll | which would both solve the race and also save us some queries | 16:44 |
| dtantsur | will require redoing some parts of RPC | 16:44 |
| dtantsur | IIRC we don't have a node record inside of ports | 16:44 |
| jroll | sigh | 16:44 |
| jroll | dtantsur: on the port object, to be clear, right | 16:45 |
| * TheJulia wonders if it is time to just rip all db access out from the api and go through RPC for it all... | 16:45 | |
| TheJulia | (long term, not short term) | 16:45 |
| * jroll wonders if it's time to rewrite the API yet | 16:45 | |
| sambetts | dtantsur: the actual exceptions get reraised, so we could catch the NodeNotFound and PortGroupNotFound exceptions in the convert with links | 16:46 |
| jroll | TheJulia: from what I'm told, our rolling upgrades depend on not doing that, I guess? | 16:46 |
| * TheJulia ponders | 16:46 | |
| openstackgerrit | Paul Belanger proposed openstack/ironic master: Stop running non-voting jobs in gate https://review.openstack.org/543543 | 16:46 |
| jroll | dtantsur: you're right about the port object, sigh | 16:46 |
| jroll | TheJulia: it made my head spin, fair warning | 16:47 |
| *** stendulker has joined #openstack-ironic | 16:47 | |
| *** links has joined #openstack-ironic | 16:47 | |
| sambetts | I thought each port has a node id field, which is a foreign key to the node's table, so I can't we join the uuid in from that? | 16:48 |
| TheJulia | jroll: gah, catch-22 | 16:48 |
| jroll | sambetts: more about the rpc object only having node_id, we'd have to add a uuid or a full node object field there | 16:48 |
| sambetts | ah right yeah :/ | 16:48 |
| *** hjensas has quit IRC | 16:49 | |
| *** olivierbourdon38 has joined #openstack-ironic | 16:49 | |
| openstackgerrit | Dongcan Ye proposed openstack/ironic master: Add meaningful exception in Neutron port show https://review.openstack.org/533640 | 16:49 |
| sambetts | i think catching this in convert with links in the only sane solution then | 16:49 |
| dtantsur | yep | 16:50 |
| jroll | idk, I think port.node should be a node object, and we should join it when needed. but that isn't backportable so probably the wrong short term solution | 16:50 |
| *** moshele has joined #openstack-ironic | 16:51 | |
| TheJulia | Everyone updating their whiteboard status(es) | 16:51 |
| sambetts | personal opinion when it comes to things like this is that because ports are a sub-resource of a node, ports should always be accessed from the node object | 16:51 |
| sambetts | i.e. the ports should be joined into the node rpc | 16:52 |
| sambetts | then you get the list of nodes, and then build the list to return by joining all the node.ports lists | 16:52 |
| *** links has quit IRC | 16:53 | |
| *** links has joined #openstack-ironic | 16:54 | |
| *** trungnv has quit IRC | 16:54 | |
| *** moshele has quit IRC | 16:55 | |
| *** hshiina has joined #openstack-ironic | 16:55 | |
| *** baha has joined #openstack-ironic | 16:55 | |
| TheJulia | #startmeeting ironic | 17:00 |
| openstack | Meeting started Mon Feb 12 17:00:08 2018 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. | 17:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 17:00 |
| *** openstack changes topic to " (Meeting topic: ironic)" | 17:00 | |
| TheJulia | o/ | 17:00 |
| dtantsur | o/ | 17:00 |
| openstack | The meeting name has been set to 'ironic' | 17:00 |
| etingof | o/ | 17:00 |
| mjturek | o/ | 17:00 |
| jroll | \o | 17:00 |
| stendulker | o/ | 17:00 |
| TheJulia | Our meeting agenda can be found on the wiki, as always! | 17:00 |
| *** mgoddard_ has joined #openstack-ironic | 17:00 | |
| hshiina | o/ | 17:00 |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic | 17:00 |
| rloo | o/ | 17:01 |
| mgoddard_ | o/ | 17:01 |
| TheJulia | #topic Announcements / Reminder | 17:01 |
| *** openstack changes topic to "Announcements / Reminder (Meeting topic: ironic)" | 17:01 | |
| TheJulia | First off, Thank you dtantsur for your hard work! | 17:01 |
| jroll | ++ | 17:01 |
| dtantsur | :) | 17:01 |
| rloo | +++ | 17:01 |
| dtantsur | and congrats TheJulia for taking this hard work ;) | 17:01 |
| etingof | dtantsur++ | 17:02 |
| jlvillal | o/ | 17:02 |
| jlvillal | +1 :) | 17:02 |
| rloo | +++ congrats! | 17:02 |
| TheJulia | I hope to meet everyone's expectations. Please remember that I too am human, as I take on the role of fearless leader. | 17:02 |
| stendulker | Congrats Julia !! | 17:02 |
| rloo | and THANK YOU TheJulia for volunteering! | 17:02 |
| etingof | ++TheJulia | 17:02 |
| * jlvillal had expectations of an in-human leader :P | 17:02 | |
| TheJulia | Anyway, time for the remaining annoucements | 17:02 |
| TheJulia | jlvillal: only if I get a flying aircraft carrier ;) | 17:02 |
| jlvillal | :) | 17:03 |
| * dtantsur announces that his visa to ireland was finally approved | 17:03 | |
| TheJulia | The PTG is coming up. Please update the PTG Planning etherpad today and over this week. | 17:03 |
| TheJulia | #link https://etherpad.openstack.org/p/ironic-rocky-ptg | 17:03 |
| * jlvillal thought we had a new core reviewer | 17:04 | |
| TheJulia | I will break the etherpad up and generate a schedule for Wednesday/Thursday on this coming friday. | 17:04 |
| TheJulia | jlvillal: thanks for the reminder! | 17:04 |
| TheJulia | #info hshiina is now a member of ironic-core, congrats! | 17:04 |
| jroll | \o/ | 17:04 |
| jlvillal | Nice! :) | 17:04 |
| hshiina | thanks, everyone | 17:05 |
| stendulker | Congrats hshiina !! | 17:05 |
| TheJulia | I think that is about it for annoucements, does anyone else have anything to annouce? | 17:05 |
| TheJulia | announce | 17:05 |
| rloo | congrats hshiina, welcome to moar reviews! :) | 17:05 |
| * jlvillal checks time in Japan and sees it is 2:05AM. Yowzer! | 17:05 | |
| rloo | ptg get together poll: https://doodle.com/poll/d4ff6m9hxg887n9q | 17:05 |
| dtantsur | when is queens final? | 17:05 |
| TheJulia | #link https://doodle.com/poll/d4ff6m9hxg887n9q | 17:05 |
| sambetts | o/ | 17:06 |
| TheJulia | Please respond to the doodle so we can schedule an evening gathering at the PTG. | 17:06 |
| rloo | if you're interested in joining the ptg get-together, please indicate your availability via the poll. i had indicated a feb 16 deadline, but i think i'd like to book a place sooner rather than later, so please sign up by tomorrow. i'll send out email too. thx. | 17:06 |
| dtantsur | ++ let's not wait till the last moment with booking | 17:06 |
| TheJulia | #info Queens final releases are slated for the week of the 19th-23rd. | 17:06 |
| TheJulia | rloo: thanks! | 17:07 |
| dtantsur | oh, so one more week to fix all the bugs | 17:07 |
| * rloo wonder, what bugs? impossible... | 17:07 | |
| TheJulia | and tests... and gates | 17:07 |
| dtantsur | rloo: 3 critical bugs.. | 17:07 |
| TheJulia | Anyway, we should move on | 17:07 |
| dtantsur | move on \o/ | 17:07 |
| TheJulia | dtantsur: is there a list on the whiteboard? | 17:07 |
| * dtantsur flies to the woods | 17:08 | |
| dtantsur | TheJulia: there is | 17:08 |
| TheJulia | #topic Review action items from previous meeting | 17:08 |
| *** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)" | 17:08 | |
| *** SerenaFeng has joined #openstack-ironic | 17:08 | |
| *** fragatina has joined #openstack-ironic | 17:08 | |
| TheJulia | Looks like our only action item was to review/triage and work on bugs last week. | 17:09 |
| *** hjensas has joined #openstack-ironic | 17:09 | |
| *** hjensas has quit IRC | 17:09 | |
| *** hjensas has joined #openstack-ironic | 17:09 | |
| *** fragatin_ has joined #openstack-ironic | 17:09 | |
| TheJulia | I think we can just move on since this week should be the same. Any disagreements? | 17:09 |
| dtantsur | +++ | 17:10 |
| TheJulia | Moving on then! | 17:10 |
| TheJulia | #topic Subteam status reports | 17:11 |
| *** openstack changes topic to "Subteam status reports (Meeting topic: ironic)" | 17:11 | |
| TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 17:11 |
| TheJulia | Starting at Line 202 | 17:11 |
| *** marios has quit IRC | 17:11 | |
| rloo | dtantsur: are you going to work on classic driver deprecation this week? (doc needs updating?) | 17:12 |
| TheJulia | FYI, for those that don't see it, dtantsur has put the list of critical Queens bugs that need to land and be backported this week under the bugs section. | 17:12 |
| dtantsur | rloo: very likely so | 17:12 |
| dtantsur | rloo: after solving the API issue we talked about | 17:13 |
| rloo | dtantsur: we should ping vendors that need to update their docs wrt classic driver deprecations | 17:13 |
| * dtantsur finally has a devstack environment to test things | 17:13 | |
| dtantsur | rloo: well, I did a call on the ML, and at least 2 vendors proposed patches | 17:13 |
| rloo | dtantsur: ok good. | 17:13 |
| * dtantsur hands TheJulia a loooong stick to poke people | 17:13 | |
| *** fragatina has quit IRC | 17:13 | |
| jroll | the critical bugs are at line 215, for anyone else that also can't read today | 17:13 |
| rloo | dtantsur: also the TODOs wrt migrating CI to hardware types. who's going to do all those? | 17:14 |
| TheJulia | dtantsur: is the end sharpened ? | 17:14 |
| dtantsur | TheJulia: just enough to make it annoying | 17:14 |
| TheJulia | dtantsur: awesome! | 17:14 |
| dtantsur | rloo: I suspect me, unless somebody wants to help | 17:14 |
| rloo | just a heads up in case you miss it, traits is almost done but we forgot one thing, it'll need to be backported (L285) | 17:14 |
| TheJulia | mgoddard: re: traits, is any further action absolutely required for this release? | 17:15 |
| * TheJulia looks at 285 | 17:15 | |
| jroll | should just be that one | 17:15 |
| TheJulia | https://review.openstack.org/#/c/543461/ | 17:16 |
| patchbot | patch 543461 - ironic - Validate instance_info.traits against node traits | 17:16 |
| rloo | it isn't clear to me what we did wrt routed network support :) L314+ | 17:16 |
| * TheJulia stats a list | 17:16 | |
| dtantsur | hjensas: mind cleaning it up please ^^^? | 17:16 |
| dtantsur | to make it clear what is to be done for queens, what is a follow-up for rocky, etc | 17:17 |
| hjensas | o/ Will do. | 17:17 |
| rloo | i am deleting the 'split away tempest plugin', meant to do that before this meeting (L423) | 17:17 |
| TheJulia | Looks like the the ansible docs need a revision, and we should likely try to land/backport | 17:18 |
| dtantsur | oh, before I forgot: TheJulia we need to document creating queens jobs for the tempest plugin in our releasing documentation | 17:18 |
| dtantsur | and, well, create queens jobs :) | 17:18 |
| mgoddard_ | TheJulia: that patch is borderline required IMO, but would be nice to get it in. I have just found a small issue with the nova virt driver that will need fixing | 17:18 |
| rloo | TheJulia: wrt bifrost L431 -- that's the bug we're fixing now, right? | 17:19 |
| TheJulia | dtantsur: is it done for this cycle? | 17:19 |
| dtantsur | TheJulia: nope, just remembered | 17:19 |
| *** e0ne has quit IRC | 17:19 | |
| dtantsur | I can do it while we talk | 17:19 |
| TheJulia | rloo: no, different but not really a big deal, just needs to be gotten to "soon" since it is all the way off in keystoneauth1 | 17:20 |
| rloo | TheJulia: :-( | 17:20 |
| TheJulia | Yeah :\ | 17:20 |
| TheJulia | Anyway, I think we've looked most everything over for subteams | 17:21 |
| TheJulia | Are we ready to move on? | 17:21 |
| rloo | + moving on. do we want to continue with these subteam statuses until after PTG, or put on hold until after PTG? | 17:21 |
| rloo | although i guess we aren't quite done with queens so maybe continue... | 17:22 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Add jobs for stable/queens https://review.openstack.org/543555 | 17:22 |
| dtantsur | TheJulia: ^^^ | 17:22 |
| TheJulia | We ought to hold off for next week, I think next week will all be discussion | 17:22 |
| TheJulia | dtantsur: thanks! | 17:22 |
| TheJulia | #topic Priorities for this week | 17:23 |
| *** openstack changes topic to "Priorities for this week (Meeting topic: ironic)" | 17:23 | |
| TheJulia | I'm going to remove the list of things from last week that are struck out | 17:23 |
| TheJulia | dtantsur: do you think we should explicitly add the list of bugs to the priority list? | 17:24 |
| rloo | TheJulia: dtantsur's doc patch for classic drivers dep: https://review.openstack.org/#/c/537959/ | 17:25 |
| patchbot | patch 537959 - ironic - Switch contributor documentation to hardware types | 17:25 |
| dtantsur | TheJulia: probably won't hurt | 17:25 |
| *** etingof is now known as etingof|afk | 17:25 | |
| TheJulia | dtantsur: can you perform that copy/paste? | 17:25 |
| dtantsur | yep | 17:25 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: releasing docs: document stable jobs for the tempest plugin https://review.openstack.org/543558 | 17:25 |
| TheJulia | I think that works order wise | 17:26 |
| TheJulia | thoughts/objections? | 17:26 |
| dtantsur | LGTM | 17:27 |
| TheJulia | Are we ready to move on? | 17:27 |
| rloo | i guess it is implicit that the traits patch is a weekly priority? | 17:27 |
| dtantsur | it's in "Required backports" | 17:27 |
| rloo | dtantsur: hence 'implicit' :) | 17:28 |
| dtantsur | :) | 17:28 |
| *** hjensas has quit IRC | 17:28 | |
| TheJulia | One thing to keep in mind, if anyone becomes aware of something that must be backported, please raise visibility as soon as possible. | 17:29 |
| TheJulia | Time to move on :) | 17:29 |
| dtantsur | I think https://review.openstack.org/542214 is nice to have | 17:29 |
| patchbot | patch 542214 - ironic-inspector - Only set switch_id in local_link_connection if it ... | 17:29 |
| TheJulia | I agree | 17:30 |
| TheJulia | #topic Bug Triaging for the week | 17:30 |
| *** openstack changes topic to "Bug Triaging for the week (Meeting topic: ironic)" | 17:30 | |
| *** serlex has quit IRC | 17:30 | |
| TheJulia | Same as last week? | 17:30 |
| dtantsur | ++ | 17:31 |
| *** rbartal has quit IRC | 17:31 | |
| TheJulia | #action Everyone to triage/review bugs in preparation for final Queens release. | 17:31 |
| TheJulia | Moving on! | 17:31 |
| TheJulia | #topic Discussion | 17:31 |
| *** openstack changes topic to "Discussion (Meeting topic: ironic)" | 17:31 | |
| TheJulia | First, and only topic it looks like, is what to do with grenade. | 17:32 |
| rloo | fix it :) | 17:32 |
| TheJulia | The problem is there is no fixing it as-is... | 17:32 |
| rloo | so it seems like grenade framework doesn't work for us/rolling upgrades | 17:32 |
| *** moshele has joined #openstack-ironic | 17:32 | |
| rloo | have we had discussion with the grenade folks in the past about it? cuz we're now trying to continue to hack to get it to work for us | 17:33 |
| rloo | and we hack something but then don't follow up. and then something breaks :-( | 17:33 |
| TheJulia | Yeah, we can't complete the nova upgrade without a nova patch in-place to handle version negotiation either. | 17:33 |
| jroll | what's the latest problem with grenade? | 17:34 |
| rloo | which doesn't mean that we shouldn't hack something now but ... | 17:34 |
| TheJulia | jroll: tl;dr sqlalchemy gets upgraded, and old nova is incompatible with newer sqlalchemy | 17:34 |
| TheJulia | *boom* | 17:34 |
| *** ijw has joined #openstack-ironic | 17:34 | |
| rloo | and in our rolling upgrades scenario, we don't upgrade nova, just ironic | 17:35 |
| * jroll thinks he needs more time than we have to fully understand the thing | 17:35 | |
| rloo | cuz the order of upgrading is ironic first, then nova | 17:35 |
| jroll | are we back to the segfault problem, is my actual question | 17:35 |
| TheJulia | And since we don't upgrade ironic-api either, we can't actually upgrade nova | 17:35 |
| TheJulia | jroll: We are! :) | 17:35 |
| *** moshele has quit IRC | 17:35 | |
| jroll | TheJulia: that seems to me like a critical bug to be fixed, likely by the nova team | 17:35 |
| * jroll recalls dansmith saying similar, and then the bug disappeared for a while | 17:36 | |
| TheJulia | A critical bug in Pike? | 17:36 |
| jroll | yes | 17:36 |
| rloo | but should old s/w be expected to work with new packages? | 17:36 |
| *** AlexeyAbashkin has quit IRC | 17:36 | |
| jroll | running software should not be expected to segfault after an apt-get upgrade. | 17:36 |
| jroll | ever, that's a bug, flat-out. | 17:37 |
| rloo | jroll: ok, in that case, it is a nova bug. | 17:37 |
| dtantsur | we don't upper-cap sqlalchemy in requirements, so we're expected to work with newer versions | 17:37 |
| dansmith | rloo: a segv after a package upgrade would be a bug in some library | 17:38 |
| jroll | I'm totally open to a conversation about whether grenade is the right tool for the job here, but it seems to me we've been doing a lot to hack around this bug, and then complaining that grenade makes those hacks hard :) | 17:38 |
| TheJulia | So then it is a nova bug | 17:38 |
| dansmith | rloo: there should be nothing you can do from python land to segv yourself | 17:38 |
| * jroll isn't sure it's a nova bug, but it's a bug with how nova interacts with the system, yes | 17:38 | |
| rloo | dansmith: that is good to know! | 17:38 |
| dansmith | jroll: you might even argue that grenade is the right tool since it's poking something that needs fixing :) | 17:39 |
| TheJulia | jroll: I think at the same time, we have an unrealistic scenario that we're executing with grenade | 17:39 |
| jroll | dansmith: yeah, I should have finished with "so that's a separate conversation" :) | 17:39 |
| dansmith | jroll: aye :) | 17:39 |
| dansmith | TheJulia: what's unrealistic about it? | 17:39 |
| dansmith | aside from the fact that nobody would deploy any of this from devstack anyway | 17:39 |
| TheJulia | dansmith: Upgrade everything but nova on the same machine without isolation of underlying shared packages | 17:39 |
| TheJulia | which we do because we can't run newer nova with older ironic | 17:40 |
| dansmith | TheJulia: I don't think that's unrealistic | 17:40 |
| dansmith | it's unideal for sure, | 17:40 |
| dansmith | but if the package versions don't prohibit it, I think people would expect that should work | 17:40 |
| jroll | would and do, unfortunately | 17:41 |
| dansmith | right | 17:41 |
| TheJulia | so will we actually get traction for nova to fix it in stable/pike? | 17:41 |
| TheJulia | Well, for a fix to land | 17:41 |
| dansmith | if there's something nova has to do, then sure, but I can't imagine what that is | 17:42 |
| rloo | i think we may need to work with nova to help pinpoint where/how it is failing... seems like if we take ironic out of the picture, nova should still segv? | 17:42 |
| TheJulia | If dtantsur's assertion that projects must be compatible with future sqlalchemy versions, then there is an extra kwarg that needs to be removed that is currently ignored I believe | 17:42 |
| TheJulia | if the underlying bytecode is removed that the python runtime is using, does it recompile the bytecode? | 17:43 |
| jroll | dansmith: I think it's less that nova needs to do something and more us begging for help because we've cumulatively put hundreds of people-hours into trying to track this down and/or fix it :( | 17:43 |
| dansmith | jroll: I hear ya | 17:43 |
| dtantsur | TheJulia: IIRC yes | 17:44 |
| dansmith | TheJulia: that shouldn't cause an segv, otherwise that'd be a python bug | 17:44 |
| rloo | doesn't nova have a rolling upgrades/grenade job? I'd think it would have barfed there too? | 17:44 |
| *** hjensas has joined #openstack-ironic | 17:45 | |
| dansmith | rloo: several of them yeah | 17:45 |
| jroll | rloo: nova-conductor is upgraded in that job (which is the service that is segfaulting) | 17:45 |
| jroll | that's why it isn't seen there | 17:45 |
| TheJulia | but do those upgrades not actually upgrade nova? | 17:45 |
| rloo | jroll: really, i thought in our job, we didn't upgrade nova. let's take it offline | 17:45 |
| jroll | rloo: correct, we do not upgrade nova. nova's grenade jobs do. nova-conductor only breaks when not upgraded. | 17:46 |
| dansmith | TheJulia: we upgrade pieces of nova in the partial job, but conductor always gets upgraded (i.e. restarted( | 17:46 |
| TheJulia | dansmith: ok | 17:46 |
| rloo | jroll: ah got it. so can we change their test to not upgrade and see if it barfs? | 17:46 |
| dansmith | rloo: no, the whole point of our grenade test is to upgrade conductor :) | 17:46 |
| TheJulia | The take-away I'm getting is we don't try and change the grenade scenario, that we hunt down and try and fix the root cause of the segfault? | 17:47 |
| dansmith | has anyone tried to reproduce this locally? | 17:47 |
| jroll | TheJulia: that's my opinion, yes | 17:47 |
| rloo | TheJulia: yup, we should fix root cause | 17:47 |
| dansmith | because doing that would let us get core files more easily and dig into what was going on when the segv is triggered | 17:47 |
| TheJulia | dansmith: I'm fairly sure I did so last week | 17:47 |
| * jroll has not tried locally | 17:47 | |
| dansmith | um okay :) | 17:48 |
| TheJulia | I wiped the machine out though | 17:48 |
| dansmith | TheJulia: does that mean you're fairly sure you reproduced it? or fairly sure you tried? | 17:48 |
| * rloo wonders why the segv appeared, then disappeared, then appeared again... | 17:48 | |
| dansmith | rloo: that's usually the nature of such things | 17:48 |
| TheJulia | dansmith: I really don't remember at this point :( | 17:49 |
| rloo | dansmith: that explains it then! | 17:49 |
| TheJulia | I think that I did, but last week was a blur | 17:49 |
| dansmith | they can be deterministic, but often not, due to ordering and timing | 17:49 |
| rloo | dansmith: so it might be hard to reproduce. great. | 17:50 |
| rloo | although zuul is having great luck reproducing | 17:50 |
| TheJulia | unless it is breaking updated bytecode that is causing the segfault.. I seem to remember the first time we ran into this we got some lsofs out of a running system where the conductor was crashing and we had some sqlalchemy files open but not all... | 17:51 |
| TheJulia | I'll continue to work on it this week, but with the constraint of not changing the job or scenario | 17:51 |
| dansmith | AFAIK, python only opens those files whilst loading them the first time, not continually | 17:52 |
| dansmith | and I don't think it ever purges them and has to reload them | 17:52 |
| jroll | I would imagine it's more about some shared library underneath getting upgraded | 17:52 |
| dansmith | yes | 17:52 |
| dansmith | I would bet on it | 17:52 |
| TheJulia | The case is the same for shared libraries | 17:53 |
| TheJulia | Open file handler don't change | 17:53 |
| dansmith | TheJulia: but shared libraries can be opened and closed, | 17:53 |
| TheJulia | it would have ot be opening a new file/library/thing that is often accessed | 17:53 |
| jroll | even when the process is forked? | 17:53 |
| dansmith | jroll: yes if it's just a fork | 17:53 |
| *** trown is now known as trown|lunch | 17:54 | |
| TheJulia | We're running out of time today | 17:54 |
| TheJulia | rloo: It doesn't look like we're goin gto get to RFEs at this point | 17:54 |
| rloo | no worries | 17:54 |
| rloo | i might poke people about them later. or not :) | 17:54 |
| TheJulia | rloo: I believe that is reasonable | 17:54 |
| TheJulia | #action TheJulia to try and reproduce the fun grenade crash situation locally and use that to try and collect data | 17:55 |
| TheJulia | Since we have only 4 minutes left, does anyone have anything else that needs to be discussed today? | 17:56 |
| * TheJulia queries crickets as a service | 17:56 | |
| * jroll has nothing | 17:57 | |
| rloo | crickets | 17:57 |
| * dtantsur too | 17:57 | |
| TheJulia | Okay, thanks everyone! | 17:57 |
| TheJulia | Have a wonderful week! | 17:57 |
| dtantsur | thanks TheJulia and congrats again | 17:57 |
| TheJulia | #endmeeting | 17:57 |
| *** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic" | 17:57 | |
| openstack | Meeting ended Mon Feb 12 17:57:43 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 17:57 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-02-12-17.00.html | 17:57 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-02-12-17.00.txt | 17:57 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-02-12-17.00.log.html | 17:57 |
| jroll | thanks | 17:58 |
| * jroll lunches | 17:58 | |
| *** mgoddard_ has quit IRC | 17:58 | |
| *** derekh has quit IRC | 18:00 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-python-agent master: Follow-up patch for documentation of rescue mode https://review.openstack.org/542339 | 18:04 |
| *** stendulker has quit IRC | 18:04 | |
| *** mriedem has joined #openstack-ironic | 18:10 | |
| *** hshiina has quit IRC | 18:11 | |
| *** harlowja has joined #openstack-ironic | 18:14 | |
| *** mriedem has left #openstack-ironic | 18:18 | |
| *** r-daneel has quit IRC | 18:27 | |
| crushil | Hi. My nodes are stuck in verifying state. What is the way to get the node out of that state or even clean up that node? | 18:28 |
| crushil | This is the current state of my node: +------------------------+------------------------+---------------+-------------+--------------------+-------------+ | 18:31 |
| crushil | | UUID | Name | Instance UUID | Power State | Provisioning State | Maintenance | | 18:31 |
| crushil | +------------------------+------------------------+---------------+-------------+--------------------+-------------+ | 18:31 |
| crushil | | eb178d43-9789-4bbb- | LOC_Cloud5_Controller1 | None | None | verifying | False | | 18:31 |
| crushil | | b8b0-175fcb15c65c | -broken | | | | | | 18:31 |
| *** r-daneel has joined #openstack-ironic | 18:35 | |
| openstackgerrit | Merged openstack/ironic-inspector stable/pike: Zuul: Remove project name https://review.openstack.org/542367 | 18:43 |
| openstackgerrit | Merged openstack/ironic-inspector master: ironic_inspector: ironic: Fix 'auth_type' when 'auth_strategy' is used https://review.openstack.org/542206 | 18:43 |
| *** nicodemos has quit IRC | 18:45 | |
| *** links has quit IRC | 18:45 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector stable/queens: ironic_inspector: ironic: Fix 'auth_type' when 'auth_strategy' is used https://review.openstack.org/543577 | 18:45 |
| *** ijw has quit IRC | 18:52 | |
| *** ijw has joined #openstack-ironic | 18:53 | |
| openstackgerrit | Merged openstack/ironic stable/queens: Prepare testing for stable/queens https://review.openstack.org/542714 | 18:54 |
| openstackgerrit | Merged openstack/ironic master: Fix grammar errors https://review.openstack.org/542734 | 18:54 |
| *** e0ne has joined #openstack-ironic | 18:55 | |
| *** dprince has quit IRC | 18:58 | |
| openstackgerrit | James E. Blair proposed openstack/ironic stable/pike: Zuul: Remove project name https://review.openstack.org/543581 | 19:02 |
| *** lucasagomes is now known as lucas-pto | 19:03 | |
| *** dprince has joined #openstack-ironic | 19:11 | |
| *** sambetts is now known as sambetts|afk | 19:13 | |
| *** rwsu has quit IRC | 19:13 | |
| *** rwsu has joined #openstack-ironic | 19:13 | |
| *** jtomasek has quit IRC | 19:14 | |
| *** dprince has quit IRC | 19:15 | |
| *** moshele has joined #openstack-ironic | 19:15 | |
| *** dprince has joined #openstack-ironic | 19:15 | |
| *** tesseract has quit IRC | 19:18 | |
| *** SerenaFeng has quit IRC | 19:19 | |
| *** racedo has quit IRC | 19:23 | |
| rloo | crushil: sorry, was in meetings. did you figure it out? did anything time out? verifying is basically checking the power state of the node. | 19:25 |
| crushil | rloo, Np. I was able to force delete and create it again, and sidestep that error. I will let you know if I see it again. :) | 19:26 |
| rloo | crushil: ok thx. it would be good to know where it got stuck, i thought we timed out wrt the power stuff, but wondering if it was at your xclarity driver level, waiting to hear back from your xclarity node? | 19:27 |
| crushil | rloo, No. This was a pxe_ssh node trying to provision a VM node | 19:28 |
| crushil | And this was on RHOSP10 which is Newton | 19:28 |
| rloo | crushil: OH. newton. what's that? :) | 19:28 |
| crushil | rloo, Haha | 19:28 |
| rloo | crushil: btw, i was wondering when your company was going to have third party CI for the xclarity stuff | 19:29 |
| crushil | We are working on it. Maybe in the next month or so. I was told we need to have it running by the end of this release? | 19:30 |
| *** moshele has quit IRC | 19:30 | |
| rloo | crushil: to be honest, i don't recall the deadlines. if you were told that, then it is probably right. | 19:31 |
| crushil | rloo, And I need to still push a doc patch and a patch fixing the nits. I'm just trying to meet a deadline for MWC right now. :) | 19:31 |
| *** e0ne has quit IRC | 19:31 | |
| crushil | After that, I will be able to resume all the work to meet all the community requirements. :) | 19:32 |
| rloo | crushil: ok, I'll let the new PTL know. TheJulia ^^. Don't worry, we won't just yank out the code. I don't think. :) In fact, I think you may be the first guinea pig (err company) that has landed a new hardware type since we added that new third-party Ci rule... | 19:34 |
| rloo | (apart from ansible driver, but they had their ducks lined up already I think) | 19:34 |
| * TheJulia pulls her brain out of her huge review queue | 19:35 | |
| dtantsur | re CI I think we agreed by the release or really soon afterwards (with voting on stable/queens as well) | 19:35 |
| rloo | dtantsur: OH. crushil ^^ | 19:35 |
| TheJulia | Yeah, As a community I seem to remember agreement was with-in six months of code landing was acceptable with a preference on towards 3 months | 19:36 |
| dtantsur | do we have it written somewhere? I think we should | 19:37 |
| rloo | dtantsur: we better have it written down. was it in that spec?... | 19:37 |
| * dtantsur does not remember | 19:38 | |
| rloo | dtantsur: ohhh. https://specs.openstack.org/openstack/ironic-specs/specs/6.2/third-party-ci.html | 19:39 |
| TheJulia | I do believe it is written down... I just don't remember where | 19:39 |
| rloo | dtantsur, TheJulia: "From Newton feature freeze forward, all new ironic drivers must show that they have a system performing CI testing and meet all infra [3] and ironic requirements in order to land code into the ironic tree." | 19:39 |
| dtantsur | yeah, I guess we used a more relaxed version here | 19:40 |
| dtantsur | which I don't regret too much, because it's kind of a chicken-and-egg problem | 19:40 |
| rloo | dtantsur: yeah. | 19:40 |
| TheJulia | oh, absolutely is a chicken/egg problem | 19:40 |
| *** e0ne has joined #openstack-ironic | 19:41 | |
| rloo | dtantsur, TheJulia: so are we good with TheJulia's 6-months-after-code-lands? | 19:41 |
| *** lbragstad_ has joined #openstack-ironic | 19:41 | |
| TheJulia | I think it was one view that we ended up actually reaching consensus, but it might just be in meeting notes somewhere. | 19:41 |
| TheJulia | s/one view/oneview/ | 19:41 |
| rloo | TheJulia: very possible that the spec is outdated, we don't go back to update after the fact. | 19:41 |
| *** e0ne has quit IRC | 19:41 | |
| dtantsur | I'm fine with "6 months after code AND before the next release's feature freeze" | 19:42 |
| TheJulia | Yeah, I feel like we need an *outdated* banner or something | 19:42 |
| *** trown|lunch is now known as trown | 19:43 | |
| *** moshele has joined #openstack-ironic | 19:43 | |
| *** lbragstad_ has quit IRC | 19:43 | |
| rloo | This is the only documentation I could find intree about third party. https://docs.openstack.org/ironic/latest/contributor/third-party-ci.html | 19:43 |
| dtantsur | not quite impressive.. | 19:44 |
| dtantsur | anyway, see you tomorrow | 19:44 |
| *** dtantsur is now known as dtantsur|afk | 19:44 | |
| dtantsur|afk | o/ | 19:44 |
| TheJulia | ugh, yeah, that needs some edigs | 19:44 |
| TheJulia | edits | 19:44 |
| TheJulia | goodnight dtantsur|afk | 19:44 |
| rloo | ciao dtantsur|afk | 19:44 |
| * TheJulia goes back to the crazy queue | 19:45 | |
| *** ijw_ has joined #openstack-ironic | 19:45 | |
| *** moshele has quit IRC | 19:46 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Add optional healthcheck middleware https://review.openstack.org/542992 | 19:48 |
| *** ijw has quit IRC | 19:48 | |
| *** Nisha_Agarwal has quit IRC | 19:50 | |
| openstackgerrit | Merged openstack/ironic-python-agent stable/queens: Update .gitreview for stable/queens https://review.openstack.org/542155 | 20:01 |
| rloo | TheJulia: just wanted to check with you, this deletes multinode grenade from gate: https://review.openstack.org/#/c/543543/ | 20:09 |
| patchbot | patch 543543 - ironic - Stop running non-voting jobs in gate | 20:09 |
| openstackgerrit | Adam Kimball proposed openstack/ironic master: Add OpenBMC class for accessing nodes using OpenBMC https://review.openstack.org/429836 | 20:09 |
| TheJulia | We really should just comment it out :\ | 20:10 |
| * TheJulia shifts location because the wind is really starting to pickup here | 20:11 | |
| rloo | TheJulia: right, but in the 'let's just get stuff in and save zuul time'. i figured this one is an easy copy/paste from the check gate. | 20:11 |
| * rloo sees TheJulia gone with the wind | 20:11 | |
| *** moshele has joined #openstack-ironic | 20:13 | |
| TheJulia | lol | 20:13 |
| openstackgerrit | Adam Kimball proposed openstack/ironic master: OpenBMC Hardware Type https://review.openstack.org/403835 | 20:14 |
| TheJulia | from that POV, I agree, no sense in making zuul do more work | 20:14 |
| *** moshele has quit IRC | 20:15 | |
| openstackgerrit | Paul Belanger proposed openstack/ironic master: Stop running non-voting jobs in gate https://review.openstack.org/543543 | 20:16 |
| *** ijw_ has quit IRC | 20:18 | |
| *** dprince has quit IRC | 20:19 | |
| *** jcoufal has quit IRC | 20:25 | |
| rloo | jlvillal: you avail to review an easy one? We don't have any queens release notes w/o this: https://review.openstack.org/#/c/542209/ | 20:27 |
| patchbot | patch 542209 - ironic - Update reno for stable/queens | 20:27 |
| *** moshele has joined #openstack-ironic | 20:27 | |
| *** hw_wutianwei has quit IRC | 20:31 | |
| *** hw_wutianwei has joined #openstack-ironic | 20:32 | |
| *** moshele has quit IRC | 20:33 | |
| TheJulia | rloo: I just took care of it | 20:33 |
| TheJulia | rloo: thanks! | 20:33 |
| rloo | TheJulia: thx, I didn't want to bother you since you have so much on your plate :) | 20:33 |
| TheJulia | <3 | 20:33 |
| rloo | TheJulia: also, you stopped being blown by the wind already? :) | 20:33 |
| TheJulia | I was sitting outside.... | 20:34 |
| TheJulia | Inside now, the wind was forecast to only be 5 knots, it is gusting at 25-30 | 20:35 |
| rloo | TheJulia: make sure Toto is inside too :) | 20:38 |
| TheJulia | did I say 25-30.... I think it just gusted to 35-ish | 20:38 |
| *** livelace-link has joined #openstack-ironic | 20:44 | |
| *** r-daneel has quit IRC | 21:17 | |
| *** r-daneel has joined #openstack-ironic | 21:17 | |
| openstackgerrit | Merged openstack/ironic master: Update reno for stable/queens https://review.openstack.org/542209 | 21:18 |
| rloo | sambetts|afk: wrt IPA API versioning, it isn't finished :-( I added a comment to the RFE bug with the outstanding items: https://bugs.launchpad.net/ironic-python-agent/+bug/1602265/comments/7 | 21:26 |
| openstack | Launchpad bug 1602265 in ironic-python-agent "[RFE] IPA API Versioning" [Wishlist,In progress] - Assigned to Sam Betts (sambetts) | 21:26 |
| *** baha has quit IRC | 21:43 | |
| *** ijw has joined #openstack-ironic | 21:45 | |
| *** bfournie has quit IRC | 21:58 | |
| *** trown is now known as trown|outtypewww | 22:05 | |
| rloo | TheJulia, jlvillal, vdrok: any of you have opinions on this RFE? I think it can be approved w/o a spec: https://bugs.launchpad.net/ironic/+bug/1748224 | 22:06 |
| openstack | Launchpad bug 1748224 in Ironic "[RFE] Stop special handling of mime extensions in the API" [Wishlist,Confirmed] | 22:06 |
| jlvillal | rloo, I saw a patch up for that... | 22:07 |
| jlvillal | rloo, Merged in fact... | 22:07 |
| *** Sukhdev has joined #openstack-ironic | 22:07 | |
| rloo | jlvillal: already? there was a patch that we merged, that 'fixed' it but allowed that old behaviour. this is to remove the old behaviour. | 22:07 |
| jlvillal | rloo, https://review.openstack.org/#/c/541289/ | 22:07 |
| patchbot | patch 541289 - ironic - Stop guessing mime types based on URLs (MERGED) | 22:07 |
| TheJulia | rloo: I'm good with without a spec for that | 22:08 |
| jlvillal | rloo, Ah. Okay another patch will still be needed. | 22:08 |
| TheJulia | :) | 22:08 |
| rloo | jlvillal: right. And... I got another ok from TheJulia, so good to go! :) | 22:08 |
| jlvillal | rloo, I'm with TheJulia on that. | 22:09 |
| * TheJulia context switches back other window | 22:09 | |
| rloo | and if anyone wants, this is a low hanging fruit RFE, ripe for execution! https://bugs.launchpad.net/ironic/+bug/1748224 | 22:09 |
| openstack | Launchpad bug 1748224 in Ironic "[RFE] Stop special handling of mime extensions in the API" [Wishlist,Confirmed] | 22:09 |
| *** threestrands has joined #openstack-ironic | 22:12 | |
| rloo | jroll: any idea whether this will need a spec? https://bugs.launchpad.net/ironic/+bug/1747466 | 22:12 |
| openstack | Launchpad bug 1747466 in Ironic "[RFE] Support alternative tempurl algorithms for swift" [Wishlist,Triaged] | 22:12 |
| *** ijw has quit IRC | 22:16 | |
| * TheJulia looks | 22:16 | |
| rloo | TheJulia: here's another one, I think this doesn't need a spec: https://bugs.launchpad.net/ironic/+bug/1743674 | 22:17 |
| openstack | Launchpad bug 1743674 in Ironic "[RFE] Fujitsu iRMC BIOS configuration" [Wishlist,In progress] - Assigned to Tuan (tuanla) | 22:17 |
| *** ijw has joined #openstack-ironic | 22:19 | |
| TheJulia | rloo: jroll: I'm good without a spec as long as there are not any additional mechanics to reach out to swift. If it works like the existing client for generation then \o/. Reaching out might require more, and even with some lightweight client driven mechanics, I don't think it really needs a spec, it is just an additional capability in my mind. | 22:21 |
| jroll | TheJulia: yeah, I'm worried we'll need to auth to swift there, which isn't strictly required today | 22:22 |
| *** crushil has quit IRC | 22:22 | |
| *** cdearborn has quit IRC | 22:23 | |
| jroll | it'd be just like the other clients, though, and we might have the code for that already | 22:23 |
| TheJulia | Yeah, I think we call swift client code for the creation today | 22:23 |
| rloo | jroll: if it is like the other clients, we should ahve the code. pas-ha made a lot of auth changes. let me see. | 22:23 |
| TheJulia | it just doesn't actually touch swift to do it because *magic* | 22:23 |
| jroll | cool, I haven't looked into it much. just saw that release note in swift and filed the bug as I knew we'd need it eventually | 22:24 |
| jroll | might be a decent feature for a newbie to hack on | 22:24 |
| rloo | jroll: i'm not sure i'd call it low-hanging fruit but i think i would be good with approving if someone could put down more notes on what needs to be done. cuz otherwise said newbie may be asking or doing something different than what you are envisioning... | 22:25 |
| rloo | jroll: the swift release notes link doesn't work cuz queens release page isn't available :-( | 22:26 |
| rloo | jroll: well, cuz 'current' release notes doesn't have that swift feature, and queens page ain't avail yet. | 22:27 |
| jroll | what the | 22:27 |
| TheJulia | likely a pike build occured and the pike renos got pushed up | 22:27 |
| rloo | TheJulia: more likely that the bot pushed a patch to generate the queens release note, and it didn't get approved yet. like our ironic one that just got approved today :) | 22:27 |
| jroll | there, link fixed :) | 22:27 |
| TheJulia | oh, that too! | 22:28 |
| TheJulia | jroll: thanks! | 22:28 |
| jroll | :) | 22:28 |
| rloo | jroll: and while i'm digesting that one, what about this one: https://bugs.launchpad.net/ironic/+bug/1748515 | 22:29 |
| openstack | Launchpad bug 1748515 in Ironic "[RFE] support healthcheck middleware" [Wishlist,In progress] - Assigned to Jim Rollenhagen (jim-rollenhagen) | 22:29 |
| rloo | jroll: looks like you're already coding it. does it need a spec? and should I trust you? :) | 22:29 |
| *** masber has quit IRC | 22:29 | |
| TheJulia | w/r/t irmc bios configuration, I'm good with it as it implements what is in the bios spec (at least, that is the idea) | 22:29 |
| rloo | TheJulia: yup, that's the idea :) ok thx, will approve it. | 22:30 |
| jroll | rloo: I imagine it doesn't need a spec. I also imagine some folks will want to have a larger openstack-wide consensus on it first, based on my conversation with the nova folks this morning. apparently that happened once: https://review.openstack.org/#/c/531456/ | 22:31 |
| patchbot | patch 531456 - oslo-specs - Initial Spec for common healthcheck middleware | 22:31 |
| jroll | rloo: I'm not going to spend the energy to figure this out for all of openstack, though, so I'm fine with just WONTFIX if that's what will happen here | 22:31 |
| *** rcernin has joined #openstack-ironic | 22:31 | |
| jroll | rloo: I'm just trying to make that middleware available in services that don't use paste, not make a healthcheck that solves all the world's problems :) | 22:32 |
| rloo | jroll: ah. ok, let me spend a bit more time on that one. i'm trying to spend as little time as possible with the rfe bugs, just 'triaging' quickly :) | 22:32 |
| jroll | rloo: yeah, fair enough :) | 22:33 |
| TheJulia | I also don't think it really needs a spec if there is community consensus. My immediate concern when I first saw the change set was that it would be useful to be able to disable the endpoint... as in run separate "public" and "private" API services with different operating settings | 22:33 |
| TheJulia | or... an operator could just put proxy config in to take care of that | 22:33 |
| jroll | right, there's a switch for it | 22:34 |
| TheJulia | in the new spec, or in the initial code posted? | 22:35 |
| jroll | TheJulia: oh, I haven't really read the new spec, sorry | 22:35 |
| jroll | rloo: I don't have much to add as to how we do the swift thing, hit /info, find a supported algorithm, generate the tempurl. maybe do something in validate() re: creds. | 22:38 |
| rloo | oh, now it is coming back to me. i can see why jroll is interested in this; I think I wrote something downstream about such a healthcheck... | 22:38 |
| jroll | rloo: haaaaaaaaaaaaaaaaaaaa. ha ha. | 22:38 |
| rloo | jroll: that's ok, i approved it :) | 22:39 |
| jroll | rloo: we use the oslo one everywhere that has paste. ironic and placement don't have paste so need to inject this in code. | 22:39 |
| jroll | cool, thank you :) | 22:39 |
| rloo | jroll: i have often wondered why we/ironic doesn't have paste. | 22:39 |
| rloo | jroll: but i never got beyond the wondering. do you know? | 22:39 |
| *** Goneri has quit IRC | 22:42 | |
| jroll | rloo: I've never wondered too much because I've never liked paste, honestly | 22:43 |
| TheJulia | jroll: validate upon conductor start might be better, unless your talking about validating a image_source url | 22:43 |
| rloo | jroll: oh, it isn't a case of liking or not, more like an 'inconsistency' cuz other projects/services use it. | 22:43 |
| jroll | rloo: sure, but I remain in happy bliss | 22:44 |
| jroll | :) | 22:44 |
| jroll | TheJulia: yeah, agree | 22:45 |
| * TheJulia has officially reached eyes glazing over for the day | 22:45 | |
| rloo | TheJulia: so I can quote you -- are you ok with approving https://bugs.launchpad.net/ironic/+bug/1748515 w/o a spec? | 22:45 |
| openstack | Launchpad bug 1748515 in Ironic "[RFE] support healthcheck middleware" [Wishlist,In progress] - Assigned to Jim Rollenhagen (jim-rollenhagen) | 22:46 |
| TheJulia | If there is community consensus on a standard, yes | 22:46 |
| * jroll runs away | 22:46 | |
| *** pmannidi has joined #openstack-ironic | 22:47 | |
| TheJulia | lol | 22:49 |
| TheJulia | jroll: there is no running away! | 22:49 |
| rloo | TheJulia: ok, so what you're saying is that it depends on the proposed spec: https://review.openstack.org/#/c/531456/, because even thought it will still be middleware, the 'api' will be diff from the existing one in oslo. | 22:50 |
| patchbot | patch 531456 - oslo-specs - Initial Spec for common healthcheck middleware | 22:50 |
| rloo | jroll: what's the benefit of implementing/hooking in existing healthcheck middleware? how will we deprecate that for the new shiny bauble? | 22:51 |
| TheJulia | yes, if we are just implementing the interaction that the spec calls to be supported, then I think we're in non-spec land for us. If we are writing something different or non-compliant, then I think a spec would be a good thing | 22:51 |
| * TheJulia begins to wonder if jroll really ran away :( | 22:51 | |
| *** jlabarre has quit IRC | 22:51 | |
| * rloo wonders why this healthcheck wasn't a rocky goal... | 22:52 | |
| * rloo thinks jroll is crying. | 22:52 | |
| rloo | he had a tendency to do that | 22:52 |
| rloo | poor boy | 22:52 |
| jroll | heh, just doing some house things | 22:53 |
| jroll | I'm not sure how we'd deprecate the old. I wasn't in the room when people decided we needed a new fancy one. it seems a long way off based on what I've seen/heard | 22:54 |
| jroll | I don't really care to wait for that to go through committee, though I'm happy to implement it when it does | 22:54 |
| jroll | I just want to make the existing oslo (read: openstack common) thing usable in ironic like it is elsewhere | 22:55 |
| rloo | jroll: is it something that we can speed up at the ptg? | 22:55 |
| jroll | rloo: TheJulia: ^ | 22:55 |
| jroll | rloo: it isn't something I care enough about to sit in a room and discuss much | 22:55 |
| rloo | jroll: or at least, let's get an idea of timeframe for the new-fangled healthcheck. maybe they have an idea of how projects can deprecate from old to new. | 22:55 |
| rloo | jroll: heh. i just don't want to have to deal with reviewing/landing code, just to have a new thing avail 6 months later. | 22:56 |
| jroll | I hear ya | 22:56 |
| TheJulia | rloo: goals are complex... | 23:00 |
| rloo | TheJulia: I'm sure. I'm not complaining, if they go with remove-mox and sighup/update configs, we're golden. almost done! | 23:01 |
| TheJulia | jroll: I'm cool with that, my literal only concern is divergence from the community as a whole which creates more work... kind of like microversion http headers | 23:02 |
| TheJulia | jroll: in other words, since it is already in oslo, I'm fine with that, but they shouldn't expect one to completely replace the other at that level either. It would hav eot be like healtcheckv2 or something silly | 23:03 |
| TheJulia | (oslo in that it is already in two projects... in theory) | 23:03 |
| jroll | TheJulia: sure. it's funny we're discussing it now, as I basically marked it WIP best I could after talking with nova this morning :) | 23:03 |
| jroll | I need to read up more on the new thing; at a glance I'm not sure it will be a thing that will get done | 23:04 |
| TheJulia | Nothing wrong with discussing something good | 23:04 |
| TheJulia | jroll: *sigh* | 23:04 |
| * jroll notices it says services that don't expose an API should be extended to expose one to add this middleware | 23:04 | |
| jroll | all of my O_o | 23:04 |
| * TheJulia *headdesks* | 23:04 | |
| rloo | jroll, TheJulia: I commented on the new spec, asked for their opinion on when it might be avail and whether they could document how to go from existing healthcheck to the proposed one. | 23:10 |
| rloo | and i refrained from saying 'i didn't really care what they did, just do something' | 23:11 |
| rloo | jroll, TheJulia: i have to take off. we shall continue this micro-saga later :D | 23:18 |
| jroll | \o | 23:19 |
| *** Sukhdev has quit IRC | 23:20 | |
| TheJulia | rloo: thanks! | 23:20 |
| *** AlexeyAbashkin has joined #openstack-ironic | 23:21 | |
| *** AlexeyAbashkin has quit IRC | 23:26 | |
| *** Sukhdev has joined #openstack-ironic | 23:32 | |
| openstackgerrit | James E. Blair proposed openstack/networking-generic-switch stable/pike: Zuul: Remove project name https://review.openstack.org/543655 | 23:44 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
