| *** ijw has quit IRC | 00:00 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs master: Enhance ilo drivers to do inband inspection https://review.openstack.org/201904 | 00:15 |
|---|---|---|
| *** ijw has joined #openstack-ironic | 00:20 | |
| *** Nisha_Agarwal has quit IRC | 00:20 | |
| *** ijw has quit IRC | 00:24 | |
| *** g3ek has quit IRC | 00:37 | |
| *** sasha21 has quit IRC | 00:41 | |
| *** g3ek has joined #openstack-ironic | 00:46 | |
| *** crushil has joined #openstack-ironic | 00:50 | |
| *** yee379 has quit IRC | 00:58 | |
| *** yee379 has joined #openstack-ironic | 00:58 | |
| *** tiendc has joined #openstack-ironic | 01:02 | |
| *** hoonetorg has quit IRC | 01:15 | |
| *** hoonetorg has joined #openstack-ironic | 01:20 | |
| *** WeiAA has joined #openstack-ironic | 01:21 | |
| *** tuanluong has joined #openstack-ironic | 01:31 | |
| *** chas has joined #openstack-ironic | 01:47 | |
| *** chas has quit IRC | 01:51 | |
| *** Sukhdev_ has quit IRC | 01:52 | |
| *** mat128 has joined #openstack-ironic | 01:55 | |
| *** yolanda has quit IRC | 01:55 | |
| *** Goneri has quit IRC | 01:58 | |
| *** zhurong has joined #openstack-ironic | 02:04 | |
| *** Nakato has quit IRC | 02:06 | |
| *** Nakato has joined #openstack-ironic | 02:09 | |
| *** jaeman has joined #openstack-ironic | 02:15 | |
| jaeman | Hi, guys. I am provisioning baremetal nova instance by Ironic with fedora image. The instance's status is ACTIVE but booting does not finish with message "A start job is running for dev-disk..." | 02:17 |
| jaeman | Please let me know if anyone know how to resolve this issue. | 02:18 |
| *** yee379 has quit IRC | 02:31 | |
| *** yee37917 has joined #openstack-ironic | 02:31 | |
| *** Nakato has quit IRC | 02:32 | |
| *** Nakato has joined #openstack-ironic | 02:33 | |
| openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient master: Extends driver-list, driver-show supporting new hardware types https://review.openstack.org/419274 | 02:42 |
| openstackgerrit | Ngo Quoc Cuong proposed openstack/ironic master: Remove translation of log messages from + ironic/drivers/modules/drac + ironic/drivers/modules/ilo + ironic/drivers/modules/network + ironic/drivers/modules/oneview https://review.openstack.org/460367 | 02:47 |
| openstackgerrit | Ngo Quoc Cuong proposed openstack/ironic master: Remove translation of log messages from + ironic/drivers/modules/drac + ironic/drivers/modules/network + ironic/drivers/modules/oneview https://review.openstack.org/460367 | 02:54 |
| *** hw_wutianwei has quit IRC | 02:58 | |
| *** hoonetorg has quit IRC | 02:59 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add context to IronicObject._from_db_object() https://review.openstack.org/460809 | 03:04 |
| *** mat128 has quit IRC | 03:15 | |
| *** hoonetorg has joined #openstack-ironic | 03:15 | |
| *** crushil has quit IRC | 03:19 | |
| *** rbudden has quit IRC | 03:19 | |
| *** aparnav has joined #openstack-ironic | 03:24 | |
| *** zhurong has quit IRC | 03:38 | |
| *** mat128 has joined #openstack-ironic | 03:47 | |
| *** chas has joined #openstack-ironic | 03:48 | |
| *** gkadam has joined #openstack-ironic | 03:51 | |
| *** chas has quit IRC | 03:52 | |
| openstackgerrit | Ngo Quoc Cuong proposed openstack/ironic master: Remove translation of log messages from + ironic/drivers/modules/drac + ironic/drivers/modules/network + ironic/drivers/modules/oneview + ironic/common + ironic/tests https://review.openstack.org/460367 | 03:57 |
| *** pmannidi has joined #openstack-ironic | 04:03 | |
| *** pmannidi has quit IRC | 04:04 | |
| *** pmannidi has joined #openstack-ironic | 04:04 | |
| *** fragatina has quit IRC | 04:11 | |
| *** jcoufal has joined #openstack-ironic | 04:13 | |
| *** amotoki has joined #openstack-ironic | 04:14 | |
| *** phuongnh has joined #openstack-ironic | 04:15 | |
| *** jcoufal has quit IRC | 04:20 | |
| *** adisky_ has joined #openstack-ironic | 04:23 | |
| *** mat128 has quit IRC | 04:27 | |
| *** jaypipes has quit IRC | 04:29 | |
| *** Sukhdev has joined #openstack-ironic | 04:30 | |
| *** udesale has joined #openstack-ironic | 04:41 | |
| *** jaypipes has joined #openstack-ironic | 04:41 | |
| *** gkadam__ has joined #openstack-ironic | 04:48 | |
| *** gkadam has quit IRC | 04:49 | |
| *** gkadam__ is now known as regain | 04:49 | |
| *** regain is now known as gkadam | 04:49 | |
| *** gkadam has quit IRC | 04:50 | |
| *** gkadam has joined #openstack-ironic | 04:51 | |
| *** karthiks has joined #openstack-ironic | 04:54 | |
| *** zhurong has joined #openstack-ironic | 05:00 | |
| *** fragatina has joined #openstack-ironic | 05:10 | |
| *** fragatina has quit IRC | 05:12 | |
| *** fragatina has joined #openstack-ironic | 05:13 | |
| *** Sukhdev has quit IRC | 05:13 | |
| *** l_jacob has joined #openstack-ironic | 05:17 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 05:25 | |
| *** causten has joined #openstack-ironic | 05:39 | |
| *** ChubYann has quit IRC | 05:42 | |
| *** yuanying_ has quit IRC | 05:57 | |
| *** jaeman has quit IRC | 05:59 | |
| *** Lambert has joined #openstack-ironic | 05:59 | |
| *** zhurong has quit IRC | 06:04 | |
| Nisha_Agarwal | morning Ironic :) | 06:10 |
| *** zhurong has joined #openstack-ironic | 06:11 | |
| *** mvpnitesh has joined #openstack-ironic | 06:12 | |
| *** causten has quit IRC | 06:13 | |
| openstackgerrit | Ngo Quoc Cuong proposed openstack/ironic master: Remove translation of log messages from ironic/drivers/modules/ and ironic/drivers/ (not include sub-folder) https://review.openstack.org/460841 | 06:13 |
| *** zhurong has quit IRC | 06:23 | |
| *** Lambert has quit IRC | 06:29 | |
| *** phuongnh has quit IRC | 06:30 | |
| *** yuanying has joined #openstack-ironic | 06:36 | |
| *** pcaruana has joined #openstack-ironic | 06:36 | |
| Nisha_Agarwal | lucas-afk, hi | 06:39 |
| *** ccamacho has joined #openstack-ironic | 06:40 | |
| *** Lambert has joined #openstack-ironic | 06:41 | |
| *** yuanying_ has joined #openstack-ironic | 06:42 | |
| *** yuanying has quit IRC | 06:42 | |
| *** e0ne has joined #openstack-ironic | 06:46 | |
| *** yuanying_ has quit IRC | 06:46 | |
| *** yuanying has joined #openstack-ironic | 06:47 | |
| *** chas has joined #openstack-ironic | 07:03 | |
| *** tesseract has joined #openstack-ironic | 07:05 | |
| *** e0ne has quit IRC | 07:19 | |
| *** Serlex has joined #openstack-ironic | 07:23 | |
| *** udesale has quit IRC | 07:30 | |
| *** tiendc_ has joined #openstack-ironic | 07:33 | |
| *** tuanluong has quit IRC | 07:34 | |
| *** tiendc has quit IRC | 07:34 | |
| *** g3ek has quit IRC | 07:38 | |
| *** pmannidi has quit IRC | 07:42 | |
| *** mgoddard has joined #openstack-ironic | 07:42 | |
| *** mvpnitesh has quit IRC | 07:45 | |
| *** mvpnitesh has joined #openstack-ironic | 07:45 | |
| openstackgerrit | Thomas Herve proposed openstack/ironic master: Fix oslo.messaging log level https://review.openstack.org/457913 | 07:46 |
| *** g3ek has joined #openstack-ironic | 07:46 | |
| Nisha_Agarwal | sambetts|afk, | 07:49 |
| Nisha_Agarwal | Hi | 07:49 |
| *** Linkmark has joined #openstack-ironic | 07:53 | |
| *** makowals has joined #openstack-ironic | 07:54 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| *** aarefiev_afk is now known as aarefiev | 08:07 | |
| aarefiev | morning all! | 08:07 |
| pas-ha | morning ironic :) | 08:09 |
| *** athomas has joined #openstack-ironic | 08:09 | |
| *** dtantsur|afk is now known as dtantsur | 08:17 | |
| dtantsur | morning ironic! | 08:17 |
| *** lucas-afk is now known as lucasagomes | 08:17 | |
| lucasagomes | morning all | 08:17 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [Devstack] Rework VMs connection logic https://review.openstack.org/453541 | 08:18 |
| *** phuongnh has joined #openstack-ironic | 08:18 | |
| *** mvpnitesh has quit IRC | 08:23 | |
| *** tiendc has joined #openstack-ironic | 08:27 | |
| *** phuongnh has quit IRC | 08:28 | |
| *** tiendc_ has quit IRC | 08:28 | |
| Nisha_Agarwal | lucasagomes, dtantsur Hi | 08:29 |
| *** phuongnh has joined #openstack-ironic | 08:29 | |
| Nisha_Agarwal | I have the latest ironic and my ironic.conf has "redfish" as the enabled hardware types | 08:29 |
| Nisha_Agarwal | but my conductor fails with error "DriverNotFoundInEntrypoint: Could not find the following items in the 'ironic.hardware.types' entrypoint: redfish, ipmi." | 08:30 |
| dtantsur | Nisha_Agarwal, it does looks like some problem with installation. as you see it cannot even find "ipmi", which was introduced last cycle | 08:31 |
| Nisha_Agarwal | i tried installing it using all the steps "sudo pip install .", "sudo pip install -e ." , and "sudo python setup.py install" | 08:32 |
| Nisha_Agarwal | but the error doesnt go off | 08:32 |
| *** racedo has joined #openstack-ironic | 08:32 | |
| Nisha_Agarwal | dtantsur, yeah but i dont know what | 08:33 |
| Nisha_Agarwal | dtantsur, i just upgraded the sushy library and restarted conductor so that it can use the latest sushy | 08:34 |
| Nisha_Agarwal | and it started failing | 08:34 |
| dtantsur | hmm | 08:34 |
| Nisha_Agarwal | so i have even upgraded ironic and reinstalled it using above commands, but it fails | 08:34 |
| Nisha_Agarwal | :( | 08:34 |
| Nisha_Agarwal | i wanted to test redfish driver with redfish hardware | 08:35 |
| lucasagomes | strange, I mean, the library doesn't do anything with entry points | 08:35 |
| lucasagomes | it seems something with the ironic installation as dtantsur pointed out | 08:36 |
| Nisha_Agarwal | last week when i faced the same issue while bringing up the stack sambetts told me to use "sudo pip install -e ." and that worked that time | 08:36 |
| dtantsur | Nisha_Agarwal, `less /usr/lib/python2.7/site-packages/ironic-*.egg-info/entry_points.txt` <- does it list redfish and ipmi? | 08:36 |
| Nisha_Agarwal | lucasagomes, yes, but i have done all the correct way of installing the ironci | 08:36 |
| dtantsur | well, assuming you have the same package layout as my centos.. | 08:36 |
| Nisha_Agarwal | dtantsur, no it doesnt list | 08:39 |
| Nisha_Agarwal | http://paste.openstack.org/show/608292/ | 08:39 |
| Nisha_Agarwal | how to get that in this | 08:40 |
| dtantsur | Nisha_Agarwal, ironic-2015.1.dev18 is OLD :) | 08:40 |
| dtantsur | something apparently went wrong with pip install. what if you delete/uninstall ironic completely, and then try it again? I think 'sudo pip install -e .' is what you want to use | 08:40 |
| *** derekh has joined #openstack-ironic | 08:40 | |
| dtantsur | oh, I think I now what's wrong | 08:41 |
| dtantsur | version 2015.1 is higher than 8.0 or whatever we have now | 08:41 |
| dtantsur | you have to uninstall ironic completely indeed | 08:41 |
| dtantsur | Nisha_Agarwal, ^^^ | 08:41 |
| *** makowals has quit IRC | 08:42 | |
| Nisha_Agarwal | dtantsur, oh | 08:43 |
| Nisha_Agarwal | if i uninstal and install i dont need to restack, correct? | 08:43 |
| *** tuanluong has joined #openstack-ironic | 08:43 | |
| Nisha_Agarwal | or if i just remove old *.egg directories will resolve the issue | 08:43 |
| Nisha_Agarwal | ? | 08:43 |
| Nisha_Agarwal | dtantsur, ^^^ | 08:44 |
| dtantsur | Nisha_Agarwal, start with 'sudo pip uninstall ironic', then manually remove if anything is left /usr/lib/python2.7/site-packages/ | 08:44 |
| Nisha_Agarwal | k | 08:45 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-specs master: Remove Pike priorities that cannot make it due to recent team changes https://review.openstack.org/460086 | 08:45 |
| Nisha_Agarwal | dtantsur, i dont need to restack? | 08:45 |
| Nisha_Agarwal | correct | 08:45 |
| Nisha_Agarwal | its there in 8.0 version but ironic conductor is picking up old one | 08:46 |
| dtantsur | Nisha_Agarwal, correct, I think you can avoid that | 08:46 |
| Nisha_Agarwal | :( | 08:46 |
| Nisha_Agarwal | ok | 08:46 |
| Nisha_Agarwal | dtantsur, yes it worked...it uninstalled all old versions by itself | 08:48 |
| Nisha_Agarwal | dtantsur, thanks | 08:48 |
| dtantsur | np | 08:48 |
| *** jaosorior_away is now known as jaosorior | 08:49 | |
| Nisha_Agarwal | dtantsur, milan had one comment on "Boot Management for inband-inspection" spec | 08:49 |
| Nisha_Agarwal | i am not the expert in inspector | 08:50 |
| Nisha_Agarwal | i need your advice | 08:50 |
| Nisha_Agarwal | dtantsur, could you help in that | 08:51 |
| Nisha_Agarwal | only one comment outstanding in that spec | 08:51 |
| *** e0ne has joined #openstack-ironic | 08:52 | |
| dtantsur | sure | 08:52 |
| *** milan has joined #openstack-ironic | 08:56 | |
| Nisha_Agarwal | lucasagomes, when i create ironic node using redfish driver i get error as in http://paste.openstack.org/show/608294/ | 08:57 |
| Nisha_Agarwal | lucasagomes, any idea | 08:57 |
| dtantsur | oh | 09:00 |
| dtantsur | this is why we need extensive logging | 09:00 |
| lucasagomes | Nisha_Agarwal, hmm not off the top of my head | 09:00 |
| lucasagomes | feels like programatic error tho | 09:00 |
| dtantsur | it certainly is | 09:01 |
| dtantsur | huh, I cannot find any use of "find" call in the sushy driver | 09:02 |
| dtantsur | Nisha_Agarwal, do you have up-to-date sushy? | 09:02 |
| *** MattMan has quit IRC | 09:03 | |
| *** MattMan has joined #openstack-ironic | 09:03 | |
| dtantsur | lucasagomes, speaking of vbmc, did you have a chance to check it's CI? I think it was down last time I saw it | 09:04 |
| *** hamzy has joined #openstack-ironic | 09:04 | |
| milan | morning dtantsur, lucasagomes, Nisha_Agarwal! :) | 09:04 |
| milan | and morning Ironic! :) | 09:04 |
| Nisha_Agarwal | dtantsur, yes it is uptodaye | 09:04 |
| lucasagomes | dtantsur, I have not, the last weekes I've been mostly looking into ovn stuff | 09:04 |
| Nisha_Agarwal | i tested latest sushy for power on and off it works on python shell | 09:05 |
| Nisha_Agarwal | lucasagomes, ^^^ | 09:05 |
| lucasagomes | dtantsur, I will take a look, Isee the patch is failing there | 09:06 |
| dtantsur | Nisha_Agarwal, I'm afraid I don't have an easy answer then.. you may have to try inserting logs/prints in various parts of the code to see where exactly it dies | 09:06 |
| Nisha_Agarwal | dtantsur, hmmm | 09:07 |
| Nisha_Agarwal | dtantsur, it looks to me something in ironic :( | 09:07 |
| lucasagomes | Nisha_Agarwal, what you get when you change the power state manually via ironic's api ? | 09:08 |
| lucasagomes | ironic node-set-power-state blah on/off | 09:08 |
| dtantsur | I think so, I just could not find the relevant place by just looking at the code. also ++ to the question ^^^ | 09:09 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [Grenade]: Do not run ir-api on primary node after upgrade https://review.openstack.org/456166 | 09:09 |
| Nisha_Agarwal | lucasagomes, i get 200 status | 09:09 |
| lucasagomes | Nisha_Agarwal, so it works ? | 09:10 |
| Nisha_Agarwal | wer state (took 0.02 seconds) from (pid=24289) reserve_node /opt/stack/ironic/ironic/conductor/task_manager.py:273 | 09:10 |
| Nisha_Agarwal | 2017-04-28 02:37:16.891 DEBUG sushy.connector [req-1cb4d0c7-342d-40ef-8c4c-1d2af59c6af3 None None] Issuing a HTTP GET request at https://15.119.209.114/redfish/v1/ with the headers "{'Content-Type': 'application/json'}" and data "" from (pid=24289) _op /opt/stack/sushy/sushy/connector.py:67 | 09:10 |
| Nisha_Agarwal | 2017-04-28 02:37:17.535 DEBUG sushy.connector [req-1cb4d0c7-342d-40ef-8c4c-1d2af59c6af3 None None] Response: Status code: 200 from (pid=24289) _op /opt/stack/sushy/sushy/connector.py:73 | 09:10 |
| Nisha_Agarwal | 2017-04-28 02:37:17.586 DEBUG ironic.conductor.task_manager [req-1cb4d0c7-342d-40ef-8c4c-1d2af59c6af3 None None] Successfully released exclusive lock for changing node power state on node 9bf33085-4e2b-49c7-b48a-a9a4bc1e83c9 (lock was held 0.74 sec) from (pid=24289) release_resources /opt/stack/ironic/ironic/conductor/task_manager.py:352 | 09:10 |
| Nisha_Agarwal | but it fails for "sync_power_state" | 09:10 |
| Nisha_Agarwal | so no power state is registered for the node | 09:10 |
| Nisha_Agarwal | let me see it looks like some constant issue | 09:10 |
| lucasagomes | ok, yeah i guess you will have to debug that | 09:11 |
| Nisha_Agarwal | yeah the node.last_error shows this " Failed to change power state to 'power off'. Error: 'int' object has no attribute 'find'" | 09:12 |
| lucasagomes | yeah i would need to give it a go, I have no env here atm | 09:13 |
| lucasagomes | so I need to build one | 09:13 |
| lucasagomes | I don't see any find() in sushy or in the drivers/modules/redfish in ironic tho | 09:15 |
| * lucasagomes wonders | 09:15 | |
| openstackgerrit | Merged openstack/ironic-inspector-specs master: Split service into API and Worker https://review.openstack.org/435517 | 09:21 |
| dtantsur | aarefiev, ^^^ \o/ | 09:22 |
| dtantsur | let's make it happen now :) | 09:22 |
| aarefiev | \o/, thanks | 09:23 |
| aarefiev | there is first patch with service functionality which I need to rebase | 09:24 |
| *** mvpnitesh has joined #openstack-ironic | 09:24 | |
| dtantsur | aarefiev, milan, a small patch we need to avoid getting broken by https://review.openstack.org/#/c/460062/: https://review.openstack.org/#/c/460216/ | 09:24 |
| patchbot | patch 460062 - openstack-dev/devstack - make USE_SCREEN=False imply USE_SYSTEMD=True | 09:24 |
| patchbot | patch 460216 - ironic-inspector - [devstack] fix working with USE_SYSTEMD=True | 09:24 |
| *** sambetts|afk is now known as sambetts | 09:31 | |
| *** jtomasek has quit IRC | 09:34 | |
| *** jtomasek has joined #openstack-ironic | 09:37 | |
| *** amarao has joined #openstack-ironic | 09:43 | |
| *** racedo has quit IRC | 09:52 | |
| *** paresh has joined #openstack-ironic | 09:58 | |
| *** udesale has joined #openstack-ironic | 09:58 | |
| * milan checking dtantsur patches | 09:58 | |
| *** jaosorior is now known as jaosorior_away | 10:00 | |
| paresh | Hi dtantsur | 10:02 |
| paresh | dtantsur Can you look into this patch https://review.openstack.org/#/c/454625/ | 10:03 |
| patchbot | patch 454625 - ironic - Change to avoid building of deploy iso when provided | 10:03 |
| dtantsur | not now, but added to my list | 10:04 |
| paresh | dtantsur thanks | 10:04 |
| *** tesseract has quit IRC | 10:04 | |
| *** paresh has quit IRC | 10:05 | |
| *** tesseract has joined #openstack-ironic | 10:08 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Remove redundant copy operation in pxe_enabled update code https://review.openstack.org/460919 | 10:14 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Updated from global requirements https://review.openstack.org/459131 | 10:16 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Allow timeout in active states https://review.openstack.org/460690 | 10:17 |
| *** chas has quit IRC | 10:18 | |
| *** Fdaisuke has joined #openstack-ironic | 10:27 | |
| *** phuongnh has quit IRC | 10:31 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Bump pydot2 to pydot3 https://review.openstack.org/460938 | 10:39 |
| *** jkilpatr has quit IRC | 10:42 | |
| * milan errands and Monday is a public holiday here; CU guys on Tuesday! Unless you're on hewbrocca 's party tonight! :P | 10:42 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic master: Validate outlet index in SNMP driver https://review.openstack.org/452716 | 10:42 |
| *** karthiks has quit IRC | 10:43 | |
| *** mat128 has joined #openstack-ironic | 10:44 | |
| *** vmud213 has joined #openstack-ironic | 10:46 | |
| *** paresh has joined #openstack-ironic | 10:47 | |
| *** milan has quit IRC | 10:47 | |
| *** sao_p has joined #openstack-ironic | 10:50 | |
| *** paresh has quit IRC | 10:50 | |
| *** sao_p has quit IRC | 10:51 | |
| *** tuanluong has quit IRC | 10:52 | |
| *** udesale__ has joined #openstack-ironic | 10:54 | |
| *** udesale has quit IRC | 10:56 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [Grenade]: Do not run ir-api on primary node after upgrade https://review.openstack.org/456166 | 10:59 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Fix `clean up` error logging formatting https://review.openstack.org/460989 | 11:07 |
| *** makowals has joined #openstack-ironic | 11:09 | |
| *** makowals has quit IRC | 11:10 | |
| dtantsur | lucasagomes, hey! two suggestions for the redfish driver, wdyt about: 1. allowing missing scheme (assume https), 2. allowing separate redfish_port? | 11:10 |
| dtantsur | this would bring it much closer to other drivers, where xxx_address is a host name or IP | 11:11 |
| *** jkilpatr has joined #openstack-ironic | 11:11 | |
| lucasagomes | dtantsur, I like 1. it's secure by default | 11:12 |
| lucasagomes | we need to change some validations for that, I think that for now we require one to add the schema as part of the address | 11:12 |
| dtantsur | I can do that, if you don't object | 11:12 |
| lucasagomes | 2. hmm it's a little strange based on the format of the address | 11:12 |
| lucasagomes | dtantsur, I'm ok with it, I mean no objections | 11:13 |
| dtantsur | lucasagomes, yeah.. we do accept client_port for ilo | 11:13 |
| lucasagomes | yeah or ipmi/drac as well | 11:13 |
| *** l_jacob has quit IRC | 11:13 | |
| dtantsur | and irmc has it configured in ironic.conf Oo | 11:13 |
| * dtantsur cries | 11:13 | |
| lucasagomes | hah | 11:14 |
| lucasagomes | one thing tho, we should probably look at unifying such crendentials | 11:14 |
| dtantsur | all this would not be a problem if tripleo did not require hardcoding it.... | 11:14 |
| lucasagomes | instead of having them in the namespace of the drivers | 11:14 |
| lucasagomes | that way we can guarantee some consistency between all driver | 11:14 |
| lucasagomes | drivers* | 11:14 |
| lucasagomes | dtantsur, *sigh* re OOO | 11:14 |
| dtantsur | ok, let me do #1, and #2 can go to tripleo itself | 11:15 |
| *** makowals has joined #openstack-ironic | 11:17 | |
| lucasagomes | ++ | 11:19 |
| *** Nisha_Agarwal has quit IRC | 11:20 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 11:21 | |
| *** Lambert has quit IRC | 11:21 | |
| *** avladu has joined #openstack-ironic | 11:22 | |
| sambetts | +1000 to standardising the driver properties | 11:25 |
| jkilpatr | d0ugal, those patches ready to test? | 11:26 |
| jkilpatr | umm it might help if I knew how to apply them in my job.... | 11:26 |
| jkilpatr | Anyways I have a 30 node ovb test cloud we can apply them to and get feedback at least daily. | 11:27 |
| dtantsur | jkilpatr, wrong channel or am I missing something? :) | 11:29 |
| *** gkadam has quit IRC | 11:29 | |
| jkilpatr | dtantsur, https://review.openstack.org/#/c/439568/ | 11:29 |
| patchbot | patch 439568 - tripleo-common - Run introspection 20 nodes at a time | 11:29 |
| jkilpatr | so I guess that's not a ironic patch exactly? | 11:30 |
| jkilpatr | ¯\_(ツ)_/¯ | 11:30 |
| dtantsur | jkilpatr, not entirely :) though I'd love some scale testing on it | 11:31 |
| jkilpatr | dtantsur, that's the idea behind this job, daily scale builds, if mainline isn't passing it's a place to test patchsets | 11:31 |
| jkilpatr | rather than "oh well lets wait until we have someone with hardware and spare time" | 11:32 |
| dtantsur | nice | 11:34 |
| *** chas has joined #openstack-ironic | 11:40 | |
| *** racedo has joined #openstack-ironic | 11:42 | |
| *** chas has quit IRC | 11:45 | |
| *** Serlex has quit IRC | 11:45 | |
| *** xavierr has joined #openstack-ironic | 11:47 | |
| openstackgerrit | Aparna proposed openstack/proliantutils master: Performs IPA erase_devices after Sanitize disk erase https://review.openstack.org/436450 | 11:49 |
| *** tesseract has quit IRC | 11:50 | |
| *** chas has joined #openstack-ironic | 11:50 | |
| *** mvpnitesh has quit IRC | 11:51 | |
| *** mvpnitesh has joined #openstack-ironic | 11:54 | |
| dtantsur | lucasagomes, oh fun. the library we use parse example.com:8080 as scheme example.com and authority 8080 >_< | 11:55 |
| *** bfournie has quit IRC | 11:55 | |
| *** ccamacho is now known as ccamacho|lunch | 11:57 | |
| openstackgerrit | Merged openstack/ironic-inspector master: [devstack] fix working with USE_SYSTEMD=True https://review.openstack.org/460216 | 11:58 |
| *** galyna has left #openstack-ironic | 11:58 | |
| *** Nisha_Agarwal has quit IRC | 12:02 | |
| *** d0ugal_ has joined #openstack-ironic | 12:02 | |
| *** zhenguo has quit IRC | 12:02 | |
| *** xavierr has quit IRC | 12:03 | |
| *** d0ugal has quit IRC | 12:04 | |
| *** tesseract has joined #openstack-ironic | 12:05 | |
| openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient master: Extends driver-list, driver-show supporting new hardware types https://review.openstack.org/419274 | 12:06 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Bring the redfish driver address parameter closer to one of other drivers https://review.openstack.org/461008 | 12:07 |
| dtantsur | lucasagomes, ^^^ | 12:07 |
| lucasagomes | dtantsur, lol | 12:10 |
| *** g3ek has quit IRC | 12:10 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Support hpsum based `update_firmware` for iLO drivers https://review.openstack.org/422572 | 12:10 |
| lucasagomes | dtantsur, will take a look | 12:10 |
| lucasagomes | /me goes grab some food quickly and then will look at it | 12:11 |
| *** lucasagomes is now known as lucas-hungry | 12:11 | |
| *** dprince has joined #openstack-ironic | 12:11 | |
| *** tiendc has quit IRC | 12:12 | |
| TheJulia | Good morning everyone | 12:15 |
| dtantsur | morning TheJulia | 12:16 |
| jroll | morning TheJulia, dtantsur, everyone else | 12:16 |
| dtantsur | and jroll :) | 12:16 |
| jroll | :) | 12:16 |
| *** chlong has quit IRC | 12:21 | |
| *** d0ugal_ is now known as d0ugal | 12:21 | |
| d0ugal | jkilpatr: I can help test in about an hour if that works for you? sorry, things a bit hectic here. | 12:21 |
| *** d0ugal is now known as Guest8049 | 12:21 | |
| *** Guest8049 has quit IRC | 12:21 | |
| jkilpatr | d0ugal, don't worry the job only run's once a day anyways, we have all day to set it up. | 12:22 |
| jkilpatr | runs* | 12:22 |
| *** d0ugal_ has joined #openstack-ironic | 12:22 | |
| d0ugal_ | jkilpatr: I've not managed to do as much as I'd like this week - had issues with my own setup, but it should be ready to test, in theory :) | 12:22 |
| d0ugal_ | back shortly | 12:22 |
| *** bcornec has joined #openstack-ironic | 12:24 | |
| *** bfournie has joined #openstack-ironic | 12:25 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic-ui master: Add node filtering by chassis uuid https://review.openstack.org/461015 | 12:29 |
| TheJulia | mgoddard: Hey, I was looking at https://review.openstack.org/#/c/460005. Are you planning on putting together an overall documentation update once all of the pieces are together | 12:31 |
| patchbot | patch 460005 - bifrost - Support storage of introspection data in Nginx | 12:31 |
| TheJulia | (personally, I'd be very +2 on landing defaults to collect introspection data for lldp data....) | 12:31 |
| dtantsur | mgoddard, TheJulia, I'm quite meh on this patch. Why not add an option to store data locally to ironic-inspector instead? | 12:33 |
| TheJulia | Fastest path to functional as opposed to waiting for an inspector release? | 12:33 |
| TheJulia | and it would be backwards compatible | 12:34 |
| dtantsur | well.. fast != best :) anyway, not going to -1 it, just a bit sad that's bifrost-specific | 12:34 |
| TheJulia | Not saying inspector should not have that feature though :) | 12:34 |
| TheJulia | More bifrost being attune to the present capabilities/defaults of the world around it. :) | 12:35 |
| *** aparnav has quit IRC | 12:38 | |
| *** gcb has quit IRC | 12:40 | |
| *** trown|outtypewww is now known as trown | 12:40 | |
| *** fragatina has quit IRC | 12:40 | |
| *** fragatina has joined #openstack-ironic | 12:41 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 12:44 | |
| *** rbudden has joined #openstack-ironic | 12:48 | |
| mgoddard | TheJulia: hi, I can put some docs together for it but currently the bifrost docs don't mention inspector at all so I'd be starting from nothing (related: https://bugs.launchpad.net/bifrost/+bug/1554141) | 12:52 |
| openstack | Launchpad bug 1554141 in bifrost "Documentation of inspector usage lacking" [High,Confirmed] | 12:52 |
| mgoddard | TheJulia, dtantsur: agree that local inspector storage might be a better option. The nginx route went from an idea to a local implementation so thought I'd share it | 12:54 |
| TheJulia | It definitely meets my "Is this useful?" bar :) | 12:56 |
| mgoddard | TheJulia: slight -1 against always collecting LLDP is that it incurs a 30 second timeout each time we check for LLDP data (which seems to be more than once for some reason, but that's a separate issue) | 12:57 |
| TheJulia | Ahh, I didn't know that. That makes sense then | 12:57 |
| sambetts | mgoddard: I think I worked out why it does it more than once | 12:58 |
| *** mat128 has quit IRC | 12:58 | |
| TheJulia | mgoddard: with regards to docs, That raises a good point. I guess when I added inspector I thought users just likely already know how to use it and that was a mistake on my part :( | 12:58 |
| sambetts | mgoddard: its because inspector calls the IPA inventory functions in a loop to wait for the interfaces to be up | 12:58 |
| mgoddard | dtantsur: one issue with local storage is that it won't scale to >1 inspector | 12:59 |
| *** mvpnitesh has quit IRC | 12:59 | |
| sambetts | mgoddard: its possible to scale it using an NFS mount to all the hosts running inspector | 12:59 |
| mgoddard | sambetts: true, but you might as well deploy swift at that point :) | 13:00 |
| TheJulia | heh | 13:00 |
| *** mat128 has joined #openstack-ironic | 13:00 | |
| dtantsur | mgoddard, hmm, does this nginx approach scale? | 13:00 |
| * dtantsur did not look in depth | 13:00 | |
| *** mvpnitesh has joined #openstack-ironic | 13:01 | |
| mgoddard | dtantsur: It scales to >1 inspectors, but not >1 nginx | 13:01 |
| TheJulia | There would need to be underlying configuration | 13:01 |
| TheJulia | such as an NFS volume supporting /httpboot | 13:01 |
| dtantsur | well, this does sound like an NFS share + local storage | 13:01 |
| *** vmud213 has quit IRC | 13:02 | |
| mgoddard | dtantsur: I'm not sure it's something you'd do in production anyway as there's no authentication | 13:02 |
| *** Goneri has joined #openstack-ironic | 13:04 | |
| mgoddard | sambetts: I only noticed the multiple LLDP passes because I failed to collect LLDP recently and noticed that it had 2 passes, succeeded once (during wait_for_interface / dhcp / something?) then failed when actually collecting the inventory | 13:04 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Test rolling upgrades https://review.openstack.org/458748 | 13:04 |
| mgoddard | sambetts: likely because my ipa lldp collection timeout == switch lldp transmit interval (30 seconds) | 13:04 |
| *** jcoufal has joined #openstack-ironic | 13:04 | |
| mgoddard | sambetts: although that is the default on both sides so may occasionally hit others | 13:05 |
| sambetts | mgoddard: there is glitchy logic in IPA regarding NICs being down etc too I have been slowly putting together a patch for it here https://review.openstack.org/#/c/434385/ | 13:06 |
| patchbot | patch 434385 - ironic-python-agent - [LLDP] Skip NICs that say they are ready but are u... | 13:06 |
| sambetts | mgoddard: but its by no means perfect yet | 13:07 |
| *** sasha2 has joined #openstack-ironic | 13:07 | |
| *** udesale__ has quit IRC | 13:07 | |
| *** ccamacho|lunch is now known as ccamacho | 13:07 | |
| *** lucas-hungry is now known as lucasagomes | 13:08 | |
| mgoddard | sambetts: I'll take a look | 13:09 |
| *** rpioso has joined #openstack-ironic | 13:10 | |
| * TheJulia wonders where her intel nuc ran off to | 13:12 | |
| mgoddard | TheJulia: back to docs, one key thing that's missing is an env-vars file that works for inspector or at least instructions on which environment variables to set. I'm using OS_TOKEN and OS_URL | 13:14 |
| mgoddard | TheJulia: you're right in that it shouldn't duplicate the inspector docs but could at least say how to access it | 13:15 |
| TheJulia | Well, OS_CLOUD would be easier for most users I think | 13:16 |
| TheJulia | since we are all moving to the osc cli | 13:16 |
| *** jaypipes is now known as leakypipes | 13:16 | |
| TheJulia | leakypipes: have you tried duct tape? | 13:17 |
| * TheJulia ducks | 13:17 | |
| leakypipes | TheJulia: unfortunately, yes :) | 13:17 |
| leakypipes | TheJulia: it's this DAMN Florida water. | 13:18 |
| mgoddard | TheJulia: perhaps, although I generally don't use it. How does that work in an environment without keystone with multiple endpoints (ironic & inspector)? | 13:18 |
| leakypipes | ;) | 13:18 |
| TheJulia | mgoddard: well, essentially the same way that env-vars uses, however osc seems super unfriendly to that path. using clouds.yaml a user could still define a "cloud" with the individual resources and select the one they want. Or.... they could re-run install and turn on keystone. | 13:20 |
| * TheJulia takes the nuc with far too many cables hanging off of it into the back room to build a fresh install. | 13:22 | |
| *** mvpnitesh has quit IRC | 13:24 | |
| mgoddard | TheJulia: I see, multiple clouds. I suppose in an keystone-less environment bifrost could generate a clouds.yml as well as/instead of env-vars | 13:26 |
| *** d0ugal_ has quit IRC | 13:26 | |
| *** d0ugal has joined #openstack-ironic | 13:27 | |
| *** d0ugal has quit IRC | 13:27 | |
| *** d0ugal has joined #openstack-ironic | 13:27 | |
| *** bcornec has quit IRC | 13:30 | |
| *** mvpnitesh has joined #openstack-ironic | 13:30 | |
| TheJulia | mgoddard: yup, I think we already have a role for writing the file, I think the pertinent data needs to be passed in | 13:32 |
| mgoddard | TheJulia: oh ok, missed that. I'll take a look | 13:32 |
| TheJulia | mgoddard: locally, I've been using keystone as of recent, but I've been mucking with neutron integration, so otherwise I'd go crazy | 13:33 |
| *** mvpnitesh has quit IRC | 13:37 | |
| *** xavierr has joined #openstack-ironic | 13:37 | |
| *** mvpnitesh has joined #openstack-ironic | 13:37 | |
| mgoddard | TheJulia: makes sense. | 13:38 |
| *** adisky_ has quit IRC | 13:39 | |
| TheJulia | s/crazy/more crazy/ | 13:39 |
| *** srobert has joined #openstack-ironic | 13:43 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Handle some ironic inspector HTTP 500 errors https://review.openstack.org/461036 | 13:44 |
| *** mat128 has quit IRC | 13:45 | |
| *** jcoufal_ has joined #openstack-ironic | 13:48 | |
| *** jcoufal has quit IRC | 13:50 | |
| *** l_jacob has joined #openstack-ironic | 13:50 | |
| *** chlong has joined #openstack-ironic | 13:53 | |
| openstackgerrit | M V P Nitesh proposed openstack/ironic master: Allow setting {provisioning,cleaning}_network in node driver_info https://review.openstack.org/460564 | 13:53 |
| *** l_jacob has quit IRC | 13:58 | |
| *** mvpnitesh has quit IRC | 13:58 | |
| *** baoli has joined #openstack-ironic | 13:59 | |
| *** makowals has quit IRC | 14:01 | |
| *** crushil has joined #openstack-ironic | 14:01 | |
| *** chlong has quit IRC | 14:07 | |
| *** Serlex has joined #openstack-ironic | 14:10 | |
| *** jaosorior_away has quit IRC | 14:10 | |
| TheJulia | Note to self, never copy huge set of disk images over wifi again :( | 14:12 |
| sambetts | :( | 14:14 |
| *** baha has joined #openstack-ironic | 14:18 | |
| *** aarefiev is now known as aarefiev_afk | 14:19 | |
| NobodyCam | Good Morning Ironic'ers ... and ... | 14:20 |
| NobodyCam | TGIF! | 14:20 |
| *** chlong has joined #openstack-ironic | 14:20 | |
| *** gcb has joined #openstack-ironic | 14:24 | |
| TheJulia | Good morning NobodyCam | 14:25 |
| dtantsur | morning NobodyCam | 14:25 |
| NobodyCam | Good Morning TheJulia & dtantsur happy friday. | 14:25 |
| dtantsur | indeed :) | 14:26 |
| NobodyCam | :) | 14:26 |
| *** causten has joined #openstack-ironic | 14:43 | |
| *** gcb has quit IRC | 14:48 | |
| *** gcb has joined #openstack-ironic | 14:49 | |
| *** dprince has quit IRC | 14:53 | |
| *** amarao has quit IRC | 14:55 | |
| *** bnemec is now known as beekneemech | 14:59 | |
| *** aarefiev_afk has quit IRC | 14:59 | |
| *** dprince has joined #openstack-ironic | 15:06 | |
| lucasagomes | dtantsur, +a the redfish patch | 15:07 |
| dtantsur | nice, thanks! | 15:09 |
| *** dfisher has joined #openstack-ironic | 15:14 | |
| dfisher | good morning! I've been working on getting devstack + Ironic up and running and was hitting all kinds of issues on Oracle Linux 7.3 with just getting the plugin packages installed. I had to edit ironic/devstack/files/rpms/ironic and change docker-io to docker.io (like in debs/ironic) to get devstack to proceed. | 15:15 |
| dfisher | is this a known issue? | 15:16 |
| *** openstackgerrit has quit IRC | 15:17 | |
| TheJulia | do the package names just differ between Centos/EPEL/Fedora and Oracle Linux? | 15:18 |
| dfisher | I'm honestly unsure how to check :( I'm actually a solaris guy! | 15:18 |
| dfisher | i know, there's like … 3 of us! | 15:18 |
| * TheJulia looks it up | 15:18 | |
| JayF | Does anyone know of a good trick to bulk-unsubscribe from gerrit emails? :/ | 15:18 |
| JayF | good morning :) | 15:19 |
| TheJulia | dfisher: docker-io is the package name in fedora, although it looks like it is deprecated/outdated | 15:20 |
| TheJulia | JayF: Did you change your settings in gerrit? | 15:20 |
| dfisher | TheJulia: where did you search for that (so I can do all the digging on my own next time) | 15:20 |
| JayF | TheJulia: oh, is there a setting to disable emails? | 15:20 |
| TheJulia | eh, notifiation preferences if memory serves | 15:20 |
| JayF | TheJulia: I'd probably prefer to unsub from all things I'm listed as a reviewer on, so if I post one in the future I'd still get notifies | 15:20 |
| JayF | but that should qualify as Good Enough(tm) | 15:21 |
| TheJulia | JayF: *nods* | 15:21 |
| TheJulia | dfisher: https://admin.fedoraproject.org/pkgdb/package/rpms/docker-io/ | 15:21 |
| * TheJulia sighs | 15:21 | |
| dfisher | so, it looks like it was simply renamed to 'docker' ? | 15:21 |
| mariojv | dtantsur: hi, i left a comment on https://review.openstack.org/#/c/460086/ - please do feel free to remove my name from priorities | 15:24 |
| patchbot | patch 460086 - ironic-specs - Remove Pike priorities that cannot make it due to ... | 15:24 |
| mariojv | i will send out an official goodbye to mailing list later today | 15:24 |
| mariojv | happy to answer any questions about existing work for whomever picks it up though | 15:24 |
| JayF | As a note: when we get our official ironic bear stickers | 15:25 |
| JayF | someone had damn well better mail me a half dozen ;) | 15:25 |
| dtantsur | :) | 15:26 |
| TheJulia | dfisher: so ubuntu has a docker and docker.io, so I'm really not sure what the difference is off-hand | 15:26 |
| dfisher | but ubuntu/debian stuff is in files/debs/ironic | 15:26 |
| dfisher | right? | 15:26 |
| TheJulia | JayF: note taken :) | 15:26 |
| *** pcaruana has quit IRC | 15:28 | |
| TheJulia | dfisher: indeed, dependencies are likely expanding. I'd have to dig into the dependencies and all | 15:29 |
| dfisher | so, i don't think we should worry about debs. | 15:29 |
| dfisher | probably just need to look at centos/fedora/other rpm things | 15:29 |
| *** Linkmark has quit IRC | 15:30 | |
| dtantsur | mariojv, thanks for update! | 15:30 |
| dtantsur | and good luck :) | 15:30 |
| TheJulia | dfisher: search for docker and docker-io on rpmfind | 15:31 |
| dfisher | Found 154 RPM for docker | 15:31 |
| dfisher | ugh. | 15:31 |
| *** openstackgerrit has joined #openstack-ironic | 15:31 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-specs master: Remove Pike priorities that cannot make it due to recent team changes https://review.openstack.org/460086 | 15:31 |
| dtantsur | new update ^^^ please review | 15:31 |
| dfisher | 0 for docker.io, 2 for docker-io | 15:31 |
| dfisher | docker-io is EL 6 only though | 15:32 |
| *** chlong has quit IRC | 15:34 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Handle some ironic inspector HTTP 500 errors https://review.openstack.org/461036 | 15:35 |
| TheJulia | dfisher: yay for inconsistent package naming \o/ | 15:35 |
| dfisher | yay! | 15:36 |
| * TheJulia fires up bifrost's install playbook on the computer normally attached to the TV... for reasons. | 15:37 | |
| *** WeiAA has quit IRC | 15:38 | |
| *** yolanda has joined #openstack-ironic | 15:43 | |
| *** ipukha has quit IRC | 15:45 | |
| *** vinsh has quit IRC | 15:48 | |
| *** g3ek has joined #openstack-ironic | 16:04 | |
| *** avladu has quit IRC | 16:04 | |
| *** trown is now known as trown|lunch | 16:07 | |
| *** racedo has quit IRC | 16:10 | |
| *** athomas has quit IRC | 16:11 | |
| *** derekh has quit IRC | 16:15 | |
| *** chlong has joined #openstack-ironic | 16:16 | |
| *** jcoufal has joined #openstack-ironic | 16:20 | |
| dtantsur | removing 2 cores in one day is a sad feeling :( especially when you know that you have to remove 1 or 2 more soon | 16:20 |
| dtantsur | changing the PTL did not bring you luck, I need to say | 16:21 |
| *** jcoufal__ has joined #openstack-ironic | 16:22 | |
| *** jcoufal_ has quit IRC | 16:22 | |
| *** jcoufal has quit IRC | 16:24 | |
| *** baha has quit IRC | 16:32 | |
| *** jcoufal has joined #openstack-ironic | 16:33 | |
| *** jcoufal__ has quit IRC | 16:34 | |
| JayF | TheJulia: I couldn't find an option in gerrit to disable notifications | 16:35 |
| TheJulia | JayF: I thought you could do it on a repo level | 16:38 |
| JayF | If you watch a repo, yes, but apparently it's not trivial to unsub from the actual patchsets | 16:39 |
| JayF | see -infra | 16:39 |
| TheJulia | :( | 16:39 |
| dtantsur | Nearly beer'o'clock for me. It's a public holiday on Monday, but I may show up for the meeting. If not - please have it anyway :) | 16:44 |
| dtantsur | See you next week, and have a great weekend! | 16:44 |
| *** dtantsur is now known as dtantsur|afk | 16:45 | |
| *** tesseract has quit IRC | 16:45 | |
| *** jcoufal_ has joined #openstack-ironic | 16:47 | |
| *** ildikov is now known as coffee_cat | 16:47 | |
| *** jcoufal has quit IRC | 16:50 | |
| openstackgerrit | Merged openstack/ironic master: Bring the redfish driver address parameter closer to one of other drivers https://review.openstack.org/461008 | 16:51 |
| *** lucasagomes is now known as lucas-afk | 16:54 | |
| *** ijw has joined #openstack-ironic | 16:54 | |
| jrist | hey my dudes | 16:55 |
| jrist | can we get some eyes on https://review.openstack.org/#/c/460353/ plz | 16:55 |
| patchbot | patch 460353 - ironic - Properly allow Ironic headers in REST API | 16:55 |
| jrist | it is effecting UI | 16:55 |
| *** ijw has quit IRC | 16:56 | |
| *** ijw has joined #openstack-ironic | 16:56 | |
| *** baha has joined #openstack-ironic | 17:01 | |
| *** ijw has quit IRC | 17:01 | |
| *** trown|lunch is now known as trown | 17:02 | |
| *** Sukhdev has joined #openstack-ironic | 17:05 | |
| *** harlowja has quit IRC | 17:06 | |
| *** jcoufal has joined #openstack-ironic | 17:07 | |
| *** e0ne has quit IRC | 17:08 | |
| *** jcoufal_ has quit IRC | 17:09 | |
| *** ChubYann has joined #openstack-ironic | 17:23 | |
| *** gcb has quit IRC | 17:24 | |
| *** amotoki has quit IRC | 17:25 | |
| *** persia has quit IRC | 17:30 | |
| *** ijw has joined #openstack-ironic | 17:37 | |
| TheJulia | dtantsur|afk: 10-4, Thank you and have a wonderful weekend! | 17:38 |
| *** ijw has quit IRC | 17:42 | |
| *** sambetts is now known as sambetts|afk | 17:47 | |
| sambetts|afk | Night all | 17:47 |
| TheJulia | goodnight sambetts|afk | 17:48 |
| anupn | Hi Ironic'ers, any idea how to access the json data sent in POST request? I am trying to access the data sent in request body using pecan while creating port via POST. Right now we pass "body=Port" and so we are getting the fields exposed in init | 17:49 |
| TheJulia | anupn: I thought that is only a put in ironic's api | 17:51 |
| * TheJulia might be totally wrong though | 17:51 | |
| anupn | Hi TheJulia, no POST also use body=Port | 17:52 |
| anupn | TheJulia, this is wrt my patch https://review.openstack.org/#/c/405586/ in which i want to check if node_name and node_uuid are both passed in the request it should throw error | 17:53 |
| patchbot | patch 405586 - ironic - Allow logical name in port creation | 17:53 |
| anupn | TheJulia, we get the data in args but for that args need to be passed "body=args" | 17:54 |
| *** causten has quit IRC | 17:56 | |
| *** e0ne has joined #openstack-ironic | 17:57 | |
| *** causten has joined #openstack-ironic | 17:58 | |
| TheJulia | Hmm, yeah, it is a post, for some reason I was thinking it was a put | 18:01 |
| *** trown is now known as trown|brb | 18:02 | |
| anupn | TheJulia, any way you think i can access the data keeping body=Port as is? | 18:03 |
| Nisha_Agarwal | JayF, hi | 18:03 |
| JayF | o/ | 18:04 |
| Nisha_Agarwal | JayF, i wanted to know the status of python 3.5 compatibility work | 18:04 |
| JayF | There was zero done on my end | 18:04 |
| Nisha_Agarwal | means from where should i pick it up | 18:04 |
| Nisha_Agarwal | so its not even started? | 18:04 |
| JayF | I was told we have a py35 job running in experimental, but I think there was a concern it wasn't actually running py35 | 18:04 |
| JayF | yep, none of that work has begun at all | 18:04 |
| Nisha_Agarwal | Ok so ideally we want to get that done for ironic module, correct? | 18:05 |
| Nisha_Agarwal | or its scope is bigger than that? | 18:05 |
| JayF | The scope of the work is up to the remaining team, but it's an OpenStack priority. | 18:05 |
| *** ijw has joined #openstack-ironic | 18:05 | |
| Nisha_Agarwal | JayF, :) | 18:05 |
| JayF | My exact goal was a voting dsvm job running py35 | 18:05 |
| JayF | which would mean the client, and ironic would work | 18:05 |
| JayF | no idea about if we want to include IPA (should just work out of the box) and/or inspector | 18:05 |
| Nisha_Agarwal | JayF, ok | 18:06 |
| Nisha_Agarwal | i think we would need to include IPA and ironic-lib atleast | 18:06 |
| Nisha_Agarwal | inspector i am not so sure | 18:06 |
| *** yolanda has quit IRC | 18:07 | |
| Nisha_Agarwal | but once we start on this, it should be possible to do for inspector also i guess | 18:07 |
| JayF | Yeah, like I said, you all get to decide what that means now. I probably would've considered inspector a 'nice to have' item | 18:07 |
| Nisha_Agarwal | JayF, :) | 18:07 |
| JayF | but it may be as simple as enabling the dsvm job and watching it pass, once the prerequisites for ironic are done | 18:07 |
| Nisha_Agarwal | ok. i read in mail you would be still available on IRC. How long is that expected? :) | 18:08 |
| Nisha_Agarwal | actually i am on FTO from May5-May22, so asking in case i need any help later will you be still there after May22 | 18:09 |
| Nisha_Agarwal | JayF, i had one query on rescue also | 18:10 |
| JayF | I'm going to be on freenode IRC for either my or freenode's lifetime, I suspect :P | 18:11 |
| Nisha_Agarwal | :) | 18:11 |
| JayF | I have no intentions of leaving here, but won't be available for more than a couple of quick questions occassionally | 18:11 |
| *** ijw has quit IRC | 18:11 | |
| JayF | and the TTL on my memory is pretty awful as well, haha | 18:11 |
| Nisha_Agarwal | :) | 18:12 |
| Nisha_Agarwal | JayF, how much work is left on rescue side? Is there anything which is not raised yet? As i understand from aparnav all the patches are raised and just review comments need to be addressed | 18:13 |
| Nisha_Agarwal | is my understanding correct? | 18:13 |
| JayF | mariojv would be more up to date on that as I was on vacation before I stopped working on Ironic | 18:13 |
| Nisha_Agarwal | :) | 18:14 |
| JayF | but AIUI it needed CI completed and workign | 18:14 |
| JayF | and the agent support for static networking is 100% undone | 18:14 |
| JayF | so the part that is working is limited to DHCP networks. | 18:14 |
| Nisha_Agarwal | ok agent part was unknown to me | 18:14 |
| Nisha_Agarwal | Ci is the target for us also | 18:14 |
| Nisha_Agarwal | :) | 18:15 |
| Nisha_Agarwal | Nova side work is started? | 18:15 |
| anupn | Nisha_Agarwal, I have some notes collected for static networking for agent, if you want I can update it clean :) | 18:15 |
| Nisha_Agarwal | for rescue | 18:15 |
| JayF | the nova patch is pretty close to done | 18:15 |
| JayF | yeah, anupn is a good person to chat with as well | 18:15 |
| JayF | he knows what's up with rescue too | 18:15 |
| JayF | anupn: you going to continue working on Ironic? | 18:15 |
| JayF | actually I think the nova patch, in order to work "properly" | 18:16 |
| anupn | JayF, :) Yup but not sure how much on rescue | 18:16 |
| JayF | has to land after the client work is done | 18:16 |
| *** yolanda has joined #openstack-ironic | 18:16 | |
| JayF | with a bump to the ironicclient version with the rescue support | 18:16 |
| Nisha_Agarwal | anupn, thanks. I would ask aparnav/stendulker to contact you. I am just taking update to understand how much is done and how much is left | 18:16 |
| JayF | If I were you, I'd focus 100% on the ironic side, including client, until that's done. Then worry about nova. | 18:16 |
| JayF | Nova won't land it until all the ironic work is done anyway. | 18:16 |
| Nisha_Agarwal | JayF, :) ok | 18:17 |
| anupn | JayF, if someone is taking up other patches, i can look into tempest patches, I volunteered for that in upstream meeting :) | 18:18 |
| anupn | Nisha_Agarwal, sure | 18:18 |
| JayF | anupn: it seems like Nisha_Agarwal is taking it over, she'd probably appreciate the help -- plus you can share some of the etherpad notes we all used to give a good handoff on the rescue work | 18:18 |
| *** vinsh has joined #openstack-ironic | 18:18 | |
| Nisha_Agarwal | JayF, anupn that would be great | 18:19 |
| *** dprince has quit IRC | 18:19 | |
| anupn | Nisha_Agarwal, JayF, yeah. Will clean up the etherpad with some detailed steps and hover you Nisha_Agarwal | 18:19 |
| Nisha_Agarwal | anupn, thanks | 18:20 |
| mariojv | yes, tempest still needs to pass | 18:20 |
| mariojv | tests are up but not passing yet | 18:20 |
| mariojv | anupn was working on it, but it seems y'all already know that | 18:20 |
| anupn | Nisha_Agarwal, it would be good to check with dtantsur|afk as someone has been signed up for rescue | 18:20 |
| anupn | in virtual meetup that happened | 18:21 |
| anupn | Nisha_Agarwal, also if you are gonna take it over may be it can be brought in the priorities | 18:22 |
| Nisha_Agarwal | anupn, wanyen signed up for rescue and now stendulker/aparnav going to take care of it. We all work in single team | 18:22 |
| anupn | Nisha_Agarwal, ah i see | 18:22 |
| anupn | Nisha_Agarwal, cool | 18:23 |
| Nisha_Agarwal | anupn, :) | 18:23 |
| Nisha_Agarwal | anupn, it would help if you can share the notes for rescue | 18:23 |
| Nisha_Agarwal | when you are done with updating them | 18:24 |
| anupn | Nisha_Agarwal, yes :) | 18:25 |
| Nisha_Agarwal | anupn, thanks :) | 18:25 |
| anupn | Nisha_Agarwal, np | 18:25 |
| TheJulia | anupn: I might be misunderstanding what your trying to do, but pdict in post() under the port controller seems to be what you want. FWIW, it seems like the uuid should always take precedence | 18:26 |
| *** vinsh has quit IRC | 18:28 | |
| *** trown|brb is now known as trown | 18:29 | |
| anupn | TheJulia, pdict will give me all that is exposed in init(), that includes both node_name and node_uuid. Is it always that we give uuid a precedence? | 18:31 |
| TheJulia | anupn: my $0.02 is that we should, although that could also be considered a bug. | 18:34 |
| anupn | TheJulia, :) just concerned as there should not be a conflict and also was asked on my patch set to take care of that, so if uuid is always a priority which I believe should be then i will include that in reno or commit msg | 18:34 |
| TheJulia | Then it makes sense that you do so | 18:35 |
| TheJulia | anupn: if the values are there, from init, what if you check to see if the value has a length in the dictionary? | 18:36 |
| *** harlowja has joined #openstack-ironic | 18:38 | |
| anupn | TheJulia, but init() will fetch both corresponding values even if either node_uuid or node_name are passed because we are appending the fields before calling setattr from init() | 18:39 |
| anupn | TheJulia, https://review.openstack.org/#/c/405586/22/ironic/api/controllers/v1/port.py@193 | 18:40 |
| patchbot | patch 405586 - ironic - Allow logical name in port creation | 18:40 |
| anupn | TheJulia, so length of that field will not be zero | 18:40 |
| TheJulia | the length of the node_name in the posted dictionary? | 18:41 |
| TheJulia | the value that is | 18:42 |
| anupn | TheJulia, yes the value of node_name would be correct node name value in the pdict that we get | 18:42 |
| anupn | TheJulia, you meant pdict, if i am not wrong? | 18:43 |
| TheJulia | well, pdict is a dictionary of post data, so yes :) | 18:44 |
| anupn | TheJulia, haha, sometimes the names get misleading so confirmed :) | 18:45 |
| anupn | TheJulia, I am trying to see if I can make use of pecan.request object but I don't find the method to access the json data sent | 18:46 |
| TheJulia | Sadly no, your going to get the slightly re-parsed data passed in via pecan, hence why I'm thinking just check the length of the data to see if any data was actually posted, since "" could have been sent along. | 18:49 |
| *** ijw has joined #openstack-ironic | 18:51 | |
| *** rbartal has joined #openstack-ironic | 18:52 | |
| *** causten has quit IRC | 18:52 | |
| anupn | TheJulia, I see, but in pdict i get node_uuid, node_name, address and node_id with correct values and no empty value for node_name | 18:55 |
| TheJulia | so if your not submitting node_name, is the value being populated for you by pecan? | 18:56 |
| anupn | TheJulia, no in request say I passed node_uuid but then a call to "port.as_dict()" will also call setters for all the fields that are added in self.fields and in my init() i am adding node_name as well in self.fields | 18:58 |
| *** e0ne has quit IRC | 18:58 | |
| *** causten has joined #openstack-ironic | 18:59 | |
| *** xavierr has quit IRC | 19:02 | |
| *** Goneri has quit IRC | 19:03 | |
| TheJulia | So you'll never know if it is actually sent, but if there is a value perhaps? | 19:04 |
| anupn | yes, and hence want to access data that is actually sent and not use pdict. Atleast to know whether both are sent or not | 19:05 |
| anupn | like as done for PATCH we are using PortPatchType that gives exact data sent in the request | 19:06 |
| TheJulia | So assuming we can't do the exact thing that was asked short of refactoring the entire post logic, what do you see as the shortest path to resolution? | 19:09 |
| *** rbartal has quit IRC | 19:10 | |
| anupn | TheJulia, I think what i can do is pass args to body, body=args and the where we are getting pdict, change it to "pdict=Port.as_dict()" | 19:13 |
| anupn | *and where | 19:14 |
| TheJulia | If that works | 19:15 |
| TheJulia | I'm not sure it is going to work, but I've largely avoided the API code | 19:16 |
| jroll | TheJulia: good decision :P | 19:19 |
| TheJulia | jroll: It was the "There be dragons!" sign | 19:20 |
| jroll | heh | 19:20 |
| anupn | TheJulia, the more shorter is mention in reno that node_uuid will be a priority, lol | 19:22 |
| * anupn joking | 19:22 | |
| anupn | jroll, hehe | 19:22 |
| TheJulia | anupn: so how might you have the desired behavior when you will always have keys provided to the post method? | 19:27 |
| *** Nisha_Agarwal has quit IRC | 19:29 | |
| anupn | TheJulia, actually i am thinking to make a change in init method, right now setattr inside init() fetch values from kwargs passed to init and kwargs should not have node_name as it is not passed in the POST request body I am digging how kwargs is getting node_name | 19:31 |
| *** kjw3 has joined #openstack-ironic | 19:32 | |
| *** g3ek has quit IRC | 19:33 | |
| *** chas has quit IRC | 19:34 | |
| *** ijw has quit IRC | 19:39 | |
| *** g3ek has joined #openstack-ironic | 19:40 | |
| *** Serlex has quit IRC | 19:41 | |
| *** ijw has joined #openstack-ironic | 19:41 | |
| *** ijw has quit IRC | 19:47 | |
| *** chas has joined #openstack-ironic | 19:54 | |
| anupn | TheJulia, Ah i figured out, in set_node_uuid we are also setting node_name to avoid multiple db access. And hence even if you pass only node_uuid in request we get correct value for node_name in pdict | 19:57 |
| anupn | TheJulia, let me make a change there, and then pdict will not have node_name :) | 19:58 |
| *** chas has quit IRC | 19:59 | |
| *** jcoufal_ has joined #openstack-ironic | 20:02 | |
| *** jcoufal has quit IRC | 20:05 | |
| jlvillal | jroll Do you know the status of the Ironic Neutron meeting? Does it happen anymore? | 20:10 |
| TheJulia | jlvillal: afaik, no | 20:11 |
| jlvillal | TheJulia, That is what I thought. | 20:11 |
| TheJulia | The slot, though, is still reserved | 20:11 |
| jroll | it does not | 20:11 |
| jlvillal | jroll, Thanks | 20:12 |
| jroll | np | 20:13 |
| *** chas has joined #openstack-ironic | 20:15 | |
| *** kjw3 has quit IRC | 20:16 | |
| *** morgabra has left #openstack-ironic | 20:17 | |
| *** causten has quit IRC | 20:18 | |
| *** causten has joined #openstack-ironic | 20:18 | |
| *** crushil has quit IRC | 20:18 | |
| *** crushil has joined #openstack-ironic | 20:19 | |
| *** chas has quit IRC | 20:20 | |
| *** jcoufal has joined #openstack-ironic | 20:22 | |
| *** causten has quit IRC | 20:22 | |
| *** jcoufal_ has quit IRC | 20:23 | |
| *** bfournie has quit IRC | 20:27 | |
| *** causten has joined #openstack-ironic | 20:28 | |
| *** mgoddard has quit IRC | 20:36 | |
| *** chas has joined #openstack-ironic | 20:36 | |
| *** chas has quit IRC | 20:41 | |
| *** ijw has joined #openstack-ironic | 20:43 | |
| *** causten has quit IRC | 20:47 | |
| *** jcoufal_ has joined #openstack-ironic | 20:49 | |
| *** jcoufal has quit IRC | 20:50 | |
| *** ijw has quit IRC | 20:51 | |
| *** causten has joined #openstack-ironic | 20:51 | |
| *** crushil has quit IRC | 20:56 | |
| *** chas has joined #openstack-ironic | 20:57 | |
| *** rpioso has quit IRC | 21:00 | |
| *** chas has quit IRC | 21:01 | |
| *** Sukhdev has quit IRC | 21:03 | |
| *** jcoufal has joined #openstack-ironic | 21:04 | |
| *** jcoufal_ has quit IRC | 21:05 | |
| *** dfisher has quit IRC | 21:09 | |
| *** ijw has joined #openstack-ironic | 21:09 | |
| *** trown is now known as trown|outtypewww | 21:10 | |
| *** ijw has quit IRC | 21:10 | |
| *** ijw has joined #openstack-ironic | 21:10 | |
| *** crushil has joined #openstack-ironic | 21:11 | |
| *** Goneri has joined #openstack-ironic | 21:13 | |
| *** harlowja has quit IRC | 21:17 | |
| *** harlowja has joined #openstack-ironic | 21:17 | |
| *** chas has joined #openstack-ironic | 21:18 | |
| *** ijw has quit IRC | 21:20 | |
| *** chas has quit IRC | 21:23 | |
| jrist | thanks TheJulia | 21:23 |
| *** srobert has quit IRC | 21:27 | |
| *** jlvillal has left #openstack-ironic | 21:30 | |
| *** jlvillal has joined #openstack-ironic | 21:30 | |
| *** baha1 has joined #openstack-ironic | 21:30 | |
| *** crushil has quit IRC | 21:32 | |
| *** baha has quit IRC | 21:33 | |
| *** ijw has joined #openstack-ironic | 21:34 | |
| *** ijw has quit IRC | 21:34 | |
| *** ijw has joined #openstack-ironic | 21:34 | |
| *** baha1 has quit IRC | 21:35 | |
| *** harlowja has quit IRC | 21:36 | |
| *** rpioso has joined #openstack-ironic | 21:36 | |
| *** bfournie has joined #openstack-ironic | 21:37 | |
| *** chlong has quit IRC | 21:39 | |
| *** chas has joined #openstack-ironic | 21:39 | |
| *** fragatin_ has joined #openstack-ironic | 21:41 | |
| *** fragatina has quit IRC | 21:43 | |
| *** chas has quit IRC | 21:43 | |
| *** causten has quit IRC | 21:43 | |
| *** causten has joined #openstack-ironic | 21:44 | |
| *** causten has quit IRC | 21:46 | |
| *** causten has joined #openstack-ironic | 21:47 | |
| *** ijw has quit IRC | 21:47 | |
| *** ijw has joined #openstack-ironic | 21:48 | |
| *** ijw has quit IRC | 21:48 | |
| *** ijw has joined #openstack-ironic | 21:48 | |
| *** baoli has quit IRC | 21:49 | |
| *** jcoufal_ has joined #openstack-ironic | 21:49 | |
| *** jcoufal has quit IRC | 21:50 | |
| *** causten has quit IRC | 21:54 | |
| *** causten has joined #openstack-ironic | 21:57 | |
| *** cmurphy has quit IRC | 21:59 | |
| *** chas has joined #openstack-ironic | 22:00 | |
| *** jcoufal has joined #openstack-ironic | 22:03 | |
| *** cmurphy has joined #openstack-ironic | 22:04 | |
| *** chas has quit IRC | 22:04 | |
| *** jcoufal_ has quit IRC | 22:05 | |
| *** rpioso has quit IRC | 22:15 | |
| *** chas has joined #openstack-ironic | 22:21 | |
| *** jcoufal has quit IRC | 22:22 | |
| *** mgoddard has joined #openstack-ironic | 22:23 | |
| *** chas has quit IRC | 22:26 | |
| *** causten has quit IRC | 22:27 | |
| *** causten has joined #openstack-ironic | 22:27 | |
| *** ijw has quit IRC | 22:28 | |
| *** JayF has quit IRC | 22:29 | |
| *** jkilpatr has quit IRC | 22:30 | |
| *** JayF has joined #openstack-ironic | 22:37 | |
| *** JayF has joined #openstack-ironic | 22:37 | |
| *** mgoddard has quit IRC | 22:37 | |
| *** baoli has joined #openstack-ironic | 22:45 | |
| *** baoli has quit IRC | 22:45 | |
| *** baoli has joined #openstack-ironic | 22:46 | |
| *** crushil has joined #openstack-ironic | 22:46 | |
| *** ijw has joined #openstack-ironic | 22:51 | |
| *** Sukhdev has joined #openstack-ironic | 22:53 | |
| *** ijw has quit IRC | 22:56 | |
| *** rama_y has quit IRC | 22:58 | |
| *** chas has joined #openstack-ironic | 23:02 | |
| *** jralph has quit IRC | 23:03 | |
| *** chas has quit IRC | 23:07 | |
| *** ijw has joined #openstack-ironic | 23:09 | |
| *** jralph has joined #openstack-ironic | 23:11 | |
| *** rama_y has joined #openstack-ironic | 23:11 | |
| *** ijw has quit IRC | 23:13 | |
| *** chas has joined #openstack-ironic | 23:23 | |
| *** chas has quit IRC | 23:28 | |
| *** harlowja has joined #openstack-ironic | 23:30 | |
| *** chas has joined #openstack-ironic | 23:44 | |
| *** chas has quit IRC | 23:48 | |
| *** crushil has quit IRC | 23:50 | |
| *** pmannidi has joined #openstack-ironic | 23:51 | |
| *** harlowja has quit IRC | 23:55 | |
| *** jkilpatr has joined #openstack-ironic | 23:56 | |
| *** slagle has quit IRC | 23:57 | |
| *** harlowja has joined #openstack-ironic | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!