*** suro-patz has quit IRC | 00:04 | |
*** garthb_ has quit IRC | 00:09 | |
*** mtanino has quit IRC | 00:21 | |
*** smoriya_ has joined #openstack-ironic | 00:25 | |
*** gmmaha has quit IRC | 00:25 | |
*** gmmaha has joined #openstack-ironic | 00:27 | |
jroll | stable IPA stuff landed, devstack changes still under test, will resume tomorrow | 00:31 |
---|---|---|
jroll | night y'all | 00:31 |
*** thrash is now known as thrash|g0ne | 00:46 | |
*** ChrisAusten has joined #openstack-ironic | 00:48 | |
*** hoangcx has joined #openstack-ironic | 00:51 | |
*** jaybeale has quit IRC | 00:55 | |
openstackgerrit | Merged openstack/ironic-python-agent: Add docstrings to all functions in Agent standby extension https://review.openstack.org/288632 | 00:57 |
*** baoli has joined #openstack-ironic | 01:00 | |
*** spandhe has quit IRC | 01:20 | |
*** mtanino has joined #openstack-ironic | 01:28 | |
clif_h | woot | 01:30 |
*** krtaylor has joined #openstack-ironic | 01:35 | |
*** suro-patz has joined #openstack-ironic | 01:39 | |
*** rbudden has joined #openstack-ironic | 01:45 | |
*** suro-patz has quit IRC | 01:53 | |
*** baoli has quit IRC | 01:58 | |
*** baoli has joined #openstack-ironic | 01:58 | |
*** Sukhdev has quit IRC | 01:59 | |
*** achanda has joined #openstack-ironic | 02:14 | |
*** Nisha has joined #openstack-ironic | 02:15 | |
*** baoli has quit IRC | 02:25 | |
*** rbudden has quit IRC | 02:34 | |
*** baoli has joined #openstack-ironic | 02:42 | |
*** sivaramakrishna has joined #openstack-ironic | 02:42 | |
*** baoli has quit IRC | 02:45 | |
*** yuanying has quit IRC | 02:51 | |
*** Nisha_away has joined #openstack-ironic | 02:59 | |
*** Nisha has quit IRC | 03:01 | |
*** jjohnson2 has joined #openstack-ironic | 03:09 | |
*** krotscheck is now known as krotscheck_vaca | 03:20 | |
openstackgerrit | Tan Lin proposed openstack/ironic: Add nodeinfo_list() to Node Object https://review.openstack.org/290313 | 03:24 |
*** spandhe has joined #openstack-ironic | 03:30 | |
*** spandhe_ has joined #openstack-ironic | 03:33 | |
*** spandhe has quit IRC | 03:34 | |
*** spandhe_ is now known as spandhe | 03:34 | |
*** harshs has joined #openstack-ironic | 03:38 | |
*** links has joined #openstack-ironic | 03:41 | |
*** yuanying has joined #openstack-ironic | 03:49 | |
*** jjohnson2 has quit IRC | 03:52 | |
*** achanda has quit IRC | 04:04 | |
*** alexpilotti has quit IRC | 04:06 | |
*** achanda has joined #openstack-ironic | 04:08 | |
*** achanda has quit IRC | 04:13 | |
*** alexpilotti has joined #openstack-ironic | 04:17 | |
*** harlowja_at_home has joined #openstack-ironic | 04:19 | |
*** alexpilotti has quit IRC | 04:21 | |
*** Sukhdev has joined #openstack-ironic | 04:25 | |
*** ndipanov has quit IRC | 04:29 | |
*** suro-patz has joined #openstack-ironic | 04:30 | |
*** vmud213 has joined #openstack-ironic | 04:34 | |
*** mtreinish has quit IRC | 04:44 | |
*** suro-patz has quit IRC | 04:45 | |
*** pradipta has joined #openstack-ironic | 04:49 | |
*** pradipta has quit IRC | 04:50 | |
*** hoangcx has quit IRC | 04:54 | |
*** rcernin has joined #openstack-ironic | 04:57 | |
*** alexpilotti has joined #openstack-ironic | 04:59 | |
*** achanda has joined #openstack-ironic | 05:02 | |
*** stevemar_ has quit IRC | 05:02 | |
*** stevemar has joined #openstack-ironic | 05:02 | |
*** alexpilotti has quit IRC | 05:03 | |
*** achanda has quit IRC | 05:10 | |
*** vinaym213 has joined #openstack-ironic | 05:13 | |
*** vmud213 has quit IRC | 05:16 | |
*** alexpilotti has joined #openstack-ironic | 05:17 | |
*** harlowja_at_home has quit IRC | 05:19 | |
*** alexpilotti has quit IRC | 05:24 | |
*** davideag_ has joined #openstack-ironic | 05:24 | |
*** ChrisAusten has quit IRC | 05:25 | |
*** davideagnello has quit IRC | 05:26 | |
*** suro-patz has joined #openstack-ironic | 05:31 | |
*** yolanda has quit IRC | 05:39 | |
*** davideag_ has quit IRC | 05:42 | |
*** ChubYann has quit IRC | 05:43 | |
*** harshs_ has joined #openstack-ironic | 05:49 | |
*** harshs has quit IRC | 05:51 | |
*** mtreinish has joined #openstack-ironic | 05:52 | |
*** ifarkas has quit IRC | 05:54 | |
*** harshs_ has quit IRC | 05:55 | |
*** harshs has joined #openstack-ironic | 05:58 | |
*** hoangcx has joined #openstack-ironic | 06:03 | |
*** harshs has quit IRC | 06:03 | |
*** spandhe has quit IRC | 06:04 | |
*** Marga_ has quit IRC | 06:06 | |
*** spandhe has joined #openstack-ironic | 06:10 | |
*** Nisha_away has quit IRC | 06:12 | |
*** yolanda has joined #openstack-ironic | 06:15 | |
*** spandhe_ has joined #openstack-ironic | 06:16 | |
*** spandhe has quit IRC | 06:17 | |
*** spandhe_ is now known as spandhe | 06:17 | |
*** harshs has joined #openstack-ironic | 06:17 | |
*** spandhe_ has joined #openstack-ironic | 06:21 | |
*** spandhe has quit IRC | 06:21 | |
*** spandhe_ is now known as spandhe | 06:21 | |
*** suro-patz has quit IRC | 06:22 | |
*** Sukhdev has quit IRC | 06:24 | |
*** Nisha_away has joined #openstack-ironic | 06:25 | |
*** Marga_ has joined #openstack-ironic | 06:25 | |
*** Marga_ has quit IRC | 06:29 | |
*** achanda has joined #openstack-ironic | 06:30 | |
*** Marga_ has joined #openstack-ironic | 06:38 | |
*** yushiro has joined #openstack-ironic | 06:39 | |
yushiro | Hi all, | 06:41 |
*** Marga_ has quit IRC | 06:43 | |
yushiro | I'd like to know about cleaning for the ironic node with agent_xxxx driver. | 06:44 |
yushiro | When timeout(clean_callback_timeout in ironic.conf) has occurred for cleaning, provision state has changed into "clean failed". | 06:48 |
yushiro | However, deploy image keeps on running and verifying heartbeat forever. | 06:49 |
yushiro | I think it should be executed power off and unmounted some virtual media. | 06:53 |
*** alexpilotti has joined #openstack-ironic | 06:56 | |
*** ohamada has joined #openstack-ironic | 06:56 | |
openstackgerrit | Tan Lin proposed openstack/ironic: Adopt Ironic's own context https://review.openstack.org/295663 | 06:57 |
yushiro | I'd like to post the bug-report but I'm not sure it's a bug or not. Therefore, would you please tell me your opinion? | 06:57 |
*** alexpilotti has quit IRC | 07:00 | |
*** spandhe has quit IRC | 07:04 | |
*** e0ne has joined #openstack-ironic | 07:07 | |
*** mgoddard has joined #openstack-ironic | 07:14 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Tempest plugin initial commit https://review.openstack.org/293925 | 07:17 |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: [WIP] Tempest: add basic test https://review.openstack.org/293926 | 07:17 |
*** arturas has joined #openstack-ironic | 07:19 | |
*** pcaruana has joined #openstack-ironic | 07:25 | |
*** harshs has quit IRC | 07:25 | |
*** e0ne has quit IRC | 07:35 | |
*** e0ne has joined #openstack-ironic | 07:35 | |
*** dtantsur|afk is now known as dtantsur | 07:37 | |
dtantsur | Morning Ironic | 07:37 |
*** Nisha_away has quit IRC | 07:38 | |
*** jcoufal has joined #openstack-ironic | 07:39 | |
*** e0ne has quit IRC | 07:40 | |
*** e0ne has joined #openstack-ironic | 07:45 | |
*** amotoki has joined #openstack-ironic | 07:52 | |
*** e0ne has quit IRC | 07:54 | |
*** Nisha has joined #openstack-ironic | 07:55 | |
*** tesseract has joined #openstack-ironic | 07:56 | |
*** tesseract is now known as Guest90877 | 07:56 | |
*** e0ne has joined #openstack-ironic | 07:57 | |
*** dtantsur is now known as dtantsur|bbl | 07:59 | |
*** ifarkas has joined #openstack-ironic | 08:02 | |
*** yolanda has quit IRC | 08:04 | |
*** ndipanov has joined #openstack-ironic | 08:06 | |
*** jistr has joined #openstack-ironic | 08:07 | |
*** e0ne has quit IRC | 08:10 | |
*** mbound has quit IRC | 08:12 | |
*** e0ne has joined #openstack-ironic | 08:13 | |
aarefiev | morning! | 08:15 |
*** baoli has joined #openstack-ironic | 08:16 | |
*** e0ne has quit IRC | 08:16 | |
*** mkovacik has quit IRC | 08:17 | |
*** alexpilotti has joined #openstack-ironic | 08:17 | |
*** e0ne has joined #openstack-ironic | 08:20 | |
*** baoli has quit IRC | 08:20 | |
*** alexpilotti has quit IRC | 08:22 | |
*** achanda has quit IRC | 08:22 | |
*** achanda has joined #openstack-ironic | 08:25 | |
*** e0ne has quit IRC | 08:26 | |
*** e0ne has joined #openstack-ironic | 08:29 | |
*** e0ne_ has joined #openstack-ironic | 08:31 | |
*** kromanenko has joined #openstack-ironic | 08:32 | |
*** kromanenko_ has joined #openstack-ironic | 08:32 | |
*** e0ne has quit IRC | 08:32 | |
*** achanda has quit IRC | 08:32 | |
*** e0ne_ has quit IRC | 08:33 | |
*** achanda has joined #openstack-ironic | 08:38 | |
*** alexpilotti has joined #openstack-ironic | 08:40 | |
*** alexpilotti has quit IRC | 08:45 | |
*** mbound has joined #openstack-ironic | 08:45 | |
*** Nisha has quit IRC | 08:46 | |
*** e0ne has joined #openstack-ironic | 08:48 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 08:49 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Allow to build user image with DIB https://review.openstack.org/256363 | 08:49 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add Link-Local-Connection info to ironic port https://review.openstack.org/256365 | 08:49 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Added operator documentation for ironic portgroups https://review.openstack.org/228496 | 08:49 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/206244 | 08:49 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: refactor ironic enroll-node code https://review.openstack.org/256364 | 08:49 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add configure_provision_network function https://review.openstack.org/256367 | 08:49 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Update Ironic VM network connection https://review.openstack.org/256366 | 08:49 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 08:49 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 08:49 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add network drivers https://review.openstack.org/285852 | 08:49 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add Ironic/Neutron integration documentation https://review.openstack.org/258596 | 08:49 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Allow to use NETWORK_DRIVERS in devstack https://review.openstack.org/293520 | 08:49 |
*** amit213 has quit IRC | 08:49 | |
*** thrash|g0ne has quit IRC | 08:51 | |
*** thrash has joined #openstack-ironic | 08:51 | |
*** thrash has quit IRC | 08:51 | |
*** thrash has joined #openstack-ironic | 08:51 | |
*** e0ne has quit IRC | 08:53 | |
*** yushiro has quit IRC | 08:54 | |
*** achanda has quit IRC | 08:57 | |
openstackgerrit | Sofiia Andriichenko proposed openstack/python-ironicclient: Add test for 'node-show --field' command https://review.openstack.org/300064 | 08:58 |
*** e0ne has joined #openstack-ironic | 08:58 | |
*** alexpilotti has joined #openstack-ironic | 09:02 | |
*** alexpilotti has quit IRC | 09:02 | |
*** mkovacik has joined #openstack-ironic | 09:04 | |
vdrok | morning ironic, dtantsur|bbl, aarefiev | 09:05 |
aarefiev | vdrok: morning | 09:05 |
vinaym213 | morning vdrok,aarefiev,dtantsur and other ironicers not listed here | 09:06 |
*** kbyrne has quit IRC | 09:06 | |
aarefiev | vinaym213 morning | 09:07 |
*** kbyrne has joined #openstack-ironic | 09:11 | |
mkovacik | morning all! | 09:13 |
mkovacik | dtantsur|bbl, ifarkas, sambetts|afk aarefiev, I've pushed new revision of https://review.openstack.org/#/c/267637/ based on yesterday's discussion, please, take a look when you've some time. Thanks! | 09:16 |
*** chihhsin has joined #openstack-ironic | 09:16 | |
ifarkas | mkovacik, sure, will take a look | 09:17 |
*** alexpilotti has joined #openstack-ironic | 09:18 | |
vdrok | morning vinaym213 mkovacik and ifarkas | 09:19 |
mkovacik | ifarkas, thanks! | 09:19 |
mkovacik | vdrok, o/ | 09:19 |
*** arturas has quit IRC | 09:23 | |
*** alexpilotti has quit IRC | 09:23 | |
*** amotoki has quit IRC | 09:27 | |
*** alexpilotti has joined #openstack-ironic | 09:33 | |
*** e0ne has quit IRC | 09:36 | |
*** amotoki has joined #openstack-ironic | 09:42 | |
*** alexpilotti has quit IRC | 09:42 | |
*** jcoufal has quit IRC | 09:42 | |
*** amotoki has quit IRC | 09:44 | |
*** amotoki has joined #openstack-ironic | 09:46 | |
*** arturas has joined #openstack-ironic | 09:46 | |
*** e0ne has joined #openstack-ironic | 09:47 | |
*** mgould has joined #openstack-ironic | 09:50 | |
*** amotoki has quit IRC | 09:51 | |
*** amotoki has joined #openstack-ironic | 09:56 | |
*** arturas has quit IRC | 09:56 | |
*** amotoki has quit IRC | 09:56 | |
mgould | morning Ironicers! | 09:57 |
*** daemontool has joined #openstack-ironic | 09:57 | |
*** achanda has joined #openstack-ironic | 09:58 | |
*** amotoki has joined #openstack-ironic | 09:59 | |
*** achanda has quit IRC | 10:02 | |
*** e0ne has quit IRC | 10:08 | |
*** e0ne has joined #openstack-ironic | 10:12 | |
*** amotoki has quit IRC | 10:16 | |
*** hoangcx has quit IRC | 10:16 | |
*** daemontool_ has joined #openstack-ironic | 10:24 | |
*** jcoufal has joined #openstack-ironic | 10:25 | |
*** ohamada_ has joined #openstack-ironic | 10:27 | |
*** ohamada has quit IRC | 10:27 | |
*** daemontool has quit IRC | 10:27 | |
*** alaski is now known as lascii | 10:38 | |
openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command https://review.openstack.org/251458 | 10:39 |
*** alexpilotti has joined #openstack-ironic | 10:40 | |
*** dtantsur|bbl is now known as dtantsur | 10:40 | |
dtantsur | mkovacik, with pleasure. only one note: due to suddenly worsen weather, I'll only put -2 on reviews today. | 10:41 |
dtantsur | thanks for your understanding | 10:41 |
*** alexpilotti has quit IRC | 10:44 | |
*** amotoki has joined #openstack-ironic | 10:45 | |
*** e0ne has quit IRC | 10:47 | |
*** sivaramakrishna has quit IRC | 10:51 | |
*** smoriya_ has quit IRC | 10:51 | |
*** e0ne has joined #openstack-ironic | 10:52 | |
sambetts|afk | Morning all | 10:58 |
*** sambetts|afk is now known as sambetts | 10:59 | |
openstackgerrit | Karim Boumedhel proposed openstack/ironic-specs: Adds a Blueprint for ovirt support in Ironic https://review.openstack.org/300410 | 10:59 |
*** trown|outtypewww is now known as trown | 10:59 | |
openstackgerrit | Karim Boumedhel proposed openstack/ironic-specs: Add cloudinit support to ovirt.py module https://review.openstack.org/300411 | 11:00 |
*** e0ne has quit IRC | 11:04 | |
mkovacik | dtantsur, yeah, but since you're staying in bed the whole day could be just -1 ;) | 11:04 |
dtantsur | :) | 11:06 |
*** Marga_ has joined #openstack-ironic | 11:07 | |
openstackgerrit | Karim Boumedhel proposed openstack/ironic-specs: Adds a Blueprint for ovirt support in Ironic https://review.openstack.org/300418 | 11:07 |
*** e0ne has joined #openstack-ironic | 11:08 | |
*** e0ne has quit IRC | 11:11 | |
*** Marga_ has quit IRC | 11:11 | |
*** baoli has joined #openstack-ironic | 11:12 | |
jroll | gooood morning, happy friday | 11:14 |
dtantsur | morning jroll | 11:15 |
jroll | hey dtantsur | 11:15 |
jroll | friendly reminder not to believe anything you read today | 11:15 |
jroll | I see neutron folks are being properly trolled on the ML | 11:16 |
mkovacik | jroll, o/ | 11:16 |
*** baoli has quit IRC | 11:16 | |
dtantsur | yep :) | 11:25 |
dtantsur | I forgot to put a spec to turn Ironic into a proper CMDB.. again.. | 11:25 |
dtantsur | :) | 11:25 |
jroll | heh | 11:26 |
trown | https://review.openstack.org/#/c/300272/ is my favorite | 11:26 |
jroll | I'm going to build a PTLaaS one year | 11:26 |
jroll | trown: idk, I think http://lists.openstack.org/pipermail/openstack-dev/2016-April/091120.html | 11:27 |
jroll | oh my, https://review.openstack.org/#/c/300274/ | 11:28 |
trown | hehe | 11:29 |
trown | should post a follow up adding Ironic back in to... could be Notronic | 11:30 |
dtantsur | LOL | 11:30 |
*** achanda has joined #openstack-ironic | 11:30 | |
*** e0ne has joined #openstack-ironic | 11:33 | |
jroll | omg yes | 11:33 |
* jroll does it | 11:33 | |
trown | :) | 11:33 |
*** amotoki has quit IRC | 11:34 | |
dtantsur | sambetts, please see if your concerns are resolved on the reapply patch | 11:35 |
*** achanda has quit IRC | 11:35 | |
sambetts | dtantsur: sure :) | 11:35 |
jroll | https://review.openstack.org/#/c/300428/ | 11:37 |
*** daemontool_ is now known as daemontool | 11:37 | |
dtantsur | +1 | 11:40 |
*** alexpilotti has joined #openstack-ironic | 11:40 | |
*** baoli has joined #openstack-ironic | 11:42 | |
*** vinay50muddu has joined #openstack-ironic | 11:43 | |
*** vinay50muddu has quit IRC | 11:43 | |
mkovacik | dtantsur, thanks for the sprinkling of my patch with the Holy Ironic Bear pee ;) | 11:44 |
*** vmud213 has joined #openstack-ironic | 11:44 | |
dtantsur | lol wut?? :D | 11:44 |
jroll | O_o | 11:45 |
dtantsur | man, we should rename our +W gerrit text to "Approved by the Holy Ironic Bear pee" | 11:45 |
*** alexpilotti has quit IRC | 11:45 | |
*** baoli_ has joined #openstack-ironic | 11:45 | |
jroll | lol | 11:45 |
*** vinaym213 has quit IRC | 11:47 | |
*** baoli has quit IRC | 11:48 | |
*** e0ne has quit IRC | 11:50 | |
sambetts | dtantsur, mkovacik: left a comment on that patch, but everything else looks good to me | 11:50 |
*** Goneri has quit IRC | 11:50 | |
jroll | welp. | 11:50 |
jroll | $ find . -type f -exec sed -i -e 's/ironic/neutronic/g' {} \; | 11:50 |
jroll | $ git st | 11:50 |
jroll | error: bad index file sha1 signature | 11:50 |
jroll | fatal: index file corrupt | 11:50 |
jroll | mistakes were made. | 11:50 |
sambetts | ohhhh god ... . | 11:51 |
TheJulia | good morning | 11:51 |
sambetts | o/ TheJulia | 11:51 |
dtantsur | oh yeah, "find ." was not the best idea, I'm afraid | 11:51 |
dtantsur | morning TheJulia | 11:51 |
jroll | yeah :D | 11:51 |
jroll | that's one way to clean local branches out | 11:51 |
dtantsur | also imagine how many things it screwed in .tox :) | 11:51 |
sambetts | jroll: http://logs.openstack.org/29/299529/4/gate/gate-tempest-dsvm-ironic-agent_ssh-src/32a27ac/ I hope this isn't an april fool | 11:52 |
dtantsur | sambetts, tripleo CI is/was widely affected by this | 11:52 |
jroll | sambetts: nope, that was a thing yesterday | 11:52 |
dtantsur | but I hope it got fixed this night | 11:52 |
sambetts | Last 2 gate runs blew up here because it this morning https://review.openstack.org/#/c/299529/ | 11:53 |
dtantsur | folks, please check my memory: we try to NOT do network requests in the driver validate() call, right? | 11:53 |
dtantsur | jroll, ^^ | 11:53 |
jroll | dtantsur: I believe so as that's synchronous | 11:53 |
TheJulia | dtantsur: re renaming +W.... omg yes | 11:53 |
openstackgerrit | Debayan Ray proposed openstack/ironic: Fix NamedTemporaryFile() OSError Exception https://review.openstack.org/293762 | 11:53 |
sambetts | dtantsur: its a sync method so I believe so | 11:53 |
dtantsur | :) | 11:53 |
*** e0ne has joined #openstack-ironic | 11:54 | |
sambetts | dtantsur: thats why I prompted the OneView guys to make their thing a periodic, and have the checks they want in validate to just be checking node properties | 11:54 |
sambetts | dtantsur: originally they were going to call out to OneView in the validate function and I was like probably not a good idea :-p | 11:55 |
dtantsur | yeah. I'm reviewing the submission of the libvirt driver to ironic-staging-drivers, and they do the same | 11:55 |
*** e0ne has quit IRC | 11:57 | |
*** dprince has joined #openstack-ironic | 11:57 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Rename ironic to neutronic https://review.openstack.org/300434 | 11:58 |
jroll | wheeeee | 11:58 |
sambetts | wow thats a huge patch :-P | 11:59 |
* sambetts things we should replace everywhere in the code that says Ironic with a {project-name} indicator so that we can rename the whole project on the fly by just changing a variable | 12:00 | |
sambetts | s/things/thinks | 12:00 |
jroll | heh | 12:01 |
trown | conflicts with 115 | 12:02 |
jroll | dtantsur: btw, https://review.openstack.org/#/c/300146/ | 12:02 |
jroll | (this is not a joke patch) | 12:03 |
trown | sambetts: that traceroute issue was infra, and should be fixed | 12:03 |
*** alexpilotti has joined #openstack-ironic | 12:03 | |
sambetts | dtantsur: my concern is that if someone didn't pay attention to the api docs and sent a body to the POST endpoint, right now it wouldn't care and it would just do a reapply, but in the future if we start doing things with the body then it would change the functionality of their call | 12:03 |
sambetts | trown: yeah, it must be taking some time for the images to propergate or something | 12:03 |
*** jjohnson2 has joined #openstack-ironic | 12:06 | |
dtantsur | sambetts, I'm not against starting such checks, just pointing our that we never did it before | 12:07 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Make tox respect upper-constraints.txt https://review.openstack.org/300146 | 12:07 |
dtantsur | mkovacik, ^^ | 12:07 |
jroll | dtantsur: good points, fixed | 12:07 |
dtantsur | +@ | 12:08 |
dtantsur | +2 | 12:08 |
sambetts | dtantsur: Does it fit into any of our other API endpoints? | 12:08 |
dtantsur | sambetts, you mean, such check? well, we could check absence of additional arguments in all other endpoints.. | 12:09 |
dtantsur | jroll, and then we have to backport the same thing to liberty and mitaka stable? | 12:09 |
*** e0ne has joined #openstack-ironic | 12:09 | |
sambetts | dtantsur: Oh yeah I guess :/ | 12:10 |
jroll | dtantsur: yeah, probably should | 12:10 |
jroll | also. my branch-based ipa builds aren't working >.> | 12:11 |
dtantsur | sambetts, well, anyway, since we plan on using body later on, we should probably check it for now. mkovacik, objections? | 12:11 |
jroll | http://logs.openstack.org/23/237b76e0d532ff5c6bf45fefc89c81d8f89a3d49/post/ironic-python-agent-buildimage-coreos/8a37d7f/console.html#_2016-04-01_00_24_55_784 | 12:11 |
dtantsur | jroll, ouch :( | 12:11 |
jroll | ಠ_ಠ| 12:11 |
jroll | is it because I didn't export BRANCH_PATH, I assume? | 12:11 |
dtantsur | probably | 12:11 |
jroll | grumble | 12:12 |
dtantsur | now to something completely different. jroll, do you have any ideas about https://bugzilla.redhat.com/show_bug.cgi?id=1317731 ? (looks like the bug is public, so you should be able to see it) | 12:12 |
openstack | bugzilla.redhat.com bug 1317731 in openstack-ironic-python-agent "[RFE]: IPA needs to support Multipathed SAN LUNs for root disk" [High,New] - Assigned to dtantsur | 12:12 |
dtantsur | dunno what it implies for upstream, maybe you've heard about someone doing that? | 12:13 |
jroll | https://review.openstack.org/300441 | 12:14 |
* TheJulia takes a look at the ipa bug since that has been a bit of a major issue in testing | 12:15 | |
jroll | dtantsur: so they want boot from volume, essentially :) | 12:15 |
jroll | dtantsur: though, I suspect we could be missing something in the image to detect that, and/or they're missing a root device hint | 12:16 |
dtantsur | yeah, but some simple variant, as I understood it.. i.e. assume the disk is already magically connected | 12:16 |
jroll | right | 12:16 |
dtantsur | jroll, missing root device hint is my current guess | 12:16 |
TheJulia | jroll: nah, its not boot from volume, it is missing root device hint | 12:16 |
jroll | ya | 12:16 |
jroll | TheJulia: they did mention iscsi :) | 12:16 |
TheJulia | true | 12:16 |
TheJulia | same thing, path deduplication | 12:17 |
jroll | yeah | 12:17 |
dtantsur | we do miss a couple of things on an image as well | 12:17 |
TheJulia | bottom line it could be a feature request although really we just need to get mpath tools running | 12:17 |
TheJulia | bad side of that is... san venders have variety in the proper way to identify alternate paths | 12:17 |
* TheJulia twitches has flashbacks | 12:18 | |
jroll | heh | 12:18 |
TheJulia | so tooling to pass in a specific multipath config file would be needed in dib | 12:18 |
TheJulia | does coreos even support multipathing? | 12:18 |
jroll | doubt it, they don't even support mdraid | 12:19 |
TheJulia | i remember searching their docs for the word a few months ago and got 0 results | 12:19 |
dtantsur | oh hmm, number of proposed virt drivers increase: https://bugs.launchpad.net/ironic/+bug/1564841 | 12:20 |
openstack | Launchpad bug 1564841 in Ironic "[RFE] Add a ovirt driver to Ironic" [Undecided,Incomplete] - Assigned to Karim Boumedhel (karmab) | 12:20 |
jroll | why do people want all these virt drivers. | 12:21 |
sambetts | jroll: why can't they use nova? | 12:21 |
sambetts | surely thats the question :-P | 12:21 |
jroll | heh | 12:21 |
dtantsur | I suspect that's for testing on something weird (again) | 12:22 |
jroll | it is, it calls out testing tripleo | 12:22 |
dtantsur | I remember a guy trying to run tripleo on ovirt nodes | 12:22 |
dtantsur | didn't quite work :) | 12:22 |
* jroll is reminded to do an rfe sweep today | 12:22 | |
* dtantsur is going through bugs as well | 12:23 | |
dtantsur | so we can sync our efforts/have quick discussions, etc | 12:23 |
jroll | dtantsur: if you want to triage rfes to just unconfirmed, I can take them from there | 12:23 |
TheJulia | dtantsur: w/r/t multipath, I kind of think it could be a simple addition to IPA to test for/handle, and try to identified the path that is not locked, or to assert a device lock on a particular path, which would kind of allow it to work in mpath environments without trully consolidating the path down to a single device where the OS is magically doing MPIO under the hood.... but in order to devel | 12:28 |
TheJulia | op that I think one of is is goign to need access to a san system that is either very buggy or very mean with lock management | 12:28 |
*** wuhg has quit IRC | 12:29 | |
TheJulia | what we've seen is one controller rejects IO on that path because persumably the lun has a lock on the other path, and saldy ipa has chosen the wrong path to try and write the disk out to | 12:30 |
dtantsur | jroll, re https://bugs.launchpad.net/python-ironicclient/+bug/1564484 I don't think we should treat writing tests as rfe | 12:30 |
openstack | Launchpad bug 1564484 in python-ironicclient "[RFE] Write tests tests for testing node-delete command for Ironic Client " [Undecided,In progress] - Assigned to Sergii Turivnyi (sturivnyi) | 12:30 |
jroll | geez, https://review.openstack.org/#/c/297479/ finally passed CI | 12:30 |
jroll | dtantsur: +1 | 12:30 |
dtantsur | TheJulia, sounds complex :( | 12:30 |
TheJulia | dtantsur: very :( | 12:30 |
jroll | forgot about client rfes :/ | 12:31 |
*** achanda has joined #openstack-ironic | 12:32 | |
jroll | dtantsur: you clearing these test RFEs out or shall I? | 12:34 |
* jroll jfdi | 12:34 | |
dtantsur | jroll, go ahead, I got dragged into another discussion | 12:35 |
jroll | yeah no worries | 12:35 |
jroll | dtantsur: any problem with me approving https://bugs.launchpad.net/ironic-python-agent/+bug/1552090 ? | 12:37 |
openstack | Launchpad bug 1552090 in ironic-python-agent "IPA cannot create UEFI bootable iso for coreOS " [Wishlist,In progress] - Assigned to Shivanand Tendulker (stendulker) | 12:37 |
*** achanda has quit IRC | 12:37 | |
dtantsur | jroll, no problem | 12:38 |
jroll | thanks | 12:38 |
*** devananda has quit IRC | 12:43 | |
*** keedya has joined #openstack-ironic | 12:43 | |
*** devananda has joined #openstack-ironic | 12:44 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 12:44 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to build user image with DIB https://review.openstack.org/256363 | 12:44 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Link-Local-Connection info to ironic port https://review.openstack.org/256365 | 12:44 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Added operator documentation for ironic portgroups https://review.openstack.org/228496 | 12:44 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: refactor ironic enroll-node code https://review.openstack.org/256364 | 12:44 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add configure_provision_network function https://review.openstack.org/256367 | 12:44 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update Ironic VM network connection https://review.openstack.org/256366 | 12:44 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 12:44 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 12:44 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add network drivers https://review.openstack.org/285852 | 12:44 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Ironic/Neutron integration documentation https://review.openstack.org/258596 | 12:44 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to use NETWORK_DRIVERS in devstack https://review.openstack.org/293520 | 12:44 |
mkovacik | jroll, dtantsur w/r the reapply, I think it's better if we avoid splitting manual from api, it doesn't make much sense; instead if we chose one day to have the ramdisk posting the data to /uuid/data/unprocessed we should have put over the same path for reapply; i still think that the idempotence isn't affected as resulting processed data is going to be manipulated so if you GET /uuid/data/unprocessed you'll have the same thing | 12:49 |
mkovacik | you put there and if you put 100 times, the data there will remain the same; what will change is the result data at /uuid/data | 12:49 |
*** dims_ has quit IRC | 12:49 | |
*** dims has joined #openstack-ironic | 12:49 | |
* jroll has no context here | 12:49 | |
dtantsur | mkovacik, " we chose one day to have the ramdisk posting the data to /uuid/data/unprocessed" this day is not very close ;) | 12:49 |
dtantsur | jroll, I think he meant to include sambetts :) | 12:49 |
jroll | probably :) | 12:50 |
mkovacik | then, sambetts, why the -1 | 12:50 |
mkovacik | yeah sorry jroll | 12:50 |
* dtantsur feels like he also has no context | 12:50 | |
dtantsur | mkovacik, I think sambetts' -1 is about enforcing that the new API does not accept a body, until we implement it | 12:50 |
jroll | heh no worries | 12:51 |
mkovacik | dtantsur, sambetts, renaming api_data_unprocessed won't help anything; I'm cool with the 400 over non-empty body | 12:51 |
mkovacik | but we may one day ;) want to change that ;) | 12:52 |
openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with port. https://review.openstack.org/259354 | 12:53 |
dtantsur | we probably will, even though it's unlikely to come from a ramdisk | 12:53 |
dtantsur | maybe more in line with what nailgun folks are doing: periodic resubmitting of introspection data | 12:53 |
dtantsur | this has interesting implications though | 12:53 |
mkovacik | dtantsur, so it would make sense to have POST trigger regular processing and PUT trigger reapply over such an endpoint | 12:54 |
openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with port. https://review.openstack.org/259354 | 12:54 |
openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for the Ironic CLI commands which checks port-create, port-show, and port-delete using them either without arguments or with incorrect arguments. https://review.openstack.org/259354 | 12:55 |
*** vinaym213 has joined #openstack-ironic | 12:55 | |
mkovacik | dtantsur, sambetts so lemme hack in the body size check with some reasonable text: "body processing not implemented yet" and keep it POST @ /uuid/data/unprocessed ? | 12:56 |
*** e0ne has quit IRC | 12:57 | |
dtantsur | I hope we already agreed on the endpoint itself ;) yeah | 12:58 |
sambetts | yup | 12:58 |
*** vmud213 has quit IRC | 12:58 | |
mkovacik | dtantsur, sambetts, ack :) | 12:59 |
*** mtanino has quit IRC | 13:01 | |
mkovacik | sambetts, dtantsur I've just realised we won't be able to deprecate it later if we chose to have ramdisk post there; it will be just 400 to empty bodies, possibly breaking users; or you think we could redirect then somehow? | 13:03 |
sambetts | ? | 13:04 |
sambetts | when we enable receiving a body we'll just remove the 400 | 13:05 |
sambetts | now: empty body = reapply, populated body = 400, the future: empty body = reapply, populated body = set and apply | 13:06 |
dtantsur | sambetts++ | 13:06 |
*** amotoki has joined #openstack-ironic | 13:06 | |
mkovacik | sambetts, put -> reapply? | 13:07 |
mkovacik | but OK | 13:07 |
sambetts | no, there is no PUT, we established ysterday we can't use PUT | 13:07 |
mkovacik | OK | 13:08 |
*** mbound has quit IRC | 13:08 | |
*** mbound has joined #openstack-ironic | 13:08 | |
*** links has quit IRC | 13:12 | |
*** mkovacik has quit IRC | 13:14 | |
*** Goneri has joined #openstack-ironic | 13:14 | |
*** e0ne has joined #openstack-ironic | 13:15 | |
*** [1]cdearborn has joined #openstack-ironic | 13:15 | |
*** jjohnson2 has quit IRC | 13:16 | |
*** jjohnson2 has joined #openstack-ironic | 13:16 | |
jroll | hm, need a simple, backportable, IPA job to trigger post jobs on all the things | 13:23 |
*** dansmith is now known as bizarroman | 13:25 | |
*** bizarroman is now known as bizarrodan | 13:25 | |
*** dims_ has joined #openstack-ironic | 13:27 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: Correct link to enabling agent drivers https://review.openstack.org/300485 | 13:28 |
jroll | there we go. | 13:28 |
*** e0ne has quit IRC | 13:29 | |
*** jistr has quit IRC | 13:32 | |
*** openstackgerrit has quit IRC | 13:33 | |
*** rbudden has joined #openstack-ironic | 13:34 | |
*** openstackgerrit has joined #openstack-ironic | 13:34 | |
*** achanda has joined #openstack-ironic | 13:34 | |
*** krtaylor has quit IRC | 13:35 | |
jroll | could use reviews on that so we can land/backport once project-config change has been applied ^ | 13:37 |
jroll | the intent is to re-kick post jobs on master and stable branches | 13:38 |
*** achanda has quit IRC | 13:39 | |
*** electrofelix has joined #openstack-ironic | 13:41 | |
dtantsur | jroll, +2 | 13:43 |
*** thiagop-afk has joined #openstack-ironic | 13:43 | |
*** thiagop-afk is now known as thiagop | 13:43 | |
thiagop | good morning folks | 13:43 |
openstackgerrit | Thiago Paiva Brito proposed openstack/ironic-specs: Dynamic allocation of nodes on OneView drivers https://review.openstack.org/275726 | 13:44 |
dtantsur | morning thiagop | 13:44 |
thiagop | hiya dtantsur | 13:45 |
dtantsur | ugh, there is a race between DHCP and IPA in the DIB build. looks like it results in inspection sometimes not returning any IP addresses | 13:45 |
dtantsur | I wonder if we should have a code waiting for IP addressed to get assigned.. | 13:46 |
*** e0ne has joined #openstack-ironic | 13:51 | |
*** mtanino has joined #openstack-ironic | 13:51 | |
*** lascii is now known as lutf-8 | 13:52 | |
NobodyCam | good morning Ironicers and of course .... | 13:52 |
NobodyCam | TGIF! | 13:52 |
dtantsur | morning NobodyCam, really TGIF :) | 13:52 |
sambetts | dtantsur: I think it should be possible to make the .service file wait until after dhcp as got an IP | 13:54 |
dtantsur | sambetts, well, I thought I did it by making it depend on network-online target | 13:54 |
dtantsur | today, however, I see it didn't happen: https://bugzilla.redhat.com/show_bug.cgi?id=1322892 | 13:55 |
openstack | bugzilla.redhat.com bug 1322892 in openstack-ironic-discoverd "No valid interfaces found during introspection" [Unspecified,New] - Assigned to dtantsur | 13:58 |
thiagop | morning TGIF NobodyCam | 13:58 |
sambetts | dtantsur: it should do looking at the docs for it, although maybe we need to do this? systemctl enable systemd-networkd-wait-online.service or systemctl enable NetworkManager-wait-online.service | 13:58 |
sambetts | https://www.freedesktop.org/wiki/Software/systemd/NetworkTarget/ | 13:58 |
dtantsur | there is no network manager on the ramdisk | 13:58 |
NobodyCam | :) | 13:58 |
sambetts | dtantsur: networkd? | 13:58 |
dtantsur | dunno, checking | 13:58 |
sambetts | dtantsur: it seems like without at least one of those, the network targets won't work | 13:58 |
sambetts | Hey NobodyCam TGIF :) | 13:58 |
dtantsur | well, I have neither, but it did solve a similar problem some time ago | 13:58 |
dtantsur | at least for our downstream RHEL builds | 13:58 |
jroll | folks, don't forget to vote for the TC, deadline is sometime next week! | 13:58 |
* dtantsur already did | 13:58 | |
* sambetts did too | 13:58 | |
* jroll just finished | 13:58 | |
*** openstack has quit IRC | 13:58 | |
*** openstack has joined #openstack-ironic | 14:00 | |
*** daemontool_ is now known as daemontool | 14:01 | |
jroll | someone mind landing https://review.openstack.org/#/c/300485/1 ? | 14:02 |
jroll | need to backport that through stable to kick off some post jobs | 14:02 |
jroll | wtf, docs job fail? | 14:03 |
dtantsur | sambetts, maybe the problem is in the weird way DIB DHCP's things: https://github.com/openstack/diskimage-builder/tree/master/elements/dhcp-all-interfaces | 14:03 |
sambetts | jroll: I've seen that a lot recently | 14:04 |
jroll | timeout, /me rechecks | 14:04 |
jroll | yeah | 14:04 |
dtantsur | me too | 14:04 |
jroll | this was vexxhost, maybe should start keeping track | 14:04 |
openstackgerrit | Sofiia Andriichenko proposed openstack/python-ironicclient: Add test for chassis-node-list https://review.openstack.org/299482 | 14:04 |
sambetts | dtantsur: yeah :/ maybe we need to make sure that the ipa.service file is after the dhcp-all-interfaces.service file? | 14:06 |
openstackgerrit | Sofiia Andriichenko proposed openstack/python-ironicclient: Add test for chassis-node-list https://review.openstack.org/299482 | 14:06 |
dtantsur | sambetts, maybe, but there's no single service, rather a family of them... | 14:07 |
*** e0ne has quit IRC | 14:07 | |
dtantsur | so I'm starting to think that we should ensure it on IPA side rather than requiring downstream to figure out all this craziness | 14:07 |
sambetts | I guess we could just add a peice of code to IPA that does, if no route to to Ironic IP blow up, and then add the retry= properties to the ipa.service | 14:08 |
thiagop | NobodyCam: have you seen the changes on the spec of dyn allocation wrt the use of periodic tasks? | 14:12 |
thiagop | NobodyCam: L88-L95 and L158-179 | 14:13 |
*** stendulker has joined #openstack-ironic | 14:14 | |
dtantsur | sambetts, we do have retry on detecting route to Ironic, it should rather be a retry on at least one IP address appearing in inventory... but I'd ask someone knowing about dhcp-all-interfaces thing first | 14:14 |
* dtantsur moves this to #tripleo | 14:15 | |
sambetts | dtantsur: maybe retry on route to inspector too for that case | 14:18 |
stendulker | dtantsur: Hi | 14:18 |
dtantsur | sambetts, maybe | 14:18 |
dtantsur | stendulker, hi | 14:18 |
stendulker | dtantsur: this review https://review.openstack.org/#/c/299573/ is based on a new feature code https://review.openstack.org/#/c/275766 | 14:18 |
stendulker | dtantsur: Is that ok? It has 2 +2s | 14:18 |
openstackgerrit | Merged openstack/ironic-python-agent: Correct link to enabling agent drivers https://review.openstack.org/300485 | 14:18 |
dtantsur | stendulker, well, this fix is independent of that feature, so I'm fine with landing it. It's a question for TheJulia what it was done this way | 14:18 |
stendulker | dtantsur: yes fix is independent | 14:18 |
stendulker | dtantsur: is basing code like this is correct for landing? | 14:18 |
dtantsur | stendulker, well, no. it won't land until the parent lands. TheJulia probably needs to rebase it on top of master | 14:18 |
stendulker | dtantsur: ok. i will -1 it with the comment | 14:18 |
stendulker | dtantsur: thank you. | 14:19 |
TheJulia | I'll rebase it off of master a little later then | 14:22 |
stendulker | TheJulia: thank you | 14:22 |
TheJulia | the previous patch really hits the issue hard, so the dependency has to be flipped in that case since ANC won't work otherwise | 14:22 |
TheJulia | well, it will work, and ilo users will find their hardware powered off | 14:23 |
stendulker | TheJulia: yes, that would be better | 14:23 |
thiagop | hiya TheJulia | 14:25 |
jroll | dtantsur: when you have a moment, liberty https://review.openstack.org/#/c/300516/ and mitaka https://review.openstack.org/#/c/300515/ | 14:25 |
jroll | no rush, going to take a break for a bit | 14:25 |
*** harshs has joined #openstack-ironic | 14:27 | |
TheJulia | well, easier rebase than I thought | 14:27 |
*** stendulker_ has joined #openstack-ironic | 14:28 | |
*** stendulker has quit IRC | 14:31 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: Don't power off non-deploying iLO nodes in takeover https://review.openstack.org/299573 | 14:34 |
jlvillal | Good morning Ironic | 14:37 |
*** vishwanathj has joined #openstack-ironic | 14:37 | |
*** stendulker_ has quit IRC | 14:38 | |
*** vinay50muddu has joined #openstack-ironic | 14:39 | |
*** ametts has joined #openstack-ironic | 14:42 | |
*** vinaym213 has quit IRC | 14:43 | |
*** harshs_ has joined #openstack-ironic | 14:44 | |
dtantsur | morning jlvillal | 14:44 |
*** vinay50muddu has quit IRC | 14:44 | |
dtantsur | sambetts, I'm looking at our downstream ramdisk code (which is a bash hell), I see things like 'sleep 10' to make sure all interfaces did receive DHCP >_< | 14:44 |
*** harshs has quit IRC | 14:45 | |
*** harshs_ is now known as harshs | 14:45 | |
dtantsur | sambetts, so next idea: loop until all interfaces get their IP's with some reasonable deadline (30 seconds?), then proceed with inspection | 14:45 |
dtantsur | wdyt? | 14:45 |
*** jistr has joined #openstack-ironic | 14:46 | |
sambetts | dtantsur: thats definatly one way to do it | 14:50 |
dtantsur | lemme try to put together a patch.. | 14:54 |
*** garthb_ has joined #openstack-ironic | 14:54 | |
*** Marga_ has joined #openstack-ironic | 14:55 | |
*** e0ne has joined #openstack-ironic | 14:57 | |
*** Marga_ has quit IRC | 14:57 | |
*** Marga_ has joined #openstack-ironic | 14:58 | |
sambetts | jroll: https://review.openstack.org/#/c/299485/3/jenkins/jobs/ironic-python-agent-jobs.yaml, AJaeger seems to want us to standardise the post jobs, that would involve standardising the build scripts right? | 14:59 |
*** Marga_ has quit IRC | 14:59 | |
*** Marga_ has joined #openstack-ironic | 15:00 | |
jroll | sambetts: yeah, I guess merging them is an option | 15:01 |
jroll | we don't have to standardize any scripts, just run both, then upload all the things | 15:02 |
jroll | this also means a bug in either script will break uploading of both images, though | 15:02 |
jroll | \o jlvillal | 15:02 |
sambetts | jroll: I didn't want to do that so that one wouldn't affect the other | 15:02 |
jroll | yeah, that's my thought | 15:02 |
*** electrofelix has quit IRC | 15:03 | |
sambetts | I messaged AJaeger on #infra about it and he seemed to want us to make the scripts both dump into a common UPLOAD folder so that we could have generic scp code etc | 15:04 |
*** mdorman has joined #openstack-ironic | 15:05 | |
jroll | sambetts: I'm kinda meh on that, it's also going to break devstack if we move them somewhere else | 15:06 |
openstackgerrit | Karim Boumedhel proposed openstack/ironic-specs: Adds a Blueprint for ovirt support in Ironic https://review.openstack.org/300418 | 15:06 |
sambetts | having something like job: ironic-python-agent-buildimage-{image-type} then using the {image-type} to select what get built | 15:07 |
jroll | oh, a template, that's fine | 15:07 |
jroll | maybe just template out the scp source path for now too | 15:07 |
*** links has joined #openstack-ironic | 15:07 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent: Wait for the interfaces to get IP addresses before inspection https://review.openstack.org/300548 | 15:13 |
dtantsur | sambetts, ^^^ | 15:13 |
jroll | 2016-04-01 15:18:08.986 | + imagebuild/coreos/full_trusty_build.sh | 15:18 |
jroll | 2016-04-01 15:18:08.986 | + BRANCH_PATH=stable-mitaka | 15:18 |
jroll | woooooo. finally. | 15:18 |
dtantsur | \o/ | 15:20 |
NobodyCam | wow :) | 15:20 |
jroll | next step is https://review.openstack.org/297479 | 15:21 |
jroll | and then all the backport craziness for that | 15:21 |
dtantsur | jroll, really dunno how to act upon https://bugs.launchpad.net/ironic/+bug/1564841. redirect to ironic-staging-drivers? | 15:21 |
openstack | Launchpad bug 1564841 in Ironic "[RFE] Add a ovirt driver to Ironic" [Undecided,In progress] - Assigned to Karim Boumedhel (karmab) | 15:21 |
jroll | dtantsur: yeah, I think so | 15:21 |
jroll | http://tarballs.openstack.org/ironic-python-agent/coreos/files/ \o/ | 15:24 |
*** harshs has quit IRC | 15:30 | |
NobodyCam | jroll: am I missing reason there is a need for the 'tr' command? | 15:30 |
dtantsur | NobodyCam, replace / with - for use in the file name | 15:31 |
NobodyCam | preventing folks from sub-pathing | 15:31 |
*** e0ne has quit IRC | 15:31 | |
dtantsur | aka stable/mitaka -> stable-mitaka | 15:31 |
NobodyCam | ah | 15:31 |
NobodyCam | TY | 15:31 |
NobodyCam | +2 with a minor nit | 15:36 |
NobodyCam | :p | 15:36 |
*** achanda has joined #openstack-ironic | 15:37 | |
*** suro-patz has joined #openstack-ironic | 15:37 | |
*** achanda has quit IRC | 15:42 | |
dtantsur | sambetts, do we expect cisco CI to pass on https://review.openstack.org/#/c/297479/ ? | 15:43 |
*** jaybeale has joined #openstack-ironic | 15:44 | |
*** krtaylor has joined #openstack-ironic | 15:44 | |
*** Guest90877 has quit IRC | 15:44 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Correct api version check conditional for node.name https://review.openstack.org/299264 | 15:45 |
NobodyCam | doh TheJulia I just saw that I made a comment on pre existing section of a doc that was not part of the change you are working on... sorry :p | 15:49 |
sambetts | dtantsur: CIMC will pass UCSM will fail | 15:49 |
dtantsur | ok, so it's safe to approve, right? | 15:49 |
sambetts | +2 | 15:50 |
*** yolanda has joined #openstack-ironic | 15:53 | |
*** links has quit IRC | 15:55 | |
openstackgerrit | Merged openstack/ironic-python-agent: Enable branch tagging during tinyipa build https://review.openstack.org/299529 | 15:55 |
jroll | woohoo, thanks for +W dtantsur | 15:57 |
*** penick has joined #openstack-ironic | 15:57 | |
dtantsur | :) | 15:58 |
* jlvillal feels somewhat vindicated in his belief that checking log messages in unit tests can have value. https://bugs.launchpad.net/ironic/+bug/1564531 | 15:58 | |
openstack | Launchpad bug 1564531 in Ironic "Unit test: ironic.tests.unit.conductor.test_manager.DoNodeCleanTestCase.test__do_next_clean_step_manual_fail_in_tear_down_cleaning incorrect" [Undecided,In progress] - Assigned to John L. Villalovos (happycamp) | 15:58 |
dtantsur | jlvillal, I personally prefer to only check if a substantial part of a message is present (e.g. node UUID). I also tend to never check log messages, only user-visible ones. | 15:59 |
jroll | whatever happened to oneview ci? | 15:59 |
dtantsur | sambetts, could you please make the ucsm job non-voting until it's ready to vote? | 16:00 |
jlvillal | dtantsur: Understood. But only because I checked the log message did I discover that the unit test was wrong. | 16:00 |
dtantsur | sambetts, otherwise it's confusing for reviewers whether we should pay attention to Cisco CI or not | 16:00 |
dtantsur | jlvillal, sigh. it happens :( | 16:01 |
dtantsur | also, today I've fixed one more bug with incorrect formatting in log messages... | 16:01 |
jlvillal | dtantsur: Oh :( Sad face for bug with log messages. | 16:01 |
jlvillal | dtantsur: :) happy face for you finding and fixing it! | 16:01 |
dtantsur | jlvillal, btw please review it: https://review.openstack.org/299887 | 16:02 |
dtantsur | a really small one | 16:02 |
jlvillal | dtantsur: will do! | 16:02 |
jroll | +2'd | 16:02 |
jlvillal | dtantsur: And if you care to look at this: https://review.openstack.org/#/c/300204/ Though not really small one | 16:02 |
jlvillal | +2 +A | 16:04 |
*** amotoki has quit IRC | 16:04 | |
dtantsur | thnx | 16:05 |
* jlvillal is that grumpy guy who doesn't find most of the April Fools jokes funny :P | 16:05 | |
jlvillal | Sometimes there are good ones, but most of them are meh... | 16:06 |
dtantsur | jlvillal, my wife said the same, except that she likes neither | 16:06 |
jlvillal | :D | 16:06 |
*** trown is now known as trown|lunch | 16:06 | |
jlvillal | I have learned to stay away from Slashdot on April 1st... | 16:06 |
jroll | lol, dtantsur and I had the same comment | 16:08 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent: Wait for the interfaces to get IP addresses before inspection https://review.openstack.org/300548 | 16:09 |
dtantsur | just added a release note ^^^ | 16:09 |
dtantsur | jroll, really lol | 16:09 |
jroll | yep :P | 16:09 |
*** garthb_ has quit IRC | 16:10 | |
dtantsur | ifarkas, hey! mind reviewing https://review.openstack.org/#/c/294467/ please? it's been around for some time already | 16:10 |
*** zzzeek has quit IRC | 16:13 | |
sambetts | dtantsur: I've made the UCS job voting: false, so any new runs should be marked non-voting from now on | 16:13 |
dtantsur | thnx | 16:13 |
*** MattMan has quit IRC | 16:13 | |
*** zzzeek has joined #openstack-ironic | 16:14 | |
*** jcoufal has quit IRC | 16:15 | |
*** suro-patz has quit IRC | 16:16 | |
*** jjohnson2 has quit IRC | 16:16 | |
*** jjohnson2 has joined #openstack-ironic | 16:17 | |
*** mbound has quit IRC | 16:19 | |
*** yolanda has quit IRC | 16:19 | |
*** karimb has joined #openstack-ironic | 16:21 | |
*** ohamada_ has quit IRC | 16:23 | |
*** e0ne has joined #openstack-ironic | 16:24 | |
jlvillal | dtantsur: jroll: Thanks for helping me learn something new on the asserts :) | 16:25 |
jroll | :D | 16:25 |
dtantsur | :) | 16:25 |
dtantsur | the mock library has a lot of funny stuff, I also learn something new from time to time | 16:26 |
jlvillal | Yeah, trying to teach the mock stuff to my new co-worker. I can seem them struggling a bit to understand it. I know when I first learned it I was going "huh????" | 16:27 |
*** suro-patz has joined #openstack-ironic | 16:27 | |
sambetts | :-P its a weird concept to learn :-P | 16:27 |
jlvillal | sambetts: +1 | 16:28 |
*** e0ne has quit IRC | 16:29 | |
jlvillal | Also some of them have never done unit testing before. And then you add in mock. | 16:29 |
*** e0ne has joined #openstack-ironic | 16:29 | |
*** Marga_ has quit IRC | 16:30 | |
*** yolanda has joined #openstack-ironic | 16:30 | |
*** openstackgerrit has quit IRC | 16:31 | |
*** rcernin has quit IRC | 16:32 | |
sambetts | been through a similar thing with a co-worker | 16:32 |
*** achanda has joined #openstack-ironic | 16:38 | |
* jlvillal waits patiently for bot to announce patch he just pushed... | 16:38 | |
*** e0ne has quit IRC | 16:39 | |
jlvillal | John Villalovos proposed openstack/ironic: Fix _do_next_clean_step_fail_in_tear_down_cleaning() https://review.openstack.org/#/c/300204/ | 16:39 |
TheJulia | heh | 16:39 |
jlvillal | I was tired of waiting. It had been a whole two minutes | 16:40 |
TheJulia | well, the bot did timeout about 01 minutes ago | 16:40 |
TheJulia | 10 | 16:40 |
jlvillal | Ah. | 16:40 |
* jlvillal disables seeing join/quit messages | 16:40 | |
*** kromanenko has quit IRC | 16:41 | |
TheJulia | Posted new revision of https://review.openstack.org/#/c/275766 - | 16:41 |
TheJulia | Active Node Creation via adopt state | 16:41 |
*** kromanenko_ has quit IRC | 16:41 | |
* TheJulia goes and digs into new bug | 16:41 | |
jroll | dtantsur: mitaka devstack changes are passing now https://review.openstack.org/#/c/299317/ | 16:43 |
NobodyCam | jlvillal: I did that a year ago :) | 16:44 |
*** pcaruana has quit IRC | 16:44 | |
* dtantsur clicks | 16:44 | |
jlvillal | NobodyCam: :) It does help the signal to noise ratio for me. | 16:45 |
*** achanda has quit IRC | 16:45 | |
*** openstackgerrit has joined #openstack-ironic | 16:46 | |
dtantsur | jroll, +2. looks like it even works with the inspector gate, good | 16:47 |
dtantsur | and now I have to run, see you next week and have a great weekend | 16:47 |
jroll | dtantsur: thanks! have a good weekend | 16:47 |
*** dtantsur is now known as dtantsur|afk | 16:47 | |
jroll | oh dtantsur, before you go | 16:47 |
mgould | dtantsur, good night! | 16:47 |
jroll | dtantsur|afk: ^ | 16:47 |
dtantsur|afk | yep? | 16:47 |
jlvillal | dtantsur: One good thing about April Fools day, both Apress and PragPub are having 40% of ebooks :) | 16:47 |
jroll | dtantsur|afk: you want a change on liberty inspector to test the devstack change on inspector? | 16:47 |
jroll | I can push one up | 16:48 |
dtantsur|afk | jroll, not sure I got the question, but please do :) | 16:48 |
dtantsur|afk | probably both mitaka and liberty, in this case | 16:48 |
jroll | dtantsur|afk: you don't have a test on devstack changes, so a testpatch on inspector liberty against that devstack change | 16:48 |
jroll | ok | 16:48 |
jroll | there's no kilo, right? | 16:48 |
dtantsur|afk | nope | 16:49 |
jroll | k | 16:49 |
jroll | thanks! | 16:49 |
dtantsur|afk | thank you | 16:49 |
* dtantsur|afk is now really afk | 16:49 | |
sambetts | goodnight dtantsur|afk | 16:54 |
*** e0ne has joined #openstack-ironic | 16:55 | |
*** Marga_ has joined #openstack-ironic | 17:02 | |
openstackgerrit | Merged openstack/ironic: Fix _do_next_clean_step_fail_in_tear_down_cleaning() https://review.openstack.org/300204 | 17:04 |
openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Tests for testing node-create command https://review.openstack.org/262055 | 17:08 |
*** trown|lunch is now known as trown | 17:09 | |
*** mgoddard has quit IRC | 17:14 | |
*** ndipanov has quit IRC | 17:14 | |
*** e0ne has quit IRC | 17:15 | |
*** Marga_ has quit IRC | 17:17 | |
*** Marga_ has joined #openstack-ironic | 17:17 | |
*** yolanda has quit IRC | 17:19 | |
*** davideagnello has joined #openstack-ironic | 17:19 | |
*** suro-patz has quit IRC | 17:21 | |
*** sambetts is now known as sambetts|afk | 17:26 | |
sambetts|afk | Night all | 17:26 |
jlvillal | sambetts|afk: Ciao | 17:26 |
openstackgerrit | xiexs proposed openstack/python-ironicclient: Add multi nodes support for "baremetal delete" command https://review.openstack.org/289676 | 17:28 |
NobodyCam | night sambetts|afk | 17:31 |
NobodyCam | night dtantsur|afk | 17:31 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: Correct api version check conditional for node.name https://review.openstack.org/299264 | 17:39 |
*** vishwanathj has quit IRC | 17:40 | |
openstackgerrit | Merged openstack/ironic-python-agent: Fix incorrect formatting in the iSCSI extension https://review.openstack.org/299887 | 17:44 |
*** suro-patz has joined #openstack-ironic | 17:46 | |
*** cdearborn has joined #openstack-ironic | 17:52 | |
*** mgould has quit IRC | 17:54 | |
*** [3]cdearborn has joined #openstack-ironic | 17:55 | |
*** vishwanathj has joined #openstack-ironic | 17:57 | |
*** jistr has quit IRC | 17:58 | |
*** suro-patz has quit IRC | 18:02 | |
jroll | devananda: you'll be interested in https://review.openstack.org/#/c/245459/ | 18:02 |
jroll | (TIL) | 18:02 |
*** harlowja has quit IRC | 18:02 | |
*** suro-patz has joined #openstack-ironic | 18:03 | |
*** piet has joined #openstack-ironic | 18:05 | |
* devananda looks | 18:05 | |
jroll | I haven't reviewed yet, but didn't realize someone was working on this | 18:05 |
devananda | oh. hah. nice | 18:05 |
*** [1]cdearborn has quit IRC | 18:06 | |
jroll | devananda: would also like your eyes on https://review.openstack.org/#/c/299317/ | 18:07 |
jroll | that's the last thing to make mitaka test mitaka ipa | 18:07 |
*** cdearborn has quit IRC | 18:10 | |
*** piet has quit IRC | 18:15 | |
*** daemontool has quit IRC | 18:18 | |
jroll | dtantsur|afk: liberty inspector stuff looks okay in terms of devstack/ipa changes, but py27 looks busted :( https://review.openstack.org/#/c/300581/ | 18:20 |
jroll | anyway, not going to block my stuff on that | 18:20 |
jroll | I'm taking off, might check back in later but otherwise have a good weekend :) | 18:21 |
*** baoli_ has quit IRC | 18:22 | |
*** garthb has joined #openstack-ironic | 18:22 | |
*** vishwanathj has quit IRC | 18:22 | |
NobodyCam | have a great weekend jroll | 18:24 |
*** piet has joined #openstack-ironic | 18:30 | |
*** baoli has joined #openstack-ironic | 18:35 | |
*** jaybeale has quit IRC | 18:39 | |
*** ifarkas has quit IRC | 18:43 | |
*** ChubYann has joined #openstack-ironic | 18:43 | |
*** keedya has quit IRC | 18:45 | |
*** Sukhdev has joined #openstack-ironic | 18:53 | |
*** david-lyle has quit IRC | 18:53 | |
*** piet has quit IRC | 18:56 | |
*** piet has joined #openstack-ironic | 18:58 | |
*** david-lyle has joined #openstack-ironic | 18:59 | |
*** Goneri has quit IRC | 18:59 | |
*** Sukhdev has quit IRC | 19:00 | |
*** suro-patz has quit IRC | 19:03 | |
*** piet has quit IRC | 19:03 | |
*** suro-patz has joined #openstack-ironic | 19:11 | |
*** mbound has joined #openstack-ironic | 19:11 | |
*** jaybeale has joined #openstack-ironic | 19:18 | |
*** baoli has quit IRC | 19:19 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/300625 | 19:29 |
*** baoli has joined #openstack-ironic | 19:29 | |
*** harshs has joined #openstack-ironic | 19:36 | |
*** harlowja has joined #openstack-ironic | 19:37 | |
*** baoli has quit IRC | 19:43 | |
TheJulia | anyone know who maintains the irmc driver? | 19:45 |
*** Sukhdev has joined #openstack-ironic | 19:52 | |
*** baoli has joined #openstack-ironic | 19:55 | |
*** garthb_ has joined #openstack-ironic | 19:57 | |
*** garthb has quit IRC | 20:00 | |
*** Sukhdev has quit IRC | 20:08 | |
*** mbound has quit IRC | 20:08 | |
*** UForgotten has quit IRC | 20:12 | |
JayF | TheJulia: might wanna read mailing list | 20:27 |
JayF | TheJulia: tl;dr: complaints about bifrost not making build artifacts | 20:27 |
*** dprince has quit IRC | 20:28 | |
*** jayc has quit IRC | 20:32 | |
*** raildo is now known as raildo-afk | 20:42 | |
* jlvillal assumes TheJulia saw the email about Bifrost build artifacts | 20:48 | |
* jlvillal also sees JayF is much faster than him... | 20:48 | |
JayF | when I'm working I tend to react to email boops very quickly, for better or worse | 20:49 |
* JayF inbox zero 4 lyfe | 20:49 | |
jlvillal | :) | 20:49 |
*** alexpilotti has quit IRC | 20:54 | |
TheJulia | ugh | 21:05 |
TheJulia | oh lovely | 21:08 |
TheJulia | I must be missing something deep inside the CI config that is just not clearly documented on project creation :\ | 21:10 |
*** rbudden has quit IRC | 21:11 | |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Discard reference when no longer needed https://review.openstack.org/300660 | 21:11 |
*** trown is now known as trown|outtypewww | 21:11 | |
*** devananda has quit IRC | 21:12 | |
*** PhilipTW has quit IRC | 21:12 | |
*** devananda has joined #openstack-ironic | 21:13 | |
*** alexpilotti has joined #openstack-ironic | 21:15 | |
*** mdorman has quit IRC | 21:16 | |
*** jjohnson2 has quit IRC | 21:19 | |
*** alexpilotti has quit IRC | 21:19 | |
TheJulia | JayF: jlvillal thank you both | 21:20 |
JayF | np | 21:20 |
*** Sukhdev has joined #openstack-ironic | 21:29 | |
*** Sukhdev has quit IRC | 21:31 | |
*** Goneri has joined #openstack-ironic | 21:31 | |
*** baoli has quit IRC | 21:36 | |
*** daemontool has joined #openstack-ironic | 21:44 | |
*** jayc has joined #openstack-ironic | 21:51 | |
*** Sukhdev has joined #openstack-ironic | 21:52 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: deployment vmedia ops should not be run when not deploying https://review.openstack.org/300674 | 21:53 |
*** Sukhdev has quit IRC | 21:55 | |
*** Sukhdev has joined #openstack-ironic | 22:00 | |
*** daemontool has quit IRC | 22:03 | |
*** Sukhdev has quit IRC | 22:07 | |
*** rbrud has quit IRC | 22:12 | |
*** lutf-8 is now known as alaski | 22:12 | |
*** alexpilotti has joined #openstack-ironic | 22:17 | |
*** alexpilotti has quit IRC | 22:22 | |
*** mbound has joined #openstack-ironic | 22:24 | |
*** Sukhdev has joined #openstack-ironic | 22:25 | |
*** mbound has quit IRC | 22:25 | |
*** mbound has joined #openstack-ironic | 22:25 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: Active Node Creation via adopt state https://review.openstack.org/275766 | 22:28 |
*** ametts has quit IRC | 22:28 | |
*** achanda has joined #openstack-ironic | 22:29 | |
*** suro-patz has quit IRC | 22:29 | |
*** karimb has quit IRC | 22:32 | |
* jlvillal only now sees jroll's April Fools patch :) | 22:39 | |
*** Marga_ has quit IRC | 22:39 | |
*** krtaylor has quit IRC | 22:44 | |
JayF | jlvillal: link? | 22:46 |
jlvillal | JayF: https://review.openstack.org/300434 | 22:47 |
*** ppiela_ has quit IRC | 22:49 | |
*** daemontool has joined #openstack-ironic | 22:51 | |
*** krtaylor has joined #openstack-ironic | 22:56 | |
*** keedya has joined #openstack-ironic | 22:58 | |
*** Marga_ has joined #openstack-ironic | 23:01 | |
*** mgoddard has joined #openstack-ironic | 23:01 | |
TheJulia | 2016-04-01 22:29:59.230 | /tmp/hudson1946662699213468843.sh: line 99: syntax error: unexpected end of file \o/ | 23:01 |
jlvillal | TheJulia: What do you mean? | 23:03 |
TheJulia | our gate is failing | 23:04 |
TheJulia | looking at recent changes in project config right now | 23:04 |
JayF | remember lib/ironic in devstack comes from us | 23:05 |
JayF | so it could be something from ironic, although we'd have hoped that'd fail gate I guess | 23:05 |
*** Marga_ has quit IRC | 23:06 | |
*** [1]cdearborn has joined #openstack-ironic | 23:06 | |
TheJulia | yeah, 99 makes me think it is coming out of something from jjb | 23:09 |
openstackgerrit | Clint 'SpamapS' Byrum proposed openstack/bifrost: Add playbook to transfer nodes to another Ironic https://review.openstack.org/299660 | 23:09 |
TheJulia | well, more so the hudson and then line 99 | 23:09 |
TheJulia | hudson part | 23:09 |
JayF | oh, gotcha | 23:11 |
JayF | that's the jeknins runner script | 23:11 |
*** mgoddard has quit IRC | 23:11 | |
TheJulia | yup, "./safe-devstack-vm-gate-wrap.sh" is apparently line 99 in the corresponding devstack-gate job that is running | 23:12 |
*** alexpilotti has joined #openstack-ironic | 23:13 | |
*** mtanino has quit IRC | 23:13 | |
*** Marga_ has joined #openstack-ironic | 23:14 | |
*** alexpilotti has quit IRC | 23:16 | |
*** alexpilotti has joined #openstack-ironic | 23:16 | |
*** daemontool has quit IRC | 23:17 | |
*** daemontool has joined #openstack-ironic | 23:18 | |
*** [3]cdearborn has quit IRC | 23:19 | |
TheJulia | hmm.. nope... off by two lines :\ | 23:20 |
*** trown|outtypewww is now known as trown | 23:21 | |
*** alexpilotti has quit IRC | 23:21 | |
*** dkehn has joined #openstack-ironic | 23:23 | |
*** Sukhdev has quit IRC | 23:23 | |
TheJulia | hmm... can't run the extracted output...... same error | 23:23 |
TheJulia | i see it | 23:30 |
*** harlowja has quit IRC | 23:36 | |
*** daemontool has quit IRC | 23:38 | |
TheJulia | https://review.openstack.org/#/c/300694/ should fix the gate, etherpad updated. | 23:39 |
*** harshs has quit IRC | 23:40 | |
*** jayc has quit IRC | 23:44 | |
TheJulia | Looks like the fix is going to land in a little bit. Would it be wrong to update the etherpad with "Everything is Awesome!" | 23:56 |
TheJulia | ? | 23:56 |
TheJulia | :) | 23:56 |
JayF | I'd just link to the patch and say "if this has landed; gate should be fixed" | 23:58 |
JayF | and let someone mark it as happy once it actually is :P | 23:58 |
TheJulia | Then my work here is done for the night | 23:59 |
TheJulia | :) | 23:59 |
JayF | TheJulia: you'll appreciate this I think, if you wanna see something awesome | 23:59 |
TheJulia | oh? | 23:59 |
JayF | TheJulia: I just replaced my keycaps on my das ultimate with this | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!