konetzed | Thanks for the help. I think I will take a serious look at liberty release and swing in tomorrow AM to ask my question again. See if cant get some of those EU guys to answer :D | 00:02 |
---|---|---|
*** david-lyle has quit IRC | 00:02 | |
*** david-lyle has joined #openstack-ironic | 00:03 | |
*** rama_y has quit IRC | 00:06 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/302853 | 00:10 |
*** sukhdev has quit IRC | 00:12 | |
*** saripurigopi has joined #openstack-ironic | 00:12 | |
JayF | NobodyCam: lucasagomes: FWIW, I'm very okay with https://review.openstack.org/#/c/291427 going in without a bug number, despite my comment on the patch, especially given the wide consensus. | 00:15 |
*** shakamunyi has joined #openstack-ironic | 00:18 | |
*** garthb has quit IRC | 00:21 | |
*** rbudden has quit IRC | 00:22 | |
*** rbudden has joined #openstack-ironic | 00:23 | |
*** shakamunyi has quit IRC | 00:23 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/302851 | 00:27 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/302852 | 00:27 |
*** spandhe has quit IRC | 00:28 | |
*** jayc has quit IRC | 00:29 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/302879 | 00:32 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/302880 | 00:32 |
openstackgerrit | Marcellin Fom Tchassem proposed openstack/ironic-inspector: Make tox respect upper-constraints.txt https://review.openstack.org/301700 | 00:33 |
*** jayc has joined #openstack-ironic | 00:41 | |
*** jjohnson2 has quit IRC | 00:41 | |
*** rbudden has quit IRC | 00:45 | |
*** hoangcx_ has joined #openstack-ironic | 00:49 | |
*** hoangcx has quit IRC | 00:49 | |
*** hoangcx_ is now known as hoangcx | 00:49 | |
*** cdearborn has joined #openstack-ironic | 00:52 | |
*** amotoki has quit IRC | 00:56 | |
*** spandhe has joined #openstack-ironic | 01:01 | |
*** smoriya_ has joined #openstack-ironic | 01:06 | |
*** spandhe has quit IRC | 01:08 | |
*** [1]cdearborn has joined #openstack-ironic | 01:09 | |
*** penick has quit IRC | 01:16 | |
jlvillal | lintan, ping? | 01:17 |
jlvillal | lintan, It would be great to get your input on: https://review.openstack.org/303048 Thanks | 01:18 |
*** shakamunyi has joined #openstack-ironic | 01:18 | |
*** amotoki has joined #openstack-ironic | 01:22 | |
*** shakamunyi has quit IRC | 01:23 | |
*** cdearborn has quit IRC | 01:24 | |
*** davidlenwell has quit IRC | 01:28 | |
*** amotoki has quit IRC | 01:29 | |
*** [1]cdearborn has quit IRC | 01:30 | |
*** davidlenwell has joined #openstack-ironic | 01:32 | |
*** baoli has joined #openstack-ironic | 01:32 | |
*** Haomeng|2 has joined #openstack-ironic | 01:35 | |
*** saripurigopi has quit IRC | 01:36 | |
*** baoli has quit IRC | 01:38 | |
*** saripurigopi has joined #openstack-ironic | 01:40 | |
*** shu-mutou has joined #openstack-ironic | 01:46 | |
*** baoli has joined #openstack-ironic | 01:47 | |
lintan | OK, let me check jivillal | 01:48 |
*** baoli has quit IRC | 01:52 | |
openstackgerrit | Tan Lin proposed openstack/ironic: Add nodeinfo_list() to Node Object https://review.openstack.org/290313 | 02:00 |
*** baoli has joined #openstack-ironic | 02:00 | |
*** saripurigopi has quit IRC | 02:03 | |
*** baoli has quit IRC | 02:07 | |
*** baoli has joined #openstack-ironic | 02:08 | |
*** davideag_ has joined #openstack-ironic | 02:13 | |
*** mtanino__ has joined #openstack-ironic | 02:15 | |
*** suro-patz has joined #openstack-ironic | 02:15 | |
*** Sukhdev_ has quit IRC | 02:15 | |
*** davideagnello has quit IRC | 02:15 | |
*** jtomasek has quit IRC | 02:15 | |
*** mtanino has quit IRC | 02:15 | |
*** baoli has quit IRC | 02:16 | |
*** jaybeale has quit IRC | 02:17 | |
*** baoli has joined #openstack-ironic | 02:17 | |
*** amotoki has joined #openstack-ironic | 02:17 | |
*** jaybeale has joined #openstack-ironic | 02:18 | |
*** vishwana_ has quit IRC | 02:18 | |
*** shakamunyi has joined #openstack-ironic | 02:19 | |
*** openstackstatus has quit IRC | 02:19 | |
*** aginwala has quit IRC | 02:19 | |
*** vishwanathj has joined #openstack-ironic | 02:19 | |
*** JieLi has joined #openstack-ironic | 02:22 | |
*** shakamunyi has quit IRC | 02:24 | |
*** baoli has quit IRC | 02:27 | |
*** aginwala has joined #openstack-ironic | 02:29 | |
openstackgerrit | Merged openstack/bifrost: Updated from global requirements https://review.openstack.org/302841 | 02:29 |
*** suro-patz has quit IRC | 02:35 | |
*** jaybeale has quit IRC | 02:36 | |
*** aginwala has quit IRC | 02:39 | |
*** JieLi has quit IRC | 02:39 | |
*** suro-patz has joined #openstack-ironic | 02:42 | |
*** yuanying has quit IRC | 02:46 | |
*** suro-patz has quit IRC | 02:50 | |
*** aginwala has joined #openstack-ironic | 02:52 | |
*** Marga_ has quit IRC | 02:53 | |
*** aginwala has quit IRC | 02:59 | |
*** Goneri has joined #openstack-ironic | 03:04 | |
openstackgerrit | Tan Lin proposed openstack/ironic: Make use of oslo-config-generator https://review.openstack.org/247331 | 03:17 |
*** shakamunyi has joined #openstack-ironic | 03:18 | |
jlvillal | lintan, If you would like to make those changes and add yourself as a co-author, feel free :) | 03:18 |
jlvillal | Otherwise I could work on it tomorrow. | 03:19 |
jlvillal | lintan, Thanks for the review! | 03:19 |
*** shakamunyi has quit IRC | 03:23 | |
openstackgerrit | Merged openstack/ironic: Adjust ipminative._reboot to comply with pyghmi contract https://review.openstack.org/264990 | 03:24 |
openstackgerrit | Dao Cong Tien proposed openstack/ironic-specs: Nova serial console support https://review.openstack.org/296869 | 03:26 |
*** Marga_ has joined #openstack-ironic | 03:27 | |
lintan | OK, I can update this :) | 03:31 |
*** Marga_ has quit IRC | 03:31 | |
*** Sukhdev has joined #openstack-ironic | 03:34 | |
*** Goneri has quit IRC | 03:37 | |
*** Marga_ has joined #openstack-ironic | 03:45 | |
openstackgerrit | Tan Lin proposed openstack/ironic: Prepare for transition to oslo-config-generator https://review.openstack.org/303048 | 03:46 |
*** Sukhdev has quit IRC | 03:47 | |
*** yuanying has joined #openstack-ironic | 03:49 | |
*** links has joined #openstack-ironic | 03:53 | |
*** saripurigopi has joined #openstack-ironic | 04:04 | |
*** harlowja_at_home has joined #openstack-ironic | 04:13 | |
*** Sukhdev has joined #openstack-ironic | 04:16 | |
*** keedya has quit IRC | 04:16 | |
*** garthb has joined #openstack-ironic | 04:37 | |
*** harlowja_at_home has quit IRC | 04:39 | |
*** aginwala has joined #openstack-ironic | 04:39 | |
*** mtanino__ has quit IRC | 04:48 | |
*** UForgotten has joined #openstack-ironic | 04:50 | |
*** rcernin has joined #openstack-ironic | 04:55 | |
*** alex_xu has quit IRC | 04:56 | |
*** shu-mutou is now known as shu-mutou-AFK | 04:57 | |
*** UForgotten has quit IRC | 04:59 | |
*** UForgotten has joined #openstack-ironic | 04:59 | |
*** stevebaker has joined #openstack-ironic | 05:01 | |
*** alex_xu has joined #openstack-ironic | 05:01 | |
*** garthb has quit IRC | 05:02 | |
openstackgerrit | Tan Lin proposed openstack/ironic: Prepare for transition to oslo-config-generator https://review.openstack.org/303048 | 05:06 |
*** links has quit IRC | 05:07 | |
*** rbrud has quit IRC | 05:12 | |
*** ChubYann has quit IRC | 05:18 | |
*** shakamunyi has joined #openstack-ironic | 05:18 | |
*** shakamunyi has quit IRC | 05:23 | |
*** links has joined #openstack-ironic | 05:24 | |
*** moshele has joined #openstack-ironic | 05:27 | |
*** suro-patz has joined #openstack-ironic | 05:40 | |
*** marios has joined #openstack-ironic | 05:46 | |
*** aginwala has quit IRC | 05:53 | |
*** Sukhdev has quit IRC | 05:54 | |
*** mvpnitesh has quit IRC | 05:56 | |
*** mvpnitesh has joined #openstack-ironic | 05:56 | |
*** saripurigopi has quit IRC | 05:58 | |
*** saripurigopi has joined #openstack-ironic | 05:58 | |
*** saripurigopi has quit IRC | 06:02 | |
*** links has quit IRC | 06:03 | |
*** amotoki has quit IRC | 06:06 | |
*** shakamunyi has joined #openstack-ironic | 06:18 | |
*** links has joined #openstack-ironic | 06:19 | |
*** shakamunyi has quit IRC | 06:23 | |
*** ndipanov has joined #openstack-ironic | 06:29 | |
*** amotoki has joined #openstack-ironic | 06:29 | |
*** amotoki has quit IRC | 06:33 | |
*** amotoki has joined #openstack-ironic | 06:33 | |
*** fangfenghua has joined #openstack-ironic | 06:44 | |
*** fangfenghua has left #openstack-ironic | 06:44 | |
*** ndipanov has quit IRC | 06:47 | |
*** amotoki has quit IRC | 06:52 | |
*** yuanying has quit IRC | 06:57 | |
*** praneshp_ has quit IRC | 07:01 | |
*** mrda is now known as mrda_away | 07:01 | |
*** suro-patz has quit IRC | 07:01 | |
*** e0ne has joined #openstack-ironic | 07:01 | |
*** jtomasek_ has joined #openstack-ironic | 07:03 | |
*** jtomasek_ has quit IRC | 07:04 | |
*** amotoki has joined #openstack-ironic | 07:05 | |
*** dtantsur|afk is now known as dtantsur | 07:12 | |
dtantsur | Morning Ironic | 07:14 |
dtantsur | konetzed, hi! I work on ironic-discoverd, feel free to ask your questions :) ironic-discoverd did work in kilo for me, and I still support it internally | 07:14 |
dtantsur | (not it's called ironic-inspector, but it's only compatible with liberty and newer) | 07:15 |
dtantsur | * now | 07:15 |
dtantsur | konetzed, I'd also recommend considering liberty, if possible. ironic-inspector got a lot of improvements, and so did ironic. | 07:17 |
*** shakamunyi has joined #openstack-ironic | 07:18 | |
*** shakamunyi has quit IRC | 07:23 | |
*** ndipanov has joined #openstack-ironic | 07:28 | |
*** alex_xu_ has joined #openstack-ironic | 07:32 | |
*** alex_xu has quit IRC | 07:34 | |
*** jtomasek has joined #openstack-ironic | 07:38 | |
*** ifarkas has joined #openstack-ironic | 07:38 | |
openstackgerrit | Merged openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/302852 | 07:48 |
*** pcaruana has joined #openstack-ironic | 07:49 | |
*** ohamada has joined #openstack-ironic | 07:52 | |
openstackgerrit | Merged openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/302879 | 07:54 |
openstackgerrit | Merged openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/302880 | 07:54 |
*** ohamada has quit IRC | 07:54 | |
*** ohamada has joined #openstack-ironic | 07:57 | |
aarefiev | morning! | 07:58 |
aarefiev | dtantsur: morning, do we want to run discovery test on the existing job? I think about separate one | 07:59 |
dtantsur | aarefiev, hmm, yeah, maybe. still, project-config is worth fixing first | 07:59 |
aarefiev | ok | 08:00 |
*** jistr has joined #openstack-ironic | 08:28 | |
*** derekh has joined #openstack-ironic | 08:28 | |
*** dtantsur is now known as dtantsur|brb | 08:30 | |
*** e0ne has quit IRC | 08:32 | |
*** openstackgerrit has quit IRC | 08:33 | |
*** athomas has joined #openstack-ironic | 08:33 | |
*** kromanenko has joined #openstack-ironic | 08:33 | |
*** openstackgerrit has joined #openstack-ironic | 08:33 | |
*** mgould has joined #openstack-ironic | 08:37 | |
mgould | morning Ironicers | 08:39 |
*** e0ne has joined #openstack-ironic | 08:40 | |
*** vmud213 has joined #openstack-ironic | 08:40 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Tempest plugin initial commit https://review.openstack.org/293925 | 08:49 |
*** moshele has quit IRC | 08:53 | |
vdrok | morning dtantsur|brb aarefiev and mgould | 09:00 |
aarefiev | vdrok: morning | 09:00 |
*** harlowja has quit IRC | 09:06 | |
vmud213 | morning ironicers | 09:10 |
sambetts | Morning all | 09:12 |
sergek | Morning All | 09:13 |
mgould | vdrok aarefiev vmud213 sambetts sergek morning! | 09:16 |
sergek | o/ | 09:17 |
vmud213 | mgould o/ | 09:17 |
vdrok | morning vmud213 sambetts and sergek | 09:17 |
vmud213 | morning vdrok | 09:18 |
*** dtantsur|brb is now known as dtantsur | 09:21 | |
*** e0ne has quit IRC | 09:25 | |
*** divya__ has joined #openstack-ironic | 09:27 | |
divya__ | i am planning to install openstack in my environment..and perfer a automated deployment..suggest me few methods | 09:28 |
divya__ | what are the openstack automate deployment available | 09:28 |
sambetts | For what purpose? testing or production? | 09:29 |
*** afaranha has quit IRC | 09:30 | |
*** yuikotak_ has joined #openstack-ironic | 09:31 | |
mgould | can someone +W https://review.openstack.org/#/c/301225 please? Already has two +2s, CI passing | 09:31 |
divya__ | hi sam.. | 09:31 |
divya__ | its for testing | 09:31 |
sambetts | devstack | 09:31 |
mgould | morning divya__ | 09:31 |
divya__ | good morning mgould | 09:31 |
divya__ | i need to form a openstack environment.. and not devstack sam | 09:32 |
divya__ | if devstack then easy for me as well | 09:32 |
*** yuikotakada has quit IRC | 09:32 | |
mgould | divya__, why can't you use devstack? | 09:32 |
*** alexpilotti has joined #openstack-ironic | 09:33 | |
divya__ | my testing requirement is openstack.. they want to test it in openstack | 09:33 |
* mgould suggests https://github.com/openstack/tripleo-quickstart/ as a possible alternative | 09:33 | |
mgould | divya__, I don't follow | 09:34 |
mgould | devstack is an installer for openstack | 09:34 |
divya__ | yea mgould.. we are planning to test in real time customer setup.. where all customer might not have devstack..they prefer openstack | 09:36 |
*** alexpilotti has quit IRC | 09:37 | |
sambetts | well then you need to find out how your customer is going to install it and use that | 09:37 |
*** subscope has joined #openstack-ironic | 09:38 | |
divya__ | they use openstack manual deployment..but i dont prefer that | 09:41 |
divya__ | so checking if there is anyother way like devstack to install openstack | 09:42 |
*** subscope has quit IRC | 09:44 | |
*** subscope has joined #openstack-ironic | 09:45 | |
*** yuikotak_ has quit IRC | 09:49 | |
mgould | well, if the environment needs to precisely match your customer's environment, you really need to do what they do | 09:49 |
mgould | any automated installer will have the same problem as devstack | 09:49 |
*** yuanying has joined #openstack-ironic | 09:50 | |
*** amotoki has quit IRC | 09:50 | |
divya__ | oh ok.. let me create my own script rather than installting manually | 09:51 |
mgould | if you just need an OpenStack install but it doesn't need to precisely match your customer's environment, you could try https://wiki.openstack.org/wiki/Packstack or https://github.com/openstack/tripleo-quickstart/ or http://openstack.astokes.org/ maybe | 09:53 |
*** yuikotakada has joined #openstack-ironic | 09:53 | |
mgould | [disclaimer: I've only used tripleo-quickstart from that list] | 09:54 |
divya__ | ok mgould | 09:56 |
mgould | np, good luck! | 09:56 |
*** ohamada has quit IRC | 09:59 | |
*** ohamada has joined #openstack-ironic | 10:00 | |
*** athomas has quit IRC | 10:07 | |
*** hoangcx has quit IRC | 10:09 | |
*** subscope has quit IRC | 10:10 | |
*** athomas has joined #openstack-ironic | 10:11 | |
*** subscope has joined #openstack-ironic | 10:15 | |
*** e0ne has joined #openstack-ironic | 10:20 | |
*** subscope has quit IRC | 10:20 | |
openstackgerrit | Aline Bousquet proposed openstack/ironic-lib: Add support for metrics https://review.openstack.org/301526 | 10:26 |
*** amotoki has joined #openstack-ironic | 10:41 | |
*** amotoki has quit IRC | 10:43 | |
sambetts | TheJulia: switching to local boot has got things passing in my CI, but I'm still frustrated I don't know what was causing the pxe to fail | 10:45 |
*** aswadr_ has joined #openstack-ironic | 10:47 | |
*** nathan-h has joined #openstack-ironic | 10:48 | |
*** mgould has quit IRC | 10:50 | |
nathan-h | vdrok: Hey, with regards to all log entries having request id, the request id obviously changes a few times during provisioning, is there any plans to have a master id , that would refer to all entries of a provisioning? | 10:51 |
*** amotoki has joined #openstack-ironic | 10:55 | |
*** mgould has joined #openstack-ironic | 11:03 | |
vdrok | hi nathan-h, lintan is working on this, I think this is the end goal | 11:03 |
*** pece has joined #openstack-ironic | 11:07 | |
nathan-h | vdrok: Thanks | 11:10 |
*** amotoki has quit IRC | 11:17 | |
*** shakamunyi has joined #openstack-ironic | 11:18 | |
*** shakamunyi has quit IRC | 11:23 | |
*** amotoki has joined #openstack-ironic | 11:24 | |
*** amotoki has quit IRC | 11:26 | |
*** alexpilotti has joined #openstack-ironic | 11:27 | |
*** keedya has joined #openstack-ironic | 11:28 | |
*** piet_ has joined #openstack-ironic | 11:30 | |
*** amotoki has joined #openstack-ironic | 11:30 | |
mat128 | good morning everyone | 11:37 |
*** subscope has joined #openstack-ironic | 11:38 | |
*** amotoki has quit IRC | 11:40 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/302851 | 11:43 |
divya__ | good morning mat128 | 11:44 |
*** divya__ has quit IRC | 11:44 | |
*** mkovacik has joined #openstack-ironic | 11:45 | |
*** dprince has joined #openstack-ironic | 11:47 | |
*** baoli has joined #openstack-ironic | 11:47 | |
*** baoli_ has joined #openstack-ironic | 11:48 | |
*** keedya has quit IRC | 11:50 | |
*** subscope has quit IRC | 11:50 | |
*** ndipanov has quit IRC | 11:51 | |
*** baoli has quit IRC | 11:51 | |
*** Marga_ has quit IRC | 11:54 | |
*** Marga_ has joined #openstack-ironic | 11:54 | |
*** sturivnyi has quit IRC | 11:56 | |
*** sturivnyi has joined #openstack-ironic | 11:57 | |
*** derekh has quit IRC | 11:58 | |
*** e0ne has quit IRC | 11:59 | |
*** ipukha has joined #openstack-ironic | 12:01 | |
*** subscope has joined #openstack-ironic | 12:03 | |
*** afaranha has joined #openstack-ironic | 12:04 | |
TheJulia | sambetts: I wouldn't call it pxe failing, the chain loader just couldn't see the disks. :\ I kind of wonder if you were to boot up ye-olde dos and if it would see the disk | 12:12 |
TheJulia | oh, and, Good morning everyone! | 12:12 |
*** ohamada_ has joined #openstack-ironic | 12:17 | |
*** ohamada has quit IRC | 12:17 | |
*** shakamunyi has joined #openstack-ironic | 12:18 | |
*** alexpilotti has quit IRC | 12:19 | |
dtantsur | morning TheJulia | 12:23 |
*** shakamunyi has quit IRC | 12:23 | |
*** thrash|g0ne is now known as thrash | 12:23 | |
openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Added functions to translate between uuid and uri https://review.openstack.org/301877 | 12:24 |
sambetts | Morning TheJulia, yeah I guess the pxe part was successful, but it was failure from the Ironic deployment perspective because syslinux couldn't boot the OS :( | 12:24 |
*** raildo-afk is now known as raildo | 12:26 | |
*** Marga_ has quit IRC | 12:26 | |
*** Goneri has joined #openstack-ironic | 12:29 | |
*** alexpilotti has joined #openstack-ironic | 12:29 | |
*** trown|outtypewww is now known as trown | 12:29 | |
*** jjohnson2 has joined #openstack-ironic | 12:29 | |
*** Goneri has quit IRC | 12:30 | |
*** alexpilotti has quit IRC | 12:30 | |
*** alexpilotti has joined #openstack-ironic | 12:30 | |
*** mtanino has joined #openstack-ironic | 12:30 | |
*** Goneri has joined #openstack-ironic | 12:33 | |
openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Added get_server_profile_by_uuid and changed model https://review.openstack.org/302225 | 12:33 |
openstackgerrit | Milan Kováčik proposed openstack/python-ironic-inspector-client: Introspection on stored data https://review.openstack.org/302195 | 12:34 |
*** alaski is now known as lascii | 12:39 | |
vdrok | morning mat128 and TheJulia | 12:40 |
openstackgerrit | Merged openstack/ironic-python-agent: Set modification time in tarfile of ramdisk logs https://review.openstack.org/301225 | 12:41 |
*** ndipanov has joined #openstack-ironic | 12:43 | |
*** derekh has joined #openstack-ironic | 12:43 | |
TheJulia | vdrok: I'm pondering the thought of returning deployfail instead... I like the tiny option box error represents, and kind of fear that someone seeing deployfail that is working against the same ironic but not aware of nodes being imported might take the wrong action. | 12:47 |
*** piet_ has quit IRC | 12:47 | |
vdrok | TheJulia: hm, yep, deployfail additionaly allows transition to active | 12:49 |
mat128 | TheJulia: I think the rational behind it was that error is essentially deletefail | 12:50 |
vdrok | otoh, if you don't plan to allow it, it does not really matter right> | 12:50 |
vdrok | so deploing -> deployfail seems more logical than deploying -> erro | 12:51 |
vdrok | *error | 12:51 |
vdrok | have to go, will be back in an hour | 12:51 |
*** mtanino has quit IRC | 12:53 | |
*** smoriya_ has quit IRC | 12:54 | |
TheJulia | mat128: it is, and that is part of what has me conflicted. A user with a node in adoptfail can only go so many places in the state machine, a user in an old client, would pretty much be limited to node-delete unless their client decided to start speaking the newer api version :) | 12:57 |
openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Extend client to add Connections https://review.openstack.org/302365 | 12:58 |
mgould | morning mat128 TheJulia | 12:58 |
TheJulia | good morning mgould | 12:58 |
*** e0ne has joined #openstack-ironic | 12:58 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: [WIP] Store ramdisk logs on preprocessing failures as well https://review.openstack.org/300011 | 13:01 |
jroll | morning friends | 13:02 |
TheJulia | good morning jroll | 13:04 |
*** jcoufal has joined #openstack-ironic | 13:04 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: [WIP] Support Ironic node names in our API https://review.openstack.org/276331 | 13:07 |
dtantsur | morning jroll, happy Friday | 13:07 |
jroll | happy friday indeed \o/ | 13:08 |
jroll | I realized this morning I need to draft summit schedule stuff | 13:08 |
jroll | so last call for things there :P | 13:08 |
*** saripurigopi has joined #openstack-ironic | 13:09 | |
mgould | morning jroll | 13:09 |
*** jayc has quit IRC | 13:09 | |
*** jayc has joined #openstack-ironic | 13:10 | |
*** xavierr has quit IRC | 13:10 | |
openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/302851 | 13:10 |
jroll | \o mgould | 13:11 |
*** piet has joined #openstack-ironic | 13:12 | |
thiagop | morning Ironic | 13:13 |
thiagop | TGIF! | 13:13 |
*** [1]cdearborn has joined #openstack-ironic | 13:13 | |
TheJulia | good morning thiagop piet | 13:13 |
piet | Morning | 13:14 |
mgould | morning thiagop piet | 13:14 |
thiagop | hiya piet | 13:14 |
thiagop | TheJulia mgould o/ | 13:14 |
piet | thiagop Hiya | 13:15 |
piet | Wakknig up | 13:15 |
jroll | dtantsur: we talked about a general inspection session at the summit, you still want that? do you also want the "inspector HA" session? | 13:16 |
piet | Had a "no show" for a usability this morning | 13:16 |
vmud213 | morning TheJulia, jroll, dtantsur | 13:16 |
dtantsur | jroll, we need at least one session. we definitely need to talk about our HA, it can be a separate session or part of our only session (which is the most likely, I guess) | 13:17 |
dtantsur | HA is the main topic we'll be discussing | 13:17 |
jroll | dtantsur: okay, thinking one session for general, and maybe further HA stuff can be friday morning | 13:17 |
jroll | dtantsur: fishbowl or workroom for the former? | 13:17 |
jroll | s/former/general session | 13:18 |
* dtantsur tries to remember his plans for the summit | 13:18 | |
dtantsur | jroll, workroom, I guess. Our main topic will be the HA, everything else on Friday. | 13:19 |
dtantsur | mkoderer, sambetts ^^^ | 13:19 |
jroll | dtantsur: aha, ok | 13:19 |
dtantsur | jroll, also we need to sync with Fuel folks, but that goes beyond only inspector | 13:19 |
dtantsur | oops, I planned to ping mkovacik, sorry mkoderer | 13:20 |
jroll | dtantsur: oh yeah, fun | 13:20 |
*** rbudden has joined #openstack-ironic | 13:21 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/302853 | 13:22 |
*** lucasagomes has quit IRC | 13:22 | |
* mkovacik closes a tab full of "yo dawgs...processing..." pics | 13:23 | |
TheJulia | vdrok: re https://review.openstack.org/#/c/275766/11/doc/source/deploy/adoption.rst comment below line 61, mentioning nova. I'm struggling to try and identify why we would mention nova, espescially since preserve_ephemeral, if an operator chose, is just an instance_info field, but I think that woudl be limited to only the most advanced operators :) | 13:23 |
mkovacik | dtantsur, OK cool | 13:23 |
mat128 | TheJulia: any thought on managing the instance part? I know this is out of scope for your spec, just wanted to talk about it | 13:24 |
*** alexpilotti has quit IRC | 13:24 | |
jroll | dtantsur: one more thing, removing dib ramdisk, I thought we had a plan to drop the test on DIB master and just eat the stable branch breaks when they happen | 13:24 |
TheJulia | mat128: as in creating a record in nova, someone was mentioning they were working on that idea a few months back, and to make the node visible in nova they would have just to populate ironic's instance_info field with the uuid from nova | 13:25 |
*** alexpilotti has joined #openstack-ironic | 13:25 | |
TheJulia | err, instance_uuid | 13:25 |
mat128 | yeah, so far (before your spec) we were going to nova boot with --az to land on a specific hypervisor, and have the "fake" driver in Ironic so nothing happens, all of this while using the old api version to end up in the active state | 13:26 |
dtantsur | jroll, yeah, probably | 13:26 |
mat128 | TheJulia: just trying to replace the fake/old api by adopt stuff | 13:26 |
jroll | dtantsur: okay, going to skip that session then if that's good with you | 13:26 |
dtantsur | jroll, TheJulia, also, do we need a session on boot from volume? | 13:26 |
dtantsur | jroll++ | 13:26 |
jroll | dtantsur: probably? I sadly have not reviewed that spec yet | 13:27 |
TheJulia | dtantsur: possibly, but the lack of spec reviews for the spec and underlying spec would kind of make it difficul to have a session on it | 13:27 |
dtantsur | fair | 13:28 |
TheJulia | Given I think we'll likely (i hope) get it approved early on in the cycle, and begin landing substrate, the real discussion for remaining things would end up being at the next major gathering, whatever that is for us :) | 13:28 |
*** lucasagomes has joined #openstack-ironic | 13:28 | |
dtantsur | such lucasagomes wow :) | 13:28 |
jroll | sambetts: woooo https://www.openstack.org/summit/austin-2016/summit-schedule/events/9491?goback=1 | 13:29 |
*** moshele has joined #openstack-ironic | 13:29 | |
jroll | sambetts: we probably still want our own session on that? | 13:29 |
dtantsur | do we? | 13:30 |
jroll | idk | 13:30 |
*** subscope has quit IRC | 13:30 | |
jroll | I'm sure there's plenty of details to work out | 13:30 |
dtantsur | we'll definitely won't go over all details in one or two sessions :) | 13:31 |
jroll | but maybe we can work most of it out there and follow up in gerrit | 13:31 |
jroll | ya | 13:31 |
TheJulia | Yeah, but maybe give time for people to bring their wants/needs to the table | 13:31 |
openstackgerrit | Milan Kováčik proposed openstack/python-ironic-inspector-client: Introspection on stored data https://review.openstack.org/302195 | 13:31 |
jroll | TheJulia: that is to say we should or should not have another session? :) | 13:32 |
TheJulia | jroll: maybe one, or have a sign and a suggestion box :) | 13:32 |
* TheJulia kind of likes the suggestion box idea | 13:32 | |
jroll | heh | 13:32 |
*** lucasagomes has quit IRC | 13:33 | |
TheJulia | The more I think about it, that might not be a bad way to get some feedback | 13:33 |
* TheJulia may also be completely crazy | 13:34 | |
*** alexpilo_ has joined #openstack-ironic | 13:34 | |
*** alexpilotti has quit IRC | 13:35 | |
openstackgerrit | Milan Kováčik proposed openstack/python-ironic-inspector-client: Introspection on stored data https://review.openstack.org/302195 | 13:37 |
*** alexpilotti has joined #openstack-ironic | 13:37 | |
*** alexpilo_ has quit IRC | 13:39 | |
*** dims is now known as dimsum__ | 13:41 | |
*** mvpnitesh has quit IRC | 13:41 | |
*** lucasagomes has joined #openstack-ironic | 13:42 | |
*** mtanino has joined #openstack-ironic | 13:42 | |
*** dansmith is now known as superdan | 13:42 | |
*** piet has quit IRC | 13:42 | |
jroll | TheJulia: I set up a suggestion box at https://bugs.launchpad.net/ironic | 13:43 |
jroll | :P | 13:43 |
openstackgerrit | Milan Kováčik proposed openstack/python-ironic-inspector-client: Introspection on stored data https://review.openstack.org/302195 | 13:43 |
TheJulia | jroll: :) | 13:43 |
*** mgoddard_ has joined #openstack-ironic | 13:45 | |
*** mgoddard has quit IRC | 13:49 | |
*** ndipanov has quit IRC | 13:50 | |
*** links has quit IRC | 13:51 | |
openstackgerrit | Merged openstack/ironic: Allow user to specify cleaning network https://review.openstack.org/291427 | 13:53 |
*** hoangcx has joined #openstack-ironic | 13:53 | |
*** subscope has joined #openstack-ironic | 13:56 | |
openstackgerrit | Thiago Paiva Brito proposed openstack/ironic-specs: Dynamic allocation of nodes on OneView drivers https://review.openstack.org/275726 | 13:58 |
*** jayc has quit IRC | 13:58 | |
sambetts | jroll: awesome! Yeah not sure about having our own session, I'm sure if it comes to it we can always organise a coffee/beer session or something to work things out on the side | 14:05 |
sambetts | on another awesome note, the tinyipa post job has merged! | 14:06 |
sambetts | we just need an IPA patch to go in to trigger its first build! | 14:07 |
jroll | sambetts: yeah, we could, and \o/ | 14:08 |
sambetts | jroll: and the patch to enable non-src tinyipa jobs has merge! | 14:09 |
sambetts | jroll: Do you know were I can look at the results of the post jobs? | 14:09 |
jroll | sambetts: logs.o.o/<first two characters of commit sha>/<commit sha> | 14:12 |
vdrok | morning jroll and thiagop | 14:15 |
thiagop | hiya vdrok | 14:15 |
jroll | \o vdrok | 14:15 |
thiagop | jroll: did you have a fine travel? | 14:16 |
jroll | thiagop: so far, so good :) | 14:17 |
jroll | currently in the middle of wyoming | 14:17 |
NobodyCam | good morning Ironicers | 14:17 |
NobodyCam | jroll: :) good to hear travel is going well | 14:18 |
vdrok | morning NobodyCam | 14:18 |
NobodyCam | morning vdrok | 14:18 |
thiagop | jroll: wait, you haven't arrived yet? o.O | 14:18 |
jroll | hey, good morning NobodyCam | 14:18 |
thiagop | good morning NobodyCam | 14:18 |
jroll | thiagop: we only left wednesday, and taking our time to see all the things :) | 14:18 |
jroll | it'll be ~38 hours of driving not including stops altogether | 14:18 |
NobodyCam | thank you for hitting the button on the review | 14:18 |
thiagop | jroll: woot | 14:18 |
vdrok | TheJulia: yep, you're right about preserve_ephemeral, ignore that :) As for error/deployfail I think about the following. node gets to adoptfail state, older client sees ERROR state, and cannot do set-provision-state deleted or rebuild, which are allowed for a node that actually is in error state. in case if it is DEPLOYFAIL, he still can not do | 14:24 |
vdrok | set-provision-state deleted, rebuild and additionally cannot do active. so both options will bring confusion | 14:24 |
sambetts | jroll: this patch merged yesterday after the post job was in and I don't see any post queue logs https://review.openstack.org/#/c/299560/ http://logs.openstack.org/60/299560/2/ | 14:24 |
sambetts | jroll: Oh yeah I forgot you were still on the road! | 14:24 |
TheJulia | vdrok: well, the only other option is to reveal new states to the client.. | 14:25 |
jroll | sambetts: it's the commit sha, and if there's a merge commit it's *that* one | 14:26 |
jroll | so e.g. last commit is at http://logs.openstack.org/2d/2d8e139f0394cdd9b713bcf97bf300ee43165bf7/ | 14:26 |
jroll | seems to be still processing | 14:26 |
sambetts | jroll: oooh, I see! So its under the commit not the change, I miss understood | 14:26 |
vdrok | TheJulia: yep. I'm not insisting on doing either of the options, just think that if one sees deployfail after deploying in older versions, it may be more logical. let's wait what others think :) | 14:28 |
*** piet has joined #openstack-ironic | 14:28 | |
jroll | sambetts: yep, no worries | 14:28 |
sambetts | jroll: Do you know how to get requirements added to the post job? mksquashfs is missing :'( | 14:30 |
jroll | dtantsur: devananda: I've got 6/10 sessions locked in at the bottom of the pad. still need to think more about the rest, might update later, would love your thoughts, we could discuss in the meeting monday as well | 14:30 |
jroll | sambetts: either install them in your post job script in the repo, or figure out how to add to base images | 14:30 |
jroll | sambetts: I think there's some sort of extra-requirements.txt that is apt packages that jobs may pick up too? only heard about it briefly so I may be completely off base | 14:31 |
TheJulia | vdrok: okay, all for other people's input :) | 14:32 |
sambetts | jroll: I thought you had to do the same thing recently for a missing requirement for the coreos image? | 14:32 |
sambetts | or was that in the built image | 14:32 |
jroll | sambetts: I did, we added to the full_trusty_build.sh, which is what the post job runs | 14:32 |
jroll | I believe we were also missing mksquashfs there :P | 14:32 |
sambetts | jroll: Ah, I see you've add the logic to install the deps based on which system | 14:33 |
jroll | sambetts: yeah, I think lucas added the fedora stuff | 14:34 |
jroll | but that's unrelated | 14:34 |
*** subscope has quit IRC | 14:34 | |
*** persia has quit IRC | 14:34 | |
*** daemontool has quit IRC | 14:35 | |
*** jjohnson2 has quit IRC | 14:36 | |
*** jjohnson2 has joined #openstack-ironic | 14:36 | |
jroll | alright, gotta bounce for now. catch you all on monday or something :) | 14:37 |
sambetts | cya jroll | 14:37 |
*** persia has joined #openstack-ironic | 14:37 | |
NobodyCam | travel safe jroll | 14:37 |
*** trown is now known as trown|relocating | 14:37 | |
TheJulia | trown|relocating: please tell me your not fleeing the state.... | 14:37 |
trown|relocating | TheJulia: lol, no just driving downtown, but with our legislature it has crossed my mind :) | 14:38 |
TheJulia | :) | 14:38 |
*** karimb has joined #openstack-ironic | 14:38 | |
TheJulia | trown|relocating: sadly I know WAY too many people who have already left :( | 14:39 |
mgould | jroll, safe travels | 14:39 |
*** karimb has quit IRC | 14:40 | |
*** karimb has joined #openstack-ironic | 14:41 | |
*** saripurigopi has quit IRC | 14:42 | |
*** piet has quit IRC | 14:46 | |
NobodyCam | morning mgould :) | 14:47 |
NobodyCam | morning sambetts :) | 14:47 |
NobodyCam | morning vdrok :) | 14:47 |
sambetts | o/ NobodyCam | 14:47 |
NobodyCam | hehehe :) | 14:47 |
* sambetts thinks NobodyCam is scripting this stuff now | 14:48 | |
NobodyCam | lol nope | 14:48 |
dtantsur | morning NobodyCam, happy Friday | 14:48 |
NobodyCam | hey hey dtantsur good morning and totally TGIF! | 14:48 |
openstackgerrit | Sam Betts proposed openstack/ironic-python-agent: Add script to install missing tinyipa dependencies https://review.openstack.org/303465 | 14:48 |
sambetts | dtantsur, jroll, NobodyCam: ^ the tinyipa post job is failing until we merge that | 14:49 |
* NobodyCam looks | 14:50 | |
*** mgoddard has joined #openstack-ironic | 14:51 | |
*** mgoddard_ has quit IRC | 14:51 | |
NobodyCam | sambetts: not bug #? | 14:52 |
sambetts | NobodyCam: the tinyipa post job is so new I've not got round to summiting a bug report for it, probably should though for tracking I guess | 14:53 |
*** jaybeale has joined #openstack-ironic | 14:55 | |
mgould | NobodyCam, morning! | 14:56 |
openstackgerrit | Sam Betts proposed openstack/ironic-python-agent: Add script to install missing tinyipa dependencies https://review.openstack.org/303465 | 14:56 |
sambetts | NobodyCam: ^ added a bug :) | 14:57 |
NobodyCam | :) | 14:57 |
NobodyCam | wow same package name across all the distros... what are the chances of that, | 14:59 |
sambetts | :D I know | 15:00 |
mgould | it's madness | 15:01 |
mgould | crops won't grow | 15:02 |
sambetts | hahaha | 15:02 |
NobodyCam | lol | 15:02 |
NobodyCam | +2 | 15:02 |
sambetts | :D thanks NobodyCam | 15:03 |
NobodyCam | sambetts: you should go to vegas | 15:03 |
sambetts | :-P | 15:03 |
*** lucasagomes has quit IRC | 15:03 | |
*** vishwanathj has quit IRC | 15:04 | |
*** subscope has joined #openstack-ironic | 15:04 | |
konetzed | dtantsur: can you run ironic-inspector on the same node that neutron is running on and they share the bridge interface or do you need a different interface for it to run on. The docs for it really dont clarify that. | 15:07 |
konetzed | dtantsur: basically i am tryign to run the full stack on one server | 15:07 |
* mgould is ssh'ed into a server running both ironic-inspector and neutron as we speak | 15:08 | |
dtantsur | konetzed, that's what tripleo does. yes, you can. neutron uses namespaces (iirc), so their dhcp servers do not overlap | 15:08 |
sambetts | konetzed: devstack single node will run inspector and neutron on the same node | 15:09 |
konetzed | ah yes never thought of devstack doing that. Thanks guys I will run through a liberty setup this morning and get ironic-inspector setup :D | 15:12 |
*** jcoufal has quit IRC | 15:12 | |
*** lucasagomes has joined #openstack-ironic | 15:12 | |
dtantsur | you're welcome :) | 15:12 |
*** moshele has quit IRC | 15:16 | |
sambetts | mgould: I never knew about .PHONY thats cool! | 15:17 |
mgould | sambetts, :-) | 15:18 |
*** cloudnull has joined #openstack-ironic | 15:18 | |
cloudnull | i have a question regarding network setups using ironic. specifically im having an issue with tftp once the node comes online. The host is getting DHCP from neutron and thall seems to work just fine, i can manually retrieve files from tftp using the cli client, but its just not working when PXE booting. Im using the ipmi agent w/ the latest mitaka release. | 15:21 |
cloudnull | nothing notable in the logs | 15:21 |
cloudnull | but the nodes gets cycled via ipmi, neutron provides dhcp, all seems well, then tftp times out | 15:22 |
cloudnull | if i boot the node manually install an OS configure the interface to get DHCP it gets the address neutron provides and again all is well. | 15:22 |
cloudnull | i can then install a tftp client and retrieve files without issues. | 15:22 |
mgould | cloudnull, is the pxe rom being requested? | 15:23 |
sambetts | how is you network configured? Does the the baremetal machine have a route to the tftp server? | 15:23 |
cloudnull | sambetts: yes it has a route. | 15:23 |
cloudnull | the network within neutron is a flat network | 15:23 |
cloudnull | on a native vlan | 15:23 |
cloudnull | mgould: not that i can see | 15:24 |
cloudnull | the console shows getting dhcp | 15:24 |
cloudnull | then "tftp open timeout" | 15:24 |
sambetts | if you do a tcp dump on the machine running the tftp server do you see any packets coming in requesting tftp? | 15:24 |
sambetts | and is the tftp server running? | 15:24 |
cloudnull | i was doing that last night, i thought i did, let me do it again. might be a min to cycle things | 15:24 |
cloudnull | tftp is running a | 15:25 |
cloudnull | installed with tftp-hpa | 15:25 |
cloudnull | w/ ubuntu 14.04.4 | 15:25 |
*** aginwala has joined #openstack-ironic | 15:25 | |
cloudnull | its running the tftp-hpa service , though i have also tried doing it with xinetd | 15:25 |
mgould | cloudnull, you could also try adding log-dhcp to dnsmasq-ironic.conf and restarting it | 15:25 |
mgould | that should log whether or not the machine being booted is trying to do the fetch | 15:26 |
cloudnull | i will add that too | 15:26 |
*** garthb has joined #openstack-ironic | 15:26 | |
mgould | sambetts, is there a special tftp server? I thought dnsmasq did its own tftp serving | 15:26 |
sambetts | dnsmasq doesn't serve tftp, it just provides the dhcp options to point the pxe firmware to the right place | 15:27 |
cloudnull | sambetts: if i ``tftp $IPADRESS`` i can get files without issues from the ironic conductor. | 15:27 |
mgould | sambetts, thanks | 15:27 |
cloudnull | so from a naive test it working | 15:27 |
jlvillal | dtantsur: FYI: I replied to your comment: https://review.openstack.org/#/c/303048/5//COMMIT_MSG | 15:27 |
jlvillal | thanks | 15:28 |
sambetts | in the console of the bm machine, when it trying to pxe, can you see what IP address is being given for the tftp server? | 15:28 |
cloudnull | no, it just times out. | 15:28 |
sambetts | my servers print it out next to its own IP it got from DHCP | 15:28 |
cloudnull | i have a screen shot | 15:29 |
*** aginwala has quit IRC | 15:29 | |
*** piet has joined #openstack-ironic | 15:29 | |
cloudnull | http://snag.gy/GWQJa.jpg | 15:29 |
sambetts | ok, if you neutron port-show the port that the machine is given, you should be able to see the dhcp options, can you see if the tftp server ip is correct | 15:30 |
cloudnull | the client ip is what neutron is providing to the node | 15:30 |
*** aginwala has joined #openstack-ironic | 15:30 | |
cloudnull | let me check | 15:30 |
*** trown|relocating is now known as trown | 15:31 | |
openstackgerrit | vinay kumar muddu proposed openstack/ironic-python-agent: Wait for all interfaces to be up before node lookup https://review.openstack.org/303503 | 15:33 |
cloudnull | ah so this may be my issue, my nodes have multiple nics and the {"opt_value": "$IP", "ip_version": 4, "opt_name": "tftp-server"} is not the correct network for the tftp-server, seems i need to set the my_ip value or similar | 15:33 |
*** mkovacik has quit IRC | 15:33 | |
* cloudnull rereading the ironic.conf sample file | 15:33 | |
sambetts | cloudnull: in the ironic.conf there is a tftp ip setting you can configure so you may need to update that | 15:34 |
*** saripurigopi has joined #openstack-ironic | 15:35 | |
*** praneshp_ has joined #openstack-ironic | 15:41 | |
cloudnull | booting a new node now and will see if that has made it happy. . | 15:44 |
cloudnull | thanks mgould and sambetts for the advice | 15:44 |
sambetts | :) | 15:44 |
mgould | np, good luck! | 15:45 |
devananda | morning, all | 15:47 |
dtantsur | morning devananda | 15:48 |
dtantsur | and I'm calling it a day, have a great weekend folks | 15:48 |
NobodyCam | good morning devananda :) | 15:49 |
mgould | morning devananda | 15:50 |
mgould | dtantsur, have a good weekend! | 15:50 |
*** dtantsur is now known as dtantsur|afk | 15:50 | |
*** ifarkas has quit IRC | 15:50 | |
NobodyCam | just fyi: I just added multiple compute host support to the newton summit etherpad | 15:50 |
*** rama_y has joined #openstack-ironic | 15:51 | |
NobodyCam | have a good weekend dtantsur|afk | 15:51 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Prepare for transition to oslo-config-generator https://review.openstack.org/303048 | 15:52 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DEMO: Make use of oslo-config-generator https://review.openstack.org/303057 | 15:52 |
jlvillal | dtantsur|afk: Bye! | 15:53 |
*** aginwala has quit IRC | 15:53 | |
*** aginwala has joined #openstack-ironic | 15:54 | |
sambetts | o/ dtantsur|afk | 15:56 |
*** aginwala has quit IRC | 15:56 | |
*** aginwala_ has joined #openstack-ironic | 15:56 | |
*** bnemec has quit IRC | 15:56 | |
sambetts | NobodyCam: I think (if I remember correctly) jroll left that out because he was pretty confident that we knew what we were doing with it, although I'd confirm with him | 15:57 |
NobodyCam | sambetts: ya :) I think it may have something to do with node claims stuff but I added just with: anything to discuss / go over with this? | 15:59 |
NobodyCam | â—ľorginal spec: https://review.openstack | 15:59 |
NobodyCam | ya | 15:59 |
NobodyCam | gah | 15:59 |
*** ipukha has quit IRC | 16:00 | |
TheJulia | although the filtering in terms of api indexing I think we bumped because there were no good ways we liked | 16:00 |
*** e0ne has quit IRC | 16:00 | |
*** saripurigopi has quit IRC | 16:01 | |
sambetts | NobodyCam: Make sense, I'm really interested in seeing where the claims API goes, because I've got some interesting ideas for abusing it :-P | 16:01 |
NobodyCam | lol | 16:01 |
TheJulia | I think everyone does :) | 16:01 |
cloudnull | sambetts mgould -- http://snag.gy/MJT1x.jpg -- great success! | 16:03 |
cloudnull | thanks again | 16:03 |
cloudnull | now off to solve other problems :) | 16:03 |
*** bnemec has joined #openstack-ironic | 16:04 | |
*** piet has quit IRC | 16:05 | |
sambetts | cloudnull: :D awesome!! | 16:07 |
*** kromanenko has quit IRC | 16:07 | |
*** amotoki has joined #openstack-ironic | 16:10 | |
cloudnull | quick question, besides building images with disk image builder, is there a cache of pre-built images/kernels that I can use for deployments ? | 16:10 |
cloudnull | im happy to build one, but figured it one existed id rather not go the route of NIH. | 16:12 |
sambetts | cloudnull: http://tarballs.openstack.org/ironic-python-agent/coreos contains prebuilt coreos IPA images, and there will soon be /tinyipa too containing tinycore linux IPA images too | 16:12 |
cloudnull | excellent | 16:13 |
mgould | cloudnull, hurrah! | 16:13 |
*** daemontool has joined #openstack-ironic | 16:13 | |
*** rbrud has joined #openstack-ironic | 16:17 | |
*** shakamunyi has joined #openstack-ironic | 16:18 | |
*** cdearborn has joined #openstack-ironic | 16:18 | |
*** aginwala_ has quit IRC | 16:19 | |
*** aginwala has joined #openstack-ironic | 16:19 | |
*** saripurigopi has joined #openstack-ironic | 16:20 | |
*** saripurigopi has quit IRC | 16:21 | |
*** nathan-h has quit IRC | 16:22 | |
*** shakamunyi has quit IRC | 16:23 | |
*** aginwala has quit IRC | 16:24 | |
*** karimb has quit IRC | 16:27 | |
*** saripurigopi has joined #openstack-ironic | 16:30 | |
*** aginwala has joined #openstack-ironic | 16:31 | |
*** [1]cdearborn has quit IRC | 16:33 | |
*** vmud213 has quit IRC | 16:33 | |
*** derekh has quit IRC | 16:33 | |
*** subscope has quit IRC | 16:35 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Prepare for transition to oslo-config-generator https://review.openstack.org/303048 | 16:41 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DEMO: Make use of oslo-config-generator https://review.openstack.org/303057 | 16:41 |
*** baoli_ has quit IRC | 16:42 | |
*** Marga_ has joined #openstack-ironic | 16:43 | |
*** harlowja has joined #openstack-ironic | 16:46 | |
*** ohamada_ has quit IRC | 16:46 | |
*** pcaruana has quit IRC | 16:46 | |
*** suro-patz has joined #openstack-ironic | 16:50 | |
*** KennethWilke has joined #openstack-ironic | 16:52 | |
*** devananda has quit IRC | 16:53 | |
NobodyCam | brb | 16:54 |
*** praneshp_ has quit IRC | 16:55 | |
*** rcernin has quit IRC | 16:55 | |
*** e0ne has joined #openstack-ironic | 16:56 | |
*** e0ne has quit IRC | 16:57 | |
jlvillal | NobodyCam: come back!!! | 16:59 |
NobodyCam | lol | 16:59 |
TheJulia | lol | 16:59 |
*** aginwala has quit IRC | 17:00 | |
*** aginwala has joined #openstack-ironic | 17:00 | |
*** aginwala has quit IRC | 17:01 | |
jlvillal | Whew, I missed you NobodyCam :) | 17:01 |
*** aginwala has joined #openstack-ironic | 17:02 | |
jlvillal | 16 days until Austin.... | 17:02 |
NobodyCam | and here I tought you were going see if I could review something :p | 17:02 |
jlvillal | NobodyCam: I'd be happy on your thoughts on: https://review.openstack.org/303048 | 17:03 |
jlvillal | But that is not why I told you to come back :) | 17:03 |
SpamapS | Hi! | 17:03 |
SpamapS | so, people, I'm playing with the patchset that adds Neutrony things to Ironic | 17:03 |
SpamapS | and it does something very bad, which I'd like to fix | 17:04 |
SpamapS | UUIDs do not belong in config files. | 17:04 |
SpamapS | we have names in our API's for this purpose. | 17:04 |
SpamapS | anybody around that knows about the neutron stuff? | 17:04 |
jlvillal | Link? | 17:05 |
TheJulia | are names globally unique in neutron? | 17:05 |
*** devananda has joined #openstack-ironic | 17:06 | |
mat128 | TheJulia: nope | 17:07 |
*** baoli has joined #openstack-ironic | 17:07 | |
mat128 | just created two networks with the same name, neutron accepts and lists them | 17:07 |
davidlenwell | hey SpamapS | 17:08 |
mat128 | SpamapS: How do you disambiguate network names? | 17:08 |
sambetts | SpamapS: Ironic has had cleaning network uuid in the config file long before the full neutron intergration work | 17:09 |
NobodyCam | jlvillal: lol | 17:10 |
NobodyCam | morning SpamapS | 17:12 |
SpamapS | davidlenwell: howdy | 17:13 |
* mgould is being kicked out of the office by the Royal Scottish Pipers Society, so must bid you all goodnight | 17:14 | |
SpamapS | mat128: you fail if there are two networks in neutron with the same name in your tenant. Because that is _an insane situation_ | 17:14 |
NobodyCam | mgould: good night and good weekend... also LOL | 17:14 |
SpamapS | sambetts: ahh, ok, that was also a maligned idea. :) | 17:14 |
SpamapS | sambetts: not saying it does not work, but it is a lot of extra complexity for no gain. | 17:14 |
mgould | NobodyCam, the hazards of subletting office space :-) | 17:14 |
*** e0ne has joined #openstack-ironic | 17:15 | |
*** mgould has quit IRC | 17:15 | |
sambetts | extra complexity? You mean you have to configure and start Ironic after starting neutron and setting up a network? | 17:15 |
*** aginwala has quit IRC | 17:16 | |
SpamapS | sambetts: not saying you shouldn't have to reconfigure it or restart it if you're making networking changes. I'm saying, having to plumb UUIDs into config files is extra complexity. | 17:16 |
TheJulia | wouldn't that just be a fact in one's configuration management system? | 17:17 |
SpamapS | after you have created everything, yes | 17:17 |
SpamapS | We went through this in Heat too with tenant and domain ids. The burden you're putting on the config management system is really undue. | 17:18 |
TheJulia | but could the creation and then fact generation be completely automated? | 17:18 |
SpamapS | It already is fully automated. However, the ordering becomes rigid. | 17:18 |
mat128 | We've had the same issue with swift_account uuid vs account name | 17:19 |
SpamapS | It's just extra work | 17:19 |
mat128 | We run things from scratch all the time (automated testing) and ended up having to query the uuid for the name | 17:19 |
mat128 | same pattern | 17:19 |
SpamapS | And there are a _lot_ more divergent config management systems than there are Ironics. | 17:19 |
*** aginwala has joined #openstack-ironic | 17:20 | |
SpamapS | also if you don't want the "fail on two network names" you can also have the "pick random one" or "pick first". | 17:21 |
mat128 | Seeing how neutron just bails out on a multiple match (Multiple network matches found for name 'test', use an ID to be more specific.), that's what I would do if we were going this route | 17:21 |
sambetts | Nova does the same for instance names | 17:24 |
TheJulia | So how would we acurately identify the exact same network every single time? | 17:24 |
*** alexpilotti has quit IRC | 17:24 | |
mat128 | TheJulia: always lookup, crash on multiple results | 17:24 |
mat128 | you might break the service simply by creating a network though | 17:24 |
TheJulia | so then it becomes who creates it first scenario | 17:25 |
TheJulia | and what do they create | 17:25 |
*** jistr has quit IRC | 17:25 | |
TheJulia | Not disagreeing, just thinking that just a name may not be the best idea, espescially if I had a custom hardwaremanager doing cleaning that needed access to some resources and my cleaning network could talk to not a single thing in the world beyond the conductors. | 17:26 |
TheJulia | but that comes down to operational needs more than anything | 17:26 |
NobodyCam | jlvillal: this is a quite a ominous note to sneak in: "Its value may be silently ignored in the future".... lol ;) | 17:26 |
jlvillal | NobodyCam: That is what oslo-config-generator prints. | 17:27 |
NobodyCam | lol ++ | 17:27 |
jlvillal | Basically trying to get as much of the content to look the same as how it will look with oslo-config-generator so we can actually see what the transition to that really changes. | 17:27 |
jlvillal | NobodyCam: and yes a little ominous :) | 17:28 |
jlvillal | NobodyCam: I have this WIP patch where I rebased lintan's patch onto my patch: https://review.openstack.org/#/c/303057/ | 17:28 |
jlvillal | So you can see what the oslo-generator patch changes to the config file more easily. | 17:28 |
*** athomas has quit IRC | 17:29 | |
NobodyCam | jlvillal: line 252 : default_log_levels seem strange to the space for the first one but then not for any of the others... (just looks strange to me) | 17:30 |
jlvillal | NobodyCam: looking... | 17:32 |
jlvillal | NobodyCam: I think the only thing that changed was the space added. | 17:34 |
NobodyCam | oh y :) | 17:34 |
NobodyCam | ya | 17:34 |
jlvillal | NobodyCam: https://review.openstack.org/#/c/303057/5/etc/ironic/ironic.conf.sample This shows what happens after oslo-config-generator change over. So no change to that line, so it matches what ocg does. | 17:35 |
* sambetts is heading off for the weekend | 17:36 | |
NobodyCam | have a great weekend sambetts | 17:36 |
jlvillal | sambetts: Enjoy! | 17:36 |
sambetts | See everyone on Monday :D | 17:36 |
*** sambetts is now known as sambetts|afk | 17:36 | |
sambetts|afk | o/ NobodyCam jlvillal | 17:37 |
SpamapS | TheJulia: I think your fears are valid, but the risks are very low. | 17:37 |
SpamapS | TheJulia: administrators create networks _almost never_ | 17:37 |
davidlenwell | SpamapS: could you be a little more vague? | 17:38 |
SpamapS | davidlenwell: it's my speciality | 17:38 |
SpamapS | something is | 17:38 |
SpamapS | or isn't | 17:38 |
SpamapS | sometimes | 17:38 |
TheJulia | lol | 17:38 |
davidlenwell | SpamapS: https://www.youtube.com/watch?v=_P_hKcZgQhY | 17:38 |
* TheJulia giggled and was immedialy asked "what are you up to?" | 17:39 | |
SpamapS | TheJulia: the appropriate response is "something" | 17:39 |
davidlenwell | SpamapS: im sorry the answer we were looking for was "stuff" | 17:40 |
SpamapS | So anyway, I think one thing to make clear is, the query for "give me the uuid of this netowrk" should only query the tenant that ironic's service user runs in. | 17:40 |
SpamapS | project | 17:40 |
TheJulia | SpamapS: that only causes more giggling, fwiw | 17:40 |
SpamapS | it uses project or it gets the hose | 17:40 |
SpamapS | davidlenwell: maybe | 17:41 |
NobodyCam | lol | 17:41 |
davidlenwell | lol | 17:41 |
davidlenwell | SpamapS: thats actually logical. | 17:41 |
*** garthb has quit IRC | 17:42 | |
*** garthb has joined #openstack-ironic | 17:42 | |
SpamapS | TheJulia: anyway, I'm not saying don't _allow_ uuid btw. | 17:43 |
SpamapS | Just, don't require it. | 17:43 |
jlvillal | https://twitter.com/stevebot/status/718185667709267969 | 17:43 |
TheJulia | Okay, in that case making it optional just gives more flexibility | 17:43 |
TheJulia | forcing it to be a name just makes me think that we're sitting around a table pushing bottle caps of burden back and forth around on a map | 17:44 |
TheJulia | SpamapS: I say submit patches! :) | 17:44 |
davidlenwell | ++ TheJulia | 17:45 |
NobodyCam | TheJulia: sounds like a game of Reversi | 17:45 |
*** piet has joined #openstack-ironic | 17:46 | |
*** praneshp_ has joined #openstack-ironic | 17:46 | |
TheJulia | NobodyCam: Kind of yeah, but likely with at least three players... not sure what that would be :) | 17:47 |
SpamapS | TheJulia: oh, I will, I just wanted to make sure I'm not _100%_ nuts on this. :) | 17:48 |
TheJulia | SpamapS: well, standard disclaimer, I don't claim to be sane | 17:48 |
*** Sukhdev has joined #openstack-ironic | 17:48 | |
SpamapS | Yeah, it's a scale | 17:49 |
jlvillal | I see what the twitter comment means, as an example: http://stackalytics.com/?module=ironic-group&user_id=cody.eilrich@ericsson.com | 17:51 |
* jlvillal now goes back to work... | 17:52 | |
*** e0ne has quit IRC | 17:52 | |
TheJulia | heh | 17:53 |
*** amotoki has quit IRC | 17:53 | |
mat128 | jlvillal: can't contribute too much :P | 17:53 |
jlvillal | :) | 17:53 |
*** Marga_ has quit IRC | 17:57 | |
*** achanda has joined #openstack-ironic | 17:57 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: Active Node Creation via adopt state https://review.openstack.org/275766 | 18:06 |
TheJulia | JayF: you around today? | 18:07 |
*** saripurigopi has quit IRC | 18:07 | |
JayF | TheJulia: ayup | 18:08 |
JayF | TheJulia: sup? | 18:08 |
*** saripurigopi has joined #openstack-ironic | 18:08 | |
TheJulia | 270902, are you thinking that we should just update the catch calling _ata_erase()? | 18:08 |
JayF | I think _ata_erase() should only return BlockDeviceEraseError exceptions | 18:09 |
JayF | assuming you're talking about the patch I thnk you are | 18:09 |
TheJulia | yes | 18:09 |
TheJulia | okay, simple enough | 18:09 |
TheJulia | Just wanted to make sure, since one line versues many others :) | 18:11 |
openstackgerrit | Merged openstack/ironic-lib: Updated from global requirements https://review.openstack.org/302969 | 18:11 |
*** achanda has quit IRC | 18:11 | |
*** lucasagomes has quit IRC | 18:14 | |
*** saripurigopi has quit IRC | 18:18 | |
*** saripurigopi has joined #openstack-ironic | 18:18 | |
*** e0ne has joined #openstack-ironic | 18:22 | |
*** lucasagomes has joined #openstack-ironic | 18:22 | |
*** rama_y has quit IRC | 18:24 | |
*** aginwala has quit IRC | 18:24 | |
*** alexpilotti has joined #openstack-ironic | 18:25 | |
*** e0ne has quit IRC | 18:25 | |
*** e0ne has joined #openstack-ironic | 18:26 | |
*** e0ne has quit IRC | 18:26 | |
*** Marga_ has joined #openstack-ironic | 18:26 | |
*** aginwala has joined #openstack-ironic | 18:28 | |
*** ChubYann has joined #openstack-ironic | 18:29 | |
*** alexpilotti has quit IRC | 18:29 | |
JayF | jlvillal: you had mentioned yesterday you might want some help diagnosing some grenade problems, here's your offer of help before I delve into something different :) | 18:30 |
jlvillal | JayF: I'm actually heading to lunch now... | 18:30 |
jlvillal | Was about to walk out the door. | 18:31 |
JayF | jlvillal: enjoy :) | 18:31 |
jlvillal | I'll see if you are busy when I get back :) And thanks! | 18:31 |
*** vishwanathj has joined #openstack-ironic | 18:49 | |
*** aginwala has quit IRC | 18:49 | |
*** aginwala has joined #openstack-ironic | 18:50 | |
NobodyCam | jlvillal: +2 with a comment on 303048 :) | 18:53 |
*** dprince has quit IRC | 18:56 | |
*** aginwala has quit IRC | 18:57 | |
*** piet has quit IRC | 18:59 | |
NobodyCam | anyone have a minute to take a look at a method rename patch: https://review.openstack.org/#/c/302224 | 19:01 |
*** piet has joined #openstack-ironic | 19:15 | |
*** alexpilotti has joined #openstack-ironic | 19:26 | |
thiagop | Folks, I'm calling it a week | 19:27 |
thiagop | have a nice weekend you all | 19:27 |
*** thiagop has quit IRC | 19:27 | |
davidlenwell | thiagop: thanks you too! | 19:27 |
*** alexpilotti has quit IRC | 19:31 | |
*** daemontool has quit IRC | 19:33 | |
*** karimb has joined #openstack-ironic | 19:34 | |
*** daemontool has joined #openstack-ironic | 19:37 | |
*** vishwanathj has quit IRC | 19:38 | |
*** vishwanathj has joined #openstack-ironic | 19:40 | |
*** vishwana_ has joined #openstack-ironic | 19:41 | |
*** moshele has joined #openstack-ironic | 19:44 | |
*** vishwanathj has quit IRC | 19:45 | |
jlvillal | NobodyCam: I responded to your comment | 19:50 |
NobodyCam | jlvillal: totally.. I did put in the comment "I have a comment in line it's more my own ramblings, please feel free to ignore" :) | 19:52 |
NobodyCam | jlvillal: totally.. I did put in the comment "I have a comment in line it's more my own ramblings, please feel free to ignore" :);p | 19:52 |
NobodyCam | :p | 19:52 |
jlvillal | NobodyCam: I saw :) | 19:52 |
jlvillal | NobodyCam: But I had to reply! I couldn't control myself... | 19:52 |
NobodyCam | hehehe | 19:52 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent: Provide fallback from ATA erase to shredding https://review.openstack.org/270902 | 19:55 |
*** rbrud has quit IRC | 19:55 | |
*** bigjools has quit IRC | 19:56 | |
*** SpamapS has quit IRC | 19:58 | |
jlvillal | NobodyCam: TheJulia: So I'm just adding to the waffling with my comment :( https://review.openstack.org/#/c/302224/ | 19:58 |
* TheJulia goes back to demanding we pain the shed purple | 19:59 | |
TheJulia | jlvillal: but very good point :) | 19:59 |
TheJulia | different word usage I guess | 19:59 |
*** aswadr_ has quit IRC | 20:00 | |
jlvillal | TheJulia: I see 'assert' and I just think Python 'assert' statement. | 20:00 |
NobodyCam | jlvillal: that is a much less bikeshed-y type comment :) | 20:00 |
TheJulia | how about _handle_previous_state_names ? | 20:01 |
NobodyCam | maybe we should ask for a spec on this and iterate there... /me *DUCKS* | 20:02 |
*** daemontool has quit IRC | 20:02 | |
jlvillal | TheJulia: sounds sort-of okay. I'm not yelling out, "Yes!" on it :) | 20:03 |
* TheJulia tosses things at NobodyCam | 20:03 | |
jlvillal | Naming is the hard part of computer science... | 20:03 |
TheJulia | indeed | 20:03 |
TheJulia | _handle_backwards_compat_api_states ? | 20:04 |
jlvillal | TheJulia: update_state_in_older_versions ? | 20:04 |
jlvillal | I'm looking at function above. | 20:04 |
jlvillal | hide_fields_in_newer_versions | 20:04 |
jlvillal | How I came up with my suggestion. | 20:04 |
JayF | def _this_is_the_review_that_doesnt_end() | 20:05 |
JayF | def _yes_it_goes_on_and_on_my_friends() | 20:05 |
JayF | :) | 20:05 |
jlvillal | LOL for real here :) | 20:05 |
*** bigjools has joined #openstack-ironic | 20:06 | |
*** bigjools has quit IRC | 20:06 | |
*** bigjools has joined #openstack-ironic | 20:06 | |
* jlvillal wonders if the Gerrit '/' search sucks in all browsers or just Firefox. So slow.... | 20:06 | |
TheJulia | JayF: lol | 20:09 |
mtreinish | jlvillal: it's all browsers | 20:09 |
jlvillal | mtreinish: Well that makes me feel better and sad at the same time. | 20:10 |
mtreinish | well I haven't tried it on lynx, but at least chrome and firefox are really bad with it | 20:10 |
* mtreinish tries lynx | 20:10 | |
JayF | who uses lynx? | 20:10 |
JayF | elinks ftw | 20:10 |
JayF | gotta have that sweet, sweet js support | 20:10 |
jlvillal | I really dislike that doing CTRL-F in my browser does not work reliably in Gerrit. | 20:10 |
JayF | it does if you go to the unified diff | 20:11 |
JayF | that's what I started doing | 20:11 |
jlvillal | Ah | 20:11 |
NobodyCam | I know we can have a Thunderdome session at summit "two names enter one name...." | 20:11 |
JayF | not for all reviews, but for when I'm looking for something in particular | 20:11 |
TheJulia | NobodyCam: lol | 20:11 |
jlvillal | NobodyCam: Good one :) | 20:12 |
NobodyCam | lol | 20:12 |
*** SpamapS has joined #openstack-ironic | 20:15 | |
*** karimb has quit IRC | 20:17 | |
JayF | TheJulia: re +2'd that IPA patch; thanks! | 20:20 |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Implement MegaRAC IPv6 SNMP destinations https://review.openstack.org/302817 | 20:22 |
JayF | NobodyCam: https://review.openstack.org/#/c/275726 I just added my +2 to this (OneView spec), IDK if any other cores have interest in this, so you might wanna land it now | 20:23 |
JayF | NobodyCam: I'll leave that to you or someone who is more familiar overall with the hp oneview stuff though | 20:24 |
NobodyCam | I'd like lucasagomes to be able to have a look at it, as he had a +2 before it changed | 20:24 |
NobodyCam | so I may hold off to monday | 20:25 |
JayF | that works for me | 20:25 |
NobodyCam | :) | 20:25 |
JayF | just figured that one has been through the ringer a little bit | 20:25 |
JayF | might make someones' weekend to get it in, lol | 20:25 |
NobodyCam | ++++ | 20:25 |
*** moshele has quit IRC | 20:25 | |
NobodyCam | yea, but he signed off already | 20:25 |
NobodyCam | lol | 20:25 |
*** daemontool has joined #openstack-ironic | 20:26 | |
TheJulia | JayF: NP | 20:28 |
openstackgerrit | Gonéri Le Bouder proposed openstack/ironic: iscsi: wipe the disk before deployment https://review.openstack.org/303605 | 20:32 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Remove deprecated option names in deploy_utils.py https://review.openstack.org/303081 | 20:36 |
*** jayc has joined #openstack-ironic | 20:42 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: Rename juno name state modification method https://review.openstack.org/302224 | 20:44 |
* TheJulia expects new paint colors to be proposed :) | 20:44 | |
*** baoli_ has joined #openstack-ironic | 20:46 | |
*** baoli has quit IRC | 20:47 | |
* NobodyCam clicks | 20:48 | |
NobodyCam | TheJulia: the new color is quite nice | 20:50 |
JayF | TheJulia: jlvillal had requested a docstring, do you mind adding one real quick before it causes +2s to get ruined? | 20:50 |
*** piet has quit IRC | 20:51 | |
TheJulia | JayF: context please | 20:53 |
JayF | TheJulia: bikeshed patch | 20:53 |
JayF | TheJulia: lol | 20:53 |
NobodyCam | i've been calling it hte purple patch | 20:53 |
TheJulia | I'm laughing and sighing at the same moment :) | 20:53 |
NobodyCam | I still like the thunderdome idea | 20:53 |
cloudnull | hey all, I'm seeing a connection problem with the ironic_python_agent -- http://snag.gy/A8T94.jpg -- I know its because the API is on load balancer VIP address the kicked host does not a route to it. so my question is, is there a way to configure that to be something else within the ironic.conf? | 20:54 |
TheJulia | oh, I didn't even see the mention for docstring | 20:54 |
cloudnull | looked through the sample file but im not seeing anything obvious | 20:54 |
* TheJulia updates | 20:54 | |
cloudnull | though it s friday so i could be seeing though my friday glasses | 20:54 |
JayF | cloudnull: configure /what/ to be something else? | 20:55 |
cloudnull | the ironic api client seems to want to talk back to the ironic api on my load balancer | 20:55 |
cloudnull | the kicked host is on a flat network via neutron | 20:56 |
JayF | You still haven't told me what setting you're trying to change, lol | 20:56 |
cloudnull | JayF: idk thats what im asking . | 20:56 |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Add notifications to Ironic https://review.openstack.org/298461 | 20:56 |
JayF | you want to add a route to the booted ramdisk? Change the ironic api url it's using? What exactly | 20:56 |
cloudnull | the ironic python client is getting that address , from the agent template? is that correct? | 20:57 |
JayF | Ah, I can show you | 20:57 |
cloudnull | show me the way | 20:57 |
cloudnull | :) | 20:57 |
JayF | https://github.com/openstack/ironic/blob/4482eaa59b3ad4f92d7097231100260a487215f4/ironic/drivers/modules/agent_config.template | 20:59 |
JayF | trying to find where that's populated though | 20:59 |
* TheJulia feels like there is a reference to a musical, but goes back to docstring | 20:59 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Remove deprecated option names in deploy_utils.py https://review.openstack.org/303081 | 21:00 |
JayF | cloudnull: https://github.com/openstack/ironic/blob/4482eaa59b3ad4f92d7097231100260a487215f4/ironic/drivers/modules/deploy_utils.py#L980 | 21:00 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Rename juno name state modification method https://review.openstack.org/302224 | 21:00 |
*** Sukhdev has quit IRC | 21:00 | |
JayF | cloudnull: alternatively, edit that agent_config.template to take whatever static value you want (it's a diff file for ipxe) if you can't/don't want to change that option | 21:00 |
cloudnull | ah ha! | 21:00 |
JayF | cloudnull: I also could see a bug/rfe for being able to make agents see a different ipa-api-url | 21:00 |
cloudnull | i can change it , i have the vi technology :) | 21:01 |
JayF | lol, well it may not be great to have it be static | 21:01 |
JayF | although I guess given it just uses that config option, already is kinda static | 21:01 |
cloudnull | is it possible for the ironic python agent to configure more than one network which could used for providing a route back to the control plain? | 21:03 |
cloudnull | asking because I've not tried it | 21:03 |
JayF | You can easily build custom IPA images that configure in pretty much whatever way you want | 21:03 |
cloudnull | ok. | 21:03 |
JayF | if you're using neutron though, I bet you can configure it for that network to push down an extra route over dhcp | 21:03 |
cloudnull | makes sense | 21:03 |
JayF | although I'm not 100% sure how you'd do that | 21:04 |
cloudnull | yea i can configure the hosts, and add routes, etc, though i'd like to keep things totally separated if at all possible. | 21:04 |
JayF | yep, I mean both ways work | 21:04 |
*** daemontool has quit IRC | 21:05 | |
cloudnull | i may go the route of custom image. | 21:05 |
JayF | the static editing of the template is likely to get you to "working" faster | 21:05 |
*** pece has quit IRC | 21:05 | |
cloudnull | depends on how much hair i want to give to it | 21:05 |
JayF | I mean, the custom image stuff is really super simple | 21:05 |
*** trown is now known as trown|outtypewww | 21:05 | |
JayF | cloudnull: if you're using coreos builder, it's all in here; https://github.com/openstack/ironic-python-agent/tree/master/imagebuild/coreos | 21:06 |
cloudnull | sadly i cant use that. though i tried. | 21:06 |
JayF | Why not? | 21:06 |
cloudnull | i need a super new kernel to work with the hosts we have in the OSIC | 21:06 |
cloudnull | or i need to slipstream the drivers in an older kernel | 21:07 |
JayF | oh, that'll be easy to fix | 21:07 |
*** raildo is now known as raildo-afk | 21:07 | |
JayF | coreos is by far the quickest route to a ramdisk with shiny new kernels | 21:07 |
cloudnull | yea. i went down the route of diskimage builder. | 21:07 |
JayF | cloudnull: https://github.com/openstack/ironic-python-agent/blob/master/imagebuild/coreos/coreos-oem-inject.py + change COREOS_VERSION to something newer. | 21:07 |
cloudnull | which is what i have working now. | 21:07 |
JayF | cloudnull: for the DIB image, I don't know much about setting that up; we use the coreos one for OnMetal, but either way you can use the pxe template edit to get past your headache | 21:08 |
*** piet has joined #openstack-ironic | 21:10 | |
* JayF goes off looking for requirements patches to blindly +1 to improve his stats /s | 21:12 | |
cloudnull | hahaha cheers | 21:12 |
cloudnull | thanks for the assist | 21:12 |
openstackgerrit | Jay Faulkner proposed openstack/ironic: Remove deprecated option names in deploy_utils.py https://review.openstack.org/303081 | 21:13 |
JayF | jlvillal: ^ fixed your commit message nit with inline editing if you wanna revote now | 21:13 |
jlvillal | JayF: Thanks! | 21:14 |
*** jjohnson2 has quit IRC | 21:14 | |
openstackgerrit | Anup Navare proposed openstack/ironic: Add node UUID to messages/exceptions https://review.openstack.org/301350 | 21:15 |
*** karimb has joined #openstack-ironic | 21:23 | |
JayF | mariojv: a thought about the notifications implementation for power state changes; there are some cases where Ironic sees a power state change that it didn't initiate (i.e. if you have the option set where Ironic updates the DB instead of the power state if it polls and the state has changed) | 21:23 |
JayF | mariojv: should we ensure those cases emit notifications too? i.e. "power state changed without ironic intervention" notifications in addition to "ironic changed the power state" notifications | 21:23 |
mariojv | maybe - where does that detection happen? | 21:24 |
openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Create helper function: create_hdparm_info() https://review.openstack.org/303620 | 21:25 |
jlvillal | TheJulia: You may or may not like that patch ^^^^ | 21:26 |
mariojv | that may be better in a separate notification (or we should rename this notification), on second thought, since this one is explicitly "*set*_power_state" | 21:27 |
JayF | mariojv: https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L2391 the conf setting it differs on is L2471 | 21:27 |
JayF | mariojv: I do think it should be a different notification, but I also think you need this one too or else there's going to be some really confusing results for people who run with force_power_state_during_sync on | 21:27 |
JayF | mariojv: i.e. they'd see a notification that a node was powered on, never see a power off notification, then see power on again | 21:28 |
JayF | mariojv: this may be stuff that was better suited to the spec, but was just thinking about how this would look as a consumer | 21:28 |
*** alexpilotti has joined #openstack-ironic | 21:28 | |
mariojv | i think at a minimum we definitely need a new start notification here: https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L2478 | 21:32 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Remove deprecated group and option names in deploy_utils.py https://review.openstack.org/303081 | 21:32 |
mariojv | since then you'll see the .end or .fail with no start during the sync | 21:32 |
*** rama_y has joined #openstack-ironic | 21:32 | |
openstackgerrit | Gonéri Le Bouder proposed openstack/ironic: iscsi: wipe the disk before deployment https://review.openstack.org/303605 | 21:33 |
*** cfarquhar_ has quit IRC | 21:33 | |
*** alexpilotti has quit IRC | 21:33 | |
JayF | mariojv: Just was thinking if the goal of notifications is being able to track node state | 21:34 |
mariojv | JayF: what do you think about having a new notification like baremetal.node.sync_power_state.{start, fail, end}? | 21:34 |
JayF | mariojv: you have to track the real-world state change, not just when Ironic pushes a button and makes the change | 21:34 |
jlvillal | JayF: Updated commit message again :) ^^^ | 21:34 |
TheJulia | jlvillal: I kind of like it... :) | 21:34 |
*** cfarquhar has joined #openstack-ironic | 21:35 | |
*** cfarquhar has quit IRC | 21:35 | |
*** cfarquhar has joined #openstack-ironic | 21:35 | |
jlvillal | TheJulia: Thanks. When I saw all those you had to add for your patch I thought, maybe we can make it a little easier... | 21:35 |
mariojv | then it'd look something like this: sync start, set power state start, set power state end, sync power state end (in the normal case where it gets out of sync) | 21:35 |
JayF | mariojv: I don't think that addresses the real thing, and I'd argue a sync_power_state inconsistency isn't DIRECTLY a failure | 21:35 |
JayF | mariojv: because a tenant can power control in band without Ironic's intervention; the states getting misaligned isn't a failure case, it's just a case | 21:35 |
JayF | mariojv: but that may be getting more pedantic than needed :) | 21:36 |
mariojv | right, .start and .end don't indicate failure | 21:36 |
mariojv | .fail indicates failure | 21:36 |
TheJulia | jlvillal: there are a few minor aspects i'm letting simmer while I make dinner :) | 21:36 |
mariojv | :P | 21:36 |
jlvillal | :) | 21:36 |
JayF | but then I don't understand how having the sync power state notifications help then? | 21:36 |
mariojv | you can include a new field in those notifs that shows the actual state of the node ironic looked up | 21:37 |
mariojv | so from a consumer perspective, you see sync_power_state.start | 21:37 |
mariojv | maybe sync_power_state.update or whatever we want to call it when we see it's changed and action is needed | 21:37 |
JayF | could you emit something like, sync_power_state.update if it updates the db | 21:37 |
mariojv | then that goes to utils.node_power_action | 21:37 |
JayF | and if it just updates teh actual node power state a power action would be triggered | 21:37 |
mariojv | emits set notifs, then power_sync.end | 21:38 |
JayF | the only thing that I would caution about this approach is it changes the notifications to be way more frequent | 21:38 |
JayF | because power state doesn't change nearly as often for a node as the power state sync runs against it | 21:38 |
JayF | so I guess I think only emitting in the case where the power state is updated in DB || if the power state is changed on the BMC as a result of the sync | 21:39 |
mariojv | true | 21:39 |
JayF | and AIUI the other piece, which uses power action, would emit already | 21:39 |
JayF | so you're just missing like a "fake" power state update | 21:39 |
JayF | you could probably even add a field in the notification to say whether it was ironic initiated or ironic observed | 21:39 |
JayF | except using better words than I did for that, lol | 21:39 |
mariojv | maybe have the .start and .end as DEBUG and the .update or whatever indicates that it's been changed in the hw as INFO | 21:39 |
JayF | hmm. It'd still be probably one of the noisiest notifications, right? | 21:40 |
JayF | have you thought of other examples of debug level notifications? | 21:40 |
mariojv | depends on how often you sync | 21:40 |
JayF | that's basically a guarantee that for N nodes, by default at least, you'll get N notifications every 5 minutes, right? | 21:40 |
JayF | power sync defaults 5m I think | 21:40 |
JayF | cfg.IntOpt('sync_power_state_interval', | 21:41 |
JayF | default=60, | 21:41 |
JayF | nope, 60s | 21:41 |
JayF | so that'd be exceptionally chatty | 21:41 |
TheJulia | I was going to say I think it is 1m | 21:41 |
mariojv | hmm | 21:41 |
JayF | and either way, as an operator, why do I care if the sync starts? | 21:41 |
JayF | I only care if it found something anomolous and takes action | 21:41 |
mariojv | i think i agree | 21:44 |
mariojv | i viewed DEBUG notifs originally as the spammiest thing you can have, like DEBUG logs | 21:44 |
*** piet has quit IRC | 21:45 | |
JayF | So I think the easiest way to make this make sense | 21:45 |
mariojv | but maybe the sync would be too spammy even for that | 21:45 |
JayF | is to have a field on the notification to say who performed the power action, or something similar | 21:45 |
JayF | when Ironic takes the power action, it would be Ironic | 21:45 |
JayF | otherwise it could be "node" or something else to indicate the power state changed outside ironic | 21:45 |
JayF | so when sync_power_state finds an inconsistency, it notifies a power state change | 21:45 |
JayF | then would notify again via node_power_action when it rectified it, if you had the option to do so enabled | 21:46 |
*** piet has joined #openstack-ironic | 21:46 | |
JayF | mariojv: summarized all this in the review, lmk if you think it reflects things fairly | 21:48 |
mariojv | i think it's a good idea | 21:49 |
JayF | I'm just thinking of what we wanted downstream, a kind of "audit log" for node actions | 21:49 |
mariojv | i think we also want to notify on errors in the sync, in addition to changes, like here https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L2418 | 21:49 |
JayF | and without this, you can't really do that, because you're missing an important piece | 21:49 |
JayF | mariojv: sure; generally, but that'd probably be a different patch than the initial one just dealing with power stuff? | 21:50 |
mariojv | so if i.e. a BMC network becomes unreachable, you get a lot of error notifs that you can alert on | 21:50 |
JayF | mariojv: unless we already send error notifications when we fail to change power state? | 21:50 |
mariojv | we do, but that piece of code is for when we retrieve the state during the sync | 21:50 |
mariojv | just extending that idea of having a full power state history of the node | 21:51 |
JayF | OK, I think it makes sense all together then | 21:51 |
mariojv | "I don't know what's here" is another "state" | 21:51 |
JayF | yeah, exactly | 21:51 |
openstackgerrit | Merged openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/302853 | 21:51 |
JayF | +1 | 21:51 |
jlvillal | NobodyCam, TheJulia, JayF: What is our policy on proposal bot changes to requirements? Is it two +2s or one +2? I'm thinking two +2s, but wanted to be sure. | 21:53 |
JayF | jlvillal: IDK what the policy is, but we rarely have trouble getting 2x+2 | 21:53 |
JayF | we have enough folks paying attention that they don't get lost in the shuffle | 21:53 |
jlvillal | Yeah. Which reminds me I wonder if the stable gate for python-ironicclient is broken. I see two approved requirements patches which have failed the gate. One for Liberty and the other for Kilo | 21:54 |
* jlvillal doesn't have the motivation to check it out on Friday afternoon with gorgeous weather outside.... | 21:55 | |
JayF | Wouldn't be too surprising, jroll had to do gymnastics to fix the "normal" gate for kilo/liberty | 21:55 |
SpamapS | Hey party people | 21:58 |
SpamapS | so I have an Ironic running next to a regular VM based nova.. | 21:59 |
SpamapS | trying to use host aggs in flavors to separate things | 21:59 |
SpamapS | but the problem I'm seeing is the scheduler that we use for ironic, picks up VM scheduling requests, and fails them because of the exact match filter | 21:59 |
*** aNupoisc has joined #openstack-ironic | 22:00 | |
*** piet has quit IRC | 22:00 | |
*** lascii is now known as alaski | 22:02 | |
*** karimb has quit IRC | 22:02 | |
*** wuhg has joined #openstack-ironic | 22:07 | |
*** piet has joined #openstack-ironic | 22:09 | |
*** kragniz has quit IRC | 22:09 | |
*** kragniz has joined #openstack-ironic | 22:10 | |
*** MattMan has quit IRC | 22:13 | |
*** MattMan has joined #openstack-ironic | 22:13 | |
SpamapS | jroll: ^^ any ieas? | 22:14 |
SpamapS | ideas | 22:14 |
*** suro-patz has quit IRC | 22:14 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Switch tox unit test command to use ostestr https://review.openstack.org/303637 | 22:15 |
JayF | SpamapS: he's moving right now, gone for most of next week too iirc | 22:16 |
JayF | SpamapS: Sorry I don't know anything about that stuff or I'd try to help ;( | 22:16 |
mtreinish | jlvillal: heh, I recognize most of that commit msg :) ^^^ | 22:16 |
jlvillal | mtreinish: :) Though I have to figure out how to do the 'cover' part | 22:17 |
jlvillal | I just realized | 22:17 |
mtreinish | jlvillal: normally I just use setup.py for that and just have a separate command for that tox job | 22:17 |
JayF | jlvillal: ...since when does IPA have working coverage jobs? | 22:18 |
jlvillal | mtreinish: Okay. we currently have: python setup.py testr --coverage {posargs:ironic_python_agent} | 22:18 |
*** baoli_ has quit IRC | 22:18 | |
*** shakamunyi has joined #openstack-ironic | 22:18 | |
jlvillal | JayF: There is a section in there :) | 22:18 |
JayF | that has no relevency to if it works ;P | 22:18 |
JayF | we used to have a pypy section at one point too | 22:19 |
JayF | lol | 22:19 |
jlvillal | mtreinish: Can I just do: s/testr/ostestr/ ?? | 22:19 |
mtreinish | jlvillal: yeah, that sounds right | 22:19 |
mtreinish | no | 22:19 |
mtreinish | because calling setup.py is where the --coverage flag is defined | 22:19 |
mtreinish | it's a pbr thing, ostestr doesn't have an equiv. (I woudln't be opposed to adding it though) | 22:20 |
jlvillal | mtreinish: Okay. Thanks. I wasn't sure since had removed 'testrepository' as a dependency. | 22:20 |
*** karimb has joined #openstack-ironic | 22:20 | |
mtreinish | jlvillal: https://github.com/openstack-dev/pbr/blob/master/pbr/testr_command.py#L117-L133 | 22:21 |
JayF | jlvillal: cover job doesn't work on master | 22:21 |
jlvillal | JayF: I'm noticing :( | 22:21 |
openstackgerrit | Merged openstack/ironic-python-agent: Provide fallback from ATA erase to shredding https://review.openstack.org/270902 | 22:21 |
mtreinish | JayF: what's wrong with it? | 22:21 |
JayF | I don't know, I'm 99% sure it never was made to work | 22:21 |
JayF | When we made IPA there was massive copying of configs, espeically around ci | 22:22 |
jlvillal | mtreinish: I think the :ironic_python_agent part makes it fail | 22:22 |
JayF | I suspect this is an artifiact of that | 22:22 |
jlvillal | python setup.py testr --coverage {posargs:ironic_python_agent} | 22:22 |
jlvillal | If I remove the ':ironic_python_agent' it does run | 22:22 |
mtreinish | jlvillal: oh I missed that, yeah I would remove that part | 22:22 |
JayF | jlvillal: that's what I'm seeing, too | 22:22 |
jlvillal | Ran 261 (+260) tests in 0.929s (+0.761s) | 22:22 |
jlvillal | PASSED (id=132) | 22:22 |
jlvillal | No data to report. | 22:22 |
JayF | yeah, it runs, but it doesn't do anything | 22:22 |
*** shakamunyi has quit IRC | 22:22 | |
JayF | Coverage.py warning: Module ironic-python-agent was never imported. | 22:23 |
JayF | Coverage.py warning: No data was collected. | 22:23 |
JayF | before that seems to be relevant | 22:23 |
*** Marga_ has quit IRC | 22:23 | |
JayF | jlvillal: python setup.py testr --coverage --coverage-package-name ironic_python_agent | 22:23 |
JayF | jlvillal: seems to make it even happier | 22:23 |
jlvillal | Ah cool | 22:24 |
mtreinish | JayF: yeah, packages where the repo name doesn't match the package name need that flag | 22:24 |
JayF | yeah, that's the case here | 22:24 |
mtreinish | basically everything with dashes has to use it :) | 22:24 |
jlvillal | JayF: python setup.py testr --coverage --coverage-package-name ironic_python_agent {posargs} | 22:25 |
mtreinish | jlvillal, JayF: fwiw, you can set it in a .coveragerc instead of on the cli, like: https://github.com/openstack/tempest-lib/blob/master/.coveragerc | 22:25 |
jlvillal | error: option --coverage-package-name not recognized | 22:26 |
mtreinish | jlvillal: try switching testr to test | 22:26 |
JayF | fwiw it wfm with testr | 22:26 |
JayF | fedora 23, ipa master | 22:26 |
*** kragniz has quit IRC | 22:27 | |
jlvillal | Hmmm. Works with 'test' on Fedora 22. | 22:27 |
*** karimb has quit IRC | 22:27 | |
jlvillal | I did remove the 'testrepository' package from the test-requirements.txt if that would make any difference | 22:27 |
*** karimb has joined #openstack-ironic | 22:27 | |
jlvillal | JayF: Does 'test' work for you too? | 22:28 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [snmp] https://review.openstack.org/303644 | 22:28 |
JayF | appears to have done the exact same thing | 22:28 |
*** Marga__ has joined #openstack-ironic | 22:28 | |
*** kragniz has joined #openstack-ironic | 22:28 | |
jlvillal | JayF: What do you mean? Also any interest in pushing your changes as a patch? | 22:29 |
mtreinish | jlvillal: it's a pbr quirk don't remember what causes it | 22:29 |
jlvillal | With an addition to .gitignore for 'cover'? | 22:29 |
JayF | jlvillal: I wasn't making changes, just trying to help you test since I saw the patch and that you said cover had to be taken care of :) | 22:29 |
*** alexpilotti has joined #openstack-ironic | 22:29 | |
JayF | jlvillal: I never let abide an opportunity to just see what happens when I do something :) | 22:30 |
jlvillal | JayF: Okay. I can push a separate patch then. | 22:30 |
JayF | jlvillal: unless you want me to push a patch fixing cover/ | 22:30 |
JayF | jlvillal: oh, if you're going to do a sep patch I'll do it real quick | 22:30 |
jlvillal | JayF: If you like too. | 22:30 |
JayF | since my repo is already setup for it | 22:30 |
jlvillal | sounds great :) | 22:30 |
jlvillal | The .gitignore should be updated to JayF | 22:30 |
*** davidlenwell has quit IRC | 22:32 | |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Fix coverage for IPA https://review.openstack.org/303646 | 22:32 |
JayF | wow, 97% test coverage | 22:33 |
*** Marga__ has quit IRC | 22:33 | |
JayF | I don't know why, but I was expecting that to be much worse | 22:33 |
JayF | how can it be showing <100% coverage on *test* files? | 22:33 |
*** alexpilotti has quit IRC | 22:34 | |
*** aNupoisc has quit IRC | 22:34 | |
*** davidlenwell has joined #openstack-ironic | 22:40 | |
*** aNupoisc has joined #openstack-ironic | 22:40 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [ssh] https://review.openstack.org/303649 | 22:41 |
jlvillal | JayF: +2 and thanks. | 22:42 |
openstackgerrit | Anup Navare proposed openstack/ironic: In node_power_action() add node.UUID to log message https://review.openstack.org/301350 | 22:48 |
*** kragniz has quit IRC | 22:49 | |
*** kragniz has joined #openstack-ironic | 22:49 | |
*** karimb has quit IRC | 22:53 | |
*** karimb has joined #openstack-ironic | 22:54 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [swift] https://review.openstack.org/303653 | 22:54 |
*** suro-patz has joined #openstack-ironic | 22:55 | |
*** Marga_ has joined #openstack-ironic | 22:56 | |
jlvillal | https://twitter.com/sempf/status/514473420277694465?lang=en Probably an old one, but I saw it for the first time this week | 22:59 |
*** suro-patz has quit IRC | 23:00 | |
*** piet has quit IRC | 23:00 | |
*** karimb has quit IRC | 23:01 | |
*** cdearborn has quit IRC | 23:03 | |
*** jayc has quit IRC | 23:04 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [virtualbox] https://review.openstack.org/303655 | 23:05 |
*** aNupoisc has quit IRC | 23:09 | |
*** aNupoisc has joined #openstack-ironic | 23:12 | |
*** garthb has quit IRC | 23:12 | |
*** suro-patz has joined #openstack-ironic | 23:14 | |
*** aNupoisc has quit IRC | 23:19 | |
*** Marga_ has quit IRC | 23:25 | |
*** suro-patz has quit IRC | 23:25 | |
*** suro-patz1 has joined #openstack-ironic | 23:25 | |
*** Marga_ has joined #openstack-ironic | 23:26 | |
*** alexpilotti has joined #openstack-ironic | 23:30 | |
*** alexpilotti has quit IRC | 23:34 | |
*** suro-patz1 has quit IRC | 23:50 | |
*** rama_y has quit IRC | 23:55 | |
*** piet has joined #openstack-ironic | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!