Thursday, 2017-07-06

*** hoonetorg has joined #openstack-ironic00:03
*** joe817 has quit IRC00:04
*** yuanying_ has joined #openstack-ironic00:05
*** joe817 has joined #openstack-ironic00:06
*** rama_y_ has quit IRC00:06
*** yuanying has quit IRC00:06
*** Nisha_ has joined #openstack-ironic00:07
*** Nisha_brb has quit IRC00:07
*** yuanying has joined #openstack-ironic00:31
*** yuanying_ has quit IRC00:31
*** masber has joined #openstack-ironic00:35
masbergood morning, I would like to start using ironic, is there a guide to start using it?00:36
masberlike s starting guide or something like that? I already have it installed through project kolla-ansible00:39
*** ijw has joined #openstack-ironic00:46
*** yuanying has quit IRC00:49
*** yuanying has joined #openstack-ironic01:02
*** litao__ has joined #openstack-ironic01:25
*** makowals has quit IRC01:38
*** makowals has joined #openstack-ironic01:39
*** rcarrillocruz has quit IRC01:42
*** ijw has quit IRC01:43
*** yuanying has quit IRC01:57
*** yuanying has joined #openstack-ironic02:00
*** rbudden has quit IRC02:16
*** rcarrillocruz has joined #openstack-ironic02:17
openstackgerritNguyen Hung Phuong proposed openstack/ironic master: Add functional API tests for volume connector and volume target  https://review.openstack.org/47371702:20
*** Nisha_brb has joined #openstack-ironic02:43
*** Nisha_ has quit IRC02:43
*** udesale has joined #openstack-ironic02:43
*** Nisha_ has joined #openstack-ironic02:51
*** Nisha_brb has quit IRC02:51
openstackgerritNisha Agarwal proposed openstack/sushy master: Adds EthernetInterface to the library  https://review.openstack.org/45253203:01
openstackgerritTuan Luong-Anh proposed openstack/python-ironicclient master: Rearrange existing documentation to fit the new standard layout  https://review.openstack.org/48001303:06
*** kiennt has joined #openstack-ironic03:10
*** d0ugal has quit IRC03:19
*** d0ugal_ has joined #openstack-ironic03:19
*** rama_y has joined #openstack-ironic03:26
openstackgerritDao Cong Tien proposed openstack/ironic master: Tempest scenario test for boot-from-volume  https://review.openstack.org/47274003:29
openstackgerritDao Cong Tien proposed openstack/ironic master: WIP/DNM:Temp patch to test boot-from-volume job  https://review.openstack.org/47685403:29
openstackgerritMerged openstack/proliantutils master: Using assertIsNone() instead of assertEqual(None)  https://review.openstack.org/48003803:48
openstackgerritAparna proposed openstack/proliantutils master: Redfish: Add support for Virtual Media Operations  https://review.openstack.org/47379803:50
openstackgerritTuan Luong-Anh proposed openstack/python-ironicclient master: Rearrange existing documentation to fit the new standard layout  https://review.openstack.org/48001303:59
openstackgerritNisha Agarwal proposed openstack/proliantutils master: WIP: Add EthernetInterface to proliantutils  https://review.openstack.org/48082404:01
*** sasha21 has quit IRC04:05
*** links has joined #openstack-ironic04:05
*** nicodemos has quit IRC04:14
*** sauloaislan has quit IRC04:14
*** xavierr has quit IRC04:17
*** ricardoas has quit IRC04:17
*** ricardoas has joined #openstack-ironic04:17
*** mrtenio|afk has quit IRC04:17
*** mrtenio|afk has joined #openstack-ironic04:18
*** sauloaislan has joined #openstack-ironic04:18
*** nicodemos has joined #openstack-ironic04:18
*** fragatin_ has joined #openstack-ironic04:22
*** fragatina has quit IRC04:26
*** fragatin_ has quit IRC04:26
*** fragatina has joined #openstack-ironic04:32
*** adisky__ has joined #openstack-ironic04:35
*** fragatina has quit IRC04:36
*** nicodemos has quit IRC04:46
*** sauloaislan has quit IRC04:47
*** ricardoas has quit IRC04:47
*** mrtenio|afk has quit IRC04:47
*** mrtenio|afk has joined #openstack-ironic04:48
*** nicodemos has joined #openstack-ironic04:49
*** ricardoas has joined #openstack-ironic04:49
*** sauloaislan has joined #openstack-ironic04:49
*** xavierr has joined #openstack-ironic04:54
*** anish_ has joined #openstack-ironic05:17
*** adisky___ has joined #openstack-ironic05:17
*** mvpnitesh has joined #openstack-ironic05:18
*** harlowja has quit IRC05:22
*** adisky__ has quit IRC05:22
*** anish has quit IRC05:22
*** devananda has quit IRC05:22
*** adisky___ is now known as adisky__05:22
*** Nisha_Agarwal has joined #openstack-ironic05:24
*** Nisha_ has quit IRC05:24
*** pc_m has quit IRC05:25
*** pc_m has joined #openstack-ironic05:27
*** karthiks has joined #openstack-ironic05:30
openstackgerritparesh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get pending boot mode  https://review.openstack.org/46798205:31
*** mjura has joined #openstack-ironic05:35
*** kiennt has quit IRC05:38
openstackgerritparesh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get current boot mode  https://review.openstack.org/46669705:39
*** devananda has joined #openstack-ironic05:43
*** ijw has joined #openstack-ironic05:50
openstackgerritmelissaml proposed openstack/proliantutils master: Replace assertTrue(isinstance()) with assertIsInstance()  https://review.openstack.org/48083405:51
*** ijw has quit IRC05:56
*** kiennt has joined #openstack-ironic05:58
*** rcernin has joined #openstack-ironic05:59
*** rbartal has joined #openstack-ironic05:59
*** litao__ has quit IRC06:00
*** ccamacho has quit IRC06:01
*** ChubYann has quit IRC06:01
*** rcernin has quit IRC06:04
*** karthiks has quit IRC06:12
*** kiennt has quit IRC06:12
*** karthiks has joined #openstack-ironic06:12
*** aszeszo has joined #openstack-ironic06:15
*** kiennt has joined #openstack-ironic06:16
*** johnthetubaguy has quit IRC06:21
*** johnthetubaguy has joined #openstack-ironic06:27
*** marios has joined #openstack-ironic06:29
*** fragatina has joined #openstack-ironic06:50
*** fragatina has quit IRC06:54
*** hshiina has joined #openstack-ironic06:55
*** ccamacho has joined #openstack-ironic06:55
openstackgerritM V P Nitesh proposed openstack/ironic master: Allow setting {provisioning,cleaning}_network in node driver_info  https://review.openstack.org/46056406:59
openstackgerritAnshul Jain proposed openstack/proliantutils master: Adds 'activate_license' for redfish systems  https://review.openstack.org/46878307:03
*** tesseract has joined #openstack-ironic07:04
*** tesseract-RH has joined #openstack-ironic07:04
*** tesseract has quit IRC07:04
*** tesseract-RH has quit IRC07:04
*** tesseract has joined #openstack-ironic07:05
openstackgerritparesh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get pending boot mode  https://review.openstack.org/46798207:12
*** mgoddard_ has joined #openstack-ironic07:14
*** jaganathan has joined #openstack-ironic07:18
openstackgerritAnkit Kumar proposed openstack/proliantutils master: Adds 'reset_ilo_credentials' for redfish systems  https://review.openstack.org/47419107:30
*** racedo has joined #openstack-ironic07:32
pas-hamorning ironic :)07:38
*** d0ugal_ has quit IRC07:46
*** d0ugal has joined #openstack-ironic07:46
*** d0ugal has quit IRC07:46
*** d0ugal has joined #openstack-ironic07:46
*** ralonsoh has joined #openstack-ironic07:46
*** mgoddard_ has quit IRC07:48
*** e0ne has joined #openstack-ironic07:59
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:01
vdrokmorning ironic and pas-ha !08:01
openstackgerritAnkit Kumar proposed openstack/proliantutils master: Adds 'reset_ilo_credentials' for redfish systems  https://review.openstack.org/47419108:07
*** racedo has quit IRC08:07
openstackgerritMerged openstack/proliantutils master: Replace assertTrue(isinstance()) with assertIsInstance()  https://review.openstack.org/48083408:10
openstackgerritVladyslav Drok proposed openstack/ironic master: Test proposed nova devstack change and our grenade  https://review.openstack.org/47822008:13
*** Nisha_Agarwal has quit IRC08:15
*** Nisha_Agarwal has joined #openstack-ironic08:16
*** cvstealt1 has joined #openstack-ironic08:17
*** lucas-afk is now known as lucasagomes08:18
*** cvstealth has quit IRC08:18
openstackgerritfupingxie proposed openstack/ironic master: Replace http with https  https://review.openstack.org/48089608:20
*** mvpnitesh has quit IRC08:20
*** mgoddard_ has joined #openstack-ironic08:21
masbergood 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 IRC08:30
*** racedo has joined #openstack-ironic08:30
*** Nisha_ has joined #openstack-ironic08:30
openstackgerritVladyslav Drok proposed openstack/ironic stable/ocata: [DNM] Get rid of sourcing stackrc in grenade settings  https://review.openstack.org/48090108:31
*** litao__ has joined #openstack-ironic08:32
openstackgerritVladyslav Drok proposed openstack/ironic stable/newton: [DNM] Get rid of sourcing stackrc in grenade settings  https://review.openstack.org/48090308:38
openstackgerritparesh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get pending boot mode  https://review.openstack.org/46798208:39
*** joe817 has quit IRC08:39
*** joe817 has joined #openstack-ironic08:39
openstackgerritVladyslav Drok proposed openstack/ironic stable/ocata: [DNM] Get rid of sourcing stackrc in grenade settings  https://review.openstack.org/48090408:40
*** amarao has joined #openstack-ironic08:40
openstackgerritVladyslav Drok proposed openstack/ironic master: [DNM] Get rid of sourcing stackrc in grenade settings  https://review.openstack.org/48090508:43
openstackgerritDao Cong Tien proposed openstack/ironic master: Adds storage_interface field to driver API  https://review.openstack.org/48090708:47
*** tuanluong has joined #openstack-ironic08:49
strigaziHi 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.html08:49
openstackgerritVladyslav Drok proposed openstack/ironic stable/newton: [DNM] Get rid of sourcing stackrc in grenade settings  https://review.openstack.org/48090308:50
strigaziIn magnum, we are good right? http://git.openstack.org/cgit/openstack/magnum/tree/magnum/tests/contrib/gate_hook.sh#n5508:50
strigazior we need to remove: http://git.openstack.org/cgit/openstack/magnum/tree/magnum/tests/contrib/gate_hook.sh#n70 ?08:51
vdrokstrigazi: I think you need to remove the _ssh drivers on line 7008:52
vdrokpas-ha: ^^08:52
strigazibtw http://git.openstack.org/cgit/openstack/ironic/tree/doc/source/dev/dev-quickstart.rst#n48908:52
vdrokheh, yeah, we need to remove that :)08:53
strigazivdrok line 55 is good?08:53
vdrokyup08:53
strigaziShould we have line 70 like IRONIC_ENABLED_DRIVERS=agent_ipmitool"08:54
strigaziWill it work? Our ironic tests are broken for months :(08:54
vdrokstrigazi: nah, should be IRONIC_ENABLED_DRIVERS=fake,agent_ipmitool,pxe_ipmitool08:55
strigazivdrok ok, I'll git it a go08:55
vdrok:)08:55
strigazis/git/give x)08:55
strigazis/git/give xD08:55
pas-hastrigazi: on line 70, just remove the *_ssh drivers from the list of enabled drivers, otherwise you seem good.08:56
openstackgerritDao Cong Tien proposed openstack/python-ironicclient master: Add support for storage_interface to node and driver CLI  https://review.openstack.org/46618608:57
pas-haI'll make a patch to dev-quickstart today (or later after the docs migration is complete enough)08:57
openstackgerritVladyslav Drok proposed openstack/ironic master: Remove _ssh drivers from dev-quickstart  https://review.openstack.org/48091508:57
vdrokpas-ha: I'm faster :)08:58
openstackgerritparesh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get pending boot mode  https://review.openstack.org/46798208:58
strigazivdrok Thanks you08:58
strigazivdrok Thank you08:58
openstackgerritVladyslav Drok proposed openstack/ironic master: Remove _ssh drivers from dev-quickstart  https://review.openstack.org/48091508:59
pas-havdrok please rebase on top of https://review.openstack.org/#/c/479875/809:00
patchbotpatch 479875 - ironic - Move doc/source/dev to doc/source/contributor09:00
openstackgerritAnshul Jain proposed openstack/proliantutils master: Adds 'activate_license' for redfish systems  https://review.openstack.org/46878309:02
openstackgerritVladyslav Drok proposed openstack/ironic master: Remove _ssh drivers from dev-quickstart  https://review.openstack.org/48091509:03
vdrokpas-ha: done09:03
pas-ha+1 :)09:04
strigazivdrok magnum: https://review.openstack.org/480920 ci: Remove *_ssh ironic drivers09:04
patchbotpatch 480920 - magnum - ci: Remove *_ssh ironic drivers09:04
pas-hastrigazi: +109:06
vdrokthanks!09:07
strigazivdrok pas-ha folks, what is your timezone? I would need some help later with our ironic job.09:07
vdrokgmt+309:08
strigazivdrok cool, same as mine09:08
*** mvpnitesh has joined #openstack-ironic09:16
openstackgerritM V P Nitesh proposed openstack/ironic master: Allow setting {provisioning,cleaning}_network in node driver_info  https://review.openstack.org/46056409:19
*** j_wong has joined #openstack-ironic09:22
j_wonghi09:23
openstackgerritVladyslav Drok proposed openstack/ironic master: Test proposed nova devstack change and our grenade  https://review.openstack.org/47822009:24
j_wongmay i ask a question?09:25
*** Nisha_ has quit IRC09:26
mvpniteshyeah j_wong, you can09:26
mvpniteshmorning ironic and vdrok09:26
vdrokmorning j_wong mvpnitesh09:26
openstackgerritAnshul Jain proposed openstack/proliantutils master: Adds 'activate_license' for redfish systems  https://review.openstack.org/46878309:29
*** bmotz has joined #openstack-ironic09:30
*** bmotz has left #openstack-ironic09:31
j_wongmy customer want use nova attach volume to the node09:31
j_wongcause they are afraid of oneday the node storage is useless09:33
j_wongis somebody has a plan to provide this?09:35
j_wongit would be best that the volume can be backup09:36
openstackgerritparesh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get current boot mode  https://review.openstack.org/46669709:44
*** igordcard has quit IRC09:48
*** igordcard has joined #openstack-ironic09:48
vdrokj_wong: yes, people are working on this http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/boot-from-volume-reference-drivers.html09:48
j_wongreally?where can I find the docs or messages about it09:49
j_wongor links09:50
openstackgerritTuan Luong-Anh proposed openstack/python-ironicclient master: Rearrange existing documentation to fit the new standard layout  https://review.openstack.org/48001309:51
vdrokj_wong: here is the first one of the chain of patches https://review.openstack.org/463908. those are not yet merged09:51
patchbotpatch 463908 - ironic - Enable cinder storage interface for generic hardware09:51
openstackgerritparesh sao proposed openstack/ironic master: Added a condition for 'ilo' hardware type  https://review.openstack.org/47534209:54
j_wongthank you very much, I had already read it,but it's not enough09:54
openstackgerritGang Xu proposed openstack/pyghmi master: Add a description of the parameters  https://review.openstack.org/48094209:55
*** livelace has joined #openstack-ironic09:57
j_wongI'm off duty now, see you09:59
openstackgerritTuan Luong-Anh proposed openstack/python-ironicclient master: Rearrange existing documentation to fit the new standard layout  https://review.openstack.org/48001310:00
*** j_wong has quit IRC10:02
*** kiennt has quit IRC10:05
openstackgerritVladyslav Drok proposed openstack/ironic master: [DNM] Get rid of sourcing stackrc in grenade settings  https://review.openstack.org/48090510:08
*** MattMan_1 has quit IRC10:09
*** MattMan_1 has joined #openstack-ironic10:10
sambetts|afkTheJulia: api-ref is shelved until phase 2 of the process apprently, not sure why but thats the plan10:12
*** sambetts|afk is now known as sambetts10:12
*** mvpnitesh has quit IRC10:14
*** mvpnitesh has joined #openstack-ironic10:19
openstackgerritAnkit Kumar proposed openstack/proliantutils master: Adds 'reset_ilo_credentials' for redfish systems  https://review.openstack.org/47419110:26
*** mvpnitesh has quit IRC10:26
*** mvpnitesh has joined #openstack-ironic10:27
openstackgerritHironori Shiina proposed openstack/ironic master: Add API for volume resources to api-ref  https://review.openstack.org/47879810:28
masberhi, could I ask a question?10:34
masberI 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
vdrokmasber: enroll state is ironic api version 1.11 if i remember correctly. the default api version used by client is 1.910:38
vdrokand in 1.9 the node is created in available state already10:39
vdrokif 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
vdrokor you can export env variable not to type it every time10:40
vdrokexport IRONIC_API_VERSION=latest or export OS_BAREMETAL_API_VERSION latest10:40
masberok, stupid question, why is not using the latest version by default?10:41
*** rbartal has quit IRC10:45
vdrokmasber: for historical reasons :) we do plan to change to latest by default for CLI client10:48
openstackgerritHironori Shiina proposed openstack/ironic master: Add storage interface to Node in api-ref  https://review.openstack.org/48097510:50
masberok, 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
masberhttp://paste.openstack.org/show/614567/10:52
*** Nisha_Agarwal has joined #openstack-ironic10:56
*** retr0h has quit IRC10:57
*** retr0h has joined #openstack-ironic10:57
masberok, 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 documentation10:58
openstackgerritAnshul Jain proposed openstack/proliantutils master: Redfish: Adds 'activate_license' for redfish systems  https://review.openstack.org/46878311:12
openstackgerritVladyslav Drok proposed openstack/ironic stable/ocata: [DNM] Get rid of sourcing stackrc in grenade settings  https://review.openstack.org/48090411:13
openstackgerritHironori Shiina proposed openstack/python-ironicclient master: Add OSC commands for volume connector  https://review.openstack.org/42705311:19
openstackgerritAparna proposed openstack/ironic master: Support hpsum based `update_firmware` for iLO drivers  https://review.openstack.org/42257211:20
openstackgerritHironori Shiina proposed openstack/python-ironicclient master: Add OSC commands for volume target  https://review.openstack.org/42773811:24
*** mat128 has joined #openstack-ironic11:26
openstackgerritAparna proposed openstack/ironic master: Support hpsum based `update_firmware` for iLO drivers  https://review.openstack.org/42257211:27
*** fellypefca has joined #openstack-ironic11:41
*** litao__ has quit IRC11:42
pas-hamasber: 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 from11:42
*** deadcells has joined #openstack-ironic11:44
masberpas-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
TheJuliasambetts: Okay, thanks for following up on that11:46
TheJuliamasber: 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-haless 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 interfaces11:47
pas-halike 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-hawith 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 code11:50
pas-hanew driver == hardware tyoe11:51
pas-hatype11:51
openstackgerritparesh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get pending boot mode  https://review.openstack.org/46798211:52
*** jaypipes has joined #openstack-ironic11:52
openstackgerritWang KaiFeng proposed openstack/ironic master: Change ramdisk log filename template  https://review.openstack.org/48100111:53
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient master: Add tests for 'baremetal port create' command  https://review.openstack.org/33288011:54
*** rbudden has joined #openstack-ironic11:57
TheJuliahshiina: 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
masbersorry 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
openstackgerritAnshul Jain proposed openstack/proliantutils master: Redfish: Adds 'activate_license' for redfish systems  https://review.openstack.org/46878312:05
*** trown|outtypewww is now known as trown12:06
hshiinaTheJulia: thank you for the advice. i will create a follow-up patch for minor fixes.12:06
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient master: Add tests for 'baremetal port create' command  https://review.openstack.org/33288012:09
*** tuanluong has quit IRC12:10
*** oanson has quit IRC12:15
*** oanson has joined #openstack-ironic12:20
openstackgerritAparna proposed openstack/ironic master: Adds functionality test for hpsum based firmware update  https://review.openstack.org/47920212:21
*** Nisha_ has joined #openstack-ironic12:22
*** Nisha_Agarwal has quit IRC12:22
*** mvpnitesh has quit IRC12:24
*** dprince has joined #openstack-ironic12:24
*** bfournie has quit IRC12:29
*** mvpnitesh has joined #openstack-ironic12:33
*** crushil has quit IRC12:34
openstackgerritAnkit Kumar proposed openstack/proliantutils master: Adds 'reset_ilo_credentials' for redfish systems  https://review.openstack.org/47419112:35
*** crushil has joined #openstack-ironic12:35
*** chlong_ has joined #openstack-ironic12:37
*** mandre has joined #openstack-ironic12:37
*** mvpnitesh has quit IRC12:39
mandrehello, 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_3912:40
mandrea 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_3412:41
*** marios has quit IRC12:42
*** marios has joined #openstack-ironic12:43
openstackgerritMark Goddard proposed openstack/ironic master: Physical network aware VIF attachment  https://review.openstack.org/47040612:49
pas-hamasber: 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-haso that there's some room and avoid a very tight fit of image to disk that might fail deployment12:52
*** Nisha_brb has joined #openstack-ironic12:54
*** Nisha_ has quit IRC12:54
masberok, 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-ironic13:01
*** marios has quit IRC13:01
*** Goneri has joined #openstack-ironic13:03
*** bfournie has joined #openstack-ironic13:04
pas-hathat 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-haBasically 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 cpu13:05
*** rloo has joined #openstack-ironic13:07
pas-hawith 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
TheJuliao/ rloo13:07
rloogood morning TheJulia, pas-ha and everyone else!13:08
pas-hamorning rloo13:08
*** marios has joined #openstack-ironic13:08
openstackgerritSam Betts proposed openstack/ironic master: Update README to point at new doc location  https://review.openstack.org/47987713:09
openstackgerritSam Betts proposed openstack/ironic master: Move ironic dbsync tool docs into doc/source/cli  https://review.openstack.org/47987613:09
openstackgerritSam Betts proposed openstack/ironic master: Move doc/source/dev to doc/source/contributor  https://review.openstack.org/47987513:09
openstackgerritSam Betts proposed openstack/ironic master: Move api-ref into new doc/source/api location  https://review.openstack.org/47987213:09
*** jcoufal has joined #openstack-ironic13:10
*** makowals has quit IRC13:11
openstackgerritparesh sao proposed openstack/proliantutils master: Redfish: Adding the ability to get pending boot mode  https://review.openstack.org/46798213:12
*** makowals has joined #openstack-ironic13:13
TheJuliappiela: betherly: Could one of you take a look at: https://review.openstack.org/#/c/480180/ or https://review.openstack.org/#/c/47998013:16
patchbotpatch 480180 - ironic-ui - Set warning-is-error for documentation builds13:16
patchbotpatch 479980 - ironic-ui - Moving ironic-ui install content to install folder13:16
ppielaTheJulia: betherly: I will do that13:17
TheJuliaThanks, feel free to land13:19
*** mjturek has joined #openstack-ironic13:22
mjturekgm ironic!13:22
TheJuliagood morning mjturek13:22
mjturekgm TheJulia13:22
* TheJulia needs coffee, lots of coffee13:23
* mjturek sips already made coffee :)13:23
mjturekTheJulia: just curious, want me to abandon the boot.ipxe patch and propose something like you're suggesting?13:24
mjturekhttps://review.openstack.org/#/c/478690/ for ref13:25
patchbotpatch 478690 - ironic - Generate iPXE template on deploy for boot from volume13:25
mjturekand the title is wrong, whoops :-| it's boot.ipxe, not ipxe template13:25
TheJuliaI 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
mjturekyour follow up to the wirein?13:25
mjtureklemme check13:25
*** dprince has quit IRC13:26
mjtureksorry I mean for skip logic13:26
TheJulialet me pull it up, it has been a hectic week13:26
mjturekno worries!13:27
TheJuliahttps://review.openstack.org/#/c/479326/1/ironic/drivers/modules/iscsi_deploy.py13:27
patchbotpatch 479326 - ironic - BFV Deploy skip minor logging, logic, and test fixes13:27
TheJuliawell, close enough, you can see the question that got raised there13:27
*** baoli has joined #openstack-ironic13:27
mjturekyep 53313:27
* TheJulia powers up the espresso machine13:28
*** igordcard has quit IRC13:28
*** igordcard has joined #openstack-ironic13:29
*** dprince has joined #openstack-ironic13:29
mjturekTheJulia: I think you're referring to this patch https://review.openstack.org/#/c/476601/ which I need to give a bit of attention13:29
patchbotpatch 476601 - ironic - [WIP] Make storage calls agnostic to deploy method.13:29
TheJuliahmm power outage13:30
mjturekno rush btw - get some coffee13:30
TheJuliayeah, coffee.  I'm surprised I made it this long without any13:31
mjtureklolol13:31
*** makowals has quit IRC13:32
*** rodrigofm has joined #openstack-ironic13:33
*** makowals has joined #openstack-ironic13:35
rodrigofmTheJulia: thanks for the reviews, if u could take a look I'd appreciate very much  https://review.openstack.org/#/c/466872/13:35
patchbotpatch 466872 - ironic - Migrate oneview driver to use python-hpOneView13:36
*** kaifeng has joined #openstack-ironic13:36
TheJuliarodrigofm: Happy to, I'll try to take a look before the end of my day.13:37
TheJuliaAhh coffeeeee13:38
*** makowals has quit IRC13:40
*** krtaylor has quit IRC13:42
TheJuliamjturek: 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 prepare13:44
patchbotpatch 478690 - ironic - Generate iPXE template on deploy for boot from volume13:44
mjturekahhh gotcha13:44
TheJuliaThe more I think about it, the more that makes sense to me13:45
*** makowals has joined #openstack-ironic13:45
mjturekTheJulia: 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 generated13:47
openstackgerritNisha Agarwal proposed openstack/sushy master: Adds EthernetInterface to the library  https://review.openstack.org/45253213:47
mjturekI haven't looked that closely at the iscsi path so I might also be missing something13:48
TheJuliayeah, I think that is do-able13:49
mjturekcooool13:50
*** trown is now known as trown|brb13:50
TheJuliaI don't think they conflict or anything so they could actually land independently, at least I've not noticed anything in the "conflicts with" list13:50
TheJuliaahh,  yeah, non-bfs efforts.  I think I could just pull the wf-1 and we could let them land independently13:51
mjturekyeah pretty sure they could, but it'd at least address vlad's comment13:51
mjturekoooo13:51
TheJulias/bfs/bfv/13:52
vdrokgood morning TheJulia rloo and mjturek13:52
mjturekmorning vdrok :)13:52
TheJuliavlad's comment is as addressed by your patch though, at least It think it was.  going back and looking again13:52
TheJuliagood morning vdrok13:52
mjturekyeah I think it is13:53
rlooMorning vdrok, mjturek!13:53
TheJuliaugh, boot interface13:53
mjturekgm rloo ;)13:53
mjturek:)*13:53
TheJuliaI will need to go re-walk the code to be sure13:53
*** livelace has quit IRC13:54
openstackgerritDan Smith proposed openstack/ironic master: Mirror base grenade by not starting nova-compute in multicell mode  https://review.openstack.org/48108913:55
*** hw_wutianwei has quit IRC13:55
dansmithjlvillal: vdrok ^ total stab in the dark13:55
*** krtaylor has joined #openstack-ironic13:56
*** hw_wutianwei has joined #openstack-ironic13:56
vdrokdansmith: it's being merged already :) https://review.openstack.org/47731313:56
patchbotpatch 477313 - ironic - Set nomulticell flag for starting nova-compute in ...13:56
dansmithvdrok: dangit! :)13:57
dansmithvdrok: does it resolve the problem?13:57
TheJuliathe 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 IRC13:59
vdrokdansmith: 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 tests13:59
patchbotpatch 478220 - ironic - Test proposed nova devstack change and our grenade13:59
dansmithokay cool14:00
vdrokdansmith: 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_10547814:00
vdrokdansmith: the first one does not seem to be related to ironic. I'll now check the second one14:01
dansmithvdrok: oh, that's still with the nomulticell flag? I figured not14:01
vdrokyes, with it14:01
dansmithI looked at the config and it _is_ set in the config14:01
vdrokhrm14:01
*** baoli has joined #openstack-ironic14:01
dansmiththat's why I suspected the nomulticell flag, because it shouldn't be starting the extra conductor with that set14:02
dansmithwell, it shouldn't be using it anyway14:02
vdrokdansmith: 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_28914:03
*** trown|brb is now known as trown14:04
openstackgerritJulia Kreger proposed openstack/ironic master: BFV Deploy skip minor logging, logic, and test fixes  https://review.openstack.org/47932614:07
TheJuliamjturek: I just added the dependency ^^^14:07
mjturekthanks TheJulia :)14:08
mjturekcool14:08
*** hshiina has quit IRC14:10
vdrokdansmith: 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 is14:12
vdrokhttp://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_10547814:12
*** mvpnitesh has joined #openstack-ironic14:16
vdrokdansmith: heh, we have one more place https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L181014:16
vdrokI'll add to that patch14:16
dansmithvdrok: ah cool (sorry I'm getting sucked into a couple other things)14:16
vdroksure, np14:16
*** deadcells has quit IRC14:18
*** links has quit IRC14:18
openstackgerritVladyslav Drok proposed openstack/ironic master: Set nomulticell flag for starting nova-compute  https://review.openstack.org/47731314:19
*** aszeszo has quit IRC14:26
*** jaganathan has quit IRC14:26
vdrokdansmith: 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_91414:31
dansmithvdrok: for grenade, but not devstack14:33
dansmithvdrok: it's so we're not trying to upgrade from non-multicell to multicell14:34
dansmithvdrok: are you saying the other place you found is not in a grenade job?14:34
vdrokdansmith: yeah, I get that. but that ^^ is not grenade, just multinode job. so there nomulticell should not be added?14:34
vdrokyup14:34
dansmithright, you should not add it in a regular devstack job14:35
vdrokthat bit of log seems to setup things fine for nova14:35
vdrokdansmith: 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_28913914:35
dansmithyeah, then it's something else14:36
dansmithbut as you can see from the config, it's in there14:36
vdrokhrm, ok, I'll revert that bit for now14:36
dansmithmaybe the devstack plugin is starting conductor before the rest of lib/nova has actually written that line to the config?14:36
dansmithshould be able to tell from the timestamps I think14:37
openstackgerritVladyslav Drok proposed openstack/ironic master: Set nomulticell flag for starting nova-compute in grenade  https://review.openstack.org/47731314:38
*** rama_y has quit IRC14:38
*** Nisha_brb has quit IRC14:39
vdrokI see start_nova_api, then start_nova_rest, then start_nova_conductor14:41
dansmithvdrok: that's in base devstack right?14:41
openstackgerritMerged openstack/ironic-ui master: Set warning-is-error for documentation builds  https://review.openstack.org/48018014:41
dansmithI don't see where in your plugin you muck with starting nova services14:41
vdrokdansmith: here https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L181014:42
dansmithyeah but not nova conductor14:43
vdrokyup, we don't touch that14:43
dansmithvdrok: 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_25104414:45
dansmithohh14:45
dansmithI think that's remote14:45
vdrokdunno what it means :)14:47
dansmithhmm, maybe not14:47
openstackgerritMerged openstack/ironic-ui master: Moving ironic-ui install content to install folder  https://review.openstack.org/47998014:49
vdrokrloo: 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 patch14:50
dansmithah, I got it.. this is cell conductor trying to talk to the apidb14:50
vdrokdansmith: is there something we need to change on our side?14:51
dansmithvdrok: I dunno why it's happening, I just see that it is.. still digging14:51
*** dprince has quit IRC14:52
rloovdrok: 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
vdrokrloo: sure, np14:53
rloovdrok: thank YOU! also wrt 477313, you 'reverted' back to patch set 5, so I will +A it again, ok?14:54
vdrokyup :)14:54
rloovdrok: done14:54
*** joe817 has quit IRC14:58
*** skazi_ has joined #openstack-ironic14:58
*** joe817 has joined #openstack-ironic14:58
dansmithvdrok: ah, this is a reschedule15:00
dansmithvdrok: it tried once to build on the subnode, failed, and then called to cell conductor to reschedule, where it failed15:01
NobodyCamGood Morning Ironic'ers :)15:02
vdrokmhm15:02
vdrokmorning NobodyCam15:02
NobodyCam:) morning vdrok :)15:02
NobodyCampas-ha: happen to be around?15:05
dansmithvdrok: 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 cell15:13
dansmiththus rescheduling (which only considers a single cell) will never actually pick the other compute15:13
*** e0ne has quit IRC15:13
dansmithvdrok: 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 now15:14
vdrokdansmith: ok, I'll do it in separate patch then and we'll see. thx!15:15
dansmithvdrok: okay thanks15:16
*** rama_y has joined #openstack-ironic15:16
*** amarao has quit IRC15:16
*** amarao has joined #openstack-ironic15:17
*** kaifeng has quit IRC15:17
openstackgerritVladyslav Drok proposed openstack/ironic master: Set nomulticell flag for multinode job  https://review.openstack.org/48111415:18
openstackgerritVladyslav Drok proposed openstack/ironic master: Set nomulticell flag for multinode job  https://review.openstack.org/48111415:19
openstackgerritVladyslav Drok proposed openstack/ironic master: Set nomulticell flag for multinode job  https://review.openstack.org/48111415:20
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: Test OPNFV CI verify job. Please ignore.  https://review.openstack.org/36760815:20
rama_yGood morning, Ironic!15:21
rloosambetts: 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-ironic15:23
sambettsrloo: no sure what you mean, dev docs == contributor docs so they go into doc/source/contributor right?15:23
sambettsnot sure*15:23
NobodyCamGood Morning rama_y15:23
rloosambetts: maybe. is that where other projects are putting their dev docs?15:23
NobodyCammorning rloo & sambetts too :)15:23
rloomorning NobodyCam!15:24
NobodyCam:)15:24
rama_ymorning NobodyCam, rloo, sambetts15:24
sambettsrloo: as far as I'm aware yes15:24
rloosambetts: are we going to address dev docs later?15:24
sambettsrloo: no, I've already put together the patch for it15:24
rloosambetts: am just looking at the bottom of your chain and trying to understand what the new world is like15:24
rama_yrloo: you meant /developer/ docs right? I wondered the same.15:24
sambettsonly thing that will not be done in phase 1 is the api-ref15:25
rloosambetts: ah, ok. i will assume you followed that etherpad and know what you are doing!15:25
rloorama_y: guess it is ../contributor/15:25
sambettsrloo: top of the chain is this patch: https://review.openstack.org/#/c/479877/15:25
patchbotpatch 479877 - ironic - Update README to point at new doc location15:25
rama_yok15:25
rloosambetts: 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
sambettsrloo: 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 headings15:26
rloosambetts: 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-ironic15:28
*** e0ne has quit IRC15:28
*** aszeszo has joined #openstack-ironic15:29
sambettsrloo: 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 more15:30
rloosambetts: 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
sambettsyeah thats the main thing, although I hope things are ruffly in the sort of place they should end up15:32
sambettsand all the links are fixed15:32
rloosambetts: ++, 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_away15:33
*** aszeszo1 has joined #openstack-ironic15:33
*** aszeszo has quit IRC15:33
sambettsrloo: 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 location15:34
*** rama_y has quit IRC15:35
rloosambetts: python API docs? under contributor? guess i don't understand15:35
*** rama_y_ has joined #openstack-ironic15:35
rloosambetts: we don't have any python API docs. Do you mean the REST API?15:35
rloosambetts: or do you mean python-ironicclient python API?15:36
rloosambetts: oh, or do you mean our python classes?15:36
sambettsrloo: no, python API docs they look like this: https://docs.openstack.org/ironic/latest/py-modindex.html15:36
rloosambetts: ok, the python classes15:37
*** skazi_ has quit IRC15:37
sambettsrloo: yeah its the change to that is descrbie in bullet 8 on the etherpad15:37
sambettsdescribed*15:38
rloosambetts: i only looked at L9-17. I said High level, didn't I? :-)15:39
sambettsyup :)15:39
rloosambetts: anyway, will think (maybe) about that later.15:39
sambettsheh yeah the spec https://specs.openstack.org/openstack/docs-specs/specs/pike/os-manuals-migration.html is a lot to read through15:39
rloosambetts: thx for making all these changes, btw15:40
sambettstaking 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 /dev15:40
sambettse.g. not all contributors are developers, so it will have developer and non-developer contribution docs15:41
sambettsnot sure we have any non-dev contribuor docs right now :/15:41
mgoddard_qq: should I upgrade IPA or ironic first?15:42
sambettsmgoddard_: IPA15:43
rloosambetts: 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
rloosambetts: 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-ironic15:44
mgoddard_thanks sambetts15:44
rloosambetts: we should maybe add a release note somewhere about the doc changes.15:44
sambettsrloo: 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 go15:45
rloosambetts: ok, that would be good. we should probably still put a release note but can do that later if we remember.15:46
TheJuliathat is my understanding as well15:46
sambettsjust checked and if you try to access: http://docs.openstack.org/developer/ironic/mitaka/ it redirects you to the new location15:47
jlvillalvdrok: 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
rloosambetts: do we have end users?15:50
jlvillals/is one of/as one of/15:50
vdrokjlvillal: morning, thanks!15:51
rloosambetts: i wonder if we want user/, or if it should all go in admin/15:51
vdrokwe still might need the one for multinode job though :(15:51
sambettsrloo: we have users (admins of nodes enrolled into ironic), and we have operators (admins of ironic the service)15:52
vdrokvsaienko 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 there15:52
rloosambetts: so would it make more sense to put that stuff under admin/ ?15:52
rloosambetts: eg https://review.openstack.org/#/c/479874/8/doc/source/user/index.rst15:52
patchbotpatch 479874 - ironic - Move operator docs into into doc/source/admin15:52
*** hshiina has joined #openstack-ironic15:52
sambettsrloo: 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-create15:53
openstackgerritHironori Shiina proposed openstack/ironic master: Enable cinder storage interface for generic hardware  https://review.openstack.org/46390815:53
sambettswhich aren't the same person, /admin is for the operator of the cloud, /user is the person given the admin role in openstack15:54
*** tesseract has quit IRC15:54
rloosambetts: the person doing ironic node-create is an operator?15:54
rloosambetts: ok, so ironic's definition of 'user' is the user that uses our REST API?15:54
sambettsrloo: yeah15:55
rloosambetts: just want to make sure that jives with what 'user/' means for the rest of OpenStack projects.15:55
rloosambetts: i'll assume it does :-)15:55
sambettsthat 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 do15:56
TheJuliaBFV meeting in #openstack-meeting-516:01
*** penick has quit IRC16:03
pas-haNobodyCam: here16:05
pas-haNobodyCam: you were looking for me?16:06
NobodyCamhey 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-haNobodyCam: 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
NobodyCamahh16:09
pas-haSame happened with some other drivers, like VirtualBox ones - they were just removed from the tree w/o moving to staging16:09
pas-hasince nobody proposed those to staging - means noone is interested or have resources to continue maintaining16:10
*** hjensas has quit IRC16:11
*** hjensas has joined #openstack-ironic16:12
*** amarao has quit IRC16:14
*** fellypefca has quit IRC16:18
*** fellypefca has joined #openstack-ironic16:19
*** ricardoas has quit IRC16:20
*** ricardoas has joined #openstack-ironic16:20
*** xavierr has quit IRC16:20
*** xavierr has joined #openstack-ironic16:20
*** lucasagomes is now known as lucas-afk16:21
*** dprince has joined #openstack-ironic16:23
*** tuanla_fujitsu has joined #openstack-ironic16:25
*** mvpnitesh has quit IRC16:30
vdrokjlvillal: rloo seems like we'll need to set nomulticell everywhere in our jobs, not only grenade16:32
*** karthiks has quit IRC16:32
rloovdrok: okey dokey16:33
*** ralonsoh has quit IRC16:34
* sambetts --> home 16:35
sambettsnight all!16:35
*** sambetts is now known as sambetts|afk16:35
rloogood night sambetts!16:35
sambetts|afkcya rloo o/16:35
*** ChubYann has joined #openstack-ironic16:36
*** racedo has quit IRC16:44
*** mduggan has joined #openstack-ironic16:46
*** hshiina has quit IRC16:47
openstackgerritPeter Piela proposed openstack/ironic-ui master: Add property-collection-editor directive  https://review.openstack.org/48115216:56
*** trown is now known as trown|lunch16:56
*** Nisha_Agarwal has joined #openstack-ironic16:57
*** mjturek1 has joined #openstack-ironic16:59
jlvillalvdrok: Oh. Okay16:59
*** mjturek has quit IRC16:59
mdugganIn 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 fai16:59
mdugganlure ?16:59
*** baoli has quit IRC17:00
*** baoli has joined #openstack-ironic17:01
*** mgoddard_ has quit IRC17:02
mdugganforgot to include the port info17:07
mdugganhttps://thepasteb.in/p/DRhjG7NqyQ8Hy17:07
*** fragatina has joined #openstack-ironic17:11
*** harlowja has joined #openstack-ironic17:13
*** zzzeek has quit IRC17:14
*** baoli has quit IRC17:16
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-inspector stable/ocata: [Devstack] cleanup upgrade settings  https://review.openstack.org/48115717:25
*** fragatina has quit IRC17:25
*** udesale has quit IRC17:25
*** baoli has joined #openstack-ironic17:25
*** fragatina has joined #openstack-ironic17:25
TheJuliamduggan: the first node is in deploy failed, which has nothing to do with ironic_inspector17:26
mdugganTheJulia: but it goes into deploy failed after the inspector lookup failure .....17:28
TheJuliamduggan: 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 IRC17:31
*** joe817 has joined #openstack-ironic17:31
*** ijw has joined #openstack-ironic17:31
*** tuanla_fujitsu has quit IRC17:32
*** zzzeek has joined #openstack-ironic17:33
*** ijw has quit IRC17:33
*** ijw has joined #openstack-ironic17:34
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-inspector stable/newton: [Devstack] cleanup upgrade settings  https://review.openstack.org/48116317:34
*** ijw has quit IRC17:35
*** krtaylor has quit IRC17:35
mdugganTheJulia: Are you saying that this IPA/inspector lookup failure is irrelevant to the node install ?17:37
*** zzzeek has quit IRC17:37
*** marios has quit IRC17:38
mdugganI'm assuming that is it this failure causes the ironic node installation to abort17:38
openstackgerritRamamani Yeleswarapu proposed openstack/ironic stable/ocata: [Devstack] cleanup upgrade settings  https://review.openstack.org/48116417:40
*** zzzeek has joined #openstack-ironic17:41
openstackgerritRamamani Yeleswarapu proposed openstack/ironic stable/newton: [Devstack] cleanup upgrade settings  https://review.openstack.org/48117017:50
TheJuliamduggan: 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
TheJuliamduggan: process wise, a booting ramdisk shouldn't call IPA17:52
TheJuliaunless it is for discovery17:52
TheJuliamduggan: 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 IRC17:53
*** fragatin_ has joined #openstack-ironic17:59
*** zzzeek has joined #openstack-ironic18:00
openstackgerritMerged openstack/ironic master: Move operator docs into into doc/source/admin  https://review.openstack.org/47987418:01
*** etingof has joined #openstack-ironic18:03
*** fragatina has quit IRC18:03
*** baoli has quit IRC18:06
*** zzzeek_ has joined #openstack-ironic18:08
*** zzzeek has quit IRC18:09
*** trown|lunch is now known as trown18:11
*** Nisha_Agarwal has quit IRC18:12
*** krtaylor has joined #openstack-ironic18:12
*** ijw has joined #openstack-ironic18:14
openstackgerritMerged openstack/python-ironicclient master: switch from oslosphinx to openstackdocstheme  https://review.openstack.org/47999918:16
*** ricardoas has quit IRC18:16
*** ijw has quit IRC18:18
*** jtomasek has joined #openstack-ironic18:18
*** krtaylor has quit IRC18:28
*** etingof has quit IRC18:29
*** aszeszo1 has quit IRC18:31
*** ijw has joined #openstack-ironic18:31
*** krtaylor has joined #openstack-ironic18:33
*** ijw has quit IRC18:36
*** fellypefca has quit IRC18:37
*** ChubYann has quit IRC18:37
*** ijw has joined #openstack-ironic18:40
*** mjura has quit IRC18:46
*** jtomasek has quit IRC18:46
*** ijw has quit IRC18:47
*** ChubYann has joined #openstack-ironic18:51
*** mjturek has joined #openstack-ironic18:55
*** mjturek1 has quit IRC18:55
*** ccamacho has quit IRC19:05
*** jtomasek has joined #openstack-ironic19:11
*** pcaruana has quit IRC19:12
*** adisky__ has quit IRC19:14
*** makowals has quit IRC19:26
*** jtomasek has quit IRC19:28
*** fragatin_ has quit IRC19:28
*** fragatina has joined #openstack-ironic19:29
*** makowals has joined #openstack-ironic19:34
openstackgerritMerged openstack/ironic master: Move doc/source/dev to doc/source/contributor  https://review.openstack.org/47987519:39
openstackgerritMerged openstack/ironic master: Move ironic dbsync tool docs into doc/source/cli  https://review.openstack.org/47987619:42
*** makowals has quit IRC19:42
openstackgerritMerged openstack/ironic master: Update README to point at new doc location  https://review.openstack.org/47987719:42
*** mjturek has quit IRC19:45
rlooTheJulia: qq, wrt https://review.openstack.org/#/c/479142/ -- it has to be totally powered off, not soft power off, right?19:55
patchbotpatch 479142 - ironic - Add node power state validation to volume resource... (MERGED)19:55
openstackgerritJohn L. Villalovos proposed openstack/ironic master: DevStack: Add configuration for boot-from-volume  https://review.openstack.org/46633319:55
TheJuliastate must be POWER_OFF19:56
rlooTheJulia: ok, thx.19:56
TheJuliano problem, thanks :)19:57
*** makowals has joined #openstack-ironic20:00
*** ijw has joined #openstack-ironic20:06
*** mjturek has joined #openstack-ironic20:13
*** ijw has quit IRC20:13
*** dprince has quit IRC20:17
*** ricardoas has joined #openstack-ironic20:29
*** ijw has joined #openstack-ironic20:32
*** mandre_away is now known as mandre20:32
*** ijw has quit IRC20:37
*** deadcells has joined #openstack-ironic20:40
*** jcoufal has quit IRC20:47
*** ijw has joined #openstack-ironic20:53
*** krtaylor has quit IRC20:54
*** trown is now known as trown|outtypewww20:56
*** ijw has quit IRC21:00
*** ijw has joined #openstack-ironic21:00
*** mandre is now known as mandre_away21:10
*** baoli has joined #openstack-ironic21:12
*** baoli has quit IRC21:13
*** baoli has joined #openstack-ironic21:16
rlooTheJulia: another qq, for volume connectors, you can have more than one associated with the same node?21:18
TheJuliayes21:19
TheJuliathere are database constraints around that and the boot_index column if memory serves21:19
*** Goneri has quit IRC21:20
rlooTheJulia: am looking and i don't see any constraint on the volume connector. but i see for volume target.21:20
rlooTheJulia: wondering if i have forgotten how the db code works.21:21
*** joe817 has quit IRC21:21
TheJuliaoh, 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 relationship21:22
TheJuliahttps://github.com/openstack/ironic/blob/master/ironic/db/sqlalchemy/alembic/versions/daa1ba02d98_add_volume_connectors_table.py#L4221:23
rlooTheJulia: ok, that jives with the code, although targets are constrained on boot_index+node_id.21:23
TheJuliayeah, that is correct21:24
rlooTheJulia: 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-ironic21:25
TheJuliaI really don't remember that, but most users will only have one21:25
rlooTheJulia: no worries, it is all becoming a blur to me anyway :)21:26
TheJuliaYeah, I know that feeling _really_ well21:26
*** krtaylor has joined #openstack-ironic21:35
*** makowals has quit IRC21:37
*** deadcells has quit IRC21:40
jlvillalAnyone else have any problems running py27 unit tests for python-ironicclient?21:43
jlvillalI am checked out to master and have done: $ tox -vvv -e py27 -r21:43
jlvillalAnd getting errors :(21:44
jlvillalrloo: ^^ ??21:44
rloojlvillal: haven't tried for awhile. although it seems to me that i may have had errors 'randomly' before. sec, let me try.21:44
rloojlvillal:  a normal 'tox' works for me. let me try with your exact cmd.21:45
jlvillalrloo: So py35 works for me. But not py2721:46
* jlvillal wonders what is wrong21:46
rloojlvillal: did you do a fresh build? maybe i should delete my .tox first.21:46
jlvillalrloo: The '-r' should delete the .tox/ directory of the target21:47
rloojlvillal: yes, i just noticed that, which is why it is taking so long :-) stay tuned...21:47
rloojlvillal: still worked for me.21:47
rloojlvillal: what kind of errors are you seeing?21:47
jlvillalrloo: Okay. Something wacky here21:48
jlvillalrloo: http://paste.openstack.org/show/614650/21:48
jlvillalAnd a bunch more like that. About 1221:48
*** makowals has joined #openstack-ironic21:49
rloojlvillal: 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
jlvillalrloo: I'm assuming something is weird on my system.21:50
rloojlvillal: i'd like to assume that too :) :(21:51
* TheJulia looks21:53
TheJuliainteresting....21:53
* TheJulia switches computers21:54
TheJuliajlvillal: current master branch passes just fine for me on py2721:56
*** makowals has quit IRC21:57
jlvillalrloo: TheJulia: I fixed it :)  $ rm -rf ~/.cache/pip/21:57
jlvillalThanks!21:57
rloojlvillal: rm is your friend! glad you figured it out.21:57
TheJuliaawesome21:58
* TheJulia goes back to not being worried21:58
*** racedo has joined #openstack-ironic22:05
*** makowals has joined #openstack-ironic22:05
*** etingof has joined #openstack-ironic22:09
*** racedo has quit IRC22:16
*** ijw has quit IRC22:16
*** bfournie has quit IRC22:16
*** ijw has joined #openstack-ironic22:18
*** sauloaislan has quit IRC22:22
*** mduggan has quit IRC22:22
*** rodrigofm has quit IRC22:22
*** nicodemos has quit IRC22:22
*** ijw has quit IRC22:22
*** dprince has joined #openstack-ironic22:25
*** ijw has joined #openstack-ironic22:26
*** hw_wutianwei1 has joined #openstack-ironic22:27
*** hw_wutianwei has quit IRC22:29
*** hw_wutianwei1 is now known as hw_wutianwei22:29
*** ijw has quit IRC22:40
*** mjturek has quit IRC22:52
*** mat128 has quit IRC23:01
openstackgerritJohn L. Villalovos proposed openstack/python-ironicclient master: WIP: Require keyword arguments to call common_filters  https://review.openstack.org/48131623:08
*** ijw has joined #openstack-ironic23:17
*** ijw has quit IRC23:22
*** baoli has quit IRC23:26
openstackgerritRichard G. Pioso proposed openstack/ironic master: Revert "Wait until iDRAC is ready before out-of-band cleaning"  https://review.openstack.org/48131823:30
*** etingof has quit IRC23:33
*** rpioso has quit IRC23:33
*** joe817 has joined #openstack-ironic23:35
*** dprince has quit IRC23:35
*** joe817 has quit IRC23:35
*** joe817 has joined #openstack-ironic23:36
*** Sukhdev has joined #openstack-ironic23:40
*** ijw has joined #openstack-ironic23:41
-openstackstatus- NOTICE: nb03.openstack.org has been cleaned up and rebooted, and should return to building rotation23:44
*** ijw has quit IRC23:48

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!