Friday, 2016-12-16

*** wajdi has quit IRC00:00
*** rama_y has quit IRC00:00
*** lindycoder has joined #openstack-ironic00:01
*** harlowja has quit IRC00:06
*** wajdi_ has quit IRC00:07
openstackgerritChris Krelle proposed openstack/ironic: [WIP / POC] Add Intel Node Manager sensor data collection  https://review.openstack.org/41046500:07
*** trown|outtypewww has quit IRC00:09
*** bfournie has joined #openstack-ironic00:11
*** trown has joined #openstack-ironic00:19
*** mtanino has quit IRC00:23
*** aNuposic has quit IRC00:24
*** causten has quit IRC00:25
*** Syed__ has quit IRC00:25
*** aNuposic has joined #openstack-ironic00:26
*** harlowja has joined #openstack-ironic00:34
*** Nisha_Agarwal has joined #openstack-ironic00:40
openstackgerritAnup Navare proposed openstack/ironic: Config drive support for Ceph  https://review.openstack.org/40763900:42
*** Nisha_Agarwal has quit IRC00:45
*** SerenaFeng has joined #openstack-ironic01:01
*** SerenaFeng has quit IRC01:02
*** SerenaFeng has joined #openstack-ironic01:03
*** aNuposic has quit IRC01:05
*** tuanluong has joined #openstack-ironic01:05
*** phuongnh has joined #openstack-ironic01:06
openstackgerritMerged openstack/ironic-ui: Don't include openstack/common in flake8 exclude list  https://review.openstack.org/40228001:06
*** rama_y has joined #openstack-ironic01:12
*** rloo has quit IRC01:20
*** SerenaFeng has quit IRC01:24
*** zhangjl has joined #openstack-ironic01:25
*** harlowja has quit IRC01:25
*** tuanluong has quit IRC01:26
*** lindycoder has quit IRC01:30
*** SerenaFeng has joined #openstack-ironic01:34
*** hw_wutianwei has joined #openstack-ironic01:44
*** jralph has quit IRC01:48
*** yufei has joined #openstack-ironic01:57
*** tuanluong has joined #openstack-ironic02:13
openstackgerritjiang wei proposed openstack/ironic: Adjust python format for util.py  https://review.openstack.org/41158302:35
*** gcb has joined #openstack-ironic02:45
*** soliosg has quit IRC02:50
*** fragatina has quit IRC03:07
*** fragatina has joined #openstack-ironic03:08
*** fragatina has quit IRC03:12
*** fragatina has joined #openstack-ironic03:12
*** fragatin_ has joined #openstack-ironic03:15
*** fragatina has quit IRC03:15
*** aNuposic has joined #openstack-ironic03:17
*** fragatin_ has quit IRC03:18
*** fragatina has joined #openstack-ironic03:19
*** fragatina has quit IRC03:24
*** rama_y has quit IRC03:24
*** amotoki has quit IRC03:33
*** amotoki has joined #openstack-ironic03:34
*** amotoki has quit IRC03:35
*** amotoki has joined #openstack-ironic03:36
*** vikrant has joined #openstack-ironic03:40
*** amotoki has quit IRC03:42
*** baoli has joined #openstack-ironic03:44
*** SerenaFeng has quit IRC03:58
*** aNuposic has quit IRC04:01
*** aNuposic has joined #openstack-ironic04:01
zer0c00lif i have to override a configuration while spawning a service can i use something like '--section-option value'04:04
zer0c00lFor example if i want to override the rabbitmq hostname04:04
zer0c00lcan i call the service with '--rabbitmq-host localhost' ?04:05
zer0c00lthis will override the options specified in --config-file ?04:05
*** aNuposic has quit IRC04:06
*** baoli has quit IRC04:17
openstackgerritTuan Luong-Anh proposed openstack/ironic-python-agent: Use oslo_serialization.base64 to follow OpenStack Python3  https://review.openstack.org/41113104:29
openstackgerritMerged openstack/ironic-lib: Use oslo_serialization.base64 to follow OpenStack Python3  https://review.openstack.org/41112405:08
*** rama_y has joined #openstack-ironic05:20
*** links has joined #openstack-ironic05:21
*** fragatina has joined #openstack-ironic05:26
openstackgerritAparna proposed openstack/proliantutils: Add Sanitize disk erase support using SSA  https://review.openstack.org/39820205:29
*** fragatina has quit IRC05:36
*** fragatina has joined #openstack-ironic05:36
*** SerenaFeng has joined #openstack-ironic05:39
*** SerenaFeng has quit IRC05:57
*** Nisha_Agarwal has joined #openstack-ironic06:00
*** jaosorior has joined #openstack-ironic06:02
*** SerenaFeng has joined #openstack-ironic06:03
*** rcernin has quit IRC06:13
*** SerenaFeng has quit IRC06:24
*** phuongnh has quit IRC06:31
*** phuongnh has joined #openstack-ironic06:32
*** rcernin has joined #openstack-ironic06:37
*** rcernin has quit IRC06:43
*** rama_y has quit IRC06:44
*** bks has joined #openstack-ironic06:49
*** mjura has joined #openstack-ironic06:53
*** rcernin has joined #openstack-ironic06:54
*** ChubYann has quit IRC06:58
*** mjura has quit IRC07:00
*** mjura has joined #openstack-ironic07:00
*** fxpester has joined #openstack-ironic07:02
*** phuongnh has quit IRC07:02
openstackgerritAndrey Shestakov proposed openstack/bifrost: Fixes to allow install Ironic on remote node  https://review.openstack.org/40921307:03
*** phuongnh has joined #openstack-ironic07:03
*** SerenaFeng has joined #openstack-ironic07:18
*** adreznec has quit IRC07:21
*** bnemec has quit IRC07:21
trananhkmaxek, hi, now I am watching for rolling upgrade feature of Ironic, I already reviewed the spec carefully, I also understood how did nova do online data migration. So I think that I am also willing to help with this task, please feel free to let me know what I can do to help you :)07:23
pas-hamorning Ironic :)07:31
tuanluongmornig pas-ha07:32
tuanluongpas-ha, Can i ask you about https://review.openstack.org/#/c/410537/07:33
patchbotpatch 410537 - ironic - Fix ironic dev-quickstart doc bug07:33
pas-haleft a comment, still think we should fix the bug first. anyway thanks for the patch as it kind of helped to discover that bug :)07:36
*** bnemec has joined #openstack-ironic07:37
*** derekh has joined #openstack-ironic07:40
*** derekh has quit IRC07:47
*** e0ne has joined #openstack-ironic07:52
openstackgerritTuan Luong-Anh proposed openstack/ironic: Fix ironic dev-quickstart doc bug  https://review.openstack.org/41053707:54
*** moshele has joined #openstack-ironic07:56
*** pcaruana has joined #openstack-ironic07:56
tuanluonghello pas-ha i don't understand why Mario Villaplana don't hit the bugs07:57
pas-hame neither, investigating it currently07:57
*** tesseract has joined #openstack-ironic07:57
*** tesseract is now known as Guest3130407:58
tuanluongpas-ha, it look like have a note about that.Note The devstack “demo” tenant is now granted the “baremetal_observer” role and thereby has read-only access to ironic’s API. This is sufficient for all the examples below. Should you want to create or modify bare metal resources directly (ie. through ironic rather than through nova) you will need to use the devstack “admin” tenant.08:02
pas-ha+08:02
tuanluongpas-ha, I thinsk my commit don't direct to your bugs. it just like a update document08:03
pas-hayou can make it "Related-Bug: #" if you like, or drop the bug ref altogether08:04
tuanluongok thanks pas-ha08:04
openstackgerritTuan Luong-Anh proposed openstack/ironic: Fix ironic dev-quickstart doc bug  https://review.openstack.org/41053708:05
*** sacharya has quit IRC08:06
tuanluongpas-ha, what Operation System you are using?08:07
*** sacharya has joined #openstack-ironic08:08
pas-haubuntu 16.0408:08
*** fragatina has quit IRC08:09
*** zhugaoxiao has quit IRC08:09
tuanluongpas-ha,me too. i am continuing don't understand why just me hit the bug08:09
*** adreznec has joined #openstack-ironic08:09
*** zhugaoxiao has joined #openstack-ironic08:10
*** charlledaniel has quit IRC08:12
*** sacharya has quit IRC08:12
*** charlledaniel has joined #openstack-ironic08:13
*** ralonsoh has joined #openstack-ironic08:15
*** e0ne has quit IRC08:29
*** yufei has quit IRC08:30
*** amoralej|off is now known as amoralej08:31
*** yufei has joined #openstack-ironic08:32
*** jaosorior has quit IRC08:32
*** jaosorior has joined #openstack-ironic08:33
*** hogepodge has quit IRC08:42
*** moshele has quit IRC08:44
*** hogepodge has joined #openstack-ironic08:44
*** gcb has quit IRC08:54
*** swatson_ has quit IRC08:54
*** ccamacho|out is now known as ccamacho08:55
*** swatson_ has joined #openstack-ironic08:55
*** jaosorior has quit IRC08:55
*** sborkows has joined #openstack-ironic08:56
sborkowsmorning, all08:57
*** ohamada has joined #openstack-ironic08:58
*** aparnav has joined #openstack-ironic08:59
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-ironic09:02
*** lucas-afk is now known as lucasagomes09:04
lucasagomesmorning all09:04
*** SerenaFeng has quit IRC09:05
*** sacharya has joined #openstack-ironic09:09
*** sacharya has quit IRC09:13
*** gcb has joined #openstack-ironic09:17
*** Romanenko_K has joined #openstack-ironic09:18
*** SerenaFeng has joined #openstack-ironic09:18
*** lucasagomes is now known as lucas-brb09:19
aarefievmorning all!09:19
openstackgerritAnton Arefiev proposed openstack/ironic-specs: Torrent-based provisioning  https://review.openstack.org/31109109:20
*** athomas has joined #openstack-ironic09:28
*** trananhkma has quit IRC09:32
sborkowsmorning lucasagomes, aarefiev09:37
*** derekh has joined #openstack-ironic09:41
*** SerenaFeng has quit IRC09:43
aarefievmorning sborkows09:44
*** Linkmark has joined #openstack-ironic09:44
*** dsneddon has quit IRC09:46
*** e0ne has joined #openstack-ironic09:52
*** mjura_ has joined #openstack-ironic09:53
*** fxpester has quit IRC09:54
sborkowsCan somebody who is familiar with IPA hardware managers look at https://review.openstack.org/#/c/391836/ ?09:58
patchbotpatch 391836 - ironic-python-agent - Add a new Hardware Manager for CNA network card09:58
*** e0ne has quit IRC10:02
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Use kvm for ironic VMs when possible  https://review.openstack.org/40094110:04
*** milan has joined #openstack-ironic10:07
milanmorning Ironic! :)10:08
*** mjura_ has quit IRC10:10
*** yufei has quit IRC10:13
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Make _send_sensors_data concurrent  https://review.openstack.org/40742910:17
sergekMorning Ironic milan aarefiev sborkows lucas-brb pas-ha10:17
sborkowsmorning milan, sergek10:18
aarefievmorning milan, sergek :)10:18
*** openstackgerrit has quit IRC10:18
milanmornin aarefiev sborkows sergek! :)10:18
sborkowssomebody knows which resources are exposed in Ironic API atm?10:20
sborkowsNode, Port, Portgroup, Chassis - anything else?10:20
*** openstackgerrit has joined #openstack-ironic10:26
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Functional tests for portgroups in ironicclient  https://review.openstack.org/29772710:26
*** priteau has joined #openstack-ironic10:30
*** lucas-brb is now known as lucasagomes10:31
openstackgerritJavier Peña proposed openstack/ironic-inspector: Fix test when running with SQLite 3.7.17 from CentOS 7  https://review.openstack.org/41174610:31
*** zhangjl has left #openstack-ironic10:34
lucasagomeshi folks, if you have some time mind taking a look at this vbmc bug: https://review.openstack.org/#/c/409755/ ? It makes vbmc return the correct errors when something goes south, w/o it, vbmc can lead to deployment failures with ironic10:46
patchbotpatch 409755 - virtualbmc - Return proper errors on BMC action failures10:46
lucasagomesbeucase ironic won't retry some operators and will think it succeed in the first attempt10:46
lucasagomesty :-)10:46
*** phuongnh has quit IRC10:48
jrolllucasagomes: lgtm, +A10:50
jrollmorning y'all10:50
lucasagomesjroll, yay cheers! Good morning10:50
lucasagomesI'm about to approve: https://review.openstack.org/#/c/398417 someone else wants to take a quick look before ?10:59
patchbotpatch 398417 - ironic - Add portgroup configuration fields10:59
sborkowsmorning jroll11:00
*** MattMan has quit IRC11:01
*** MattMan has joined #openstack-ironic11:02
openstackgerritGalyna Zholtkevych proposed openstack/ironic: ETAGs maintenance for all api node's requests  https://review.openstack.org/39221311:04
*** sacharya has joined #openstack-ironic11:06
*** e0ne has joined #openstack-ironic11:07
*** MattMan has quit IRC11:10
*** MattMan has joined #openstack-ironic11:10
milanmorning lucasagomes jroll! :)11:10
lucasagomesmilan, yo! morning11:11
milan:)11:11
*** sacharya has quit IRC11:11
* milan -> food&relocate11:11
*** bks has quit IRC11:12
*** hw_wutianwei has quit IRC11:13
pas-hamorning jroll, mind removing your -2? :) https://review.openstack.org/#/c/397847/11:15
patchbotpatch 397847 - ironic - Remove iBoot, WoL and AMT drivers11:15
*** MattMan has quit IRC11:15
*** milan has quit IRC11:16
*** MattMan has joined #openstack-ironic11:16
*** fxpester has joined #openstack-ironic11:17
jrollpas-ha: I had it on my todo list for today, thanks for the reminder :)11:20
jrolldone11:20
jrollI'll try to get to landing that today11:21
pas-hathanks :)11:21
vdrokgood morning all! pas-ha tuanluong lucasagomes aarefiev jroll :)11:23
aarefievmorning vdrok, jroll11:24
jrollmorning lucasagomes, sborkows, pas-ha, vdrok, aarefiev :)11:25
*** bdeschenes has joined #openstack-ironic11:26
openstackgerritVladyslav Drok proposed openstack/ironic: Fix policy dict checkers  https://review.openstack.org/41176411:31
sborkowsCan somebody who is familiar with IPA hardware managers look at https://review.openstack.org/#/c/391836/ ?11:33
patchbotpatch 391836 - ironic-python-agent - Add a new Hardware Manager for CNA network card11:33
jrollsborkows: I'll put it on my list for today11:34
sborkowsjroll, thanks :)11:34
jrollnp :)11:35
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Functional tests for port groups in ironicclient  https://review.openstack.org/29772711:38
*** skramaja has joined #openstack-ironic11:38
*** jaosorior has joined #openstack-ironic11:38
openstackgerritMerged openstack/virtualbmc: Return proper errors on BMC action failures  https://review.openstack.org/40975511:39
lucasagomesw00t11:40
jrolldtantsur|afk: would you reconsider your vote here given the comments? https://review.openstack.org/#/c/404436/11:43
patchbotpatch 404436 - ironic - Move CONF.service_available.ironic to our plugin11:43
*** jkilpatr has quit IRC11:44
*** MattMan has quit IRC11:51
*** MattMan has joined #openstack-ironic11:51
*** milan|afk has joined #openstack-ironic11:51
*** MattMan has quit IRC11:53
openstackgerritMerged openstack/python-ironicclient: Fix API object representation in unittests  https://review.openstack.org/41011211:53
*** MattMan has joined #openstack-ironic11:53
*** MattMan has quit IRC11:55
*** MattMan has joined #openstack-ironic11:56
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add ironic resources CRUD notifications  https://review.openstack.org/35654111:58
lucasagomesanother quick one: https://review.openstack.org/#/c/410753/ I need to make the name more generic to be able to change the images in devstack to a image that supports UEFI (and bonding)12:01
patchbotpatch 410753 - ironic - DevStack: Make $IRONIC_IMAGE_NAME less dependent o...12:01
jrolllucasagomes: done12:06
lucasagomesjroll, gracias!12:06
*** Nisha_Agarwal has quit IRC12:06
jrollde nada :)12:06
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add node maintenance notifications  https://review.openstack.org/39623912:08
openstackgerritYuriy Zveryanskyy proposed openstack/ironic: Add node console notifications  https://review.openstack.org/39781212:10
*** dtantsur|afk is now known as dtantsur12:16
dtantsurhey ironic12:16
*** jkilpatr has joined #openstack-ironic12:16
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Describe possible exception in docstring  https://review.openstack.org/41140812:16
dtantsurjroll, did they fixed grenade to properly configure tempest?12:17
aparnavhey jroll, Can you take a look at spec https://review.openstack.org/#/c/410161/ ?12:17
patchbotpatch 410161 - ironic-specs - In-band hpsum firmware update for iLO drivers12:17
vdrokmorning dtantsur12:19
openstackgerritVasyl Saienko proposed openstack/ironic: Add Virtual Network Interface REST APIs  https://review.openstack.org/40424012:20
openstackgerritVasyl Saienko proposed openstack/ironic: Add Virtual Network Interface RPC APIs  https://review.openstack.org/40423912:20
openstackgerritVasyl Saienko proposed openstack/ironic: Add Virtual Network Interface Driver APIs  https://review.openstack.org/32704612:20
jrolldtantsur: yes, that patch landed12:20
jrolldtantsur: and good morning :)12:20
jrollaparnav: I will add it to my todo list but that list is very long and this is low on the priority list12:21
dtantsurjroll, after short investigation, I've approved your patch, seems good nowadays12:21
aparnavjroll, thanks12:21
jrolldtantsur: awesome, thank you12:22
* jroll finds if backports needed and rebases those12:22
*** aparnav has quit IRC12:22
openstackgerritMerged openstack/ironic: Add portgroup configuration fields  https://review.openstack.org/39841712:22
dtantsurI think backports are needed, because we use stable branch devstack + master tempest, so they better not diverge12:22
dtantsurjroll, ^^^12:22
jrolldtantsur: yeah, agree12:23
dtantsuranybody has any other requests for me? I have a few more hours online :)12:23
*** ohamada has quit IRC12:24
*** ohamada has joined #openstack-ironic12:24
jrolldtantsur: I'll have backports shortly for you to approve :)12:25
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Catch unknown exceptions in validate driver ifaces  https://review.openstack.org/40615712:25
vdrokdtantsur: my request is, on the first of january, go into the store and ask if the bread is fresh :D12:25
dtantsurLOL true12:26
openstackgerritVasyl Saienko proposed openstack/ironic: Remove grenade config workaround  https://review.openstack.org/41178112:26
lucasagomesvdrok, he will probably say "it's from last year"12:28
vdroklucasagomes: yeah, here this joke is so old, that it's even funny sometimes :)12:28
lucasagomeshah right12:29
vdrokkind of a tradition even12:29
openstackgerritJim Rollenhagen proposed openstack/ironic: Move CONF.service_available.ironic to our plugin  https://review.openstack.org/40609512:30
openstackgerritJim Rollenhagen proposed openstack/ironic: Turn off tempest's multitenant network tests  https://review.openstack.org/40609412:30
jrolldtantsur: ^ there we go, I guess probably wait for master to merge before approving the second12:30
jrolland then those merge and we can finally merge the tempest patch12:31
* jroll wipes his brow12:31
dtantsurheh12:32
jrolldtantsur: I think I'll have time to POC the driver comp API stuff next week12:33
jrolland can finish it up when we're back in january12:33
dtantsurawesome12:33
* jroll is also behind on reviews a bit12:34
*** tuanluong has quit IRC12:35
dtantsurjroll, mind participating in some bikeshedding? see comments on https://review.openstack.org/#/c/400678/12:35
patchbotpatch 400678 - ironic - Introduce generic hardware types12:35
*** lucasagomes is now known as lucas-hungry12:35
jrolldtantsur: only if I can kill the bikeshedding :)12:36
jrolldtantsur: I also like 'direct'12:37
dtantsurk, will change today12:37
dtantsurhopefully we can merge this before I leave12:37
jrollcool, I'll review after that then :)12:37
jrollhope so too12:37
dtantsurs/today/right now/ actually12:37
jrollis today your last day?12:37
jrollheh12:37
dtantsurmy bus is at 6pm, now it's nearly 2pm :)12:38
jrolllooks like JayF was good with it other than the http thing12:38
jrolldtantsur: posted a couple other quick comments, only one needs action (and is easy!)12:41
*** links has quit IRC12:42
*** ohamada has quit IRC12:43
*** ohamada_ has joined #openstack-ironic12:43
nicodemosgood morning, TGIF!12:44
dtantsurfinally booked the PTG ticket, yay!12:44
*** dprince has joined #openstack-ironic12:46
jrollwoo12:47
*** bcornec2 has joined #openstack-ironic12:47
openstackgerritMerged openstack/ironic: Make CONF.debug also reflect on IPA  https://review.openstack.org/41016812:53
*** gcb has quit IRC12:54
vsaienk0folks please review: https://review.openstack.org/#/c/362130 - add portgroup support to OSC plugin, it already had +2 from dtantsur and now has +2 from rloo, might be we can merge it soon?12:54
patchbotpatch 362130 - python-ironicclient - Add portgroup support to osc plugin12:54
openstackgerritDmitry Tantsur proposed openstack/ironic: Introduce generic hardware types  https://review.openstack.org/40067812:54
dtantsurjroll, ^^^12:54
jrolldtantsur: cool, one moment12:56
*** priteau has quit IRC12:56
vsaienk0dtantsur: we may have different agents running on ramdisk (ipa, ansible, something else) might be include agent_type|name to the name of deploy interface? i.e. direct_ipa, direct_ansible wdyt?12:56
dtantsurvsaienk0, well, the same method should work the same no matter which agent12:57
dtantsurso it's "direct" or "ansible". they're not even close right now12:58
openstackgerritDmitry Tantsur proposed openstack/ironic: [WIP] Create a table to track loaded interfaces  https://review.openstack.org/40981212:58
*** zhs_ has quit IRC12:58
dtantsurjroll, this is something I won't finish probably ^^^13:01
dtantsurso if anybody wants to overtake - feel free to13:01
jrolldtantsur: okay, and I'll need that to validate setting the interface via API, yes?13:01
dtantsurjroll, no, this is needed for conductors to check on start up13:02
dtantsurjroll, see http://specs.openstack.org/openstack/ironic-specs/specs/approved/driver-composition-reform.html#data-model-impact13:02
jrolldtantsur: ah, right13:02
*** vikrant has quit IRC13:02
* jroll wonders if/how we validate an interface is loaded when it's used in api13:03
dtantsurjroll, this is already merged13:03
jrollit isn't in the spec13:03
jrollaha, cool13:03
jrollthat's done in the conductor yeah?13:03
dtantsurjroll, https://github.com/openstack/ironic/blob/master/ironic/common/driver_factory.py#L170-L24013:03
dtantsuryep13:03
jrollk13:04
jrolldtantsur: thanks13:04
* jroll feeds animals and then reviews 40067813:04
*** sacharya has joined #openstack-ironic13:07
*** baoli has joined #openstack-ironic13:09
TheJuliagood morning13:10
vsaienk0dtantsur: direct is to abstract as for me. When we have two deploy interfaces direct and agent theirs names are based on completely different characteristics. Why no use agent instead of direct?13:10
*** sacharya has quit IRC13:11
dtantsurvsaienk0, because it's wrong13:12
dtantsurboth iscsi and direct use the same agent13:12
dtantsurthe name "agent" came from times when we had different ramdisks for pxe_ipmitool and agent_ipmitool13:12
dtantsurby "direct" I'm trying to convey that the image is downloaded straight (directly) to the machine, not via conductor and iscsi13:13
vsaienk0dtantsur: so with ansible agent the images also are downloaded dirrectly to the machine, but we can't have two interfaces with the same name13:15
dtantsurI think "ansible" is a proper name, no? :)13:15
vsaienk0ansible is proper but it uses direct download method too :)13:16
dtantsurI'm open to suggestions that are not wrong (e.g. "agent") and do not expose implementation details (e.g. "direct_ipa")13:16
*** jcoufal has joined #openstack-ironic13:20
TheJuliaI'm not sure that is entirely possible, we could name it anything realistically, and an operator would know need to learn the method utilized via documentation, which means the details are exposed.... So why not expose in the first place and make it easy for the operator, as opposed to concealing it?13:20
vsaienk0dtantsur: only administrator will see deploy driver, not end user. So calling deploy interface direct_ipa is safe, We can't use direct_agent as we may have different agents on ramdisk. I prefer to be specific here. With the same, I'm ok with any kind of name, and don't think the name should be blocker for this feature.13:26
dtantsur"only administrator will see deploy driver, not end user" for some definition of "end user". also why do you expect administrators to know what IPA means?13:27
TheJulias/administrator/knowledgeable ironic users/ ?13:27
*** fragatina has joined #openstack-ironic13:29
vsaienk0dtantsur: you are right administrator is not always cloud deployer13:31
*** amotoki has joined #openstack-ironic13:32
vsaienk0but if we call deploy interface ansible we expose implementation details anyway13:34
dtantsurwell, it's a usable implementation detail - they can extend things with ansible13:35
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Create port with port group option  https://review.openstack.org/41180613:35
openstackgerritVasyl Saienko proposed openstack/python-ironicclient: Add interface attach/detach support  https://review.openstack.org/36442013:36
vdrokdtantsur: but the operator should know the image he has built and uploaded to be used as deploy image?13:38
dtantsurvdrok, if operator built the image - yes13:39
openstackgerritDerek Higgins proposed openstack/ironic: Wrap iscsi portal in []'s if IPv6  https://review.openstack.org/41180913:39
openstackgerritMerged openstack/ironic: DevStack: Make $IRONIC_IMAGE_NAME less dependent of the name in DevStack  https://review.openstack.org/41075313:40
openstackgerritMerged openstack/ironic: Move CONF.service_available.ironic to our plugin  https://review.openstack.org/40443613:41
openstackgerritDerek Higgins proposed openstack/ironic-python-agent: Advertise the correct address when using IPv6  https://review.openstack.org/41181713:42
openstackgerritDerek Higgins proposed openstack/ironic-python-agent: Agent: Listen for connections on both IPv4 and IPv6 ports  https://review.openstack.org/41181813:42
openstackgerritDerek Higgins proposed openstack/ironic-python-agent: lio: Listen for connections on both IPv4 and IPv6 ports  https://review.openstack.org/41181913:42
vsaienk0jroll: patch to ironic has been just merged, but the jobs wasn't triggered at tempest: https://review.openstack.org/#/c/355586/ they should do +A one more time13:43
patchbotpatch 355586 - tempest - Removing baremetal tests from tempest tree13:43
jrollvsaienk0: still need the newton patches: https://review.openstack.org/#/c/406095/13:44
patchbotpatch 406095 - ironic (stable/newton) - Move CONF.service_available.ironic to our plugin13:44
vdrokdtantsur: I feel like it should be <generic method>_<specific agent or tool used to deploy>. if an administrator does perform upgrade of ironic, (s)he sometimes may need to upgrade ipa too, so I'd expect them to know what it is.13:44
jrolldtantsur: ^^ if you could take a look please :)13:44
*** chihhsin has quit IRC13:44
vsaienk0jroll: yeah, just realized it, so we need a backport https://review.openstack.org/#/c/407502/ to newton right?13:45
patchbotpatch 407502 - openstack-dev/grenade - Use same tempest.conf for base and target (MERGED)13:45
*** chihhsin has joined #openstack-ironic13:45
jrollvsaienk0: no, because newton->master upgrade testing uses master grenade13:46
dtantsurjroll, +A13:46
jrolldtantsur: and the other half of the chain :) https://review.openstack.org/#/c/406094/13:47
patchbotpatch 406094 - ironic (stable/newton) - Turn off tempest's multitenant network tests13:47
*** amoralej is now known as amoralej|lunch13:47
jrollthank you!13:49
*** lucas-hungry is now known as lucasagomes13:49
*** lindycoder has joined #openstack-ironic13:50
*** fxpester has quit IRC13:54
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Create port with specific port group UUID  https://review.openstack.org/41182613:57
jrolldtantsur: +2 on generic hw types thing13:59
jrollnice work13:59
*** [1]cdearborn has joined #openstack-ironic13:59
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Create port with specific port group UUID  https://review.openstack.org/41182613:59
*** ppiela has joined #openstack-ironic14:00
*** chlong has quit IRC14:00
*** ralonsoh_ has joined #openstack-ironic14:00
dtantsurcool!14:01
dtantsurjroll, do you think folks are really needed on PTG Mon-Tue?14:01
jrolldtantsur: only if they're interested in cross-project things14:02
jrollI recommend everyone get involved in those things, if nothing else just to meet people and learn14:02
jrollbut can't force anyone14:02
dtantsurI am, but I'm also interested in not skipping two weekends in a row.. sigh14:02
jrollyeah, long flights are long :(14:02
jrolldtantsur: I also plan for people to leave friday night, fwiw14:02
dtantsurthat's what we plan on14:03
jrollokay14:03
*** ralonsoh__ has joined #openstack-ironic14:03
*** ralonsoh has quit IRC14:03
*** sborkows has quit IRC14:04
*** ralonsoh has joined #openstack-ironic14:04
*** baoli has quit IRC14:04
dtantsurjroll, btw, wdyt about Delta Airlines?14:05
jrolldtantsur: that's what I use every time, they've always been good to me14:05
jrollI hate them slightly less than other airlines14:06
dtantsurcool14:06
jrollthey tend to use air france for europe which isn't bad14:06
jrolland ATL is their main hub14:06
*** ralonsoh_ has quit IRC14:06
*** ralonsoh__ has quit IRC14:06
*** Goneri has joined #openstack-ironic14:07
*** bcornec2 has quit IRC14:07
*** amotoki has quit IRC14:08
*** rbudden has joined #openstack-ironic14:11
openstackgerritVasyl Saienko proposed openstack/ironic: Add Virtual Network Interface REST APIs  https://review.openstack.org/40424014:12
openstackgerritVasyl Saienko proposed openstack/ironic: Add Virtual Network Interface RPC APIs  https://review.openstack.org/40423914:12
openstackgerritVasyl Saienko proposed openstack/ironic: Add Virtual Network Interface Driver APIs  https://review.openstack.org/32704614:12
*** sborkows has joined #openstack-ironic14:14
*** baoli has joined #openstack-ironic14:15
*** wajdi has joined #openstack-ironic14:15
*** wajdi_ has joined #openstack-ironic14:16
*** wajdi has quit IRC14:19
dtantsurjroll, ok, I'm done booking now14:20
*** e0ne has quit IRC14:20
sborkows Can somebody who is familiar with IPA hardware managers look at https://review.openstack.org/#/c/391836/ ?14:20
patchbotpatch 391836 - ironic-python-agent - Add a new Hardware Manager for CNA network card14:20
*** sborkows has quit IRC14:20
*** e0ne has joined #openstack-ironic14:21
jrolldtantsur: \o/14:21
*** trown is now known as trown|brb14:22
*** zackf has quit IRC14:23
*** amotoki has joined #openstack-ironic14:25
*** zackf has joined #openstack-ironic14:28
*** e0ne has quit IRC14:30
*** amotoki has quit IRC14:32
*** amoralej|lunch is now known as amoralej14:36
*** trown|brb is now known as trown14:37
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Create port with specific port group UUID in OSC  https://review.openstack.org/41184014:38
*** dansmith is now known as superdan14:38
*** e0ne has joined #openstack-ironic14:38
*** tuan_luong has joined #openstack-ironic14:40
bfourniemorning Ironic!  Based on milan|afk's suggestion, I’m looking at using the python construct library in https://review.openstack.org/#/c/406496/.  I don’t think this library is included by any other openstack components. Is there anything that needs to be done so that it can be used, beyond adding it to ironic-inspector/requirements.txt?14:41
patchbotpatch 406496 - ironic-inspector - Add plugin to process basic management LLDP TLVs14:41
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Create port with specific port group UUID in OSC  https://review.openstack.org/41184014:44
jrollbfournie: yes, it needs to be added here: https://github.com/openstack/requirements/blob/master/global-requirements.txt14:44
jrollsee the readme there for how to do that and the questions that need to be answered14:44
*** baoli has quit IRC14:46
bfourniejroll: thank you!14:46
*** godp1301 has joined #openstack-ironic14:46
jrollno problem :)14:49
*** nicodemos has quit IRC14:56
*** nicodemos has joined #openstack-ironic14:57
*** milan|afk has quit IRC14:59
*** tuan_luong has quit IRC14:59
*** milan has joined #openstack-ironic15:00
*** causten has joined #openstack-ironic15:03
*** bcornec2 has joined #openstack-ironic15:05
openstackgerritVasyl Saienko proposed openstack/ironic: Add Virtual Network Interface REST APIs  https://review.openstack.org/40424015:05
*** trown is now known as trown|afk15:08
*** sacharya has joined #openstack-ironic15:08
*** sborkows has joined #openstack-ironic15:08
*** sacharya has quit IRC15:13
*** jaosorior has quit IRC15:15
*** derekh has quit IRC15:15
NobodyCamGood Morning Ironic'er and ...15:15
NobodyCamOFC15:16
NobodyCamTGIF15:16
lucasagomesNobodyCam, morning15:16
NobodyCamMorning lucasagomes :)15:17
TheJuliagood morning NobodyCam15:17
milanmorning NobodyCam TheJulia! :) TGIF! :)15:18
NobodyCamGood Morning TheJulia & milan15:18
jroll\o NobodyCam, TheJulia :)15:18
NobodyCam:)15:18
NobodyCamhey hey jroll :)15:18
openstackgerritMerged openstack/ironic-inspector: Fix test when running with SQLite 3.7.17 from CentOS 7  https://review.openstack.org/41174615:18
dtantsurmorning NobodyCam15:19
milansambetts, hey you here? :)15:19
jrollhe's out for the year, milan15:19
dtantsurfolks, I'm leaving for my vacation :) see you in January, I guess15:19
dtantsurreach out via hangouts, if you need something15:19
milanjroll, I thought so, just his nick is different today ;)15:20
jrollmilan: probably re-connect15:20
jrolldtantsur: have fun!15:20
milandtantsur, have a nice trip! :)15:20
dtantsurthanks :)15:20
NobodyCammorning dtantsur :)15:21
*** chlong has joined #openstack-ironic15:21
milanbtw me dtantsur joining Atlanta PTG15:21
milan* me &15:22
*** mtanino has joined #openstack-ironic15:22
*** rama_y has joined #openstack-ironic15:23
*** sborkows has quit IRC15:23
*** dtantsur is now known as dtantsur|afk15:24
milanaarefiev, so our patches are sort of blocked till new year? ;)15:25
TheJuliaAwesome15:25
milanTheJulia going too? :)15:26
TheJuliaI am, I'm currently planning on driving and being in the area all week.15:26
milan\o/ beers :)15:27
openstackgerritSofiia Andriichenko proposed openstack/ironic: This adds a tempest tests for creating a node:  https://review.openstack.org/41119715:27
milanwe just booked hotel&flight, sort of last minute15:27
aarefievmilan: yeah, we don't have quorum :)15:29
milanjroll, wdyt^ :)15:29
aarefievHave a good vacation dtantsur|afk15:29
*** mjura has quit IRC15:29
NobodyCamhave a great tie dtantsur|afk15:30
NobodyCamtime even :p15:30
milanlol15:30
milan:)15:30
aarefievand morning NobodyCam :)15:30
NobodyCammorning aarefiev :)15:30
aarefievmilan: btw did you investigate why marker doesn't work in status list?15:31
jrollmilan: what do I think about what? :P15:31
* milan looks at the patches: https://review.openstack.org/#/c/375045/ https://review.openstack.org/#/c/408116/ which should be OK 15:32
patchbotpatch 375045 - ironic-inspector-specs - Endpoint to list introspection statuses15:32
patchbotpatch 408116 - python-ironic-inspector-client - List introspection statuses support15:32
milanjroll, we're out of cores to approve patches till new year :D15:32
jrollmilan: lol15:32
jrollif there's anything urgent I could also approve15:33
milanaarefiev, not yet, spent half a day looking into flights to atlanta15:33
milanjroll, nothing urgent15:34
jrollmilan: ok cool, guess it's time for you to take a vacation too :P15:34
milanjroll, yeah I just wish I had enough PTO to take :D15:34
jrollmilan: just make sure you're on irc, nobody will know :)15:35
milanheh true :D15:35
* jroll doesn't actually advocate not working when told to15:35
aarefievyeah milan bot :)15:35
milanlol have to implement one :D face roll bot15:36
*** wajdi_ has quit IRC15:36
JayFdtantsur|afk: jroll: I'm OK with direct, but ramdisk might would have been more informative15:39
* jroll doesn't think either are very informative without some docs15:39
JayFmariojv: ^ you should re-review the generic hardware types patch15:40
JayFuh oh15:40
JayFjroll: isn't generic hardware type supposed to be in setup.cfg on that patch, too?15:40
jrolluh, maybe15:41
jrollwhich patch is it again?15:41
JayFhttps://review.openstack.org/#/c/400678/13/setup.cfg15:41
patchbotpatch 400678 - ironic - Introduce generic hardware types15:41
JayFthe class gets added15:41
JayFbut not the entrypoint15:41
jrollyeah, seems like it should15:41
jrollI can add it if dtantsur|afk is gone already15:41
JayFI mean, I could too15:42
jroll¯\_(ツ)_/¯15:42
JayFjust sad b/c we could've landed it15:42
JayFlol15:42
*** MattMan_1 has joined #openstack-ironic15:42
jrollI mean, we could15:42
jrollthough I guess idk the intent15:42
JayFthere are no test cases for the generic hardware manager15:42
JayFer, hardware type15:42
jrollactually, yeah, it probably isn't meant to be exposed15:43
openstackgerritMerged openstack/ironic: Turn off tempest's multitenant network tests  https://review.openstack.org/40609415:43
openstackgerritMerged openstack/ironic: Move CONF.service_available.ironic to our plugin  https://review.openstack.org/40609515:43
JayFbut yet the manual management has a test case15:43
JayFand an entrypoint15:43
jrollcheck the commit message, last paragraph15:43
jrollthe generichardware class also doesn't have any supported power/management interfaces15:43
jrollso it isn't usable on its own15:43
jrollthis seems correct15:43
JayFso generichardware type15:44
JayFis just like a base class?15:44
mariojvlooking again15:44
jrollseems so15:44
jrolloh yeah15:44
JayFI guess I don't understand, looking at this15:44
jrollclass GenericHardware(hardware_type.AbstractHardwareType):15:44
jroll    """Abstract base class representing generic hardware.15:44
patchbotError: No closing quotation15:44
jroll"abstract"15:44
JayFthe GenericHardware appears to matc exactly whhat I'd want to run15:44
jrolldoes it?15:44
JayFit inherits from that15:44
jrollwhat power interface do you want? :)15:45
*** MattMan_1 has quit IRC15:45
JayFoooh15:45
jrollno, abstract in the docstring15:45
JayFooooooooh15:45
JayFokay, that makes a lot more sense15:45
JayF+215:45
JayFI didn't approve it yet, going to let mariojv take a look and wondering if more cores should have a look15:45
JayFif not I can land it wen mario is done15:46
mariojvso the logic behind "direct" vs. agent is that there are multiple types of agents, so we want to just specify the image download mechanism?15:46
mariojvJayF: ^ ?15:46
mariojvfor the deploy interface naming15:46
JayFI think the logic is more15:46
JayFagent describes the implementation detail15:46
TheJuliaI <3 it when 1370+ tests failed15:46
jrolland agents are used for everything15:46
JayFwhereas direct speaks more to method15:46
JayFah, agents used for all is a really good point15:46
mariojvok - better than "http"15:47
mariojvJayF: +115:47
mariojvwrt asking for more core reviews - i expect a lot of people are gone for the holidays?15:48
TheJuliaI'm not, but I won't be able to get to it until later today15:49
TheJuliawoot, now only 189 tests failing15:55
mariojvat this rate you'll have -1370 tests failing :D15:56
*** Syed__ has joined #openstack-ironic15:56
openstackgerritGalyna Zholtkevych proposed openstack/python-ironicclient: [WIP] POC: Full-featured objects with ETAGs supporting  https://review.openstack.org/40033515:56
* jroll steps away for a while15:56
*** Guest31304 has quit IRC16:02
*** Romanenko_K has quit IRC16:04
*** MattMan has left #openstack-ironic16:05
*** Nisha_Agarwal has joined #openstack-ironic16:06
mariojvi'm out on PTO for the rest of the day. have a great weekend ironic o/16:06
*** rama_y has quit IRC16:07
*** baoli has joined #openstack-ironic16:10
vdrokmariojv: you too! :)16:13
openstackgerritVladyslav Drok proposed openstack/ironic: Add test to ensure policy is always authorized  https://review.openstack.org/35017716:16
openstackgerritVladyslav Drok proposed openstack/ironic: Fix policy dict checkers  https://review.openstack.org/41176416:16
vdrokif anyone has a bit of time ^^ spent couple of hours trying to make that work16:16
vdrokespecially the second one in chain :)16:17
openstackgerritGalyna Zholtkevych proposed openstack/ironic: ETAGs maintenance for all api node's requests  https://review.openstack.org/39221316:19
*** MattMan has joined #openstack-ironic16:21
*** vinsh has quit IRC16:22
*** Linkmark has quit IRC16:23
*** rcernin has quit IRC16:23
*** pcaruana has quit IRC16:25
*** MattMan has left #openstack-ironic16:27
*** MattMan has joined #openstack-ironic16:28
*** chlong has quit IRC16:29
*** causten has quit IRC16:36
*** causten has joined #openstack-ironic16:36
*** soliosg has joined #openstack-ironic16:38
*** causten has quit IRC16:38
JayFvsaienk0: agents are used for both iscsi and direct methods16:40
JayFvsaienk0: which is why agent is not a great choice for that either16:40
*** baoli has quit IRC16:40
openstackgerritVladyslav Drok proposed openstack/ironic: Add test to ensure policy is always authorized  https://review.openstack.org/35017716:40
vsaienk0JayF: I don't think that agent is good choice because we may use other agents than ipa16:41
JayFyeah16:41
JayFI think direct is "good enough"16:41
vsaienk0JayF: but different agents may use direct method16:41
JayFfor the ansible case, we'd just call it ansible16:41
JayFlol16:41
vsaienk0I doubt why no call it ipa, ansible or direct_ipa, direct_ansible16:42
openstackgerritAndrey Shestakov proposed openstack/bifrost: Fixes to allow install Ironic on remote node  https://review.openstack.org/40921316:42
JayFthere's no requirement for the agent driver, on the ironic side, to use IPA16:43
JayFit could be any other agent implementing the same api16:43
vsaienk0JayF, but the IPA is term related to community implementation16:43
JayFyeah16:43
JayFI mean, I really disliked http16:43
JayFI'm OK with direct16:43
JayFwould probably be ok with direct_ipa, but not simply ipa16:44
JayFbut I'd also rather just get this code in rather than everyone argue about naming over the holidays :P16:44
*** baoli has joined #openstack-ironic16:44
*** baoli has quit IRC16:45
*** bdeschenes has quit IRC16:46
milanlucasagomes, w/r https://review.openstack.org/#/c/408552 I guess the behaviour changed a bit, right?16:46
patchbotpatch 408552 - ironic-inspector - Use the device hints matching mechanism from ironi...16:46
* milan wonders whether we need a release note there16:46
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Migrate to session for glance client  https://review.openstack.org/41076716:47
lucasagomesmilan, it hasn't changed, well, the code in ironic-lib is backward compat (that's why the tests won't fail) but now inspector would also be able to handle operators in the root device hints16:48
lucasagomesso that's a change yeah16:48
milanlucasagomes, exactly my point16:48
milanwdyt?16:49
lucasagomesmilan, about a release note ? I can add one yeah16:49
milanyeah16:49
milanlucasagomes, thx16:49
lucasagomeswill do that in a few16:49
milancool16:49
* milan votes -1 meanwhile :P16:49
*** baoli has joined #openstack-ironic16:50
*** causten has joined #openstack-ironic16:50
*** rama_y has joined #openstack-ironic16:52
*** chlong has joined #openstack-ironic16:53
*** ralonsoh has quit IRC16:53
openstackgerritLucas Alvares Gomes proposed openstack/ironic-inspector: Use the device hints matching mechanism from ironic-lib  https://review.openstack.org/40855216:56
lucasagomesmilan, ^ lemme know if it's good enough16:56
milanlucasagomes, thx16:56
* milan looks16:56
lucasagomesmilan, re: new dependency, apparently we don't add 'em to the release notes (couldn't find any example) so I left it out16:56
milanOK cool16:56
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Keystone session for swift-temp-url generation  https://review.openstack.org/41076416:58
milanlucasagomes, +217:02
lucasagomeso/17:02
*** rcernin has joined #openstack-ironic17:04
*** sacharya has joined #openstack-ironic17:09
jlvillaljroll, vsaienk0 : Not sure if you saw: https://review.openstack.org/#/c/397775/  I don't know what changed to cause it to stop working.17:15
patchbotpatch 397775 - ironic - DNM: Testing 35558617:15
*** bcornec2 has quit IRC17:16
*** causten has quit IRC17:18
*** athomas has quit IRC17:20
lucasagomesjroll, you think it would be fine to pin on an image like https://review.openstack.org/#/c/375095/ ? This is for the UEFI work17:25
patchbotpatch 375095 - openstack-dev/devstack - Ironic: Set default images for UEFI deployment17:25
*** causten has joined #openstack-ironic17:32
*** milan has quit IRC17:44
openstackgerritVladyslav Drok proposed openstack/ironic: Add test to ensure policy is always authorized  https://review.openstack.org/35017717:47
jlvillalJayF: jroll: Any reason not to +A https://review.openstack.org/#/c/400678/  ?17:49
patchbotpatch 400678 - ironic - Introduce generic hardware types17:49
*** vinsh has joined #openstack-ironic17:50
*** [1]cdearborn has quit IRC17:50
*** Nisha_Agarwal has quit IRC17:50
*** bdeschenes has joined #openstack-ironic17:51
TheJuliajlvillal: I was going to try and review it this afternoon17:51
jlvillalTheJulia: Then I won't +A it :)17:52
TheJuliathank you17:53
*** bdeschenes has quit IRC17:55
vdrokmorning TheJulia NobodyCam and jlvillal !17:56
jlvillalHi vdrok :)17:56
NobodyCamGood Morning vdrok :)17:57
xavierrmorning ironic17:57
NobodyCammorning xavierr :)17:57
vdrokxavierr: morning :)17:57
xavierrhey TheJulia jlvillal vdrok NobodyCam o/17:57
xavierrvdrok: have a minute?17:57
jlvillalHi xavierr :)17:57
vdrokxavierr: yeah, kind of17:57
NobodyCammorning jlvillal17:57
*** bdeschenes has joined #openstack-ironic17:58
jlvillalNobodyCam: :)17:58
xavierrvdrok: https://review.openstack.org/#/c/408298/1..3/ironic/drivers/modules/agent_base_vendor.py17:58
patchbotpatch 408298 - ironic - Shutdown server before change boot order for agent...17:58
vdrokxavierr: yup, what's up with that? is it still needed?17:58
xavierrvdrok: after remove the 'if iwdi' thing 3 jobs are breaking in the patch17:58
vdroklooking17:59
xavierrvdrok: maybe we really need that17:59
xavierrtwo ipa-wholedisk-agent_ipmitool-tinyipa breaking and multitenant also18:00
*** dprince has quit IRC18:02
*** zhugaoxiao has quit IRC18:02
*** zhugaoxiao has joined #openstack-ironic18:03
vdrokxavierr: hmm, all the jobs that are failing are agent_* driver with whole disk image18:03
*** bdeschenes has quit IRC18:03
vdroklemme look at the code a bit more18:04
xavierryeah18:04
xavierrok :)18:04
*** ohamada_ has quit IRC18:07
vdrokhmm, I see what happens but I'm not sure if it's an OK behaviour18:14
xavierrbefore remove the 'if' thing it was working18:15
vdrokjroll: have a bit of time? https://review.openstack.org/#/c/408298/1/ironic/drivers/modules/agent_base_vendor.py , my comment on L59418:15
patchbotpatch 408298 - ironic - Shutdown server before change boot order for agent...18:15
vdrokit appears that boot.prepare_instance relies on get_boot_option for node18:16
* xavierr is thinking about use deploy_utils' try_set_boot_device instead18:16
vdrokand default boot option is netboot18:16
*** srart has joined #openstack-ironic18:16
vdrokso we have some sort of logic duplication here and there18:18
vdrokin reboot_and_finish_deploy, we rely only on iwdi18:18
vdrokwhile in boot.prepare_instance we rely on boot option18:19
vdrokso with the logic here https://review.openstack.org/#/c/408298/1/ironic/drivers/modules/agent_base_vendor.py first boot interface will set boot device pxe and prepare the configs, then reboot_and_finish_deploy will switch it back to disk18:20
patchbotpatch 408298 - ironic - Shutdown server before change boot order for agent...18:20
vdrokwhich seems strange18:20
xavierrany suggestion?18:22
xavierr:)18:22
vdrokxavierr: so as you can see from above, first the boot interface will set PXE boot and prepare pxe env, then agent deploy will set it back to disk, with different logic. I'm not sure what's the best fix on that. leaving that iwdi check would work but... dunno18:22
lucasagomesfolks, calling it a day18:22
lucasagomeshave a great evening all!18:22
lucasagomesenjoy the weekend18:22
vdrokgood night lucasagomes18:22
vdrokenjoy too :)18:23
xavierrbye lucasagomes :)18:23
*** aNuposic has joined #openstack-ironic18:23
lucasagomeso/18:23
*** lucasagomes is now known as lucas-afk18:23
jlvillallucas-afk: Ciao18:23
xavierrvdrok: I'll keep the iwdi until we get a new code review on that, agree? Just to Jenkins +1 it... :)18:24
nicodemosbye lucas-afk18:26
vdrokxavierr: maybe it's ok for now. tho I'd prefer more eyes on that, as that switching back and forth does not feel right to me18:26
vdroki mean that switching boot devices18:26
xavierrvdrok: understood :)18:26
xavierrty vdrok18:26
vdroknp :)18:26
jrolljlvillal: :rageface:18:27
jlvillaljroll: Yeah...  Understandable.18:27
jlvillalI'm not sure what happened recently18:27
jrollit seems like CONF.service_available.ironic isn't being registered18:29
jrollthis happened before, trying to recall how I untangled that18:29
jlvillaljroll: But I see it in the tempest file18:29
jrollright, but the conf option isn't being registered, for it to be loaded from config18:29
jrollhttp://logs.openstack.org/75/397775/6/check/gate-grenade-dsvm-ironic-ubuntu-xenial/d74a8fe/logs/old/tempest.txt.gz#_2016-12-16_15_28_44_63318:30
jlvillalhttp://logs.openstack.org/75/397775/6/check/gate-grenade-dsvm-ironic-ubuntu-xenial/d74a8fe/logs/old/tempest_conf.txt.gz18:30
jrollyes18:30
jlvillalUh, strange18:30
jrollyou can put anything you want there, if the option doesn't exist it won't be loaded18:30
*** trown|afk is now known as trown18:31
jrollI wonder if something went wonky with the depends-on18:32
openstackgerritMerged openstack/ironic-inspector: Add drac_address to ipmi_address_fields  https://review.openstack.org/41135218:32
jrolljlvillal: looks like you rechecked it before the newton change merged, but after approval: https://review.openstack.org/#/c/406095/18:34
patchbotpatch 406095 - ironic (stable/newton) - Move CONF.service_available.ironic to our plugin (MERGED)18:34
jrolland all the references I see on the old side of grenade look like it isn't pulling in the ironic patch18:34
jrollbah, no, I lied http://logs.openstack.org/75/397775/6/check/gate-grenade-dsvm-ironic-ubuntu-xenial/d74a8fe/logs/grenade.sh.txt.gz#_2016-12-16_15_17_43_69118:35
jlvillaljroll: Oh, I thought it had merged when I did recheck18:35
jrolloh, the tempest plugin isn't loaded18:38
jrollwat18:38
jlvillalI'm looking through the tempest code to see if any changes landed recently that look suspicious18:38
jrollno, it's just, we aren't loading the tempest plugin http://logs.openstack.org/75/397775/6/check/gate-grenade-dsvm-ironic-ubuntu-xenial/d74a8fe/logs/grenade.sh.txt.gz#_2016-12-16_15_28_45_81618:38
*** jralph has joined #openstack-ironic18:38
jrollwhich I fixed once in project-config18:39
jlvillaljroll: I notice some project config changes yesterday to ironic.yaml18:39
jrollyes, I am reading them18:39
jrollclark's changes lgtm18:40
* jroll is suspicious of https://review.openstack.org/#/c/409750/18:40
patchbotpatch 409750 - openstack-infra/project-config - Add ironic tempest plugin on early stage (MERGED)18:40
jrolllast time grenade passed was before that18:41
patchbotthat18:41
jrollsigh18:41
* jroll flips tables, makes coffee, deals with this shortly18:41
*** bdeschenes has joined #openstack-ironic18:41
*** dprince has joined #openstack-ironic18:41
jlvillalheh18:41
jlvillalaarefiev: stay out of sight ;)18:41
openstackgerritHugo Nicodemos proposed openstack/ironic: Reusing oneview_client when possible  https://review.openstack.org/41194718:41
jrollsafe to assume two builders with ironic-vars-early will collide?18:42
jrollmaybe not18:42
jlvillaljroll: Should not, should append18:43
jrolloh, grenade doesn't use ironic-vars-early18:43
jrollwhy's grenade so special here, sigh18:43
jlvillaljroll: ding ding ding!18:44
jlvillaljroll: Well I have a thing to make it less special in there. I just haven't moved it over yet18:44
jlvillalI did create an ironic-grenade builder18:44
jrollright, but it isn't used :P18:44
jlvillaljroll: Yeah, not yet.18:45
jlvillal:(18:45
jrolljlvillal: https://review.openstack.org/41194918:47
patchbotpatch 411949 - openstack-infra/project-config - Use ironic-vars-early in grenade job18:47
jrollok now I coffee18:47
*** fragatina has quit IRC18:47
jlvillaljroll: I could try to make the builder less special if you like.18:47
openstackgerritXavier proposed openstack/ironic: Shutdown server before change boot order for agent drivers  https://review.openstack.org/40829818:49
jrolljlvillal: that would be helpful, but isn't urgent :)18:51
jrollaarefiev: btw, since you got mentioned, this isn't your fault :)18:51
jlvillaljroll: Looking at it now. Fairly simple. But taking awhile for me to make sure it looks good18:51
jrollit's strictly a technology/process problem, not a people problem (like all problems in software)18:51
jrolljlvillal: cool, thank you18:51
*** baoli has quit IRC18:52
jrolloff topic: if you listen to the rogue one soundtrack on spotify, it changes the seek bar https://puu.sh/sRI7s/1be9dc67b1.png18:52
xavierrjroll: wow18:53
jlvillaljroll: https://review.openstack.org/#/c/411956/18:58
patchbotpatch 411956 - openstack-infra/project-config - ironic: Have the grenade builder use devstack-virt...18:58
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-inspector: Update documentation to deploy Ironic Inspector with DevStack  https://review.openstack.org/41045619:00
JayFjlvillal: you sure that's going to resolve down to the exact same variables?19:00
*** clif_h has quit IRC19:01
*** ChubYann has joined #openstack-ironic19:01
jlvillalJayF: I believe so. From looking at it. But would love others to look at it.19:01
JayFjlvillal: DEVSTACK_GATE_GRENADE=pullup is getting dropped, afaict19:02
jlvillalJayF: Argh!19:02
JayFjlvillal: I haven't checked them all, but I know that's one of htem19:02
JayFjlvillal: I just wonder if with holidays coming up if maybe just adding ironic-early-vars to that would be preferable to the bigger change19:03
JayFjlvillal: to avoid breaking stuff when wedon't know how many ifnra folks will be around to land fixes19:03
jlvillalJayF: Fixed.19:03
jlvillalJayF: Thanks. Good catch.19:03
*** clif has joined #openstack-ironic19:04
*** clif is now known as clif_h19:04
jlvillalJayF: I reviewed that and saw pull-up and was thinking, yep I put the ironic-grenade builder in there so that gets populated. But I didn't actually put the ironic-grenade builder in there :(19:04
jlvillalThe problem with reviewing your own code19:05
jlvillalJayF: And no problem for me to wait until after the New Year.19:05
jrollyeah, I agree, let's wait19:07
jlvillalThough to play devil's advocate, this is the best time of the year for a gate failure :)19:07
JayFimpact of failure is lower19:07
JayFbut MTTF is going to be higher19:07
jlvillalTrue19:07
jlvillaljroll: JayF: I rebased it on top of jroll's patch. And did workflow -119:10
openstackgerritJulia Kreger proposed openstack/ironic: Add storage_interface to base driver class  https://review.openstack.org/34800619:11
jrolljlvillal: thanks!19:12
*** baoli has joined #openstack-ironic19:15
*** wajdi has joined #openstack-ironic19:16
*** e0ne has quit IRC19:17
openstackgerritMerged openstack/ironic-python-agent: Use oslo-config-generator for sample config  https://review.openstack.org/39898919:42
*** amoralej is now known as amoralej|off19:48
TheJuliafound an issue with driver comp :(19:52
vdrokgood night everyone!19:53
*** aNuposic has quit IRC19:55
*** fragatina has joined #openstack-ironic19:55
jlvillalTheJulia: Good thing I waited!19:55
jlvillalvdrok: Good night19:55
*** clenimar has quit IRC19:55
TheJuliajlvillal: I +A'ed that one, problem is the underlying changes to testing19:56
TheJuliaI can't define storage_interface, without object conversion blowing out  now19:56
jlvillalTheJulia: Oh.19:56
TheJuliaonly way to settle that is tell the base driver "Hey, there is this new interface, doesn't really exist yet"19:56
TheJuliaExcept, all of the merged tests now enumerate on that list expecting them to be fully functional interfaces19:56
TheJuliatl;dr, can only add interfaces presently in oneshot merges19:57
TheJuliawhich... blocks rolling upgrades19:57
* TheJulia thinks object conversion shouldn't force the interface definition19:58
openstackgerritMerged openstack/ironic: Introduce generic hardware types  https://review.openstack.org/40067819:58
* TheJulia plugs laptop into power for a little bit and short break20:00
*** rama_y has quit IRC20:07
*** rama_y has joined #openstack-ironic20:07
*** e0ne has joined #openstack-ironic20:09
*** jtomasek has quit IRC20:33
*** chlong has quit IRC20:48
*** jcoufal has quit IRC20:48
*** rama_y has quit IRC20:50
*** fragatina has quit IRC20:53
*** jcoufal has joined #openstack-ironic21:01
*** jcoufal has quit IRC21:01
*** dsneddon has joined #openstack-ironic21:07
*** dprince has quit IRC21:08
TheJuliaThe object has to be with the interface, the db field can be separate21:14
JayFI'm about to leave, I'll be on PTO all next week, and won't be back until 12/27.21:16
JayFAn email to my personal email will get read eventually if anyone needs me, if it's someting urgent my co-workers have my number21:16
JayFhave a nice holiday everyone o/21:16
jroll\o cheers JayF21:17
* jroll should leave too, have a good weekend y'all21:17
JayFo/21:17
openstackgerritJay Faulkner proposed openstack/ironic: Add API and conductor methods for [un]rescue  https://review.openstack.org/35083121:18
*** e0ne has quit IRC21:18
TheJuliahave a good vacation JayF21:22
openstackgerritJulia Kreger proposed openstack/ironic: Add storage_interface DB field  https://review.openstack.org/34800521:25
*** bdeschenes has quit IRC21:31
*** soliosg has quit IRC21:40
*** rama_y has joined #openstack-ironic21:46
*** aNuposic has joined #openstack-ironic21:53
*** vinsh has quit IRC21:55
*** trown is now known as trown|outtypewww22:05
*** godp1301 has quit IRC22:06
*** lindycoder has quit IRC22:07
*** wajdi has quit IRC22:07
*** causten has quit IRC22:16
openstackgerritMerged openstack/ironic: Remove iBoot, WoL and AMT drivers  https://review.openstack.org/39784722:16
openstackgerritJulia Kreger proposed openstack/ironic: Wire in storage interface attach/detach operations  https://review.openstack.org/40629022:18
openstackgerritJulia Kreger proposed openstack/ironic: Add Cinder storage driver  https://review.openstack.org/36619722:18
openstackgerritJulia Kreger proposed openstack/ironic: Add base cinder common interface  https://review.openstack.org/35562522:18
openstackgerritJulia Kreger proposed openstack/ironic: Add storage_interface to base driver class  https://review.openstack.org/34800622:18
*** jkilpatr has quit IRC22:27
TheJuliaHave a good weekend everyone!22:28
*** gabriel-bezerra has quit IRC22:28
*** mrtenio has quit IRC22:28
*** ricardoas has quit IRC22:28
*** charlledaniel has quit IRC22:28
*** xavierr has quit IRC22:28
*** nicodemos has quit IRC22:28
*** mtanin___ has joined #openstack-ironic22:40
*** mtanino has quit IRC22:42
*** jkilpatr has joined #openstack-ironic22:50
*** fragatin_ has joined #openstack-ironic22:51
*** fragatin_ has quit IRC22:53
*** fragatina has joined #openstack-ironic22:56
*** rbudden has quit IRC22:59
*** fragatina has quit IRC23:00
*** godp1301 has joined #openstack-ironic23:04
*** rbudden has joined #openstack-ironic23:14
openstackgerritChris Krelle proposed openstack/ironic: [WIP / POC] Add Intel Node Manager sensor data collection  https://review.openstack.org/41046523:17
NobodyCamhave a great weekend TheJulia23:18
*** godp1301 has quit IRC23:19
*** rbudden has quit IRC23:27
*** baoli has quit IRC23:34
*** awiddersheim has quit IRC23:54
*** aNuposic has quit IRC23:56

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