*** ijw has joined #openstack-ironic | 00:01 | |
*** vgadiraj has quit IRC | 00:03 | |
*** yuanying_ has joined #openstack-ironic | 00:03 | |
*** ijw has quit IRC | 00:03 | |
*** ijw has joined #openstack-ironic | 00:04 | |
*** penick has quit IRC | 00:04 | |
*** jtomasek has quit IRC | 00:05 | |
*** penick has joined #openstack-ironic | 00:09 | |
*** mtanino has quit IRC | 00:27 | |
*** jkilpatr_ has quit IRC | 00:47 | |
*** hoangcx has joined #openstack-ironic | 00:47 | |
*** rpioso has quit IRC | 00:48 | |
*** soliosg has quit IRC | 00:51 | |
*** phuongnh has joined #openstack-ironic | 00:57 | |
*** anush has joined #openstack-ironic | 00:59 | |
*** jkilpatr_ has joined #openstack-ironic | 01:01 | |
*** SerenaFeng has joined #openstack-ironic | 01:07 | |
*** anush has quit IRC | 01:11 | |
*** cdearborn has joined #openstack-ironic | 01:11 | |
*** jkilpatr_ has quit IRC | 01:18 | |
*** jkilpatr_ has joined #openstack-ironic | 01:24 | |
*** jkilpatr_ has quit IRC | 01:28 | |
*** vsaienko has quit IRC | 01:33 | |
*** yufei has joined #openstack-ironic | 01:53 | |
*** anush has joined #openstack-ironic | 02:01 | |
*** anush has quit IRC | 02:02 | |
*** zhangjl has joined #openstack-ironic | 02:05 | |
*** rama_y has quit IRC | 02:06 | |
yufei | hi, JayF, are you online? | 02:06 |
---|---|---|
yufei | I see your comments at https://review.openstack.org/#/c/399857/, but I’m confused, do you mean that the api_url should have a default value instead of None? | 02:09 |
patchbot | patch 399857 - ironic-python-agent - Skip API releated work if no api url configured | 02:09 |
*** baoli has joined #openstack-ironic | 02:11 | |
*** syed__ has quit IRC | 02:16 | |
*** yufei has quit IRC | 02:18 | |
*** yufei has joined #openstack-ironic | 02:24 | |
*** rajinir has quit IRC | 02:26 | |
*** aNuposic has quit IRC | 02:38 | |
*** penick has quit IRC | 02:42 | |
*** jschwarz has quit IRC | 02:45 | |
*** jschwarz has joined #openstack-ironic | 02:49 | |
*** ijw has quit IRC | 02:53 | |
*** ijw has joined #openstack-ironic | 02:53 | |
*** ijw has quit IRC | 02:53 | |
*** yuanying_ has quit IRC | 03:00 | |
*** links has joined #openstack-ironic | 03:10 | |
*** links has quit IRC | 03:10 | |
*** gcb has joined #openstack-ironic | 03:15 | |
*** syed__ has joined #openstack-ironic | 03:16 | |
*** rloo has quit IRC | 03:29 | |
*** SerenaFeng has quit IRC | 03:30 | |
*** cdearborn has quit IRC | 03:31 | |
*** alexpilotti has quit IRC | 03:40 | |
*** alexpilotti has joined #openstack-ironic | 03:42 | |
*** baoli has quit IRC | 03:48 | |
*** ChubYann has quit IRC | 03:58 | |
*** jkilpatr has joined #openstack-ironic | 03:58 | |
*** yuanying has joined #openstack-ironic | 03:59 | |
*** tuanluong has joined #openstack-ironic | 04:06 | |
*** jkilpatr has quit IRC | 04:07 | |
*** Nisha has joined #openstack-ironic | 04:11 | |
*** Nisha is now known as Guest31238 | 04:12 | |
*** ChubYann has joined #openstack-ironic | 04:13 | |
openstackgerrit | Joanna Taryma proposed openstack/ironic: Fixed allowing non-uuids as node UUID for port and portgroup. https://review.openstack.org/403910 | 04:13 |
*** jkilpatr has joined #openstack-ironic | 04:26 | |
*** Guest31238 is now known as Nisha_ | 04:28 | |
*** jkilpatr has quit IRC | 04:30 | |
*** jkilpatr has joined #openstack-ironic | 04:33 | |
*** Nisha_ is now known as Nisha_Agarwal | 04:36 | |
*** Nisha_Agarwal is now known as Nisha_brb | 04:37 | |
*** Nisha_brb is now known as Nisha_away | 04:37 | |
*** Nisha_away has quit IRC | 04:37 | |
*** Nisha_Agarwal has joined #openstack-ironic | 04:38 | |
*** jkilpatr has quit IRC | 04:38 | |
*** Nisha_Agarwal is now known as Nisha_ | 04:39 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Enhance ilo drivers to do inband inspection https://review.openstack.org/201904 | 04:45 |
*** baoli has joined #openstack-ironic | 04:49 | |
*** vikrant has joined #openstack-ironic | 04:54 | |
*** aparnav has joined #openstack-ironic | 04:56 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Add new capabilities to ironic https://review.openstack.org/338138 | 04:58 |
*** baoli has quit IRC | 05:01 | |
*** jkilpatr has joined #openstack-ironic | 05:02 | |
*** jrist has quit IRC | 05:03 | |
*** moshele has joined #openstack-ironic | 05:09 | |
*** zhangjl has quit IRC | 05:35 | |
*** SerenaFeng has joined #openstack-ironic | 05:37 | |
*** bks has joined #openstack-ironic | 05:46 | |
*** Nisha_ has quit IRC | 05:53 | |
*** yufei has quit IRC | 05:54 | |
*** yufei has joined #openstack-ironic | 05:55 | |
*** pester has joined #openstack-ironic | 05:58 | |
*** gcb has quit IRC | 06:08 | |
*** gcb has joined #openstack-ironic | 06:09 | |
*** Nisha_ has joined #openstack-ironic | 06:11 | |
*** gcb has quit IRC | 06:21 | |
*** jaosorior has joined #openstack-ironic | 06:22 | |
*** aNuposic has joined #openstack-ironic | 06:29 | |
*** gcb has joined #openstack-ironic | 06:41 | |
*** zhangjl has joined #openstack-ironic | 06:44 | |
*** fxpester has joined #openstack-ironic | 06:45 | |
*** vsaienko has joined #openstack-ironic | 06:47 | |
*** vsaienko has quit IRC | 06:47 | |
*** mjura has joined #openstack-ironic | 06:53 | |
*** ijw has joined #openstack-ironic | 06:56 | |
*** baoli has joined #openstack-ironic | 06:58 | |
*** mjura has quit IRC | 07:01 | |
*** mjura has joined #openstack-ironic | 07:01 | |
*** Nisha_ has quit IRC | 07:02 | |
*** baoli has quit IRC | 07:02 | |
pas-ha | morning Ironicers :) | 07:06 |
*** jaosorior has quit IRC | 07:07 | |
*** jaosorior has joined #openstack-ironic | 07:08 | |
*** vsaienko has joined #openstack-ironic | 07:10 | |
*** syed__ has quit IRC | 07:16 | |
*** yuanying has quit IRC | 07:20 | |
*** pcaruana has joined #openstack-ironic | 07:21 | |
*** ChubYann has quit IRC | 07:21 | |
*** ElCoyote_ has quit IRC | 07:23 | |
*** rbartal has joined #openstack-ironic | 07:26 | |
*** yuanying has joined #openstack-ironic | 07:26 | |
*** makowals has joined #openstack-ironic | 07:28 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from OneView drivers https://review.openstack.org/397846 | 07:31 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove iBoot, WoL and AMT drivers https://review.openstack.org/397847 | 07:31 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from most drivers https://review.openstack.org/397844 | 07:31 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from iLO drvers https://review.openstack.org/397845 | 07:31 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru completely https://review.openstack.org/397848 | 07:31 |
*** vsaienko has quit IRC | 07:38 | |
*** aNuposic has quit IRC | 07:40 | |
*** vsaienko has joined #openstack-ironic | 07:40 | |
*** aNuposic has joined #openstack-ironic | 07:41 | |
*** Nisha_ has joined #openstack-ironic | 07:47 | |
vsaienk0 | sambetts: I will try to check your patch today | 07:48 |
vsaienk0 | jlvillal: unfortunately I don't, I've never launched grenade locally :( | 07:49 |
*** fxpester has quit IRC | 07:50 | |
*** vsaienko has quit IRC | 07:53 | |
*** Nisha_ has quit IRC | 07:59 | |
*** baoli has joined #openstack-ironic | 07:59 | |
*** SerenaFeng has quit IRC | 08:01 | |
*** SerenaFeng has joined #openstack-ironic | 08:01 | |
*** e0ne has joined #openstack-ironic | 08:02 | |
*** baoli has quit IRC | 08:04 | |
*** yufei has quit IRC | 08:11 | |
*** yufei has joined #openstack-ironic | 08:13 | |
*** jpich has joined #openstack-ironic | 08:16 | |
*** ralonsoh has joined #openstack-ironic | 08:17 | |
*** yuanying has quit IRC | 08:17 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Switch func3 environment to Python 3.5 https://review.openstack.org/404072 | 08:19 |
*** SerenaFeng has quit IRC | 08:19 | |
*** SerenaFeng has joined #openstack-ironic | 08:22 | |
*** SerenaFeng has quit IRC | 08:22 | |
*** SerenaFeng has joined #openstack-ironic | 08:22 | |
*** fxpester has joined #openstack-ironic | 08:23 | |
*** e0ne has quit IRC | 08:25 | |
*** yolanda has quit IRC | 08:31 | |
*** snozhka has joined #openstack-ironic | 08:31 | |
*** alexpilo_ has joined #openstack-ironic | 08:35 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui: Imported Translations from Zanata https://review.openstack.org/403336 | 08:35 |
*** alexpilotti has quit IRC | 08:36 | |
*** ijw has quit IRC | 08:38 | |
*** alexpilotti has joined #openstack-ironic | 08:40 | |
*** alexpilo_ has quit IRC | 08:41 | |
*** yolanda has joined #openstack-ironic | 08:43 | |
openstackgerrit | jiang wei proposed openstack/ironic: Move root controller tests to functional dir https://review.openstack.org/356188 | 08:43 |
openstackgerrit | jiang wei proposed openstack/ironic: Move api driver tests to functional dir https://review.openstack.org/350365 | 08:43 |
openstackgerrit | jiang wei proposed openstack/ironic: Move port api tests to functional dir https://review.openstack.org/356287 | 08:43 |
openstackgerrit | jiang wei proposed openstack/ironic: Prepare for functional tests https://review.openstack.org/349421 | 08:43 |
*** alexpilo_ has joined #openstack-ironic | 08:50 | |
*** alexpilotti has quit IRC | 08:51 | |
openstackgerrit | jiang wei proposed openstack/ironic: Move root controller tests to functional dir https://review.openstack.org/356188 | 08:53 |
*** alexpilotti has joined #openstack-ironic | 08:55 | |
*** alexpilo_ has quit IRC | 08:55 | |
*** rcernin has joined #openstack-ironic | 08:56 | |
*** vsaienko has joined #openstack-ironic | 08:58 | |
*** vsaienko has quit IRC | 08:58 | |
vdrok | good morning ironic and pas-ha ! | 08:59 |
vdrok | and morning vsaienk0 | 08:59 |
*** alexpilotti has quit IRC | 08:59 | |
*** mgould|afk is now known as mgould | 09:00 | |
*** zzzeek has quit IRC | 09:00 | |
mgould | morning Ironic! | 09:00 |
vdrok | morning mgould | 09:00 |
*** zzzeek has joined #openstack-ironic | 09:00 | |
snozhka | morning mgould! | 09:00 |
mgould | morning vsaienk0 vdrok | 09:00 |
mgould | morning snozhka | 09:00 |
*** baoli has joined #openstack-ironic | 09:00 | |
*** pmannidi has quit IRC | 09:00 | |
vsaienk0 | morning mgould, vdrok, pas-ha, milan, dtantsur, lucasagomes and all Ironic'ers! | 09:01 |
mgould | morning pas-ha | 09:02 |
pas-ha | morning mgould vdrok vsaienk0 snozhka | 09:02 |
snozhka | morning pas-ha | 09:03 |
*** baoli has quit IRC | 09:05 | |
*** alexpilotti has joined #openstack-ironic | 09:06 | |
*** phuongnh has quit IRC | 09:06 | |
*** alineb has joined #openstack-ironic | 09:11 | |
*** milan has joined #openstack-ironic | 09:12 | |
milan | morning Ironic! :) | 09:13 |
*** alexpilotti has quit IRC | 09:15 | |
*** alexpilotti has joined #openstack-ironic | 09:16 | |
vdrok | morning milan | 09:18 |
*** sborkows has joined #openstack-ironic | 09:18 | |
milan | morning vdrok! :) | 09:18 |
*** derekh has joined #openstack-ironic | 09:18 | |
milan | vdrok, any plans for Brno visit? ;) | 09:18 |
sborkows | morning, all | 09:19 |
vdrok | vdrok: heh, not in the nearest future unfortunately | 09:19 |
vdrok | morning sborkows | 09:19 |
milan | sborkows, morning! :) | 09:19 |
vdrok | milan: ouch, replied to myself :) | 09:19 |
milan | vdrok, yeah :D | 09:19 |
milan | vdrok, sad, Christmas coming, the cities gonna have a lot of ghluhwine opportunities ;) | 09:20 |
vdrok | milan: don't even know how to pronounce that word :) in russian its name is easier | 09:23 |
milan | vdrok, what's the russian version? :) | 09:23 |
vdrok | glintwine | 09:23 |
milan | not far :D | 09:24 |
*** e0ne has joined #openstack-ironic | 09:24 | |
*** lucas-afk is now known as lucasagomes | 09:25 | |
lucasagomes | morning all | 09:25 |
milan | morning lucasagomes! :) | 09:25 |
milan | lucasagomes, let's open a dishwasher shop? ;) | 09:26 |
lucasagomes | milan, sure, to mine bitcoins ? | 09:26 |
milan | lucasagomes ;) | 09:27 |
openstackgerrit | Merged openstack/ironic-inspector: Don't recreate db during grenade upgrade https://review.openstack.org/403377 | 09:28 |
vdrok | lucasagomes: dtantsur|afk https://review.openstack.org/386734 | 09:28 |
patchbot | patch 386734 - releases - remove ironic-staging-drivers (MERGED) | 09:28 |
vdrok | seems like we're not in releases anymore | 09:28 |
lucasagomes | vdrok, urgh... so we have to manually release it to pip | 09:28 |
lucasagomes | pypi* | 09:28 |
*** ohamada has joined #openstack-ironic | 09:29 | |
vdrok | lucasagomes: yuriyz wanted to rewrite the root hints patch with operators, but it won't happen in the nearest time | 09:29 |
vdrok | lucasagomes: so seems that we can release without it for now, if you're ok with it | 09:30 |
lucasagomes | sure, np | 09:30 |
vdrok | lucasagomes: if you don't have time for it, I can do that, jsut don't know the process :) | 09:31 |
*** zackf has quit IRC | 09:31 | |
lucasagomes | vdrok, yeah, since now we have to release it directly on pypi I have to look how to as well | 09:31 |
lucasagomes | vdrok, lemme see | 09:31 |
vdrok | thanks! | 09:31 |
*** Romanenko_K has joined #openstack-ironic | 09:32 | |
openstackgerrit | paresh sao proposed openstack/proliantutils: Change to disable target DHCP in iLO https://review.openstack.org/404111 | 09:35 |
*** e0ne has quit IRC | 09:39 | |
*** e0ne has joined #openstack-ironic | 09:40 | |
*** openstackgerrit has quit IRC | 09:48 | |
*** makowals_ has joined #openstack-ironic | 09:48 | |
*** openstackgerrit has joined #openstack-ironic | 09:49 | |
*** aNuposic has quit IRC | 09:49 | |
*** SerenaFeng has quit IRC | 09:50 | |
*** makowals has quit IRC | 09:51 | |
*** e0ne has quit IRC | 09:51 | |
*** e0ne has joined #openstack-ironic | 09:52 | |
*** yolanda has quit IRC | 09:55 | |
*** yolanda has joined #openstack-ironic | 09:57 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic-python-agent: [POC] BitTorrent image provisioning https://review.openstack.org/404120 | 09:58 |
*** jschwarz has quit IRC | 09:59 | |
*** abalfour has quit IRC | 10:00 | |
*** abalfour has joined #openstack-ironic | 10:01 | |
*** baoli has joined #openstack-ironic | 10:01 | |
openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic: Python 3 Compatible JSON https://review.openstack.org/404122 | 10:02 |
*** SerenaFeng has joined #openstack-ironic | 10:03 | |
*** baoli has quit IRC | 10:06 | |
openstackgerrit | Merged openstack/ironic: Update iptables rules and services IPs for multinode https://review.openstack.org/368648 | 10:06 |
*** jschwarz has joined #openstack-ironic | 10:10 | |
*** yufei has quit IRC | 10:10 | |
*** hshiina has joined #openstack-ironic | 10:12 | |
*** SerenaFeng has quit IRC | 10:12 | |
*** zhangjl has quit IRC | 10:15 | |
*** SerenaFeng has joined #openstack-ironic | 10:16 | |
sambetts|afk | Morning all | 10:19 |
*** sambetts|afk is now known as sambetts | 10:19 | |
lucasagomes | sambetts, morning | 10:21 |
sambetts | o/ lucasagomes | 10:21 |
*** hoangcx has quit IRC | 10:26 | |
moshele | aarefiev: hi can you review again https://review.openstack.org/#/c/264257/ | 10:28 |
patchbot | patch 264257 - ironic-inspector - Adding InfiniBand Support | 10:28 |
aarefiev | moshele: sure, I'll take a look today | 10:28 |
moshele | sambetts: morning, can you review https://review.openstack.org/#/c/399671/ | 10:30 |
patchbot | patch 399671 - ironic-inspector - Add lldp system name to switch_info | 10:30 |
lucasagomes | sambetts, now, something odd: http://paste.openstack.org/show/590773/ | 10:30 |
lucasagomes | sambetts, the serial is "abcdefg" it appears on the second command, but lsblk (or udev) just overwrite the serial with the wwn | 10:30 |
*** parlos has joined #openstack-ironic | 10:35 | |
*** e0ne has quit IRC | 10:40 | |
*** e0ne_ has joined #openstack-ironic | 10:40 | |
sambetts | lucasagomes: -_- | 10:46 |
*** yufei has joined #openstack-ironic | 10:46 | |
vdrok | morning sambetts | 10:54 |
vsaienk0 | morning sambetts | 10:54 |
sambetts | hey vdrok, vsaienk0 | 10:55 |
sambetts | vsaienk0, I heard congratulations are in order, I hope everything went ok? | 10:55 |
vsaienk0 | sambetts: could you please resolve merge conflict on network refactoring patch https://review.openstack.org/#/c/392959/ all multinode patches are merged. | 10:55 |
patchbot | patch 392959 - ironic - Rework Ironic devstack baremetal network simulation | 10:55 |
sambetts | vsaienk0: yup its on my list, problem is everything that merges blows it up | 10:56 |
vsaienk0 | sambetts: yeah, all is great thanks! I have two daughters now, and a lot of non-sleeping nights :) | 10:57 |
sambetts | vsaienk0: twins?! | 10:57 |
vsaienk0 | older daughter is 3 years old | 10:59 |
sambetts | ah! Congrats | 10:59 |
*** baoli has joined #openstack-ironic | 11:02 | |
*** baoli has quit IRC | 11:07 | |
*** Nisha_ has joined #openstack-ironic | 11:07 | |
*** SerenaFeng has quit IRC | 11:09 | |
*** dtantsur|afk is now known as dtantsur | 11:10 | |
*** fxpester has quit IRC | 11:10 | |
dtantsur | morning Ironic | 11:10 |
*** cdearborn has joined #openstack-ironic | 11:10 | |
dtantsur | lucasagomes, vdrok, I can release stuff | 11:11 |
dtantsur | assuming I have push tag rights on ironic-staging-drivers, that is | 11:12 |
vdrok | morning dtantsur | 11:12 |
lucasagomes | dtantsur, morning | 11:12 |
lucasagomes | dtantsur, can you check it ? | 11:12 |
dtantsur | sure | 11:12 |
vdrok | but to release on pypi, you still have to be the owner? | 11:13 |
dtantsur | vdrok, I don't, openstackci should be an admin on pypi | 11:13 |
dtantsur | lucasagomes, I need to be added to https://review.openstack.org/#/admin/groups/1259,members to be able to release it | 11:13 |
vdrok | aha | 11:14 |
lucasagomes | dtantsur, done | 11:14 |
lucasagomes | vdrok, will add you too, so we are not a red hat only thing that can release it. Ok ? | 11:14 |
vdrok | lucasagomes: sure :) | 11:14 |
lucasagomes | done | 11:15 |
dtantsur | essentially the procedure is to push a signed tag. you can tell me what version to use and from which commit, if you don't want to mess with gpg | 11:15 |
dtantsur | lucasagomes, ^^^ | 11:16 |
lucasagomes | dtantsur, currently it's 0.3.0... so, I would say 0.4.0 | 11:16 |
vdrok | dtantsur: https://github.com/openstack/ironic-staging-drivers/commit/9de213f26f5825fed56d5c930340ea17fcf0042e this one I think | 11:16 |
lucasagomes | last commit: 9de213f26f5825fed56d5c930340ea17fcf0042e | 11:16 |
patchbot | 9de213f26f5825fed56d5c930340ea17fcf0042e | 11:16 |
dtantsur | doing now | 11:16 |
* dtantsur will do a few sanity checks first | 11:16 | |
lucasagomes | dtantsur, is there a script somewhere to release it ? (I think dhellmann have once pointed me to one) | 11:17 |
dtantsur | well, not any more, I think | 11:17 |
dtantsur | anyway, it's not too hard nowadays | 11:18 |
lucasagomes | dtantsur, docs at least ? | 11:18 |
dtantsur | git tag -s x.y.z && git push gerrit x.y.z :) | 11:19 |
lucasagomes | hah | 11:19 |
dtantsur | tag created, now waiting for publishing jobs | 11:20 |
dtantsur | check http://status.openstack.org/zuul/ (you can input ironic-staging in search field) | 11:20 |
*** bks has quit IRC | 11:21 | |
vdrok | dtantsur: awesome, thanks! | 11:21 |
sambetts | dtantsur, lucasagomes: the disadvantage of not using the script is the script throws a bunch of metadata in the tag message, which is used by infra jobs to generate things like the email annoucing the release | 11:22 |
dtantsur | sambetts, well, I think announcement for self-released projects are broken anyway | 11:22 |
lucasagomes | dtantsur, thanks | 11:22 |
dtantsur | I think the depend on openstack/release nowadays | 11:22 |
vdrok | yeah, from what I've read in docs thats ^ the case | 11:23 |
vdrok | we are not in openstack/releases anymore | 11:23 |
sambetts | I think that is just down to the release script reading that repo to create the metadata, you can manually inject the metadata and their publish jobs for that work from what I've seen | 11:24 |
*** jtomasek has joined #openstack-ironic | 11:26 | |
dtantsur | lucasagomes, vdrok, https://pypi.python.org/pypi/ironic-staging-drivers/0.4.0 | 11:26 |
lucasagomes | w00t | 11:26 |
vdrok | yup :) | 11:26 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Add new capabilities to ironic https://review.openstack.org/338138 | 11:34 |
*** vikrant has quit IRC | 11:37 | |
*** vikrant has joined #openstack-ironic | 11:43 | |
*** galyna has joined #openstack-ironic | 11:54 | |
*** Nisha_ has quit IRC | 11:54 | |
openstackgerrit | Sergii Nozhka proposed openstack/ironic: Added test for root_device hints functionality. https://review.openstack.org/404174 | 11:57 |
snozhka | dtantsur, lucasagomes hi guys! | 11:58 |
lucasagomes | snozhka, yo! How are you ? | 11:59 |
snozhka | lucasagomes, good, dude, how are you ? | 11:59 |
snozhka | lucasagomes, i've created review for root device hints, but it's not yet ready, i'll update it till eow | 12:00 |
dtantsur | hey snozhka | 12:00 |
*** aparnav has quit IRC | 12:00 | |
lucasagomes | snozhka, good good | 12:00 |
snozhka | lucasagomes, dtantsur created review to not loose changes due to laptop changes | 12:00 |
lucasagomes | snozhka, https://review.openstack.org/404156 | 12:01 |
patchbot | patch 404156 - tripleo-quickstart - Allow customizing the hard-drive Serial and/or WWN | 12:01 |
snozhka | dtantsur, hey! how are you ? | 12:01 |
dtantsur | cool! hanging out with mgould in Brno :) | 12:01 |
snozhka | dtantsur, cool! drink antoher beer this evening! | 12:02 |
snozhka | lucasagomes, cool!! i can +1 it! | 12:02 |
*** baoli has joined #openstack-ironic | 12:03 | |
lucasagomes | snozhka, I still have to figure out how to include virtio_scsi in tinyipa for testing it in the upstream ironic gate | 12:03 |
lucasagomes | but will do soon | 12:03 |
snozhka | lucasagomes, it's great! | 12:03 |
*** milan has quit IRC | 12:03 | |
*** milan has joined #openstack-ironic | 12:04 | |
*** baoli has quit IRC | 12:07 | |
jroll | vsaienk0: congrats! :) | 12:08 |
sambetts | lucasagomes: I did some reasearch regarding that and it seems like we might have to build the module from source | 12:08 |
lucasagomes | sambetts, hmm | 12:09 |
sambetts | lucasagomes: actually! Maybe not!!! | 12:10 |
sambetts | lucasagomes: oh :/ | 12:11 |
sambetts | hmmm | 12:11 |
*** anush has joined #openstack-ironic | 12:13 | |
*** tuanluong has quit IRC | 12:14 | |
sambetts | lucasagomes: virtio_scsi is missing from the tinycore package that includes alllllll the scsi drivers ;( | 12:17 |
sambetts | lucasagomes: http://distro.ibiblio.org/tinycorelinux/7.x/x86_64/tcz/scsi-4.2.9-tinycore64.tcz.list | 12:17 |
*** sborkows has quit IRC | 12:20 | |
lucasagomes | sambetts, and apparently their bug tracker is a forum heh (which needs a log in for searching) | 12:23 |
* lucasagomes register | 12:24 | |
*** anush has quit IRC | 12:24 | |
sambetts | lucasagomes: yeah... | 12:24 |
sambetts | lucasagomes: in the previous version of tinycore 6.x.x virtio_scsi is there | 12:26 |
sambetts | in the scsi package | 12:26 |
sambetts | http://distro.ibiblio.org/tinycorelinux/6.x/x86_64/tcz/scsi-3.16.6-tinycore64.tcz.list | 12:26 |
lucasagomes | ugh... I wonder why it's been removed | 12:26 |
sambetts | not sure, either that or someone forgot to compile it | 12:26 |
sambetts | XD | 12:26 |
vsaienk0 | jroll thanks! :) | 12:27 |
lucasagomes | sambetts, there's two posts about re-enabling it on 7.X for 64bits | 12:28 |
lucasagomes | now... apparently they didn't do that (yet ?) | 12:29 |
*** ohamada has quit IRC | 12:29 | |
sambetts | lucasagomes: http://tinycorelinux.net/7.x/x86_64/release/src/kernel/config-4.2.9-tinycore64 looking here I guess not | 12:34 |
lucasagomes | sambetts, indeed :-( | 12:35 |
dtantsur | vsaienk0, ping re https://review.openstack.org/#/c/357983/ | 12:35 |
patchbot | patch 357983 - ironic - Support names for {cleaning,provisioning}_network | 12:35 |
dtantsur | vsaienk0, do we expect cleaning_network to be a provider network always? | 12:36 |
lucasagomes | sambetts, yeah, apparently the solution won't be that easy | 12:36 |
sambetts | lucasagomes: most likely way for us to do it is to build the driver from source during build | 12:37 |
vsaienk0 | dtantsur: yes, as we should have access to server during cleaning | 12:37 |
lucasagomes | sambetts, ++ | 12:37 |
*** Nisha_ has joined #openstack-ironic | 12:38 | |
dtantsur | vsaienk0, so we can filter out only networks that are provider ones? they can be created only by admin, right? | 12:38 |
sambetts | just looking into how to modify the tinyipa build script to do it | 12:38 |
openstackgerrit | Markos Chandras proposed openstack/bifrost: Test OPNFV CI verify job. Please ignore. https://review.openstack.org/367608 | 12:39 |
nicodemos | good morning. | 12:40 |
*** vsaienk0 has left #openstack-ironic | 12:41 | |
lucasagomes | sambetts, with the source we probably just need a "make modules && make install" | 12:41 |
*** vsaienk0 has joined #openstack-ironic | 12:41 | |
lucasagomes | probably depmod -a as well | 12:41 |
lucasagomes | you are chroot with a fake uname right ? | 12:41 |
lucasagomes | chrooting* | 12:41 |
sambetts | lucasagomes: yup | 12:41 |
vsaienk0 | dtantsur: I thought we may search for networks that were created by user/tenant that ironic uses to talk to neutron https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L2032 | 12:42 |
dtantsur | vsaienk0, these are technical users | 12:42 |
dtantsur | actual networks will be created from admin/admin, not from ironic/service | 12:43 |
vsaienk0 | dtantsur, jroll: all ironic-multinode patches are merged https://review.openstack.org/#/c/404070/ if you do not mind we may set ironic-multinode job in non-voting mode for a while, and when we consider it as stable we may replace current ironic-multitenancy by ironic-multinode-multitenancy | 12:44 |
patchbot | patch 404070 - openstack-infra/project-config - Move ironic-multinode non-voting job to check queue | 12:44 |
jroll | vsaienk0: \o/ \o/ \o/ | 12:45 |
vsaienk0 | jroll: also I think we need a volunteer who can write test for takeover :) | 12:45 |
*** bfournie has quit IRC | 12:46 | |
dtantsur | also morning jroll | 12:46 |
jroll | vsaienk0: heh, yes we should do that :) | 12:46 |
jroll | morning! | 12:46 |
* jroll is busy with performance eval things this morning | 12:46 | |
dtantsur | vsaienk0, so, if we want to check users, we need to provide separate fields for user/tenant. I'd try the provider network approach first. | 12:47 |
*** vikrant has quit IRC | 12:48 | |
*** athomas has joined #openstack-ironic | 12:49 | |
*** athomas has quit IRC | 12:49 | |
*** athomas has joined #openstack-ironic | 12:50 | |
TheJulia | Good morning everyone! | 12:52 |
lucasagomes | TheJulia, morning | 12:52 |
*** lucasagomes is now known as lucas-hungry | 12:52 | |
dtantsur | morning TheJulia | 12:53 |
*** dprince has joined #openstack-ironic | 12:54 | |
openstackgerrit | Sam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation https://review.openstack.org/392959 | 12:54 |
sambetts | vsaienk0: updated ^ | 12:54 |
vdrok | morning jroll nicodemos and TheJulia | 12:55 |
vsaienk0 | sambetts: ok I will check patch today | 12:56 |
*** gcb has quit IRC | 12:56 | |
*** athomas has quit IRC | 12:57 | |
*** vikrant has joined #openstack-ironic | 13:00 | |
*** trown|outtypewww is now known as trown | 13:03 | |
*** baoli has joined #openstack-ironic | 13:03 | |
*** jtomasek has quit IRC | 13:04 | |
*** athomas has joined #openstack-ironic | 13:07 | |
*** baoli has quit IRC | 13:08 | |
*** athomas has quit IRC | 13:14 | |
*** athomas has joined #openstack-ironic | 13:15 | |
*** bfournie has joined #openstack-ironic | 13:19 | |
*** jcoufal has joined #openstack-ironic | 13:22 | |
*** jrist has joined #openstack-ironic | 13:23 | |
TheJulia | dtantsur: Looking at 336626. Any thoughts yet on the py35 job failures? | 13:24 |
dtantsur | TheJulia, no, I haven't (busy with other stuff right now). will do soon. | 13:26 |
openstackgerrit | Miles Gould proposed openstack/ironic: Add release names & numbers to API version history https://review.openstack.org/404211 | 13:28 |
*** hoangcx has joined #openstack-ironic | 13:28 | |
*** srobert has joined #openstack-ironic | 13:31 | |
*** jcoufal_ has joined #openstack-ironic | 13:31 | |
*** jcoufal has quit IRC | 13:34 | |
*** jtomasek has joined #openstack-ironic | 13:40 | |
*** ohamada has joined #openstack-ironic | 13:41 | |
*** yufei has quit IRC | 13:49 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Change node create notifications https://review.openstack.org/404224 | 13:50 |
*** lucas-hungry is now known as lucasagomes | 13:54 | |
*** Nisha_ has quit IRC | 13:56 | |
*** ralonsoh_ has joined #openstack-ironic | 13:56 | |
*** ralonsoh has quit IRC | 13:57 | |
*** jheroux has joined #openstack-ironic | 13:58 | |
sambetts | lucasagomes: tell a lie! as it turns out virtio_scsi is included in tinycore, but its compiled as a builtin not as a module | 13:58 |
sambetts | lucasagomes: so theory is no hackry required | 13:58 |
lucasagomes | sambetts, oh, I didn't see virtio_scsi=y config hmm | 13:59 |
sambetts | lucasagomes: backwards thing is in the .config file virtio_scsi is called CONFIG_SCSI_VIRTIO ... | 13:59 |
lucasagomes | ok, that's good news! | 13:59 |
sambetts | lucasagomes: yeah :D | 13:59 |
sambetts | lucasagomes: I only discovered it because I've added the lines required to compile the driver to the build script and it refused to build it into a module | 14:00 |
*** jkilpatr has quit IRC | 14:01 | |
lucasagomes | sambetts, hah | 14:02 |
lucasagomes | sambetts, idk if you saw but snozhka is already working on some tempest tests for root device hints | 14:03 |
* lucasagomes is building a devstack env for testing tcl | 14:03 | |
*** baoli has joined #openstack-ironic | 14:03 | |
sambetts | awesome :) I hope it works if we switch over to scsi, it seems like even with mainstream OSs though (your centos test a an example) don't play that nice with virtual disks | 14:04 |
pas-ha | yay, nova fix for postgres is merged! | 14:05 |
*** rloo has joined #openstack-ironic | 14:05 | |
lucasagomes | sambetts, yeah | 14:05 |
sambetts | pas-ha: oh!? thats a thing?! is that whats been causing the DBErrors I've been seeing? | 14:05 |
pas-ha | yes | 14:05 |
*** lindycoder has joined #openstack-ironic | 14:05 | |
sambetts | \o/ I thought I borked something | 14:06 |
pas-ha | in our postgres job that was | 14:06 |
sambetts | yeah | 14:06 |
*** Goneri has joined #openstack-ironic | 14:09 | |
*** glonlas has joined #openstack-ironic | 14:09 | |
*** glonlas has quit IRC | 14:10 | |
*** glonlas has joined #openstack-ironic | 14:11 | |
*** parlos has quit IRC | 14:15 | |
*** jkilpatr has joined #openstack-ironic | 14:16 | |
*** fxpester has joined #openstack-ironic | 14:18 | |
rloo | hi and good morning all you ironic'ers out there, lucasagomes, sambetts, pas-ha | 14:18 |
lucasagomes | good morning rloo | 14:19 |
*** daemontool has joined #openstack-ironic | 14:19 | |
*** fxpester|2 has joined #openstack-ironic | 14:20 | |
*** fxpester|3 has joined #openstack-ironic | 14:22 | |
*** fxpester has quit IRC | 14:23 | |
sambetts | hey rloo | 14:25 |
*** fxpester|2 has quit IRC | 14:25 | |
*** ElCoyote_ has joined #openstack-ironic | 14:26 | |
*** krtaylor has quit IRC | 14:28 | |
dtantsur | morning rloo | 14:28 |
rloo | hiya dtantsur | 14:28 |
*** srobert has quit IRC | 14:29 | |
openstackgerrit | Sam Betts proposed openstack/ironic: Add Virtual Network Interface Driver APIs https://review.openstack.org/327046 | 14:30 |
openstackgerrit | Sam Betts proposed openstack/ironic: Add Virtual Network Interface RPC APIs https://review.openstack.org/404239 | 14:30 |
openstackgerrit | Sam Betts proposed openstack/ironic: Add Virtual Network Interface REST APIs https://review.openstack.org/404240 | 14:30 |
sambetts | jroll: ^^ | 14:30 |
*** fxpester has joined #openstack-ironic | 14:32 | |
jroll | sambetts: nice! | 14:33 |
openstackgerrit | Merged openstack/ironic-inspector: Switch func3 environment to Python 3.5 https://review.openstack.org/404072 | 14:34 |
lucasagomes | sambetts, no dice setting the disk controller to "scsi" with tinyipa :-/ | 14:35 |
*** fxpester|3 has quit IRC | 14:35 | |
* lucasagomes investigates more | 14:35 | |
*** krtaylor has joined #openstack-ironic | 14:36 | |
sambetts | lucasagomes: :( does the virtio_scsi module get loaded? It should automatically right if a device of the right type is detected? | 14:36 |
lucasagomes | sambetts, yeah, maybe we need some udev rules to make it load | 14:36 |
lucasagomes | idk, I need to look deeper into it | 14:37 |
sambetts | lucasagomes: either that or we might need to inject a modprobe into the init script like we tried before | 14:37 |
sambetts | lucasagomes: need to check lsmod though to see what modules are being loaded | 14:37 |
lucasagomes | sambetts, indeed, you still have that patch up ? /me checks for it | 14:38 |
*** sborkows has joined #openstack-ironic | 14:38 | |
openstackgerrit | Sam Betts proposed openstack/ironic-python-agent: [WIP] Make sure we load virtio blk modules for testing in VMs https://review.openstack.org/402136 | 14:39 |
sambetts | lucasagomes: ^^ | 14:39 |
lucasagomes | sambetts, ty, I will configure a user so I can log in tinyipa using virsh console | 14:39 |
openstackgerrit | Miles Gould proposed openstack/ironic: Fix docs error about OOB RAID support https://review.openstack.org/404248 | 14:40 |
sambetts | lucasagomes: I updated it to call modprobe on virtio_scsi, but if its a builtin then it should load automagically I think | 14:40 |
lucasagomes | sambetts, that's my understanding as well... well let's see once I'm logged in I can play with it | 14:40 |
dtantsur | vsaienk0, do you have principal objections to me trying to limit networks to provider ones for https://review.openstack.org/357983 ? | 14:41 |
patchbot | patch 357983 - ironic - Support names for {cleaning,provisioning}_network | 14:41 |
*** rloo has quit IRC | 14:42 | |
*** vikrant has quit IRC | 14:42 | |
*** rloo has joined #openstack-ironic | 14:42 | |
sambetts | lucasagomes: tinyipa should autologin to the TC user | 14:43 |
*** moshele has quit IRC | 14:44 | |
vsaienk0 | dtantsur: might be add mention that network names are not uniq and if user set network name instead of uuid he is responsible for keep that name unique. | 14:44 |
dtantsur | vsaienk0, sure, that was my plan | 14:44 |
vsaienk0 | dtantsur: I'm fully ok with that :) | 14:45 |
*** moshele has joined #openstack-ironic | 14:46 | |
dtantsur | cool | 14:46 |
aarefiev | dtantsur: are you fine with manually enable|disable DHCP in case InfiniBand | 14:46 |
*** mrtenio has joined #openstack-ironic | 14:47 | |
aarefiev | https://review.openstack.org/#/c/264257/34/doc/source/usage.rst | 14:47 |
patchbot | patch 264257 - ironic-inspector - Adding InfiniBand Support | 14:47 |
* dtantsur needs to check | 14:50 | |
*** mtanino has joined #openstack-ironic | 14:51 | |
dtantsur | this barely sounds like "support" for infiniband... | 14:51 |
*** fxpester|2 has joined #openstack-ironic | 14:53 | |
milan | aarefiev, dtantsur, sambetts morning guys! :) The obligatory states-patch review appeal: https://review.openstack.org/#/c/348943/ | 14:53 |
patchbot | patch 348943 - ironic-inspector - Introducing node introspection state management | 14:53 |
lucasagomes | sambetts, right, yeah I don't get a shell... probably because we are executing the bootlocal.sh | 14:54 |
* lucasagomes modifies it | 14:54 | |
sambetts | lucasagomes: you should get a shell, if you hit enter it should appear, its just being swamped by the output of IPA | 14:55 |
*** parlos has joined #openstack-ironic | 14:55 | |
sambetts | lucasagomes: the first thing I do for testing is type "sudo killall python" to kill IPA | 14:55 |
sambetts | lucasagomes: then you should be able to do everything you need too | 14:55 |
*** fxpester has quit IRC | 14:56 | |
*** hshiina has quit IRC | 14:56 | |
*** trown is now known as trown|brb | 14:57 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add ironic resources CRUD notifications https://review.openstack.org/356541 | 14:58 |
*** glonlas_ has joined #openstack-ironic | 15:00 | |
*** glonlas_ has quit IRC | 15:00 | |
*** glonlas_ has joined #openstack-ironic | 15:00 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_connectors and volume_targets to task https://review.openstack.org/366196 | 15:01 |
*** glonlas has quit IRC | 15:03 | |
*** fxpester has joined #openstack-ironic | 15:05 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Check baremetal subcommands in OSC help output https://review.openstack.org/404258 | 15:05 |
*** jaosorior has quit IRC | 15:07 | |
*** fxpester|2 has quit IRC | 15:07 | |
*** jaosorior has joined #openstack-ironic | 15:08 | |
*** hoangcx has quit IRC | 15:11 | |
*** fxpester|2 has joined #openstack-ironic | 15:14 | |
*** trown|brb is now known as trown | 15:15 | |
*** fxpester|3 has joined #openstack-ironic | 15:17 | |
*** fxpester has quit IRC | 15:17 | |
*** glonlas_ has quit IRC | 15:18 | |
*** Nisha_ has joined #openstack-ironic | 15:18 | |
*** srobert has joined #openstack-ironic | 15:18 | |
*** fxpester|2 has quit IRC | 15:19 | |
*** fxpester has joined #openstack-ironic | 15:20 | |
*** glonlas has joined #openstack-ironic | 15:20 | |
*** glonlas has quit IRC | 15:20 | |
*** moshele has quit IRC | 15:20 | |
*** glonlas has joined #openstack-ironic | 15:21 | |
*** fxpester|3 has quit IRC | 15:22 | |
*** rajinir has joined #openstack-ironic | 15:26 | |
*** snozhka is now known as snozhka|afk | 15:26 | |
mariojv | yuriyz: hi - i agree with vdrok's comment on https://review.openstack.org/#/c/385459/ - if you want to take a look when you're free | 15:27 |
patchbot | patch 385459 - ironic - Fix stable states validation in FSM | 15:27 |
*** chlong has joined #openstack-ironic | 15:27 | |
mariojv | TheJulia: JayF: you may want to reconsider the +2s on that, based on the comment ^ | 15:27 |
mariojv | basically target state is being cleared where it might not be good to do that | 15:28 |
*** Nisha_ has quit IRC | 15:28 | |
mariojv | i think either the calling code should be fixed to make ironic not keep target state in error / enroll cases, or the code being modified should be changed to keep that existing behavior | 15:28 |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node maintenance notifications https://review.openstack.org/396239 | 15:30 |
JayF | mariojv: okie dokie | 15:31 |
TheJulia | yeouch | 15:31 |
mariojv | thanks | 15:31 |
TheJulia | yeah, then it can't know the idfference between manual or automated cleaning | 15:31 |
mariojv | yep | 15:31 |
mariojv | b/c CLEANWAIT | 15:31 |
mariojv | i'm surprised tests didn't get that | 15:32 |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node console notifications https://review.openstack.org/397812 | 15:33 |
*** fxpester|2 has joined #openstack-ironic | 15:34 | |
sambetts | lucasagomes: any luck? | 15:34 |
JayF | mariojv: making a comment that we should add a test to exercise that is a good idea | 15:34 |
TheJulia | yeah, I just basically said the same thing | 15:35 |
TheJulia | s/said/posted/ | 15:35 |
NobodyCam | Good Morning Ironic'ers | 15:35 |
*** fxpester has quit IRC | 15:36 | |
vdrok | morning mariojv JayF and NobodyCam | 15:37 |
JayF | o/ | 15:37 |
mariojv | morning vdrok | 15:37 |
*** syed__ has joined #openstack-ironic | 15:37 | |
NobodyCam | morning vdrok, JayF, mariojv | 15:37 |
*** fxpester has joined #openstack-ironic | 15:38 | |
vdrok | and good morning rloo | 15:38 |
*** Nisha_ has joined #openstack-ironic | 15:38 | |
lucasagomes | sambetts, not yet :-( | 15:38 |
sambetts | lucasagomes: :( | 15:38 |
lucasagomes | sambetts, I'm also helping debugging a deployment problem (downstream stuff) so I'm a bit slow | 15:39 |
*** wajdi has joined #openstack-ironic | 15:39 | |
dtantsur | morning mariojv, JayF, NobodyCam | 15:40 |
sambetts | lucasagomes: thats ok, is there a way to make ironic boot into IPA and stay there and then access the console? | 15:40 |
vdrok | rloo: re your comment about using CONF.fatal_exception_format_errors -- this says about formatting, while the problem that is solved is about ability to send the kwargs over rpc | 15:40 |
NobodyCam | morning dtantsur, lucasagomes, sambetts :) | 15:40 |
lucasagomes | sambetts, yeah, in a hacky way heh you can just kill the ir-conductor after the node goes into deploy wait :-) | 15:40 |
*** fxpester|2 has quit IRC | 15:40 | |
vdrok | rloo: in theory, there may be an object, that can not be json serialized, can not be converted to string, but can be converted to integer, and in the string format operation it is substituted as %d. in this case, formatting is OK, sending over rpc is not | 15:41 |
rloo | vdrok: yeah, i know it isn't exactly the same thing but i thought we could use it anyway. it is sort of similar if i stretch it. | 15:42 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Support names for {cleaning,provisioning}_network https://review.openstack.org/357983 | 15:42 |
sambetts | lucasagomes: does it work to put the node in maintenace then trigger manual cleaning? | 15:42 |
*** ricardoas has left #openstack-ironic | 15:42 | |
rloo | vdrok: the patch seems to indicate that it will always work / be serialized, no 500 error, yet the code has that new exception. | 15:42 |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Power off node in oneview https://review.openstack.org/360330 | 15:43 |
vdrok | rloo: there won't be 500 exception saying timed out waiting for response :) | 15:43 |
rloo | vdrok: oh, what happens when that exception is raised? | 15:43 |
*** soliosg has joined #openstack-ironic | 15:44 | |
vdrok | rloo: that what happens right now, if some methods tries to send non-serializable object in the exception from the conductor to api, conductor fails to serialize it, message is not sent, api times out waiting for response | 15:44 |
vdrok | rloo: when the new exception is raised, its contents are serializable 100% and the api gets the response correctly and displays it to the user | 15:45 |
vdrok | that exception is still 500 as it is programmer error in fact | 15:46 |
*** fxpester|2 has joined #openstack-ironic | 15:46 | |
vdrok | but it at least gives some info on what's actually wrong | 15:46 |
rloo | vdrok: ok, so the user will still see an exception. | 15:46 |
vdrok | yup | 15:46 |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Power off node in oneview https://review.openstack.org/360330 | 15:46 |
sambetts | lucasagomes: how do you connect to the VM to work on it, did you change anything to make that work? | 15:46 |
rloo | vdrok: at the very least, we should change the release notes | 15:46 |
*** fxpester|3 has joined #openstack-ironic | 15:48 | |
rloo | vdrok: also, since i didn't review it in detail so i'm going to ask. that exception -- are you including the original msg_fmt string w/o it being filled out? that could be useful (if we don't want to use that CONF to decide whether to raise exception or only spit out the msg-fmt string) | 15:48 |
vdrok | rloo: would changing the last sentence to "Starting with this release, it is checked that everything contained in an exception is serializable, otherwise it leads to InvalidExceptionContents." | 15:48 |
*** fxpester has quit IRC | 15:48 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Power off node in oneview https://review.openstack.org/360330 | 15:48 |
*** ricardoas has joined #openstack-ironic | 15:49 | |
rajinir | Zuul problem, need help: There is a change item in the queue that says "Unknown Project" and zuul is throwing exception. How do I fix it? | 15:49 |
*** vgadiraj has joined #openstack-ironic | 15:49 | |
sambetts | vdrok, rloo: I'd rather the user saw the error that they actually got even if the data is unserializable, e.g. Failed to boot node X because could not set flag *unserializable* seems more useful than "An exception was raised but it had unserializable data so we're not telling you want it was" | 15:49 |
*** bnemec has joined #openstack-ironic | 15:49 | |
rloo | sambetts: ++ | 15:50 |
vdrok | sambetts: rloo heh, the only problem with that, how to make it clear that unserializable is not the actual value :) | 15:50 |
*** fxpester|2 has quit IRC | 15:51 | |
vdrok | I'm not good at naming things, suggestions welcome :) | 15:51 |
*** vgadiraj has quit IRC | 15:51 | |
rajinir | Zuul: Is there a way to dequeue a pending job that has this "Unknown Project" problem so I don't have to restart Zuul | 15:52 |
vdrok | sambetts: rloo also in that case, we'd need to handle the parameter types, %d, %s, %f etc.? | 15:52 |
vdrok | as unserializable we won't be able to convert unserializable to string | 15:53 |
vdrok | ugh ^( | 15:53 |
vdrok | we won't be able to convert unserializable to integert | 15:53 |
rloo | vdrok, sambetts: i think for now, unless you think it is a hack, we could just clear kwargs. I think if we do that, it would end up going to L112: https://review.openstack.org/#/c/396355/8/ironic/common/exception.py | 15:53 |
patchbot | patch 396355 - ironic - Make all IronicExceptions RPC-serializable | 15:53 |
rloo | vdrok, sambetts: given that this is going to be a rare occurrence... | 15:54 |
sambetts | rloo. vdrok: I was just thinking that! we could just kill the kwarg, and then it'd be left as something like %{node_id}s or whatever | 15:54 |
jroll | rajinir: most people in here probably can't answer those questions, best ask #openstack-infra | 15:54 |
vdrok | rloo: so I think on the conductor side, we still need to at least log the error, on the api side, yeah, maybe we can just pass over whatever | 15:54 |
rajinir | jroll: Thank you | 15:55 |
rloo | vdrok: def, on either side, we should log the errors. | 15:55 |
sambetts | vdrok: yeah we totally should log it conductor side# | 15:55 |
*** baoli has quit IRC | 15:55 | |
lucasagomes | sambetts, hmm you probably can with manual cleaning yes | 15:55 |
rajinir | jroll: This is caused by openstack/ironic-staging-drivers project changes. FYI if you care | 15:55 |
vdrok | sambetts: rloo OK, that makes sense. updating. thanks for reviews! | 15:55 |
*** baoli has joined #openstack-ironic | 15:55 | |
jroll | rajinir: what changes? | 15:55 |
lucasagomes | sambetts, I connected via the virsh console, you have to edit the libvirt xml. Basically edit it to stop redirect the console to a file | 15:55 |
rajinir | https://review.openstack.org/#/c/397849/ | 15:56 |
patchbot | patch 397849 - ironic-staging-drivers - Remove agent vendor passthru from drivers | 15:56 |
lucasagomes | sambetts, the serial and console tags should look like: <serial type='pty' /> <console type='pty' /> | 15:56 |
lucasagomes | sambetts, that should do it | 15:56 |
rajinir | https://usercontent.irccloud-cdn.com/file/s4ugg9S6/ZuulStuck | 15:57 |
lucasagomes | sambetts, I also see we start the openssh server in the bootlocal.sh script, I haven't tried to ssh (will do) | 15:57 |
rajinir | jroll: See above | 15:58 |
jroll | rajinir: ah, perhaps ironic-staging-drivers isn't recognized by your zuul somehow? still recommend you ask -infra | 15:58 |
sambetts | lucasagomes: cool got it :) | 15:58 |
rajinir | jroll: Sure will do, ty | 15:58 |
jroll | np | 15:59 |
sambetts | lucasagomes: yeah managed to get it to hold IPA open by doing node manage -> node provide -> node maintainece = true | 15:59 |
*** madorn has quit IRC | 15:59 | |
lucasagomes | cool | 16:00 |
rloo | yuriyz: this needs to be approved first: https://review.openstack.org/#/c/404224/, before https://review.openstack.org/#/c/356541/? | 16:00 |
patchbot | patch 404224 - ironic-specs - Change node create notifications | 16:00 |
patchbot | patch 356541 - ironic - Add ironic resources CRUD notifications | 16:00 |
lucasagomes | sambetts, long ago I wanted to add an option in ironic to keep the node powered on in case the deployment failed (for debugging purpose) | 16:00 |
JayF | lucasagomes: I am pretty sure we landed that? | 16:00 |
lucasagomes | but I've abandoned the patch after some discussion | 16:00 |
JayF | lucasagomes: didn't I get a patch in that did just that? | 16:00 |
lucasagomes | JayF, did we ? /me checks | 16:01 |
JayF | lucasagomes: https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L995 | 16:01 |
lucasagomes | JayF, oh yeah indeed | 16:01 |
lucasagomes | https://review.openstack.org/#/c/259119/ | 16:01 |
patchbot | patch 259119 - ironic - Config to stop powering off nodes on failure (MERGED) | 16:01 |
lucasagomes | sambetts, ^] | 16:01 |
patchbot | Error: Spurious "]". You may want to quote your arguments with double quotes in order to prevent extra brackets from being evaluated as nested commands. | 16:01 |
sambetts | cool! | 16:02 |
*** strigazi_AFK is now known as strigazi | 16:02 | |
*** rcernin has quit IRC | 16:03 | |
lucasagomes | JayF, thanks, I totally forgot it | 16:03 |
JayF | np | 16:03 |
*** mjura has quit IRC | 16:03 | |
sambetts | lucasagomes: btw did you manage to get into a shell through the console? I normally test tinyIpa using an iso build, not through console so I'm a little lost haha | 16:03 |
*** jaosorior has quit IRC | 16:05 | |
*** anush has joined #openstack-ironic | 16:05 | |
*** chlong has quit IRC | 16:07 | |
*** glonlas_ has joined #openstack-ironic | 16:07 | |
*** glonlas has quit IRC | 16:10 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Migrate to keystone session glance and swift https://review.openstack.org/352289 | 16:11 |
jlvillal | vsaienk0: Thanks, I thought if someone knew about grenade, you would be a good possibility :) | 16:13 |
*** glonlas_ has quit IRC | 16:14 | |
*** aNuposic has joined #openstack-ironic | 16:15 | |
JayF | dtantsur: I'm thinking https://bugs.launchpad.net/ironic/+bug/1521197 is not a good bug (What ^^ that patch fixes) | 16:15 |
openstack | Launchpad bug 1521197 in Ironic "swift-related code needs refactoring" [Wishlist,In progress] - Assigned to Galyna Zholtkevych (gzholtkevych) | 16:15 |
JayF | dtantsur: basically don't we have one swift_tempurl_key that's supposed to be in the glance images account, and one that's supposed to be used for ironic to store stuff in swift? | 16:16 |
JayF | galyna: ^ | 16:16 |
*** rpioso has joined #openstack-ironic | 16:16 | |
* dtantsur does not remember | 16:16 | |
jroll | yes, those can be separate accounts IIRC | 16:17 |
*** glonlas has joined #openstack-ironic | 16:18 | |
jroll | (an account can only have one tempurl key | 16:18 |
jroll | ) | 16:18 |
*** vgadiraj has joined #openstack-ironic | 16:19 | |
JayF | That's what I thought, I put a -1 and a question here: https://review.openstack.org/#/c/352289/52 | 16:19 |
patchbot | patch 352289 - ironic - Migrate to keystone session glance and swift | 16:19 |
*** chlong has joined #openstack-ironic | 16:20 | |
openstackgerrit | Merged openstack/ironic: Remove file prefix parameter from lockutils methods https://review.openstack.org/383706 | 16:20 |
*** rbartal has quit IRC | 16:22 | |
lucasagomes | sambetts, yup, but I changed the bootsync.sh script :-/ | 16:23 |
*** sborkows has quit IRC | 16:23 | |
sambetts | lucasagomes: ah :/ | 16:24 |
*** e0ne_ has quit IRC | 16:24 | |
*** fxpester|3 has quit IRC | 16:28 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Move interface validation from API to conductor side https://review.openstack.org/401311 | 16:29 |
*** jschwarz has quit IRC | 16:32 | |
*** konetzed has joined #openstack-ironic | 16:33 | |
konetzed | Anyone have issues with the ubuntu liberty release of Iroinc getting stuck on reservations of nodes and never actually powering on the physical server? | 16:34 |
*** milan has quit IRC | 16:34 | |
*** jschwarz has joined #openstack-ironic | 16:35 | |
*** pcaruana has quit IRC | 16:36 | |
*** zackf has joined #openstack-ironic | 16:37 | |
JayF | konetzed: liberty is EOL and no longer supported. I don't know of any specific bugs relating to that, but you'll have much more success overall with a more recent release of OpenStack. | 16:37 |
*** vgadiraj has quit IRC | 16:38 | |
konetzed | JayF: tnx i figured that, it had been rock solid for us and I think with more users coming online we are hitting some goofy race condition. Every now and then the nodes just get stuck being reserved in db but never move any farther on the prov process | 16:39 |
*** mtani____ has joined #openstack-ironic | 16:39 | |
JayF | konetzed: it could be any one of hundreds of bugs we've fixed in the intervening year+ since it was released :) | 16:39 |
konetzed | I will work out upgrading to Mitaka | 16:40 |
JayF | Newton is released now fwiw :D | 16:40 |
*** vgadiraj has joined #openstack-ironic | 16:40 | |
JayF | although mitaka is still in support so that's clearly better than liberty | 16:40 |
konetzed | doh! lol ill work out Newton upgrade then | 16:40 |
*** mtanino has quit IRC | 16:40 | |
konetzed | Btw loving Ironic, lets us control ~100 machines for CI | 16:41 |
jroll | <3 | 16:41 |
lucasagomes | :-) | 16:41 |
jroll | konetzed: could be an unresponsive BMC, too, maybe | 16:41 |
jroll | or slow bmc | 16:42 |
jroll | there's a power sync loop that checks power status every (5 by default I think?) minutes | 16:42 |
konetzed | jroll: maybe slow bmc, i have hit it with a ipmitool status check and they are pretty responsive | 16:42 |
jroll | if a bmc takes minutes to respond, that could be it | 16:42 |
jroll | ah, hm | 16:42 |
* JayF wonders what the default value was for conductor worker threads durign liberty | 16:43 | |
JayF | might need a bump? | 16:43 |
JayF | I know at some point we bumped that default a bunch, to a more sane value | 16:43 |
konetzed | JayF: i did bump them up last night to see if it would solve the problem | 16:44 |
konetzed | Any known issues you guys run into with supermicro boxes? | 16:45 |
*** trown is now known as trown|lunch | 16:47 | |
lucasagomes | sambetts, btw, I've access to a tinyipa shell right, in case you want me to check anything | 16:47 |
jroll | konetzed: I haven't heard of many, other than IPMI being IPMI | 16:49 |
konetzed | thanks guys I will work out neuton upgrade :D | 16:50 |
*** mtanino has joined #openstack-ironic | 16:52 | |
*** mtani____ has quit IRC | 16:53 | |
*** rama_y has joined #openstack-ironic | 16:56 | |
*** hamzy has quit IRC | 17:02 | |
sambetts | lucasagomes: oh nice, the main thing to check is to make sure that the drivers are being loaded successfully or not, I did wonder if there were cases where it wasn't detecting the "hardware" correctly so wasn't loading the driver | 17:02 |
sambetts | lucasagomes: although I guess thats hard to check with builtin drivers right? because you can't just lsmod | 17:03 |
lucasagomes | sambetts, yeah static built drivers doesn't appear in the lsmod. We need to check the modules.builtin file (which is not present here idk why) | 17:03 |
lucasagomes | we can build one just running make modules in the source tho | 17:03 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Introduce generic hardware types https://review.openstack.org/400678 | 17:03 |
lucasagomes | sambetts, exactly | 17:03 |
lucasagomes | sambetts, I was thinking of rebuilding the tinyipa kernle with virtio_scsi=m | 17:03 |
lucasagomes | instead of static | 17:03 |
dtantsur | see you tomorrow | 17:04 |
*** dtantsur is now known as dtantsur|afk | 17:04 | |
sambetts | o/ dtantsur|afk | 17:04 |
lucasagomes | dtantsur|afk, see ya | 17:04 |
*** baoli has quit IRC | 17:05 | |
mrtenio | Hi Ironic. | 17:06 |
mrtenio | JayF hi, could I pm you? | 17:08 |
JayF | Usually it's better to talk in channel, so everyone can help and benefit from the discussion. | 17:08 |
JayF | If there's something that's required to be private -- like a security issue -- please file it in launchpad and mark it as private security. | 17:08 |
*** chlong has quit IRC | 17:10 | |
mrtenio | I understand. When you say to bump the required version, do you mean to add the information to the releasenote? Or do you mean to add that information to the requirements? Thank you. | 17:10 |
JayF | ah, I know what patch you're talkign about | 17:11 |
JayF | my confusion was this: it was listed as depending on a patch for the python-oneviewclient | 17:11 |
JayF | but looking at it, that fix to python-oneviewclient hasn't been released yet, and the driver-requirements.txt have not been updated in the ironic patch. | 17:11 |
sambetts | lucasagomes: perhaps looking at /sys/bus/virtio_scsi might tell us if things are being picked up or not? | 17:11 |
JayF | If it truly depended on the oneviewclient fixes, those steps would have to be done as well. | 17:12 |
*** anush has quit IRC | 17:12 | |
*** glonlas has quit IRC | 17:12 | |
*** baoli has joined #openstack-ironic | 17:12 | |
*** nicodemos has quit IRC | 17:14 | |
lucasagomes | sambetts, bash-4.3# ls /sys/bus/virtio_scsi | 17:15 |
lucasagomes | ls: cannot access /sys/bus/virtio_scsi: No such file or directory | 17:15 |
lucasagomes | sambetts, I'm recompiling the kernel with virtio_scsi as a module | 17:15 |
lucasagomes | I will add it to the ramdisk after and repack it | 17:16 |
*** nicodemos has joined #openstack-ironic | 17:16 | |
sambetts | lucasagomes: ah ok, that exists on mine | 17:16 |
mrtenio | That makes sense, it does depend on the new version. I am gonna update the requirements. Thank you for noticing it. | 17:17 |
JayF | mrtenio: no problem! That's what code reviews are for. Good luck1 | 17:17 |
sambetts | lucasagomes: /sys/bus/virtio/drivers/virtio_scsi | 17:17 |
sambetts | lucasagomes: my mistake | 17:17 |
lucasagomes | sambetts, oh yeah, that exist | 17:17 |
lucasagomes | bash-4.3# ls /sys/bus/virtio/drivers/virtio_scsi/ | 17:17 |
lucasagomes | bind uevent unbind | 17:17 |
sambetts | hmm | 17:19 |
*** anush has joined #openstack-ironic | 17:20 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Make all IronicExceptions RPC-serializable https://review.openstack.org/396355 | 17:20 |
*** zzzeek has quit IRC | 17:20 | |
*** jpich has quit IRC | 17:21 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Make all IronicExceptions RPC-serializable https://review.openstack.org/396355 | 17:23 |
vdrok | rloo: sambetts hopefully that makes more sense ^^ | 17:23 |
*** aNuposic has quit IRC | 17:24 | |
*** chlong has joined #openstack-ironic | 17:24 | |
*** zzzeek has joined #openstack-ironic | 17:24 | |
*** makowals_ has quit IRC | 17:25 | |
rloo | thx vdrok, will look in a few | 17:25 |
openstackgerrit | Stenio Araujo proposed openstack/ironic: Adds another validation step when using dynamic allocation https://review.openstack.org/377073 | 17:26 |
*** krtaylor has quit IRC | 17:29 | |
*** priteau has joined #openstack-ironic | 17:38 | |
*** daemontool has quit IRC | 17:39 | |
*** anush has quit IRC | 17:40 | |
sambetts | cool thanks vdrok | 17:40 |
*** penick has joined #openstack-ironic | 17:42 | |
*** zzzeek has quit IRC | 17:45 | |
lucasagomes | I'm calling it a day | 17:47 |
vdrok | rloo: sambetts oh, and by the way, it seems that we won't be able to say "portgroup.address is mandatory in >=1.28 but obligatory in 1.26-27" | 17:47 |
lucasagomes | have a great evening all | 17:47 |
vdrok | night lucasagomes | 17:47 |
pas-ha | bye lucasagomes | 17:47 |
rloo | bye lucasagomes | 17:47 |
lucasagomes | sambetts, I let you know tomorrow if I found anything new | 17:47 |
vdrok | as how it's done right now is mandatory=True in wsme attr definition | 17:47 |
lucasagomes | sambetts, (compiling the modules atm) | 17:47 |
vdrok | which is not api version aware | 17:48 |
sambetts | lucasagomes: thanks :) | 17:48 |
rloo | vdrok: ? what's the diff between mandatory & obligatory? | 17:48 |
lucasagomes | sambetts, if not, I will check if I need anything else... in any case I can enable things later in the menuconfig | 17:48 |
rloo | vdrok: I thought we decided it was optional | 17:48 |
vdrok | rloo: ouch :) s/obligatory/optional | 17:48 |
*** lucasagomes is now known as lucas-afk | 17:48 | |
rloo | vdrok: ha ha. | 17:48 |
vdrok | mandatory is used correctly :) | 17:48 |
*** zzzeek has joined #openstack-ironic | 17:48 | |
rloo | vdrok: so portgroup.address is optional in 1.26 & 1.27, but we're making it mandatory in >= 1.28? | 17:49 |
vdrok | or, mandatory in old microversion, optional in new | 17:49 |
vdrok | correct :D | 17:49 |
rloo | vdrok: maybe you should go home... relax... | 17:49 |
sambetts | lucasagomes: yeah, I've not had much experience with build kernels from source, so your probably have more luck than me | 17:49 |
vdrok | rloo: yeah, definitely a bad day for writing | 17:50 |
rloo | vdrok: heh. where is it that you want to say/write that, but cannot? | 17:50 |
*** dprince has quit IRC | 17:50 | |
*** dprince has joined #openstack-ironic | 17:50 | |
rloo | vdrok: or do you mean that we cannot make portgroups.address optional? | 17:51 |
vdrok | rloo: I'll try to be short | 17:51 |
vdrok | this place | 17:51 |
vdrok | https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/portgroup.py#L74 | 17:51 |
vdrok | wsme.attr is not microversion aware | 17:51 |
rloo | vdrok: oh. that's what you mean. | 17:51 |
*** derekh has quit IRC | 17:51 | |
*** anush has joined #openstack-ironic | 17:51 | |
rloo | vdrok: honestly, i don't care, if it isn't mandatory any more, it isn't going to break anyone | 17:52 |
vdrok | unless we hack the arguments on the fly, we won't be able to say mandatory=true if old microversion, mandatory=false if new | 17:52 |
vdrok | rloo: that's good, no hacks then :) | 17:52 |
rloo | vdrok: as far as i am concerned, it probably doesn't make sense for anyone to use portgroups until after 1.28 or 1.29, after you add the bonding stuff. | 17:52 |
*** parlos has quit IRC | 17:53 | |
rloo | vdrok: yeah, no need for hacks. unless someone can convince us otherwise. | 17:54 |
*** strigazi is now known as strigazi_AFK | 17:54 | |
*** aNuposic has joined #openstack-ironic | 17:57 | |
*** Romanenko_K has quit IRC | 17:57 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Move heartbeat processing to separate mixin class https://review.openstack.org/404364 | 17:59 |
*** ralonsoh_ has quit IRC | 18:02 | |
*** ijw has joined #openstack-ironic | 18:05 | |
*** vgadiraj has quit IRC | 18:09 | |
*** ohamada has quit IRC | 18:11 | |
*** soliosg has quit IRC | 18:13 | |
*** baoli has quit IRC | 18:13 | |
*** baoli has joined #openstack-ironic | 18:14 | |
*** soliosg has joined #openstack-ironic | 18:14 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Bifrost keystone follow-up https://review.openstack.org/404370 | 18:17 |
*** alineb has quit IRC | 18:18 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Bifrost keystone follow-up https://review.openstack.org/404370 | 18:20 |
*** glonlas has joined #openstack-ironic | 18:20 | |
*** glonlas has quit IRC | 18:20 | |
*** trown|lunch is now known as trown | 18:22 | |
*** glonlas has joined #openstack-ironic | 18:23 | |
*** crinkle has quit IRC | 18:23 | |
*** athomas has quit IRC | 18:25 | |
*** dprince has quit IRC | 18:25 | |
*** dprince has joined #openstack-ironic | 18:26 | |
openstackgerrit | Annie Lezil proposed openstack/ironic-inspector-specs: Network Interface Resource using Biosdevname https://review.openstack.org/404372 | 18:28 |
*** crinkle has joined #openstack-ironic | 18:28 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Test devstack-gate test-matrix.py changes https://review.openstack.org/404373 | 18:29 |
*** makowals has joined #openstack-ironic | 18:30 | |
*** e0ne has joined #openstack-ironic | 18:37 | |
*** vgadiraj has joined #openstack-ironic | 18:44 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add portgroup configuration fields https://review.openstack.org/398417 | 18:46 |
*** Nisha_brb has joined #openstack-ironic | 18:46 | |
*** krtaylor has joined #openstack-ironic | 18:46 | |
*** [1]cdearborn has joined #openstack-ironic | 18:48 | |
*** penick has quit IRC | 18:49 | |
*** Nisha_ has quit IRC | 18:50 | |
*** e0ne has quit IRC | 18:52 | |
*** penick has joined #openstack-ironic | 18:55 | |
*** athomas has joined #openstack-ironic | 18:55 | |
*** baoli has quit IRC | 18:57 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add portgroup configuration fields https://review.openstack.org/398417 | 18:58 |
*** cdearborn has quit IRC | 19:02 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add portgroup configuration fields https://review.openstack.org/398417 | 19:03 |
*** athomas has quit IRC | 19:03 | |
vdrok | good night everyone! | 19:04 |
nicodemos | night vdrok | 19:04 |
*** cdearborn has joined #openstack-ironic | 19:04 | |
*** Nisha_brb has quit IRC | 19:07 | |
*** hamzy has joined #openstack-ironic | 19:11 | |
*** baoli has joined #openstack-ironic | 19:11 | |
NobodyCam | night vdrok | 19:11 |
*** e0ne has joined #openstack-ironic | 19:12 | |
*** jkilpatr has quit IRC | 19:12 | |
e0ne | jroll: hi Jim, you told me at the Summit that you've got some issue using python-brick-cinderclient-ext with ironic | 19:13 |
jroll | e0ne: I don't quite recall, but I think maybe I wanted to be able to choose which IP gets passed to cinder (e.g. have a box with two nics, only one can reach the volume host) | 19:14 |
openstackgerrit | Annie Lezil proposed openstack/ironic-inspector-specs: Collect NIC name given by BIOS https://review.openstack.org/404372 | 19:15 |
e0ne | jroll: I've got it. I'll try to reproduce it and share update with you tomorrow | 19:16 |
jroll | e0ne: well, I hadn't tried it yet, was more curious what the behavior would be :) | 19:16 |
e0ne | jroll: I didn't try it with several NICs too | 19:17 |
*** athomas has joined #openstack-ironic | 19:17 | |
*** [1]cdearborn has quit IRC | 19:19 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Bifrost keystone follow-up https://review.openstack.org/404370 | 19:23 |
*** ijw has quit IRC | 19:28 | |
openstackgerrit | Martin Roy proposed openstack/ironic: DONOTMERGE-change IRONIC_DEPLOY_DRIVER to pxe_snmp https://review.openstack.org/388168 | 19:29 |
*** e0ne has quit IRC | 19:30 | |
openstackgerrit | Martin Roy proposed openstack/ironic: DONOTMERGE-change IRONIC_DEPLOY_DRIVER to agent_snmp https://review.openstack.org/388168 | 19:31 |
*** vsaienko has joined #openstack-ironic | 19:31 | |
*** penick has quit IRC | 19:32 | |
*** e0ne has joined #openstack-ironic | 19:32 | |
*** penick has joined #openstack-ironic | 19:34 | |
openstackgerrit | Martin Roy proposed openstack/ironic: Add virtualpdu to ironic devstack plugin https://review.openstack.org/388154 | 19:35 |
openstackgerrit | Merged openstack/bifrost: Add keystone installation playbook https://review.openstack.org/380225 | 19:43 |
*** ChubYann has joined #openstack-ironic | 19:50 | |
*** vsaienko has quit IRC | 19:55 | |
*** vsaienko has joined #openstack-ironic | 19:58 | |
*** syed__ is now known as syed_ | 19:59 | |
*** syed_ is now known as syed | 20:00 | |
*** syed is now known as Syed | 20:00 | |
jlvillal | rajinir: Get your own patch to do dell-recheck on! :P | 20:07 |
jlvillal | rajinir: I noticed you using my WIP/DNM patch for your rechecks :D | 20:08 |
rajinir | jlvillal: sorry, was trying to get the builds going that were blocked by a zuul bug | 20:08 |
jlvillal | rajinir: No worries | 20:08 |
jlvillal | rajinir: I just found it amusing. | 20:09 |
* jlvillal has a strange sense of humor. | 20:09 | |
rajinir | jlvillal: ;) | 20:10 |
jlvillal | rajinir: I was thinking, ooh someone reviewed my patch! Then, oh it is my WIP/DNM patch | 20:10 |
*** glonlas has quit IRC | 20:10 | |
*** Goneri has quit IRC | 20:11 | |
*** glonlas has joined #openstack-ironic | 20:15 | |
mrda | Morning Ironic | 20:17 |
NobodyCam | morning mrda :) | 20:18 |
mrda | NobodyCam: o/ | 20:18 |
NobodyCam | :) | 20:18 |
mrda | jlvillal: did you want to do the thing in 2 hours minus 20 minutes? Does that work for you? | 20:19 |
*** priteau has quit IRC | 20:20 | |
*** priteau has joined #openstack-ironic | 20:20 | |
*** baoli has quit IRC | 20:23 | |
openstackgerrit | Annie Lezil proposed openstack/ironic-python-agent: Trying to create a package for biosdevname https://review.openstack.org/396406 | 20:23 |
*** baoli has joined #openstack-ironic | 20:23 | |
*** priteau has quit IRC | 20:26 | |
*** priteau has joined #openstack-ironic | 20:26 | |
*** vsaienko has quit IRC | 20:28 | |
*** [1]cdearborn has joined #openstack-ironic | 20:30 | |
*** vsaienko has joined #openstack-ironic | 20:30 | |
JayF | Is there anyone other than dtantsur|afk who approves/looks at inspector RFEs? | 20:33 |
sambetts | JayF: I try to look at them, but I've been off the ball recently | 20:34 |
JayF | sambetts: I know rama_y is working on https://bugs.launchpad.net/ironic-inspector/+bug/1635057 but feedback sorta seems to have slowed down | 20:35 |
openstack | Launchpad bug 1635057 in ironic-python-agent "[RFE] Collect processor, memory and bios info using dmidecode" [Wishlist,Incomplete] - Assigned to Ramamani Yeleswarapu (ramamani-yeleswarapu) | 20:35 |
JayF | and I think the implementation is like, 75%+ done so any attention is appreciated | 20:36 |
*** vsaienko has quit IRC | 20:37 | |
sambetts | JayF: sure I've added a note to look at it tomorrow | 20:37 |
rama_y | JayF, sambetts: thanks for looking at my RFE. | 20:37 |
JayF | Thanks Sam, any feedback is appreciated \o/ | 20:37 |
*** vsaienko has joined #openstack-ironic | 20:38 | |
*** priteau has quit IRC | 20:38 | |
* sambetts -> home | 20:38 | |
sambetts | cy'all tomorrow | 20:39 |
*** priteau has joined #openstack-ironic | 20:39 | |
*** sambetts is now known as sambetts|afk | 20:39 | |
*** vgadiraj has quit IRC | 20:41 | |
JayF | o/ | 20:41 |
*** cdearborn has quit IRC | 20:42 | |
*** priteau has quit IRC | 20:46 | |
*** priteau has joined #openstack-ironic | 20:46 | |
NobodyCam | night sambetts|afk | 20:47 |
*** vsaienko has quit IRC | 20:49 | |
jlvillal | mrda, Sure | 20:52 |
openstackgerrit | Annie Lezil proposed openstack/ironic-python-agent: Trying to create a package for biosdevname https://review.openstack.org/396406 | 20:54 |
jroll | JayF: aarefiev and milan are also good people to bug about those | 20:54 |
openstackgerrit | Joanna Taryma proposed openstack/ironic: Fixed allowing non-uuids as node UUID for port and portgroup. https://review.openstack.org/403910 | 20:55 |
openstackgerrit | Martin Roy proposed openstack/ironic: Add virtualpdu to ironic devstack plugin https://review.openstack.org/388154 | 21:03 |
*** Goneri has joined #openstack-ironic | 21:04 | |
*** rbudden has joined #openstack-ironic | 21:06 | |
*** ijw has joined #openstack-ironic | 21:06 | |
*** MattMan has quit IRC | 21:07 | |
*** MattMan has joined #openstack-ironic | 21:07 | |
*** ijw has quit IRC | 21:07 | |
*** ijw has joined #openstack-ironic | 21:08 | |
openstackgerrit | Stenio Araujo proposed openstack/ironic: Adds another validation step when using dynamic allocation https://review.openstack.org/377073 | 21:09 |
*** srobert has quit IRC | 21:13 | |
*** jschwarz has quit IRC | 21:16 | |
*** jcoufal_ has quit IRC | 21:17 | |
*** jschwarz has joined #openstack-ironic | 21:17 | |
*** jkilpatr has joined #openstack-ironic | 21:19 | |
*** anush has quit IRC | 21:21 | |
jroll | dtantsur|afk: tomorrow I'd like your review on https://review.openstack.org/#/c/404422/1 - see also the comments here: https://review.openstack.org/#/c/355586/ | 21:22 |
patchbot | patch 404422 - openstack-infra/project-config - Load ironic tempest plugin for ironic-inspector | 21:22 |
patchbot | patch 355586 - tempest - Removing baremetal tests from tempest tree | 21:22 |
*** penick has quit IRC | 21:25 | |
*** penick has joined #openstack-ironic | 21:25 | |
*** krtaylor has quit IRC | 21:26 | |
*** jschwarz has quit IRC | 21:26 | |
*** jschwarz has joined #openstack-ironic | 21:27 | |
*** penick has quit IRC | 21:30 | |
*** e0ne has quit IRC | 21:31 | |
*** penick has joined #openstack-ironic | 21:35 | |
mariojv | aNuposic: hey, i left some comments on your RFE: https://bugs.launchpad.net/ironic/+bug/1642719 | 21:41 |
openstack | Launchpad bug 1642719 in Ironic "[RFE] RADOSGW authentication mechanism for storing config drive in Ceph" [Wishlist,New] - Assigned to Anup (anup-d-navare) | 21:41 |
*** vsaienko has joined #openstack-ironic | 21:41 | |
mariojv | aNuposic: i'm not sure if you need a separate RFE for the actual ceph support, or if they can both be taken care of in that one | 21:41 |
JayF | oh I'll have a look at that too | 21:43 |
*** e0ne has joined #openstack-ironic | 21:46 | |
*** e0ne has quit IRC | 21:47 | |
mariojv | i'm not sure if there's anywhere else that ceph support is going to be added | 21:47 |
mariojv | would be good to clarify and get it out of the way early, though | 21:47 |
aNuposic | mariojv: we already have ceph support | 21:47 |
mariojv | iirc we currently support downloading the ramdisk for ceph, maybe | 21:47 |
mariojv | *from ceph | 21:47 |
aNuposic | so this RFE mainly is focused on different authentication and config drive in Ceph | 21:48 |
JayF | mariojv: no | 21:48 |
JayF | mariojv: oh, you're right -- we support ceph temp urls | 21:48 |
aNuposic | mariojv: so did you meant a separate RFE for config drive in ceph? | 21:48 |
JayF | mariojv: for downloading ramdisks; that's it | 21:49 |
jroll | for ramdisks or for images? | 21:49 |
mariojv | oh, just for tempurls http://docs.openstack.org/developer/ironic/deploy/radosgw.html | 21:49 |
mariojv | yep | 21:49 |
*** ijw has quit IRC | 21:49 | |
mariojv | images i think | 21:49 |
*** glonlas has quit IRC | 21:49 | |
mariojv | it's actually used as a glance backend it seems | 21:49 |
JayF | oh, that's it | 21:49 |
JayF | I'm thinking ramdisk/agent because that's where the tempurl is sent | 21:50 |
mariojv | right | 21:50 |
aNuposic | yeah glance has the support for ceph | 21:50 |
*** ijw has joined #openstack-ironic | 21:50 | |
aNuposic | i believe temp urls are used for fetching instance images | 21:50 |
mariojv | aNuposic: feel free to ping me in irc in this channel when the RFE gets updated | 21:51 |
mariojv | thanks | 21:51 |
mariojv | aNuposic: that's correct | 21:51 |
mariojv | some people use the term "deploy images" but it's the same thing | 21:51 |
aNuposic | mariojv: Sure thanks. I will comment on the RFE | 21:51 |
*** jkilpatr has quit IRC | 21:53 | |
*** ijw has quit IRC | 21:55 | |
openstackgerrit | Joanna Taryma proposed openstack/ironic: Fixed allowing non-uuids as node UUID for port and portgroup. https://review.openstack.org/403910 | 21:57 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Move CONF.service_available.ironic to our plugin https://review.openstack.org/404436 | 21:58 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: DNM: Testing 355586 https://review.openstack.org/397775 | 21:59 |
jroll | ^ the chain ending at 404436 should be ready assuming qa team is happy with 355586, have a W-1 in case | 21:59 |
*** trown is now known as trown|outtypewww | 22:00 | |
*** priteau has quit IRC | 22:08 | |
*** jtomasek has quit IRC | 22:08 | |
rloo | does anyone know what/if there is anything wrong with the gate? i'm seeing a few failures in various patches | 22:08 |
*** jrist has quit IRC | 22:08 | |
rloo | mariojv: this is done, right? http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/notifications.html | 22:11 |
*** chlong has quit IRC | 22:12 | |
mariojv | rloo: yup! | 22:14 |
JayF | rloo: no problems afaik, if you want to link me one of the patches I can dig in some | 22:14 |
rloo | JayF: https://review.openstack.org/#/c/285220/ | 22:15 |
patchbot | patch 285220 - ironic - Add RPCs to support volume target operations | 22:15 |
*** penick has quit IRC | 22:15 | |
*** rbudden has quit IRC | 22:15 | |
JayF | grenade /o\ | 22:15 |
rloo | JayF: this one is really bad: https://review.openstack.org/#/c/396355/ | 22:15 |
patchbot | patch 396355 - ironic - Make all IronicExceptions RPC-serializable | 22:15 |
*** penick has joined #openstack-ironic | 22:16 | |
rloo | JayF: I'm sure ^^ is transient. or some of it :) | 22:16 |
*** lindycoder has quit IRC | 22:16 | |
*** penick has quit IRC | 22:17 | |
*** penick_ has joined #openstack-ironic | 22:17 | |
*** athomas has quit IRC | 22:17 | |
jroll | if a patch fails pep8, unit tests, and some of the integration tests, I assume it's the patch :P | 22:18 |
*** wajdi has quit IRC | 22:18 | |
* jroll looks at it anyway | 22:18 | |
jroll | 2016-11-29 17:37:23.180650 | ConnectTimeout: HTTPConnectionPool(host='mirror.regionone.osic-cloud1.openstack.org', port=80): Max retries exceeded with url: | 22:18 |
jroll | aha | 22:18 |
rloo | jroll: transient. we hope! | 22:19 |
JayF | http://logs.openstack.org/20/285220/43/check/gate-grenade-dsvm-ironic/892560d/logs/grenade.sh.txt.gz#_2016-11-29_18_11_11_906 | 22:19 |
JayF | this failure is more interesting | 22:19 |
JayF | tempest smoke failed in the old/ side of grenade due to SecurityGroupInUse | 22:19 |
JayF | but since the old/ side failed, it can't be the patch, maybe something landed in stable/newton to make it unhappy? | 22:19 |
rloo | jroll: something in neutron? | 22:20 |
rloo | jroll, sorry , i meant the other JayF | 22:20 |
jroll | heh | 22:20 |
JayF | rloo: or tempest /me peers at jroll | 22:21 |
jroll | I didn't do it | 22:21 |
rloo | JayF: we didn't backport the security changes, did we? | 22:21 |
jroll | I see other things passing grenade that ran after that one so ¯\_(ツ)_/¯ | 22:21 |
JayF | nothing has landed in ironic stable/newton for almost a week | 22:22 |
JayF | yeah; I suspect it's transient | 22:22 |
rloo | jroll: YUCK. things should fail all the time if they want to fail. | 22:22 |
JayF | I just try to dig anyway so I know what it looks like | 22:22 |
JayF | so if it happens again I can lose days of my life finding it | 22:22 |
* jroll looks at the clock and decides to see if it's still transient in the morning | 22:22 | |
jroll | see y'all tomorrow! | 22:22 |
rloo | bye jroll | 22:23 |
rloo | thx JayF and jroll for looking! | 22:23 |
JayF | I'll always look at gate screwiness, it's all production troubleshooting without the stress | 22:23 |
JayF | lol | 22:23 |
rloo | JayF: i still haven't forgotten, you still owe us a tutorial... | 22:23 |
JayF | I can give it off the cuff whenever folks want to recieve it :) | 22:24 |
* jroll thinks JayF needs a blog | 22:24 | |
rloo | JayF: heh. I'm waiting for the crowds to get restless first! :) | 22:25 |
* jroll really disappears | 22:25 | |
JayF | rloo: yeah I can't find anything related to that failure, if it happens again throw up the jay-signal and I'll respond :P | 22:25 |
rloo | JayF: thx. I'll wait til tomorrow to see if it is still occurring. | 22:25 |
*** chas has joined #openstack-ironic | 22:30 | |
*** rcernin has joined #openstack-ironic | 22:31 | |
*** chas has quit IRC | 22:34 | |
*** dprince has quit IRC | 22:35 | |
*** jschwarz has quit IRC | 22:39 | |
*** jschwarz has joined #openstack-ironic | 22:40 | |
*** jheroux has quit IRC | 22:45 | |
*** ijw has joined #openstack-ironic | 22:51 | |
*** baoli has quit IRC | 22:52 | |
*** aNuposic has quit IRC | 22:52 | |
*** ijw has quit IRC | 22:54 | |
*** ijw has joined #openstack-ironic | 22:55 | |
*** slagle has quit IRC | 22:57 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Trying to test grenade multi-tenant experimental job https://review.openstack.org/403895 | 22:58 |
*** lindycoder has joined #openstack-ironic | 23:01 | |
*** bfournie has quit IRC | 23:01 | |
*** rcernin has quit IRC | 23:01 | |
*** bfournie has joined #openstack-ironic | 23:02 | |
*** bfournie has quit IRC | 23:02 | |
*** rcernin has joined #openstack-ironic | 23:03 | |
*** rcernin has quit IRC | 23:04 | |
*** lindycoder has quit IRC | 23:04 | |
*** rcernin has joined #openstack-ironic | 23:04 | |
*** lindycoder has joined #openstack-ironic | 23:04 | |
*** ijw has quit IRC | 23:13 | |
*** yuanying has joined #openstack-ironic | 23:16 | |
*** ijw has joined #openstack-ironic | 23:17 | |
*** aNuposic has joined #openstack-ironic | 23:18 | |
*** aNuposic has quit IRC | 23:18 | |
*** soliosg has quit IRC | 23:22 | |
*** ijw has quit IRC | 23:22 | |
*** [1]cdearborn has quit IRC | 23:23 | |
*** wajdi has joined #openstack-ironic | 23:30 | |
jlvillal | So have we EOL the Liberty release for openstack/ironic? | 23:32 |
*** bfournie has joined #openstack-ironic | 23:33 | |
jlvillal | Okay, looks like we will be: https://gist.github.com/tbreeds/93cd346c37aa46269456f56649f0a4ac#file-liberty_eol_data-txt-L1 | 23:34 |
*** pmannidi has joined #openstack-ironic | 23:35 | |
*** lindycoder has quit IRC | 23:37 | |
*** rcernin has quit IRC | 23:41 | |
*** wajdi has quit IRC | 23:45 | |
jlvillal | JayF: If you can look at: https://review.openstack.org/#/c/404479/ it would be appreciated. | 23:48 |
patchbot | patch 404479 - openstack-infra/project-config - Disable devstack plugin for builder 'ironic-grenade' | 23:48 |
*** vsaienko has quit IRC | 23:56 | |
*** ijw has joined #openstack-ironic | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!