| *** harlowja has quit IRC | 00:05 | |
| *** Sukhdev has quit IRC | 00:07 | |
| JayF | Time for my typical end of day review requesting :P -- https://review.openstack.org/#/c/259135/ already has 1x+2 and 2x+1, would love additional reviews as I think this one should be good to land. https://review.openstack.org/263842 also needs reviews, but it's probably a little bit away from landing, but I'd really appreciate feedback on it especially at this point | 00:07 |
|---|---|---|
| *** Sukhdev has joined #openstack-ironic | 00:10 | |
| *** garthb has quit IRC | 00:16 | |
| *** suro-patz has quit IRC | 00:16 | |
| *** achanda has joined #openstack-ironic | 00:18 | |
| *** achanda has quit IRC | 00:24 | |
| *** baoli has joined #openstack-ironic | 00:32 | |
| *** baoli_ has joined #openstack-ironic | 00:34 | |
| *** penick has quit IRC | 00:36 | |
| *** mtanino_ has quit IRC | 00:37 | |
| *** mtanino__ is now known as mtanino_ | 00:37 | |
| *** baoli has quit IRC | 00:38 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Fix full_trusty_build once and for all https://review.openstack.org/298962 | 00:40 |
| *** baoli_ has quit IRC | 00:45 | |
| *** suro-patz has joined #openstack-ironic | 00:48 | |
| *** Marga_ has quit IRC | 00:53 | |
| *** hoangcx has joined #openstack-ironic | 00:54 | |
| *** harlowja has joined #openstack-ironic | 00:54 | |
| *** hoangcx has quit IRC | 01:01 | |
| *** hoangcx has joined #openstack-ironic | 01:01 | |
| *** baoli has joined #openstack-ironic | 01:06 | |
| *** achanda has joined #openstack-ironic | 01:06 | |
| *** Marga_ has joined #openstack-ironic | 01:06 | |
| *** Sukhdev has quit IRC | 01:08 | |
| *** baoli has quit IRC | 01:10 | |
| *** Marga_ has quit IRC | 01:11 | |
| *** baoli has joined #openstack-ironic | 01:11 | |
| *** baoli has quit IRC | 01:12 | |
| *** ChrisAusten has joined #openstack-ironic | 01:21 | |
| *** ChrisAusten has quit IRC | 01:21 | |
| *** yhvh has quit IRC | 01:29 | |
| *** yhvh has joined #openstack-ironic | 01:30 | |
| *** raginbajin has quit IRC | 01:30 | |
| *** krotscheck_dcm has quit IRC | 01:31 | |
| *** raginbajin has joined #openstack-ironic | 01:31 | |
| *** krotscheck has joined #openstack-ironic | 01:33 | |
| *** baoli has joined #openstack-ironic | 01:34 | |
| *** spandhe has quit IRC | 01:38 | |
| *** mtanino has joined #openstack-ironic | 01:39 | |
| *** baoli has quit IRC | 01:41 | |
| *** baoli has joined #openstack-ironic | 01:43 | |
| *** suro-patz has quit IRC | 01:51 | |
| *** baoli has quit IRC | 01:53 | |
| *** dims has quit IRC | 01:59 | |
| *** dims has joined #openstack-ironic | 02:04 | |
| *** saripurigopi has quit IRC | 02:25 | |
| *** hoangcx has quit IRC | 02:26 | |
| *** achanda has quit IRC | 02:27 | |
| *** mannidi__ has joined #openstack-ironic | 02:28 | |
| *** achanda has joined #openstack-ironic | 02:31 | |
| *** suro-patz has joined #openstack-ironic | 02:34 | |
| *** PhilTW has quit IRC | 02:35 | |
| *** hoangcx has joined #openstack-ironic | 02:35 | |
| *** Nakato_ is now known as Nakato | 02:44 | |
| *** dims_ has joined #openstack-ironic | 02:45 | |
| *** spandhe has joined #openstack-ironic | 02:45 | |
| *** suro-patz has quit IRC | 02:46 | |
| *** dims has quit IRC | 02:46 | |
| *** hoangcx has quit IRC | 02:50 | |
| *** hoangcx has joined #openstack-ironic | 02:51 | |
| *** Nisha has joined #openstack-ironic | 02:53 | |
| *** phuongnh has joined #openstack-ironic | 02:58 | |
| openstackgerrit | Yushiro FURUKAWA proposed openstack/ironic: Insert validation for cleaning with maintenance mode https://review.openstack.org/299097 | 03:07 |
| *** Marga_ has joined #openstack-ironic | 03:14 | |
| *** Marga_ has quit IRC | 03:18 | |
| *** Marga_ has joined #openstack-ironic | 03:21 | |
| *** yuanying has quit IRC | 03:22 | |
| *** baoli has joined #openstack-ironic | 03:28 | |
| *** rbudden has quit IRC | 03:32 | |
| *** baoli has quit IRC | 03:32 | |
| *** jaybeale has joined #openstack-ironic | 03:33 | |
| *** harshs has joined #openstack-ironic | 03:36 | |
| *** Marga_ has quit IRC | 03:36 | |
| *** Marga_ has joined #openstack-ironic | 03:37 | |
| *** links has joined #openstack-ironic | 03:40 | |
| openstackgerrit | Merged openstack/proliantutils: Adds test coverage in tox.ini https://review.openstack.org/293936 | 03:42 |
| *** hoangcx has quit IRC | 03:43 | |
| *** phuongnh has quit IRC | 03:43 | |
| *** hoangcx has joined #openstack-ironic | 03:48 | |
| *** spandhe_ has joined #openstack-ironic | 03:49 | |
| *** spandhe has quit IRC | 03:51 | |
| *** spandhe_ is now known as spandhe | 03:51 | |
| *** achanda has quit IRC | 04:03 | |
| openstackgerrit | Yushiro FURUKAWA proposed openstack/ironic: Insert validation for cleaning with maintenance mode https://review.openstack.org/299097 | 04:05 |
| *** hoangcx_ has joined #openstack-ironic | 04:12 | |
| *** saripurigopi has joined #openstack-ironic | 04:12 | |
| *** hoangcx_ has quit IRC | 04:13 | |
| *** hoangcx_ has joined #openstack-ironic | 04:13 | |
| *** hoangcx has quit IRC | 04:14 | |
| openstackgerrit | Merged openstack/python-ironicclient: Fix pep8 stderr warning regarding __all__ defined as list https://review.openstack.org/298355 | 04:14 |
| *** hoangcx_ is now known as hoangcx | 04:14 | |
| *** harshs has quit IRC | 04:17 | |
| *** harshs has joined #openstack-ironic | 04:22 | |
| *** ndipanov has quit IRC | 04:23 | |
| *** yuanying has joined #openstack-ironic | 04:24 | |
| *** vinm213 has joined #openstack-ironic | 04:31 | |
| *** jaybeale has quit IRC | 04:33 | |
| *** harlowja_at_home has joined #openstack-ironic | 04:33 | |
| *** garthb has joined #openstack-ironic | 04:34 | |
| *** achanda has joined #openstack-ironic | 04:37 | |
| *** Nisha has quit IRC | 04:55 | |
| *** vishwanathj is now known as vishwanathj_zzz | 05:03 | |
| *** Andy_Yan has joined #openstack-ironic | 05:18 | |
| *** spandhe has quit IRC | 05:19 | |
| *** mannidi__ has quit IRC | 05:20 | |
| *** mtanino has quit IRC | 05:21 | |
| *** spandhe has joined #openstack-ironic | 05:22 | |
| *** saripurigopi has quit IRC | 05:23 | |
| *** Sukhdev has joined #openstack-ironic | 05:28 | |
| *** vinm213 has quit IRC | 05:37 | |
| *** vmud213 has joined #openstack-ironic | 05:39 | |
| *** garthb has quit IRC | 05:39 | |
| *** rbrud has quit IRC | 05:39 | |
| *** Sukhdev has quit IRC | 05:39 | |
| *** spandhe has quit IRC | 05:43 | |
| *** ChubYann has quit IRC | 05:55 | |
| *** deray has joined #openstack-ironic | 05:57 | |
| *** deray has quit IRC | 05:58 | |
| *** harlowja_at_home has quit IRC | 06:01 | |
| *** saripurigopi has joined #openstack-ironic | 06:05 | |
| *** Sukhdev has joined #openstack-ironic | 06:09 | |
| *** saripurigopi has quit IRC | 06:12 | |
| *** saripurigopi has joined #openstack-ironic | 06:12 | |
| *** harshs has quit IRC | 06:16 | |
| *** rcernin has joined #openstack-ironic | 06:26 | |
| *** daemontool has joined #openstack-ironic | 06:37 | |
| *** tesseract has joined #openstack-ironic | 06:45 | |
| *** tesseract is now known as Guest23157 | 06:46 | |
| *** Nisha has joined #openstack-ironic | 06:50 | |
| *** saripurigopi has quit IRC | 06:50 | |
| *** aarefiev_ has joined #openstack-ironic | 06:56 | |
| *** jtomasek_ has joined #openstack-ironic | 07:02 | |
| *** Sukhdev has quit IRC | 07:08 | |
| *** moshele has joined #openstack-ironic | 07:25 | |
| *** ifarkas has joined #openstack-ironic | 07:28 | |
| *** daemontool has quit IRC | 07:30 | |
| *** daemontool has joined #openstack-ironic | 07:41 | |
| *** jtomasek has quit IRC | 07:58 | |
| *** mbound has joined #openstack-ironic | 07:59 | |
| *** hoangcx has quit IRC | 08:00 | |
| *** mkoderer__ has quit IRC | 08:00 | |
| *** hoangcx has joined #openstack-ironic | 08:00 | |
| *** jtomasek_ has quit IRC | 08:01 | |
| *** mkoderer__ has joined #openstack-ironic | 08:03 | |
| pas-ha | morning Ironic | 08:04 |
| *** kromanenko has joined #openstack-ironic | 08:13 | |
| *** _degorenko|afk is now known as degorenko | 08:18 | |
| *** ndipanov has joined #openstack-ironic | 08:20 | |
| *** ohamada has joined #openstack-ironic | 08:20 | |
| *** achanda has quit IRC | 08:23 | |
| *** lucas-dinner is now known as lucasagomes | 08:23 | |
| lucasagomes | morning all | 08:24 |
| lucasagomes | jroll, o/ | 08:24 |
| *** jcoufal has joined #openstack-ironic | 08:24 | |
| *** jistr has joined #openstack-ironic | 08:25 | |
| *** hoangcx has quit IRC | 08:29 | |
| *** derekh has joined #openstack-ironic | 08:29 | |
| *** smoriya_ has quit IRC | 08:30 | |
| *** afaranha has quit IRC | 08:30 | |
| *** mgould has joined #openstack-ironic | 08:41 | |
| mgould | morning Ironicers! | 08:45 |
| *** e0ne has joined #openstack-ironic | 08:48 | |
| vmud213 | morning mgould, lucasagomes, pas-ha and others | 08:48 |
| mgould | vmud213, lucasagomes pas-ha morning! | 08:48 |
| lucasagomes | o/ | 08:59 |
| mkovacik | morning all! | 09:03 |
| sergek | morning Ironic! | 09:04 |
| *** alexpilotti has joined #openstack-ironic | 09:04 | |
| *** achanda has joined #openstack-ironic | 09:06 | |
| *** hoangcx has joined #openstack-ironic | 09:07 | |
| *** smoriya_ has joined #openstack-ironic | 09:07 | |
| *** pece has joined #openstack-ironic | 09:08 | |
| *** sambetts|afk is now known as sambetts | 09:11 | |
| sambetts | Morning all | 09:11 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 09:12 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: WIP: add network drivers https://review.openstack.org/285852 | 09:12 |
| mgould | sergek, sambetts morning! | 09:14 |
| sergek | o/ mgould | 09:15 |
| sambetts | o/ mgould sergek | 09:15 |
| sergek | o/ sambetts | 09:15 |
| *** daemontool has quit IRC | 09:17 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add support for dib based agent ramdisk in lib/ironic https://review.openstack.org/264579 | 09:23 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP Add tests for cleaning https://review.openstack.org/299237 | 09:23 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 09:26 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Allow to build user image with DIB https://review.openstack.org/256363 | 09:26 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add Link-Local-Connection info to ironic port https://review.openstack.org/256365 | 09:26 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Added operator documentation for ironic portgroups https://review.openstack.org/228496 | 09:26 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: refactor ironic enroll-node code https://review.openstack.org/256364 | 09:26 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add configure_provision_network function https://review.openstack.org/256367 | 09:26 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Update Ironic VM network connection https://review.openstack.org/256366 | 09:26 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 09:26 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add Ironic/Neutron integration documentation https://review.openstack.org/258596 | 09:26 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Allow to use NETWORK_DRIVERS in devstack https://review.openstack.org/293520 | 09:26 |
| openstackgerrit | Tan Lin proposed openstack/ironic-specs: Support online upgrade https://review.openstack.org/299245 | 09:36 |
| *** lucasagomes has quit IRC | 09:38 | |
| *** dtantsur|afk is now known as dtantsur | 09:39 | |
| dtantsur | Morning Ironic | 09:39 |
| sambetts | Morning dtantsur | 09:39 |
| mgould | morning dtantsur | 09:40 |
| openstackgerrit | Tan Lin proposed openstack/ironic-specs: Support online upgrade https://review.openstack.org/299245 | 09:41 |
| *** lucasagomes has joined #openstack-ironic | 09:43 | |
| openstackgerrit | Merged openstack/ironic: Remove unneeded use of task.release_resources() https://review.openstack.org/296149 | 09:47 |
| *** Andy_Yan has quit IRC | 09:49 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Prevent create/update node name "" in lower API version https://review.openstack.org/299264 | 09:53 |
| vdrok | morning lucasagomes mgould mkovacik sambetts and dtantsur | 10:01 |
| lucasagomes | vdrok, o/ | 10:02 |
| mgould | vdrok, morning! | 10:02 |
| vdrok | lucasagomes: please take a look at https://review.openstack.org/#/c/276076/ https://review.openstack.org/#/c/295863/ when you have some time | 10:08 |
| lucasagomes | vdrok, will do | 10:08 |
| mkovacik | morning vdrok ! | 10:13 |
| *** achanda has quit IRC | 10:14 | |
| *** achanda has joined #openstack-ironic | 10:16 | |
| *** moshele has quit IRC | 10:18 | |
| Nisha | morning Ironicers | 10:18 |
| Nisha | i am stuck in CI....i am facing this issue on all the slaves http://paste.openstack.org/show/492422/ | 10:19 |
| Nisha | the main server from where slaves are created can do all the git operations | 10:20 |
| sambetts | Nisha: what OS are you using for your slaves? | 10:20 |
| Nisha | Ubuntu | 10:20 |
| Nisha | this was working till feb | 10:21 |
| Nisha | suddenly the issue started and can be seen on all the slaves :( | 10:21 |
| sambetts | sounds like something to do with the version of openssl you have installed doing some quick googleing | 10:21 |
| Nisha | but the openssl on main server and slaves are same | 10:21 |
| sambetts | I would try updating openssl in the slave | 10:22 |
| mkovacik | ifarkas, aarefiev, sambetts hi guys, please take look at https://review.openstack.org/#/c/267637/ when you've got a minute. Thanks! | 10:23 |
| Nisha | i tried already | 10:23 |
| Nisha | :( | 10:23 |
| Nisha | this is the version of openssl which works for teh main server but not in slavesopenssl 1.0.1f-1ubuntu2.18 | 10:23 |
| *** achanda has quit IRC | 10:27 | |
| *** hoangcx has quit IRC | 10:32 | |
| vdrok | Nisha: why not use https://git.openstack.org/cgit/openstack-dev/devstack ? | 10:34 |
| vdrok | and morning | 10:34 |
| Nisha | vdrok, that also gives same error | 10:34 |
| Nisha | :( | 10:34 |
| dtantsur | sounds like some proxy tried to get into HTTPS session with changing the certificate | 10:36 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 10:38 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to build user image with DIB https://review.openstack.org/256363 | 10:38 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Link-Local-Connection info to ironic port https://review.openstack.org/256365 | 10:38 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Added operator documentation for ironic portgroups https://review.openstack.org/228496 | 10:38 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/206244 | 10:39 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: refactor ironic enroll-node code https://review.openstack.org/256364 | 10:39 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add configure_provision_network function https://review.openstack.org/256367 | 10:39 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update Ironic VM network connection https://review.openstack.org/256366 | 10:39 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 10:39 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 10:39 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: add network drivers https://review.openstack.org/285852 | 10:39 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Ironic/Neutron integration documentation https://review.openstack.org/258596 | 10:39 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to use NETWORK_DRIVERS in devstack https://review.openstack.org/293520 | 10:39 |
| openstackgerrit | Merged openstack/ironic-webclient: Updated UX for Default Cloud dropdown https://review.openstack.org/294080 | 10:44 |
| *** moshele has joined #openstack-ironic | 10:52 | |
| *** aarefiev_ has quit IRC | 10:53 | |
| *** daemontool has joined #openstack-ironic | 10:57 | |
| *** smoriya_ has quit IRC | 11:03 | |
| *** dprince has joined #openstack-ironic | 11:05 | |
| jroll | mornin' | 11:11 |
| jroll | Nisha: I agree with dtantsur, there's a proxy or some other machine in the middle there | 11:12 |
| vdrok | morning jroll | 11:12 |
| dtantsur | morning jroll | 11:12 |
| -openstackstatus- NOTICE: Gate on project-config is currently broken due to IRC tests. The problem has been detected and we are working to fix the issue as soon as possible. | 11:14 | |
| jroll | ayyyyy http://logs.openstack.org/47/473b6b4661480e0a2e2c363553e48d62c7145272/post/ironic-python-agent-buildimage-coreos/b870122/console.html | 11:14 |
| jroll | \o/ | 11:14 |
| jroll | dtantsur: mind a look at the backports for this when you have a few? https://review.openstack.org/#/q/status:open+project:openstack/ironic-python-agent++topic:bug/1561756 | 11:15 |
| jroll | I think I'd like tonyb or mriedem ack for the BRANCH_PATH one, since it's not a bug | 11:15 |
| jroll | but I don't expect opposition since it's fixing the gate | 11:15 |
| dtantsur | :) | 11:16 |
| dtantsur | I'd say broken gate IS a bug | 11:16 |
| jroll | well | 11:16 |
| dtantsur | jroll, some changes clearly need rebase there | 11:16 |
| jroll | the branch_path one makes it put out the stable version, not fix the job :) | 11:17 |
| jroll | wat | 11:17 |
| jroll | :( | 11:17 |
| * jroll does it | 11:17 | |
| dtantsur | I'll have a quick snack in the meanwhile | 11:17 |
| jroll | wait, what needs a rebase? | 11:17 |
| jroll | they all show successful builds | 11:17 |
| openstackgerrit | Merged openstack/ironic: Move testcases related to parse_instance_info() https://review.openstack.org/297186 | 11:20 |
| *** dprince has quit IRC | 11:20 | |
| sambetts | Morning jroll | 11:20 |
| jroll | \o sambetts | 11:21 |
| *** achanda has joined #openstack-ironic | 11:27 | |
| *** mkovacik_ has joined #openstack-ironic | 11:30 | |
| TheJulia | good morning | 11:31 |
| *** mkovacik has quit IRC | 11:32 | |
| dtantsur | jroll, https://review.openstack.org/#/c/298965/2 has outdated parent | 11:32 |
| dtantsur | ditto https://review.openstack.org/#/c/298966/ | 11:32 |
| jroll | dtantsur: oh, it's just a commit message update, should be okay | 11:33 |
| jroll | but I can push the button | 11:33 |
| dtantsur | I'm not sure how gerrit treats that | 11:33 |
| jroll | I think gerrit handles that stuff better now | 11:33 |
| dtantsur | really? we can try then | 11:33 |
| jroll | heh | 11:33 |
| jroll | I just rebased, sorry | 11:33 |
| *** achanda has quit IRC | 11:33 | |
| dtantsur | thanks | 11:34 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 11:37 |
| TheJulia | gerrit only handles that kind of stuff a little better noww :) | 11:37 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 11:37 |
| dtantsur | jroll, hmm, the inspector src job really looks broken on liberty.. | 11:40 |
| * TheJulia tries to wake up | 11:42 | |
| jroll | dtantsur: the one I'm looking at looks like a timeout | 11:44 |
| lucasagomes | TheJulia, jroll morning | 11:44 |
| dtantsur | oooops... I'm starting to suspect it never worked in the first place :D | 11:44 |
| dtantsur | maybe I need to backport the devstack plugin change | 11:45 |
| jroll | dtantsur: never, or more recently? | 11:45 |
| jroll | heh | 11:45 |
| jroll | morning lucasagomes, TheJulia :) | 11:45 |
| dtantsur | I probably need to backport https://github.com/openstack/ironic-inspector/commit/c926e1e36ff64718395699712ffb96d0f808e5e5 indeed | 11:45 |
| jroll | dtantsur: looks like it's using coreos, but not from source http://logs.openstack.org/35/297735/3/check/gate-ironic-inspector-dsvm-ipa-src-nv/88026b5/logs/devstacklog.txt.gz#_2016-03-29_21_11_33_072 | 11:46 |
| dtantsur | I don't think it even uses coreos.. | 11:47 |
| *** openstackgerrit has quit IRC | 11:47 | |
| *** dprince has joined #openstack-ironic | 11:47 | |
| jroll | it certainly downloads it | 11:47 |
| dtantsur | jroll, this is for deploy, the inspector job only build the ramdisk from source for inspection process itself | 11:48 |
| dtantsur | not sure why I did that actually... | 11:48 |
| jroll | oh O_o | 11:48 |
| *** openstackgerrit has joined #openstack-ironic | 11:48 | |
| dtantsur | yeah, now it looks like a strange decision. probably I had something in mind back then :D | 11:48 |
| jroll | ah yep I see the dib stuff now | 11:48 |
| dtantsur | lemme try to cherry-pick a couple of mitaka changes | 11:49 |
| dtantsur | https://review.openstack.org/299327 is a patch we need, I think | 11:50 |
| *** aarefiev_ has joined #openstack-ironic | 11:51 | |
| TheJulia | the only reason for home automation.... have the 15A circuit that connects to all the test equipment receieve power. | 11:53 |
| openstackgerrit | Merged openstack/ironic-webclient: Removed border from header https://review.openstack.org/294081 | 12:02 |
| *** trown|outtypewww is now known as trown | 12:05 | |
| openstackgerrit | Merged openstack/ironic-webclient: Adjusted flow for header elements https://review.openstack.org/294114 | 12:05 |
| openstackgerrit | Merged openstack/ironic-webclient: Introduced btn-gradient styles. https://review.openstack.org/294082 | 12:05 |
| openstackgerrit | Merged openstack/ironic-webclient: Created common navigation header https://review.openstack.org/295329 | 12:05 |
| openstackgerrit | Merged openstack/ironic-webclient: Added node list control buttons https://review.openstack.org/295330 | 12:05 |
| openstackgerrit | Merged openstack/ironic-webclient: Applied table styles and column ordering to node list https://review.openstack.org/295399 | 12:06 |
| *** sinval has joined #openstack-ironic | 12:10 | |
| *** raildo-afk is now known as raildo | 12:11 | |
| *** thrash|g0ne is now known as thrash | 12:11 | |
| *** Nisha has quit IRC | 12:15 | |
| *** david-lyle_ has joined #openstack-ironic | 12:17 | |
| *** david-lyle has quit IRC | 12:17 | |
| *** Nisha has joined #openstack-ironic | 12:18 | |
| *** jjohnson2 has joined #openstack-ironic | 12:18 | |
| openstackgerrit | Merged openstack/ironic-inspector: Update versions on the release notes page https://review.openstack.org/298768 | 12:24 |
| *** lucasagomes is now known as lucas-hungry | 12:25 | |
| *** daemontool has quit IRC | 12:27 | |
| *** daemontool has joined #openstack-ironic | 12:32 | |
| *** openstackgerrit has quit IRC | 12:33 | |
| *** openstackgerrit has joined #openstack-ironic | 12:33 | |
| *** mbound has quit IRC | 12:34 | |
| *** mbound has joined #openstack-ironic | 12:36 | |
| *** baoli has joined #openstack-ironic | 12:39 | |
| *** baoli has quit IRC | 12:39 | |
| *** baoli has joined #openstack-ironic | 12:39 | |
| *** gridinv has quit IRC | 12:46 | |
| *** e0ne has quit IRC | 12:48 | |
| *** Nisha has quit IRC | 12:52 | |
| dtantsur | jroll, looks like we're good to land https://review.openstack.org/#/c/299326/ and https://review.openstack.org/#/c/299327/ | 12:52 |
| *** baoli_ has joined #openstack-ironic | 12:54 | |
| jroll | dtantsur: is the "don't download every time" really needed, or just an optimization? | 12:55 |
| dtantsur | jroll, not really needed, but the 2nd patch does not apply cleanly without it | 12:55 |
| jroll | dtantsur: fair enough | 12:55 |
| dtantsur | so I thought it's safe to backport both (since none touch the actual code we ship to people) | 12:56 |
| jroll | +A for both | 12:56 |
| jroll | ya | 12:56 |
| dtantsur | thanks! | 12:56 |
| jroll | np, thank you | 12:56 |
| *** Nisha has joined #openstack-ironic | 12:57 | |
| *** baoli has quit IRC | 12:57 | |
| *** sinval has quit IRC | 12:57 | |
| Nisha | vdrok, there is no proxy in between as i can do "apt-get install" without any issue | 12:59 |
| jroll | Nisha: there is something in the middle, github's certificate is not self-signed, nor signed by ‘/C=AU/ST=Some-State/O=Internet Widgits Pty Ltd’ | 13:06 |
| jroll | or your dns is broken | 13:07 |
| jroll | Nisha: when you run "dig +short github.com", what do you get? | 13:07 |
| jroll | ignore me, it's in your paste | 13:08 |
| jroll | your dns looks fine, there must be a proxy or an attacker in between you and github | 13:08 |
| jroll | from my end, at that IP, I see the cert is signed by digicert: | 13:10 |
| jroll | depth=1 /C=US/O=DigiCert Inc/OU=www.digicert.com/CN=DigiCert SHA2 Extended Validation Server CA | 13:10 |
| jroll | rather, issuer=/C=US/O=DigiCert Inc/OU=www.digicert.com/CN=DigiCert SHA2 Extended Validation Server CA | 13:11 |
| jroll | (same thing) | 13:11 |
| Nisha | jroll, i get dig +short github.com | 13:11 |
| Nisha | 192.30.252.128 | 13:11 |
| jroll | yeah | 13:11 |
| jroll | that looks fine, something is proxying your traffic to github | 13:11 |
| *** rbudden has joined #openstack-ironic | 13:15 | |
| *** mbound has quit IRC | 13:16 | |
| Nisha | jroll, no idea what is happening but there is no proxy as i can do "apt-get install" without any problem | 13:16 |
| *** mbound has joined #openstack-ironic | 13:17 | |
| *** openstackgerrit has quit IRC | 13:18 | |
| *** klindgren_ has joined #openstack-ironic | 13:19 | |
| jroll | Nisha: that just tells you that either apt isn't verifying certificates or apt traffic isn't being inspected by the proxy | 13:19 |
| *** openstackgerrit has joined #openstack-ironic | 13:19 | |
| *** Marga__ has joined #openstack-ironic | 13:19 | |
| jroll | Nisha: there's clearly something between you and github manipulating your traffic, if you're getting a self-signed cert for github | 13:19 |
| jroll | either way, it's nothing we can help you with :( | 13:19 |
| Nisha | jroll, :( | 13:19 |
| *** klindgren has quit IRC | 13:21 | |
| sambetts | googling for Internet Widgits Pty Ltd, does not give good results | 13:21 |
| jroll | sambetts: that's the default when generating a self-signed cert iirc | 13:21 |
| *** Marga_ has quit IRC | 13:21 | |
| sambetts | jroll: oh right :/ | 13:21 |
| sambetts | Nisha: perhaps try doing a trace to see where the packet goes on its way to github? | 13:22 |
| jroll | yeah, confirmed that is the default, as well as AU/Some-State | 13:23 |
| Nisha | jroll, yeah | 13:24 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 13:28 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 13:29 |
| *** links has quit IRC | 13:30 | |
| *** achanda has joined #openstack-ironic | 13:30 | |
| *** [1]cdearborn has joined #openstack-ironic | 13:35 | |
| *** lucas-hungry is now known as lucasagomes | 13:36 | |
| *** achanda has quit IRC | 13:36 | |
| *** trown is now known as trown|afk | 13:37 | |
| sambetts | jroll, Nisha's server that he's seeing this failure on is a VM, when he does a traceroute the first IP is the IP address of the host, is it possible the VM's network configuration could be to blame, e.g. the packets are proxying through the host or something? | 13:37 |
| sambetts | jroll: pulling a straws now :-P | 13:37 |
| jroll | sambetts: only if there's something on the host listening on 443 and terminating ssl, and then re-encrypting to github | 13:38 |
| jroll | sambetts: also, she :) | 13:39 |
| sambetts | oops :-P, when she does a curl https://github.com/openstack-dev/devstack --insecure, she's getting this: http://paste.openstack.org/show/492446/ | 13:40 |
| sambetts | jroll: definatly sounds like something is getting in the way right? | 13:40 |
| jroll | O_o | 13:41 |
| jroll | yes | 13:41 |
| jroll | you know what | 13:41 |
| jroll | I wonder if something in that environment thinks 192.0.0.0/8 is private IP space and that IP is allocated on some internal network | 13:41 |
| sambetts | I did wonder if there might be an ip collision :/ | 13:42 |
| jroll | I think traceroute would suss that out, but not sure | 13:42 |
| *** mkovacik_ has quit IRC | 13:42 | |
| NobodyCam | good morning ironicers :) | 13:42 |
| sambetts | jroll: http://paste.openstack.org/show/492448/ | 13:42 |
| sambetts | jroll: thats her traceroute results :/ | 13:43 |
| jroll | welp | 13:43 |
| sambetts | NobodyCam: Hey! | 13:43 |
| *** mkovacik_ has joined #openstack-ironic | 13:43 | |
| jroll | I assume nothing interesting in routes? | 13:43 |
| jroll | the route is probably in the host or up the stack somewhere :/ | 13:43 |
| *** mkovacik_ has quit IRC | 13:43 | |
| sambetts | jroll: she said that 192.168.122.1 is the ip addr of her VM host | 13:43 |
| jroll | right | 13:43 |
| mgould | NobodyCam, morning | 13:44 |
| jroll | but yeah, this is the point where I put in a ticket with IT and ask what's going on | 13:44 |
| NobodyCam | morning sambetts jroll JayF devananda jlvillal dtantsur vdrok mgould TheJulia lucasagomes and all others not listed here :) | 13:44 |
| jroll | hiya NobodyCam, mgould | 13:44 |
| mgould | hi jroll | 13:44 |
| sambetts | jroll: ++, shes headed off to pick her son up now, but I think thats the point its gotten to | 13:45 |
| jroll | sambetts: cool | 13:45 |
| *** mgoddard_ has joined #openstack-ironic | 13:45 | |
| jroll | yeah, like I said not much we can do :) | 13:45 |
| *** mkovacik_ has joined #openstack-ironic | 13:45 | |
| jroll | unless the VM has a special route for that address going out some private interface | 13:45 |
| sambetts | jroll: the weird thing is she said that on the host it works fine :/ | 13:46 |
| jroll | orly! | 13:46 |
| jroll | maybe it is a route on the vm then | 13:46 |
| sambetts | I suggested starting a fresh VM to see if she sees the same problem there | 13:47 |
| jroll | +1 | 13:47 |
| jroll | (but also check the VM's route table, heh) | 13:47 |
| *** mkovacik has joined #openstack-ironic | 13:48 | |
| lucasagomes | NobodyCam, hi there, good morning | 13:48 |
| *** mkovacik_ has quit IRC | 13:48 | |
| *** mgoddard has quit IRC | 13:48 | |
| NobodyCam | :) | 13:49 |
| vdrok | morning NobodyCam ! | 13:49 |
| openstackgerrit | Merged openstack/ironic-inspector: Remove downgrades from migrations https://review.openstack.org/297635 | 13:49 |
| *** e0ne has joined #openstack-ironic | 13:50 | |
| *** clenimar has joined #openstack-ironic | 13:51 | |
| *** ametts has joined #openstack-ironic | 13:57 | |
| dtantsur | morning NobodyCam | 13:57 |
| *** mkovacik has quit IRC | 13:58 | |
| NobodyCam | morning :) | 13:59 |
| *** caiobo has joined #openstack-ironic | 13:59 | |
| jroll | dtantsur: I just realized that until those IPA patches are merged on stable, stable commits on IPA overwrite the published image >.> | 14:00 |
| dtantsur | oooops | 14:03 |
| sambetts | jroll: is this the | 14:03 |
| sambetts | bug/1561756 | 14:03 |
| sambetts | patches? | 14:03 |
| *** mgould has quit IRC | 14:03 | |
| jroll | sambetts: ya | 14:04 |
| jroll | it's been a problem for a while, so it's whatever | 14:04 |
| sambetts | jroll: whats the process for adding a post job to build/upload a tinyipa image? | 14:05 |
| *** mgoddard_ has quit IRC | 14:06 | |
| *** mgoddard has joined #openstack-ironic | 14:06 | |
| jroll | sambetts: make a script that can do it, then something like https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic-python-agent-jobs.yaml#L16 | 14:06 |
| sambetts | jroll: oh cool! | 14:06 |
| jlvillal | jroll, Is https://bugs.launchpad.net/ironic/+bug/1561100 the correct bug for https://review.openstack.org/#/c/247331/ ? | 14:09 |
| openstack | Launchpad bug 1561100 in Ironic "[RFE] Centralize config options in Ironic" [Undecided,In progress] - Assigned to Tan Lin (tan-lin-good) | 14:09 |
| jlvillal | If so I'll drop my -1 | 14:09 |
| NobodyCam | oh speaking of post job. I haven't checked did it work? | 14:10 |
| jroll | NobodyCam: yep :) | 14:10 |
| NobodyCam | awesome :) | 14:10 |
| NobodyCam | (happy dance) | 14:10 |
| jroll | sambetts: oh, one other thing, once you go to use the pre-built image in devstack, you'll also want a separate job from source for ipa changes | 14:10 |
| jlvillal | Good morning Ironic | 14:11 |
| sambetts | jroll: makes sense :) | 14:11 |
| *** Nisha has quit IRC | 14:11 | |
| *** e0ne has quit IRC | 14:11 | |
| jroll | hey hey jlvillal | 14:12 |
| jroll | jlvillal: I did request that we do oslo-config-gen first in the RFE, or we could make a separate one, either works for me | 14:13 |
| jlvillal | jroll, Okay. I'll drop the -1 and we can update the RFE to be a bit more specific. Thanks. | 14:13 |
| jroll | jlvillal: though, separate RFE we could approve now, and the centralization rfe is waiting for the cross-project spec | 14:13 |
| jroll | so maybe separate is better, and we can land that code | 14:14 |
| jlvillal | jroll, Ah good call | 14:14 |
| *** e0ne has joined #openstack-ironic | 14:16 | |
| vdrok | morning jlvillal | 14:20 |
| jlvillal | vdrok, Hi :) | 14:20 |
| *** trown|afk is now known as trown | 14:21 | |
| NobodyCam | morning jlvillal | 14:27 |
| *** links has joined #openstack-ironic | 14:30 | |
| *** Nisha has joined #openstack-ironic | 14:32 | |
| *** smoriya_ has joined #openstack-ironic | 14:34 | |
| *** spandhe has joined #openstack-ironic | 14:35 | |
| *** spandhe has quit IRC | 14:40 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Functional tests for portgroups in ironicclient https://review.openstack.org/297727 | 14:41 |
| *** Sukhdev has joined #openstack-ironic | 14:45 | |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic: Avoid throwing exception during lookup https://review.openstack.org/299450 | 14:45 |
| *** mtanino has joined #openstack-ironic | 14:45 | |
| *** links has quit IRC | 14:49 | |
| *** dtantsur is now known as dtantsur|bbl | 14:52 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Allow fetching IPA ramdisk with branch name https://review.openstack.org/297479 | 14:53 |
| *** garthb has joined #openstack-ironic | 14:55 | |
| vmud213 | jroll: Thanks for reviewing the patch. | 14:57 |
| jroll | vmud213: not a problem, does it make sense? | 14:57 |
| *** ametts has quit IRC | 14:57 | |
| *** thiagop has joined #openstack-ironic | 14:58 | |
| vmud213 | I think when the node fails to get the interfaces(when the interfaces is empty) lookup is getting called repeatedly | 14:58 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Prevent create/update node name "" in lower API version https://review.openstack.org/299264 | 14:58 |
| vmud213 | i.e after NodeNotFound exception is raised | 14:58 |
| vmud213 | i have attached a log in the bug report | 14:59 |
| vmud213 | Would you mind giving it a look | 14:59 |
| jroll | ooo, yeah IPA will retry | 14:59 |
| jroll | will look | 14:59 |
| thiagop | Good morning Ironic | 14:59 |
| jroll | vmud213: I don't think it will fetch interfaces again though | 14:59 |
| vmud213 | I observed it mostly when used fedora based ramdisk | 15:00 |
| vmud213 | In the scenario that i have included, except interfaces other inventory is populated. And later after some time it could even get the interfaces info | 15:01 |
| jroll | oh wow, it does re-fetch hw info | 15:02 |
| jroll | that is odd | 15:02 |
| vmud213 | hmm :) | 15:02 |
| jroll | I wonder if the fedora ramdisk brings up the agent before networking, hi lucasagomes :D | 15:02 |
| jroll | those come from sysfs | 15:02 |
| vmud213 | though the issue is not consistent, i have seen this couple of times.. | 15:03 |
| thiagop | jroll: wrt periodic tasks on that dynamic allocation stuff, wdyt about the periodic task changing the state of the node? Is it a problem? | 15:03 |
| lucasagomes | hi there | 15:03 |
| * lucasagomes reads | 15:03 | |
| vmud213 | yes.IPA readsit from /sys/class/net/.. | 15:03 |
| thiagop | JayF: ^ | 15:03 |
| jroll | thiagop: that seems fine, yeah | 15:03 |
| jroll | lucasagomes: tl;dr IPA lookup() the first time doesn't have interfaces, it retries, eventually it does have them | 15:03 |
| vmud213 | May be IPA service starts well before the system stablizes | 15:04 |
| lucasagomes | jroll, gotcha, maybe we need to change the services | 15:04 |
| thiagop | jroll: we meant to move the node to 'manageable' when the OneView user takes the node to ensure it'll pass cleaning after being returned | 15:04 |
| lucasagomes | right now it has a https://github.com/openstack/diskimage-builder/blob/master/elements/ironic-agent/install.d/ironic-agent-source-install/ironic-python-agent.service#L3 | 15:04 |
| jroll | lucasagomes: hm, maybe needs to depend on whatever brings up sysfs too | 15:06 |
| jroll | vmud213: in any case, the patch still needs work, left a note in the patch | 15:06 |
| lucasagomes | jroll, yeah, apparently there's a "After=network-online.target" | 15:06 |
| jroll | heh | 15:06 |
| lucasagomes | which should wait for the network to be configured | 15:06 |
| *** moshele has left #openstack-ironic | 15:07 | |
| *** moshele has joined #openstack-ironic | 15:07 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 15:10 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 15:10 |
| vmud213 | jroll: sure will look into your comments | 15:14 |
| vmud213 | thanks | 15:14 |
| jroll | np | 15:15 |
| *** vishwanathj_zzz is now known as vishwanathj | 15:16 | |
| *** mgould has joined #openstack-ironic | 15:17 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Allow fetching IPA ramdisk with branch name https://review.openstack.org/297479 | 15:19 |
| *** vmud213 has quit IRC | 15:19 | |
| *** Guest23157 has quit IRC | 15:21 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 15:24 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Allow to build user image with DIB https://review.openstack.org/256363 | 15:24 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add Link-Local-Connection info to ironic port https://review.openstack.org/256365 | 15:24 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Added operator documentation for ironic portgroups https://review.openstack.org/228496 | 15:24 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: refactor ironic enroll-node code https://review.openstack.org/256364 | 15:24 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add configure_provision_network function https://review.openstack.org/256367 | 15:24 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Update Ironic VM network connection https://review.openstack.org/256366 | 15:24 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 15:24 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 15:24 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: WIP: add network drivers https://review.openstack.org/285852 | 15:24 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add Ironic/Neutron integration documentation https://review.openstack.org/258596 | 15:24 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Allow to use NETWORK_DRIVERS in devstack https://review.openstack.org/293520 | 15:24 |
| *** Sukhdev has quit IRC | 15:25 | |
| *** Sukhdev has joined #openstack-ironic | 15:26 | |
| *** achanda has joined #openstack-ironic | 15:34 | |
| *** Sukhdev has quit IRC | 15:35 | |
| *** achanda has quit IRC | 15:39 | |
| *** rbrud has joined #openstack-ironic | 15:39 | |
| *** moshele has quit IRC | 15:39 | |
| sambetts | Every time you do that ^ my CI queue cries :-P | 15:41 |
| openstackgerrit | Sofiia Andriichenko proposed openstack/python-ironicclient: Add test for chassis-node-list https://review.openstack.org/299482 | 15:41 |
| openstackgerrit | Sofiia Andriichenko proposed openstack/python-ironicclient: Add new tests https://review.openstack.org/299483 | 15:41 |
| *** yolanda has quit IRC | 15:42 | |
| jroll | lol | 15:42 |
| *** dprince has quit IRC | 15:43 | |
| *** rbrud has quit IRC | 15:43 | |
| *** dprince has joined #openstack-ironic | 15:43 | |
| *** rbrud has joined #openstack-ironic | 15:43 | |
| sambetts | jroll: Something like that https://review.openstack.org/#/c/299485/ | 15:44 |
| *** piet has joined #openstack-ironic | 15:46 | |
| *** yolanda has joined #openstack-ironic | 15:46 | |
| jroll | sambetts: basically, left you a comment | 15:47 |
| sambetts | jroll: good point about the cp + rm thing :-P its because i originally had if branch_path != master around the rms | 15:48 |
| jroll | sambetts: yeah, for coreos we used to publish without the branch, so it's only there for compat | 15:48 |
| JayF | oooh nice, we're going to publish the tinyipa image \o/ | 15:48 |
| JayF | exciting | 15:48 |
| jroll | ya | 15:48 |
| jroll | morning JayF :) | 15:49 |
| sambetts | o/ JayF | 15:49 |
| *** smoriya_ has quit IRC | 15:50 | |
| sambetts | jroll: thats what I thought so I removed it :-P | 15:50 |
| *** jaybeale has joined #openstack-ironic | 15:50 | |
| *** ametts has joined #openstack-ironic | 15:51 | |
| jroll | :) | 15:51 |
| sambetts | jroll: Do you think I should publish a tar ball too or just the files? I wasn't sure | 15:52 |
| JayF | jroll: qq re: renos -- is Tan Lin right on this review of the reno? https://review.openstack.org/#/c/263842/5/releasenotes/notes/bug-1531229-set-boot-device-before-cleaning-92498dfcd0cd80ae.yaml (wondering about use of deprecation: vs upgrade: sections) | 15:53 |
| jroll | sambetts: some people like tarballs, I guess, can't hurt to be consistent | 15:53 |
| sambetts | makes sense | 15:53 |
| jroll | JayF: yeah, I like to cover both sections tbh | 15:53 |
| jroll | really people should read both, but just to be safe | 15:54 |
| *** Sukhdev has joined #openstack-ironic | 15:54 | |
| *** klindgren_ has quit IRC | 15:54 | |
| JayF | aight | 15:56 |
| JayF | https://review.openstack.org/#/c/259135/ has 2x+2, 4x+1 if someone wants to review/approve it :D | 15:57 |
| lucasagomes | JayF, looking | 15:58 |
| JayF | Thanks Lucas \o/ | 15:58 |
| *** mgoddard_ has joined #openstack-ironic | 15:59 | |
| *** rajinir has joined #openstack-ironic | 16:00 | |
| *** suro-patz has joined #openstack-ironic | 16:00 | |
| *** klindgren_ has joined #openstack-ironic | 16:01 | |
| lucasagomes | JayF, boom +a | 16:02 |
| JayF | \o/ | 16:02 |
| *** penick has joined #openstack-ironic | 16:02 | |
| JayF | another Rackspaceism of Ironic crushinated | 16:02 |
| *** mgoddard has quit IRC | 16:02 | |
| lucasagomes | :-D | 16:05 |
| lucasagomes | bring them all | 16:06 |
| JayF | I'm working on it, lol | 16:06 |
| *** dtantsur|bbl is now known as dtantsur | 16:06 | |
| JayF | https://review.openstack.org/#/c/263842 is my other active patch along those lines; but that one aleady has some feedback and I should get that responded to in the next few minutes | 16:06 |
| *** penick has quit IRC | 16:07 | |
| *** stevemar has quit IRC | 16:08 | |
| *** penick has joined #openstack-ironic | 16:09 | |
| *** jistr has quit IRC | 16:10 | |
| *** mkovacik has joined #openstack-ironic | 16:10 | |
| *** penick_ has joined #openstack-ironic | 16:12 | |
| devananda | JayF: \o/ | 16:14 |
| devananda | also, good morning everyone :) | 16:14 |
| jroll | ohai devananda | 16:14 |
| jroll | devananda: I come bearing reviews for you | 16:14 |
| *** penick has quit IRC | 16:14 | |
| *** penick_ is now known as penick | 16:14 | |
| *** suro-patz has quit IRC | 16:14 | |
| jroll | devananda: when you have a few, https://review.openstack.org/#/q/status:open+project:openstack/ironic-python-agent+topic:bug/1561756 | 16:15 |
| jroll | the one ci failure is rechecking | 16:15 |
| *** jcoufal has quit IRC | 16:16 | |
| JayF | is openstackgerrit bot having a bad day? Didn't see my patch posted in here | 16:17 |
| lucasagomes | devananda, morning | 16:18 |
| JayF | https://review.openstack.org/#/c/263842/ updated now, fwiw, even though openstackgerrit doesn't wanna talk about it :P | 16:18 |
| *** daemontool has quit IRC | 16:18 | |
| thiagop | morning devananda | 16:18 |
| *** openstack has joined #openstack-ironic | 17:05 | |
| *** jtomasek_ has joined #openstack-ironic | 17:09 | |
| *** PhilipTW has joined #openstack-ironic | 17:10 | |
| dtantsur | see you tomorrow | 17:15 |
| NobodyCam | night dtantsur | 17:16 |
| *** dtantsur is now known as dtantsur|afk | 17:16 | |
| *** harshs has joined #openstack-ironic | 17:16 | |
| *** Haomeng has quit IRC | 17:18 | |
| *** suro-patz has joined #openstack-ironic | 17:20 | |
| * Nisha also calling it a day...no luck with my CI issues | 17:20 | |
| * sambetts is also heading off | 17:21 | |
| NobodyCam | night Nisha / sambetts | 17:22 |
| *** sambetts is now known as sambetts|afk | 17:22 | |
| jlvillal | sambetts|afk: Ciao! | 17:24 |
| jlvillal | Nisha: Ciao! | 17:24 |
| jlvillal | dtantsur|afk: Ciao! | 17:25 |
| * jlvillal hopes he didn't miss anyone.... | 17:25 | |
| *** moshele has joined #openstack-ironic | 17:25 | |
| *** degorenko is now known as _degorenko|afk | 17:25 | |
| jlvillal | lucasagomes: Ciao! | 17:25 |
| *** electrofelix has quit IRC | 17:28 | |
| *** Goneri has quit IRC | 17:28 | |
| *** spandhe has joined #openstack-ironic | 17:29 | |
| *** trown|lunch is now known as trown | 17:30 | |
| *** piet has quit IRC | 17:30 | |
| * mgould -> home; good night! | 17:31 | |
| NobodyCam | night mgould | 17:31 |
| *** mgould has quit IRC | 17:31 | |
| *** jtomasek_ has quit IRC | 17:31 | |
| *** kromanenko has quit IRC | 17:41 | |
| *** Goneri has joined #openstack-ironic | 17:42 | |
| *** e0ne has quit IRC | 17:45 | |
| *** jjohnson2 has quit IRC | 17:47 | |
| *** jjohnson2 has joined #openstack-ironic | 17:48 | |
| *** Sukhdev has quit IRC | 17:48 | |
| JayF | https://review.openstack.org/#/c/246036/ has 2x+2 and 8x+1 if someone wants to workflow it | 17:50 |
| *** piet has joined #openstack-ironic | 17:50 | |
| *** moshele has quit IRC | 17:51 | |
| NobodyCam | JayF: has the dependent patch landed? | 17:51 |
| JayF | ah good point, nope | 17:52 |
| JayF | lol | 17:52 |
| *** Goneri has quit IRC | 17:52 | |
| NobodyCam | :p | 17:52 |
| *** Goneri has joined #openstack-ironic | 17:52 | |
| JayF | dtantsur|afk: going to approve https://review.openstack.org/#/c/290876 and address your nits in a followup patch | 17:52 |
| NobodyCam | old gerrit was much better at showing that :-p | 17:53 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Improve __str__ method of BaseCommandResult https://review.openstack.org/299560 | 17:57 |
| *** e0ne has joined #openstack-ironic | 18:00 | |
| *** thrash|f00dz is now known as thrash | 18:01 | |
| JayF | https://review.openstack.org/#/c/291427/ 2x+2, 6x+1s, and no dependent patches :P | 18:02 |
| JayF | I guess, and depends on nothing would be the more clearer way to say it | 18:02 |
| *** ohamada has quit IRC | 18:04 | |
| *** Nisha has quit IRC | 18:06 | |
| *** piet has quit IRC | 18:07 | |
| *** piet has joined #openstack-ironic | 18:07 | |
| *** daemontool has quit IRC | 18:07 | |
| *** piet has quit IRC | 18:11 | |
| *** piet has joined #openstack-ironic | 18:11 | |
| *** e0ne has quit IRC | 18:11 | |
| *** moshele has joined #openstack-ironic | 18:13 | |
| *** Marga__ has quit IRC | 18:17 | |
| *** e0ne has joined #openstack-ironic | 18:24 | |
| *** e0ne has quit IRC | 18:26 | |
| *** davidlenwell has quit IRC | 18:27 | |
| *** mgoddard_ has joined #openstack-ironic | 18:28 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Don't power off non-deploying iLO nodes in takeover https://review.openstack.org/299573 | 18:29 |
| *** piet has quit IRC | 18:29 | |
| *** mgoddard_ has quit IRC | 18:30 | |
| *** baoli_ has quit IRC | 18:30 | |
| *** mgoddard_ has joined #openstack-ironic | 18:31 | |
| *** e0ne has joined #openstack-ironic | 18:31 | |
| *** mgoddard has quit IRC | 18:31 | |
| *** e0ne has quit IRC | 18:31 | |
| *** e0ne has joined #openstack-ironic | 18:33 | |
| *** e0ne has quit IRC | 18:33 | |
| *** davidlenwell has joined #openstack-ironic | 18:34 | |
| *** e0ne has joined #openstack-ironic | 18:35 | |
| *** piet has joined #openstack-ironic | 18:35 | |
| *** mgoddard_ has quit IRC | 18:35 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Don't power off non-deploying iLO nodes in takeover https://review.openstack.org/299573 | 18:36 |
| *** e0ne has quit IRC | 18:36 | |
| *** e0ne has joined #openstack-ironic | 18:37 | |
| *** e0ne has quit IRC | 18:37 | |
| *** piet has quit IRC | 18:38 | |
| *** piet has joined #openstack-ironic | 18:38 | |
| *** Marga_ has joined #openstack-ironic | 18:42 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Active Node Creation via adopt state https://review.openstack.org/275766 | 18:43 |
| *** Sukhdev has joined #openstack-ironic | 18:44 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Add __str__ method to BaseCommandResult https://review.openstack.org/290876 | 18:46 |
| *** krotscheck is now known as krotscheck_dcm | 18:46 | |
| *** piet has quit IRC | 18:50 | |
| *** piet has joined #openstack-ironic | 18:50 | |
| *** moshele has quit IRC | 18:51 | |
| openstackgerrit | Saulo-Aislan proposed openstack/python-oneviewclient: Add connections to Server Profile https://review.openstack.org/299579 | 18:53 |
| *** piet has quit IRC | 18:56 | |
| *** piet has joined #openstack-ironic | 18:58 | |
| openstackgerrit | Julia Kreger proposed openstack/python-ironicclient: Client addition for Active Node Creation verb adopt https://review.openstack.org/275767 | 18:59 |
| *** ChubYann has joined #openstack-ironic | 18:59 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Make agent driver always PXE before cleaning https://review.openstack.org/263842 | 19:00 |
| *** mgoddard has joined #openstack-ironic | 19:02 | |
| *** baoli has joined #openstack-ironic | 19:05 | |
| *** aarefiev_ has quit IRC | 19:05 | |
| *** e0ne has joined #openstack-ironic | 19:11 | |
| *** mgoddard has quit IRC | 19:12 | |
| *** e0ne has quit IRC | 19:13 | |
| openstackgerrit | Merged openstack/ironic: Readability fixes for cleaning_reboot code https://review.openstack.org/299524 | 19:14 |
| *** jtomasek_ has joined #openstack-ironic | 19:15 | |
| *** piet has quit IRC | 19:20 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Don't power off non-deploying iLO nodes in takeover https://review.openstack.org/299573 | 19:20 |
| *** piet has joined #openstack-ironic | 19:21 | |
| *** achanda has quit IRC | 19:24 | |
| *** jaybeale has joined #openstack-ironic | 19:30 | |
| *** jaybeale_ has joined #openstack-ironic | 19:31 | |
| *** jaybeale has quit IRC | 19:31 | |
| *** Sukhdev has quit IRC | 19:44 | |
| *** baoli has quit IRC | 19:47 | |
| *** mbound has joined #openstack-ironic | 19:48 | |
| *** baoli has joined #openstack-ironic | 19:49 | |
| *** baoli has quit IRC | 19:49 | |
| *** baoli has joined #openstack-ironic | 19:50 | |
| *** achanda has joined #openstack-ironic | 19:51 | |
| *** piet has quit IRC | 19:51 | |
| mrda | Morning Ironic | 19:57 |
| NobodyCam | morning mrda :) | 19:58 |
| mrda | hey NobodyCam | 19:58 |
| openstackgerrit | Andreas Jaeger proposed openstack/ironic: Remove unused constraints line https://review.openstack.org/299600 | 19:59 |
| *** AJaeger has joined #openstack-ironic | 19:59 | |
| AJaeger | jroll: hope that 299600 is ok to avoid any confusions on constraint usage. Or do you want to do constraints properly soon? | 20:00 |
| jroll | AJaeger: yeah, I'd rather just do it properly | 20:00 |
| *** jtomasek_ has quit IRC | 20:00 | |
| AJaeger | jroll: soon? Then I abandon mine. Or you do yours on top of mine. I'm happy to abandon... | 20:00 |
| jroll | AJaeger: yeah, I'll do it, go ahead and abandon. thanks though :) | 20:01 |
| AJaeger | ;) | 20:01 |
| *** baoli has quit IRC | 20:01 | |
| *** piet has joined #openstack-ironic | 20:02 | |
| openstackgerrit | Clif Houck proposed openstack/ironic-python-agent: Add docstrings to all functions in Agent standby extension https://review.openstack.org/288632 | 20:04 |
| *** AJaeger has left #openstack-ironic | 20:05 | |
| clif_h | ^ jroll JayF think its golden this time | 20:05 |
| clif_h | at least I'll get a change in before the summit | 20:05 |
| TheJulia | clif_h: nice | 20:08 |
| TheJulia | clif_h: line 72, should that be an upper case O in InvalidCOmmandParamsError ? | 20:09 |
| clif_h | lol no D: D: D: | 20:10 |
| * clif_h fixes | 20:10 | |
| TheJulia | clif_h: you may also want to look at line 107, since its not the contents, its the actual path to the image | 20:10 |
| TheJulia | when I read contents, I think block of data in memory | 20:10 |
| mag009_ | guys can someone explain to me how can I deploy over uefi by using ironic in standalone ? | 20:11 |
| clif_h | TheJulia: ah, right | 20:11 |
| clif_h | fixing | 20:11 |
| mag009_ | I get this error : Failed to prepare to deploy. Error: Failed to get IP address for any | | 20:11 |
| TheJulia | mag009_: Sure, give me about 10 minutes to finish reviewing whats on my screen | 20:11 |
| mag009_ | the dhcp is working I just enrolled my node using uefi/grub | 20:12 |
| mag009_ | sure thx | 20:12 |
| TheJulia | clif_h: you may want to note that image_info is an unused param on 108 | 20:12 |
| clif_h | TheJulia: fixed line 67 too | 20:12 |
| clif_h | TheJulia: this function was recently added, any reason why image_info is unused? | 20:13 |
| TheJulia | no clue... | 20:13 |
| TheJulia | I just looked and went "Hmm.... not used" | 20:13 |
| clif_h | It's only used once :| | 20:13 |
| clif_h | yeah | 20:13 |
| clif_h | I see that now | 20:14 |
| clif_h | well, not going to modify code in this patch | 20:14 |
| TheJulia | understood and appreciated :) | 20:14 |
| *** piet has quit IRC | 20:14 | |
| TheJulia | aww... I can't put the config drive on an FTP server *ducks* | 20:15 |
| clif_h | just add support for ftps (idk if that's even a thing0 | 20:15 |
| TheJulia | it actually is believe it or not | 20:15 |
| *** pleia2 has quit IRC | 20:19 | |
| *** pleia2 has joined #openstack-ironic | 20:20 | |
| TheJulia | mag009_: So failed ip address for any |? But your saying dhcp is working? do you mean that you can talk to the BMC and that it got an IP via the DHCP server? | 20:21 |
| clif_h | TheJulia: see anything else? | 20:21 |
| TheJulia | clif_h: almost to the end and not yet, really nice job :) | 20:21 |
| clif_h | thanks! just let me know when you're done then I'll push the next patch revision | 20:21 |
| * jroll promotes clif_h to tech writer | 20:22 | |
| mag009_ | no I mean the real IP pxe interface | 20:22 |
| clif_h | oh god | 20:22 |
| TheJulia | lol | 20:22 |
| mag009_ | bmc is out-of-band (drac) used to control the power | 20:22 |
| TheJulia | jroll: that does come with the manditory "write the user guide" requirement, yes? | 20:22 |
| clif_h | I need to do this for arsenal too | 20:22 |
| jroll | TheJulia: and install guide, and downstream docs | 20:22 |
| TheJulia | We should all talk about our doc at the summit, even if it is a 10 minute discussion. I feel like a lot of the files are just WAY to long for a a new user to consume and results in ERRINFORMATIONOVERLOAD | 20:23 |
| TheJulia | but my brain is wired a little differently | 20:23 |
| TheJulia | mag009_: so PXE gets an IP... what doesn't get an IP then? | 20:24 |
| TheJulia | mag009_: or better yet, where are you seeing the process break down? | 20:24 |
| mag009_ | TheJulia: as soon as I deploy an image it end up with this error | 20:24 |
| mag009_ | the server does not even turn on | 20:24 |
| TheJulia | clif_h: line 582, method name is run_image, but the description is something completely different :) And thats it :) | 20:25 |
| jroll | TheJulia: yeah, I think that's a fair determination | 20:25 |
| clif_h | TheJulia: I mean, my doc is what it does though lol | 20:25 |
| TheJulia | clif_h: true.... | 20:26 |
| TheJulia | *shrug* | 20:26 |
| * clif_h updates | 20:26 | |
| jroll | TheJulia: mag009_: I think UEFI depends on using neutron for dhcp... :/ :/ :/ | 20:26 |
| TheJulia | mag009_: Interesting, what is the error? | 20:26 |
| jroll | that's bad and we should feel bad | 20:26 |
| mag009_ | I guess thats my question :) | 20:27 |
| mag009_ | does it depend on neutron ? | 20:27 |
| TheJulia | jroll: not entirely sure on that, but I've got a wip'ed patch to look at it | 20:27 |
| TheJulia | only had to order hardware to test it with :) | 20:27 |
| jroll | https://github.com/openstack/ironic/blob/master/ironic/common/pxe_utils.py#L116 | 20:27 |
| jroll | comes from... | 20:27 |
| jroll | https://github.com/openstack/ironic/blob/master/ironic/common/pxe_utils.py#L247 | 20:27 |
| mag009_ | I guess it need the ip so it can write the files | 20:28 |
| jroll | yeah, I don't quite understand why | 20:28 |
| JayF | jroll: lucasagomes has a patch up to send root device hints via node obj to ipa on deploy | 20:28 |
| jroll | I feel like elilo doesn't do mac address matching or something | 20:28 |
| JayF | jroll: I think that's the best path to fixing up that class of bug | 20:28 |
| TheJulia | jroll: :\ | 20:29 |
| jroll | JayF: ECONTEXT | 20:29 |
| JayF | ah, still that's another reason UEFI would depend on neutron for dhcp | 20:29 |
| JayF | lol | 20:29 |
| jroll | TheJulia: mag009_: sounds like with ipxe enabled this shuold work okay, though | 20:29 |
| jroll | I thought bifrost did ipxe by default | 20:29 |
| TheJulia | jroll: it does | 20:30 |
| openstackgerrit | Clif Houck proposed openstack/ironic-python-agent: Add docstrings to all functions in Agent standby extension https://review.openstack.org/288632 | 20:30 |
| mag009_ | ill try again with ipxe | 20:31 |
| TheJulia | I'd need to context switch into UEFI+ipxe and see what we actually get... the to do list never ends :) | 20:31 |
| jroll | mag009_: cool, I'm eager to hear how it works :) | 20:31 |
| jroll | I think someone recently brought up "why are we using elilo" | 20:32 |
| jroll | still using* | 20:32 |
| mag009_ | btw I'm not using bitfrost I just installed ironic along side openstack | 20:32 |
| mag009_ | but I cannot use neutron ( we are using calico L3) plugin | 20:32 |
| mag009_ | so I'm using ironic in standalone mode | 20:32 |
| TheJulia | well, ipxe config for dhcp is fairly strait forward | 20:33 |
| mag009_ | yes I guess... lol I didn't have much success the first time. missing something maybe. | 20:33 |
| mag009_ | not maybe probably* | 20:33 |
| TheJulia | since your would have had to set your dhcp server up to reply for pxe anyway :) | 20:33 |
| jroll | mag009_: ah I see | 20:33 |
| *** achanda has quit IRC | 20:37 | |
| *** trown is now known as trown|outtypewww | 20:37 | |
| EmilienM | hi, by testing puppet-ironic, we found this bug: https://bugs.launchpad.net/ironic/+bug/1564075 | 20:39 |
| openstack | Launchpad bug 1564075 in Ironic "ironic-conductor does not retry to connect to rpc if connection failure" [Undecided,New] | 20:39 |
| EmilienM | let me know if it miss infos, I can easily reproduce it | 20:39 |
| jroll | EmilienM: fun, thanks for the report | 20:41 |
| jroll | master? | 20:41 |
| EmilienM | jroll: yeah. Our CI runs master | 20:41 |
| jroll | cool, ty | 20:41 |
| EmilienM | np! thx for looking | 20:41 |
| jroll | EmilienM: I do feel like there's been debates in the past if this is correct behavior or not, in other projects, but we will take a look | 20:42 |
| EmilienM | jroll: I found the same bug in Aodh Listener | 20:42 |
| EmilienM | but other projects, like nova-conductor, etc work | 20:43 |
| jroll | EmilienM: I don't disagree we differ from other projects here | 20:43 |
| EmilienM | for us, it's a bootstrap issue, that we can solve in manifests. But for operators, it can be more critical maybe, I think conductor should be able to reconnect if amqp is back | 20:44 |
| jroll | yeah, I think I'd prefer the same, just needs some thought | 20:47 |
| *** openstackgerrit has quit IRC | 20:48 | |
| *** openstackgerrit has joined #openstack-ironic | 20:48 | |
| *** piet has joined #openstack-ironic | 20:49 | |
| *** achanda has joined #openstack-ironic | 20:56 | |
| *** achanda has quit IRC | 20:57 | |
| *** achanda has joined #openstack-ironic | 20:58 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Don't power off non-deploying iLO nodes in takeover https://review.openstack.org/299573 | 20:59 |
| *** Sukhdev has joined #openstack-ironic | 20:59 | |
| *** raildo is now known as raildo-afk | 21:08 | |
| *** dprince has quit IRC | 21:09 | |
| *** clenimar has quit IRC | 21:16 | |
| *** Sukhdev has quit IRC | 21:29 | |
| *** rbudden has quit IRC | 21:30 | |
| *** achanda has quit IRC | 21:32 | |
| *** jjohnson2 has quit IRC | 21:37 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack/ironic-specs: Dynamic allocation of nodes on OneView drivers https://review.openstack.org/275726 | 21:44 |
| thiagop | jroll JayF: ^^ | 21:44 |
| thiagop | Already with periodic tasks approach | 21:44 |
| jroll | cool, thanks thiagop | 21:45 |
| *** piet has quit IRC | 21:45 | |
| JayF | will have a look, thanks | 21:46 |
| *** davidlenwell has quit IRC | 21:48 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Enable Grenade usage as a plugin https://review.openstack.org/298967 | 21:49 |
| *** Sukhdev has joined #openstack-ironic | 21:50 | |
| thiagop | Well, I'm calling it a day (and a Sprint). See you tomorrow! | 21:51 |
| NobodyCam | night thiagop | 21:53 |
| *** thiagop has quit IRC | 21:53 | |
| *** zer0c00l has quit IRC | 21:54 | |
| *** zer0c00l has joined #openstack-ironic | 21:54 | |
| *** pece has quit IRC | 21:55 | |
| *** davidlenwell has joined #openstack-ironic | 21:55 | |
| * jroll also out for the night, moving things to do | 21:55 | |
| JayF | heh, he's going to have a book waiting for him on that spec when he gets in tomorrow :) | 21:55 |
| jroll | see y'all | 21:55 |
| NobodyCam | have a good night jroll | 21:55 |
| NobodyCam | moving? | 21:55 |
| jroll | yes sir | 21:55 |
| NobodyCam | wow | 21:55 |
| * JayF moving in summer, too | 21:56 | |
| jroll | the real fun starts next week, I'll let y'all know more by monday | 21:56 |
| NobodyCam | :) | 21:56 |
| *** [1]cdearborn has quit IRC | 21:57 | |
| *** piet has joined #openstack-ironic | 21:58 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Active Node Creation via adopt state https://review.openstack.org/275766 | 22:03 |
| TheJulia | Moving is soo not fun. :( | 22:07 |
| *** jaybeale_ has quit IRC | 22:19 | |
| *** vishwanathj has quit IRC | 22:27 | |
| *** mbound has quit IRC | 22:28 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack/ironic-specs: Dynamic allocation of nodes on OneView drivers https://review.openstack.org/275726 | 22:31 |
| *** ametts has quit IRC | 22:57 | |
| TheJulia | okay, I think my brain is telling me it is time to call it a day | 22:59 |
| *** piet has quit IRC | 22:59 | |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack/bifrost: Add playbook to transfer nodes to another Ironic https://review.openstack.org/299660 | 23:11 |
| SpamapS | TheJulia: ^ | 23:27 |
| SpamapS | TheJulia: I'm not entirely certain about the 'transfer_names' thing... | 23:27 |
| SpamapS | TheJulia: might be better to just tack on a second static inventory that puts the names you want transferred into an ansible group.. but this allows me to do some wacky things. ;) | 23:27 |
| TheJulia | SpamapS: no active nodes I take it? :) | 23:28 |
| SpamapS | TheJulia: right, filter out active.. filter based on some variables. | 23:29 |
| TheJulia | well, one day active could be supported *grin* | 23:29 |
| TheJulia | although, I need to work on the code in shade to do that | 23:29 |
| SpamapS | I'm also not certain this even works | 23:29 |
| devananda | TheJulia: I don't htink active nodes would fit this use case | 23:29 |
| devananda | but I can imagine other ones it would | 23:30 |
| SpamapS | for my current use case, I actually want this to be the list: | 23:30 |
| devananda | so being able to filter is good | 23:30 |
| SpamapS | BMCS|selectattr('group', 'equalto', 'compute-ironic')|map(attribute='name') | list | 23:30 |
| TheJulia | filtering ++ | 23:30 |
| SpamapS | BMCS being a list of dicts that have name and group in it. | 23:30 |
| devananda | this reminds me, I need to push up my changes to set up isc-dhcpd to servie IP addreses to BMCS | 23:31 |
| SpamapS | so I'm hoping I can actually just write my own playbook that calls the role, and sets transfer_names: {{ ^^ that ungodly mess }} ;) | 23:31 |
| devananda | maybe I'll do that now while my envs rebuild :) | 23:31 |
| SpamapS | While I have your attention, can I delete nodes that are in maintenance mode? | 23:31 |
| TheJulia | you can | 23:32 |
| SpamapS | kk, good | 23:32 |
| TheJulia | well, node-delete, set-node-provision-state I'm unsure about off the top of my head | 23:32 |
| SpamapS | I need to node-delete | 23:33 |
| SpamapS | basically I want to tell bifrost's ironic to be hands-off, and then delete the node after I've successfully added it to the other ironic. | 23:33 |
| devananda | put a node in maintenance mode, and you can "node-delete" regardless of anything else | 23:33 |
| devananda | IIRC, there was a bug in kilo and it didn't work back then | 23:34 |
| devananda | but it has for a while now | 23:34 |
| SpamapS | I think I hit that bug, which is why I asked. :) | 23:34 |
| TheJulia | SpamapS: awesome | 23:34 |
| devananda | SpamapS: your role might also want to do some validation on the receiving ironic end | 23:34 |
| TheJulia | SpamapS: works as of last night on master :) | 23:34 |
| SpamapS | wonderful :) | 23:34 |
| SpamapS | devananda: I'll take any suggestion you have for that. | 23:35 |
| devananda | SpamapS: eg, add the node with initial state of "enroll", then call "node-set-provision-state manage" -- this causes ironic to validate the driver's ability to manage the node | 23:35 |
| devananda | if the driver can't, eg, reach the IPMI, it will not reach the manageable state, and record the error | 23:35 |
| SpamapS | Ahh, that sounds like a good idea. | 23:35 |
| devananda | at which point you might want to NOT delete it from bifrost :) | 23:36 |
| SpamapS | will os_ironic block on that? | 23:36 |
| devananda | I do not know | 23:36 |
| SpamapS | kk | 23:36 |
| devananda | pretty sure set-prov-state calls are all async, even in shade | 23:36 |
| SpamapS | devananda: it also strikes me that we actually need to have a way to prevent it from coming back the next time we enroll things in bifrost. | 23:36 |
| devananda | so you may need to set an async or wait/poll thing in ansible | 23:36 |
| devananda | SpamapS: ooooh. for our cases, youre right | 23:37 |
| SpamapS | But I think that's more our problem | 23:37 |
| devananda | yah | 23:37 |
| SpamapS | in that we just need to set a list of groups that shouldn't be enrolled | 23:37 |
| SpamapS | and we can basically just version that group name | 23:37 |
| TheJulia | SpamapS: set wait on os_ironic and I think that will work for you, but you'll want to check the status afterwards of course | 23:37 |
| SpamapS | compute-ironic2 for the next set | 23:37 |
| SpamapS | that way we can still enroll/inspect them | 23:37 |
| TheJulia | SpamapS: note, wait support is in devel, not in stable-2.0 for ansible | 23:38 |
| devananda | SpamapS: sounds reasonble. batch the hardware imports, as it were | 23:38 |
| SpamapS | TheJulia: luckily, we have decided to pull master for -core :-P | 23:38 |
| TheJulia | SpamapS: living dangerously :) | 23:38 |
| SpamapS | devananda: yeah, it also means we can probably trigger transfers with a single commit. | 23:38 |
| devananda | SpamapS: ++ | 23:38 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Improve __str__ method of BaseCommandResult https://review.openstack.org/299560 | 23:38 |
| SpamapS | TheJulia: now you see the violence inherent in the system | 23:38 |
| TheJulia | SpamapS: ahh... yes.... | 23:39 |
| *** jjohnson2 has joined #openstack-ironic | 23:40 | |
| *** smoriya_ has joined #openstack-ironic | 23:40 | |
| TheJulia | SpamapS: going to appologize now on the shade stuff, if you need a set of eyes, feel free to ping me, I've just not been able to spend much time on it the past six months | 23:40 |
| SpamapS | TheJulia: Oh hey, I haven't had any upstream time the last 3. It's just the way things are right now. ;) | 23:40 |
| TheJulia | :) | 23:41 |
| TheJulia | I think we all get that since we all go through that from time to time :) | 23:41 |
| TheJulia | "you need ?what? deliverd... that uses jelly beans to count?!?" | 23:42 |
| *** achanda has joined #openstack-ironic | 23:45 | |
| * TheJulia thought it was funny :) | 23:47 | |
| SpamapS | I tittered | 23:48 |
| SpamapS | but I didn't lol | 23:48 |
| SpamapS | so :-P | 23:48 |
| SpamapS | but not :-| | 23:48 |
| TheJulia | okay, I was slightly worried there :) | 23:49 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!