*** hoonetorg has joined #openstack-ironic | 00:03 | |
*** joe817 has quit IRC | 00:04 | |
*** yuanying_ has joined #openstack-ironic | 00:05 | |
*** joe817 has joined #openstack-ironic | 00:06 | |
*** rama_y_ has quit IRC | 00:06 | |
*** yuanying has quit IRC | 00:06 | |
*** Nisha_ has joined #openstack-ironic | 00:07 | |
*** Nisha_brb has quit IRC | 00:07 | |
*** yuanying has joined #openstack-ironic | 00:31 | |
*** yuanying_ has quit IRC | 00:31 | |
*** masber has joined #openstack-ironic | 00:35 | |
masber | good morning, I would like to start using ironic, is there a guide to start using it? | 00:36 |
---|---|---|
masber | like s starting guide or something like that? I already have it installed through project kolla-ansible | 00:39 |
*** ijw has joined #openstack-ironic | 00:46 | |
*** yuanying has quit IRC | 00:49 | |
*** yuanying has joined #openstack-ironic | 01:02 | |
*** litao__ has joined #openstack-ironic | 01:25 | |
*** makowals has quit IRC | 01:38 | |
*** makowals has joined #openstack-ironic | 01:39 | |
*** rcarrillocruz has quit IRC | 01:42 | |
*** ijw has quit IRC | 01:43 | |
*** yuanying has quit IRC | 01:57 | |
*** yuanying has joined #openstack-ironic | 02:00 | |
*** rbudden has quit IRC | 02:16 | |
*** rcarrillocruz has joined #openstack-ironic | 02:17 | |
openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic master: Add functional API tests for volume connector and volume target https://review.openstack.org/473717 | 02:20 |
*** Nisha_brb has joined #openstack-ironic | 02:43 | |
*** Nisha_ has quit IRC | 02:43 | |
*** udesale has joined #openstack-ironic | 02:43 | |
*** Nisha_ has joined #openstack-ironic | 02:51 | |
*** Nisha_brb has quit IRC | 02:51 | |
openstackgerrit | Nisha Agarwal proposed openstack/sushy master: Adds EthernetInterface to the library https://review.openstack.org/452532 | 03:01 |
openstackgerrit | Tuan Luong-Anh proposed openstack/python-ironicclient master: Rearrange existing documentation to fit the new standard layout https://review.openstack.org/480013 | 03:06 |
*** kiennt has joined #openstack-ironic | 03:10 | |
*** d0ugal has quit IRC | 03:19 | |
*** d0ugal_ has joined #openstack-ironic | 03:19 | |
*** rama_y has joined #openstack-ironic | 03:26 | |
openstackgerrit | Dao Cong Tien proposed openstack/ironic master: Tempest scenario test for boot-from-volume https://review.openstack.org/472740 | 03:29 |
openstackgerrit | Dao Cong Tien proposed openstack/ironic master: WIP/DNM:Temp patch to test boot-from-volume job https://review.openstack.org/476854 | 03:29 |
openstackgerrit | Merged openstack/proliantutils master: Using assertIsNone() instead of assertEqual(None) https://review.openstack.org/480038 | 03:48 |
openstackgerrit | Aparna proposed openstack/proliantutils master: Redfish: Add support for Virtual Media Operations https://review.openstack.org/473798 | 03:50 |
openstackgerrit | Tuan Luong-Anh proposed openstack/python-ironicclient master: Rearrange existing documentation to fit the new standard layout https://review.openstack.org/480013 | 03:59 |
openstackgerrit | Nisha Agarwal proposed openstack/proliantutils master: WIP: Add EthernetInterface to proliantutils https://review.openstack.org/480824 | 04:01 |
*** sasha21 has quit IRC | 04:05 | |
*** links has joined #openstack-ironic | 04:05 | |
*** nicodemos has quit IRC | 04:14 | |
*** sauloaislan has quit IRC | 04:14 | |
*** xavierr has quit IRC | 04:17 | |
*** ricardoas has quit IRC | 04:17 | |
*** ricardoas has joined #openstack-ironic | 04:17 | |
*** mrtenio|afk has quit IRC | 04:17 | |
*** mrtenio|afk has joined #openstack-ironic | 04:18 | |
*** sauloaislan has joined #openstack-ironic | 04:18 | |
*** nicodemos has joined #openstack-ironic | 04:18 | |
*** fragatin_ has joined #openstack-ironic | 04:22 | |
*** fragatina has quit IRC | 04:26 | |
*** fragatin_ has quit IRC | 04:26 | |
*** fragatina has joined #openstack-ironic | 04:32 | |
*** adisky__ has joined #openstack-ironic | 04:35 | |
*** fragatina has quit IRC | 04:36 | |
*** nicodemos has quit IRC | 04:46 | |
*** sauloaislan has quit IRC | 04:47 | |
*** ricardoas has quit IRC | 04:47 | |
*** mrtenio|afk has quit IRC | 04:47 | |
*** mrtenio|afk has joined #openstack-ironic | 04:48 | |
*** nicodemos has joined #openstack-ironic | 04:49 | |
*** ricardoas has joined #openstack-ironic | 04:49 | |
*** sauloaislan has joined #openstack-ironic | 04:49 | |
*** xavierr has joined #openstack-ironic | 04:54 | |
*** anish_ has joined #openstack-ironic | 05:17 | |
*** adisky___ has joined #openstack-ironic | 05:17 | |
*** mvpnitesh has joined #openstack-ironic | 05:18 | |
*** harlowja has quit IRC | 05:22 | |
*** adisky__ has quit IRC | 05:22 | |
*** anish has quit IRC | 05:22 | |
*** devananda has quit IRC | 05:22 | |
*** adisky___ is now known as adisky__ | 05:22 | |
*** Nisha_Agarwal has joined #openstack-ironic | 05:24 | |
*** Nisha_ has quit IRC | 05:24 | |
*** pc_m has quit IRC | 05:25 | |
*** pc_m has joined #openstack-ironic | 05:27 | |
*** karthiks has joined #openstack-ironic | 05:30 | |
openstackgerrit | paresh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get pending boot mode https://review.openstack.org/467982 | 05:31 |
*** mjura has joined #openstack-ironic | 05:35 | |
*** kiennt has quit IRC | 05:38 | |
openstackgerrit | paresh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get current boot mode https://review.openstack.org/466697 | 05:39 |
*** devananda has joined #openstack-ironic | 05:43 | |
*** ijw has joined #openstack-ironic | 05:50 | |
openstackgerrit | melissaml proposed openstack/proliantutils master: Replace assertTrue(isinstance()) with assertIsInstance() https://review.openstack.org/480834 | 05:51 |
*** ijw has quit IRC | 05:56 | |
*** kiennt has joined #openstack-ironic | 05:58 | |
*** rcernin has joined #openstack-ironic | 05:59 | |
*** rbartal has joined #openstack-ironic | 05:59 | |
*** litao__ has quit IRC | 06:00 | |
*** ccamacho has quit IRC | 06:01 | |
*** ChubYann has quit IRC | 06:01 | |
*** rcernin has quit IRC | 06:04 | |
*** karthiks has quit IRC | 06:12 | |
*** kiennt has quit IRC | 06:12 | |
*** karthiks has joined #openstack-ironic | 06:12 | |
*** aszeszo has joined #openstack-ironic | 06:15 | |
*** kiennt has joined #openstack-ironic | 06:16 | |
*** johnthetubaguy has quit IRC | 06:21 | |
*** johnthetubaguy has joined #openstack-ironic | 06:27 | |
*** marios has joined #openstack-ironic | 06:29 | |
*** fragatina has joined #openstack-ironic | 06:50 | |
*** fragatina has quit IRC | 06:54 | |
*** hshiina has joined #openstack-ironic | 06:55 | |
*** ccamacho has joined #openstack-ironic | 06:55 | |
openstackgerrit | M V P Nitesh proposed openstack/ironic master: Allow setting {provisioning,cleaning}_network in node driver_info https://review.openstack.org/460564 | 06:59 |
openstackgerrit | Anshul Jain proposed openstack/proliantutils master: Adds 'activate_license' for redfish systems https://review.openstack.org/468783 | 07:03 |
*** tesseract has joined #openstack-ironic | 07:04 | |
*** tesseract-RH has joined #openstack-ironic | 07:04 | |
*** tesseract has quit IRC | 07:04 | |
*** tesseract-RH has quit IRC | 07:04 | |
*** tesseract has joined #openstack-ironic | 07:05 | |
openstackgerrit | paresh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get pending boot mode https://review.openstack.org/467982 | 07:12 |
*** mgoddard_ has joined #openstack-ironic | 07:14 | |
*** jaganathan has joined #openstack-ironic | 07:18 | |
openstackgerrit | Ankit Kumar proposed openstack/proliantutils master: Adds 'reset_ilo_credentials' for redfish systems https://review.openstack.org/474191 | 07:30 |
*** racedo has joined #openstack-ironic | 07:32 | |
pas-ha | morning ironic :) | 07:38 |
*** d0ugal_ has quit IRC | 07:46 | |
*** d0ugal has joined #openstack-ironic | 07:46 | |
*** d0ugal has quit IRC | 07:46 | |
*** d0ugal has joined #openstack-ironic | 07:46 | |
*** ralonsoh has joined #openstack-ironic | 07:46 | |
*** mgoddard_ has quit IRC | 07:48 | |
*** e0ne has joined #openstack-ironic | 07:59 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-ironic | 08:01 | |
vdrok | morning ironic and pas-ha ! | 08:01 |
openstackgerrit | Ankit Kumar proposed openstack/proliantutils master: Adds 'reset_ilo_credentials' for redfish systems https://review.openstack.org/474191 | 08:07 |
*** racedo has quit IRC | 08:07 | |
openstackgerrit | Merged openstack/proliantutils master: Replace assertTrue(isinstance()) with assertIsInstance() https://review.openstack.org/480834 | 08:10 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Test proposed nova devstack change and our grenade https://review.openstack.org/478220 | 08:13 |
*** Nisha_Agarwal has quit IRC | 08:15 | |
*** Nisha_Agarwal has joined #openstack-ironic | 08:16 | |
*** cvstealt1 has joined #openstack-ironic | 08:17 | |
*** lucas-afk is now known as lucasagomes | 08:18 | |
*** cvstealth has quit IRC | 08:18 | |
openstackgerrit | fupingxie proposed openstack/ironic master: Replace http with https https://review.openstack.org/480896 | 08:20 |
*** mvpnitesh has quit IRC | 08:20 | |
*** mgoddard_ has joined #openstack-ironic | 08:21 | |
masber | good morning all, I am trying to setup ironic and have one question, does the conductor node needs to have access tot he IPMI network? | 08:27 |
*** Nisha_Agarwal has quit IRC | 08:30 | |
*** racedo has joined #openstack-ironic | 08:30 | |
*** Nisha_ has joined #openstack-ironic | 08:30 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic stable/ocata: [DNM] Get rid of sourcing stackrc in grenade settings https://review.openstack.org/480901 | 08:31 |
*** litao__ has joined #openstack-ironic | 08:32 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic stable/newton: [DNM] Get rid of sourcing stackrc in grenade settings https://review.openstack.org/480903 | 08:38 |
openstackgerrit | paresh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get pending boot mode https://review.openstack.org/467982 | 08:39 |
*** joe817 has quit IRC | 08:39 | |
*** joe817 has joined #openstack-ironic | 08:39 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic stable/ocata: [DNM] Get rid of sourcing stackrc in grenade settings https://review.openstack.org/480904 | 08:40 |
*** amarao has joined #openstack-ironic | 08:40 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic master: [DNM] Get rid of sourcing stackrc in grenade settings https://review.openstack.org/480905 | 08:43 |
openstackgerrit | Dao Cong Tien proposed openstack/ironic master: Adds storage_interface field to driver API https://review.openstack.org/480907 | 08:47 |
*** tuanluong has joined #openstack-ironic | 08:49 | |
strigazi | Hi ironic team, I'm from magnum and I saw the mail for the ssh based drivers. http://lists.openstack.org/pipermail/openstack-dev/2017-July/119284.html | 08:49 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic stable/newton: [DNM] Get rid of sourcing stackrc in grenade settings https://review.openstack.org/480903 | 08:50 |
strigazi | In magnum, we are good right? http://git.openstack.org/cgit/openstack/magnum/tree/magnum/tests/contrib/gate_hook.sh#n55 | 08:50 |
strigazi | or we need to remove: http://git.openstack.org/cgit/openstack/magnum/tree/magnum/tests/contrib/gate_hook.sh#n70 ? | 08:51 |
vdrok | strigazi: I think you need to remove the _ssh drivers on line 70 | 08:52 |
vdrok | pas-ha: ^^ | 08:52 |
strigazi | btw http://git.openstack.org/cgit/openstack/ironic/tree/doc/source/dev/dev-quickstart.rst#n489 | 08:52 |
vdrok | heh, yeah, we need to remove that :) | 08:53 |
strigazi | vdrok line 55 is good? | 08:53 |
vdrok | yup | 08:53 |
strigazi | Should we have line 70 like IRONIC_ENABLED_DRIVERS=agent_ipmitool" | 08:54 |
strigazi | Will it work? Our ironic tests are broken for months :( | 08:54 |
vdrok | strigazi: nah, should be IRONIC_ENABLED_DRIVERS=fake,agent_ipmitool,pxe_ipmitool | 08:55 |
strigazi | vdrok ok, I'll git it a go | 08:55 |
vdrok | :) | 08:55 |
strigazi | s/git/give x) | 08:55 |
strigazi | s/git/give xD | 08:55 |
pas-ha | strigazi: on line 70, just remove the *_ssh drivers from the list of enabled drivers, otherwise you seem good. | 08:56 |
openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient master: Add support for storage_interface to node and driver CLI https://review.openstack.org/466186 | 08:57 |
pas-ha | I'll make a patch to dev-quickstart today (or later after the docs migration is complete enough) | 08:57 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Remove _ssh drivers from dev-quickstart https://review.openstack.org/480915 | 08:57 |
vdrok | pas-ha: I'm faster :) | 08:58 |
openstackgerrit | paresh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get pending boot mode https://review.openstack.org/467982 | 08:58 |
strigazi | vdrok Thanks you | 08:58 |
strigazi | vdrok Thank you | 08:58 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Remove _ssh drivers from dev-quickstart https://review.openstack.org/480915 | 08:59 |
pas-ha | vdrok please rebase on top of https://review.openstack.org/#/c/479875/8 | 09:00 |
patchbot | patch 479875 - ironic - Move doc/source/dev to doc/source/contributor | 09:00 |
openstackgerrit | Anshul Jain proposed openstack/proliantutils master: Adds 'activate_license' for redfish systems https://review.openstack.org/468783 | 09:02 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Remove _ssh drivers from dev-quickstart https://review.openstack.org/480915 | 09:03 |
vdrok | pas-ha: done | 09:03 |
pas-ha | +1 :) | 09:04 |
strigazi | vdrok magnum: https://review.openstack.org/480920 ci: Remove *_ssh ironic drivers | 09:04 |
patchbot | patch 480920 - magnum - ci: Remove *_ssh ironic drivers | 09:04 |
pas-ha | strigazi: +1 | 09:06 |
vdrok | thanks! | 09:07 |
strigazi | vdrok pas-ha folks, what is your timezone? I would need some help later with our ironic job. | 09:07 |
vdrok | gmt+3 | 09:08 |
strigazi | vdrok cool, same as mine | 09:08 |
*** mvpnitesh has joined #openstack-ironic | 09:16 | |
openstackgerrit | M V P Nitesh proposed openstack/ironic master: Allow setting {provisioning,cleaning}_network in node driver_info https://review.openstack.org/460564 | 09:19 |
*** j_wong has joined #openstack-ironic | 09:22 | |
j_wong | hi | 09:23 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Test proposed nova devstack change and our grenade https://review.openstack.org/478220 | 09:24 |
j_wong | may i ask a question? | 09:25 |
*** Nisha_ has quit IRC | 09:26 | |
mvpnitesh | yeah j_wong, you can | 09:26 |
mvpnitesh | morning ironic and vdrok | 09:26 |
vdrok | morning j_wong mvpnitesh | 09:26 |
openstackgerrit | Anshul Jain proposed openstack/proliantutils master: Adds 'activate_license' for redfish systems https://review.openstack.org/468783 | 09:29 |
*** bmotz has joined #openstack-ironic | 09:30 | |
*** bmotz has left #openstack-ironic | 09:31 | |
j_wong | my customer want use nova attach volume to the node | 09:31 |
j_wong | cause they are afraid of oneday the node storage is useless | 09:33 |
j_wong | is somebody has a plan to provide this? | 09:35 |
j_wong | it would be best that the volume can be backup | 09:36 |
openstackgerrit | paresh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get current boot mode https://review.openstack.org/466697 | 09:44 |
*** igordcard has quit IRC | 09:48 | |
*** igordcard has joined #openstack-ironic | 09:48 | |
vdrok | j_wong: yes, people are working on this http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/boot-from-volume-reference-drivers.html | 09:48 |
j_wong | really?where can I find the docs or messages about it | 09:49 |
j_wong | or links | 09:50 |
openstackgerrit | Tuan Luong-Anh proposed openstack/python-ironicclient master: Rearrange existing documentation to fit the new standard layout https://review.openstack.org/480013 | 09:51 |
vdrok | j_wong: here is the first one of the chain of patches https://review.openstack.org/463908. those are not yet merged | 09:51 |
patchbot | patch 463908 - ironic - Enable cinder storage interface for generic hardware | 09:51 |
openstackgerrit | paresh sao proposed openstack/ironic master: Added a condition for 'ilo' hardware type https://review.openstack.org/475342 | 09:54 |
j_wong | thank you very much, I had already read it,but it's not enough | 09:54 |
openstackgerrit | Gang Xu proposed openstack/pyghmi master: Add a description of the parameters https://review.openstack.org/480942 | 09:55 |
*** livelace has joined #openstack-ironic | 09:57 | |
j_wong | I'm off duty now, see you | 09:59 |
openstackgerrit | Tuan Luong-Anh proposed openstack/python-ironicclient master: Rearrange existing documentation to fit the new standard layout https://review.openstack.org/480013 | 10:00 |
*** j_wong has quit IRC | 10:02 | |
*** kiennt has quit IRC | 10:05 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic master: [DNM] Get rid of sourcing stackrc in grenade settings https://review.openstack.org/480905 | 10:08 |
*** MattMan_1 has quit IRC | 10:09 | |
*** MattMan_1 has joined #openstack-ironic | 10:10 | |
sambetts|afk | TheJulia: api-ref is shelved until phase 2 of the process apprently, not sure why but thats the plan | 10:12 |
*** sambetts|afk is now known as sambetts | 10:12 | |
*** mvpnitesh has quit IRC | 10:14 | |
*** mvpnitesh has joined #openstack-ironic | 10:19 | |
openstackgerrit | Ankit Kumar proposed openstack/proliantutils master: Adds 'reset_ilo_credentials' for redfish systems https://review.openstack.org/474191 | 10:26 |
*** mvpnitesh has quit IRC | 10:26 | |
*** mvpnitesh has joined #openstack-ironic | 10:27 | |
openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add API for volume resources to api-ref https://review.openstack.org/478798 | 10:28 |
masber | hi, could I ask a question? | 10:34 |
masber | I am playing with ironic for first time, I am following this doc https://docs.openstack.org/ironic/latest/install/enrollment.html which says that a node is in enroll state after creation, in my case the sate is available, I wondering if I am doing it wrong? http://paste.openstack.org/show/614565/ | 10:37 |
vdrok | masber: enroll state is ironic api version 1.11 if i remember correctly. the default api version used by client is 1.9 | 10:38 |
vdrok | and in 1.9 the node is created in available state already | 10:39 |
vdrok | if you run the create command with latest version it will be in enroll. done like "ironic --ironic-api-version latest ..." or "openstack --os-baremetal-api-version latest ..." | 10:39 |
vdrok | or you can export env variable not to type it every time | 10:40 |
vdrok | export IRONIC_API_VERSION=latest or export OS_BAREMETAL_API_VERSION latest | 10:40 |
masber | ok, stupid question, why is not using the latest version by default? | 10:41 |
*** rbartal has quit IRC | 10:45 | |
vdrok | masber: for historical reasons :) we do plan to change to latest by default for CLI client | 10:48 |
openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add storage interface to Node in api-ref https://review.openstack.org/480975 | 10:50 |
masber | ok, so I used api 1.31 to create the node and it is in enroll state but then I tried to edit it changing the deploy_interface=direct and raid_agent=agent but I got a message saying For node f6e53df6-a9f4-4873-aa22-f146cb483bdf with driver pxe_ipmitool, these node fields must be set to None: deploy_interface,raid_interface. (HTTP 400) | 10:51 |
masber | http://paste.openstack.org/show/614567/ | 10:52 |
*** Nisha_Agarwal has joined #openstack-ironic | 10:56 | |
*** retr0h has quit IRC | 10:57 | |
*** retr0h has joined #openstack-ironic | 10:57 | |
masber | ok, this only happens with driver pxe_ipmitool and not with ipmi, this may lead to confusion for people like me who just copy and paste commands without knowing what they are doing. It would be good to fix the documentation | 10:58 |
openstackgerrit | Anshul Jain proposed openstack/proliantutils master: Redfish: Adds 'activate_license' for redfish systems https://review.openstack.org/468783 | 11:12 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic stable/ocata: [DNM] Get rid of sourcing stackrc in grenade settings https://review.openstack.org/480904 | 11:13 |
openstackgerrit | Hironori Shiina proposed openstack/python-ironicclient master: Add OSC commands for volume connector https://review.openstack.org/427053 | 11:19 |
openstackgerrit | Aparna proposed openstack/ironic master: Support hpsum based `update_firmware` for iLO drivers https://review.openstack.org/422572 | 11:20 |
openstackgerrit | Hironori Shiina proposed openstack/python-ironicclient master: Add OSC commands for volume target https://review.openstack.org/427738 | 11:24 |
*** mat128 has joined #openstack-ironic | 11:26 | |
openstackgerrit | Aparna proposed openstack/ironic master: Support hpsum based `update_firmware` for iLO drivers https://review.openstack.org/422572 | 11:27 |
*** fellypefca has joined #openstack-ironic | 11:41 | |
*** litao__ has quit IRC | 11:42 | |
pas-ha | masber: to be able to change interfaces you must first change the driver itself to e.g. ipmi. Ironic has old style drivers (like pxe_ipmitool) where all the driver interfaces are set in stone, and new-style, dynamic drivers (e.g. ipmi) that have a list of supported interfaces to choose from | 11:42 |
*** deadcells has joined #openstack-ironic | 11:44 | |
masber | pas-ha, what is the benefit of having to setup the drivers interfaces manually when I can have them already available in a old style driver? | 11:44 |
TheJulia | sambetts: Okay, thanks for following up on that | 11:46 |
TheJulia | masber: so if you set the hardware type, you will get a list of default interfaces that are supported, and you may wish to then choose deployment style. | 11:47 |
pas-ha | less of driver exposion.. with old style drivers you'd have to write a separate Python class in Ironic to allow for particular combination of driver interfaces | 11:47 |
pas-ha | like you have 2 implementations for deploy, 2 implementations for power/management, 2 for boot, 2-3 for console... that means 2x2x2x3 separate python classes to account for all possibilities.. | 11:49 |
pas-ha | with new drivers, each driver knows which implementations it in principle supports, so operators can mix and match w/o need for custom code and service restarts to enable the new code | 11:50 |
pas-ha | new driver == hardware tyoe | 11:51 |
pas-ha | type | 11:51 |
openstackgerrit | paresh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get pending boot mode https://review.openstack.org/467982 | 11:52 |
*** jaypipes has joined #openstack-ironic | 11:52 | |
openstackgerrit | Wang KaiFeng proposed openstack/ironic master: Change ramdisk log filename template https://review.openstack.org/481001 | 11:53 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Add tests for 'baremetal port create' command https://review.openstack.org/332880 | 11:54 |
*** rbudden has joined #openstack-ironic | 11:57 | |
TheJulia | hshiina: You may want to consider just creating a follow-up patch to address any future nits/minor changes in the python-ironicclient changes moving forward. | 12:00 |
masber | sorry another question, according to this doc https://docs.openstack.org/ironic/latest/install/enrollment.html the value of local_bg must match the size of the root device. My question is, is the size (value) returned by lsblk the right one? | 12:02 |
openstackgerrit | Anshul Jain proposed openstack/proliantutils master: Redfish: Adds 'activate_license' for redfish systems https://review.openstack.org/468783 | 12:05 |
*** trown|outtypewww is now known as trown | 12:06 | |
hshiina | TheJulia: thank you for the advice. i will create a follow-up patch for minor fixes. | 12:06 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Add tests for 'baremetal port create' command https://review.openstack.org/332880 | 12:09 |
*** tuanluong has quit IRC | 12:10 | |
*** oanson has quit IRC | 12:15 | |
*** oanson has joined #openstack-ironic | 12:20 | |
openstackgerrit | Aparna proposed openstack/ironic master: Adds functionality test for hpsum based firmware update https://review.openstack.org/479202 | 12:21 |
*** Nisha_ has joined #openstack-ironic | 12:22 | |
*** Nisha_Agarwal has quit IRC | 12:22 | |
*** mvpnitesh has quit IRC | 12:24 | |
*** dprince has joined #openstack-ironic | 12:24 | |
*** bfournie has quit IRC | 12:29 | |
*** mvpnitesh has joined #openstack-ironic | 12:33 | |
*** crushil has quit IRC | 12:34 | |
openstackgerrit | Ankit Kumar proposed openstack/proliantutils master: Adds 'reset_ilo_credentials' for redfish systems https://review.openstack.org/474191 | 12:35 |
*** crushil has joined #openstack-ironic | 12:35 | |
*** chlong_ has joined #openstack-ironic | 12:37 | |
*** mandre has joined #openstack-ironic | 12:37 | |
*** mvpnitesh has quit IRC | 12:39 | |
mandre | hello, what could be a reason for ironic-inspector to fail to start? We're seing this in the tripleo ci, http://logs.openstack.org/53/476153/12/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha-oooq/56903c3/logs/undercloud/var/log/messages.txt.gz#_Jul__6_07_48_39 | 12:40 |
mandre | a little earlier, we see that ironic-inspector fails to setup iptables rules, http://logs.openstack.org/53/476153/12/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha-oooq/56903c3/logs/undercloud/var/log/messages.txt.gz#_Jul__6_07_48_34 | 12:41 |
*** marios has quit IRC | 12:42 | |
*** marios has joined #openstack-ironic | 12:43 | |
openstackgerrit | Mark Goddard proposed openstack/ironic master: Physical network aware VIF attachment https://review.openstack.org/470406 | 12:49 |
pas-ha | masber: I would presume yes (that should be GiB in fact). But when you enroll the nodes, register those with size of 1GB smaller than actual disk size so that there's room for configdrive and partition table created for partition images. | 12:51 |
pas-ha | so that there's some room and avoid a very tight fit of image to disk that might fail deployment | 12:52 |
*** Nisha_brb has joined #openstack-ironic | 12:54 | |
*** Nisha_ has quit IRC | 12:54 | |
masber | ok, I see, another question, why do I need to specify the cpu, disk and ram on the flavor but also on the node properties? why I can't just setup the properties on the flavor and just point the node to that flavor so I only need to define the properties once? | 13:01 |
*** sasha21 has joined #openstack-ironic | 13:01 | |
*** marios has quit IRC | 13:01 | |
*** Goneri has joined #openstack-ironic | 13:03 | |
*** bfournie has joined #openstack-ironic | 13:04 | |
pas-ha | that would mean that you ever deploy the node with the same flavor, which might be suboptimal re resource utilization - for example you have 2 flavors 4GB root + 1GB swap, and 3GB root + 2GB swap, and both can be deployed on a node with 5GB disk, but you'd be able to deploy only one of them. | 13:05 |
pas-ha | Basically when nova recieves request to build a (baremetal) instance with specific flavor, it will go out to ironic to find which node can fulfill that flavor request re disk, ram and cpu | 13:05 |
*** rloo has joined #openstack-ironic | 13:07 | |
pas-ha | with recent work in nova scheduler and placement API a concept of resource providers and resource classes is introduced, and ironic itself does already supports it by possibility to assign a resource type to the node. When all that is finalized in nova, and flavors are replaced by resource types, it would be more or less what you describe. | 13:07 |
TheJulia | o/ rloo | 13:07 |
rloo | good morning TheJulia, pas-ha and everyone else! | 13:08 |
pas-ha | morning rloo | 13:08 |
*** marios has joined #openstack-ironic | 13:08 | |
openstackgerrit | Sam Betts proposed openstack/ironic master: Update README to point at new doc location https://review.openstack.org/479877 | 13:09 |
openstackgerrit | Sam Betts proposed openstack/ironic master: Move ironic dbsync tool docs into doc/source/cli https://review.openstack.org/479876 | 13:09 |
openstackgerrit | Sam Betts proposed openstack/ironic master: Move doc/source/dev to doc/source/contributor https://review.openstack.org/479875 | 13:09 |
openstackgerrit | Sam Betts proposed openstack/ironic master: Move api-ref into new doc/source/api location https://review.openstack.org/479872 | 13:09 |
*** jcoufal has joined #openstack-ironic | 13:10 | |
*** makowals has quit IRC | 13:11 | |
openstackgerrit | paresh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get pending boot mode https://review.openstack.org/467982 | 13:12 |
*** makowals has joined #openstack-ironic | 13:13 | |
TheJulia | ppiela: betherly: Could one of you take a look at: https://review.openstack.org/#/c/480180/ or https://review.openstack.org/#/c/479980 | 13:16 |
patchbot | patch 480180 - ironic-ui - Set warning-is-error for documentation builds | 13:16 |
patchbot | patch 479980 - ironic-ui - Moving ironic-ui install content to install folder | 13:16 |
ppiela | TheJulia: betherly: I will do that | 13:17 |
TheJulia | Thanks, feel free to land | 13:19 |
*** mjturek has joined #openstack-ironic | 13:22 | |
mjturek | gm ironic! | 13:22 |
TheJulia | good morning mjturek | 13:22 |
mjturek | gm TheJulia | 13:22 |
* TheJulia needs coffee, lots of coffee | 13:23 | |
* mjturek sips already made coffee :) | 13:23 | |
mjturek | TheJulia: just curious, want me to abandon the boot.ipxe patch and propose something like you're suggesting? | 13:24 |
mjturek | https://review.openstack.org/#/c/478690/ for ref | 13:25 |
patchbot | patch 478690 - ironic - Generate iPXE template on deploy for boot from volume | 13:25 |
mjturek | and the title is wrong, whoops :-| it's boot.ipxe, not ipxe template | 13:25 |
TheJulia | I wanted to talk to you about that, because I think we had a necessary overlap between that and the one I was working to address some of the other items. | 13:25 |
mjturek | your follow up to the wirein? | 13:25 |
mjturek | lemme check | 13:25 |
*** dprince has quit IRC | 13:26 | |
mjturek | sorry I mean for skip logic | 13:26 |
TheJulia | let me pull it up, it has been a hectic week | 13:26 |
mjturek | no worries! | 13:27 |
TheJulia | https://review.openstack.org/#/c/479326/1/ironic/drivers/modules/iscsi_deploy.py | 13:27 |
patchbot | patch 479326 - ironic - BFV Deploy skip minor logging, logic, and test fixes | 13:27 |
TheJulia | well, close enough, you can see the question that got raised there | 13:27 |
*** baoli has joined #openstack-ironic | 13:27 | |
mjturek | yep 533 | 13:27 |
* TheJulia powers up the espresso machine | 13:28 | |
*** igordcard has quit IRC | 13:28 | |
*** igordcard has joined #openstack-ironic | 13:29 | |
*** dprince has joined #openstack-ironic | 13:29 | |
mjturek | TheJulia: I think you're referring to this patch https://review.openstack.org/#/c/476601/ which I need to give a bit of attention | 13:29 |
patchbot | patch 476601 - ironic - [WIP] Make storage calls agnostic to deploy method. | 13:29 |
TheJulia | hmm power outage | 13:30 |
mjturek | no rush btw - get some coffee | 13:30 |
TheJulia | yeah, coffee. I'm surprised I made it this long without any | 13:31 |
mjturek | lolol | 13:31 |
*** makowals has quit IRC | 13:32 | |
*** rodrigofm has joined #openstack-ironic | 13:33 | |
*** makowals has joined #openstack-ironic | 13:35 | |
rodrigofm | TheJulia: thanks for the reviews, if u could take a look I'd appreciate very much https://review.openstack.org/#/c/466872/ | 13:35 |
patchbot | patch 466872 - ironic - Migrate oneview driver to use python-hpOneView | 13:36 |
*** kaifeng has joined #openstack-ironic | 13:36 | |
TheJulia | rodrigofm: Happy to, I'll try to take a look before the end of my day. | 13:37 |
TheJulia | Ahh coffeeeee | 13:38 |
*** makowals has quit IRC | 13:40 | |
*** krtaylor has quit IRC | 13:42 | |
TheJulia | mjturek: so I was thinking about https://review.openstack.org/#/c/478690/2/ironic/drivers/modules/pxe.py which actually helps guard the excess work from occurring. Which I actually think makes sense to pair the two at this point and just move forward with doing it upon prepare | 13:44 |
patchbot | patch 478690 - ironic - Generate iPXE template on deploy for boot from volume | 13:44 |
mjturek | ahhh gotcha | 13:44 |
TheJulia | The more I think about it, the more that makes sense to me | 13:45 |
*** makowals has joined #openstack-ironic | 13:45 | |
mjturek | TheJulia: correct me if I'm wrong, but then couldn't you just rebase on top of mine? You'll hit prepare_ramdisk and it'll return after boot.ipxe is generated | 13:47 |
openstackgerrit | Nisha Agarwal proposed openstack/sushy master: Adds EthernetInterface to the library https://review.openstack.org/452532 | 13:47 |
mjturek | I haven't looked that closely at the iscsi path so I might also be missing something | 13:48 |
TheJulia | yeah, I think that is do-able | 13:49 |
mjturek | cooool | 13:50 |
*** trown is now known as trown|brb | 13:50 | |
TheJulia | I don't think they conflict or anything so they could actually land independently, at least I've not noticed anything in the "conflicts with" list | 13:50 |
TheJulia | ahh, yeah, non-bfs efforts. I think I could just pull the wf-1 and we could let them land independently | 13:51 |
mjturek | yeah pretty sure they could, but it'd at least address vlad's comment | 13:51 |
mjturek | oooo | 13:51 |
TheJulia | s/bfs/bfv/ | 13:52 |
vdrok | good morning TheJulia rloo and mjturek | 13:52 |
mjturek | morning vdrok :) | 13:52 |
TheJulia | vlad's comment is as addressed by your patch though, at least It think it was. going back and looking again | 13:52 |
TheJulia | good morning vdrok | 13:52 |
mjturek | yeah I think it is | 13:53 |
rloo | Morning vdrok, mjturek! | 13:53 |
TheJulia | ugh, boot interface | 13:53 |
mjturek | gm rloo ;) | 13:53 |
mjturek | :)* | 13:53 |
TheJulia | I will need to go re-walk the code to be sure | 13:53 |
*** livelace has quit IRC | 13:54 | |
openstackgerrit | Dan Smith proposed openstack/ironic master: Mirror base grenade by not starting nova-compute in multicell mode https://review.openstack.org/481089 | 13:55 |
*** hw_wutianwei has quit IRC | 13:55 | |
dansmith | jlvillal: vdrok ^ total stab in the dark | 13:55 |
*** krtaylor has joined #openstack-ironic | 13:56 | |
*** hw_wutianwei has joined #openstack-ironic | 13:56 | |
vdrok | dansmith: it's being merged already :) https://review.openstack.org/477313 | 13:56 |
patchbot | patch 477313 - ironic - Set nomulticell flag for starting nova-compute in ... | 13:56 |
dansmith | vdrok: dangit! :) | 13:57 |
dansmith | vdrok: does it resolve the problem? | 13:57 |
TheJulia | the DNM patches that were for testing off of it were passing their tests, so it was a positive sign, at least to me. :) | 13:59 |
*** baoli has quit IRC | 13:59 | |
vdrok | dansmith: it does, we also need a backport to networking-generic-switch , so that multinode grenade passes. we'll merge it. the patch that incorporates all the fixes is at https://review.openstack.org/478220. but there are failures now in smoke tests | 13:59 |
patchbot | patch 478220 - ironic - Test proposed nova devstack change and our grenade | 13:59 |
dansmith | okay cool | 14:00 |
vdrok | dansmith: http://logs.openstack.org/20/478220/3/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode-ubuntu-xenial/1b9068f/console.html#_2017-07-06_10_40_44_105478 and http://logs.openstack.org/20/478220/3/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode-ubuntu-xenial/1b9068f/console.html#_2017-07-06_10_40_44_105478 | 14:00 |
vdrok | dansmith: the first one does not seem to be related to ironic. I'll now check the second one | 14:01 |
dansmith | vdrok: oh, that's still with the nomulticell flag? I figured not | 14:01 |
vdrok | yes, with it | 14:01 |
dansmith | I looked at the config and it _is_ set in the config | 14:01 |
vdrok | hrm | 14:01 |
*** baoli has joined #openstack-ironic | 14:01 | |
dansmith | that's why I suspected the nomulticell flag, because it shouldn't be starting the extra conductor with that set | 14:02 |
dansmith | well, it shouldn't be using it anyway | 14:02 |
vdrok | dansmith: in new phase there is nomulticell set http://logs.openstack.org/20/478220/3/check/gate-grenade-dsvm-ironic-ubuntu-xenial/4cdcd54/logs/grenade.sh.txt.gz#_2017-07-06_10_54_22_289 | 14:03 |
*** trown|brb is now known as trown | 14:04 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: BFV Deploy skip minor logging, logic, and test fixes https://review.openstack.org/479326 | 14:07 |
TheJulia | mjturek: I just added the dependency ^^^ | 14:07 |
mjturek | thanks TheJulia :) | 14:08 |
mjturek | cool | 14:08 |
*** hshiina has quit IRC | 14:10 | |
vdrok | dansmith: ah, so the failure of grenade job there seems to be because of http://logs.openstack.org/20/478220/3/check/gate-grenade-dsvm-ironic-ubuntu-xenial/4cdcd54/logs/new/screen-q-svc.txt.gz?level=ERROR#_2017-07-06_11_16_35_536, we did report this bug some time ago in neutron. so the only test that's failing is | 14:12 |
vdrok | http://logs.openstack.org/20/478220/3/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode-ubuntu-xenial/1b9068f/console.html#_2017-07-06_10_40_44_105478 | 14:12 |
*** mvpnitesh has joined #openstack-ironic | 14:16 | |
vdrok | dansmith: heh, we have one more place https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L1810 | 14:16 |
vdrok | I'll add to that patch | 14:16 |
dansmith | vdrok: ah cool (sorry I'm getting sucked into a couple other things) | 14:16 |
vdrok | sure, np | 14:16 |
*** deadcells has quit IRC | 14:18 | |
*** links has quit IRC | 14:18 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Set nomulticell flag for starting nova-compute https://review.openstack.org/477313 | 14:19 |
*** aszeszo has quit IRC | 14:26 | |
*** jaganathan has quit IRC | 14:26 | |
vdrok | dansmith: just to clarify, the bit that I've added sets nomulticell flag for our multinode job, not only grenade. but from the previous version log I see that nova itself starts compute with multicell? http://logs.openstack.org/20/478220/3/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode-ubuntu-xenial/1b9068f/logs/devstacklog.txt.gz#_2017-07-06_09_59_04_914 | 14:31 |
dansmith | vdrok: for grenade, but not devstack | 14:33 |
dansmith | vdrok: it's so we're not trying to upgrade from non-multicell to multicell | 14:34 |
dansmith | vdrok: are you saying the other place you found is not in a grenade job? | 14:34 |
vdrok | dansmith: yeah, I get that. but that ^^ is not grenade, just multinode job. so there nomulticell should not be added? | 14:34 |
vdrok | yup | 14:34 |
dansmith | right, you should not add it in a regular devstack job | 14:35 |
vdrok | that bit of log seems to setup things fine for nova | 14:35 |
vdrok | dansmith: and yet there is this http://logs.openstack.org/20/478220/3/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode-ubuntu-xenial/1b9068f/logs/screen-n-cond-cell1.txt.gz#_Jul_06_10_19_16_289139 | 14:35 |
dansmith | yeah, then it's something else | 14:36 |
dansmith | but as you can see from the config, it's in there | 14:36 |
vdrok | hrm, ok, I'll revert that bit for now | 14:36 |
dansmith | maybe the devstack plugin is starting conductor before the rest of lib/nova has actually written that line to the config? | 14:36 |
dansmith | should be able to tell from the timestamps I think | 14:37 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Set nomulticell flag for starting nova-compute in grenade https://review.openstack.org/477313 | 14:38 |
*** rama_y has quit IRC | 14:38 | |
*** Nisha_brb has quit IRC | 14:39 | |
vdrok | I see start_nova_api, then start_nova_rest, then start_nova_conductor | 14:41 |
dansmith | vdrok: that's in base devstack right? | 14:41 |
openstackgerrit | Merged openstack/ironic-ui master: Set warning-is-error for documentation builds https://review.openstack.org/480180 | 14:41 |
dansmith | I don't see where in your plugin you muck with starting nova services | 14:41 |
vdrok | dansmith: here https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L1810 | 14:42 |
dansmith | yeah but not nova conductor | 14:43 |
vdrok | yup, we don't touch that | 14:43 |
dansmith | vdrok: in that same log it looks set: http://logs.openstack.org/20/478220/3/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-multinode-ubuntu-xenial/1b9068f/logs/screen-n-cond-cell1.txt.gz#_Jul_06_09_59_09_251044 | 14:45 |
dansmith | ohh | 14:45 |
dansmith | I think that's remote | 14:45 |
vdrok | dunno what it means :) | 14:47 |
dansmith | hmm, maybe not | 14:47 |
openstackgerrit | Merged openstack/ironic-ui master: Moving ironic-ui install content to install folder https://review.openstack.org/479980 | 14:49 |
vdrok | rloo: also, regarding that thing with tags. the way you propose it works fine, but I'm not sure I get why we need the remove_smth flag, left a comment on your patch | 14:50 |
dansmith | ah, I got it.. this is cell conductor trying to talk to the apidb | 14:50 |
vdrok | dansmith: is there something we need to change on our side? | 14:51 |
dansmith | vdrok: I dunno why it's happening, I just see that it is.. still digging | 14:51 |
*** dprince has quit IRC | 14:52 | |
rloo | vdrok: thx, haven't gotten to that in my emails yet. i may not get to it until next week; want to focus on high priority things today. | 14:52 |
vdrok | rloo: sure, np | 14:53 |
rloo | vdrok: thank YOU! also wrt 477313, you 'reverted' back to patch set 5, so I will +A it again, ok? | 14:54 |
vdrok | yup :) | 14:54 |
rloo | vdrok: done | 14:54 |
*** joe817 has quit IRC | 14:58 | |
*** skazi_ has joined #openstack-ironic | 14:58 | |
*** joe817 has joined #openstack-ironic | 14:58 | |
dansmith | vdrok: ah, this is a reschedule | 15:00 |
dansmith | vdrok: it tried once to build on the subnode, failed, and then called to cell conductor to reschedule, where it failed | 15:01 |
NobodyCam | Good Morning Ironic'ers :) | 15:02 |
vdrok | mhm | 15:02 |
vdrok | morning NobodyCam | 15:02 |
NobodyCam | :) morning vdrok :) | 15:02 |
NobodyCam | pas-ha: happen to be around? | 15:05 |
dansmith | vdrok: so, if you're hitting reschedules, multicell is always going to be a problem for you in the gate, since each compute node is in a different cell | 15:13 |
dansmith | thus rescheduling (which only considers a single cell) will never actually pick the other compute | 15:13 |
*** e0ne has quit IRC | 15:13 | |
dansmith | vdrok: so I guess you should slap the nomulticell flag back on your regular devstack run and see if that gets you working again, and if so, we'll just have to proceed like that for now | 15:14 |
vdrok | dansmith: ok, I'll do it in separate patch then and we'll see. thx! | 15:15 |
dansmith | vdrok: okay thanks | 15:16 |
*** rama_y has joined #openstack-ironic | 15:16 | |
*** amarao has quit IRC | 15:16 | |
*** amarao has joined #openstack-ironic | 15:17 | |
*** kaifeng has quit IRC | 15:17 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Set nomulticell flag for multinode job https://review.openstack.org/481114 | 15:18 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Set nomulticell flag for multinode job https://review.openstack.org/481114 | 15:19 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Set nomulticell flag for multinode job https://review.openstack.org/481114 | 15:20 |
openstackgerrit | Markos Chandras (hwoarang) proposed openstack/bifrost master: Test OPNFV CI verify job. Please ignore. https://review.openstack.org/367608 | 15:20 |
rama_y | Good morning, Ironic! | 15:21 |
rloo | sambetts: wrt ironic doc migration, it isn't clear where the dev docs should go (looking at etherpad https://etherpad.openstack.org/p/doc-migration-tracking) | 15:22 |
*** rpioso has joined #openstack-ironic | 15:23 | |
sambetts | rloo: no sure what you mean, dev docs == contributor docs so they go into doc/source/contributor right? | 15:23 |
sambetts | not sure* | 15:23 |
NobodyCam | Good Morning rama_y | 15:23 |
rloo | sambetts: maybe. is that where other projects are putting their dev docs? | 15:23 |
NobodyCam | morning rloo & sambetts too :) | 15:23 |
rloo | morning NobodyCam! | 15:24 |
NobodyCam | :) | 15:24 |
rama_y | morning NobodyCam, rloo, sambetts | 15:24 |
sambetts | rloo: as far as I'm aware yes | 15:24 |
rloo | sambetts: are we going to address dev docs later? | 15:24 |
sambetts | rloo: no, I've already put together the patch for it | 15:24 |
rloo | sambetts: am just looking at the bottom of your chain and trying to understand what the new world is like | 15:24 |
rama_y | rloo: you meant /developer/ docs right? I wondered the same. | 15:24 |
sambetts | only thing that will not be done in phase 1 is the api-ref | 15:25 |
rloo | sambetts: ah, ok. i will assume you followed that etherpad and know what you are doing! | 15:25 |
rloo | rama_y: guess it is ../contributor/ | 15:25 |
sambetts | rloo: top of the chain is this patch: https://review.openstack.org/#/c/479877/ | 15:25 |
patchbot | patch 479877 - ironic - Update README to point at new doc location | 15:25 |
rama_y | ok | 15:25 |
rloo | sambetts: which means we might need a nicer landing page at https://docs.openstack.org/ironic/latest/ but maybe you've already done that. | 15:25 |
sambetts | rloo: the patchs clean up the landing page as they move the docs into their new locations, we're not currently sure what the final form of the main index page should look like so it right now just links to each of the new sub-docs under each of the headings | 15:26 |
rloo | sambetts: hmm, the link to contributor is specifically on how to contribute. well, will look into that later. let's get your patches landed first. | 15:27 |
*** e0ne has joined #openstack-ironic | 15:28 | |
*** e0ne has quit IRC | 15:28 | |
*** aszeszo has joined #openstack-ironic | 15:29 | |
sambetts | rloo: there is much refactoring of what docs are where that will need doing, these patches basically get the really broadstrokes down so that our docs aren't broken any more | 15:30 |
rloo | sambetts: right, so i'm not going to do a detailed review of all the pages, just make sure they are moved for now :-) | 15:31 |
sambetts | yeah thats the main thing, although I hope things are ruffly in the sort of place they should end up | 15:32 |
sambetts | and all the links are fixed | 15:32 |
rloo | sambetts: ++, that's why i wanted a top level overview :-) I think the developer docs are the only questionable one but i may have others later. | 15:32 |
*** mandre is now known as mandre_away | 15:33 | |
*** aszeszo1 has joined #openstack-ironic | 15:33 | |
*** aszeszo has quit IRC | 15:33 | |
sambetts | rloo: based on the descriptions on the etherpad I can't see the devleop docs fitting into any of the other categories, contributor is described as "anything having to do with contributing to the project" which I think devs docs are and based on the fact we're supposed to put our python API docs in contributor/api I think it all points to that being the right location | 15:34 |
*** rama_y has quit IRC | 15:35 | |
rloo | sambetts: python API docs? under contributor? guess i don't understand | 15:35 |
*** rama_y_ has joined #openstack-ironic | 15:35 | |
rloo | sambetts: we don't have any python API docs. Do you mean the REST API? | 15:35 |
rloo | sambetts: or do you mean python-ironicclient python API? | 15:36 |
rloo | sambetts: oh, or do you mean our python classes? | 15:36 |
sambetts | rloo: no, python API docs they look like this: https://docs.openstack.org/ironic/latest/py-modindex.html | 15:36 |
rloo | sambetts: ok, the python classes | 15:37 |
*** skazi_ has quit IRC | 15:37 | |
sambetts | rloo: yeah its the change to that is descrbie in bullet 8 on the etherpad | 15:37 |
sambetts | described* | 15:38 |
rloo | sambetts: i only looked at L9-17. I said High level, didn't I? :-) | 15:39 |
sambetts | yup :) | 15:39 |
rloo | sambetts: anyway, will think (maybe) about that later. | 15:39 |
sambetts | heh yeah the spec https://specs.openstack.org/openstack/docs-specs/specs/pike/os-manuals-migration.html is a lot to read through | 15:39 |
rloo | sambetts: thx for making all these changes, btw | 15:40 |
sambetts | taking a quick look at the contributor section on the spec it has more information, and it seems to imply the contributor is a superset of /dev | 15:40 |
sambetts | e.g. not all contributors are developers, so it will have developer and non-developer contribution docs | 15:41 |
sambetts | not sure we have any non-dev contribuor docs right now :/ | 15:41 |
mgoddard_ | qq: should I upgrade IPA or ironic first? | 15:42 |
sambetts | mgoddard_: IPA | 15:43 |
rloo | sambetts: ok. we can discuss later. let's get the other stuff landed first. | 15:43 |
sambetts | (although we're working to make it so it doesn't matter with version negotiation between IPA and Ironic) | 15:43 |
rloo | sambetts: one other thing. i am guessing we do not care that we have broken links, we aren't going to provide redirects to the new doc structure. | 15:43 |
*** penick has joined #openstack-ironic | 15:44 | |
mgoddard_ | thanks sambetts | 15:44 |
rloo | sambetts: we should maybe add a release note somewhere about the doc changes. | 15:44 |
sambetts | rloo: I think the current plan is that the redirects are handled above us, e.g. anyone trying to access the old location is redirected to the new doc.o.o/ironic/latest site regardless of where they were trying to go | 15:45 |
rloo | sambetts: ok, that would be good. we should probably still put a release note but can do that later if we remember. | 15:46 |
TheJulia | that is my understanding as well | 15:46 |
sambetts | just checked and if you try to access: http://docs.openstack.org/developer/ironic/mitaka/ it redirects you to the new location | 15:47 |
jlvillal | vdrok: I have to run to a meeting. But I did do a recheck on that patch is one of the dependent patches had changed. | 15:50 |
rloo | sambetts: do we have end users? | 15:50 |
jlvillal | s/is one of/as one of/ | 15:50 |
vdrok | jlvillal: morning, thanks! | 15:51 |
rloo | sambetts: i wonder if we want user/, or if it should all go in admin/ | 15:51 |
vdrok | we still might need the one for multinode job though :( | 15:51 |
sambetts | rloo: we have users (admins of nodes enrolled into ironic), and we have operators (admins of ironic the service) | 15:52 |
vdrok | vsaienko says that multinode job has a pretty high reschedule rate because of lots of concurrent tests, so not sure we'll be able to do multicell there | 15:52 |
rloo | sambetts: so would it make more sense to put that stuff under admin/ ? | 15:52 |
rloo | sambetts: eg https://review.openstack.org/#/c/479874/8/doc/source/user/index.rst | 15:52 |
patchbot | patch 479874 - ironic - Move operator docs into into doc/source/admin | 15:52 |
*** hshiina has joined #openstack-ironic | 15:52 | |
sambetts | rloo: I don't think so, I think /admin is for people operating ironic, deciding which drivers to load, configuring the number of threads etc etc, where as /user is for the person doing ironic node-create | 15:53 |
openstackgerrit | Hironori Shiina proposed openstack/ironic master: Enable cinder storage interface for generic hardware https://review.openstack.org/463908 | 15:53 |
sambetts | which aren't the same person, /admin is for the operator of the cloud, /user is the person given the admin role in openstack | 15:54 |
*** tesseract has quit IRC | 15:54 | |
rloo | sambetts: the person doing ironic node-create is an operator? | 15:54 |
rloo | sambetts: ok, so ironic's definition of 'user' is the user that uses our REST API? | 15:54 |
sambetts | rloo: yeah | 15:55 |
rloo | sambetts: just want to make sure that jives with what 'user/' means for the rest of OpenStack projects. | 15:55 |
rloo | sambetts: i'll assume it does :-) | 15:55 |
sambetts | that is my understanding, the descriptions around the /admin folder is more about operating the service e.g. maintaining it, configuring it, upgrading it, vs /user is about using the service to do the thing it was built to do | 15:56 |
TheJulia | BFV meeting in #openstack-meeting-5 | 16:01 |
*** penick has quit IRC | 16:03 | |
pas-ha | NobodyCam: here | 16:05 |
pas-ha | NobodyCam: you were looking for me? | 16:06 |
NobodyCam | hey hey Morning pas-ha :) just wanted to check in with on the removal of the ssh drivers... I check staging and didn't see them there, I thought the plan was to land them there before removing them from main tree? | 16:07 |
pas-ha | NobodyCam: AFAIR the idea was that whoever needs them and more importantly wishes to continue supporting them should move them to the staging drivers himself. | 16:08 |
NobodyCam | ahh | 16:09 |
pas-ha | Same happened with some other drivers, like VirtualBox ones - they were just removed from the tree w/o moving to staging | 16:09 |
pas-ha | since nobody proposed those to staging - means noone is interested or have resources to continue maintaining | 16:10 |
*** hjensas has quit IRC | 16:11 | |
*** hjensas has joined #openstack-ironic | 16:12 | |
*** amarao has quit IRC | 16:14 | |
*** fellypefca has quit IRC | 16:18 | |
*** fellypefca has joined #openstack-ironic | 16:19 | |
*** ricardoas has quit IRC | 16:20 | |
*** ricardoas has joined #openstack-ironic | 16:20 | |
*** xavierr has quit IRC | 16:20 | |
*** xavierr has joined #openstack-ironic | 16:20 | |
*** lucasagomes is now known as lucas-afk | 16:21 | |
*** dprince has joined #openstack-ironic | 16:23 | |
*** tuanla_fujitsu has joined #openstack-ironic | 16:25 | |
*** mvpnitesh has quit IRC | 16:30 | |
vdrok | jlvillal: rloo seems like we'll need to set nomulticell everywhere in our jobs, not only grenade | 16:32 |
*** karthiks has quit IRC | 16:32 | |
rloo | vdrok: okey dokey | 16:33 |
*** ralonsoh has quit IRC | 16:34 | |
* sambetts --> home | 16:35 | |
sambetts | night all! | 16:35 |
*** sambetts is now known as sambetts|afk | 16:35 | |
rloo | good night sambetts! | 16:35 |
sambetts|afk | cya rloo o/ | 16:35 |
*** ChubYann has joined #openstack-ironic | 16:36 | |
*** racedo has quit IRC | 16:44 | |
*** mduggan has joined #openstack-ironic | 16:46 | |
*** hshiina has quit IRC | 16:47 | |
openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Add property-collection-editor directive https://review.openstack.org/481152 | 16:56 |
*** trown is now known as trown|lunch | 16:56 | |
*** Nisha_Agarwal has joined #openstack-ironic | 16:57 | |
*** mjturek1 has joined #openstack-ironic | 16:59 | |
jlvillal | vdrok: Oh. Okay | 16:59 |
*** mjturek has quit IRC | 16:59 | |
mduggan | In the ironic-inspector logs my IPA lookup is failing, claiming that there is no node matching the bmc IP address and the mac address of the provisioning interface. Node show output is here: https://pastebin.com/kDFD4VHG and inspector log output here: https://pastebin.com/uZXjEAYH . Any suggestions on how to proceed with the install having this fai | 16:59 |
mduggan | lure ? | 16:59 |
*** baoli has quit IRC | 17:00 | |
*** baoli has joined #openstack-ironic | 17:01 | |
*** mgoddard_ has quit IRC | 17:02 | |
mduggan | forgot to include the port info | 17:07 |
mduggan | https://thepasteb.in/p/DRhjG7NqyQ8Hy | 17:07 |
*** fragatina has joined #openstack-ironic | 17:11 | |
*** harlowja has joined #openstack-ironic | 17:13 | |
*** zzzeek has quit IRC | 17:14 | |
*** baoli has quit IRC | 17:16 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-inspector stable/ocata: [Devstack] cleanup upgrade settings https://review.openstack.org/481157 | 17:25 |
*** fragatina has quit IRC | 17:25 | |
*** udesale has quit IRC | 17:25 | |
*** baoli has joined #openstack-ironic | 17:25 | |
*** fragatina has joined #openstack-ironic | 17:25 | |
TheJulia | mduggan: the first node is in deploy failed, which has nothing to do with ironic_inspector | 17:26 |
mduggan | TheJulia: but it goes into deploy failed after the inspector lookup failure ..... | 17:28 |
TheJulia | mduggan: Yeah, seeing that with the time stamps. Most likely your physical node is plugged into whatever network your using for discovery. AIUI, Neutron wouldn't be in control of that network, so it can't actually offer the specific DHCP options to tell the node where to heartbeat to in order to establish communication with ironic. | 17:31 |
*** joe817 has quit IRC | 17:31 | |
*** joe817 has joined #openstack-ironic | 17:31 | |
*** ijw has joined #openstack-ironic | 17:31 | |
*** tuanla_fujitsu has quit IRC | 17:32 | |
*** zzzeek has joined #openstack-ironic | 17:33 | |
*** ijw has quit IRC | 17:33 | |
*** ijw has joined #openstack-ironic | 17:34 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-inspector stable/newton: [Devstack] cleanup upgrade settings https://review.openstack.org/481163 | 17:34 |
*** ijw has quit IRC | 17:35 | |
*** krtaylor has quit IRC | 17:35 | |
mduggan | TheJulia: Are you saying that this IPA/inspector lookup failure is irrelevant to the node install ? | 17:37 |
*** zzzeek has quit IRC | 17:37 | |
*** marios has quit IRC | 17:38 | |
mduggan | I'm assuming that is it this failure causes the ironic node installation to abort | 17:38 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic stable/ocata: [Devstack] cleanup upgrade settings https://review.openstack.org/481164 | 17:40 |
*** zzzeek has joined #openstack-ironic | 17:41 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic stable/newton: [Devstack] cleanup upgrade settings https://review.openstack.org/481170 | 17:50 |
TheJulia | mduggan: it is a symptom of the wrong parameters being loaded, specifically discovery parameters I guess. Oddly, the port lookup fails. It might only want to match on all the ports. Sadly inspector is not my strongest area. | 17:51 |
TheJulia | mduggan: process wise, a booting ramdisk shouldn't call IPA | 17:52 |
TheJulia | unless it is for discovery | 17:52 |
TheJulia | mduggan: also, set your client protocol version to latest and make sure pxe_enabled is set to True on the ports you want to use for deployment. | 17:53 |
*** zzzeek has quit IRC | 17:53 | |
*** fragatin_ has joined #openstack-ironic | 17:59 | |
*** zzzeek has joined #openstack-ironic | 18:00 | |
openstackgerrit | Merged openstack/ironic master: Move operator docs into into doc/source/admin https://review.openstack.org/479874 | 18:01 |
*** etingof has joined #openstack-ironic | 18:03 | |
*** fragatina has quit IRC | 18:03 | |
*** baoli has quit IRC | 18:06 | |
*** zzzeek_ has joined #openstack-ironic | 18:08 | |
*** zzzeek has quit IRC | 18:09 | |
*** trown|lunch is now known as trown | 18:11 | |
*** Nisha_Agarwal has quit IRC | 18:12 | |
*** krtaylor has joined #openstack-ironic | 18:12 | |
*** ijw has joined #openstack-ironic | 18:14 | |
openstackgerrit | Merged openstack/python-ironicclient master: switch from oslosphinx to openstackdocstheme https://review.openstack.org/479999 | 18:16 |
*** ricardoas has quit IRC | 18:16 | |
*** ijw has quit IRC | 18:18 | |
*** jtomasek has joined #openstack-ironic | 18:18 | |
*** krtaylor has quit IRC | 18:28 | |
*** etingof has quit IRC | 18:29 | |
*** aszeszo1 has quit IRC | 18:31 | |
*** ijw has joined #openstack-ironic | 18:31 | |
*** krtaylor has joined #openstack-ironic | 18:33 | |
*** ijw has quit IRC | 18:36 | |
*** fellypefca has quit IRC | 18:37 | |
*** ChubYann has quit IRC | 18:37 | |
*** ijw has joined #openstack-ironic | 18:40 | |
*** mjura has quit IRC | 18:46 | |
*** jtomasek has quit IRC | 18:46 | |
*** ijw has quit IRC | 18:47 | |
*** ChubYann has joined #openstack-ironic | 18:51 | |
*** mjturek has joined #openstack-ironic | 18:55 | |
*** mjturek1 has quit IRC | 18:55 | |
*** ccamacho has quit IRC | 19:05 | |
*** jtomasek has joined #openstack-ironic | 19:11 | |
*** pcaruana has quit IRC | 19:12 | |
*** adisky__ has quit IRC | 19:14 | |
*** makowals has quit IRC | 19:26 | |
*** jtomasek has quit IRC | 19:28 | |
*** fragatin_ has quit IRC | 19:28 | |
*** fragatina has joined #openstack-ironic | 19:29 | |
*** makowals has joined #openstack-ironic | 19:34 | |
openstackgerrit | Merged openstack/ironic master: Move doc/source/dev to doc/source/contributor https://review.openstack.org/479875 | 19:39 |
openstackgerrit | Merged openstack/ironic master: Move ironic dbsync tool docs into doc/source/cli https://review.openstack.org/479876 | 19:42 |
*** makowals has quit IRC | 19:42 | |
openstackgerrit | Merged openstack/ironic master: Update README to point at new doc location https://review.openstack.org/479877 | 19:42 |
*** mjturek has quit IRC | 19:45 | |
rloo | TheJulia: qq, wrt https://review.openstack.org/#/c/479142/ -- it has to be totally powered off, not soft power off, right? | 19:55 |
patchbot | patch 479142 - ironic - Add node power state validation to volume resource... (MERGED) | 19:55 |
openstackgerrit | John L. Villalovos proposed openstack/ironic master: DevStack: Add configuration for boot-from-volume https://review.openstack.org/466333 | 19:55 |
TheJulia | state must be POWER_OFF | 19:56 |
rloo | TheJulia: ok, thx. | 19:56 |
TheJulia | no problem, thanks :) | 19:57 |
*** makowals has joined #openstack-ironic | 20:00 | |
*** ijw has joined #openstack-ironic | 20:06 | |
*** mjturek has joined #openstack-ironic | 20:13 | |
*** ijw has quit IRC | 20:13 | |
*** dprince has quit IRC | 20:17 | |
*** ricardoas has joined #openstack-ironic | 20:29 | |
*** ijw has joined #openstack-ironic | 20:32 | |
*** mandre_away is now known as mandre | 20:32 | |
*** ijw has quit IRC | 20:37 | |
*** deadcells has joined #openstack-ironic | 20:40 | |
*** jcoufal has quit IRC | 20:47 | |
*** ijw has joined #openstack-ironic | 20:53 | |
*** krtaylor has quit IRC | 20:54 | |
*** trown is now known as trown|outtypewww | 20:56 | |
*** ijw has quit IRC | 21:00 | |
*** ijw has joined #openstack-ironic | 21:00 | |
*** mandre is now known as mandre_away | 21:10 | |
*** baoli has joined #openstack-ironic | 21:12 | |
*** baoli has quit IRC | 21:13 | |
*** baoli has joined #openstack-ironic | 21:16 | |
rloo | TheJulia: another qq, for volume connectors, you can have more than one associated with the same node? | 21:18 |
TheJulia | yes | 21:19 |
TheJulia | there are database constraints around that and the boot_index column if memory serves | 21:19 |
*** Goneri has quit IRC | 21:20 | |
rloo | TheJulia: am looking and i don't see any constraint on the volume connector. but i see for volume target. | 21:20 |
rloo | TheJulia: wondering if i have forgotten how the db code works. | 21:21 |
*** joe817 has quit IRC | 21:21 | |
TheJulia | oh, no, connectors only have constraint on on the property data if memory serves, the targets are constrained on boot_index, since it can be a many to many relationship | 21:22 |
TheJulia | https://github.com/openstack/ironic/blob/master/ironic/db/sqlalchemy/alembic/versions/daa1ba02d98_add_volume_connectors_table.py#L42 | 21:23 |
rloo | TheJulia: ok, that jives with the code, although targets are constrained on boot_index+node_id. | 21:23 |
TheJulia | yeah, that is correct | 21:24 |
rloo | TheJulia: and i just checked with the volume spec. I asked cuz the client patch i am reviewing seems to indicate there is only one connector per node. i think i may have noted that in the python API client patch too. so thought i should verify. thx. | 21:24 |
*** Nisha_Agarwal has joined #openstack-ironic | 21:25 | |
TheJulia | I really don't remember that, but most users will only have one | 21:25 |
rloo | TheJulia: no worries, it is all becoming a blur to me anyway :) | 21:26 |
TheJulia | Yeah, I know that feeling _really_ well | 21:26 |
*** krtaylor has joined #openstack-ironic | 21:35 | |
*** makowals has quit IRC | 21:37 | |
*** deadcells has quit IRC | 21:40 | |
jlvillal | Anyone else have any problems running py27 unit tests for python-ironicclient? | 21:43 |
jlvillal | I am checked out to master and have done: $ tox -vvv -e py27 -r | 21:43 |
jlvillal | And getting errors :( | 21:44 |
jlvillal | rloo: ^^ ?? | 21:44 |
rloo | jlvillal: haven't tried for awhile. although it seems to me that i may have had errors 'randomly' before. sec, let me try. | 21:44 |
rloo | jlvillal: a normal 'tox' works for me. let me try with your exact cmd. | 21:45 |
jlvillal | rloo: So py35 works for me. But not py27 | 21:46 |
* jlvillal wonders what is wrong | 21:46 | |
rloo | jlvillal: did you do a fresh build? maybe i should delete my .tox first. | 21:46 |
jlvillal | rloo: The '-r' should delete the .tox/ directory of the target | 21:47 |
rloo | jlvillal: yes, i just noticed that, which is why it is taking so long :-) stay tuned... | 21:47 |
rloo | jlvillal: still worked for me. | 21:47 |
rloo | jlvillal: what kind of errors are you seeing? | 21:47 |
jlvillal | rloo: Okay. Something wacky here | 21:48 |
jlvillal | rloo: http://paste.openstack.org/show/614650/ | 21:48 |
jlvillal | And a bunch more like that. About 12 | 21:48 |
*** makowals has joined #openstack-ironic | 21:49 | |
rloo | jlvillal: that doesn't look familiar to me. although i know that vdrok has a patch to change the auth stuff in the client, which shouldn't affect master. i don't think it has landed. | 21:49 |
jlvillal | rloo: I'm assuming something is weird on my system. | 21:50 |
rloo | jlvillal: i'd like to assume that too :) :( | 21:51 |
* TheJulia looks | 21:53 | |
TheJulia | interesting.... | 21:53 |
* TheJulia switches computers | 21:54 | |
TheJulia | jlvillal: current master branch passes just fine for me on py27 | 21:56 |
*** makowals has quit IRC | 21:57 | |
jlvillal | rloo: TheJulia: I fixed it :) $ rm -rf ~/.cache/pip/ | 21:57 |
jlvillal | Thanks! | 21:57 |
rloo | jlvillal: rm is your friend! glad you figured it out. | 21:57 |
TheJulia | awesome | 21:58 |
* TheJulia goes back to not being worried | 21:58 | |
*** racedo has joined #openstack-ironic | 22:05 | |
*** makowals has joined #openstack-ironic | 22:05 | |
*** etingof has joined #openstack-ironic | 22:09 | |
*** racedo has quit IRC | 22:16 | |
*** ijw has quit IRC | 22:16 | |
*** bfournie has quit IRC | 22:16 | |
*** ijw has joined #openstack-ironic | 22:18 | |
*** sauloaislan has quit IRC | 22:22 | |
*** mduggan has quit IRC | 22:22 | |
*** rodrigofm has quit IRC | 22:22 | |
*** nicodemos has quit IRC | 22:22 | |
*** ijw has quit IRC | 22:22 | |
*** dprince has joined #openstack-ironic | 22:25 | |
*** ijw has joined #openstack-ironic | 22:26 | |
*** hw_wutianwei1 has joined #openstack-ironic | 22:27 | |
*** hw_wutianwei has quit IRC | 22:29 | |
*** hw_wutianwei1 is now known as hw_wutianwei | 22:29 | |
*** ijw has quit IRC | 22:40 | |
*** mjturek has quit IRC | 22:52 | |
*** mat128 has quit IRC | 23:01 | |
openstackgerrit | John L. Villalovos proposed openstack/python-ironicclient master: WIP: Require keyword arguments to call common_filters https://review.openstack.org/481316 | 23:08 |
*** ijw has joined #openstack-ironic | 23:17 | |
*** ijw has quit IRC | 23:22 | |
*** baoli has quit IRC | 23:26 | |
openstackgerrit | Richard G. Pioso proposed openstack/ironic master: Revert "Wait until iDRAC is ready before out-of-band cleaning" https://review.openstack.org/481318 | 23:30 |
*** etingof has quit IRC | 23:33 | |
*** rpioso has quit IRC | 23:33 | |
*** joe817 has joined #openstack-ironic | 23:35 | |
*** dprince has quit IRC | 23:35 | |
*** joe817 has quit IRC | 23:35 | |
*** joe817 has joined #openstack-ironic | 23:36 | |
*** Sukhdev has joined #openstack-ironic | 23:40 | |
*** ijw has joined #openstack-ironic | 23:41 | |
-openstackstatus- NOTICE: nb03.openstack.org has been cleaned up and rebooted, and should return to building rotation | 23:44 | |
*** ijw has quit IRC | 23:48 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!