| *** chas_ has quit IRC | 00:03 | |
| *** bfournie has joined #openstack-ironic | 00:05 | |
| *** absubram has quit IRC | 00:13 | |
| *** ijw has quit IRC | 00:18 | |
| *** ijw has joined #openstack-ironic | 00:19 | |
| *** ijw has quit IRC | 00:31 | |
| *** godp1301_ has joined #openstack-ironic | 00:34 | |
| *** rpioso has quit IRC | 00:38 | |
| *** rama_y has quit IRC | 00:38 | |
| *** causten has quit IRC | 00:39 | |
| *** jcoufal has quit IRC | 00:50 | |
| *** amotoki has joined #openstack-ironic | 00:52 | |
| *** ijw has joined #openstack-ironic | 00:52 | |
| *** hoangcx has joined #openstack-ironic | 00:54 | |
| *** bdeschenes has quit IRC | 00:55 | |
| *** ijw has quit IRC | 00:57 | |
| *** amotoki has quit IRC | 00:57 | |
| *** ijw has joined #openstack-ironic | 01:02 | |
| *** phuongnh has joined #openstack-ironic | 01:02 | |
| *** ijw has quit IRC | 01:07 | |
| *** godp1301 has joined #openstack-ironic | 01:14 | |
| *** rajinir has quit IRC | 01:16 | |
| *** godp1301_ has quit IRC | 01:17 | |
| *** hw_wutianwei has joined #openstack-ironic | 01:17 | |
| *** aNuposic has joined #openstack-ironic | 01:20 | |
| *** Sukhdev has quit IRC | 01:22 | |
| *** amotoki has joined #openstack-ironic | 01:29 | |
| *** godp1301 has quit IRC | 01:30 | |
| hogepodge | I see that snapshots are on the wish list, and I just found out that the chameleon project has implemented it for their ironic-based cloud | 01:32 |
|---|---|---|
| hogepodge | Anyone want new to connect you with them to see what they did? | 01:32 |
| hogepodge | Want me to connect you with them that is | 01:32 |
| *** ijw has joined #openstack-ironic | 01:37 | |
| *** godp1301 has joined #openstack-ironic | 01:40 | |
| *** causten has joined #openstack-ironic | 01:43 | |
| *** causten has quit IRC | 01:47 | |
| *** shuquan has joined #openstack-ironic | 01:48 | |
| *** causten_ has joined #openstack-ironic | 01:48 | |
| *** causten_ has joined #openstack-ironic | 01:49 | |
| *** gcb has joined #openstack-ironic | 01:50 | |
| *** causten_ has quit IRC | 01:53 | |
| *** links has joined #openstack-ironic | 01:55 | |
| *** zhangjl has joined #openstack-ironic | 01:59 | |
| *** shuquan_ has joined #openstack-ironic | 02:02 | |
| *** shuquan has quit IRC | 02:03 | |
| *** links has quit IRC | 02:07 | |
| *** causten has joined #openstack-ironic | 02:09 | |
| *** shuquan_ has quit IRC | 02:09 | |
| *** causten_ has joined #openstack-ironic | 02:10 | |
| *** causten__ has joined #openstack-ironic | 02:11 | |
| *** causten has quit IRC | 02:14 | |
| *** causten_ has quit IRC | 02:15 | |
| *** shuquan has joined #openstack-ironic | 02:15 | |
| *** godp1301 has quit IRC | 02:19 | |
| *** shuquan has quit IRC | 02:20 | |
| *** causten has joined #openstack-ironic | 02:33 | |
| *** syed__ has quit IRC | 02:35 | |
| *** causten__ has quit IRC | 02:36 | |
| *** chas has joined #openstack-ironic | 02:40 | |
| *** chas has quit IRC | 02:45 | |
| *** aNuposic has quit IRC | 02:49 | |
| *** shuquan has joined #openstack-ironic | 02:50 | |
| *** yuanying has quit IRC | 02:52 | |
| *** rloo has quit IRC | 02:57 | |
| *** ijw has quit IRC | 03:02 | |
| *** baoli has joined #openstack-ironic | 03:03 | |
| *** zhangyufei has joined #openstack-ironic | 03:05 | |
| *** chas has joined #openstack-ironic | 03:08 | |
| *** ijw has joined #openstack-ironic | 03:08 | |
| *** causten_ has joined #openstack-ironic | 03:12 | |
| *** chas has quit IRC | 03:12 | |
| *** rbrud has joined #openstack-ironic | 03:13 | |
| *** ijw has quit IRC | 03:13 | |
| *** causten has quit IRC | 03:15 | |
| *** causten has joined #openstack-ironic | 03:18 | |
| *** causten__ has joined #openstack-ironic | 03:19 | |
| *** causten_ has quit IRC | 03:21 | |
| *** vikrant has joined #openstack-ironic | 03:22 | |
| *** causten has quit IRC | 03:23 | |
| *** causten has joined #openstack-ironic | 03:28 | |
| *** chas has joined #openstack-ironic | 03:29 | |
| *** causten__ has quit IRC | 03:31 | |
| *** causten_ has joined #openstack-ironic | 03:32 | |
| *** chas has quit IRC | 03:33 | |
| *** causten has quit IRC | 03:34 | |
| *** ajr1 has joined #openstack-ironic | 03:34 | |
| *** zhangyufei has quit IRC | 03:35 | |
| *** shuquan has quit IRC | 03:35 | |
| *** godp1301 has joined #openstack-ironic | 03:40 | |
| *** fragatin_ has joined #openstack-ironic | 03:47 | |
| *** yuanying has joined #openstack-ironic | 03:50 | |
| *** fragatina has quit IRC | 03:51 | |
| *** fragatin_ has quit IRC | 03:52 | |
| *** baoli has quit IRC | 03:52 | |
| *** vikrant is now known as vikrant|brb | 03:57 | |
| *** Nisha has joined #openstack-ironic | 04:01 | |
| *** Nisha is now known as Guest32802 | 04:02 | |
| *** godp1301 has quit IRC | 04:06 | |
| *** amotoki has quit IRC | 04:09 | |
| *** vikrant|brb is now known as vikrant | 04:10 | |
| *** priteau has joined #openstack-ironic | 04:13 | |
| *** Sukhdev has joined #openstack-ironic | 04:14 | |
| *** priteau has quit IRC | 04:17 | |
| *** chas has joined #openstack-ironic | 04:22 | |
| *** bks has joined #openstack-ironic | 04:24 | |
| *** chas has quit IRC | 04:28 | |
| *** vmud213 has joined #openstack-ironic | 04:30 | |
| *** rbrud has quit IRC | 04:39 | |
| *** ajr1 has quit IRC | 04:40 | |
| *** ajr1 has joined #openstack-ironic | 04:40 | |
| *** khamtamtun has joined #openstack-ironic | 04:40 | |
| *** khamtamtun has quit IRC | 04:44 | |
| *** godp1301 has joined #openstack-ironic | 04:45 | |
| *** krtaylor has joined #openstack-ironic | 04:45 | |
| *** godp1301 has quit IRC | 04:52 | |
| *** godp1301 has joined #openstack-ironic | 04:53 | |
| *** ajr1 has quit IRC | 04:54 | |
| *** zhangyufei has joined #openstack-ironic | 04:55 | |
| *** godp1301 has quit IRC | 04:55 | |
| *** shuquan has joined #openstack-ironic | 04:57 | |
| *** mvpnitesh has joined #openstack-ironic | 04:59 | |
| *** ijw has joined #openstack-ironic | 05:01 | |
| mvpnitesh | Hi guys , Good morning | 05:08 |
| mvpnitesh | i'm trying to provision a baremetal, in /tftpboot folder is created with the node id. Inside that config file is also getting created, but iscsi_target_iqn, deployment_id,deployment_key,ironic_api_url are not getting popullated | 05:09 |
| mvpnitesh | i was able to provision baremetal for Kilo, Liberty and Mitaka . But in Newton i was not able to do it, should i takecare of any configuration in particular for this Newton release | 05:11 |
| *** dims has quit IRC | 05:20 | |
| *** Sukhdev has quit IRC | 05:25 | |
| *** pawnesh has joined #openstack-ironic | 05:32 | |
| *** rama_y has joined #openstack-ironic | 05:34 | |
| *** aparnav has joined #openstack-ironic | 05:41 | |
| *** moshele has joined #openstack-ironic | 05:44 | |
| *** zhangyufei has quit IRC | 05:50 | |
| *** mvpnitesh has quit IRC | 05:51 | |
| *** mvpnitesh has joined #openstack-ironic | 05:51 | |
| *** harlowja has quit IRC | 05:55 | |
| *** gcb has quit IRC | 05:58 | |
| *** dims has joined #openstack-ironic | 05:59 | |
| *** gcb has joined #openstack-ironic | 05:59 | |
| *** vmud213 has quit IRC | 05:59 | |
| *** vmud213 has joined #openstack-ironic | 06:00 | |
| *** hshiina has joined #openstack-ironic | 06:00 | |
| *** amotoki has joined #openstack-ironic | 06:10 | |
| *** ChubYann has quit IRC | 06:10 | |
| *** zhangyufei has joined #openstack-ironic | 06:16 | |
| *** jaosorior has joined #openstack-ironic | 06:16 | |
| *** chas has joined #openstack-ironic | 06:17 | |
| zhangyufei | Is your node in enroll state？ | 06:18 |
| zhangyufei | @mvpnitesh | 06:19 |
| openstackgerrit | Moshe Levi proposed openstack/ironic-python-agent: Add a new Hardware Manager for Mellanox NICs https://review.openstack.org/334816 | 06:19 |
| mvpnitesh | @zhangyufei : When i'm trying to provision , it is going to deploy failed state | 06:21 |
| *** chas has quit IRC | 06:21 | |
| zhangyufei | Do you get any log info in ironic conductor? I’m testing Newton now, looks good for me. | 06:28 |
| *** harlowja_at_home has joined #openstack-ironic | 06:32 | |
| *** harlowja_at_home has quit IRC | 06:33 | |
| mvpnitesh | nothing in the logs , in the baremetal the deploy_ramdisk and deploy_kernal are getting copied and when deploying it is getting stuck | 06:37 |
| mvpnitesh | but , i can provision the same baremetal using Mitaka | 06:39 |
| *** jaosorior has quit IRC | 06:41 | |
| *** jaosorior has joined #openstack-ironic | 06:41 | |
| *** rama_y has quit IRC | 06:45 | |
| *** chas has joined #openstack-ironic | 06:45 | |
| *** chas has quit IRC | 06:49 | |
| *** moshele has quit IRC | 06:52 | |
| zhangyufei | When you provision the machine, did it enter the PXE mode successfully? If is, did it call back successfully?Maybe the error log is on the machine’s screen. | 06:55 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled https://review.openstack.org/393745 | 06:55 |
| *** tuanluong has joined #openstack-ironic | 07:10 | |
| *** causten has joined #openstack-ironic | 07:11 | |
| *** mjura has joined #openstack-ironic | 07:11 | |
| *** chas has joined #openstack-ironic | 07:12 | |
| *** causten has quit IRC | 07:13 | |
| *** causten_ has quit IRC | 07:14 | |
| *** chas has quit IRC | 07:16 | |
| mvpnitesh | @<zhangyufei> : yes, it was in wait-call-back-state , No errors in ironic conductor , It will be in wait-call-back-state for 1 hr and then it would enter deploy failed. | 07:18 |
| mvpnitesh | but using the same machine we were able to provision a baremetal using Mitaka | 07:18 |
| *** mjura has quit IRC | 07:19 | |
| *** mjura has joined #openstack-ironic | 07:19 | |
| zhangyufei | if it is in wait call back states, you can ssh to it, and read the ironic python agent’s log | 07:25 |
| zhangyufei | http://docs.openstack.org/developer/ironic-python-agent/troubleshooting.html | 07:27 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Move `deploy_forces_oob_reboot` to deploy drivers https://review.openstack.org/397843 | 07:28 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from OneView drivers https://review.openstack.org/397846 | 07:28 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove iBoot, WoL and AMT drivers https://review.openstack.org/397847 | 07:28 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from most drivers https://review.openstack.org/397844 | 07:28 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru from iLO drvers https://review.openstack.org/397845 | 07:28 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Remove agent vendor passthru completely https://review.openstack.org/397848 | 07:28 |
| zhangyufei | I got this kind of error before, because i use hostname in keystone endpoint, the node to be deploed can’t resolve the hostnames, so it can’t call back. | 07:29 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: PoC Manage VNC console on iDRAC via WS-MAN API https://review.openstack.org/396661 | 07:29 |
| mvpnitesh | ok thanks @Zhangyuafei . I'll check it | 07:31 |
| vsaienk0 | sambetts: you can use local.conf from the CI http://logs.openstack.org/73/368173/41/experimental/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-multinode-nv/3594741/logs/ just remove ERROR_ON_CLONE=True or set to False and might be launch services in screen USE_SCREEN=True, it will work locally | 07:31 |
| vsaienk0 | sambetts: and add [[local|localrc]] to the file head | 07:32 |
| *** vsaienko has joined #openstack-ironic | 07:32 | |
| mvpnitesh | @zhangyufei: Do i need to install ironic-python-agent in this Newton Openstack release | 07:33 |
| zhangyufei | the agent runs was installed in the ramdisk , you don’t need to install it on the server | 07:34 |
| *** jaosorior has quit IRC | 07:34 | |
| *** pcaruana has joined #openstack-ironic | 07:41 | |
| *** moshele has joined #openstack-ironic | 07:42 | |
| *** jaosorior has joined #openstack-ironic | 07:54 | |
| *** gcb has quit IRC | 08:02 | |
| *** karthiks has quit IRC | 08:07 | |
| *** ccamacho has joined #openstack-ironic | 08:07 | |
| *** priteau has joined #openstack-ironic | 08:14 | |
| *** gcb has joined #openstack-ironic | 08:15 | |
| *** priteau has quit IRC | 08:18 | |
| vdrok | good morning ironic! | 08:19 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic-python-agent: Python 3 Compatible JSON https://review.openstack.org/363605 | 08:19 |
| *** vmud213 has quit IRC | 08:20 | |
| *** ralonsoh has joined #openstack-ironic | 08:20 | |
| *** karthiks has joined #openstack-ironic | 08:22 | |
| *** vmud213 has joined #openstack-ironic | 08:23 | |
| vsaienk0 | Morning Ironic'ers, vdrok, sambetts, dtantsur, lucasagomes, mgould and others! | 08:24 |
| vdrok | morning vsaienk0 | 08:25 |
| *** alexpilotti has joined #openstack-ironic | 08:26 | |
| *** gcb has quit IRC | 08:27 | |
| *** e0ne has joined #openstack-ironic | 08:30 | |
| *** gcb has joined #openstack-ironic | 08:30 | |
| *** alexpilotti has quit IRC | 08:30 | |
| *** alexpilotti has joined #openstack-ironic | 08:30 | |
| *** jpich has joined #openstack-ironic | 08:31 | |
| *** vsaienko has quit IRC | 08:32 | |
| *** mvpnitesh has quit IRC | 08:41 | |
| tuanluong | morning ironic | 08:44 |
| vdrok | morning tuanluong | 08:49 |
| tuanluong | mornig vdrok | 08:49 |
| *** athomas has joined #openstack-ironic | 08:56 | |
| *** Guest32802 has quit IRC | 08:58 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** e0ne has quit IRC | 09:02 | |
| *** dtantsur|afk is now known as dtantsur | 09:03 | |
| dtantsur | Morning Ironic | 09:03 |
| *** zzzeek has joined #openstack-ironic | 09:03 | |
| vdrok | morning dtantsur | 09:03 |
| dtantsur | jroll, JayF, I do plan to propose all stable/newton releases today (CET afternoon) | 09:06 |
| openstackgerrit | Aparna proposed openstack/proliantutils: Add Sanitize disk erase support using ssa https://review.openstack.org/398202 | 09:06 |
| *** karthiks has quit IRC | 09:06 | |
| aarefiev | morning all! | 09:08 |
| *** ohamada has joined #openstack-ironic | 09:13 | |
| *** hshiina has quit IRC | 09:14 | |
| *** causten has joined #openstack-ironic | 09:14 | |
| *** karthiks has joined #openstack-ironic | 09:19 | |
| *** causten has quit IRC | 09:19 | |
| *** makowals has quit IRC | 09:22 | |
| *** makowals has joined #openstack-ironic | 09:22 | |
| *** bdeschenes has joined #openstack-ironic | 09:24 | |
| vdrok | morning aarefiev | 09:26 |
| aarefiev | hey vdrok | 09:29 |
| *** chas has joined #openstack-ironic | 09:31 | |
| *** lucas-afk is now known as lucasagomes | 09:32 | |
| *** dtantsur is now known as dtantsur|afk | 09:33 | |
| *** dtantsur|afk is now known as dtantsur|bbl | 09:33 | |
| lucasagomes | morning all | 09:33 |
| *** makowals has quit IRC | 09:36 | |
| *** makowals has joined #openstack-ironic | 09:36 | |
| openstackgerrit | octopuszhang proposed openstack/ironic-inspector: Add !requiretty to sudoers config in install document https://review.openstack.org/398227 | 09:40 |
| *** derekh has joined #openstack-ironic | 09:41 | |
| openstackgerrit | octopuszhang proposed openstack/ironic-inspector: Add !requiretty to sudoers config in install document https://review.openstack.org/398227 | 09:45 |
| *** roeyc1 has joined #openstack-ironic | 09:45 | |
| aarefiev | morning lucasagomes | 09:47 |
| *** chas has quit IRC | 09:52 | |
| *** chas has joined #openstack-ironic | 09:52 | |
| vdrok | good morning lucasagomes | 09:56 |
| *** chas has quit IRC | 09:57 | |
| *** priteau has joined #openstack-ironic | 09:58 | |
| *** chas has joined #openstack-ironic | 09:58 | |
| *** milan has joined #openstack-ironic | 10:02 | |
| *** zhangyufei has quit IRC | 10:02 | |
| *** chas has quit IRC | 10:02 | |
| *** chas has joined #openstack-ironic | 10:04 | |
| *** e0ne has joined #openstack-ironic | 10:05 | |
| *** chas has quit IRC | 10:07 | |
| *** chas has joined #openstack-ironic | 10:07 | |
| *** mvpnitesh has joined #openstack-ironic | 10:11 | |
| *** dstepanenko_ is now known as dstepanenko | 10:13 | |
| milan | morning Ironic! :) | 10:13 |
| *** shuquan has quit IRC | 10:14 | |
| *** hoangcx has quit IRC | 10:16 | |
| *** chas has quit IRC | 10:18 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add portgroup to api-ref https://review.openstack.org/393834 | 10:19 |
| *** phuongnh has quit IRC | 10:20 | |
| *** tuanluong has quit IRC | 10:22 | |
| *** chas has joined #openstack-ironic | 10:24 | |
| *** sambetts|afk is now known as sambetts | 10:28 | |
| sambetts | Morning all | 10:29 |
| milan | morning sambetts! :) | 10:29 |
| *** chas has quit IRC | 10:29 | |
| sambetts | o/ milan | 10:29 |
| vdrok | good morning sambetts | 10:34 |
| vdrok | and morning milan ! :) | 10:34 |
| milan | morning vdrok! :) | 10:34 |
| mvpnitesh | Morning all , i'm trying to provision baremetal using "pxe_ipmitool" driver. When i make the node-provision state to active , the baremetal is getting the ip and deploy kernal and deploy ramdisk. The node is going to wait-call-back state and for 1 hr the node is moving to deploy failed state | 10:36 |
| mvpnitesh | please help me , i'm not getting any error in ironic-conductor.log or ironic-api.log | 10:36 |
| *** bdeschenes has quit IRC | 10:37 | |
| sambetts | mvpnitesh: are you using Ironic in OpenStack? | 10:37 |
| sergek_ | morning vdrok , milan , sambetts , aarefiev , vsaienk0 , pas-ha | 10:37 |
| milan | morning sergek_ ! :) | 10:37 |
| mvpnitesh | sambetts: yes | 10:37 |
| vdrok | good morning sergek_ | 10:37 |
| aarefiev | morning sergek_, milan, sambetts | 10:37 |
| milan | morning aarefiev! :) | 10:38 |
| sambetts | mvpnitesh: After enrolling and getting the nodes into the avaible state in Ironic, you should do all deployment through nova | 10:38 |
| mvpnitesh | but till mitaka i was able to deploy through ironic by changing provision state to active | 10:39 |
| *** ipukha has joined #openstack-ironic | 10:40 | |
| sambetts | mvpnitesh: are you setting the instance information and creating a neutron port manually? | 10:40 |
| mvpnitesh | sambetts: yes | 10:41 |
| vdrok | mvpnitesh: what's the network_interface value in this node? | 10:41 |
| mvpnitesh | sambetts: I was able to provision the same bare metal using Mitaka , but in Newton release i'm not able to provision it. If i check the difference /tftpboot/node-id/config," disk= iscsi_target_iqn= deployment_id= deployment_key= ironic_api_url= " values are not getting popullated | 10:42 |
| patchbot | Error: No closing quotation | 10:42 |
| mvpnitesh | vdrok: network_interface value is None, nothing is present | 10:44 |
| *** zhangjl has quit IRC | 10:45 | |
| mvpnitesh | i created a ironic-port and neutron port using the MAC address of the baremetal and neutron port id to extra/vif_port_id field in ironic port | 10:46 |
| vdrok | mvpnitesh: please try to do node-show with --ironic-api-version latest, network_interface can't be Nonw | 10:47 |
| *** chas has joined #openstack-ironic | 10:49 | |
| *** zhangyufei has joined #openstack-ironic | 10:53 | |
| *** chas has quit IRC | 10:55 | |
| *** mgould|afk is now known as mgould | 10:58 | |
| mvpnitesh | i've done node-show and the values beside network interfaces nothing is there | 10:59 |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/397908 | 10:59 |
| *** bks has quit IRC | 11:04 | |
| mvpnitesh | in /tftpboot/node-id/config ," disk= iscsi_target_iqn= deployment_id= deployment_key= ironic_api_url= boot_option= " are not getting populated | 11:05 |
| patchbot | Error: No closing quotation | 11:05 |
| *** chas has joined #openstack-ironic | 11:09 | |
| lucasagomes | mvpnitesh, these values were only used by the old bash ramdisk (prior to IPA) AFAIR, so they have been removed since | 11:13 |
| *** chas has quit IRC | 11:13 | |
| lucasagomes | mvpnitesh, which deploy ramdisk are you using ? You know if it's IPA or the old bash ramdisk (created with the "deploy-ironic" diskimage-builder element) | 11:14 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Docs: Document using operators with root device hints https://review.openstack.org/386714 | 11:14 |
| mvpnitesh | @lucasagomes: So what should i do now , i'm using old deploy ramdisk | 11:14 |
| mvpnitesh | should i create a new deploy ramdisk using IPA ?? | 11:15 |
| lucasagomes | mvpnitesh, you need to update it to use IPA | 11:15 |
| *** chas has joined #openstack-ironic | 11:15 | |
| lucasagomes | mvpnitesh, yes, you can use DIB if you want with the ironic-agent element | 11:15 |
| lucasagomes | (or agent-ironic, I don't remember) | 11:15 |
| lucasagomes | or if you don't want to use DIB there's also other ways to generate the ramdisk: https://github.com/openstack/ironic-python-agent/tree/master/imagebuild | 11:16 |
| mvpnitesh | @lucasagomes : can you tell me how i can update and use IPA | 11:16 |
| mvpnitesh | should i install IPA in my current ironic setup ? | 11:16 |
| lucasagomes | mvpnitesh, #1. generate the ramdisk + kernel (or download it) #2. Upload it to glance #3. Update the ironic nodes "driver_info/deploy_kernel" and "driver_info/deploy_ramdisk" fields to point to the UUID in glance | 11:17 |
| sambetts | mvpnitesh: http://docs.openstack.org/project-install-guide/baremetal/draft/configure-integration.html#create-and-add-images-to-the-image-service step 3 there | 11:17 |
| lucasagomes | and that should be it | 11:17 |
| lucasagomes | mvpnitesh, no you don't need to install anything, the ramdisk should be a drop-in replacement | 11:18 |
| mvpnitesh | thankyou @ lucasagomes , sambetts | 11:18 |
| lucasagomes | np | 11:18 |
| milan | aarefiev, sambetts, mind taking look at: https://review.openstack.org/#/c/344921/ ? | 11:18 |
| patchbot | patch 344921 - ironic-inspector - Add API for listing all introspection statuses | 11:18 |
| *** athomas has quit IRC | 11:18 | |
| *** chas has quit IRC | 11:19 | |
| lucasagomes | mvpnitesh, btw as a note, it would be good to check http://docs.openstack.org/releasenotes/ironic/ prior to updating ironic | 11:19 |
| lucasagomes | check the upgrade notes for the version you are going to upgrade to | 11:19 |
| lucasagomes | because there will be the information you need to know to go from one release to another | 11:20 |
| *** chas has joined #openstack-ironic | 11:21 | |
| *** athomas has joined #openstack-ironic | 11:22 | |
| aarefiev | milan: sure, I 'll take a look in minute | 11:22 |
| milan | aarefiev, thanks! | 11:22 |
| mvpnitesh | thanks lucasagomes | 11:24 |
| *** jaosorior is now known as jaosorior_lunch | 11:25 | |
| *** chas has quit IRC | 11:25 | |
| *** amotoki_ has joined #openstack-ironic | 11:30 | |
| *** amotoki has quit IRC | 11:33 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: DevStack: Configure nodes/environment to boot in UEFI mode https://review.openstack.org/374988 | 11:33 |
| *** jkilpatr has quit IRC | 11:33 | |
| openstackgerrit | Charlle Daniel proposed openstack/ironic: Ensures that OneView nodes are free for use by Ironic https://review.openstack.org/372762 | 11:42 |
| *** dtantsur|bbl is now known as dtantsur | 11:44 | |
| dtantsur | morning whoever I missed, lucasagomes, aarefiev, milan, vdrok, sambetts, sergek_ | 11:45 |
| *** vmud213 has quit IRC | 11:45 | |
| lucasagomes | o/ | 11:45 |
| aarefiev | morning dtantsur | 11:45 |
| milan | morning dtantsur! :) | 11:45 |
| zhangyufei | morning | 11:46 |
| *** chas has joined #openstack-ironic | 11:51 | |
| *** chas has quit IRC | 11:55 | |
| jroll | hogepodge: yes, I'd love to chat with them | 11:56 |
| jroll | morning everyone | 11:57 |
| *** aparnav has quit IRC | 11:57 | |
| sambetts | o/ jroll | 11:57 |
| milan | morning jroll! :) | 11:58 |
| *** aparnav has joined #openstack-ironic | 12:00 | |
| *** kvcobb has quit IRC | 12:01 | |
| *** dprince has joined #openstack-ironic | 12:01 | |
| *** jaosorior_lunch is now known as jaosorior | 12:09 | |
| vdrok | good morning jroll | 12:10 |
| lucasagomes | morning o/ | 12:10 |
| *** lucasagomes is now known as lucas-hungry | 12:11 | |
| * jroll has a dentist appt, will bbl | 12:11 | |
| *** jkilpatr has joined #openstack-ironic | 12:12 | |
| *** aparnav has quit IRC | 12:12 | |
| xavierr|afk | morning all | 12:13 |
| *** xavierr|afk is now known as xavierr | 12:14 | |
| xavierr | hey milan o/ | 12:14 |
| *** pc_m has quit IRC | 12:14 | |
| milan | hey xavierr! :) | 12:14 |
| *** g3ek has quit IRC | 12:14 | |
| *** vsaienko has joined #openstack-ironic | 12:15 | |
| *** dprince has quit IRC | 12:15 | |
| *** rook-out has quit IRC | 12:15 | |
| *** dprince has joined #openstack-ironic | 12:16 | |
| *** pc_m has joined #openstack-ironic | 12:16 | |
| *** g3ek has joined #openstack-ironic | 12:17 | |
| mgould | morning xavierr lucas-hungry sambetts milan dtantsur zhangyufei | 12:18 |
| *** rook has joined #openstack-ironic | 12:18 | |
| milan | morning mgould! :) | 12:18 |
| *** rook is now known as Guest45665 | 12:18 | |
| *** chas has joined #openstack-ironic | 12:22 | |
| openstackgerrit | Merged openstack/ironic-inspector: Add !requiretty to sudoers config in install document https://review.openstack.org/398227 | 12:25 |
| xavierr | mgould: hey miles :) | 12:25 |
| *** chas has quit IRC | 12:26 | |
| *** vikrant has quit IRC | 12:29 | |
| mgould | is yuriyz around? | 12:31 |
| yuriyz | hi mgould | 12:31 |
| mgould | hi yuriyz | 12:31 |
| mgould | have you seen my responses to your comments on https://review.openstack.org/#/c/381991/ ? | 12:31 |
| patchbot | patch 381991 - ironic-specs - API Evolution - ETAG identifiers | 12:31 |
| mgould | [I only just posted them, so probably not] | 12:33 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Virtual Network Interface APIs https://review.openstack.org/327046 | 12:34 |
| mgould | yuriyz: tl;dr, galyna is talking about client-side caches, and the whole point of ETags is so client-side caches can be properly invalidated | 12:34 |
| *** priteau has quit IRC | 12:34 | |
| yuriyz | mgould probably details about client software should be moved from "Client (CLI) impact" section | 12:35 |
| mgould | moved *from* that section? | 12:35 |
| mgould | on the grounds that caching issues are common to all clients, not just the CLI client? | 12:36 |
| *** godp1301 has joined #openstack-ironic | 12:36 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DNM: Test patch to verify bond https://review.openstack.org/388660 | 12:36 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Ironic devstack portgroup support https://review.openstack.org/381743 | 12:36 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Rely on portgroup standalone_ports_supported https://review.openstack.org/360647 | 12:36 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Tempest tests for portgroups https://review.openstack.org/382476 | 12:36 |
| *** hw_wutianwei has quit IRC | 12:37 | |
| yuriyz | mgould I leave a comments after few min | 12:38 |
| *** gcb has quit IRC | 12:38 | |
| * mgould just left another one | 12:38 | |
| mgould | your question's reasonable if you were asking about where python-ironicclient will keep its cache - sorry! | 12:38 |
| *** fxpester has joined #openstack-ironic | 12:40 | |
| *** godp1301 has quit IRC | 12:40 | |
| *** gcb has joined #openstack-ironic | 12:42 | |
| openstackgerrit | Charlle Daniel proposed openstack/ironic: Ensures that OneView nodes are free for use by Ironic https://review.openstack.org/372762 | 12:43 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: ETAG supporting to enhance API evolution https://review.openstack.org/392213 | 12:46 |
| *** priteau has joined #openstack-ironic | 12:48 | |
| *** moshele has quit IRC | 12:49 | |
| * dtantsur got some wine and is ready to continue working | 12:51 | |
| *** chas has joined #openstack-ironic | 12:52 | |
| sambetts | dtantsur: its 5pm somewhere right ;) | 12:54 |
| dtantsur | sambetts, 2pm, but it never prevented me from getting some good wine | 12:54 |
| dtantsur | ah, *somewhere* | 12:54 |
| sambetts | haha ;) | 12:54 |
| dtantsur | yeah, I still live by Russian clock I guess | 12:54 |
| *** chas has quit IRC | 12:57 | |
| sambetts | :-P | 12:58 |
| nicodemos | good morning | 12:59 |
| *** chas has joined #openstack-ironic | 13:00 | |
| mgould | morning nicodemos | 13:01 |
| mgould | dtantsur: it's still only 4pm in Moscow :-) | 13:02 |
| dtantsur | good time to start drinking | 13:02 |
| mgould | but it's 6pm in yekaterinburg and 2300 in vladivostok | 13:03 |
| *** moshele has joined #openstack-ironic | 13:03 | |
| *** ccamacho is now known as ccamacho|lunch | 13:04 | |
| zhangyufei | It’s already 21pm in china | 13:04 |
| zhangyufei | hi, does ironic-inspector support fake-token? | 13:04 |
| dtantsur | zhangyufei, you mean working without keystone authentication? yes, it does | 13:04 |
| zhangyufei | ok..thank you | 13:05 |
| *** moshele has quit IRC | 13:06 | |
| dtantsur | I'm pretty sure that's how Bifrost uses it | 13:07 |
| zhangyufei | I just configure ironic with fake token, and ironic-inspector with keystone, and get a Authentication Required error,it seem that I should use same auth_strategy. | 13:07 |
| dtantsur | I think so too | 13:08 |
| *** moshele has joined #openstack-ironic | 13:09 | |
| zhangyufei | Maybe I can add a note for it, in inspection install guide | 13:09 |
| zhangyufei | I will try to push a patch for this later…thank you | 13:10 |
| yuriyz | mgould left some comments on the spec, agree that if cache strategy will be defined outside python-ironicclient and customized for use case, we should not have many troubles | 13:10 |
| mgould | yuriyz: cool | 13:10 |
| *** trown|outtypewww is now known as trown | 13:11 | |
| *** causten has joined #openstack-ironic | 13:16 | |
| *** lucas-hungry is now known as lucasagomes | 13:20 | |
| *** bdeschenes has joined #openstack-ironic | 13:20 | |
| dtantsur | sambetts, dunno if you're a proper person to ask, but have you seen https://review.openstack.org/397959 ? | 13:21 |
| patchbot | patch 397959 - tripleo-common - Remove python-networking-cisco from overcloud-full... | 13:21 |
| *** causten has quit IRC | 13:21 | |
| vsaienk0 | sambetts: the CI failing to attach/detach vif because of https://review.openstack.org/#/c/327046/22/ironic/api/controllers/v1/node.py@1122 | 13:21 |
| patchbot | patch 327046 - ironic - Add Virtual Network Interface APIs | 13:21 |
| vsaienk0 | sambetts: v1/nodes/{node}/validate returns 404 | 13:21 |
| *** jheroux has joined #openstack-ironic | 13:24 | |
| dmellado | vsaienk0: dtantsur o/ | 13:26 |
| dmellado | g' day | 13:26 |
| dtantsur | hey | 13:26 |
| dmellado | do you fuys know if anton arefiev is going to workflow+1 this patch? | 13:26 |
| dmellado | https://review.openstack.org/#/c/366806/ | 13:26 |
| patchbot | patch 366806 - tempest - Revert "Fix exec_command to hang indefinitely" | 13:26 |
| dmellado | otherwise I'll re-propose it | 13:26 |
| dmellado | %s/fuys/guys | 13:26 |
| dtantsur | dmellado, he's aarefiev here, but I don't think other qa folks will approve it | 13:27 |
| dtantsur | I think it breaks someone else (though not in openstack CI) | 13:27 |
| dmellado | I added them to at least open up the discussion | 13:27 |
| *** bdeschenes has quit IRC | 13:27 | |
| dtantsur | for the record, I'm fine with the revert | 13:27 |
| dmellado | in any case I'd love to check what's the issue, as I was only able to reproduce that with ironic tempest tests enabled... | 13:28 |
| dtantsur | aarefiev and I have spent a few fix trying to figure out. we were not even able to reproduce it locally. | 13:28 |
| *** vsaienko has quit IRC | 13:29 | |
| dmellado | dtantsur: I can give you an environment if you want to check it out | 13:29 |
| mat128 | good morning ironic | 13:30 |
| dtantsur | I'm sorry, I'm afraid I won't help much. I know very little about how networking works in OpenStack.. | 13:30 |
| dtantsur | morning mat128 | 13:30 |
| mat128 | fun fact: my 3 year old daughter says pixie boots looks angry - https://dtantsur.github.io/talks/devconf2015/img/Ironic_mascot_color.png | 13:30 |
| dtantsur | well, not AS angry as the new one ;) | 13:31 |
| mat128 | haha | 13:31 |
| mat128 | yup | 13:31 |
| *** iurygregory has joined #openstack-ironic | 13:31 | |
| aarefiev | dmellado: mtreinish was trying to refactor ssh code https://review.openstack.org/#/c/367478/ | 13:32 |
| patchbot | patch 367478 - tempest - Refactor ssh exec command (ABANDONED) | 13:32 |
| dmellado | aarefiev: I see, hmmm well | 13:33 |
| *** bfournie has quit IRC | 13:33 | |
| dmellado | mtreinish: will be most probably out for quite some weeks | 13:33 |
| dmellado | but I'll have a look at that whenever I get the chance | 13:34 |
| dmellado | aarefiev: for now, could you put your patch to +1 | 13:34 |
| mat128 | dtantsur: just showed her both of them and she prefers the new one | 13:34 |
| dmellado | or do you still plan to do some more work on it? | 13:34 |
| mat128 | dtantsur: also says it looks more angry | 13:34 |
| dmellado | heh, didn't look at your mascot | 13:35 |
| dmellado | xD | 13:35 |
| dtantsur | mat128, ah, so she just like the angry variant :) fair enough | 13:36 |
| *** amotoki_ has quit IRC | 13:36 | |
| dmellado | aarefiev: thanks! | 13:37 |
| aarefiev | dmellado: basically I don't know why it works for ironic and doesn't for inspector, I assume there is possible some network misconfiguration | 13:39 |
| *** zhangyufei has quit IRC | 13:39 | |
| dmellado | interesting | 13:40 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add ironic resources CRUD notifications https://review.openstack.org/356541 | 13:40 |
| dmellado | were you also experiencing the same test timouts that I mention in my comment there? | 13:40 |
| aarefiev | it doesn't hang all the time, sometimes ping just fails | 13:40 |
| aarefiev | dmellado: I saw hangs, it was deadlock in paramiko | 13:42 |
| *** e0ne has quit IRC | 13:42 | |
| *** shuquan has joined #openstack-ironic | 13:47 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 13:47 |
| *** hw_wutianwei has joined #openstack-ironic | 13:49 | |
| *** Administrator_ has quit IRC | 13:51 | |
| *** Administrator_ has joined #openstack-ironic | 13:52 | |
| mrtenio | Good morning Ironic. | 13:56 |
| TheJulia | good morning | 13:56 |
| *** causten has joined #openstack-ironic | 13:57 | |
| *** amotoki has joined #openstack-ironic | 13:59 | |
| *** kvcobb has joined #openstack-ironic | 14:00 | |
| vdrok | good morning xavierr dmellado mat128 mrtenio and TheJulia | 14:01 |
| *** rbartal has quit IRC | 14:01 | |
| *** bfournie has joined #openstack-ironic | 14:02 | |
| *** rloo has joined #openstack-ironic | 14:03 | |
| *** Goneri has joined #openstack-ironic | 14:06 | |
| mrtenio | Hi TheJulia, vdrok, I am following the updates to this patch https://review.openstack.org/#/c/372762/, lucasagomes noticied that to verify if the node is in use by OneView shouldn't be part of the validate, but specific to methods that need it. However, a similar validation is done within the validate of management. I opened this bug https://bugs.launchpad.net/ironic/+bug/1642266, and I wonder if it is correct to state that. | 14:08 |
| openstack | Launchpad bug 1642266 in Ironic "The OneView Driver validate method within the module management raises an exception if the node is not in use by Ironic." [Undecided,New] - Assigned to Stenio Araujo (stenioaraujo) | 14:08 |
| mrtenio | Thanks | 14:08 |
| patchbot | patch 372762 - ironic - Ensures that OneView nodes are free for use by Ironic | 14:08 |
| lucasagomes | mrtenio, vdrok TheJulia I find it unusual to have it on validate(), because validate() is used to validate the information passed to the node instead of whether the hardware is being used or not | 14:09 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add test for ironic port-list command https://review.openstack.org/329068 | 14:11 |
| mrtenio | lucasgomes, I agree with you. I was wondering if the same should be made with the validate within management. | 14:12 |
| *** baoli has joined #openstack-ironic | 14:13 | |
| *** Guest45665 is now known as rook | 14:13 | |
| *** Administrator_ has quit IRC | 14:13 | |
| *** Administrator_ has joined #openstack-ironic | 14:14 | |
| *** Guest32802 has joined #openstack-ironic | 14:14 | |
| *** jcoufal has joined #openstack-ironic | 14:14 | |
| *** baoli has quit IRC | 14:14 | |
| openstackgerrit | Moshe Levi proposed openstack/ironic-python-agent: Add a new Hardware Manager for Mellanox NICs https://review.openstack.org/334816 | 14:15 |
| *** baoli has joined #openstack-ironic | 14:15 | |
| *** lindycoder has joined #openstack-ironic | 14:16 | |
| *** Guest32802 is now known as Nisha | 14:16 | |
| TheJulia | lucasagomes: however, validate does provide a mechanism to fast fail prior to attempting to deploy a node as nova calls it before trying attempting to set provision state, although I think the same failure would occur. I think it is a YMMV thing per driver and it's use context, which only the that driver can determine the right path for them. | 14:17 |
| *** Nisha is now known as Guest12289 | 14:17 | |
| dtantsur | TheJulia, "fast" is key here :) as long as we don't do network calls in validate(), all is fine | 14:17 |
| lucasagomes | TheJulia, yeah I agree it's a faster mechanism (and in the review I said I don't know if there's a better place for this check). But still, validate() is sync from our API | 14:18 |
| lucasagomes | and all drivers so far have used it to validate the information in driver_info() | 14:18 |
| *** baoli_ has joined #openstack-ironic | 14:18 | |
| * dtantsur starts proposing newton releases | 14:18 | |
| TheJulia | Well, is it acceptable for the driver team, is the question in my mind. I'm in total agreement otherwise. :) | 14:18 |
| lucasagomes | so I think to have one driver that fails or succeed on validate() without changing any info is actually strange | 14:18 |
| lucasagomes | TheJulia, right yeah | 14:19 |
| TheJulia | lucasagomes: So, the sync nature of validate, I seem to remember there is a note in the ipmi driver, or was, that basically "we really can't actually go check this because of threads and sync nature of the call" | 14:19 |
| lucasagomes | TheJulia, yeah, we don't really want to have sync calls that talks to a BMC in the API (tho we have it for set boot device I think) | 14:20 |
| lucasagomes | but we removed it from the validate() in ipmitool awhile ago indeed | 14:20 |
| lucasagomes | now it's performed when the node go to manageable state (will check) | 14:20 |
| *** baoli has quit IRC | 14:21 | |
| lucasagomes | so it actually validates if the information is correct and not only present) | 14:21 |
| lucasagomes | https://review.openstack.org/#/c/99121/ | 14:21 |
| patchbot | patch 99121 - ironic - Stop ipmitool.validate from touching the BMC (MERGED) | 14:21 |
| * TheJulia wonders what the better way would be for the case where a bmc may or may not be ready to take a request | 14:22 | |
| * TheJulia thinks maintenance mode | 14:22 | |
| lucasagomes | TheJulia, yeah, it's a strange case with oneview because the provision state "available" should kinda indicate that it's available for being picked by the scheduler | 14:23 |
| lucasagomes | but in their cases that's not really true | 14:23 |
| TheJulia | yeah | 14:23 |
| lucasagomes | it's an odd scenario where you have basically two systems using the same pool of nodes | 14:23 |
| lucasagomes | s/kinda// | 14:23 |
| TheJulia | :) | 14:24 |
| openstackgerrit | Merged openstack/pyghmi: Consolidate IMM date string parsing https://review.openstack.org/397801 | 14:24 |
| lucasagomes | TheJulia, so idk, maybe validate() would be the best cost&benefit place to have it | 14:24 |
| lucasagomes | but I would like to have more eyes on that behaviour before agreeing with it, cause it's def an exception | 14:24 |
| dtantsur | validate is called way too often to afford long running stuff there... I think we can it before nearly every request in a sync fashion (thus blocking oslo.msg threads, etc) | 14:25 |
| dtantsur | I remember devananda highly against doing anything like that | 14:25 |
| TheJulia | That is how I feel and I think it works from a sanding of a one view user is going to have a very specific setup, and should (hopefully) have an understanding of it's behaviors | 14:26 |
| TheJulia | that being said, I know that team has been benchmarking their work and the numbers are actually kind of impressive against the oneview datacenter simulator. | 14:26 |
| mvpnitesh | @lucasagomes, sambetts : thanks alot for the help, I was able to provision a baremetal | 14:26 |
| lucasagomes | mvpnitesh, cool! Glad it worked! | 14:26 |
| TheJulia | That being said, the benchmarks I saw, if oneview's speed was degraded, naturally it translated through to ironic via the driver, but it was for all one view use | 14:28 |
| lucasagomes | TheJulia, cool... but yeah dtantsur, I don't know whether we are going to find a perfect place for such check in Ironic | 14:28 |
| *** ccamacho|lunch is now known as ccamacho | 14:28 | |
| xavierr | by verifying if hardware is in use by oneview in validate() avoid operator to provide some action that is invalid | 14:28 |
| xavierr | according to dy-allocation model | 14:28 |
| dtantsur | we probably don't.. I'm just saying that validate() may block the whole ironic essentially, if you end up consuming all RPC threads | 14:28 |
| lucasagomes | I wonder if we could have some layer that would do such checks, e.g, when the node is taken by the other system <something> puts the node in ironic as maintenance | 14:28 |
| lucasagomes | things like that | 14:29 |
| dtantsur | lucasagomes, a periodic task in the driver? | 14:29 |
| lucasagomes | dtantsur, perhaps yeah | 14:29 |
| openstackgerrit | Merged openstack/ironic-python-agent: Python 3 Compatible JSON https://review.openstack.org/363605 | 14:29 |
| xavierr | however, we can change this to add those validation in the methods the provide the action like 'set_boot_device' or 'set_power_state' | 14:30 |
| TheJulia | xavierr: But, then you double your API interactions I take it? | 14:31 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node maintenance notifications https://review.openstack.org/396239 | 14:31 |
| *** amotoki_ has joined #openstack-ironic | 14:31 | |
| xavierr | TheJulia: why double API interactions? :) | 14:32 |
| TheJulia | xavierr: if your checking oneview api for if the node is in use for every action, I would see that increasing the oneview api load. Maybe I'm waaaay off context there. But moving that would also mean provisioning could be started, go into a fail state because operations can't be performed | 14:34 |
| *** amotoki has quit IRC | 14:34 | |
| TheJulia | which, seems totally bad and wasteful too :) | 14:34 |
| *** causten has quit IRC | 14:34 | |
| *** causten has joined #openstack-ironic | 14:35 | |
| *** e0ne has joined #openstack-ironic | 14:36 | |
| xavierr | TheJulia: yes, that the point. block the operator of do some actions or try to provide them and fail?! :) | 14:38 |
| xavierr | lucasagomes idea is great. have a layer that check if hardware is in use by <thing> and keep that node reserved | 14:40 |
| xavierr | would be awesome for us | 14:40 |
| *** amotoki_ has quit IRC | 14:41 | |
| *** jaosorior has quit IRC | 14:41 | |
| *** jaosorior has joined #openstack-ironic | 14:41 | |
| *** shuquan has quit IRC | 14:42 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Docs: Document using operators with root device hints https://review.openstack.org/386714 | 14:42 |
| lucasagomes | rloo, ^ I just changed the commit message from Partial-Bug to Closes-Bug | 14:43 |
| lucasagomes | good catch | 14:43 |
| *** shuquan has joined #openstack-ironic | 14:44 | |
| TheJulia | xavierr: a locking reservation checking plugin interface? | 14:45 |
| *** mgoddard_ has joined #openstack-ironic | 14:46 | |
| *** rama_y has joined #openstack-ironic | 14:46 | |
| TheJulia | That seemed like an awesome idea when I said it, not so much now that I'm thinking about it | 14:46 |
| *** e0ne has quit IRC | 14:46 | |
| rloo | lucasagomes: i didn't really care about it but thx for updating it! | 14:47 |
| *** jaosorior has quit IRC | 14:47 | |
| *** chas has quit IRC | 14:47 | |
| rloo | hi and good morning everyone, lucasagomes, TheJulia, xavierr, dtantsur | 14:47 |
| dtantsur | morning rloo | 14:47 |
| lucasagomes | rloo, yeah I saw you +2'd it, it would take me 2s to update the commit message (via gerrit) that I just did it | 14:47 |
| lucasagomes | ty | 14:47 |
| lucasagomes | rloo, morning | 14:47 |
| *** chas has joined #openstack-ironic | 14:47 | |
| xavierr | morning rloo | 14:47 |
| * jroll is back | 14:48 | |
| xavierr | TheJulia: I think so, it would take care of check if node is from to be provisioned for us. what is your feeling about that? | 14:49 |
| *** mgoddard has quit IRC | 14:49 | |
| rloo | jroll: were you going to post a summary of the summit? | 14:49 |
| jroll | rloo: I've been meaning to, but haven't written up a thing | 14:49 |
| * jroll feels pretty bad about that | 14:49 | |
| xavierr | s/is from/is free/ | 14:49 |
| rloo | jroll: I'm surprised JayF didn't hound you for it | 14:49 |
| *** vmud213 has joined #openstack-ironic | 14:49 | |
| *** hw_wutianwei has quit IRC | 14:50 | |
| jroll | rloo: well, I talk to jay often | 14:50 |
| TheJulia | good morning rloo | 14:50 |
| rloo | jroll: oh, so JayF can write it for you? :) | 14:50 |
| jroll | heh | 14:51 |
| jroll | rloo: not sure if it's worth writing one at this point | 14:51 |
| *** rbrud has joined #openstack-ironic | 14:51 | |
| jroll | or, well, if I remember things well enough for it to be useful | 14:51 |
| TheJulia | xavierr: Not entirely sure, my brain is spinning on how it would work :) I would hope that there is a index on the column we would hit in oneview. | 14:52 |
| TheJulia | xavierr: this is one of these moments I miss weekly calls with you guys | 14:52 |
| rloo | jroll: dunno. i know what happened. but not everyone was there, so it seems like a good thing to do | 14:52 |
| *** sabeen has joined #openstack-ironic | 14:52 | |
| jroll | rloo: yeah, just unclear to me if there's things worth writing that aren't already encoded in docs or being worked on or whatever | 14:52 |
| *** hw_wutianwei has joined #openstack-ironic | 14:53 | |
| rloo | jroll: too bad mat128 wasn't there. he did a great summary of the last mid-cycle. | 14:53 |
| xavierr | TheJulia: hahaha we miss totally those calls with you! | 14:54 |
| mat128 | rloo: heh thanks | 14:54 |
| jroll | +1 | 14:54 |
| *** mvpnitesh has quit IRC | 14:58 | |
| dtantsur | jroll, lined up these releases: https://review.openstack.org/#/q/topic:ironic-newton have I missed something? bifrost, ironic-ui? | 14:58 |
| *** mgoddard has joined #openstack-ironic | 14:58 | |
| *** mgoddard_ has quit IRC | 14:59 | |
| * jroll looks | 14:59 | |
| *** jcoufal has quit IRC | 14:59 | |
| dtantsur | vbmc seems branchless | 14:59 |
| jroll | dunno if bifrost is worth a release: https://review.openstack.org/#/q/project:openstack/bifrost+branch:stable/newton | 15:00 |
| jroll | maybe ironic-ui because translations https://review.openstack.org/#/q/project:openstack/ironic-ui+branch:stable/newton | 15:00 |
| *** priteau has quit IRC | 15:00 | |
| dtantsur | jroll, I did request a release for ironic-inspector-client which had similar changes to bifrost | 15:00 |
| dtantsur | ironic-ui ack | 15:00 |
| jroll | dtantsur: ah, maybe then | 15:01 |
| *** e0ne has joined #openstack-ironic | 15:01 | |
| *** jcoufal has joined #openstack-ironic | 15:01 | |
| dtantsur | k, doing both | 15:01 |
| dtantsur | TheJulia, any objections to technical bifrost release from newton? | 15:03 |
| TheJulia | dtantsur: no objection, although don't see much value in one at the moment :) | 15:03 |
| dtantsur | well, we have commits, we should let them out :) | 15:03 |
| TheJulia | indeed | 15:03 |
| xavierr | dtantsur: will you release a new ironic stable/newton? | 15:05 |
| dtantsur | xavierr, all our projects, all of them :) | 15:05 |
| xavierr | dtantsur: ok, tks :) | 15:05 |
| dtantsur | all releases requested: https://review.openstack.org/#/q/topic:ironic-newton | 15:06 |
| vdrok | morning rloo | 15:08 |
| vdrok | rloo: so you want to land portgroup api-ref? found more issues there | 15:08 |
| rloo | vdrok: hi. did i already land it? | 15:09 |
| vdrok | rloo: nope :) | 15:10 |
| rloo | vdrok: oh. i think i meant to and forgot. | 15:10 |
| rloo | vdrok: how many more issues? | 15:10 |
| rloo | vdrok: it is a pain to look through all those files | 15:10 |
| vdrok | rloo: 9 more, these are about parameters | 15:11 |
| rloo | vdrok: oh. so mostly changes in one file? | 15:11 |
| vdrok | rloo: in 4 files :( | 15:11 |
| rloo | vdrok: are you ok if i land it and you submit a followon patch? | 15:11 |
| vdrok | rloo: ok, lemme just post them then | 15:12 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add test for ironic port-list command https://review.openstack.org/329068 | 15:13 |
| *** priteau has joined #openstack-ironic | 15:13 | |
| rloo | thx vdrok | 15:14 |
| *** hw_wutianwei has quit IRC | 15:17 | |
| openstackgerrit | Merged openstack/ironic: Add portgroup to api-ref https://review.openstack.org/393834 | 15:19 |
| hogepodge | jroll: I got more information from them. They do the snapshots with an agent in their VMs, which sadly makes it a not universally applicable solution. | 15:21 |
| jroll | hogepodge: ah. we kind of all have the idea in our head of how it would work more generally (offline snapshots, boot into ramdisk and do the thing) | 15:22 |
| JayF | jroll: it's worth doing one | 15:22 |
| JayF | rloo: thanks, now i have a reason to hound jroll | 15:22 |
| * JayF howls in jroll's direction | 15:23 | |
| jroll | what the what | 15:23 |
| mat128 | JayF: isnt that a bit too far from him to hear? :) | 15:23 |
| rloo | thx JayF! :D | 15:23 |
| *** Goneri has quit IRC | 15:23 | |
| *** baoli_ has quit IRC | 15:24 | |
| *** absubram has joined #openstack-ironic | 15:25 | |
| jroll | JayF: I guess my question is, at this point (where lots of those decisions and such are documented), is it the best use of my time | 15:25 |
| JayF | jroll: I think that makes it easier to do; you can talk abuot what was discussed and link the specs where it was decided | 15:25 |
| dtantsur | zigo, hi, around? | 15:26 |
| JayF | jroll: at a minimum annnuoncing that we've nailed down priorities + a link would have some value | 15:26 |
| zigo | dtantsur: Yup, what's up ? | 15:26 |
| dtantsur | zigo, do you run unit tests on building the ironic package? I've just tried it in RDO and got 14 failures: https://review.rdoproject.org/jenkins/job/DLRN-rpmbuild/5076/console | 15:26 |
| jroll | JayF: mmmm, okay. | 15:26 |
| zigo | dtantsur: I do. | 15:26 |
| dtantsur | zigo, do you have to disable/workaround any tests? | 15:27 |
| zigo | dtantsur: Is your funcsigs up-to-date ? | 15:27 |
| zigo | dtantsur: I haven't worked on OpenStack Debian packages since Newton, so if that's Ocata, maybe there's new failures. | 15:27 |
| dtantsur | good question :) which version do you have? | 15:28 |
| dtantsur | yes, this is Ocata | 15:28 |
| zigo | I got 1.0.2. | 15:28 |
| zigo | u? | 15:28 |
| dtantsur | python2-funcsigs.noarch 0:0.4-2.el7 | 15:28 |
| dtantsur | looks kinda old | 15:28 |
| zigo | Right. | 15:28 |
| zigo | This looks like the Mitaka reqs. | 15:29 |
| *** priteau has quit IRC | 15:29 | |
| dtantsur | well, it's in Ocata too... https://github.com/openstack/requirements/blob/master/global-requirements.txt#L54 | 15:29 |
| *** rajinir has joined #openstack-ironic | 15:30 | |
| *** rpioso has joined #openstack-ironic | 15:30 | |
| dtantsur | anyway, thanks zigo, I'll check it | 15:31 |
| *** syed__ has joined #openstack-ironic | 15:31 | |
| dtantsur | yeah, I can reproduce it locally, if I install funcsigs 0.4. fun. | 15:32 |
| zigo | dtantsur: Yet-another-we-don't-test-lower-bounds pb... :P | 15:32 |
| zigo | I'd fill a bug against Ironic to make it bump. | 15:32 |
| dtantsur | well, it should go to global-requirements and potentially even affects stable branches... | 15:32 |
| mat128 | zigo, dtantsur: maybe we should also have a lower-constraints ? | 15:33 |
| zigo | mat128: It's currently 0.4 which doesn't work with today's ironic. | 15:33 |
| dtantsur | global-requirements is lower constrain essentially | 15:33 |
| mat128 | dtantsur: but is there anything running (tox/gates/etc) with the lowest versions of everything? | 15:34 |
| *** priteau has joined #openstack-ironic | 15:34 | |
| *** vmud213 has quit IRC | 15:34 | |
| *** d[^_^]b has quit IRC | 15:35 | |
| *** Goneri has joined #openstack-ironic | 15:35 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add portgroup configuration fields https://review.openstack.org/398417 | 15:35 |
| dtantsur | mat128, nope, that's what zigo is talking about | 15:35 |
| mat128 | dtantsur: thats why I was suggesting generating a lower-constraints file and having gates running with it :) | 15:36 |
| mat128 | we're all on the same page, using different words | 15:36 |
| * mat128 apologies | 15:36 | |
| dtantsur | k, I agree | 15:36 |
| *** vmud213 has joined #openstack-ironic | 15:43 | |
| *** shuquan has quit IRC | 15:45 | |
| *** rbartal has joined #openstack-ironic | 15:45 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled https://review.openstack.org/393745 | 15:46 |
| JayF | the first two patches in the remaining notifications line: https://review.openstack.org/#/c/348437/ both have my +2 and a +1 from mario, it would be a good thing to land if folks are looking for something to review | 15:47 |
| patchbot | patch 348437 - ironic - Add node provision state change notification | 15:47 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled https://review.openstack.org/393745 | 15:48 |
| rloo | JayF: it was on my list to look at notifications today. I'm mostly done with portgroups :) | 15:51 |
| * dtantsur could use a bit more reviews on his next driver comp patch https://review.openstack.org/396681 | 15:52 | |
| patchbot | patch 396681 - ironic - Add node (database and objects) fields for all int... | 15:52 |
| JayF | i'll add it to my queue | 15:53 |
| *** sukhdev has joined #openstack-ironic | 15:53 | |
| rloo | dtantsur: oh, i forgot about that. was going to do it yesterday until you made more changes | 15:53 |
| *** fxpester has quit IRC | 15:53 | |
| dtantsur | yeah, I realized that this change is pretty useless without objects change, so I merged them | 15:54 |
| *** Sukhdev_ has joined #openstack-ironic | 15:58 | |
| *** ptx0 has joined #openstack-ironic | 16:00 | |
| jroll | rloo: JayF: thanks for lighting the fire under me, sent the thing | 16:02 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic-specs: API Evolution - ETAG identifiers https://review.openstack.org/381991 | 16:02 |
| *** Guest12289 has quit IRC | 16:03 | |
| rloo | jroll: looks good. Thanks for posting it! So that's what we did on the beach... :) | 16:05 |
| *** mjura has quit IRC | 16:06 | |
| jroll | heh | 16:06 |
| jroll | I left out the beach conversations :P | 16:06 |
| *** dougs2 has joined #openstack-ironic | 16:06 | |
| *** dougs1 has quit IRC | 16:08 | |
| openstackgerrit | Merged openstack/ironic: Docs: Document using operators with root device hints https://review.openstack.org/386714 | 16:09 |
| jroll | ^^ woot | 16:10 |
| rloo | yay! jroll, you'll do the admin-stuff related to root device hints? :) | 16:10 |
| *** clenimar has joined #openstack-ironic | 16:11 | |
| jroll | rloo: shore thing | 16:11 |
| JayF | hah! | 16:12 |
| rloo | :D as far as I know, jroll was the only one that didn't stay on shore the whole time. | 16:13 |
| jroll | that water was amazing | 16:13 |
| rloo | JayF: it was horrible, you didn't miss anything! :) | 16:14 |
| JayF | lol | 16:14 |
| JayF | I certainly didn't miss the long, cramped flight | 16:14 |
| JayF | I travelled a *lot* in 2015, I'm honestly not upset about sticking mostly around the house this year. | 16:14 |
| JayF | at least from a personal do-i-want-to-go standpoint ;) | 16:15 |
| rloo | JayF: i hear you. I don't like travelling for work myself. | 16:15 |
| rloo | JayF: the sacrifices I make for ironic... | 16:15 |
| JayF | I am looking forward to the ptg, and should know in a week or two if I'm going to get to go | 16:16 |
| JayF | my brother + his family are in Atlanta, and I won't see them at Christmastime this year b/c my SIL is having their third kid around christmas | 16:16 |
| *** causten_ has joined #openstack-ironic | 16:16 | |
| JayF | so that'd be nice, to get to meet the new kiddo while working on ironic :) | 16:16 |
| rloo | JayF: anyone you want me to bug, to give you permission? :) | 16:17 |
| JayF | rloo: of all the things I do, I bug people the best, so I don't think I need any help there ;) | 16:17 |
| rloo | JayF: so very true! | 16:18 |
| *** causten has quit IRC | 16:20 | |
| *** dfisher has joined #openstack-ironic | 16:22 | |
| dfisher | what's the difference between the 'manageable' and 'available' provisioning state in Ironic (Mitaka)? | 16:23 |
| NobodyCam | Good Morning Ironic'ers | 16:23 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled https://review.openstack.org/393745 | 16:24 |
| *** dougs1 has joined #openstack-ironic | 16:25 | |
| rloo | morning NobodyCam | 16:25 |
| jroll | \o NobodyCam | 16:25 |
| rloo | dfisher: 'available' means the node is available for provisioning/deploying | 16:25 |
| NobodyCam | Good Morning rloo & jroll :) | 16:25 |
| rloo | dfisher: 'manageable' means it can be 'managed'. which means what you ask. sec, let me get the diagram | 16:25 |
| dfisher | rloo: thank you! | 16:25 |
| rloo | dfisher: http://docs.openstack.org/developer/ironic/dev/states.html | 16:26 |
| rloo | dfisher: does that help? | 16:26 |
| *** moshele has quit IRC | 16:26 | |
| dfisher | let me look at the diagram | 16:26 |
| dfisher | and i'll get back to you :) | 16:26 |
| *** roeyc1 has quit IRC | 16:26 | |
| *** ricardoas has joined #openstack-ironic | 16:27 | |
| JayF | dfisher: I sorta think of managable as a "parking" state. The machine won't be provisioned to, but is still managed by ironic. It's the state you initiate things like manual cleaning from. | 16:27 |
| dfisher | but I can set the provision-state to active on a manageable node … right? | 16:28 |
| *** dougs2 has quit IRC | 16:28 | |
| JayF | yes; that will take the node from managable, through automated cleaning if availabe, and set ti to active | 16:28 |
| JayF | but the api verb is actually 'provide', not active | 16:28 |
| jroll | s/active/available | 16:28 |
| JayF | ironic node-set-provision-state my-node provide | 16:28 |
| JayF | jroll: dfisher ^ what jroll said. managable -> cleaning -> available | 16:28 |
| jroll | and available -> deploying -> active | 16:29 |
| jroll | can't go manageable to active | 16:29 |
| *** rama_y has quit IRC | 16:29 | |
| dfisher | nono. i meant the CLI invocation | 16:29 |
| dfisher | literally 'ironic node-set-provision-state mynode active' | 16:29 |
| dfisher | takes it from manageable -> deploying -> active | 16:29 |
| dfisher | right? | 16:29 |
| JayF | dfisher: no | 16:29 |
| dfisher | erm. | 16:29 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled https://review.openstack.org/393745 | 16:29 |
| JayF | dfisher: you cannot make that transition. you have to be in available | 16:29 |
| *** krtaylor has quit IRC | 16:29 | |
| JayF | dfisher: http://docs.openstack.org/developer/ironic/_images/states.svg maps it out | 16:30 |
| dfisher | right. rloo linked that one | 16:30 |
| JayF | only paths to active are managable -> adopting -> active (if using our support for adopting predeployed instances) | 16:30 |
| openstackgerrit | Charlle Daniel proposed openstack/ironic: Ensures that OneView nodes are free for use by Ironic https://review.openstack.org/372762 | 16:30 |
| JayF | and mangable -> available; then available -> deploying -> active | 16:30 |
| JayF | although deploying a node manually through the cli requries more than just changing the provision state | 16:31 |
| dfisher | sure. | 16:31 |
| vdrok | morning NobodyCam | 16:31 |
| dfisher | JayF/rloo/jroll: thanks for the clarity | 16:32 |
| JayF | dfisher: no problem. curious what you're trying to do though :) | 16:32 |
| NobodyCam | Morning vdork :) | 16:32 |
| dfisher | just clarify it in my head | 16:33 |
| dfisher | and i'm the oddball doing Ironic on Solaris :) | 16:33 |
| JayF | using it to deploy solaris, or trying to run the control plane on solaris? | 16:34 |
| JayF | I'd actually imagine either one isn't that difficult... | 16:34 |
| dfisher | yes to both. | 16:34 |
| mgould | Russian speakers! I have a favour to ask. | 16:34 |
| mgould | Is | 16:34 |
| mgould | dammit | 16:35 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Use stable/newton ironic-inspector for functional tests https://review.openstack.org/398474 | 16:35 |
| dfisher | my coworker submitted a spec months ago for our driver but it was declined | 16:35 |
| lucasagomes | dfisher, that said there's a graph in our docs with the possible transitions | 16:35 |
| lucasagomes | dfisher, http://docs.openstack.org/developer/ironic/dev/states.html | 16:35 |
| JayF | dfisher: I bet you could deploy whole-disk solaris images from a solaris control plane with the agent driver with no further changes to ironic | 16:35 |
| clif_h | alineb: https://github.com/ClifHouck/boot-devstack/ | 16:35 |
| JayF | dfisher: but it's always possible I'm wrong ;) | 16:35 |
| mgould | is https://docs.google.com/document/d/1IMU8b9GK6wsIzi_lTEoP5oLvN9mZkBhFwIE67F9fvlo/edit?usp=sharing a reasonable translation of https://docs.google.com/document/d/1EGWh4KMKtJkEfzavM7YqY6gsbjWeJygnaa1ATzG9SwY/edit?usp=sharing ? | 16:35 |
| dfisher | oh, we're using our driver. | 16:35 |
| mgould | [I wrote the English version, but not the Russian one] | 16:35 |
| lucasagomes | JayF, I think solaris has it's own mechanism for deploying | 16:36 |
| alineb | clif_h: thanks | 16:36 |
| lucasagomes | JayF, kinda like xclarify for lenovo (tho lenovo would be possible to just use a normal ironic driver) | 16:36 |
| dfisher | lucasgomes: we do. | 16:36 |
| mgould | ^^^ dtantsur snozhka aarefiev vdrok vsaienk0 galyna pas-ha | 16:36 |
| JayF | oh, so like the equivalent of like, a solaris kickstart | 16:36 |
| JayF | instead of deploying an image | 16:36 |
| dfisher | well | 16:36 |
| lucasagomes | JayF, yeah, something like that | 16:36 |
| dfisher | sort of both, really | 16:36 |
| patchbot | both, of really | 16:36 |
| dfisher | we use Solaris Automated Install (AI) to install a disk image | 16:36 |
| jroll | patchbot always cracks me up. | 16:36 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow bifrost to work with selinux enabled https://review.openstack.org/393745 | 16:36 |
| dfisher | but, AI is jumpstart v2.0 for Solaris 11+ | 16:37 |
| mat128 | dfisher: is it something like anaconda/kickstart? | 16:38 |
| dfisher | yes | 16:38 |
| sambetts | so Ironic doesn't do kickstart style installs | 16:38 |
| mat128 | we tend to favour image deployment, but everythign is possible | 16:38 |
| sambetts | it does image based deployment, similar to VMs | 16:38 |
| dtantsur | aarefiev, sambetts, milan, up for quick review of https://review.openstack.org/#/c/398474/? I realized I missed it from the release.. | 16:38 |
| patchbot | patch 398474 - python-ironic-inspector-client (stable/newton) - Use stable/newton ironic-inspector for functional ... | 16:38 |
| mat128 | your image could be the kickstart install that reboots to disk once done :) | 16:38 |
| pas-ha | mgould: just couple of small nits re declinations, if you'd allow comments I'll propose fixes | 16:39 |
| dfisher | sambetts: we can do image based OR package based. | 16:39 |
| lucasagomes | [off-topic] kickstart does image install as well (and tarball fwiw) | 16:39 |
| sambetts | dfisher: but I assume your replacing the ironic ramdisk at that point? | 16:39 |
| dfisher | yes | 16:39 |
| dfisher | the images we use have bootable media in them. | 16:39 |
| vdrok | mgould: heh, will review in a bit :) | 16:39 |
| vdrok | first page looks mostly OK | 16:39 |
| dfisher | *or* the AI server provides the ramdisk over the wire | 16:39 |
| milan | dtantsur, ack | 16:40 |
| mgould | vdrok: thanks! | 16:40 |
| dtantsur | mgould, the translation looks reasonable enough, although I was surprised by word "Первовосхождение" | 16:40 |
| *** rbartal has quit IRC | 16:41 | |
| mgould | /me googles | 16:41 |
| mgould | "first ascent"? | 16:41 |
| mgould | link with comments enabled: https://docs.google.com/document/d/1IMU8b9GK6wsIzi_lTEoP5oLvN9mZkBhFwIE67F9fvlo/edit?usp=sharing | 16:42 |
| *** krtaylor has joined #openstack-ironic | 16:43 | |
| mgould | Фирж should probably be Фэргий | 16:43 |
| dtantsur | mgould, it's not wrong, I've just never seen it as one word, as opposed to two regular words "первое восхождение" | 16:44 |
| * mgould nods | 16:44 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Update the alembic migration section in the developer FAQ https://review.openstack.org/398486 | 16:44 |
| * vdrok agrees with dtantsur | 16:45 | |
| *** e0ne has quit IRC | 16:45 | |
| *** strigazi is now known as strigazi_AFK | 16:46 | |
| rloo | dtantsur: reviewed https://review.openstack.org/#/c/396681. do we have to change the notifications payload? | 16:47 |
| patchbot | patch 396681 - ironic - Add node (database and objects) fields for all int... | 16:47 |
| dtantsur | I'm thinking about that right now :) | 16:47 |
| * jroll also reviewed it | 16:47 | |
| jroll | we probably don't need to change notifications until it's in the API | 16:47 |
| jroll | as it won't change right now | 16:48 |
| vdrok | mgould: also I'm not sure in the term "ребро", "хребет" sounds more familiar to me -- ridge | 16:48 |
| dtantsur | rloo, I really feel like we should merge rescue asap... | 16:48 |
| jroll | though, doesn't hurt to do it now, right? | 16:48 |
| dtantsur | jroll, rloo, if we think it should not be there, I'd rather depend on rescue work.. how close is it? | 16:48 |
| rloo | jroll: notifications aren't in the API. hmm, i've forgotten how the versioning of that is done. | 16:48 |
| *** jistr is now known as jistr|biab | 16:48 | |
| rloo | dtantsur: i think i saw one patch for rescue, but have no idea if there is only one patch and what the status is. JayF, do you know the status of rescue work? | 16:49 |
| jroll | rloo: well, I mean, the field can't possibly change values until it's in the API, so we don't need to notify about changes, right? | 16:49 |
| jroll | or should we send it out anyway | 16:49 |
| jroll | there's one patch up for rescue afaik, maybe two | 16:49 |
| JayF | I would not wait on rescue, honestly | 16:50 |
| rloo | jroll: it will be in the payload that the notification sends | 16:50 |
| JayF | the api and conductor methods are up, the driver side implementation exists in some downstream-rackspace form, but needs a lot of work | 16:50 |
| jroll | rloo: yeah, but if it can't change, does it need to be in the payload? is it useful? | 16:50 |
| rloo | jroll: i can't remember though how the notification knows which version to use | 16:50 |
| mgould | vdrok: that may just be jargon; English-speaking climbers sometimes use "rib" to mean "ridge" - I'll ask ITMC :-) | 16:50 |
| rloo | jroll: well, that's my quesiton. lets not put it in. | 16:50 |
| jroll | rloo: I'm not very up to speed on how notifications work | 16:50 |
| *** kvcobb has quit IRC | 16:51 | |
| rloo | jroll: everything in the payload gets put in the notification that is emitted. it isn't a delta/change. | 16:51 |
| jroll | no, I understand that | 16:51 |
| jroll | is it useful to consumers if it's always null though | 16:51 |
| jroll | is my question | 16:51 |
| dtantsur | jroll, rloo, next question: should I exclude rescue from my work completely for now? | 16:51 |
| rloo | jroll: it seems confusing to me. we haven't yet modified the API so that if you do a node-show, you'll see those interface fields. yet, if you are using notifications, you'll see those fields being null. | 16:52 |
| jroll | rloo: right | 16:52 |
| rloo | dtantsur: i would prefer if you excluded rescue. i didn't know if you could cuz it was a variable in the base driver/whatever. | 16:52 |
| jroll | dtantsur: hm, I don't see a harm in including it in the plumbing, but I could go either way | 16:52 |
| *** absubram has quit IRC | 16:53 | |
| rloo | jroll: i hope that makes sense, just reading it. i mean, you won't see the new interfaces in a node-show, but you will in a notification. which seems wrong. | 16:53 |
| dtantsur | jroll, well, then we need to draw a line somewhere | 16:53 |
| mgould | dtantsur, pas-ha, vdrok, snozhka: thank you very much! | 16:53 |
| rloo | dtantsur: if it is easier, just don't add rescue at all now | 16:53 |
| jroll | dtantsur: the API? idk. we could leave it out altogether. | 16:53 |
| mgould | and thanks everyone else for your patience :-) | 16:53 |
| jroll | dtantsur: the interface exists, though, people could have out of tree drivers that have rescue right now, right? | 16:53 |
| dtantsur | jroll, they can if they have a full downstream implementation.. | 16:54 |
| jroll | well, they could without implementing everything, it just wouldn't do anything :P | 16:54 |
| jroll | but yeah | 16:54 |
| dtantsur | I'm fine whatever route we take | 16:55 |
| jroll | again, I could go either way here | 16:55 |
| rloo | if we want to support some hypothetical third party driver, then i think we should have rescue upto the node object, but not past that. | 16:56 |
| JayF | It's worth noting that I think the only reason RescueInterface is upstream right now was to make rebasing downstream patches easier for OnMetal ... | 16:56 |
| dtantsur | adding it to objects, but not adding it to notifications about its changes also looks a bit strange to me.. | 16:56 |
| JayF | like there was nobody clamoring for a rescue interface for an out of tree driver is all I'm saying | 16:56 |
| jroll | jlvill-travel: krtaylor: sambetts: are we doing qa meeting today? | 16:56 |
| jroll | JayF: no, RescueInterface existed when we started working on ironic | 16:56 |
| rloo | my suggestion is NOT to add rescue for now. let's see how the rest of the plumbing goes. we can always add it in a later patch in this series. | 16:56 |
| dtantsur | rloo, not add at all? even to the database? /me is fine with it, just clarifying | 16:57 |
| rloo | dtantsur: yeah,even to the db. that makes it cleaner i think. | 16:57 |
| JayF | jroll: ...are you sure? | 16:57 |
| jroll | JayF: 98% | 16:57 |
| dtantsur | rloo, fine, please leave the comment on the patch | 16:57 |
| JayF | I'm going to check git, because that sounds wrongism | 16:57 |
| JayF | *wrongish | 16:57 |
| dtantsur | I'm leaving for a day, and tomorrow is a public holiday | 16:58 |
| dtantsur | so see you on Friday | 16:58 |
| rloo | dtantsur: ok. enjoy your holiday! (see you on Monday) | 16:58 |
| jroll | JayF: is in icehouse https://github.com/openstack/ironic/blob/icehouse-eol/ironic/drivers/base.py#L269 | 16:58 |
| jroll | dtantsur: enjoy :) | 16:58 |
| *** dtantsur is now known as dtantsur|afk | 16:58 | |
| mgould | dtantsur|afk: o/ | 16:59 |
| *** vmud213 has quit IRC | 16:59 | |
| jroll | JayF: at least as early as june 2013 https://github.com/openstack/ironic/commit/59d5bea14ac752e4d3e70f17f926584dc1ab9257 | 16:59 |
| jroll | which is when I joined rackspace | 16:59 |
| jroll | er, before I joined | 16:59 |
| *** vmud213 has joined #openstack-ironic | 16:59 | |
| jroll | (dec 2013) | 16:59 |
| JayF | what the hell am I remembering then!? | 16:59 |
| JayF | maybe we -1'd the patch to remove it, and instead it was just pulled from the api? | 16:59 |
| *** dfisher has left #openstack-ironic | 17:00 | |
| jroll | JayF: idk, but you aren't remembering all the painful rebases I had to do :D | 17:00 |
| JayF | lol | 17:00 |
| *** absubram has joined #openstack-ironic | 17:00 | |
| krtaylor | jroll, yes, ironic-qa meeting started | 17:02 |
| jroll | d'oh | 17:02 |
| *** rbrud has quit IRC | 17:02 | |
| *** aNuposic has joined #openstack-ironic | 17:04 | |
| *** Sukhdev_ has quit IRC | 17:05 | |
| *** dougs1 has quit IRC | 17:10 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: PoC Manage VNC console on iDRAC via WS-MAN API https://review.openstack.org/396661 | 17:14 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node console notifications https://review.openstack.org/397812 | 17:21 |
| *** vmud213 has quit IRC | 17:22 | |
| * jroll -> lunch | 17:23 | |
| *** chas has quit IRC | 17:24 | |
| *** chas has joined #openstack-ironic | 17:24 | |
| mariojv | where is console info stored on a node? | 17:28 |
| mariojv | driver_info maybe? | 17:29 |
| *** chas has quit IRC | 17:29 | |
| mariojv | i know that there's a top-level console_enabled property but am not sure where anything else is | 17:29 |
| *** baoli has joined #openstack-ironic | 17:30 | |
| *** lennyb has quit IRC | 17:35 | |
| *** jistr|biab is now known as jistr | 17:36 | |
| *** absubram has quit IRC | 17:37 | |
| *** priteau has quit IRC | 17:38 | |
| *** causten_ has quit IRC | 17:38 | |
| rloo | mariojv: should be some stuff in driver_info | 17:38 |
| *** causten has joined #openstack-ironic | 17:38 | |
| rloo | mariojv: don't know if there is anything documented here: http://docs.openstack.org/developer/ironic/deploy/console.html | 17:38 |
| *** priteau has joined #openstack-ironic | 17:39 | |
| *** aNuposic has quit IRC | 17:40 | |
| *** fragatina has joined #openstack-ironic | 17:41 | |
| mat128 | mariojv: iirc some of the stuff is stored in driver_internal_info | 17:41 |
| mariojv | thanks | 17:41 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Add API for listing all introspection statuses https://review.openstack.org/344921 | 17:41 |
| milan | aarefiev, there were just two options w/ the decorator: ugly and uglier :D ^ | 17:42 |
| *** causten has quit IRC | 17:43 | |
| *** rama_y has joined #openstack-ironic | 17:43 | |
| lucasagomes | calling it a day, have a great evening all :-) | 17:47 |
| *** lennyb has joined #openstack-ironic | 17:47 | |
| *** lucasagomes is now known as lucas-afk | 17:47 | |
| *** derekh has quit IRC | 17:47 | |
| NobodyCam | night lucas-afk | 17:47 |
| lucas-afk | NobodyCam, morning Chris! | 17:47 |
| rloo | bye lucas-afk | 17:47 |
| NobodyCam | :) | 17:47 |
| *** david-lyle has quit IRC | 17:48 | |
| *** david-lyle has joined #openstack-ironic | 17:48 | |
| *** rpioso is now known as rpioso-afk | 17:49 | |
| *** trown is now known as trown|lunch | 17:52 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Followup to 0335e81a8787 https://review.openstack.org/398515 | 17:54 |
| *** priteau has quit IRC | 17:56 | |
| vdrok | good night everyone! | 17:57 |
| mgould | vdrok: good night! | 17:58 |
| NobodyCam | night vdrok | 17:58 |
| *** ralonsoh has quit IRC | 17:58 | |
| nicodemos | night, vdrok | 17:58 |
| *** milan has quit IRC | 17:58 | |
| *** ohamada has quit IRC | 18:00 | |
| *** fragatin_ has joined #openstack-ironic | 18:04 | |
| *** makowals_ has joined #openstack-ironic | 18:05 | |
| *** moshele has joined #openstack-ironic | 18:05 | |
| mrtenio | Good night vdrok | 18:07 |
| *** makowals_ has quit IRC | 18:07 | |
| *** makowals_ has joined #openstack-ironic | 18:08 | |
| *** fragatina has quit IRC | 18:08 | |
| *** sukhdev has quit IRC | 18:11 | |
| *** mgould is now known as mgould|afk | 18:12 | |
| *** e0ne has joined #openstack-ironic | 18:12 | |
| *** priteau has joined #openstack-ironic | 18:13 | |
| *** absubram has joined #openstack-ironic | 18:14 | |
| *** harlowja has joined #openstack-ironic | 18:14 | |
| *** rbrud has joined #openstack-ironic | 18:16 | |
| moshele | JayF:hi, I fixed you comments https://review.openstack.org/#/c/385693/ can you review again | 18:17 |
| patchbot | patch 385693 - nova - placement: update client to set parent provider | 18:17 |
| JayF | moshele: wrong jay :) | 18:18 |
| JayF | moshele: you're looking for jaypipes | 18:18 |
| sambetts | Night all | 18:18 |
| sambetts | o/ | 18:18 |
| *** sambetts is now known as sambetts|afk | 18:18 | |
| JayF | moshele: actually .. sure you llinked the right patch? | 18:18 |
| moshele | JayF: that true, but I wanted you to review https://review.openstack.org/#/c/385693/ | 18:18 |
| patchbot | patch 385693 - nova - placement: update client to set parent provider | 18:18 |
| JayF | I mean, I can, but I'm not really a nova expert :) | 18:18 |
| *** vsaienko has joined #openstack-ironic | 18:19 | |
| moshele | sorry copy paste mistake I meant this https://review.openstack.org/#/c/334816/ | 18:19 |
| patchbot | patch 334816 - ironic-python-agent - Add a new Hardware Manager for Mellanox NICs | 18:19 |
| moshele | I do need jaypipes for the other one :) | 18:20 |
| JayF | moshele: straight up perfect example of a minimal hardware manager addition :) good work, +2 | 18:21 |
| moshele | JayF: thanks, In the future we will extend it to other things :) | 18:22 |
| JayF | moshele: hmm. one thing, gotta see why that inspector job is failing | 18:22 |
| JayF | aha! I finally saw it happen in a gate | 18:23 |
| JayF | http://logs.openstack.org/16/334816/18/check/gate-tempest-dsvm-ironic-inspector-src-nv/e7b40f4/logs/devstacklog.txt.gz#_2016-11-16_14_38_04_590 | 18:24 |
| JayF | ironic-inspector plugin loaded before ironic did | 18:24 |
| JayF | aslezil: ^ the thing that happened in your devstack happened in the gate | 18:24 |
| JayF | nope, lib/iroinc ran first | 18:26 |
| JayF | but it also ran cleanup? | 18:26 |
| JayF | wtf | 18:26 |
| JayF | http://logs.openstack.org/16/334816/18/check/gate-tempest-dsvm-ironic-inspector-src-nv/e7b40f4/logs/devstacklog.txt.gz#_2016-11-16_14_30_40_276 | 18:27 |
| JayF | why is that even running? That reemoves the ironic.conf before inspector can modify it | 18:27 |
| JayF | jroll: ^ nerd snipe | 18:29 |
| JayF | looking at https://github.com/openstack/ironic/blob/master/devstack/plugin.sh | 18:30 |
| JayF | it looks like the `stack install` phase of lib/ironic occurred | 18:30 |
| JayF | but then ironic-inspector ran out of order afterwards? or somethign like that | 18:30 |
| *** Administrator_ has quit IRC | 18:32 | |
| *** Administrator_ has joined #openstack-ironic | 18:32 | |
| harlowja | rloo jroll https://review.openstack.org/#/c/397264/ fyi | 18:33 |
| patchbot | patch 397264 - tooz - Add a hashring implementation | 18:33 |
| JayF | So after gettting all the way down the rabbithole, ironic-inspector plugin.sh stack post-config is called | 18:33 |
| JayF | before ironic plugin.sh stack post-config | 18:33 |
| mariojv | does anyone have an example command for ironic/devstack/tools/ironic/scripts/create-node.sh? | 18:33 |
| JayF | so dependencies on ironic are not being handled, and that's how the job failed | 18:33 |
| JayF | mariojv: devstack/lib/ironic in the repo | 18:33 |
| *** aNuposic has joined #openstack-ironic | 18:34 | |
| mariojv | i'm trying to help aNuposic create a new baremetal node to test out discovery with inspector | 18:34 |
| mariojv | thanks, those are all env vars, guess i will backtrace them | 18:34 |
| * JayF wonders if you can just copy out the xml | 18:34 | |
| *** livelace has joined #openstack-ironic | 18:34 | |
| JayF | and change the name | 18:34 |
| mariojv | i don't think so | 18:34 |
| mariojv | create-node.sh sets up a bridge, links it to ovs, etc. | 18:34 |
| rloo | harlowja: thx. may not get to it until next week though | 18:35 |
| harlowja | k | 18:35 |
| *** jpich has quit IRC | 18:36 | |
| *** e0ne has quit IRC | 18:36 | |
| *** jkilpatr_ has joined #openstack-ironic | 18:42 | |
| JayF | I filed https://bugs.launchpad.net/ironic-inspector/+bug/1642362 about the gate issue I'm tracking | 18:43 |
| openstack | Launchpad bug 1642362 in Ironic Inspector "Devstack plugin sometimes loads out of order in gate" [Undecided,New] - Assigned to Jay Faulkner (jason-oldos) | 18:43 |
| *** jkilpatr has quit IRC | 18:43 | |
| *** athomas has quit IRC | 18:50 | |
| *** chlong has joined #openstack-ironic | 18:53 | |
| jroll | JayF: oh, that's fun | 18:56 |
| JayF | jroll: yeah, I'm talking about it in -infra now | 18:56 |
| JayF | jroll: I can't tell why | 18:56 |
| jroll | cool | 18:56 |
| jroll | thanks | 18:56 |
| JayF | I'm trying to figure out why this job: http://logs.openstack.org/06/396406/4/check/gate-tempest-dsvm-ironic-inspector-src-nv/69842b0/logs/localrc.txt.gz has ironic plugin first in localrc, but http://logs.openstack.org/16/334816/18/check/gate-tempest-dsvm-ironic-inspector-src-nv/e7b40f4/logs/localrc.txt.gz has inspector before ironic | 18:56 |
| jroll | right right | 18:56 |
| JayF | there are no changes to project-config or devstack-gate that would cause the change in behavior | 18:56 |
| JayF | and infra swears it's deterministic | 18:57 |
| JayF | so someone is wrong. probably me, but idk where | 18:57 |
| jroll | JayF: when did the bad one run? | 18:57 |
| JayF | both of those are from this mornign | 18:57 |
| jroll | hrm, ok | 18:57 |
| mariojv | aNuposic: i think the thing you're trying to ensure works is already tested in the gate: https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/test/inspector_tempest_plugin/tests/test_discovery.py#L117 | 18:58 |
| JayF | 11/16 14:38 (nonworking) vs 11/15 16:36 (working) | 18:59 |
| aNuposic | mariojv: I see | 18:59 |
| JayF | so there's more of a time span between them than I realized | 18:59 |
| mariojv | aNuposic: if you wanted, you could probably add a test for the enroll state there and improve that test upstream | 19:00 |
| mariojv | then you won't have to worry about replicating all the virsh calls locally | 19:00 |
| JayF | there were ironic changes merged to project-config between now and then | 19:00 |
| mariojv | aNuposic: another strategy i didn't think about was that you don't actually have to create a new node really | 19:00 |
| mariojv | aNuposic: you could just delete the node from ironic (ironic node-delete ...), power it on with virsh, and make sure it pxe boots / enrolls | 19:00 |
| *** jcoufal has quit IRC | 19:01 | |
| mariojv | aNuposic: i think the gate test is better but it wouldn't hurt to do both | 19:01 |
| mariojv | aNuposic: does that make sense? | 19:02 |
| *** penick has joined #openstack-ironic | 19:03 | |
| *** trown|lunch is now known as trown | 19:05 | |
| jroll | JayF: I feel like this broke it | 19:06 |
| jroll | willing to bet the ironic-extra-vars thing (with a comment about optional builders) is what enables inspector | 19:06 |
| *** Sukhdev has joined #openstack-ironic | 19:06 | |
| jroll | https://github.com/openstack-infra/project-config/commit/d6a054376d2956a15b79a40e530bcbf67a7dda65 | 19:06 |
| JayF | jroll: it's https://github.com/openstack-infra/project-config/commit/5dd7714098289c28bc9be9214eda90d4c6389651 | 19:06 |
| JayF | jroll: clark just got to the bottom of it in -infra | 19:07 |
| jroll | same patch yeah | 19:07 |
| jroll | jlvill-travel isn't here | 19:07 |
| JayF | oh, lol | 19:07 |
| JayF | yeah I'll own fixing it | 19:07 |
| JayF | after seeing why it was misordered in the first place | 19:07 |
| jroll | JayF: I'd revert it first, afaik that wasn't fixing anything voting | 19:09 |
| jroll | (as I don't remember any breakage) | 19:09 |
| JayF | jroll: I mean, given clark identified the ordering change as what broke it, I'd rather not completely undo someones work | 19:09 |
| JayF | jroll: and there was nothing in the review indicating that the change in ordering was explicit at all | 19:10 |
| JayF | well, I guess if I don't know wtf is up | 19:10 |
| JayF | I should revert and let jlvill-travel know | 19:10 |
| jroll | JayF: I mean | 19:10 |
| jroll | I can explain this commit | 19:10 |
| jroll | though I'm not sure what it was fixing | 19:10 |
| jroll | the only reason the order changed is so that IRONIC_DISABLE_DEVSTACK_PLUGIN variable could be read | 19:10 |
| jroll | afaict | 19:10 |
| JayF | yeah, which means moving the ordering | 19:11 |
| JayF | completely craters the whole patch anyway | 19:11 |
| jroll | right | 19:11 |
| jroll | might as well revert | 19:11 |
| jroll | and going by commit message / comments, we likely want to stop enabling the ironic devstack plugin in our grenade plugin? | 19:11 |
| jroll | although dunno what implications that has | 19:11 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic-inspector: [WIP/DNM] Test discovered nodes are in ENROLL state https://review.openstack.org/398546 | 19:12 |
| mariojv | aNuposic: ^ let's see if that passes | 19:13 |
| *** jcoufal has joined #openstack-ironic | 19:16 | |
| jroll | JayF: btw, that has broken inspector's gate altogether | 19:19 |
| jroll | https://review.openstack.org/#/c/344921/ | 19:19 |
| patchbot | patch 344921 - ironic-inspector - Add API for listing all introspection statuses | 19:19 |
| JayF | jroll: ah, I didn't see any failures in inspector | 19:19 |
| jroll | will be reporting as failing shortly | 19:19 |
| JayF | that was running tests when I looked though :) | 19:19 |
| JayF | aha | 19:19 |
| jroll | still is, but zuul tells me it's broken | 19:20 |
| JayF | makes sense. | 19:20 |
| *** g3ek has quit IRC | 19:20 | |
| JayF | Glad I caught it and got clark's help to track it down | 19:20 |
| JayF | I saw that change but didn't grok what it meant until he pointed it out | 19:20 |
| jroll | +1 | 19:20 |
| mariojv | for some reason inspector gate was failing on a notification patch set earlier, too | 19:20 |
| mariojv | so i don't think it's isolated | 19:20 |
| jroll | yep | 19:20 |
| JayF | mariojv: it's easy to see if it's caused by this: look in the localrc file in logs, if enable_plugin inspector before enable_plugin ironic you have a winner | 19:21 |
| JayF | rloo: we absolutely mask image_url at the api level | 19:21 |
| JayF | rloo: behind the same policy we use for password and other secrets, since it can be a swift temp url | 19:21 |
| aNuposic | mariojv: Okay, if this test pass then i believe mass discovery works | 19:24 |
| JayF | I think inspector has a discovery job, doesn't it? in the gate? | 19:26 |
| jroll | indeed | 19:26 |
| *** shobhar_ has joined #openstack-ironic | 19:26 | |
| mariojv | that's what i was pointing out | 19:28 |
| mariojv | i just added a line to make sure they start out in ENROLL | 19:28 |
| jroll | now... that job will probably fail currently, to be clear :) | 19:28 |
| mariojv | yeah :) | 19:28 |
| jroll | https://review.openstack.org/#/c/398545/ | 19:28 |
| patchbot | patch 398545 - openstack-infra/project-config - Revert "Disable devstack plugin for builder 'ironi... | 19:28 |
| mariojv | aNuposic: gate is broken right now, so we'll have to run "recheck" after that's fixed | 19:28 |
| aNuposic | mariojv: Okay | 19:29 |
| *** absubram has quit IRC | 19:30 | |
| *** g3ek has joined #openstack-ironic | 19:30 | |
| *** ChubYann has joined #openstack-ironic | 19:32 | |
| *** vsaienko has quit IRC | 19:34 | |
| *** rpioso-afk is now known as rpioso | 19:34 | |
| *** betherly_ has joined #openstack-ironic | 19:35 | |
| *** ijw has quit IRC | 19:35 | |
| jroll | does anyone recall the magic to force a library to be pulled in from source, rather than pip, in devstack? | 19:36 |
| JayF | jroll: https://github.com/openstack-dev/devstack/blob/master/stackrc#L211 what you're looking fo? | 19:38 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add ability to time out on individual clean steps https://review.openstack.org/391554 | 19:38 |
| jroll | JayF: probably, just trying to think if I can just throw that in our devstack plugin or what | 19:39 |
| jroll | just want to test a thing | 19:39 |
| JayF | I think that's more intended for localrc | 19:39 |
| jroll | yeah, which is ironic/devstack/settings | 19:39 |
| rloo | JayF, jroll: fwiw, I'm not sure when jlvill-travel will be back. might be tomorrow, or in a few days, he wasn't sure | 19:39 |
| jroll | rloo: yeah, not too worried, I think vsaienk0 is in that loop as well | 19:40 |
| JayF | rloo: it's not a big deal, I dropped him a line to let him know I have a patch up to revert his code | 19:40 |
| rloo | JayF: thx wrt masking image_url. I did a quick grep and didn't see it, might have grep'd for the wrong thing. | 19:40 |
| JayF | rloo: yeah; it went in when we standardized it all with policy, because it sorta came out that it needed to be masked too. So many secrets :P | 19:40 |
| rloo | jroll, JayF: ok, just don't want you to wait from him before taking action :) | 19:41 |
| jroll | rloo: nope, revert is up :) | 19:41 |
| *** fragatin_ has quit IRC | 19:41 | |
| *** fragatina has joined #openstack-ironic | 19:41 | |
| rloo | JayF, jroll: do you know when inspector was broken -- we usually put a date/time in the etherpad. | 19:41 |
| JayF | rloo: sometime in the last twelve hours, I think the first failed patch in inspector just finsihed :) I found it b/c of a nv job on ipa | 19:42 |
| jroll | rloo: this says 11 hours ago https://github.com/openstack-infra/project-config/commit/5dd7714098289c28bc9be9214eda90d4c6389651 | 19:42 |
| rloo | ok, i'll update the etherpad. thx jroll, JayF. | 19:42 |
| *** absubram has joined #openstack-ironic | 19:42 | |
| jroll | np, ty | 19:42 |
| * jroll currently reviewing oslo-ified hashring O_O | 19:43 | |
| *** jkilpatr_ has quit IRC | 19:44 | |
| rloo | thx for reviewing that jroll! | 19:45 |
| *** e0ne has joined #openstack-ironic | 19:49 | |
| *** e0ne has quit IRC | 19:50 | |
| *** pcaruana has quit IRC | 19:51 | |
| *** e0ne has joined #openstack-ironic | 19:52 | |
| *** e0ne has quit IRC | 19:54 | |
| *** causten has joined #openstack-ironic | 19:57 | |
| *** jkilpatr_ has joined #openstack-ironic | 19:57 | |
| *** vsaienko has joined #openstack-ironic | 19:57 | |
| jroll | not a problem | 19:58 |
| jroll | pretty close to current code | 19:58 |
| anish | dtantsur|afk: mgould|afk you're both afk, but what's your take on dell machines that don't follow wsman spec. I'm seeing a couple of machines where wsman is either just missing an element or two, or returning values outside of what's defined in the spec | 20:00 |
| *** e0ne has joined #openstack-ironic | 20:01 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Serve iPXE configs from Ironic API https://review.openstack.org/369438 | 20:01 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Implement Driver API for dynamic iPXE config https://review.openstack.org/394398 | 20:02 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Add RPC API to get iPXE template and options https://review.openstack.org/394399 | 20:02 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Convert iPXE boot script to Jinja template https://review.openstack.org/394396 | 20:02 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: DNM test enabled dynamic iPXE https://review.openstack.org/391943 | 20:02 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Allow not writing pxe configs to disk https://review.openstack.org/394397 | 20:02 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Convert iPXE boot config to proper Jinja template https://review.openstack.org/393462 | 20:02 |
| *** jcoufal has quit IRC | 20:05 | |
| *** vsaienko has quit IRC | 20:05 | |
| *** penick has quit IRC | 20:09 | |
| *** jcoufal has joined #openstack-ironic | 20:12 | |
| *** e0ne has quit IRC | 20:12 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Make agent driver always PXE before cleaning https://review.openstack.org/263842 | 20:13 |
| *** ijw_ has joined #openstack-ironic | 20:13 | |
| mariojv | aNuposic: that gate test failed, but it's because of the gate bug, not b/c of the test i added | 20:15 |
| mariojv | you can tell because inspector plugin was loaded before ironic plugin: http://logs.openstack.org/46/398546/1/check/gate-tempest-dsvm-ironic-inspector-discovery/ce71fc1/console.html | 20:15 |
| mariojv | is there anything that needs a review for the fix? it sounds like jroll and JayF figured out root cause | 20:15 |
| JayF | mariojv: there's a project-config patch up that has one +2 on it | 20:16 |
| JayF | mariojv: not much you can do | 20:16 |
| mariojv | alright | 20:16 |
| mariojv | thanks | 20:16 |
| jroll | yerp | 20:16 |
| JayF | I ping'd clark about it, since he's familiar but never voted on the patch | 20:17 |
| *** ijw_ has quit IRC | 20:19 | |
| mrda | Morning Ironic | 20:19 |
| mariojv | morning mrda | 20:19 |
| mrda | o/ | 20:19 |
| jroll | \o | 20:20 |
| *** e0ne has joined #openstack-ironic | 20:20 | |
| mat128 | wife has contractions, maybe we're on our way to the hospital | 20:22 |
| jroll | O_O | 20:22 |
| mat128 | in any case, I will be on PTO for ~5 weeks | 20:22 |
| rloo | jroll: if you have some time, would be good to get your eyes on https://review.openstack.org/#/c/396610/. sambetts|afk too. | 20:22 |
| patchbot | patch 396610 - ironic-specs - Add bonding configuration fields to portgroups | 20:22 |
| jroll | mat128: congrats and good luck! | 20:22 |
| rloo | mat128: congrats!!! | 20:22 |
| mat128 | thanks :) | 20:23 |
| mat128 | I'll try to keep you updated :) | 20:23 |
| jroll | \o/ | 20:23 |
| jroll | rloo: we're *still* changing that spec? jeez | 20:23 |
| JayF | mat128: congratulations! | 20:23 |
| *** Sukhdev has quit IRC | 20:23 | |
| *** penick has joined #openstack-ironic | 20:24 | |
| rloo | jroll: for the bonding. i think you and/or sambetts|afk had discussed this earlier this week or last week | 20:24 |
| jroll | indeed. | 20:25 |
| jroll | big update. | 20:25 |
| rloo | jroll: i was wondering if we could have broken that spec up into separate specs but it is what it is. | 20:25 |
| * jroll has a meeting now, though | 20:25 | |
| rloo | jroll: no worries, just want to make sure you and sambetts|afk are on board with this since it affects nova | 20:25 |
| jroll | right | 20:25 |
| * jroll stars it | 20:25 | |
| rloo | jroll: which reminds me -- the nova portgroups spec!!!! | 20:25 |
| jroll | rloo: I plugged it in the nova channel this morning | 20:26 |
| jroll | I'll actually bother people tomorrow if nobody go tit | 20:26 |
| jroll | (they didn't get it) | 20:26 |
| rloo | jroll: thx. we will have to pull out the big guns tomorrow... | 20:26 |
| * jroll loads some magazines | 20:27 | |
| *** priteau has quit IRC | 20:27 | |
| *** priteau has joined #openstack-ironic | 20:27 | |
| *** sabeen has quit IRC | 20:28 | |
| JayF | aNuposic: mariojv: project-config fix for the inspector gate just merged | 20:29 |
| mariojv | \o/ | 20:29 |
| aNuposic | Nice | 20:29 |
| mariojv | JayF: does that take effect immediately? | 20:29 |
| mariojv | if so i'll re-run | 20:29 |
| JayF | mariojv: I think you have to wait for a zuul puppet run | 20:30 |
| mariojv | oops | 20:30 |
| JayF | on zuul.openstack.org | 20:30 |
| JayF | bottom of page | 20:30 |
| JayF | Last reconfigured: Wed Nov 16 2016 11:26:55 GMT-0800 (PST) | 20:30 |
| patchbot | (PST) | 20:30 |
| JayF | so you might be in good shape anyway | 20:30 |
| mariojv | \o/ | 20:30 |
| JayF | patchbot: never change, you and your bad almost certainly exploitable bad string matches | 20:30 |
| *** ijw has joined #openstack-ironic | 20:31 | |
| *** priteau has quit IRC | 20:32 | |
| *** kvcobb has joined #openstack-ironic | 20:37 | |
| *** jkilpatr_ has quit IRC | 20:37 | |
| *** sabeen has joined #openstack-ironic | 20:38 | |
| *** jkilpatr has joined #openstack-ironic | 20:38 | |
| *** jkilpatr has quit IRC | 20:43 | |
| rama_y | project-config | 20:48 |
| rama_y | sorry | 20:49 |
| *** vsaienko has joined #openstack-ironic | 20:53 | |
| *** jkilpatr has joined #openstack-ironic | 20:56 | |
| *** e0ne has quit IRC | 20:56 | |
| *** rcernin has joined #openstack-ironic | 20:58 | |
| *** krtaylor has quit IRC | 20:58 | |
| *** MattMan has quit IRC | 21:03 | |
| *** MattMan has joined #openstack-ironic | 21:03 | |
| *** aNuposic has quit IRC | 21:04 | |
| xavierr | hey Ironic, I'm getting an error while launching an instance 'error finding disk or partition device' http://paste.openstack.org/show/589511/ | 21:05 |
| *** makowals_ has quit IRC | 21:05 | |
| xavierr | hardware issue and/or bad configuration? | 21:06 |
| JayF | That typically means it couldn't find a disk to deploy onto | 21:07 |
| jroll | it actually tells you exactly the problem | 21:08 |
| xavierr | JayF: any idea how to debug/solve? | 21:09 |
| xavierr | :) | 21:09 |
| JayF | See if you can find a disk? | 21:09 |
| JayF | :) | 21:09 |
| xavierr | haha ok, Jay | 21:09 |
| jroll | xavierr: check out the code in IPA for get_deploy_device or whatever it's called, it's pretty straightforward | 21:10 |
| *** Goneri has quit IRC | 21:10 | |
| JayF | get_os_install_device() :) | 21:11 |
| JayF | it's in hardware.py | 21:11 |
| JayF | jroll: mariojv: want a fun little review? https://review.openstack.org/#/c/334816/ not a priority, but it's straightforward, and idk who else would know things about ipa hardware managers other than rackers :) | 21:11 |
| patchbot | patch 334816 - ironic-python-agent - Add a new Hardware Manager for Mellanox NICs | 21:11 |
| xavierr | reading... thanks jroll JayF | 21:12 |
| *** dprince has quit IRC | 21:13 | |
| jroll | JayF: lots of major changes there for a lack of release note or anything at all in the commit message... | 21:17 |
| JayF | jroll: yeah, it needs a release note, that's a good call | 21:17 |
| jroll | JayF: left a review as such | 21:18 |
| jroll | I also wonder if that should be broken up into the core changes and the hw manager addition | 21:18 |
| JayF | I'm not sure what value that would add | 21:19 |
| jroll | they just seem like two atomic things | 21:19 |
| JayF | the first piece is completely not useful without a manager using it, right? | 21:19 |
| jroll | the dispatch is | 21:20 |
| jroll | the refactoring makes sense by itself too | 21:20 |
| JayF | I mean, if as someone reviewing you think so, feel free to comment that way, I'm honestly not convinced it's valuable, espeically when you think about how small the commit w/just the mlnx hardware manager would be alone | 21:21 |
| jroll | the hardware manager code is 230 lines, that isn't terribly small | 21:21 |
| *** jkilpatr has quit IRC | 21:25 | |
| jlvill-travel | JayF, Did you figure out why the ordering change broke inspector? | 21:26 |
| JayF | jlvill-travel: clark had an idea in -infra, but I have the 'results' in the bug | 21:26 |
| jlvill-travel | JayF, I'm confused why the ordering change would break it. Do you have a link to the bug? | 21:27 |
| jlvill-travel | I haven't fully read the back-scroll | 21:27 |
| JayF | jlvill-travel: the tl;dr is: enable_plugin is processed in the order they are in local.conf | 21:28 |
| JayF | jlvill-travel: after your change, for $reasons I don't fully understand, inspector jobs had the inspector plugin enabled before the ironic plugin | 21:28 |
| JayF | jlvill-travel: I think it has something to do with the ordering change w/ironic-extra-varsa | 21:28 |
| jlvill-travel | JayF, Ah okay. I can see that then. If enabling ironic-inspector plugin requires ironic plugin also | 21:29 |
| JayF | jlvill-travel: exactly, yeah | 21:29 |
| JayF | jlvill-travel: so the enable_plugin ironic in local.conf always must be before enable_plugin ironic-inspector | 21:29 |
| jroll | oh you know what we could do | 21:30 |
| jlvill-travel | JayF, Okay, that makes sense. | 21:30 |
| jroll | I think it needs to go the other way | 21:30 |
| jroll | no idk, ignore me | 21:31 |
| * jlvill-travel will ponder on this... | 21:31 | |
| jroll | jlvill-travel: I think if we had extra-vars that were something like SHOULD_ENABLE_FOO_PLUGIN and did both after | 21:32 |
| JayF | jlvill-travel: yeah; it's not a big deal, just wanted to make sure you knew your patch was reverted before it sent you down a post-travel rabbithole :) | 21:32 |
| jlvill-travel | jroll, Yeah. Let me think about it. | 21:32 |
| jlvill-travel | JayF, Thanks! | 21:32 |
| *** aNuposic has joined #openstack-ironic | 21:32 | |
| jroll | :) | 21:32 |
| *** krtaylor has joined #openstack-ironic | 21:33 | |
| jlvill-travel | One option might be to add the plugin in the ironic-grenade "job" that we have. | 21:33 |
| jlvill-travel | And then remove it from the grenade directory in ironic/devstack/upgrade/settings | 21:33 |
| jroll | yeah, that's the other route | 21:33 |
| jlvill-travel | I was just worried about the moving pieces... | 21:34 |
| jlvill-travel | Coordinating it. | 21:34 |
| *** krtaylor has quit IRC | 21:37 | |
| *** krtaylor has joined #openstack-ironic | 21:45 | |
| *** krtaylor has quit IRC | 21:47 | |
| *** Sukhdev has joined #openstack-ironic | 21:49 | |
| JayF | jroll: sanity check? https://review.openstack.org/#/c/398595/ | 21:49 |
| patchbot | patch 398595 - openstack-infra/project-config - Add ipa-example-hardware-managers | 21:49 |
| jroll | JayF: I did one of these this week and failed hard, I'm probably not the best sanity check :) | 21:50 |
| JayF | jroll: that sounds more like you're the best sanity check then | 21:50 |
| JayF | lol | 21:50 |
| jroll | JayF: I haven't looked into fixing it though :P | 21:50 |
| *** krtaylor has joined #openstack-ironic | 21:50 | |
| jroll | JayF: I'm not 100% sure on the job config bits but I think it looks right | 21:51 |
| *** trown is now known as trown|outtypewww | 21:52 | |
| *** krtaylor has quit IRC | 21:53 | |
| *** krtaylor has joined #openstack-ironic | 21:56 | |
| * jroll done for the day, see y'all tomorrow | 21:56 | |
| TheJulia | goodnight night jroll | 21:58 |
| TheJulia | err | 21:58 |
| * TheJulia gives up typing | 21:58 | |
| jroll | :D | 21:58 |
| *** krtaylor has quit IRC | 21:59 | |
| *** krtaylor has joined #openstack-ironic | 22:00 | |
| *** lindycoder has quit IRC | 22:06 | |
| *** betherly has quit IRC | 22:10 | |
| *** betherly_ is now known as betherly | 22:10 | |
| *** lifeless has quit IRC | 22:11 | |
| *** Goneri has joined #openstack-ironic | 22:11 | |
| *** lifeless has joined #openstack-ironic | 22:12 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client: Use stable/newton ironic-inspector for functional tests https://review.openstack.org/398474 | 22:20 |
| *** chas has joined #openstack-ironic | 22:23 | |
| *** absubram has quit IRC | 22:26 | |
| *** vsaienko has quit IRC | 22:27 | |
| *** chas has quit IRC | 22:27 | |
| *** penick has quit IRC | 22:29 | |
| openstackgerrit | Jay Carman proposed openstack/pyghmi: Raise IpmiException on error checking power state https://review.openstack.org/398629 | 22:29 |
| *** chas has joined #openstack-ironic | 22:31 | |
| *** chas has quit IRC | 22:33 | |
| *** chas has joined #openstack-ironic | 22:34 | |
| *** jcoufal has quit IRC | 22:34 | |
| *** absubram has joined #openstack-ironic | 22:37 | |
| mariojv | JayF: i'll take a look again when that's not workflow -1 if i remember | 22:40 |
| mariojv | ah, i'm already on there as a reviewer | 22:40 |
| JayF | mariojv: yeah, jroll -1'd it so moshe is fixing it | 22:41 |
| *** Goneri has quit IRC | 22:44 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce VolumeConnector object https://review.openstack.org/214584 | 22:45 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce new RPCs to support volume connector operation https://review.openstack.org/214585 | 22:45 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_connectors and volume_targets to task https://review.openstack.org/366196 | 22:45 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce new RPCs to support volume target operations https://review.openstack.org/285220 | 22:45 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce VolumeTarget object https://review.openstack.org/285219 | 22:45 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_targets table to save target information https://review.openstack.org/285218 | 22:45 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_connector table to save connector information https://review.openstack.org/200983 | 22:45 |
| *** aNuposic has quit IRC | 22:47 | |
| *** aNuposic has joined #openstack-ironic | 22:47 | |
| openstackgerrit | Sukhdev Kapur proposed openstack/ironic: Add support for Security Groups for baremetal servers https://review.openstack.org/361451 | 22:50 |
| *** rama_y has quit IRC | 22:50 | |
| *** chlong has quit IRC | 22:59 | |
| *** jkilpatr has joined #openstack-ironic | 23:01 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add storage_interface DB field and object https://review.openstack.org/348005 | 23:04 |
| *** jheroux has quit IRC | 23:04 | |
| *** penick has joined #openstack-ironic | 23:07 | |
| *** krtaylor has quit IRC | 23:09 | |
| *** chlong has joined #openstack-ironic | 23:17 | |
| *** penick has quit IRC | 23:20 | |
| *** penick has joined #openstack-ironic | 23:20 | |
| *** godp1301 has joined #openstack-ironic | 23:23 | |
| *** penick has quit IRC | 23:23 | |
| *** chlong has quit IRC | 23:27 | |
| *** godp1301 has quit IRC | 23:27 | |
| *** aNuposic has quit IRC | 23:37 | |
| *** shobhar_ has quit IRC | 23:37 | |
| *** zhugaoxiao has joined #openstack-ironic | 23:38 | |
| *** khamtamtun has joined #openstack-ironic | 23:39 | |
| *** Administrator_ has quit IRC | 23:40 | |
| *** rbrud has quit IRC | 23:41 | |
| *** khamtamtun has quit IRC | 23:42 | |
| *** david-lyle_ has joined #openstack-ironic | 23:50 | |
| *** david-lyle has quit IRC | 23:51 | |
| *** rcernin has quit IRC | 23:52 | |
| *** syed__ has quit IRC | 23:55 | |
| *** godp1301 has joined #openstack-ironic | 23:58 | |
| *** rpioso has quit IRC | 23:59 | |
| *** baoli has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
