| *** suro-patz has quit IRC | 00:05 | |
| *** chlong has quit IRC | 00:11 | |
| *** rama_y has quit IRC | 00:12 | |
| *** Sukhdev has quit IRC | 00:13 | |
| *** garthb has quit IRC | 00:18 | |
| *** achanda has quit IRC | 00:19 | |
| *** mtanino has quit IRC | 00:24 | |
| *** sambetts|afk has quit IRC | 00:26 | |
| *** jcoufal has joined #openstack-ironic | 00:27 | |
| *** sambetts has joined #openstack-ironic | 00:33 | |
| *** praneshp_ has quit IRC | 00:37 | |
| *** spandhe has quit IRC | 00:42 | |
| *** shu-mutou-AFK is now known as shu-mutou | 00:43 | |
| *** Sukhdev has joined #openstack-ironic | 00:49 | |
| *** Sukhdev has quit IRC | 00:49 | |
| *** jaybeale has quit IRC | 00:50 | |
| *** piet has quit IRC | 00:56 | |
| *** achanda has joined #openstack-ironic | 01:19 | |
| *** jaybeale has joined #openstack-ironic | 01:20 | |
| *** piet has joined #openstack-ironic | 01:22 | |
| *** achanda has quit IRC | 01:23 | |
| *** vishwanathj has quit IRC | 01:26 | |
| *** penick has quit IRC | 01:30 | |
| *** achanda has joined #openstack-ironic | 01:34 | |
| *** mtanino has joined #openstack-ironic | 01:34 | |
| *** chlong has joined #openstack-ironic | 01:36 | |
| *** baoli has joined #openstack-ironic | 01:42 | |
| openstackgerrit | Merged openstack/ironic: Add xinetd and its TFTP configuration in Install Guide https://review.openstack.org/261429 | 01:45 |
|---|---|---|
| *** ChrisAusten has joined #openstack-ironic | 01:45 | |
| *** hoangcx has joined #openstack-ironic | 01:50 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-lib: Support configdrive in iscsi deploy for raw images https://review.openstack.org/230924 | 01:52 |
| *** baoli has quit IRC | 01:52 | |
| *** Cameron_C has quit IRC | 01:55 | |
| *** cameron has joined #openstack-ironic | 01:57 | |
| *** baoli has joined #openstack-ironic | 01:58 | |
| *** spandhe has joined #openstack-ironic | 01:58 | |
| *** achanda has quit IRC | 01:59 | |
| *** saripurigopi1 has quit IRC | 01:59 | |
| *** mtreinish has quit IRC | 01:59 | |
| *** harshs has quit IRC | 02:00 | |
| *** mtreinish has joined #openstack-ironic | 02:01 | |
| *** achanda has joined #openstack-ironic | 02:05 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Support configdrive in iscsi deploy for raw images https://review.openstack.org/225115 | 02:11 |
| *** baoli has quit IRC | 02:13 | |
| *** baoli has joined #openstack-ironic | 02:18 | |
| *** baoli has quit IRC | 02:20 | |
| *** jaybeale has quit IRC | 02:22 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Restart consoles on conductor startup https://review.openstack.org/257252 | 02:23 |
| *** baoli has joined #openstack-ironic | 02:29 | |
| *** baoli has quit IRC | 02:31 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Remove AMT driver https://review.openstack.org/296867 | 02:33 |
| *** cdearborn has quit IRC | 02:38 | |
| *** baoli has joined #openstack-ironic | 02:39 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic-specs: Nova serial console support. https://review.openstack.org/296869 | 02:41 |
| *** baoli has quit IRC | 02:41 | |
| *** baoli has joined #openstack-ironic | 02:42 | |
| *** sivaramakrishna has joined #openstack-ironic | 02:43 | |
| *** jcoufal has quit IRC | 02:47 | |
| *** baoli has quit IRC | 02:48 | |
| sivaramakrishna | sambetts:can you review https://review.openstack.org/#/c/213262 | 02:49 |
| *** baoli has joined #openstack-ironic | 02:51 | |
| *** baoli has quit IRC | 02:54 | |
| *** vishwanathj has joined #openstack-ironic | 03:02 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-python-agent: Use ironic-lib to create configdrive https://review.openstack.org/296466 | 03:06 |
| *** saripurigopi has joined #openstack-ironic | 03:09 | |
| *** piet has quit IRC | 03:09 | |
| *** praneshp_ has joined #openstack-ironic | 03:10 | |
| *** piet has joined #openstack-ironic | 03:22 | |
| *** mannidi_ has joined #openstack-ironic | 03:24 | |
| *** hoangcx has quit IRC | 03:25 | |
| *** piet has quit IRC | 03:26 | |
| *** mannidi__ has joined #openstack-ironic | 03:26 | |
| *** hoangcx has joined #openstack-ironic | 03:26 | |
| *** links has joined #openstack-ironic | 03:27 | |
| *** mannidi_ has quit IRC | 03:30 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic-specs: Dynamically allocate console ports https://review.openstack.org/249876 | 03:35 |
| *** praneshp__ has joined #openstack-ironic | 03:43 | |
| *** praneshp_ has quit IRC | 03:46 | |
| *** achanda has quit IRC | 03:49 | |
| *** appu103_ has joined #openstack-ironic | 03:52 | |
| *** mariojv has quit IRC | 03:53 | |
| *** alineb has quit IRC | 03:53 | |
| *** praneshp__ has quit IRC | 03:55 | |
| *** praneshp_ has joined #openstack-ironic | 03:56 | |
| *** Marga_ has quit IRC | 03:58 | |
| *** Marga_ has joined #openstack-ironic | 03:59 | |
| *** Marga_ has quit IRC | 03:59 | |
| *** Marga_ has joined #openstack-ironic | 03:59 | |
| *** spandhe has quit IRC | 04:01 | |
| *** absubram has joined #openstack-ironic | 04:07 | |
| *** absubram_ has joined #openstack-ironic | 04:08 | |
| *** absubram has quit IRC | 04:12 | |
| *** absubram_ is now known as absubram | 04:12 | |
| *** mariojv has joined #openstack-ironic | 04:16 | |
| *** jaybeale has joined #openstack-ironic | 04:16 | |
| *** praneshp_ has quit IRC | 04:23 | |
| *** vishwanathj is now known as vishwanathj_zzz | 04:24 | |
| *** penick has joined #openstack-ironic | 04:33 | |
| *** absubram has quit IRC | 04:36 | |
| *** appu103_ is now known as aparnav | 04:39 | |
| *** jaybeale has quit IRC | 04:52 | |
| *** chlong has quit IRC | 04:56 | |
| *** watanabe_isao has joined #openstack-ironic | 04:57 | |
| *** achanda has joined #openstack-ironic | 04:59 | |
| watanabe_isao | Hello team. I noticed ironic code has been removed from tree of devstack. Does anyone know what should I do to install ironic with Devstack now? | 04:59 |
| watanabe_isao | ^^^BTW, the commit was https://github.com/openstack-dev/devstack/commit/1195a5b7394fc5b7a1cb1415978e9997701f5af1 | 05:02 |
| watanabe_isao | 29 Dec 2015 | 05:02 |
| *** baoli has joined #openstack-ironic | 05:06 | |
| *** baoli has quit IRC | 05:11 | |
| *** chlong has joined #openstack-ironic | 05:12 | |
| *** penick_ has joined #openstack-ironic | 05:16 | |
| *** penick has quit IRC | 05:17 | |
| *** penick_ is now known as penick | 05:17 | |
| watanabe_isao | enable_plugin ironic git://git.openstack.org/openstack/ironic | 05:18 |
| watanabe_isao | ^^^Figure out myself.:) | 05:18 |
| *** penick has quit IRC | 05:35 | |
| *** mtanino has quit IRC | 05:35 | |
| *** alineb has joined #openstack-ironic | 05:41 | |
| *** harlowja_at_home has quit IRC | 05:43 | |
| *** pcaruana has quit IRC | 05:48 | |
| *** chlong has quit IRC | 06:08 | |
| openstackgerrit | Aparna proposed openstack/ironic: Doc update to enable HTTPS in Glance and Ironic comm https://review.openstack.org/280533 | 06:10 |
| *** suro-patz has joined #openstack-ironic | 06:22 | |
| *** chlong has joined #openstack-ironic | 06:22 | |
| *** rcernin has joined #openstack-ironic | 06:23 | |
| *** ChubYann has quit IRC | 06:39 | |
| *** suro-patz has quit IRC | 06:42 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Make use of oslo-config-generator https://review.openstack.org/247331 | 06:42 |
| *** ishant has joined #openstack-ironic | 06:45 | |
| *** moshele has joined #openstack-ironic | 06:48 | |
| sivaramakrishna | lazy_prince: good afternoon! | 06:50 |
| *** saibarspeis has joined #openstack-ironic | 07:00 | |
| *** hoangcx has quit IRC | 07:01 | |
| *** hoangcx has joined #openstack-ironic | 07:04 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add nodes tagging suport - objs https://review.openstack.org/233357 | 07:05 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add DB layer support for node tags filter https://review.openstack.org/272624 | 07:05 |
| *** lifeless has quit IRC | 07:06 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add nodes tagging suport - objs https://review.openstack.org/233357 | 07:17 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add DB layer support for node tags filter https://review.openstack.org/272624 | 07:17 |
| *** Marga_ has quit IRC | 07:27 | |
| *** saripurigopi has quit IRC | 07:30 | |
| *** pcaruana has joined #openstack-ironic | 07:32 | |
| *** daemontool has joined #openstack-ironic | 07:33 | |
| *** achanda has quit IRC | 07:33 | |
| *** achanda has joined #openstack-ironic | 07:37 | |
| *** Marga_ has joined #openstack-ironic | 07:40 | |
| *** tesseract has joined #openstack-ironic | 07:41 | |
| *** tesseract is now known as Guest27982 | 07:42 | |
| *** alexpilo_ has quit IRC | 07:43 | |
| *** Marga_ has quit IRC | 07:45 | |
| *** ndipanov has quit IRC | 07:46 | |
| *** alexpilotti has joined #openstack-ironic | 07:50 | |
| *** achanda has quit IRC | 07:56 | |
| *** ifarkas has joined #openstack-ironic | 07:59 | |
| *** athomas has joined #openstack-ironic | 08:09 | |
| *** achanda has joined #openstack-ironic | 08:10 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Make use of oslo-config-generator https://review.openstack.org/247331 | 08:17 |
| openstackgerrit | Haomeng,Wang proposed openstack/ironic-python-agent: add new portal_port argument into iscsi.start_iscsi_target api https://review.openstack.org/272420 | 08:23 |
| *** mbound has joined #openstack-ironic | 08:26 | |
| *** e0ne has joined #openstack-ironic | 08:29 | |
| *** chlong has quit IRC | 08:38 | |
| *** achanda has quit IRC | 08:45 | |
| *** mannidi__ has quit IRC | 08:48 | |
| *** saibarspeis has quit IRC | 08:49 | |
| *** early has quit IRC | 08:50 | |
| *** achanda has joined #openstack-ironic | 08:56 | |
| *** watanabe_isao has quit IRC | 08:59 | |
| *** shu-mutou is now known as shu-mutou-AFK | 09:00 | |
| *** sinh_ has joined #openstack-ironic | 09:07 | |
| *** sinh has quit IRC | 09:10 | |
| *** lucas-dinner is now known as lucasagomes | 09:11 | |
| lucasagomes | morning all | 09:12 |
| *** kromanenko has joined #openstack-ironic | 09:13 | |
| *** ndipanov has joined #openstack-ironic | 09:21 | |
| *** hoangcx has quit IRC | 09:23 | |
| *** hoangcx has joined #openstack-ironic | 09:25 | |
| pas-ha | morning Ironic | 09:25 |
| *** achanda has quit IRC | 09:26 | |
| *** saibarspeis has joined #openstack-ironic | 09:27 | |
| *** ndipanov has quit IRC | 09:28 | |
| *** ndipanov has joined #openstack-ironic | 09:29 | |
| alineb | morning all | 09:32 |
| *** jcoufal has joined #openstack-ironic | 09:33 | |
| *** athomas has quit IRC | 09:45 | |
| *** athomas has joined #openstack-ironic | 09:50 | |
| *** _degorenko|afk is now known as degorenko | 09:51 | |
| *** early has joined #openstack-ironic | 09:53 | |
| *** ishant has quit IRC | 09:57 | |
| *** ohamada has joined #openstack-ironic | 09:59 | |
| *** smoriya_ has quit IRC | 10:01 | |
| *** chihhsin has quit IRC | 10:07 | |
| *** moorryan has joined #openstack-ironic | 10:11 | |
| *** mkovacik__ has joined #openstack-ironic | 10:11 | |
| yuriyz | morning Ironic | 10:19 |
| BadCub | morning lucasagomes pas-ha alineb yuriyz | 10:21 |
| *** early has quit IRC | 10:21 | |
| *** electrofelix has joined #openstack-ironic | 10:22 | |
| pas-ha | morning BadCub | 10:23 |
| lucasagomes | BadCub, yuriyz pas-ha alineb good morning (-: | 10:23 |
| BadCub | heya lucasagomes Sorry we didn't make it over :( | 10:26 |
| alineb | o/ yuriyz BadCub pas-ha lucasagomes | 10:26 |
| lucasagomes | BadCub, hey, oh no worries about it. I'm sure we will have other opportunities to meet up in Dublin (or somewhere else) | 10:30 |
| *** early has joined #openstack-ironic | 10:30 | |
| BadCub | lucasagomes: absolutely!!! | 10:31 |
| sambetts | Morning all | 10:31 |
| *** mgould has joined #openstack-ironic | 10:33 | |
| sergek | o/ | 10:38 |
| openstackgerrit | Merged openstack/ironic: Delete bridge "brbm" in devstack/unstack.sh https://review.openstack.org/293900 | 10:38 |
| *** ndipanov has quit IRC | 10:38 | |
| *** hoangcx has quit IRC | 10:41 | |
| *** ohamada has quit IRC | 10:42 | |
| vdrok | morning ironic, lucasagomes BadCub alineb sambetts | 10:43 |
| *** daemontool has quit IRC | 10:46 | |
| sambetts | lucasagomes: I wonder if I've just found a bug in the Ironic host manager ... | 10:48 |
| lucasagomes | sambetts, hi there! | 10:49 |
| lucasagomes | oh what happened? | 10:49 |
| sambetts | lucasagomes: http://paste.openstack.org/show/491688/ | 10:50 |
| lucasagomes | sambetts, oh, strange how compute.free_disk_gb got set to None | 10:52 |
| sambetts | lucasagomes: yeah... I'm seeing this straight after I do a node-update | 10:52 |
| lucasagomes | sambetts, may worth looking at other host managers | 10:53 |
| lucasagomes | sambetts, scheduler/host_manager does something similar https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L190-L201 | 10:54 |
| lucasagomes | sambetts, maybe what we are missing is: https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L184-L186 | 10:55 |
| lucasagomes | to make sure it's populated I guess | 10:55 |
| * lucasagomes digs into it | 10:55 | |
| sambetts | lucasagomes: Either that or I wondered if there was a problem with the method that is retreiving the node information and sending it to the host manager | 10:56 |
| sambetts | lucasagomes: do you know where that happens? | 10:57 |
| lucasagomes | sambetts, right, all nodes has their properties (cpus, etc...) set right? | 10:57 |
| lucasagomes | sambetts, I believe get_info() or something like that in the driver | 10:57 |
| sambetts | lucasagomes: yup, when I'm seeing this is directly after running devstack sucessfully I update the node added by the IPMI devstack code to use my CIMC driver | 10:58 |
| sambetts | lucasagomes: and all the required fields are added by devstack | 10:59 |
| lucasagomes | right | 10:59 |
| sambetts | lucasagomes: it seems to be a timing thing because sometimes it works and sometimes I see this error, and it leads to my temepest tests failing :( | 10:59 |
| sambetts | lucasagomes: there is another log literally 3 milliseconds before and free_disk_db=0 in that log | 11:01 |
| *** aparnav has quit IRC | 11:01 | |
| lucasagomes | sambetts, wondering if this https://bugs.launchpad.net/ironic/+bug/1248022 has something to do | 11:01 |
| openstack | Launchpad bug 1248022 in Ironic "Nova scheduler not updated immediately when a baremetal node is added or removed" [Medium,Triaged] | 11:01 |
| sambetts | lucasagomes: http://paste.openstack.org/show/491690/ | 11:01 |
| lucasagomes | cause it takes a while to the things to get propagated (it's a periodic task) | 11:01 |
| *** ohamada has joined #openstack-ironic | 11:02 | |
| sambetts | lucasagomes: I added a 2 minute sleep to my script to try to make sure that happens :/ and I believe this is happening at the point that nova is trying update the resources :/ | 11:04 |
| *** sivaramakrishna has quit IRC | 11:06 | |
| *** saibarspeis has quit IRC | 11:06 | |
| *** lifeless has joined #openstack-ironic | 11:07 | |
| *** moorryan has quit IRC | 11:08 | |
| sambetts | lucasagomes: how does an Ironic compute node get added to nova? | 11:09 |
| *** baoli has joined #openstack-ironic | 11:11 | |
| *** jcoufal has quit IRC | 11:13 | |
| lucasagomes | sambetts, yeah I bet it may work with the 2min sleep (if updating the resources is the problem there) | 11:13 |
| lucasagomes | sambetts, not sure if I get the question correctly, but they just run another nova compute with the ironic driver loaded (and a different hostname) | 11:13 |
| sambetts | lucasagomes: Yeah, I've just found the code in the virt driver, I thought it might have been in the scheduler code | 11:14 |
| *** baoli_ has joined #openstack-ironic | 11:18 | |
| *** baoli_ has quit IRC | 11:20 | |
| *** baoli_ has joined #openstack-ironic | 11:20 | |
| *** baoli has quit IRC | 11:21 | |
| sambetts | lucasagomes: looking at the ComputeNode object in nova free_disk_gb is a nullable field and I guess it'll default to null if not set by the update_from_virt_driver method | 11:22 |
| lucasagomes | sambetts, oh :-/ right | 11:23 |
| sambetts | lucasagomes: https://github.com/openstack/nova/blob/master/nova/objects/compute_node.py#L345 | 11:23 |
| lucasagomes | yeah maybe that conditional at the beggining at the method checking if things were updated would be a good ida | 11:23 |
| *** electrofelix has quit IRC | 11:26 | |
| *** baoli_ has quit IRC | 11:31 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add flavor_id baremetal tempest option https://review.openstack.org/293020 | 11:31 |
| *** baoli has joined #openstack-ironic | 11:31 | |
| sambetts | :/ lucasagomes hmmm interestingly its not the node that I added that is causing the schedular to blow up | 11:34 |
| sambetts | lucasagomes: it seems to be one of the ones added during the tempest API scenario | 11:34 |
| *** baoli has quit IRC | 11:35 | |
| sambetts | lucasagomes: API tests* | 11:35 |
| *** baoli has joined #openstack-ironic | 11:35 | |
| lucasagomes | oh :-/ | 11:38 |
| lucasagomes | sambetts, perhaps we should handle TypeErrors on that method then (when we try to multiply the values and things like that) | 11:38 |
| *** ndipanov has joined #openstack-ironic | 11:44 | |
| sambetts | lucasagomes: hmm, really stuck... I'm just looking at the n-sch logs for a test run that passed and the logs look completely different, there's no "Update host state....." logs anywhere to be seen only "Successfully synced instances from host" messages | 11:50 |
| TheJulia | good morning | 11:51 |
| sambetts | o/ TheJulia | 11:51 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add flavor_id baremetal tempest option https://review.openstack.org/293020 | 11:51 |
| lucasagomes | sambetts, :-/ | 11:53 |
| lucasagomes | sambetts, so the 2 min delay didn't work ? | 11:53 |
| *** ohamada has quit IRC | 11:54 | |
| sambetts | lucasagomes: no, that n-sch error is happening during the tempest run after the node has synced correctly once, in the paste I sent you its trying to sync one of the nodes added by tempest using the fake driver... | 11:55 |
| *** mannidi__ has joined #openstack-ironic | 11:56 | |
| yhvh | sambetts, jroll: my colleague is working on this bug https://bugs.launchpad.net/ironic/+bug/1536572 and I'd like to confirm your position on the sshd method he proposed | 11:58 |
| openstack | Launchpad bug 1536572 in Ironic "[RFE] Add CLI serial over lan support to Ironic" [Wishlist,Confirmed] | 11:58 |
| *** trown|outtypewww is now known as trown | 11:58 | |
| yhvh | the conductor still starts the ipmitool connection, but this is accessed over ssh | 11:58 |
| yhvh | I don't see a security risk if keys are checked for validity | 11:59 |
| mkovacik__ | sambetts, hi, I just got some positive feedback on the reapply introspection over stored data patch; https://review.openstack.org/#/c/267637/16 would you mind reviewing? | 12:00 |
| mkovacik__ | ifarkas, hi, I could use your feedback, too ;) ^ | 12:01 |
| openstackgerrit | Haomeng,Wang proposed openstack/ironic-python-agent: add new portal_port argument into iscsi.start_iscsi_target api https://review.openstack.org/272420 | 12:01 |
| *** jcoufal has joined #openstack-ironic | 12:03 | |
| *** electrofelix has joined #openstack-ironic | 12:04 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: CoreOS image building: Include support dnf support https://review.openstack.org/297064 | 12:05 |
| *** daemontool has joined #openstack-ironic | 12:07 | |
| *** moshele has quit IRC | 12:08 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: CoreOS image building: Add dnf support https://review.openstack.org/297064 | 12:09 |
| * lucasagomes changes commit message | 12:09 | |
| *** daemontool has quit IRC | 12:15 | |
| *** mbound has quit IRC | 12:16 | |
| *** mbound has joined #openstack-ironic | 12:18 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add test for --wrap option for driver-properties output https://review.openstack.org/297077 | 12:25 |
| *** mtanino has joined #openstack-ironic | 12:25 | |
| *** sinval has joined #openstack-ironic | 12:27 | |
| sinval | good morning | 12:27 |
| *** chihhsin has joined #openstack-ironic | 12:27 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: CoreOS Image building: Ensure docker is started https://review.openstack.org/297079 | 12:28 |
| lucasagomes | jroll, ^ is an attempt to fix https://launchpad.net/bugs/1559383 (post job to build the image), I'm not sure how to test the post job tho, I can test the code locally only | 12:29 |
| openstack | Launchpad bug 1559383 in Ironic "IPA post jobs appear to be failing" [Medium,New] - Assigned to Lucas Alvares Gomes (lucasagomes) | 12:29 |
| *** electrofelix has quit IRC | 12:39 | |
| sambetts | lucasagomes: do we authenticate our shellinabox connections? | 12:48 |
| ifarkas | mkovacik__, sure, will look at the patch | 12:48 |
| mkovacik__ | ifarkas, thanks! | 12:49 |
| openstackgerrit | Merged openstack/bifrost: Correct test-bifrost-venv script https://review.openstack.org/296611 | 12:55 |
| *** mannidi__ has quit IRC | 12:58 | |
| *** mbound has quit IRC | 12:58 | |
| *** mbound has joined #openstack-ironic | 12:58 | |
| *** Goneri has joined #openstack-ironic | 13:00 | |
| *** piet has joined #openstack-ironic | 13:01 | |
| *** cdearborn has joined #openstack-ironic | 13:03 | |
| lucasagomes | sambetts, not by default AFAIK, I think you can configure it to | 13:05 |
| * lucasagomes checks | 13:05 | |
| lucasagomes | sambetts, https://review.openstack.org/#/c/289671/ | 13:05 |
| *** jaybeale has joined #openstack-ironic | 13:18 | |
| *** rbudden has joined #openstack-ironic | 13:22 | |
| *** links has quit IRC | 13:28 | |
| openstackgerrit | Haomeng,Wang proposed openstack/ironic-python-agent: add new portal_port argument into iscsi.start_iscsi_target api https://review.openstack.org/272420 | 13:30 |
| *** absubram has joined #openstack-ironic | 13:33 | |
| lucasagomes | sambetts, any progress on that problem? | 13:33 |
| *** krtaylor has quit IRC | 13:33 | |
| *** ChrisAusten has quit IRC | 13:34 | |
| *** absubram_ has joined #openstack-ironic | 13:34 | |
| *** moshele has joined #openstack-ironic | 13:34 | |
| sambetts | lucasagomes: not really, I'm wondering if it has something to do with the problem with the devstack tempest/ironic plugin loading order and not configuring something correctly | 13:34 |
| sambetts | lucasagomes: I'm pulling at straws though | 13:35 |
| *** absubram has quit IRC | 13:38 | |
| *** absubram_ is now known as absubram | 13:38 | |
| *** krtaylor has joined #openstack-ironic | 13:45 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 13:46 |
| *** jaybeale has quit IRC | 13:48 | |
| *** jaybeale has joined #openstack-ironic | 13:48 | |
| *** achanda has joined #openstack-ironic | 13:50 | |
| *** achanda has quit IRC | 13:50 | |
| *** daemontool has joined #openstack-ironic | 13:52 | |
| jroll | morning all :) | 13:55 |
| sambetts | jroll: o/ | 13:55 |
| jroll | yhvh: yeah, that's fine I think, needs a spec for sure | 13:55 |
| NobodyCam | Good Morning Ironicers | 13:55 |
| sambetts | jroll: I left a big comment on the RFE, I'd like to know what you thinl | 13:55 |
| jroll | yhvh: I just wanted to be clear that the client should not be initiating ipmi traffic but rather the conductor | 13:56 |
| jroll | sambetts: part of my comment yesterday was "tl;dr I agree with Sam here." | 13:56 |
| jroll | :) | 13:56 |
| jroll | lucasagomes: cool, though I'd be willing to bet it's the same bug as https://review.openstack.org/#/c/294343/ | 13:56 |
| lucasagomes | jroll, morning | 13:56 |
| sambetts | jroll: :) I'd like to see it align with the shellinabox implemention if possible | 13:57 |
| lucasagomes | sambetts, :-( ack | 13:57 |
| lucasagomes | jroll, yeah it could be related indeed | 13:57 |
| jroll | sambetts: yeah, just ssh instead of websockets? | 13:57 |
| sambetts | jroll: yup | 13:57 |
| jroll | nod | 13:57 |
| jroll | lucasagomes: you might need to ask in infra how to test | 13:57 |
| *** jaypipes has joined #openstack-ironic | 13:57 | |
| jroll | or actually I can | 13:57 |
| lucasagomes | jroll, up to u, I'm just finishing something here and I can ask them | 13:58 |
| jroll | I just did :P | 13:58 |
| lucasagomes | heh ok | 13:59 |
| lucasagomes | thanks | 13:59 |
| jroll | np, thanks for looking at that | 13:59 |
| sambetts | jroll: have you ever seen this in the n-sch logs http://paste.openstack.org/show/491688/ | 13:59 |
| sambetts | jroll: me and lucas are quite confused | 13:59 |
| jroll | sambetts: yeah I was reading about that in scrollback a bit, I don't believe I have | 14:00 |
| lucasagomes | np, in any case having the daemon to start automatically is useful (I always forget to start it on fedora) | 14:00 |
| jroll | sambetts: what's the chance you can get the full scheduler and compute logs? :) | 14:00 |
| sambetts | jroll: pretty easy | 14:00 |
| jroll | yeah, that would help | 14:00 |
| jroll | and you don't have any nova patches, right? | 14:01 |
| sambetts | jroll: No, I'm seeing this in my CI which is running straight devstack + ironic tempest | 14:01 |
| openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Extend python-oneviewclient to Logical Interconnect https://review.openstack.org/297171 | 14:01 |
| *** mgoddard_ has joined #openstack-ironic | 14:01 | |
| sambetts | jroll: http://192.133.158.2:8080/view/openstack%20ironic/job/tempest-dsvm-ironic-pxe_iscsi_cimc/472/ | 14:02 |
| jroll | sambetts: cool, yeah I can help if you have logs - btw, what's the node-update you mentioned you're doing? | 14:02 |
| jroll | nice | 14:02 |
| *** mtanino has quit IRC | 14:03 | |
| sambetts | jroll: basically lucasagomes added the code to the devstack plugin to enroll the nodes using the ipmi driver, so I let that do its work then after devstack is finished I flip the node over to use my CIMC driver | 14:04 |
| *** moshele has quit IRC | 14:04 | |
| jroll | sambetts: ah, so shouldn't affect it | 14:04 |
| *** mgoddard has quit IRC | 14:04 | |
| sambetts | jroll: it shouldn't I even added in a 2 minute wait after i do the node update to make sure that the hypervisor stats propogate after the node-update | 14:05 |
| sambetts | the node that's blowing up in the n-sch is one of the one added during the tempest API tests, as far as I can tell | 14:05 |
| jroll | sambetts: right | 14:06 |
| *** alexpilotti has quit IRC | 14:07 | |
| *** alexpilotti has joined #openstack-ironic | 14:07 | |
| *** ChrisAusten has joined #openstack-ironic | 14:09 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: Install apparmor b/c Docker.io has undeclared dep https://review.openstack.org/297177 | 14:10 |
| jroll | lucasagomes: ^ I think that's the real fix | 14:10 |
| NobodyCam | I thought that landed already :( | 14:10 |
| lucasagomes | jroll, nice, did you figure out how to test it? | 14:11 |
| jroll | NobodyCam: hehhhhhh. | 14:11 |
| jroll | lucasagomes: it can be done by infra admins logging into jenkins and manually running it with the new code | 14:11 |
| lucasagomes | jroll, gotcha +2 | 14:11 |
| jroll | so I vote we just land it (there's precedent here) and see what happens | 14:11 |
| lucasagomes | yeah won't hurt | 14:11 |
| jroll | NobodyCam: it keeps popping back up, first on devstack stable branches, now in the IPA build job | 14:12 |
| NobodyCam | ahh | 14:12 |
| *** alexpilotti has quit IRC | 14:12 | |
| jroll | lucasagomes: jay found this yesterday, we should land it as well https://review.openstack.org/#/c/296837/ | 14:13 |
| NobodyCam | jroll: +2, hold +a until test results are in | 14:13 |
| jroll | NobodyCam: sure, thanks | 14:13 |
| lucasagomes | jroll, ouch hah | 14:14 |
| lucasagomes | tricky man, +a | 14:14 |
| jroll | thanks | 14:15 |
| jroll | new images are faster apparently :) | 14:15 |
| *** alexpilotti has joined #openstack-ironic | 14:18 | |
| lucasagomes | o/ | 14:18 |
| *** mgoddard_ has quit IRC | 14:23 | |
| *** mgoddard has joined #openstack-ironic | 14:23 | |
| *** alexpilotti has quit IRC | 14:23 | |
| openstackgerrit | Merged openstack/ironic: Fix race in hash ring refresh unit test https://review.openstack.org/296837 | 14:23 |
| vdrok | morning jroll and NobodyCam ! | 14:23 |
| NobodyCam | morning vdrok :) | 14:24 |
| *** aginwala has joined #openstack-ironic | 14:24 | |
| yhvh | jroll, sambetts: thanks for clarification - we'll work up a spec and show the code we have so far | 14:24 |
| jroll | yhvh: perfect, thanks! | 14:24 |
| *** aginwala has quit IRC | 14:29 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Update reno for stable/mitaka https://review.openstack.org/296428 | 14:31 |
| jroll | that and this https://review.openstack.org/#/c/295591/ should be easy reviews, last things to open up newton | 14:31 |
| *** vishwanathj_zzz is now known as vishwanathj | 14:33 | |
| *** ametts has joined #openstack-ironic | 14:33 | |
| *** aginwala has joined #openstack-ironic | 14:35 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Move testcases related to parse_instance_info() https://review.openstack.org/297186 | 14:36 |
| mariojv | good morning ironic | 14:37 |
| vdrok | morning mariojv | 14:37 |
| NobodyCam | morning mariojv :) | 14:37 |
| jroll | aarefiev: hey, are you still working on the policy spec or can someone else pick that up | 14:38 |
| mariojv | does anyone happen to know if jcaustin is still working on notifications? i have quite a bit of code for it, not quite ready yet, but am wondering if it's a good use of time | 14:38 |
| jroll | devananda: ^ | 14:38 |
| mariojv | iirc he said something like having it in ~a week about a month ago, so i'm curious | 14:38 |
| jroll | mariojv: I think the plan was you two work together so at least put up what you have and add him as a reviewer, can go from there | 14:39 |
| aarefiev | jroll: hi, I'm not now | 14:39 |
| *** hparekh has quit IRC | 14:39 | |
| jroll | aarefiev: ok, thanks | 14:39 |
| mariojv | sgtm, ty jroll | 14:40 |
| jroll | np | 14:41 |
| sambetts | jroll: you set --concurrency=1 in the gate when running tempest, could that be the part I'm missing from my gate? | 14:44 |
| sambetts | jroll: your basically turning parallel testing off right? | 14:44 |
| jroll | sambetts: possibly, I didn't get to look at your logs yet | 14:44 |
| jroll | yes | 14:44 |
| jroll | I wonder if api tests are conflicting with the nova exercise | 14:44 |
| jroll | I need to step away for breakfast though | 14:45 |
| sambetts | Yeah thats what I'm thinking if they are running in parallel | 14:45 |
| *** absubram has quit IRC | 14:45 | |
| *** electrofelix has joined #openstack-ironic | 14:49 | |
| *** electrofelix has quit IRC | 14:50 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Use truorfalce function instead of specific value https://review.openstack.org/297192 | 14:51 |
| *** hparekh has joined #openstack-ironic | 14:52 | |
| *** piet has quit IRC | 14:54 | |
| *** mag009_ has joined #openstack-ironic | 14:54 | |
| mag009_ | hey | 14:54 |
| NobodyCam | hi mag009_ | 14:54 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Fix: Duplicated driver causes conductor to fail https://review.openstack.org/297194 | 14:54 |
| mag009_ | question regarding the uefi is it possible to boot from uefi using dhcp that is not controlled by neutron ? | 14:54 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Use truorfalse function instead of specific value https://review.openstack.org/297192 | 14:55 |
| lucasagomes | mag009_, you mean via PXE boot? | 14:56 |
| mag009_ | yep | 14:56 |
| lucasagomes | mag009_, yes, but you have to configure the external DHCP to send the right PXE/iPXE image for the EFI nodes | 14:56 |
| TheJulia | mag009_: yes, you just need to put the settings in dhcp accordingly | 14:56 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 14:56 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to build user image with DIB https://review.openstack.org/256363 | 14:56 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Link-Local-Connection info to ironic port https://review.openstack.org/256365 | 14:56 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Added operator documentation for ironic portgroups https://review.openstack.org/228496 | 14:56 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/206244 | 14:56 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: refactor ironic enroll-node code https://review.openstack.org/256364 | 14:56 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add configure_provision_network function https://review.openstack.org/256367 | 14:56 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update Ironic VM network connection https://review.openstack.org/256366 | 14:57 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 14:57 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 14:57 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: add network drivers https://review.openstack.org/285852 | 14:57 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Ironic/Neutron integration documentation https://review.openstack.org/258596 | 14:57 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to use NETWORK_DRIVERS in devstack https://review.openstack.org/293520 | 14:57 |
| NobodyCam | ya all are to quick to reply :) | 14:57 |
| NobodyCam | hehehe | 14:57 |
| mag009_ | well my dhcp poin to the file name : filename "elilo.efi" | 14:57 |
| mag009_ | but I guess I need to tell the dhcp to use the ram + kernel image ? | 14:57 |
| *** harlowja_at_home has joined #openstack-ironic | 14:58 | |
| lucasagomes | mag009_, no the DHCP doesn't know anything about ramdisk + kernel | 14:58 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Use trueorfalse function instead of specific value https://review.openstack.org/297192 | 14:58 |
| lucasagomes | mag009_, the download of the kernel/ramdisk should be done by the network bootloader (elilo in ur case) | 14:59 |
| lucasagomes | the configuration file should specify where to it should get the ramdisk/kernel from | 14:59 |
| mag009_ | ok the elilo.conf file is missing | 15:00 |
| *** piet has joined #openstack-ironic | 15:00 | |
| mag009_ | so I need to set that up.. | 15:00 |
| lucasagomes | mag009_, just one more note on the DHCP config. If you have an hybrid envrioment (EFI + BIOS) you can create some conditionals to the DHCP configuration to look at the a dhcp option called "client-arch" to know if the node being booted is EFI or BIOS | 15:00 |
| lucasagomes | mag009_, right | 15:00 |
| mag009_ | it'd be nice to have that added to the doc ;) | 15:00 |
| lucasagomes | mag009_, also you have to use elilo? cause it's quite old | 15:00 |
| lucasagomes | honestly I think you should go for iPXE | 15:00 |
| mag009_ | i suppose I can use grub | 15:00 |
| lucasagomes | mag009_, right | 15:01 |
| mag009_ | or ipxe. | 15:01 |
| *** mbound has quit IRC | 15:01 | |
| mag009_ | but let me test with grub | 15:01 |
| lucasagomes | sure | 15:01 |
| mag009_ | I had an issue with grub | 15:01 |
| mag009_ | when I tried to deploy it was saying something about it couldn't get an ip | 15:01 |
| mag009_ | and the deploy failed right away | 15:02 |
| *** links has joined #openstack-ironic | 15:02 | |
| *** mgoddard_ has joined #openstack-ironic | 15:05 | |
| *** mgoddard has quit IRC | 15:09 | |
| *** lucasagomes_ has joined #openstack-ironic | 15:11 | |
| *** Shalmazar has joined #openstack-ironic | 15:11 | |
| *** adreznec_ has joined #openstack-ironic | 15:12 | |
| mag009_ | Failed to get IP address for any | | 15:13 |
| mag009_ | | | port on node c61f3fa9-a978-4f75-8f01-1dd418ebe7b2 | 15:13 |
| mag009_ | when I deploy with uefi | 15:13 |
| *** aginwala has quit IRC | 15:13 | |
| *** hparekh has quit IRC | 15:13 | |
| *** baoli has quit IRC | 15:13 | |
| *** lifeless has quit IRC | 15:13 | |
| *** david-lyle has quit IRC | 15:13 | |
| *** yuanying has quit IRC | 15:13 | |
| *** ppiela has quit IRC | 15:13 | |
| *** zzzeek has quit IRC | 15:13 | |
| *** lucasagomes has quit IRC | 15:13 | |
| *** adreznec has quit IRC | 15:13 | |
| *** Madasi has quit IRC | 15:13 | |
| *** Shalmazar is now known as Madasi | 15:13 | |
| *** zzzeek has joined #openstack-ironic | 15:13 | |
| *** saripurigopi has joined #openstack-ironic | 15:14 | |
| *** mgoddard_ has quit IRC | 15:15 | |
| *** mgoddard has joined #openstack-ironic | 15:16 | |
| lucasagomes_ | mag009_, :-/ not sure how others does it, with iPXE you can explicitly call the "dhcp" command to get an IP addr | 15:16 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Use get_admin_context() to create the context object https://review.openstack.org/246036 | 15:16 |
| lucasagomes_ | http://ipxe.org/cmd/dhcp | 15:16 |
| *** Guest27982 has quit IRC | 15:17 | |
| *** rcernin has quit IRC | 15:17 | |
| lucasagomes_ | mag009_, you know if at the boot time (before the image gets chainloaded) the node is getting an IP ? | 15:17 |
| *** david-lyle has joined #openstack-ironic | 15:17 | |
| mag009_ | well when I do a deploy .. it doesn't boot at all | 15:18 |
| *** links has quit IRC | 15:18 | |
| mag009_ | it just fail | 15:18 |
| mag009_ | doesn't even power on the machine | 15:18 |
| mag009_ | if I manually boot the machine it enter the grub | 15:18 |
| mag009_ | so yes I get an ip | 15:18 |
| *** adreznec_ is now known as adreznec | 15:19 | |
| mag009_ | i'll give a try with ipxe... lol | 15:21 |
| *** baoli has joined #openstack-ironic | 15:25 | |
| *** hparekh has joined #openstack-ironic | 15:29 | |
| *** lifeless has joined #openstack-ironic | 15:29 | |
| *** yuanying has joined #openstack-ironic | 15:29 | |
| *** ppiela has joined #openstack-ironic | 15:29 | |
| *** Guest27982 has joined #openstack-ironic | 15:30 | |
| *** rcernin has joined #openstack-ironic | 15:30 | |
| *** lifeless has quit IRC | 15:30 | |
| *** e0ne has quit IRC | 15:30 | |
| *** lifeless has joined #openstack-ironic | 15:30 | |
| *** e0ne has joined #openstack-ironic | 15:30 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Fix: Duplicated driver causes conductor to fail https://review.openstack.org/297194 | 15:33 |
| lucasagomes_ | mag009_, heh ok | 15:33 |
| jroll | could some cores get these patches in please? :) https://review.openstack.org/#/c/296428/ https://review.openstack.org/#/c/295591/ | 15:34 |
| * lucasagomes_ is on a call, will do after | 15:35 | |
| *** lucasagomes_ is now known as lucasagomes | 15:36 | |
| jroll | thanks | 15:36 |
| devananda | morning, all | 15:38 |
| sambetts | o/ devananda | 15:38 |
| mgould | morning devananda | 15:38 |
| *** absubram has joined #openstack-ironic | 15:39 | |
| jroll | hiya devananda | 15:40 |
| TheJulia | good morning devananda | 15:41 |
| mag009_ | i'll catch up later | 15:43 |
| *** garthb has joined #openstack-ironic | 15:44 | |
| *** mgoddard_ has joined #openstack-ironic | 15:45 | |
| NobodyCam | good morning devananda | 15:46 |
| *** mgoddard has quit IRC | 15:49 | |
| lucasagomes | devananda, TheJulia morning | 15:50 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Fix: Duplicated driver causes conductor to fail https://review.openstack.org/297194 | 15:51 |
| openstackgerrit | Merged openstack/ironic: Update reno for stable/mitaka https://review.openstack.org/296428 | 15:54 |
| *** mbound has joined #openstack-ironic | 15:55 | |
| *** ifarkas has quit IRC | 16:00 | |
| *** Haomeng has quit IRC | 16:05 | |
| *** ChrisAusten has quit IRC | 16:06 | |
| *** trown is now known as trown|lunch | 16:09 | |
| *** ifarkas has joined #openstack-ironic | 16:10 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Install apparmor b/c Docker.io has undeclared dep https://review.openstack.org/297177 | 16:13 |
| openstackgerrit | Merged openstack/ironic-python-agent: Update reno for stable/mitaka https://review.openstack.org/295591 | 16:14 |
| *** e0ne has quit IRC | 16:14 | |
| *** mgoddard_ has quit IRC | 16:14 | |
| *** mgoddard has joined #openstack-ironic | 16:14 | |
| *** mbound has quit IRC | 16:17 | |
| *** mgoddard_ has joined #openstack-ironic | 16:18 | |
| *** mgoddard has quit IRC | 16:21 | |
| *** Marga_ has joined #openstack-ironic | 16:21 | |
| *** Marga_ has quit IRC | 16:22 | |
| *** Marga_ has joined #openstack-ironic | 16:23 | |
| lucasagomes | is it only here or gerrit is really slow now? | 16:30 |
| jlvillal | lucasagomes, I can't connect very well :( | 16:30 |
| *** zzzeek has quit IRC | 16:30 | |
| lucasagomes | :-/ | 16:30 |
| sambetts | lucasagomes: its slow for me too | 16:30 |
| TheJulia | seems very slow to me | 16:30 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Clean up test node post data https://review.openstack.org/297248 | 16:32 |
| *** zzzeek has joined #openstack-ironic | 16:33 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: CoreOS image building: Add dnf support https://review.openstack.org/297064 | 16:34 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: CoreOS Image building: Ensure docker is started https://review.openstack.org/297079 | 16:34 |
| *** harshs has joined #openstack-ironic | 16:40 | |
| *** praneshp_ has joined #openstack-ironic | 16:40 | |
| JayF | is gerrit down/unhappy? | 16:42 |
| NobodyCam | I just got a error but went thru on second attempt | 16:44 |
| lucasagomes | JayF, yes :-/ | 16:45 |
| JayF | yeah, -infra confirms it, they were already looking into it | 16:46 |
| *** mbound has joined #openstack-ironic | 16:46 | |
| *** mbound has quit IRC | 16:46 | |
| *** achanda has joined #openstack-ironic | 16:47 | |
| *** mgoddard_ has quit IRC | 16:47 | |
| *** mgoddard has joined #openstack-ironic | 16:47 | |
| *** Nisha has joined #openstack-ironic | 16:48 | |
| sambetts | can't access it at all now :( | 16:48 |
| NobodyCam | ugh :( | 16:48 |
| *** suro-patz has joined #openstack-ironic | 16:53 | |
| sambetts | ugh ... this is making debugging my CI really hard.. | 16:54 |
| *** mbound has joined #openstack-ironic | 16:55 | |
| *** odyssey4me has quit IRC | 16:55 | |
| *** odyssey4me has joined #openstack-ironic | 16:55 | |
| *** e0ne has joined #openstack-ironic | 16:56 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector-specs: High Availability for Ironic Inspector https://review.openstack.org/253675 | 16:57 |
| jroll | TheJulia: hey, how's the bifrost release coming along? | 17:01 |
| *** openstackgerrit has quit IRC | 17:01 | |
| cinerama | jroll: we cut earlier this week | 17:02 |
| TheJulia | jroll: done | 17:02 |
| *** mbound has quit IRC | 17:02 | |
| *** e0ne_ has joined #openstack-ironic | 17:02 | |
| *** openstackgerrit has joined #openstack-ironic | 17:02 | |
| jroll | oh, awesome! thank you :) | 17:02 |
| * jroll sends newton is open email shortly | 17:02 | |
| Nisha | :) | 17:02 |
| TheJulia | :) | 17:02 |
| jroll | lucasagomes: if you want to watch that post job we fixed: https://jenkins01.openstack.org/job/ironic-python-agent-branch-tarball/30/consoleFull | 17:02 |
| devananda | \o/ | 17:02 |
| lucasagomes | jroll, o/ | 17:03 |
| lucasagomes | *fingers crossed* | 17:03 |
| JayF | I feel dumbish for not realizing we had to fix the full_trusty_build script too :( | 17:03 |
| *** e0ne has quit IRC | 17:03 | |
| jroll | JayF: heh, no worries | 17:04 |
| lucasagomes | 2016-03-24 17:04:12.037 | Finished: SUCCESS | 17:04 |
| jroll | JayF: I even noticed it was broken, filed a bug, but didn't realize why | 17:04 |
| jroll | oh wait, is that the wrong job? | 17:05 |
| jroll | it is, dang | 17:05 |
| jroll | ironic-python-agent-buildimage-coreos is still queued | 17:05 |
| lucasagomes | oh the post didn't started yet | 17:05 |
| lucasagomes | yeah | 17:05 |
| jroll | my bad | 17:05 |
| lucasagomes | it's been in the qeue for 51min :-) I bet things are very slow today | 17:05 |
| jroll | post queue seems to be slow often | 17:06 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Added node tags in ironic-api https://review.openstack.org/250478 | 17:06 |
| JayF | post queue is lowest priority jobs | 17:08 |
| *** mkovacik__ has quit IRC | 17:09 | |
| sambetts | jroll, lucasagomes: With a change like this https://review.openstack.org/#/c/256364/45 should we make it backwards compatible, because I'm seeing failures in my CI because we use the IRONIC_IPMIINFO_FILE in our CI and anything in that chain above that commit fails because it trys to look for a file that doesn't exist | 17:09 |
| jroll | sambetts: we need to either make it backwards compat, or give a proper heads up to folks before landing it | 17:10 |
| jroll | probably the former is best | 17:10 |
| jroll | (if possible) | 17:10 |
| lucasagomes | yeah, if it's possible to make it backward compat we should do it | 17:10 |
| * jroll brb | 17:11 | |
| *** saripurigopi has quit IRC | 17:13 | |
| *** ifarkas has quit IRC | 17:16 | |
| *** e0ne_ has quit IRC | 17:17 | |
| *** MattMan has quit IRC | 17:19 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Follow up patch of 843ce0a16160f2e2710ef0901028453cd9a0357c https://review.openstack.org/297266 | 17:20 |
| *** MattMan has joined #openstack-ironic | 17:20 | |
| *** jaypipes has quit IRC | 17:23 | |
| lucasagomes | I'm calling it a day here, have a great evening all | 17:26 |
| sambetts | o/ lucasagomes | 17:26 |
| NobodyCam | have a good night lucasagomes | 17:27 |
| lucasagomes | tomorrow is holiday here so I see you guys on tuesday (monday's also holiday) | 17:27 |
| NobodyCam | wow must be nice :) | 17:27 |
| lucasagomes | enjoy the [extended] weekend | 17:27 |
| NobodyCam | you too | 17:27 |
| sambetts | lucasagomes: thanks lucasagomes | 17:27 |
| *** lucasagomes is now known as lucas-afk | 17:28 | |
| NobodyCam | question: has anyone tested: localboot + BIOS boot mode with GPT disk ?? | 17:32 |
| *** trown|lunch is now known as trown | 17:32 | |
| sambetts | NobodyCam: no but I am interested in it because I've had failures deploying nodes with local disk >2TB | 17:33 |
| NobodyCam | sambetts: ya | 17:33 |
| TheJulia | I seem to remember one of my friends wrote a blog post about doing it... | 17:33 |
| *** degorenko is now known as _degorenko|afk | 17:35 | |
| *** saripurigopi has joined #openstack-ironic | 17:35 | |
| *** rcernin has quit IRC | 17:36 | |
| TheJulia | well, turns out written by someone else | 17:37 |
| TheJulia | http://www.anchor.com.au/blog/2012/10/booting-large-gpt-disks-without-efi/ | 17:38 |
| NobodyCam | thank you TheJulia :) | 17:39 |
| TheJulia | Looking at it, I'm fairly certian that is almost identical to how I set it up on a few of my machines | 17:39 |
| *** pcaruana has quit IRC | 17:40 | |
| *** spandhe has joined #openstack-ironic | 17:41 | |
| sambetts | NobodyCam, TheJulia: I guess this would be for a whole disk image right, so manually setup before ironic gets involved? Do you know if IPA do it for a partition image? | 17:41 |
| Nisha | sambetts, it should be done by ironic-lib for partition images | 17:42 |
| *** ndipanov has quit IRC | 17:42 | |
| Nisha | sambetts, we need to check that | 17:42 |
| TheJulia | sambetts: I don't see why not since I could manually make an image that represents that composition, and then attempt to deploy it | 17:43 |
| Nisha | TheJulia, so were you able to deploy successfully the above combination? | 17:44 |
| TheJulia | Nisha: with Ironic, I've never tried. One of my test machines is GPT+bios boot and I've set it up before. It would be fairly trivial to make a raw image representing that | 17:45 |
| Nisha | TheJulia, ok. | 17:45 |
| Nisha | does disk_label applies to whole disk images also? | 17:46 |
| TheJulia | I don't think so | 17:46 |
| TheJulia | I don't have that code path, at least as it pertains to ironic-lib in my head at the moment | 17:46 |
| sambetts | Last time I tried to deploy an image onto a node with a disk great than 2TB it blew up after the first boot because cloud init expanded the partition to the full size of the disk | 17:47 |
| TheJulia | sambetts: yeah, the end of the first partition, or at least the partition with the kernel/ramdisk for booting has to be in the first 31 bits of addressable space on bios booting | 17:48 |
| TheJulia | so if the filesystem is grown, it can't grok it, and kaboom | 17:48 |
| TheJulia | auto-grow is evil | 17:48 |
| sambetts | if IPA could detect the real disk size and realise it was >2TB then install a gpt partition it would work right? | 17:50 |
| jroll | sambetts: is this agent or iscsi drivers? | 17:50 |
| TheJulia | sambetts: as in a partition image write-out? | 17:51 |
| sambetts | iscsi with a partition image | 17:51 |
| jroll | I feel like it does that, but I could be wrong | 17:51 |
| jroll | there is a way to hint it to do gpt | 17:52 |
| *** spandhe has quit IRC | 17:52 | |
| sambetts | jroll: adding concurrency=1 to my CI seems to have prevented those weird errors with the n-sch | 17:52 |
| TheJulia | i seem to remember there is a hint as well | 17:52 |
| jroll | nice | 17:52 |
| TheJulia | but its been a while since I've dug through any of that | 17:52 |
| jroll | Nisha: where does the disk_label thing go, capabilities? instance_info? | 17:53 |
| Nisha | instance_info | 17:53 |
| jroll | sambetts: instance_info/disk_label gpt | 17:53 |
| jroll | ty Nisha | 17:53 |
| sambetts | I wonder if we should automate it for when we detect disk size >1/2TB | 17:54 |
| *** ChubYann has joined #openstack-ironic | 17:54 | |
| jroll | I feel like there was a bug/rfe for that, I thought we did it but idk | 17:55 |
| *** kromanenko has quit IRC | 17:55 | |
| jroll | sambetts: https://review.openstack.org/#/c/225556/ | 17:55 |
| jroll | so yeah, not done | 17:56 |
| TheJulia | sambetts: I think that would be a good idea, there are some caveats to that though, we mgiht have to verify the partition image in a raw state, depending on the format and contents, that nothing is beyond the barrier | 17:56 |
| Nisha | jroll, we tested above combination but it didnt work...so was just asking if its always expect to work | 17:56 |
| jroll | Nisha: which combination? | 17:57 |
| jroll | what didn't work? | 17:57 |
| Nisha | localboot + bios + gpt | 17:57 |
| Nisha | disk label* | 17:57 |
| jroll | I don't know enough to know if that should work, unfortunately | 17:58 |
| *** spandhe has joined #openstack-ironic | 17:58 | |
| sambetts | jroll: ah I see | 17:58 |
| *** saripurigopi has quit IRC | 17:59 | |
| Nisha | jroll, hmm | 17:59 |
| TheJulia | I would expect it wouldn't work unless we had code to explicitly make it work, at least for partition images | 17:59 |
| Nisha | TheJulia, we dont have anything special in ironic_lib for it...as far as i see the code | 17:59 |
| TheJulia | Nisha: that is kind of what I remember right now | 18:00 |
| jroll | I assume it's just the bootloader install that's broken, right? | 18:00 |
| *** saripurigopi has joined #openstack-ironic | 18:01 | |
| Nisha | jroll, no i guess its the partition creation which i see is broken...i am not sure though unless its investigated | 18:01 |
| jroll | only on gpt? | 18:01 |
| Nisha | in Bios mode GPT partition creation | 18:01 |
| Nisha | I am not really sure | 18:02 |
| jroll | hm | 18:02 |
| * sambetts is calling it a day | 18:03 | |
| NobodyCam | night sambetts | 18:03 |
| jroll | nn \o | 18:04 |
| sambetts | as lucas-afk said its a holiday for us tomorrow and monday so I'll see you all next week :) | 18:04 |
| sambetts | o/ NobodyCam jroll | 18:05 |
| *** sambetts is now known as sambetts|afk | 18:05 | |
| * NobodyCam want a holiday too :) | 18:05 | |
| Nisha | NobodyCam, jroll, TheJulia this will need an investigation which i will see on Monday...we also have holiday tomorrow (not on Monday though) :) | 18:08 |
| * Nisha is calling it a day. | 18:08 | |
| NobodyCam | have good (long) weekend Nisha | 18:09 |
| Nisha | NobodyCam, :) | 18:09 |
| TheJulia | have a wonderful weekend | 18:09 |
| *** piet has quit IRC | 18:09 | |
| JayF | https://review.openstack.org/#/c/293373/ has 1x+2 and 5x+1 if someone wants to land it | 18:10 |
| JayF | I'm tired of seeing all the same patches in no-negative-feedback on my dash :P | 18:10 |
| *** jaybeale has quit IRC | 18:10 | |
| Nisha | TheJulia, Thanks... just before i go ...ther is one more topic which i have faced and i am not sure wher eit should be fixed but it affects us though | 18:10 |
| * NobodyCam looks | 18:11 | |
| TheJulia | Nisha: oh? | 18:12 |
| Nisha | jroll, TheJulia NobodyCam JayF during my testing with DIB IPA image, when i used fedora based deploy ramdisk, it failed to do localboot | 18:12 |
| Nisha | but the same works with ubunto ramdisk | 18:12 |
| Nisha | the reason is it always fails to install boot loader | 18:12 |
| TheJulia | fun :\ | 18:13 |
| Nisha | when we use fedora based ramdisk | 18:13 |
| JayF | I'd file a bug with all the details then | 18:13 |
| JayF | including any logs from the failed install | 18:13 |
| TheJulia | what JayF said | 18:13 |
| JayF | without logs there is so extermination :) | 18:13 |
| *** athomas has quit IRC | 18:13 | |
| Nisha | Jayf, the point is still incomplete here | 18:13 |
| Nisha | when we use ubunto ramdisk and fedora partition image, then the image falls to the ramdisk prompt(dracut) | 18:14 |
| TheJulia | Nisha: I've found individual dib revisions can get very focused on a single use case or OS, which ends up breaking others, and that we've just had to find the issues and submit fixes | 18:14 |
| Nisha | so to me it doesnt seem to be an issue in DIB/IPA/Ironic | 18:14 |
| Nisha | it seemed to me sisue with fedora kernel | 18:14 |
| Nisha | TheJulia, no this was faced by rameshg87 also earlier .... | 18:15 |
| Nisha | last year i guess (around Auguts / Sept) | 18:15 |
| *** piet has joined #openstack-ironic | 18:16 | |
| TheJulia | Nisha: they sound like two distinctly separate issues, file or see if the bugs are out there and raise awareness | 18:16 |
| TheJulia | That way we have full context and can begin to identify possible causes and remedies | 18:17 |
| Nisha | TheJulia, i dont have the bugzilla access, but i got this link from someone on this | 18:18 |
| Nisha | https://bugzilla.redhat.com/show_bug.cgi?id=1188876 | 18:18 |
| openstack | Nisha: Error: Error getting bugzilla.redhat.com bug #1188876: NotPermitted | 18:18 |
| Nisha | i am not sure what is there in this link as i cant see the details | 18:18 |
| TheJulia | trown: any chance you can take a look at that bug Nisha is talking about? | 18:18 |
| * Nisha is now logging off...just wanted to see if anyone else knows the issue... | 18:19 | |
| trown | hmm RHEL6 bug, yuck | 18:20 |
| TheJulia | pertinant to kind of what we're talking about? | 18:21 |
| trown | TheJulia: that is a RHEL6 bug about UEFI support, not sure the context | 18:21 |
| NobodyCam | JayF: jroll: +a'd | 18:21 |
| trown | reading up the context seems to be DIB, and that bug is not DIB related... | 18:22 |
| TheJulia | trown: thank you | 18:22 |
| trown | np | 18:22 |
| jlvillal | Does anyone remember who Alex Weeks works for? I was looking at this and saw he was assigned as the owner: https://bugs.launchpad.net/ironic/+bug/1526219 | 18:28 |
| openstack | Launchpad bug 1526219 in Ironic "[RFE] Add pluggable metrics backend for Ironic and IPA" [Wishlist,Confirmed] - Assigned to Alex Weeks (alex-weeks) | 18:28 |
| zer0c00l | Nisha: ping. If you have a bug, i would be happy to look at it. | 18:28 |
| jlvillal | I can't find any info in Stackalytics on him. | 18:28 |
| jroll | jlvillal: he used to work with us | 18:29 |
| jroll | now a not-openstack company | 18:29 |
| jlvillal | jroll: Do you think then he won't be working on that bug anymore? | 18:29 |
| NobodyCam | simple follow up patch 4 x +1's https://review.openstack.org/#/c/296621 | 18:29 |
| jroll | jlvillal: correct - alineb was actually talking about picking that up this morning | 18:30 |
| jlvillal | jroll: We were looking at bugs that maybe our team could look into here at Intel. | 18:30 |
| jroll | but you can unassign | 18:30 |
| jroll | ah | 18:30 |
| jroll | jlvillal: osic or no? | 18:30 |
| jlvillal | jroll: Non-osic | 18:30 |
| jroll | ah ok | 18:30 |
| jlvillal | jroll: Same boss :) | 18:30 |
| jroll | just curious :) | 18:30 |
| jlvillal | jroll: I'll assume then alineb will be picking it up then. | 18:31 |
| jroll | so yeah alineb was going to pick that up, she's out til tuesday but you could chat with her about it | 18:31 |
| jroll | (she's in the UK time zone) | 18:31 |
| jlvillal | jroll: Thank! | 18:31 |
| *** daemontool has quit IRC | 18:32 | |
| *** Sukhdev has joined #openstack-ironic | 18:33 | |
| JayF | jlvillal: I hope I'll work with aline on that too | 18:33 |
| JayF | jlvillal: so metrics might already be dogpiled :) | 18:33 |
| JayF | jlvillal: have any desire to help on the nova side for rescue? heh | 18:33 |
| jlvillal | JayF: Link? Trying to find some things for our new developers to contribute to. | 18:33 |
| JayF | this is probably not suitable for a noob :) | 18:34 |
| jlvillal | To help them learn and improve Ironic at the same time. | 18:34 |
| JayF | needs bp in nova and the spec in ironic isn't quite done yet | 18:34 |
| jlvillal | Yeah... | 18:34 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add user and project domains to ironic context https://review.openstack.org/295371 | 18:34 |
| *** Sukhdev has quit IRC | 18:39 | |
| * mgould -> home; good night, everyone, and have a great weekend! | 18:41 | |
| JayF | Both of these are updated and I would like reviews if anyone has the time; https://review.openstack.org/#/c/259135/ (support reboot_requested in agent clean steps) and https://review.openstack.org/#/c/259119/ (new config option to leave nodes on post-deploy-failure) | 18:41 |
| NobodyCam | night mgould | 18:41 |
| mgould | NobodyCam, good night! | 18:44 |
| *** mgould has quit IRC | 18:44 | |
| *** Guest27982 has quit IRC | 18:46 | |
| *** Marga_ has quit IRC | 18:46 | |
| *** piet has quit IRC | 18:47 | |
| *** mtanino has joined #openstack-ironic | 18:48 | |
| *** Nisha has quit IRC | 18:52 | |
| *** achanda has quit IRC | 18:53 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Migrate to using keystoneauth Sessions https://review.openstack.org/236982 | 18:55 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Update devstack plugin with new auth options https://review.openstack.org/297312 | 18:55 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector: Set config options for keystoneauth. https://review.openstack.org/294703 | 19:02 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector: Use keystoneauth for Ironic and Swift clients https://review.openstack.org/286510 | 19:02 |
| *** piet has joined #openstack-ironic | 19:02 | |
| *** spandhe has quit IRC | 19:04 | |
| *** daemontool has joined #openstack-ironic | 19:07 | |
| *** jaybeale has joined #openstack-ironic | 19:09 | |
| *** Marga_ has joined #openstack-ironic | 19:12 | |
| *** spandhe has joined #openstack-ironic | 19:16 | |
| *** ChrisAusten has joined #openstack-ironic | 19:17 | |
| *** baoli has quit IRC | 19:18 | |
| *** achanda has joined #openstack-ironic | 19:24 | |
| *** daemontool has quit IRC | 19:25 | |
| *** baoli has joined #openstack-ironic | 19:29 | |
| *** mgoddard has quit IRC | 19:29 | |
| *** davidlenwell has quit IRC | 19:35 | |
| *** piet has quit IRC | 19:37 | |
| *** davidlenwell has joined #openstack-ironic | 19:38 | |
| *** wuhao has quit IRC | 19:38 | |
| *** e0ne has joined #openstack-ironic | 19:41 | |
| *** baoli has quit IRC | 19:41 | |
| *** baoli has joined #openstack-ironic | 19:43 | |
| *** Marga_ has quit IRC | 19:46 | |
| *** Marga_ has joined #openstack-ironic | 19:46 | |
| *** baoli has quit IRC | 19:48 | |
| *** achanda has quit IRC | 19:55 | |
| jroll | NobodyCam: https://review.openstack.org/#/c/297122/ | 20:01 |
| jroll | mind landing that? per ML we need to release a new version with that asap | 20:01 |
| jroll | (because it's a Y bump in x.y.z and we don't want to end up releasing a Y bump in master first | 20:02 |
| NobodyCam | looking now :) | 20:02 |
| jroll | thanks | 20:02 |
| *** davidlenwell has quit IRC | 20:02 | |
| NobodyCam | LGTM +a | 20:03 |
| jroll | ty | 20:03 |
| NobodyCam | oh speaking of. there are a couple stable/kilo patches out there too | 20:03 |
| jroll | yep, been working with mriedem on that | 20:04 |
| jroll | the CI was completely busted | 20:04 |
| NobodyCam | :) | 20:04 |
| jroll | those patches can come in once this lands https://review.openstack.org/#/c/271391/6 | 20:04 |
| jroll | which depends on https://review.openstack.org/#/c/296555/ | 20:05 |
| jroll | both are in the pipe | 20:05 |
| jroll | NobodyCam: also, I just realized that first one is a stable branch and afaik you aren't on the stable core list so wth? | 20:05 |
| jroll | oh ironic-lib doesn't have stable acls | 20:06 |
| jroll | hrm | 20:06 |
| NobodyCam | :p | 20:06 |
| openstackgerrit | Merged openstack/ironic: Fix: Duplicated driver causes conductor to fail https://review.openstack.org/297194 | 20:06 |
| * jroll fixes | 20:06 | |
| jroll | thanks for the help though! | 20:06 |
| NobodyCam | ofc :) | 20:06 |
| * NobodyCam goes back to his pizza :) | 20:07 | |
| *** baoli has joined #openstack-ironic | 20:10 | |
| *** davidlenwell has joined #openstack-ironic | 20:12 | |
| *** e0ne has quit IRC | 20:21 | |
| *** Sukhdev has joined #openstack-ironic | 20:24 | |
| *** piet has joined #openstack-ironic | 20:25 | |
| *** achanda has joined #openstack-ironic | 20:28 | |
| *** achanda has quit IRC | 20:31 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [console] https://review.openstack.org/295946 | 20:31 |
| NobodyCam | JayF: just look'n at 259119 is there a nova patch? | 20:32 |
| JayF | NobodyCam: no; lucas-afk suggested he might write it | 20:32 |
| JayF | NobodyCam: but it's helpful anyway for various reasons, including standalone mode | 20:32 |
| openstackgerrit | Merged openstack/ironic: [Devstack]Add ability to enable shellinabox SSL certificate https://review.openstack.org/289671 | 20:40 |
| *** trown is now known as trown|outtypewww | 20:41 | |
| *** Goneri has quit IRC | 20:44 | |
| *** moshele has joined #openstack-ironic | 20:45 | |
| NobodyCam | JayF: +2 ... holding +a to others a little time to re-review.. if it hasn't landed by morning we'll push the button | 20:47 |
| NobodyCam | s/to others/to allow others/ | 20:47 |
| *** openstackgerrit has quit IRC | 20:48 | |
| *** openstackgerrit has joined #openstack-ironic | 20:49 | |
| *** Marga_ has quit IRC | 20:53 | |
| *** Sukhdev has quit IRC | 20:53 | |
| *** Marga_ has joined #openstack-ironic | 20:53 | |
| *** mdorman has joined #openstack-ironic | 20:55 | |
| *** moshele has quit IRC | 20:56 | |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Provide attach remote media function https://review.openstack.org/296047 | 21:03 |
| *** mrda has quit IRC | 21:07 | |
| *** mrda has joined #openstack-ironic | 21:07 | |
| *** pas-ha has quit IRC | 21:08 | |
| *** pas-ha has joined #openstack-ironic | 21:09 | |
| *** rama_y has joined #openstack-ironic | 21:10 | |
| *** baoli has quit IRC | 21:23 | |
| jlvillal | NobodyCam: JayF: I did leave a couple comments on https://review.openstack.org/#/c/259119/ | 21:24 |
| jlvillal | But I'm fine with it being merged :) | 21:24 |
| NobodyCam | :) that's why I gave some time :) | 21:24 |
| *** harshs has quit IRC | 21:24 | |
| *** suro-patz has quit IRC | 21:25 | |
| *** suro-patz has joined #openstack-ironic | 21:25 | |
| *** harshs has joined #openstack-ironic | 21:26 | |
| *** suro-patz has quit IRC | 21:30 | |
| *** spandhe has quit IRC | 21:32 | |
| *** suro-patz has joined #openstack-ironic | 21:32 | |
| *** spandhe has joined #openstack-ironic | 21:34 | |
| *** yonglihe has quit IRC | 21:34 | |
| *** patrickeast has quit IRC | 21:34 | |
| *** evgenyl has quit IRC | 21:34 | |
| *** harlowja_at_home has quit IRC | 21:34 | |
| *** yonglihe has joined #openstack-ironic | 21:35 | |
| *** devananda has quit IRC | 21:35 | |
| *** evgenyl has joined #openstack-ironic | 21:35 | |
| *** patrickeast has joined #openstack-ironic | 21:37 | |
| jlvillal | NobodyCam: https://review.openstack.org/#/c/295293/7/devstack/lib/ironic What do you think of my comment? | 21:39 |
| jlvillal | Bike-shedding? :) | 21:39 |
| NobodyCam | will look in a minute | 21:40 |
| *** openstack has joined #openstack-ironic | 21:50 | |
| NobodyCam | are going to do the follow up? | 21:51 |
| NobodyCam | I can | 21:51 |
| jlvillal | NobodyCam: If you like, feel free :) | 21:53 |
| NobodyCam | :) | 21:53 |
| jlvillal | NobodyCam: I left another comment about 'sort' and '-o'. The specifically call out it being acceptable to use if for in-place sort. | 21:56 |
| jlvillal | NobodyCam: One extra line seems simpler to me than all the if/else/fi stuff :) | 21:56 |
| *** ametts has quit IRC | 21:57 | |
| *** spandhe has quit IRC | 21:58 | |
| *** spandhe has joined #openstack-ironic | 22:00 | |
| *** harshs has joined #openstack-ironic | 22:00 | |
| *** mdorman has joined #openstack-ironic | 22:00 | |
| *** hparekh has joined #openstack-ironic | 22:00 | |
| *** yuanying has joined #openstack-ironic | 22:00 | |
| *** ppiela has joined #openstack-ironic | 22:00 | |
| *** rcrit_ has quit IRC | 22:01 | |
| *** penick has joined #openstack-ironic | 22:01 | |
| *** rcrit_ has joined #openstack-ironic | 22:02 | |
| *** baoli has joined #openstack-ironic | 22:09 | |
| *** baoli has quit IRC | 22:09 | |
| *** baoli has joined #openstack-ironic | 22:10 | |
| *** cdearborn has quit IRC | 22:15 | |
| NobodyCam | humm I did something wrong: ! [remote rejected] HEAD -> refs/publish/master/patch_295293 (squash commits first) | 22:15 |
| *** jaybeale has quit IRC | 22:17 | |
| openstackgerrit | Merged openstack/ironic: Config to stop powering off nodes on failure https://review.openstack.org/259119 | 22:20 |
| jlvillal | NobodyCam: Did you figure it out? | 22:26 |
| NobodyCam | oh sorry got distracted by email | 22:27 |
| jlvillal | :) | 22:27 |
| openstackgerrit | Chris Krelle proposed openstack/ironic: follow up to Idc1603d7f6d849d9c1a745b6d50d75ceb414d5c9 https://review.openstack.org/297391 | 22:30 |
| NobodyCam | :-p | 22:30 |
| openstackgerrit | Merged openstack/ironic: Update authorized_keys with new key only https://review.openstack.org/295293 | 22:31 |
| NobodyCam | jlvillal: somehting like that ?? | 22:32 |
| jlvillal | NobodyCam: Yep :) | 22:33 |
| jlvillal | NobodyCam: I'm slightly confused about the '.pub' change thing | 22:33 |
| NobodyCam | that was my comment on the review. we test for file xyz but use file xyz.pub | 22:35 |
| jlvillal | NobodyCam: Okay. But isn't that just being used to create the directory? | 22:35 |
| NobodyCam | if the file was placed by a config eng like pupet or such then there may infact be no .pub file | 22:35 |
| NobodyCam | line 831 | 22:36 |
| jlvillal | NobodyCam: Okay. So if the pub file doesn't exist, we should generate a new keypair? | 22:36 |
| NobodyCam | we could error ? | 22:36 |
| NobodyCam | but I figured just make a new pair | 22:36 |
| jlvillal | NobodyCam: Works for me. Though... | 22:37 |
| * jlvillal wonders if should check to make sure both exist... | 22:37 | |
| NobodyCam | I'm happy to adjust | 22:38 |
| jlvillal | if [[ ! -e $IRONIC_KEY_FILE -o ! -e ${IRONIC_KEY_FILE}.pub ]] | 22:38 |
| jlvillal | something like that. | 22:38 |
| NobodyCam | ya | 22:38 |
| * jlvillal likes the var wrapped in {} since he wasn't sure that the '.' was an invalid character in an environment variable | 22:38 | |
| jlvillal | NobodyCam: Don't trust my IRC bash commands though ;) | 22:39 |
| NobodyCam | lol | 22:39 |
| jlvillal | I have NOT tested them! | 22:39 |
| NobodyCam | let give it time to generate some comments :) | 22:39 |
| NobodyCam | *lets | 22:40 |
| jlvillal | I did not know that 'jam' was your middle name NobodyCam | 22:41 |
| NobodyCam | lol james chopped to jam by email folks :-p | 22:42 |
| jlvillal | Heh, that is funny :) | 22:42 |
| NobodyCam | hehehe | 22:42 |
| jlvillal | NobodyCam: I feel sorry for someone with the middle name Assa... | 22:43 |
| * jlvillal thinks of other ways truncating to three letters could be wrong... | 22:44 | |
| NobodyCam | lol | 22:44 |
| NobodyCam | :p | 22:44 |
| *** penick has quit IRC | 22:44 | |
| openstackgerrit | Merged openstack/ironic: Update ironic_ssh_check method https://review.openstack.org/293373 | 22:48 |
| jlvillal | NobodyCam: Left some comments. Feel free to grumble at me under your breath ;) | 22:49 |
| NobodyCam | lol na... but prob woth fix until tomorrow :p | 22:50 |
| JayF | jlvillal: fwiw i tested it | 22:51 |
| NobodyCam | oh I did use the change-id for the depends on didn't I | 22:51 |
| JayF | jlvillal: $BLAH.pub returns properly | 22:51 |
| jlvillal | NobodyCam: No hurry. Thanks for following up on it! You rock!!! | 22:51 |
| NobodyCam | (blush) | 22:51 |
| NobodyCam | JayF: Thank you :) | 22:51 |
| jlvillal | JayF: Thanks :) I had to test it too to be sure... | 22:51 |
| * jlvillal plays with IRC coloring... | 22:52 | |
| * NobodyCam lol ... because he uses a near default irssi config | 22:52 | |
| * jroll lol ... because he uses weechat :P | 22:53 | |
| NobodyCam | :) | 22:54 |
| *** chlong has joined #openstack-ironic | 22:55 | |
| *** vishwanathj has quit IRC | 22:55 | |
| *** rbudden has quit IRC | 22:59 | |
| *** cfarquhar has quit IRC | 23:00 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Unify testing scripts https://review.openstack.org/297424 | 23:00 |
| *** cfarquhar has joined #openstack-ironic | 23:00 | |
| *** cfarquhar has quit IRC | 23:00 | |
| *** cfarquhar has joined #openstack-ironic | 23:00 | |
| *** jcoufal has quit IRC | 23:06 | |
| *** davidlenwell has quit IRC | 23:07 | |
| *** aNupoisc has joined #openstack-ironic | 23:08 | |
| jroll | any specs cores want to approve my RFEs? :) https://bugs.launchpad.net/ironic/+bug/1561756 https://bugs.launchpad.net/ironic/+bug/1561760 https://bugs.launchpad.net/ironic/+bug/1561773 | 23:08 |
| openstack | Launchpad bug 1561756 in Ironic "[RFE] Use stable branch of agent when testing stable branches" [Wishlist,Confirmed] | 23:08 |
| openstack | Launchpad bug 1561760 in Ironic "[RFE] CI job to run full tempest suite against ironic" [Wishlist,Confirmed] | 23:08 |
| openstack | Launchpad bug 1561773 in Ironic "[RFE] Run grenade tests against Ironic in CI" [Wishlist,Confirmed] | 23:08 |
| JayF | jroll: I think they need specs. All of them. | 23:12 |
| JayF | lol | 23:12 |
| jroll | ಠ_ಠ | 23:12 |
| JayF | jroll: how do I do that? Haven't approved any specs since rfe stuff started | 23:13 |
| jroll | approval process btw is switch the tags from rfe to rfe-approved | 23:13 |
| NobodyCam | I was just about to head out for dinner.. is tomorrow ok ? | 23:13 |
| jroll | we can also wait to run through those in the meeting | 23:13 |
| jroll | yeah I don't care, I still might start working on them | 23:13 |
| JayF | https://bugs.launchpad.net/ironic/+bug/1561756 approved? | 23:13 |
| openstack | Launchpad bug 1561756 in Ironic "[RFE] Use stable branch of agent when testing stable branches" [Wishlist,Confirmed] | 23:13 |
| jroll | :) | 23:13 |
| JayF | NobodyCam: I'm a specs core; I got this | 23:14 |
| jroll | that looks correct, yes | 23:14 |
| JayF | NobodyCam: I can approve WHILE I eat the latest lunch ever, you go single task your dinner though :P | 23:14 |
| jroll | lol | 23:14 |
| NobodyCam | w00t you ROCK JayF | 23:14 |
| NobodyCam | lol | 23:14 |
| *** davidlenwell has joined #openstack-ironic | 23:14 | |
| JayF | jroll: https://bugs.launchpad.net/ironic/+bug/1561760 is... interesting | 23:14 |
| openstack | Launchpad bug 1561760 in Ironic "[RFE] CI job to run full tempest suite against ironic" [Wishlist,Confirmed] | 23:14 |
| jroll | JayF: nova folks have been asking for that for years | 23:15 |
| JayF | jroll: How is that diff than what we're doing today? | 23:15 |
| jroll | we don't run *any* of nova's standard tests | 23:15 |
| jroll | only our own | 23:15 |
| jroll | which exercise much more functionality / edge cases | 23:15 |
| JayF | Why haven't we run all of them historically? | 23:15 |
| jroll | because nobody has put in the time to make it work | 23:15 |
| JayF | I'm just highly suspicious the "run them quickly" requirement is unpossible | 23:15 |
| jroll | primarily runtime/performance problems | 23:15 |
| jroll | right, it might be impossible today | 23:16 |
| JayF | Which is my primary pause in approving it | 23:16 |
| jroll | but we should work towards it | 23:16 |
| JayF | sure, I guess so | 23:16 |
| JayF | I presume the grenade rfe is of the same ilk? | 23:16 |
| JayF | just s/tempest/grenade/ | 23:16 |
| jroll | but I mean, we can chat about it in meeting | 23:16 |
| jroll | yeah | 23:16 |
| JayF | I just wanna make sure *I* understand if I'm doing it | 23:16 |
| jroll | if you're hesitant let's bring it to the meeting monday | 23:16 |
| JayF | doesn't seem controversial but I'm not a rubber stamp | 23:16 |
| JayF | no I mean it makes sense to have a bug to point work against to get towards that goal | 23:17 |
| JayF | I don't think it's controversial or anything; just wanted to make sure I understood it | 23:17 |
| jroll | cool | 23:17 |
| JayF | jroll: ptl or not you'll never bully me into doing something I don't wanna do, lol | 23:17 |
| jroll | that's what you think :) | 23:17 |
| JayF | approvinated | 23:17 |
| jroll | ty sir | 23:17 |
| devananda | this seems surprising to me, but does our nova driver not support an ironic service that requires SSL ? | 23:25 |
| jroll | it does, afaik | 23:27 |
| jroll | what are you seeing? | 23:27 |
| *** suro-patz has quit IRC | 23:28 | |
| devananda | I know the python client does, but it doens't look like the nova driver is able to pass --os-cacert to the client | 23:28 |
| devananda | there's nothing in client_wrapper.py that sets it, and no CONF option to specify it | 23:30 |
| jroll | oh, does it support an ironic service that requires SSL with certs signed by a non-standard CA? :) | 23:30 |
| jroll | I have no clue. | 23:30 |
| devananda | hah | 23:30 |
| devananda | right. it does not | 23:30 |
| jroll | that's fun! | 23:30 |
| devananda | whereas the vmware driver does | 23:30 |
| devananda | and, ofc, so do all the other openstack services which Nova talks to | 23:30 |
| jroll | yeah, I see no reason not to | 23:30 |
| devananda | because self signed certs are a thing | 23:31 |
| jroll | probably just nobody has decided to do it | 23:31 |
| devananda | cool. /me files bug | 23:31 |
| devananda | patch will be forthcoming, once i fix this | 23:31 |
| * jroll points devananda at letsencrypt | 23:31 | |
| devananda | whusdat? | 23:31 |
| jroll | O_o | 23:31 |
| jroll | the best thing ever | 23:31 |
| jroll | https://letsencrypt.org/ | 23:31 |
| devananda | oh oh | 23:31 |
| devananda | yea | 23:31 |
| jroll | it's easier than self-signed certs | 23:31 |
| jroll | and actually validates :) | 23:31 |
| devananda | nice | 23:32 |
| devananda | I should start using that for all my things | 23:32 |
| jroll | indeed | 23:32 |
| jroll | cron job to renew | 23:32 |
| jroll | easy peasy | 23:32 |
| JayF | I mean, we should still fix the support gap | 23:36 |
| JayF | btw devananda ty for the review and approval earlier ;) I have less universally interesting patches if you wanna keep helping me out :P | 23:36 |
| jroll | yeah, I trust deva is fixing it | 23:37 |
| devananda | JayF: heh. sure. maybe? | 23:38 |
| devananda | I really want to fix this SSl thing right now, though | 23:38 |
| jlvillal | jroll: I'm still using startssl instead of letsencrypt | 23:38 |
| jlvillal | :) | 23:38 |
| jlvillal | Both are free. | 23:39 |
| jroll | jlvillal: I have two reasons | 23:39 |
| *** ChrisAusten has quit IRC | 23:39 | |
| JayF | devananda: https://review.openstack.org/#/c/259135/ is what I was talking about; but it's not urgent | 23:39 |
| jroll | 1) letsencrypt is stupid simple to automate the signing and renewal of certs, I have it all ansible'd out | 23:39 |
| JayF | you might see more random little patches from me as I'm trying to get rid of any downstream patches we have, so we can run pure upstream ironic | 23:40 |
| JayF | and there are a small handful of little things like that | 23:40 |
| jroll | 2) startssl has a history of forcing customers to pay to revoke certificates when things go wrong | 23:40 |
| jlvillal | Ah. | 23:40 |
| jroll | jlvillal: https://raim.codingfarm.de/blog/2014/04/12/distrusting-startssl/ | 23:40 |
| jlvillal | Though on 1). I keep going, I don't want to run that program.... :) | 23:40 |
| JayF | Why wouldn't you want to? It's open source, and sponsored by the eff now. | 23:41 |
| jlvillal | But I should attempt it again. | 23:41 |
| *** harshs has quit IRC | 23:41 | |
| jroll | you'd rather use some crappy website? :) | 23:41 |
| * jroll recommends this client: https://github.com/kuba/simp_le | 23:41 | |
| jlvillal | One problem is I haven't figured out how to do it for internal websites. | 23:41 |
| jlvillal | nas.int.mydomain.com | 23:41 |
| jlvillal | Which is not accessible from the internet but only inside my house. | 23:42 |
| jroll | right, that case is problematic | 23:42 |
| * jlvillal is crazy and actually creates SSL certs for internal systems in his house. | 23:42 | |
| jroll | I believe there is a way, though | 23:42 |
| jroll | as you should :) | 23:42 |
| *** piet has quit IRC | 23:42 | |
| jroll | NobodyCam: stable/kilo CI should be happy now, I've rechecked all the things :) | 23:42 |
| JayF | whoa, nice job jroll | 23:43 |
| jlvillal | That was when I tried to use letsencrypt and sort of went hrmmm. And went and created one on startssl. | 23:43 |
| jlvillal | But for my znc host on the internet I will try letsencrypt when the current one expires. | 23:43 |
| jroll | JayF: thank mriedem, he did most of the work | 23:43 |
| JayF | whoa, nice job mriedem | 23:44 |
| jlvillal | jroll: I like the idea of full tempest CI working. Well to be honest if 'smoke' ran that would be a good thing. | 23:44 |
| jroll | jlvillal: smoke is a subset of full, so :) | 23:44 |
| jlvillal | Yep :) | 23:45 |
| *** spandhe has quit IRC | 23:48 | |
| JayF | jroll: https://review.openstack.org/#/c/296403 stable backports don't need multiple +2s, right? I think you can land that | 23:49 |
| JayF | jroll: and tests finally passed on it :) | 23:49 |
| *** suro-patz has joined #openstack-ironic | 23:49 | |
| *** praneshp_ has quit IRC | 23:50 | |
| JayF | another super easy one to land: https://review.openstack.org/#/c/296621/ 1x+2 (jim) and 5x+1 | 23:50 |
| jroll | JayF: they do, unless another stable maintainer proposed it | 23:50 |
| JayF | ah | 23:50 |
| JayF | I just know I don't have a +2 there and don't know who else does | 23:50 |
| jroll | so that's devananda or dtantsur or anyone on the general global stable team | 23:50 |
| JayF | aha | 23:51 |
| JayF | another low hanging review: https://review.openstack.org/#/c/297064 already has my core review in IPA, super simple change | 23:51 |
| * JayF trying to get things merged | 23:51 | |
| jroll | JayF: you can see the acls by clicking the gear next to the project name, then access at the top | 23:51 |
| JayF | nice, will do that next time | 23:51 |
| jroll | and then clicking into the group | 23:51 |
| jroll | so e.g. you end up at https://review.openstack.org/#/admin/groups/950,members | 23:51 |
| jlvillal | JayF: done on https://review.openstack.org/#/c/296621/ | 23:52 |
| JayF | another easy one https://review.openstack.org/#/c/297192 | 23:52 |
| *** shu-mutou-AFK is now known as shu-mutou | 23:52 | |
| JayF | I hate seeing things with a bunch of votes and not benig merged yet, especially when they are small changes | 23:52 |
| JayF | making someone rebase a 2 line change is evil :P | 23:52 |
| jroll | someone likes the 'no negative feedback' dash | 23:53 |
| jroll | +A | 23:53 |
| JayF | I *love* the no negative feedback dash | 23:53 |
| JayF | it's either an easy "me too", or I catch something nobody else has caught yet | 23:53 |
| jroll | jlvillal: btw, I made 'gate' an official tag in launchpad, we should totally use it | 23:54 |
| jroll | yeah | 23:54 |
| jlvillal | jroll: +1 | 23:54 |
| JayF | plus I feel like if someone gets a bunch of + reviews, and no -, we sorta have a social contract to merge their stuff if they got it to that point (assuming we actually want it, etc etc) | 23:54 |
| jlvillal | jroll: Not as easy review :) https://review.openstack.org/287306 | 23:56 |
| jlvillal | But not that hard.... | 23:56 |
| JayF | I think I have +1s on almost all outstanding patches, I keep clicking links and seeing familiar things, haha | 23:56 |
| jlvillal | JayF: I try to use this: http://www.sodarock.com/~john/dash/ironic.html | 23:57 |
| JayF | that looks similar to what I use? | 23:57 |
| JayF | https://review.openstack.org/#/dashboard/?foreach=%28project%3Aopenstack%2Fironic+OR+project%3Aopenstack%2Fpython-ironicclient+OR+project%3Aopenstack%2Fironic-python-agent+OR+project%3Aopenstack%2Fironic-specs%29+status%3Aopen+NOT+label%3ACode-Review%3C%3D-2&title=Ironic+Inbox&My+Patches+Requiring+Attention=owner%3Aself+%28label%3AVerified-1%252cjenkins+OR+label%3ACode-Review-1+OR+label%3AWorkflow%3 | 23:57 |
| JayF | C%3D-1%29&Ironic+Specs=NOT+label%3AWorkflow%3E%3D1+NOT+owner%3Aself+project%3Aopenstack%2Fironic-specs&Needs+Approval=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D-1+label%3AVerified%3E%3D1%252cjenkins+NOT+owner%3Aself+label%3ACode-Review%3E%3D2+NOT+label%3ACode-Review-1&Needs+Reverify=label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D-1+label%3AVerified%3C%3D-1%252cjenkins&5+Days+Without+F | 23:57 |
| JayF | eedback=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D-1+label%3AVerified%3E%3D1%252cjenkins+NOT+owner%3Aself+NOT+project%3Aopenstack%2Fironic-specs+NOT+label%3ACode-Review%3C%3D2+age%3A5d&No+Negative+Feedback=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D-1+label%3AVerified%3E%3D1%252cjenkins+NOT+owner%3Aself+NOT+project%3Aopenstack%2Fironic-specs+NOT+label%3ACode-Review%3C%3D-1+NOT | 23:57 |
| JayF | +label%3ACode-Review%3E%3D2+limit%3A50&With+Negative+Feedback=NOT+label%3AWorkflow%3C%3D-1+NOT+label%3AVerified%3C%3D-1%252cjenkins+NOT+owner%3Aself+NOT+project%3Aopenstack%2Fironic-specs+label%3ACode-Review-1+limit%3A20&Work+In+Progress+Or+Unverified=NOT+label%3AWorkflow%3E%3D1+NOT+owner%3Aself+%28label%3AWorkflow%3C%3D-1+OR+label%3AVerified%3C%3D-1%252cjenkins%29+limit%3A20 | 23:57 |
| JayF | oh holy url, sorry about that | 23:57 |
| jlvillal | LOL | 23:57 |
| JayF | lol | 23:57 |
| jlvillal | JayF: But yes, pretty much very similar. Taken from dash-creator | 23:58 |
| jlvillal | Minor mods in ordering | 23:58 |
| JayF | well this was the one that was in the whiteboard for a while | 23:58 |
| JayF | as the review dashboard | 23:58 |
| JayF | and I bookmarked it | 23:58 |
| jroll | omg | 23:58 |
| JayF | I know I need to update it b/c it doesn't see any ironic-lib stuff today | 23:58 |
| jroll | /kick JayF spam! | 23:58 |
| JayF | jroll: SPAM?!? https://www.youtube.com/watch?v=anwy2MPT5RE | 23:58 |
| jroll | JayF: you know about https://github.com/openstack/gerrit-dash-creator right? | 23:58 |
| openstackgerrit | Merged openstack/ironic: Addressing nits on I2984cd9d469622a65201fd9d50f964b144cce625 https://review.openstack.org/296621 | 23:59 |
| JayF | yep, when I managed the team used it to make an "onmetal team upstream" dash | 23:59 |
| jroll | oh neat | 23:59 |
| JayF | it was depressing and got moreso the closer we got to v2 launch, haha | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!