Wednesday, 2016-11-02

*** shobhar_ has quit IRC00:17
*** jerrygb has joined #openstack-ironic00:17
*** jerrygb_ has joined #openstack-ironic00:20
*** jerrygb has quit IRC00:23
*** mtanino has quit IRC00:35
*** awiddersheim has quit IRC00:45
*** maratoid_ has quit IRC00:45
*** baoli has joined #openstack-ironic00:47
*** baoli has quit IRC00:48
*** baoli has joined #openstack-ironic00:50
*** baoli_ has joined #openstack-ironic00:51
*** jerrygb_ has quit IRC00:52
rlooJayF: wrt 392114, don't think so. looks like it is fixing a bug. w/o that change, things will fail, right? i added a comment to the patch00:52
*** rloo has quit IRC00:53
*** baoli has quit IRC00:55
*** jkilpatr has quit IRC00:55
*** baoli_ has quit IRC00:56
*** tuanluong has joined #openstack-ironic00:59
*** hoangcx has joined #openstack-ironic01:01
*** hw_wutianwei has joined #openstack-ironic01:07
*** aignatov has quit IRC01:08
*** aignatov has joined #openstack-ironic01:11
*** [1]cdearborn has quit IRC01:17
*** awiddersheim has joined #openstack-ironic01:27
*** baoli has joined #openstack-ironic01:33
*** sabeen has joined #openstack-ironic01:40
*** mtanino has joined #openstack-ironic01:42
*** zhangyufei has joined #openstack-ironic01:54
*** baoli has quit IRC01:59
*** baoli has joined #openstack-ironic01:59
*** mtanino has quit IRC02:01
*** baoli has quit IRC02:04
*** syed_ has quit IRC02:05
*** jerrygb has joined #openstack-ironic02:05
*** jvgrant has quit IRC02:14
*** jvgrant has joined #openstack-ironic02:14
*** zhangyufei has quit IRC02:15
*** zhangyufei has joined #openstack-ironic02:29
openstackgerritNaohiro Tamura proposed openstack/ironic: Ipmitool power driver for soft reboot and soft power off  https://review.openstack.org/21673802:53
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/38989603:03
openstackgerritChangBo Guo(gcb) proposed openstack/python-ironicclient: Use function import_versioned_module from oslo.utils  https://review.openstack.org/39239503:23
*** mtanino has joined #openstack-ironic03:33
*** vikrant has joined #openstack-ironic03:35
*** vinsh has quit IRC03:36
*** sabeen has quit IRC03:36
*** lifeless has quit IRC03:41
*** lifeless has joined #openstack-ironic03:42
*** phuongnh has joined #openstack-ironic03:44
*** mcarden has joined #openstack-ironic03:48
*** absubram has joined #openstack-ironic04:04
*** absubram_ has joined #openstack-ironic04:05
*** absubram has quit IRC04:08
*** absubram_ is now known as absubram04:08
openstackgerritTuan Luong-Anh proposed openstack/ironic: Fix setting persistent boot device does not work  https://review.openstack.org/37632204:10
openstackgerritTuan Luong-Anh proposed openstack/ironic: Fix setting persistent boot device does not work  https://review.openstack.org/37632204:11
openstackgerritChangBo Guo(gcb) proposed openstack/python-ironicclient: Use function import_versioned_module from oslo.utils  https://review.openstack.org/39239504:11
*** yuikotakadamori has quit IRC04:13
openstackgerritTuan Luong-Anh proposed openstack/ironic: Fix setting persistent boot device does not work  https://review.openstack.org/37632204:19
*** jerrygb has quit IRC04:19
*** jerrygb has joined #openstack-ironic04:24
*** yuanying has quit IRC04:24
*** chlong has quit IRC04:24
*** shobhar_ has joined #openstack-ironic04:27
*** shobhar_ has quit IRC04:28
*** Nisha_away has joined #openstack-ironic04:32
*** yuanying has joined #openstack-ironic04:36
*** absubram has quit IRC04:37
*** mtanino has quit IRC04:48
*** hoangcx has quit IRC04:54
*** jerrygb has quit IRC05:03
*** jerrygb has joined #openstack-ironic05:06
openstackgerritChangBo Guo(gcb) proposed openstack/python-ironicclient: Use function import_versioned_module from oslo.utils  https://review.openstack.org/39239505:36
*** vinsh has joined #openstack-ironic05:37
*** vinsh has quit IRC05:41
*** vmud213 has joined #openstack-ironic05:47
*** maratoid_ has joined #openstack-ironic05:47
*** jerrygb has quit IRC05:53
*** jerrygb has joined #openstack-ironic05:57
*** hshiina has joined #openstack-ironic06:04
*** Nisha_away has quit IRC06:12
*** ChubYann has quit IRC06:18
*** jerrygb has quit IRC06:22
*** jerrygb has joined #openstack-ironic06:27
*** Nisha_away has joined #openstack-ironic06:30
*** Nisha_brb has joined #openstack-ironic06:31
*** Nisha_away has quit IRC06:35
*** moshele has joined #openstack-ironic06:40
*** zhangyufei has quit IRC06:42
*** zhangyufei has joined #openstack-ironic06:43
*** yuanying has quit IRC06:46
*** moshele has quit IRC06:46
openstackgerritTuan Luong-Anh proposed openstack/python-ironicclient: Trivial fix: Unified formats of log_xxx function  https://review.openstack.org/37634306:49
*** yuanying has joined #openstack-ironic06:49
*** rcernin has joined #openstack-ironic06:50
*** mjura has joined #openstack-ironic07:04
*** jerrygb has quit IRC07:06
*** jerrygb has joined #openstack-ironic07:10
*** mjura has quit IRC07:11
*** ohamada has joined #openstack-ironic07:11
*** mjura has joined #openstack-ironic07:11
*** rbartal has joined #openstack-ironic07:13
*** tesseract has joined #openstack-ironic07:17
*** tesseract is now known as Guest4871207:17
*** Nisha_brb has quit IRC07:28
openstackgerrityolanda.robla proposed openstack/bifrost: Enable upper requirements on bifrost  https://review.openstack.org/39182807:29
*** fragatina has joined #openstack-ironic07:30
*** fragatina has quit IRC07:39
*** fragatina has joined #openstack-ironic07:39
*** jerrygb has quit IRC07:42
*** jerrygb has joined #openstack-ironic07:46
*** openstackgerrit has quit IRC07:48
*** openstackgerrit has joined #openstack-ironic07:48
*** Nisha_brb has joined #openstack-ironic07:50
*** jerrygb_ has joined #openstack-ironic07:50
*** ccamacho has joined #openstack-ironic07:51
*** snozhka has quit IRC07:51
*** jerrygb has quit IRC07:52
*** moshele has joined #openstack-ironic07:53
*** snozhka has joined #openstack-ironic07:55
*** jerrygb has joined #openstack-ironic07:56
*** jerrygb_ has quit IRC07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:01
*** jerrygb has quit IRC08:08
*** pcaruana has joined #openstack-ironic08:09
openstackgerritMerged openstack/ironic-inspector: Switch func3 environment to Python 3.5  https://review.openstack.org/39181808:12
*** jerrygb has joined #openstack-ironic08:13
*** Nisha_brb has quit IRC08:14
*** maratoid_ has quit IRC08:22
*** Nisha_away has joined #openstack-ironic08:26
*** jerrygb_ has joined #openstack-ironic08:27
*** athomas has joined #openstack-ironic08:28
*** xek__ is now known as xek08:28
*** jerrygb has quit IRC08:29
xekmorning :)08:31
*** mgoddard has joined #openstack-ironic08:37
*** skazi has quit IRC08:42
openstackgerritVasyl Saienko proposed openstack/python-ironicclient: OSC add capability to remove node/chassis_uuid  https://review.openstack.org/39181208:42
*** skazi has joined #openstack-ironic08:42
*** jaosorior has joined #openstack-ironic08:45
*** jpich has joined #openstack-ironic08:47
*** lucas-afk is now known as lucasagomes08:51
lucasagomesmorning all08:51
Nisha_awaylucasagomes, MORNING09:06
openstackgerritZhou Ya proposed openstack/ironic-python-agent: Get numa_node info when collecting pci devices info  https://review.openstack.org/36924509:06
*** jerrygb_ has quit IRC09:08
*** jerrygb has joined #openstack-ironic09:12
*** jaosorior has quit IRC09:14
*** jaosorior has joined #openstack-ironic09:14
*** Nisha_away has quit IRC09:15
*** Nisha_away has joined #openstack-ironic09:16
*** dougs1 has joined #openstack-ironic09:17
milanmorning lucasagomes, Nisha_away, xek! :)09:19
*** dtantsur|afk is now known as dtantsur09:21
dtantsurMorning Ironic!09:21
milanmorning dtantsur! :)09:23
* milan relocates09:25
*** derekh has joined #openstack-ironic09:26
*** milan has quit IRC09:26
openstackgerritVasyl Saienko proposed openstack/python-ironicclient: OSC add capability to set chassis-uuid for node  https://review.openstack.org/39249509:32
*** galyna has joined #openstack-ironic09:34
*** hshiina has quit IRC09:35
*** vinsh has joined #openstack-ironic09:38
*** vinsh has quit IRC09:43
openstackgerritLucas Alvares Gomes proposed openstack/ironic: API: lookup() ignore malformed MAC addresses  https://review.openstack.org/39211409:43
*** kirankv has quit IRC09:44
*** strigazi_AFK is now known as strigazi09:44
Nisha_awaymorning dtantsur09:45
dtantsurhey Nisha_away, mikal09:45
*** Nisha_away is now known as Nisha109:45
dtantsurooops, that was for milan :)09:45
vsaienk0morning dtantsur, aarefiev, lucasagomes, sambetts, milan, mgould and all Ironic'ers!09:47
dtantsurhey vsaienk009:47
*** jaosorior has quit IRC09:48
*** ijw has joined #openstack-ironic09:49
*** jaosorior has joined #openstack-ironic09:49
*** ijw has quit IRC09:49
*** ijw has joined #openstack-ironic09:50
*** kirankv has joined #openstack-ironic09:52
*** jerrygb has quit IRC09:54
*** dprince has joined #openstack-ironic09:56
*** jerrygb has joined #openstack-ironic09:58
lucasagomesvsaienk0, hi there!10:01
*** snecklifter has joined #openstack-ironic10:03
*** vmud213 has quit IRC10:06
*** vmud213 has joined #openstack-ironic10:07
*** jaosorior is now known as jaosorior_lunch10:08
*** phuongnh has quit IRC10:08
openstackgerritGalyna Zholtkevych proposed openstack/ironic: ETAG supporting to enhance API evolution  https://review.openstack.org/39221310:08
zhangyufeiSome one can help review this patch about deploy image doc? https://review.openstack.org/#/c/387082/10:09
patchbotpatch 387082 - ironic - Add the way to get the deploy ram disks10:09
*** milan has joined #openstack-ironic10:10
openstackgerritAparna proposed openstack/ironic-specs: Generic way to configure clean step priorites in Automated cleaning  https://review.openstack.org/36261510:12
milanmorning Ironic! :)10:12
*** zhangyufei has quit IRC10:14
*** Nisha1 has quit IRC10:14
*** tuanluong has quit IRC10:16
*** e0ne has joined #openstack-ironic10:16
*** aparnav has joined #openstack-ironic10:17
*** kirankv has quit IRC10:17
*** ishant|away has quit IRC10:17
*** lazy_prince has quit IRC10:17
*** sambetts|afk is now known as sambetts10:18
*** e0ne_ has joined #openstack-ironic10:18
sambettsMorning all10:18
milanmorning sambetts! :)10:18
dtantsurmorning sambetts, milan!10:19
sambettshey dtantsur, milan10:19
*** e0ne has quit IRC10:21
vsaienk0morning sambetts!10:21
milanmorning vsaienk0! :)10:21
*** harbor_log is now known as potdirect10:22
lucasagomessambetts, hey there10:23
*** ishant|away has joined #openstack-ironic10:24
*** kirankv has joined #openstack-ironic10:24
*** lazy_prince has joined #openstack-ironic10:25
sambettso/ lucasagomes10:25
*** hw_wutianwei has quit IRC10:28
*** portdirect has joined #openstack-ironic10:34
*** mgould|afk is now known as mgould10:36
mgouldmorning Ironic!10:36
mgouldmorning milan sambetts vsaienk0 lucasagomes10:36
lucasagomeshi there10:36
*** potdirect has quit IRC10:37
dtantsurmorning mgould10:37
milanmorning mgould! :)10:38
openstackgerritVasyl Saienko proposed openstack/ironic: Switch to Neutron Flat network.  https://review.openstack.org/34069510:40
openstackgerritVasyl Saienko proposed openstack/ironic: Allow to use FLAT network setup  https://review.openstack.org/33438210:40
openstackgerritVasyl Saienko proposed openstack/ironic: Add IRONIC_TEST_NETWORK_NAME devstack plugin var  https://review.openstack.org/33811710:40
*** portdirect has quit IRC10:48
*** [1]cdearborn has joined #openstack-ironic10:49
openstackgerritMerged openstack/ironic: Log currently known iSCSI devices when we retry waiting for iSCSI target  https://review.openstack.org/38373010:50
*** e0ne has joined #openstack-ironic10:54
*** e0ne_ has quit IRC10:54
sambettsvsaienk0: Hey! Did you see my comment on the flat network patch?10:58
sambettsvsaienk0: I'm having quite a bit of success getting the simulation to be more like a real neutron deployment10:59
vsaienk0sambetts: I'm sorry. I've missed that10:59
sambettsvsaienk0: I have flat networks working with my new way, I'm hoping to try and get vlan networks working today11:00
sambettsvsaienk0: it code I have now really simplifys what we have to do in devstack to setup the environment which is soo nice11:01
sambettsthe code*11:01
openstackgerritMerged openstack/ironic: DRAC get_bios_config() passthru causes exception  https://review.openstack.org/39157111:02
sambettsI like the change you made to change the USE_FLAT variable to a string11:03
*** g3ek has quit IRC11:04
openstackgerritDmitry Tantsur proposed openstack/ironic: DRAC get_bios_config() passthru causes exception  https://review.openstack.org/39254811:07
*** g3ek has joined #openstack-ironic11:07
openstackgerritDmitry Tantsur proposed openstack/ironic: Remove "dhcp" command from the iPXE template  https://review.openstack.org/39255211:09
openstackgerritDmitry Tantsur proposed openstack/ironic: Add a missing error check in ipmitool driver's reboot  https://review.openstack.org/39255311:10
* dtantsur is proactively backporting stuff ^^^11:10
*** portdirect has joined #openstack-ironic11:11
dtantsuraarefiev, ugh, the discovery job seems broken: http://logs.openstack.org/31/389331/2/check/gate-tempest-dsvm-ironic-inspector-discovery/f52e5c2/11:12
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent: Fix config drive writing script  https://review.openstack.org/39255711:16
dtantsurvdrok, mind backporting https://review.openstack.org/#/c/387664/ to stable/newton please? looks like an important thing, I can't cleanly backport it due to merge conflicts11:19
patchbotpatch 387664 - ironic-lib - Wait for a config drive device to be available (MERGED)11:19
* dtantsur is done with proactvie backports11:20
*** jaosorior_lunch is now known as jaosorior11:21
*** hshiina has joined #openstack-ironic11:23
* dtantsur -> reboot for updates brb11:25
*** daemontool has joined #openstack-ironic11:25
*** bfournie has quit IRC11:26
*** bfournie has joined #openstack-ironic11:26
*** jkilpatr has joined #openstack-ironic11:27
vdrokdtantsur: is it OK if I'll do it tomorrow? On pto today. And morning everyone :)11:28
lucasagomesvdrok, totally, if you want I can backport it for ya in a sec11:30
lucasagomesvdrok, morning11:30
vdroklucasagomes: thanks!11:33
*** Administrator__ has quit IRC11:34
*** Administrator__ has joined #openstack-ironic11:35
mgouldmorning vdrok11:36
mgouldalso, what are you doing here? Go and enjoy your day off :-)11:36
*** zhugaoxiao has joined #openstack-ironic11:37
dtantsurvdrok, morning! tomorrow or any other day. enjoy your pto!11:38
*** baoli has joined #openstack-ironic11:38
vdrokMorning mgould , that's all because of IRC client on my phone :) morning dtantsur11:38
*** Administrator__ has quit IRC11:40
mat128morning everyone11:41
mgouldmorning mat12811:42
*** baoli_ has joined #openstack-ironic11:42
openstackgerritLucas Alvares Gomes proposed openstack/ironic-lib: Wait for a config drive device to be available  https://review.openstack.org/39257311:43
lucasagomesvdrok, ^ (don't look at it today!)11:43
lucasagomesthere was a merge conflict btw11:43
*** baoli has quit IRC11:45
openstackgerritMerged openstack/ironic-ui: Eliminate use of a now obsolete admin basePath variable  https://review.openstack.org/38870711:45
*** mtanino has joined #openstack-ironic11:46
*** mtanino has quit IRC11:52
jrollmorning y'all11:58
sambettso/ jroll11:58
*** jcoufal has joined #openstack-ironic11:59
lucasagomesjroll, morning12:02
jrolldtantsur: all those backports are on newton yeah? I can roll through reviewing them12:02
dtantsurjroll, yep, all newton. I think mitaka is critical-only right now. also good morning :)12:02
jrollcool12:03
jrollmorning :)12:03
*** krtaylor has joined #openstack-ironic12:03
*** moshele has quit IRC12:03
lucasagomesjroll, yup, I want to backport the infiniband/invalid mac fix to newton as well once it's merged upstream12:05
* lucasagomes goes get some food12:05
*** lucasagomes is now known as lucas-hungry12:05
openstackgerritAline Bousquet proposed openstack/ironic: Put nodes in CLEANWAIT during a cleaning reboot  https://review.openstack.org/39258412:07
jrolldtantsur: fyi, added to the whiteboard, but this should cover all stable reviews http://bit.ly/2fE0QTj12:07
dtantsurcool!12:07
*** aparnav has quit IRC12:13
*** moshele has joined #openstack-ironic12:14
*** rbrud has joined #openstack-ironic12:15
*** hshiina has quit IRC12:16
jrolldtantsur: +A'd some, one is failing unit tests, lucas' backport needs your +212:18
dtantsurdamn tests, let's make them non-voting12:19
jrollheh12:19
dtantsur:-P12:19
milanmorning jroll! :)12:19
milanmorning mat128, vdrok! :)12:19
jrollhey milan12:20
mgouldmorning jroll12:21
jroll\o mgould12:22
*** cdearborn has joined #openstack-ironic12:28
*** vikrant has quit IRC12:30
*** ohamada has quit IRC12:32
*** ohamada has joined #openstack-ironic12:33
*** baoli_ has quit IRC12:34
*** baoli has joined #openstack-ironic12:36
openstackgerritDmitry Tantsur proposed openstack/ironic: Remove "dhcp" command from the iPXE template  https://review.openstack.org/39255212:39
*** jerrygb has quit IRC12:42
*** e0ne has quit IRC12:42
*** yolanda has quit IRC12:44
*** yolanda has joined #openstack-ironic12:44
*** [1]cdearborn has quit IRC12:44
openstackgerritMerged openstack/ironic-python-agent: Fix config drive writing script  https://review.openstack.org/39255712:44
*** jerrygb has joined #openstack-ironic12:45
*** jheroux has joined #openstack-ironic12:47
*** hshiina has joined #openstack-ironic12:51
*** dougs1 has quit IRC12:55
*** dougs1 has joined #openstack-ironic12:56
*** ccamacho is now known as ccamacho|lunch12:59
*** vmud213 has quit IRC13:02
*** snecklifter has quit IRC13:03
*** snecklifter has joined #openstack-ironic13:03
*** Goneri has joined #openstack-ironic13:07
*** rloo has joined #openstack-ironic13:08
*** mtanino has joined #openstack-ironic13:11
openstackgerritMerged openstack/ironic: DRAC get_bios_config() passthru causes exception  https://review.openstack.org/39254813:12
*** [1]cdearborn has joined #openstack-ironic13:16
openstackgerritMerged openstack/ironic: Add a missing error check in ipmitool driver's reboot  https://review.openstack.org/39255313:16
*** ralonsoh has joined #openstack-ironic13:20
*** lucas-hungry is now known as lucasagomes13:22
*** jkilpatr has quit IRC13:27
*** hshiina has quit IRC13:28
*** ijw has quit IRC13:28
*** absubram has joined #openstack-ironic13:29
*** cdearborn has quit IRC13:30
pas-hado we have somewhere a matrix/table clearly describing what gate job tests what combination of deploy options?13:33
dtantsurnot really..13:33
pas-haI'd probably start an etherpad for this13:33
rloopas-ha: ++ it should be a wiki or a doc somewhere, but etherpad is a good start13:35
rloohi and good morning ironic'ers :)13:35
*** baoli_ has joined #openstack-ironic13:36
*** baoli has quit IRC13:36
*** baoli has joined #openstack-ironic13:37
dtantsurmorning rloo13:38
*** baoli__ has joined #openstack-ironic13:38
dtantsurI think etherpad is better, provided that we're going to change it soon13:38
*** jkilpatr has joined #openstack-ironic13:39
lucasagomesrloo, morning13:40
*** vinsh has joined #openstack-ironic13:40
rloodtantsur: yes, that's why it is a good start. when (if?) it 'stabilizes', we should move it13:40
dtantsur++13:40
rloohi lucasagomes, dtantsur!13:41
*** baoli_ has quit IRC13:41
*** baoli has quit IRC13:42
*** baoli__ has quit IRC13:43
*** baoli has joined #openstack-ironic13:44
*** vinsh has quit IRC13:44
*** jkilpatr has quit IRC13:45
*** ccamacho|lunch is now known as ccamacho13:48
*** jaosorior has quit IRC13:53
*** mjturek has joined #openstack-ironic13:53
*** jkilpatr has joined #openstack-ironic13:57
*** baoli has quit IRC13:58
mjturekhamzy: krtaylor: I'm thinking this could be the new table structure for MI so we can support other drivers http://imgur.com/zV9oMlY. thoughts?13:58
mjturekdict represents fields in the table13:59
rloomorning JayF! Just saw your comment on 392114. wrt Aline's patch 390558. good question. if it needs a microversion bump, then the 'bug' should be an RFE and it needs to be approved, no?13:59
patchbothttps://review.openstack.org/#/c/390558/ - ironic - Add possibility to remove chassis_uuid from a node13:59
*** baoli has joined #openstack-ironic13:59
*** ppiela has joined #openstack-ironic13:59
JayFrloo: I guess so; tbh I didn't think alineb's patch should require a microversion on initial review13:59
JayFrloo: if it's possib13:59
JayF*if it's possible to create a node w/o chassis uuid, it should be possible to remove chassis uuid from a node14:00
rlooJayF: i'm still coming out of the fog of not having heads down in ironic. not sure yet where i fall wrt needing microversion or not.14:00
rlooJayF: what I'm wondering is why we made chassis uuid a mandatory field (this is based on the bug description, i didn't look at the code yet.)14:01
JayFI'm not super familiar with our microversion rules, I should go read the spec, but the rule of thumb I use is "if I had an API client, would this change likely break me?"14:01
JayFand in the case of both of those patches, I think the answer is clearly no14:01
dtantsurthis is not how people see microversions (even though I do agree with your definition)14:01
rlooJayF: it isn't just that. IF we add a new 'feature', that also requires a microversion bump.14:01
rlooJayF: new features don't necessarily break existing setups14:02
dtantsurmicroversions are designed like they're designed for 1. visibility of changes, 2. avoiding breaking changes, 3. to force people to use microversions14:02
dtantsurpoint #2 is clearly not applicable to both cases, #1 and #3 are questionable14:03
openstackgerritLucas Alvares Gomes proposed openstack/ironic: API: lookup() ignore malformed MAC addresses  https://review.openstack.org/39211414:03
dtantsurfor me the border is if the change actually fixes something broken or rather adds something missing14:03
milanmorning JayF, rloo! :)14:04
rloohi milan14:04
JayFso playing the devils advocate for ^ that patch, wouldn't adding support for non-ethernet macs count as a feature?14:04
rlooJayF: it doesn't add support for non-ethernet macs.14:04
dtantsurJayF, for what definition of support?14:04
dtantsurrloo++ we just stop crashing if they happen to be on the same machine14:05
rloodtantsur: exactly.14:05
lucasagomesrloo, JayF I think the reason why chassis_uuid couldn't be removed is just bad design14:05
JayFWell but I look back to what caused this error -- when we added support in IPA For fibre channel macs14:05
dtantsurlike, if we failed deploy for machines with AMD CPU (as opposed to Intel ones), I'd also call it a bug14:05
dtantsurJayF, that's not correct. it's caused by lookup API introduction IIRC14:05
JayFI don't actually think either change needs a microversion, I'm just trying to make the point that I think it's still pretty subjective :/14:05
lucasagomesthat said, I also don't know if we should have a microversion on alines' patch. It sounds like somehting we would like to get fixed across multiple versions (cause it's a bug)14:06
rlooJayF: i agree, the microversion thing wasn't that clear. did the api group issue some microversion spec that clarifies it?14:06
dtantsurJayF, I've made a strict input check on MAC format, hence it started failing if a NIC is present with a different hw address format. it has not relation to the ongoing Infiniband effort14:06
* lucasagomes don't like microversion to be a "bug compat" thing14:06
dtantsuralso note that if we start version bug fixes, they'll become not backportable14:07
rloolucasagomes: when you say the chassis uuid thing was due to a bad design, what do you mean by that?14:07
JayFdtantsur: interesting, I thought it was a change we added to IPA to send back extra macs, but looking at the git log it's clear you're right14:07
lucasagomesrloo, I mean that, when we designed the API we didn't understand the relation between the resources very well14:07
lucasagomesrloo, the API in general was a great PoC in my view, we didn't know ReST, wsme, etc... at the time14:08
rloolucasagomes: so there isn't anything inherent in our underlying code that makes a chassis uuid mandatory for a node, we just coded it that we wanted it mandatory?14:08
dtantsurrloo, JayF, another approach to this: you'll benefit from the MAC fix without any changes to your code; you won't benefit from chassis_uuid fix without actually starting deleting chassis_uuid. hence the discoverability aspec of microversioning kicks in here.14:08
lucasagomesrloo, right. I don't think there's anything in the code that depends on chassis uuid being mandatory for nodes14:09
* dtantsur is not sure whether his comment clarifies things or makes them even more compex14:09
JayFdtantsur: at least it tells me exactly where we're splitting the hair, lol14:09
lucasagomesI think that one idea of microversion is so that, the same version across different clouds (different vendors) will work exactly the same way. So, in the chassis_uuid thing (or even the MAC) that won't be true if we do not microversion it14:10
rlooJayF: so when I look at the code (via Aline's patch) I see that it is coded explicitly that chassi_uuid is not removable14:10
rlooJayF: which, in conjunction to what lucasagomes said, makes me think we/someone made a conscious decision for chassis_uuid to be mandatory14:11
rlooJayF: which makes me think it requires a microversion.14:11
JayFYeah, with what dtantsur said I can see where the distinction is now14:11
rlooJayF: cuz it was a 'feature', not a bug.14:11
lucasagomesbut I don't really agree with that goal, being compatible is cool, but being "bug compatible" is dummy14:11
JayFI think I'm with lucasagomes in that I'd prefer the distinction be less strict, but I'm OK with whatever as long as I understand it :)14:11
* rloo worries that we'll get back into the discussion of why we added microversions and i don't want to upset/remind dtantsur of those discussions...14:12
dtantsurheh :)14:12
* dtantsur has no clue why, but he got used to them14:13
lucasagomesdtantsur, human nature :-)14:13
*** causten has joined #openstack-ironic14:13
*** baoli has quit IRC14:15
pas-hastarted etherpad describing tested combinations per gate job https://etherpad.openstack.org/p/ironic-gate-jobs-described (WiP)14:15
*** baoli has joined #openstack-ironic14:15
*** baoli_ has joined #openstack-ironic14:16
rloothx pas-ha. would you mind sending out email to the devlist about that etherpad?14:16
pas-hasure14:16
*** e0ne has joined #openstack-ironic14:19
*** baoli has quit IRC14:20
JayFrloo: so can we swap over the bug for aline's patch to an rfe and approve it? Did you wanna dig deeper into it first?14:22
pas-harloo: done14:23
rlooJayF: give me about 5 more minutes; just looking at existing code now.14:23
rloopas-ha: thx!14:23
JayFperfect, going to go finish making breakfast then :D14:23
*** daemontool has quit IRC14:31
*** daemontool has joined #openstack-ironic14:32
*** baoli_ has quit IRC14:32
*** anush has quit IRC14:36
openstackgerritJim Rollenhagen proposed openstack/ironic: Clarify ironic governance requirements and process  https://review.openstack.org/39268514:36
jroll^^ happy fun time14:37
jrollI need all cores on that at some point, also sent an email14:37
*** vinsh has joined #openstack-ironic14:39
vsaienk0dtantsur: are you planning to work on https://review.openstack.org/#/c/357983?14:41
patchbotpatch 357983 - ironic - Support names for {cleaning,provisioning}_network (ABANDONED)14:41
dtantsurvsaienk0, maybe later. feel free to overtake, if you want to14:41
*** jaosorior has joined #openstack-ironic14:41
rloodtantsur, lucasagomes, JayF: wrt ability to delete node.chassis_uuid, are you all for it? I am updating the bug/RFE, just need to put down who is for it so I can approve it.14:41
JayF+214:42
dtantsur+214:42
lucasagomesrloo, I'm good with being able to delete node.chassis_uuid. +114:42
*** dmellado is now known as dmellado|lunch14:42
*** zackf has joined #openstack-ironic14:42
jroll+214:43
rloothx lucasagomes. oh, i didn't even see the other +2s. thx all!14:43
jroll:D14:43
rlooprocess-related question, jroll etc. if the RFE touches python-ironicclient, we add that project to the bug too, right? vs creating a new bug?14:44
jrollrloo: yeah, that's good14:44
*** wajdi has joined #openstack-ironic14:46
openstackgerritFélix Bouliane proposed openstack/ironic: DONOTMERGE-change IRONIC_DEPLOY_DRIVER to pxe_snmp  https://review.openstack.org/38816814:46
*** mgoddard_ has joined #openstack-ironic14:47
*** wajdi_ has joined #openstack-ironic14:48
*** mgoddard has quit IRC14:50
*** wajdi has quit IRC14:51
*** mtanino has quit IRC14:52
*** mgoddard_ has quit IRC14:53
*** jjohnson2 has quit IRC14:53
*** mtanino has joined #openstack-ironic14:53
*** mgoddard_ has joined #openstack-ironic14:53
*** dougs1 has quit IRC14:53
*** jjohnson2 has joined #openstack-ironic14:54
*** sai has quit IRC14:55
*** dougs1 has joined #openstack-ironic14:55
*** syed_ has joined #openstack-ironic14:55
vsaienk0rloo, dtantsur, sambetts, jroll: https://review.openstack.org/#/c/390558/ is in conflict with Portgroup API patch, can we block it until we land portgroup API patch? it is pretty big and already has two +214:56
patchbotpatch 390558 - ironic - Add possibility to remove chassis_uuid from a node14:56
openstackgerritMerged openstack/ironic-lib: Wait for a config drive device to be available  https://review.openstack.org/39257314:56
jrollvsaienk0: ++14:57
*** yufei has joined #openstack-ironic14:57
jrollvsaienk0: -1'd with a note14:58
*** daemontool has quit IRC14:58
*** sai has joined #openstack-ironic14:58
vsaienk0thanks jroll!14:59
rloovsaienk0: thx for reminder. portgroups is on my high priority list. i shouldn't have gotten distracted! :)14:59
*** jcoufal has quit IRC14:59
*** jcoufal has joined #openstack-ironic15:00
*** rbartal has quit IRC15:00
rloovsaienk0: do you have a link to the portgroup patches?15:00
rloovsaienk0: i see this from trello, not sure it is correct: https://review.openstack.org/#/q/status:open+branch:master+topic:bug/152640315:01
vsaienk0rloo: first patch https://review.openstack.org/#/c/347549/15:01
patchbotpatch 347549 - ironic - Add PortGroups API15:01
*** mgoddard_ has quit IRC15:02
*** mgoddard has joined #openstack-ironic15:02
rloovsaienk0: thx. that patch shows 5 other related changes, yet when i look at the topic, i see https://review.openstack.org/#/q/topic:bug/161875415:03
openstackgerritLucas Alvares Gomes proposed openstack/ironic: API: lookup() ignore malformed MAC addresses  https://review.openstack.org/39211415:04
rloovsaienk0: and do you have any idea wrt the link that is in trello? should I delete that?15:04
vsaienk0rloo: we decided to track portgroups by separate RFE 1618754. Filtering by topic:bug/1618754 should return a list of all related portgroup patches15:04
*** moshele has left #openstack-ironic15:05
vsaienk0rloo: I don't have write access to trello, if you have please update it to https://review.openstack.org/#/q/topic:bug/161875415:05
*** moshele has joined #openstack-ironic15:05
rloovsaienk0: ok, will do...15:05
hamzymjturek, are you proposing using four different tables? hrm. I am almost finished with a patch that just uses one field of user supplied data15:08
openstackgerritMerged openstack/ironic: Remove "dhcp" command from the iPXE template  https://review.openstack.org/39255215:09
JayFSo is there a reason we haven't landed portgroup API patch yet? Myself and Yuriy have +2 on it15:09
*** e0ne has quit IRC15:09
JayFmaybe some folks could take a look today and we can land that?15:09
rlooJayF: i'll look today. (or you could land it if you want)15:10
JayFHonestly, for API changes I'm always a little bit happier if more folks get a look15:10
JayFI don't like to be the deciding core vote on those :)15:10
mjturekhamzy: well, a table per driver (not all at once, just as support is needed), and a join table so you can get the right info from the right table. It's more complex, but it's a better structure15:11
mjturekhamzy: but I suppose having json blobs is fine for now15:11
hamzymjturek, What is the end goal of molteniron? Provide locking with data or complex database operations?15:12
mjturekhamzy: locking with data... but that doesn't mean we shouldn't strive for a well structured DB15:13
*** snozhka has quit IRC15:13
*** snozhka has joined #openstack-ironic15:14
hamzymjturek, I don't have a lot of experience in databases. I can understand the concept of locking one row in one table. In your picture the Nodes table is the one being locked?15:15
hamzyand all of the other tables are queryable?15:15
mgouldmjturek, hamzy: are you going to query on the contents of those JSON blobs?15:16
*** srobert has joined #openstack-ironic15:16
mgouldIME storing JSON in strings in an RDBMS is a fertile source of pain15:16
hamzymgould, my initial rewrite looks like this http://paste.openstack.org/show/587653/15:17
*** anush has joined #openstack-ironic15:17
hamzythe status field is the lock15:17
mjturekhamzy: yeah it's the same flow15:17
rloohey dtantsur: wrt https://review.openstack.org/#/c/390558/3//COMMIT_MSG, i thought we used partial-bug for all patches related to an rfe/bug? or do we just do it per project per bug? (now i am wondering if in past rfes, we've added the client as a separate project)15:18
patchbotpatch 390558 - ironic - Add possibility to remove chassis_uuid from a node15:18
mjturekmgould: I don't think we need to query the data in the JSON, we just need to get it15:18
mgouldOK, that's probably fine15:18
hamzymgould, is there problems with encoding and corruption when storing JSON in a database?15:18
mgouldthat, and it makes your query logic awful and slow15:19
hamzymgould, is there a safe datastore for JSON that is also simple?15:20
mgouldqueries that would be a nice efficient join turn into a mess of string parsing15:20
NobodyCamGood Morning Ironic'ers15:20
mgouldmorning NobodyCam15:20
NobodyCammorning mgould :)15:21
*** e0ne has joined #openstack-ironic15:21
mgouldhamzy: sadly, I don't have a good answer to that in our case15:21
hamzyunderstood15:21
mgouldrecent versions of Postgres (which we can't rely on in Ironic) support native JSON queries15:21
mgouldI don't know enough about the NoSQL space to answer your safety concerns15:22
milanmorning NobodyCam! :)15:22
hamzybecause that postgress would have to run and be supported on all the platforms, right?15:22
mgouldit probably does, but we can't rely on it being there15:22
mgouldrecent versions of MySQL also support native JSON queries (in an incompatible way, of course)15:23
NobodyCammorning milan15:23
mgouldwhen this came up a few months ago, the conclusion was that we couldn't rely on users having an up-to-date MySQL install15:23
mgouldI suspect at our scale (say a few tens of thousands of rows?) most NoSQL databases are safe enough :-)15:24
mgouldbut yeah, CouchDB or ElasticSearch are designed for searching arbitrary JSON blobs efficiently15:28
*** sandriichenko has joined #openstack-ironic15:28
*** jcoufal has quit IRC15:28
mgouldand if you know the structure in advance, it's quite possibly worth expressing it as a relational schema15:29
mgould[IME, as a developer and not an operator :-)]15:29
mariojvrloo: do you or lintan want help with any particular piece of rolling upgrades? looking at https://review.openstack.org/#/c/306357 it seems like the basic framework is underway, maybe i can help with unit tests or docs?15:29
patchbotpatch 306357 - ironic - [PoC]Ironic rolling upgrade framework - Make servi...15:29
rloomariojv: so xek is going to take the lead on rolling upgrades. he might appreciate help on unit tests or docs, dunno.15:30
mariojvalternatively maybe jlvill-travel / vdrok / vsaienk0 need help with multinode grenade for that? i don't know much about it but am willing to learn :)15:30
openstackgerritOpenStack Proposal Bot proposed openstack/bifrost: Updated from global requirements  https://review.openstack.org/38987615:30
mariojvok, cool15:30
rloomariojv: i think we would welcome more folks knowing about grenade etc :)15:30
*** moshele has quit IRC15:32
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/39273015:32
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/39273115:32
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/38989615:32
openstackgerritSzymon Borkowski proposed openstack/ironic-python-agent: Add a new Hardware Manager for X710 network card  https://review.openstack.org/39183615:34
*** jcoufal has joined #openstack-ironic15:34
*** sabeen has joined #openstack-ironic15:36
*** g3ek has quit IRC15:36
*** harlowja_at_home has joined #openstack-ironic15:38
*** mmedvede has quit IRC15:38
*** g3ek has joined #openstack-ironic15:39
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements  https://review.openstack.org/38993815:40
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/39282015:40
dtantsurrloo, sorry for the delay. closes-bug closes only one project side of the bug15:40
dtantsurrloo, so if you plan on only one patch for ironic repo, you should use closes-bug15:40
dtantsurno matter how many projects are added to the bug15:40
dtantsurotherwise the ironic part of the bug will stay opened, and I'll have to manually close it one day :)15:41
rloodtantsur: gotcha.15:42
*** rama_y has joined #openstack-ironic15:42
* lucasagomes still can't get his head around the project-config yaml's15:44
dtantsurlucasagomes, haha, I can imagine. need some help?15:45
mgouldlucasagomes: anything I can help with?15:45
mgouldsnap15:45
hamzymgould, thanks for the input!15:45
mgouldhamzy: np, sorry for the wall of text15:45
lucasagomesdtantsur, mgould hah thanks. Lemme try to get the first version of the patch up and I may shout for help if it fails15:46
lucasagomesI'm switching the jobs to use ipmitool instead of ssh btw15:46
dtantsuraha, that's fun15:46
*** yufei has quit IRC15:47
*** dmellado|lunch is now known as dmellado15:47
dtantsurlucasagomes, tl;dr: jenkins/jobs/ironic defines job templates, jenkins/jobs/projects creates actual jobs definitions from these templates, zuul/layout insert jobs into project's gate15:47
lucasagomesdtantsur, thanks that helps!15:48
lucasagomessometimes I just feel like s/pxe_ssh/pxe_ipmitool15:48
lucasagomesbut that won't work I think15:48
lucasagomesheh15:48
mgouldlucasagomes: try it and see what breaks :-)15:49
dtantsuryeah, you need duplicate jobs to be able to run different ones on stable/{liberty,mitaka} and newer15:49
dtantsurlucasagomes, make sure to move new jobs to xenial btw.. they'll request it anyway from you15:49
lucasagomesdtantsur, we should do it separately, right ?15:49
lucasagomesI've seem you changed the inspector ones in the git logs15:50
*** mmedvede has joined #openstack-ironic15:50
dtantsurlucasagomes, well... if you do it separately, you'll have to duplicate jobs for trusty and xenial first :)15:50
* dtantsur is not sure we haven't done it already though15:50
lucasagomesoh dear15:50
mgoulddtantsur: move to xenial, then move to pxe_ipmitool?15:50
dtantsurmgould++15:50
dtantsurlucasagomes, I think we first move ipmitool jobs to xenial, if they're not there already15:51
dtantsurlucasagomes, then we switch ipmitool to voting and shut down ssh jobs15:51
lucasagomesdtantsur, mgould yeah, sounds more sane15:51
* lucasagomes does it15:51
*** krtaylor has quit IRC15:54
*** sabeen has quit IRC15:56
*** snecklifter has quit IRC16:01
*** [1]cdearborn has quit IRC16:02
*** harlowja_at_home has quit IRC16:03
*** ohamada has quit IRC16:04
*** rbartal has joined #openstack-ironic16:04
*** ohamada has joined #openstack-ironic16:05
*** ohamada has quit IRC16:05
*** pcaruana has quit IRC16:06
*** sabeen has joined #openstack-ironic16:06
*** rbrud has quit IRC16:06
*** mgoddard_ has joined #openstack-ironic16:07
*** rcernin has quit IRC16:09
*** anush has quit IRC16:10
*** mgoddard has quit IRC16:10
*** sabeen has quit IRC16:10
*** sabeen has joined #openstack-ironic16:11
hwoarangTheJulia: cinerama: any idea about the recent "No module named vine.five" ci failures?16:15
*** rpioso has joined #openstack-ironic16:16
*** causten has quit IRC16:17
hwoarangplus, this error in the ci job is interesting: "2016-11-02 16:06:44.579114 | mkdir: cannot create directory ‘/opt/stack’: Permission denied"16:17
hwoaranghttp://logs.openstack.org/79/389079/2/check/gate-bifrost-integration-tinyipa/5402386/console.html16:17
*** causten has joined #openstack-ironic16:18
*** causten_ has joined #openstack-ironic16:19
*** causten has quit IRC16:22
*** baoli has joined #openstack-ironic16:22
mat128hwoarang: https://bugs.launchpad.net/ceilometer/+bug/163832316:26
openstackLaunchpad bug 1638323 in Ceilometer "gate fails reporting 'No module named vine.five'" [Undecided,In progress] - Assigned to Hanxi Liu (hanxi-liu)16:26
*** jkilpatr has quit IRC16:26
mat128hwoarang: tl;dr, pin kombu<4.016:26
hwoarangoh thanks mat12816:26
*** jaosorior has quit IRC16:28
*** mjura has quit IRC16:33
*** Guest48712 has quit IRC16:34
*** mmedvede has quit IRC16:37
*** rbrud has joined #openstack-ironic16:37
*** jkilpatr has joined #openstack-ironic16:39
*** e0ne has quit IRC16:40
*** baoli has quit IRC16:42
*** baoli has joined #openstack-ironic16:42
*** aNuposic has joined #openstack-ironic16:44
*** mmedvede has joined #openstack-ironic16:45
jrollso jlvill-travel is out and I don't see krtaylor, maybe we skip the qa meeting today?16:46
* dtantsur wants to skip too16:47
dtantsur(like if I ever attended, lol)16:47
*** e0ne has joined #openstack-ironic16:48
jrollheh16:50
jroll"dtantsur will skip per usual"16:50
dtantsurright16:51
dtantsurwell, after I committed to do some CI refactoring, I should probably start attending16:51
jrollfair enough16:51
dtantsuralso it's now 6pm, not 7pm (ditto for v2 meeting)16:51
jroll\o/16:52
JayFI am pretty sure that when DST hits in America (this weekend)16:52
JayFI'm going to keep the same working hours in UTC16:52
JayFMy wife typically gets home from teaching around 3:30pm, and that would put me getting off around 4pm, so we get a longer afternoon and I have incentive to not waste so much time in the morning :)16:53
*** kvcobb has joined #openstack-ironic16:53
dtantsurhah, great16:53
dtantsurI work random hours16:53
JayFI work a very consistent 8-5 (1500-2400 UTC), sometimes leaving early if I start early or skip taking a lunch break16:54
JayFMy favorite thing about the job right now is how consistent the hours are16:54
dtantsur8am is deep night for me16:54
JayFI used to be a stay-up-late-wake-up-late guy, but when I got married, and she kept a strict early bed/early rise schedule, I decided to not fight it :)16:55
dtantsurhah, I see. we're both night owls, soooo16:55
*** shobhar_ has joined #openstack-ironic16:56
JayFYeah; makes sense. I'm usually in bed by 9p, asleep by 10p16:56
JayFI know for most folks that's just when they're getting started :D16:56
dtantsurwell.. I try not to do any work after 9pm :)16:57
*** milan has quit IRC16:57
*** krtaylor has joined #openstack-ironic16:58
* mgould tries to be in bed by 2330, but often fails17:01
*** causten_ has quit IRC17:01
*** mgoddard has joined #openstack-ironic17:02
*** mgoddard_ has quit IRC17:02
*** causten has joined #openstack-ironic17:02
openstackgerritGalyna Zholtkevych proposed openstack/ironic: ETAG supporting to enhance API evolution  https://review.openstack.org/39221317:03
dtantsurleaving early today, see you tomorrow17:06
*** dtantsur is now known as dtantsur|afk17:06
JayFo/17:07
*** causten has quit IRC17:10
lucasagomesnot sure if it's enough but: https://review.openstack.org/#/c/392890/17:11
patchbotpatch 392890 - openstack-infra/project-config - Switch ironic-ipa jobs to Xenial17:11
JayFI'll review lucas17:11
lucasagomesJayF, thanks, I mean it's my first attempt on it. Not sure what other jobs we should switch as well (if not all)17:12
JayFthat looks sane to me17:12
JayFlucasagomes: the only thing I'm curious about is if that would impact stable jobs at all17:12
JayFlucasagomes: aiui all our jobs should be running xenial at this point17:12
lucasagomesyeah, good point on the stables... let's see if I can figure it out17:13
JayFIt's really hard to know exactly what you're impacting when changing those jobs17:13
JayFzuul v3 will make it a lot more clear b/c we can actually make changes in-tree and see them take effect17:14
mgouldJayF: hurrah!17:14
*** rbudden has joined #openstack-ironic17:14
lucasagomesoh that would be perfect. That's exactly my fear because I see inspector there, I see granade I don't wanna touch those at the moment17:14
lucasagomes(running tox locally worked for the templates with my changes)17:15
JayFlucasagomes: xenial is what is going to be supported for pike though, so we gotta get all our jobs working there17:15
JayFI'm wondering why we have the strange mix we have17:15
JayFi.e. why py34/35 jobs aren't running on xenial yet17:16
JayFlucasagomes: that's another thing; all the jobs running xenial have -xenial in the name; idk if we'd want to continue that17:16
lucasagomesJayF, yeah or functional... or why postgres have xenial and trusty there17:16
JayFayup17:16
JayFmight be worth an email to the mailing list to get a grip on what is transitioning where17:17
lucasagomesJayF, re: name... no idea17:17
JayFand if there were known issues that are tribally known but not documented17:17
lucasagomesvsaienk0, it was you that hit some problems with xenial ?17:17
mgoulddtantsur|afk spent a lot of time fighting with it, IIRC17:18
* lucasagomes remembers the MTU thingy17:18
mgouldlucasagomes: https://review.openstack.org/38465217:19
patchbotpatch 384652 - ironic - Fix broken xenial job (MERGED)17:19
*** harlowja has quit IRC17:21
*** kvcobb has quit IRC17:25
*** e0ne has quit IRC17:27
*** e0ne has joined #openstack-ironic17:28
*** e0ne has quit IRC17:28
*** e0ne has joined #openstack-ironic17:29
lucasagomesright, I will hunt reviews/expert on that and working on changing the jobs to use ipmitool tomorrow17:31
lucasagomescalling it a day all, have a great evening!17:31
JayFo/ have a good evening17:31
mgouldlucasagomes: gnite!17:31
*** lucasagomes is now known as lucas-afk17:32
anishdtantsur|afk: can anyone else do the second review for all the python-dracclient patches ?17:34
mgouldanish: looking now17:36
*** [1]cdearborn has joined #openstack-ironic17:36
*** milan has joined #openstack-ironic17:37
*** baoli has quit IRC17:38
*** e0ne has quit IRC17:39
* mgould had totally forgotten he was now a core on python-dracclient17:39
mgouldmwahahaha17:39
anishheh, thanks mgould !17:40
mgouldI took over from ifarkas, but the handover was quite abrupt17:40
*** fragatina has quit IRC17:40
openstackgerritOleksandr Berezovskyi proposed openstack/ironic-python-agent: CPU inspection now supports multiple processors  https://review.openstack.org/39129017:40
*** ccamacho is now known as ccamacho|out17:42
*** harlowja has joined #openstack-ironic17:44
mgouldanish: all +W'ed, thanks for the patches!17:46
anishman, that's some velocity :)17:47
mgouldthey were all pretty simple, and dtantsur|afk had already +2ed them :-)17:47
*** jpich has quit IRC17:47
anishrelated question : I am adding sensor + draccard profile support to python-dracclient17:47
anishmy code's ready and working17:47
mgouldcool17:48
anishbut I can't find the the schedule for ironic on spec/blueprint submissions17:48
mgouldwe decided at the summit that python-dracclient is being dropped from the ironic project's governance17:49
mgouldso it doesn't actually matter what ironic does here17:49
anishohh, interesting17:49
anishdoes that remove the spec requirement or just the schedule requirements17:49
mgoulder, dunno17:49
openstackgerritOleksandr Berezovskyi proposed openstack/ironic-python-agent: CPU inspection now supports multiple processors  https://review.openstack.org/39129017:50
mgouldI think it means that the python-dracclient cores can now do whatever the hell we like?17:50
*** rbartal has quit IRC17:50
anishdoes it get replaced by python-redfish or something, or is it just being spun out of ironic ?17:50
TheJuliaIronic does not have an official freeze, so their is no sxhwdule17:50
*** athomas has quit IRC17:50
mgouldTheJulia: ah, thanks17:50
mgouldanish: yes to both, IIRC17:50
mgouldone moment...17:50
TheJuliaAdditionally sub-projects could always have their own cores :)17:51
mgouldanish: https://etherpad.openstack.org/p/ironic-ocata-summit-contributor-meetup17:51
*** athomas has joined #openstack-ironic17:51
TheJuliaanish: think of it this way.  If the code in the ironic repo talking to python-dracclient requires additions or changes for features, then a spec would be required.17:52
mgouldanish: docs patch explaining this here: https://review.openstack.org/#/c/392685/17:52
patchbotpatch 392685 - ironic - Clarify ironic governance requirements and process17:52
*** aNuposic has quit IRC17:52
anishTheJulia: thanks will submit a spec, that was my original plan anyways17:52
mgouldanish: cool, I look forward to reading it17:53
*** aNuposic has joined #openstack-ironic17:53
mgouldbut the python-dracclient side can be submitted without a spec, I think17:53
TheJuliaIt can17:53
mgould\o/17:53
anishoh I misread that17:53
TheJuliaWell, more than anything that is up to the cores over that repo17:53
anishfaster patches !17:53
anishso is there a future-redfish-plans doc somewhere ? The only reason I started looking at python-dracclient was dell redfish support was woeful17:54
mgouldheh17:54
anishapparently that's changed with the 2.40 drac release17:54
anish(or so they claim)17:54
* mgould hasn't looked17:55
TheJuliaanish:  there is a spec in review for redfish support17:55
TheJuliabcornec has been championing that effort17:55
mgouldhttps://review.openstack.org/#/c/184653/17:55
patchbotpatch 184653 - ironic-specs - Propose Redfish support into Ironic17:55
*** anush has joined #openstack-ironic17:55
*** derekh has quit IRC17:55
TheJuliaThank you mgould17:56
anishoh man, how did I miss this17:56
mgouldanish: AIUI it's something we'd very much like to have in Ironic, but neither the protocol itself nor the implementations are quite ready yet17:56
mgouldanish: there's a lotta stuff going on, you can't pay attention to it all :-)17:56
mgouldmore reviews on that spec always welcome, of course17:57
anishmy experience with redfish was it's similar but not the same between vendors.17:57
mgouldyeah17:57
mgouldbcornec gave a talk at the summit in which he mentioned a few key bits which were still vendor-defined17:58
TheJuliaMore than anything, we need a way to do CI testing for a redfish driver which is the true blocker, although I think we will see rapid progress on that17:58
mgouldyeah17:58
mgouldTheJulia: would it be enough to test against the reference virtual implementation?17:59
mgouldor do we need to test against actual redfish-supporting hardware?17:59
anishiirc, that is incomplete no (assuming you mean simulator)17:59
JayFmgould: the rule is that it has to have ci17:59
JayFmgould: if it's emulated ci; that might be sufficient17:59
TheJuliamgould: for power/boot control, the consensus that I encountered at the summit was yes.  More advanced things like vendor specific things would need to be bolt-ons with appropriate CI though18:00
mgouldJayF: yeah, I think DMTF provide a docker container running a redfish server that emulates a BMC18:00
TheJuliaThey do18:00
mgouldanish: https://www.openstack.org/summit/barcelona-2016/summit-schedule/events/16231/empowering-ironic-with-redfish-support18:00
anishmgould: I think that was devananda18:01
mgouldanish: sorry, what was?18:01
anishhttps://github.com/devananda/python-redfish18:01
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Introducing node introspection state management  https://review.openstack.org/34894318:01
anishthe simulator18:01
TheJuliabcornec:  perhaps a weekly meeting or call, or something to push it forward18:01
*** ralonsoh has quit IRC18:01
mgouldanish: that looks like a client library to me, or does that contain the simulator as well?18:02
* TheJulia gets out laptop to type faster18:02
anishmgould: yeah, the layout is just confusing https://github.com/devananda/python-redfish/tree/master/dmtf18:02
mgouldOK, thanks18:03
anishbcornec: very interested in any sort of regular meetings. we're doing python-dracclient and our own redfish work internally, I would rather do it for openstack18:03
mgouldanish: OK, looks like it includes a binary of the simulator and some code to unpack it into a docker container18:04
anishayup18:04
mgouldbut anyway, that's the thing I meant :-)18:04
mgouldvideo of bcornec's talk: https://www.youtube.com/watch?v=KxRo5cRpj6k18:05
TheJuliaWhat I suggested, which is horrible, but hopefully minimally sufficent was to put up a rest api similar to virtualbmc's ipmi server, and only implement the endpoints required for testing the node.18:05
mgouldthat's me asking annoying questions about vendor extensions at the end18:05
anishhaha18:05
anishvendor extensions are the worst18:05
TheJuliaanish: fyi, bcornec is based out of UTC+118:05
TheJuliatl;dr we're leaving him messages in his irc client that he will see tomorrow :)18:06
anishspain I'm guessing ?18:06
mgouldFrance, I think18:06
TheJuliaYup, France18:07
mgouldTheJulia: so write our own Redfish BMC emulator implementing the subset of the API we care about?18:07
TheJuliamgould: bingo18:07
anishor strip out the unwanted parts of the API ?18:08
anishthe simulator just reads a giant json iirc and serves it back18:08
TheJuliaWe would actually want to wire the redfish api to control a local VM through virsh18:08
TheJuliaso /redfish/v1/servers/testvm1 could map to testvm1 in virsh18:08
* mgould wonders if we could add the functionality into virtualbmc18:08
TheJuliamgould: that ;)18:09
mgouldyeah, that sounds relatively doable18:09
* mgould hasn't looked deeply at the virtualbmc code, though18:09
mgouldhttps://github.com/umago/virtualbmc, for those following along18:10
TheJuliaThe library it leverages, if memory serves, makes it rather easy since I started prototyping something similar before lucas created virtualbmc18:10
* anish is following curiously18:11
TheJuliaor https://github.com/openstack/virtualbmc18:11
mgouldah, cool18:11
*** athomas has quit IRC18:11
mgouldOK, so it subclasses pyghmi.ipmi.bmc and makes libvirt calls to simulate the effects of each IPMI command18:14
*** anush has quit IRC18:14
*** jvgrant_ has joined #openstack-ironic18:15
anishhrm, the virtualbmc part is already part of the spec18:16
anishsomeone was thinking ahead !18:16
mgouldalways nice when that happens :-)18:16
* TheJulia smiles18:16
*** openstackgerrit has quit IRC18:18
TheJuliaI really want to see redfish support get in.  I also chatted with openbmc folk in BCN, I know they were going to talk with bcornec some because standardization == ++good18:18
anishsame here, at oscon18:18
*** openstackgerrit has joined #openstack-ironic18:18
mgouldhell yes18:18
mgouldanish: FWIW, the python-dracclient cores are listed at https://review.openstack.org/#/admin/groups/1032,members18:18
anishthey seem to have stalled a bit though, they were almost there last time I talked to them18:18
*** jvgrant has quit IRC18:18
anishTheJulia: was it shenki or redbeard ?18:19
TheJuliaopenbmc already has a virsh wrapper for their rest api though :)18:19
*** jkilpatr has quit IRC18:19
TheJuliaanish: I'm horrible mapping names to faces, and then remembering names.  It was one of the folks at IBM18:19
anishTheJulia: I was just about to ask if someone has done the reverse for virsh18:19
anishI think they rely heavily on qemu for their testing/development so the virsh wrapper makes total sense18:20
TheJuliathere was also chatter of possibly doing the same as redfish, putting up an a driver in ironic for openbmc, but it really seems like there is a desire to head in a common direction18:21
mgouldTheJulia: does OpenBMC define its own protocol?18:22
TheJuliamgould: So, based on the ad-hoc discussion where I introduced people, my impression was that they did18:22
mgouldbah18:23
mgouldyeah, looks like it: https://github.com/facebook/openbmc/blob/master/common/recipes-rest/rest-api/rest-api_0.2.bb18:24
anishwrong openbmc https://github.com/openbmc/openbmc18:25
mgouldgah18:25
anish(not the same)18:25
anishbecause, why make things less confusing18:25
anishI understood the openbmc thing differently. they basically concluded they need to support redfish to not be obsolete18:26
mgouldaha: https://github.com/openbmc/docs/blob/master/rest-api.md18:26
*** anush has joined #openstack-ironic18:26
mgouldright, /me -> home18:29
mgouldgood night, everybody!18:29
mgouldanish: clearly whoever got there second should rebrand to LibreBMC18:29
TheJuliagoodnight mgould18:29
anishnight mgould18:30
mgouldgoodnight TheJulia anish18:30
*** mgould is now known as mgould|afk18:30
* TheJulia ponders coffee18:31
* anish sips coffee stares at TheJulia 18:31
anishTIL, /me swallows commas18:32
* TheJulia now really kind of wants coffee, but wonders about commas instead.18:32
TheJuliaanish: I think it may be your client18:32
*** jkilpatr has joined #openstack-ironic18:32
anishpossibly, it's an old version of irssi18:33
TheJuliaWeird...18:34
*** shobhar_ has quit IRC18:37
*** pester has quit IRC18:39
openstackgerritJim Rollenhagen proposed openstack/ironic: Clarify ironic governance requirements and process  https://review.openstack.org/39268518:40
* jroll chugs coffee18:40
*** shobhar_ has joined #openstack-ironic18:42
JayFfwiw facebook sponsors openbmc18:43
JayFso I suspect those two projects might be the same thing, just at different points in time18:44
sambettsis openBMC linked with the opencompute hardware stuff ?18:44
anishfrom what I heard from the openbmc ppl last time I met them, not so18:44
* TheJulia awaits for rendered doc output from CI18:44
JayFsambetts: openbmc is used by some opencompute platforms, such as barreleye (the openpower stuff)18:45
JayFsambetts: but not actually affiliated18:45
*** baoli has joined #openstack-ironic18:45
sambettsah cool18:45
JayFWhen I was at the OCP summit about a year ago or so18:45
sambettsI looked into some of the opencompute chassis stuff at a summit or two back and it was pretty cool18:45
JayFI talked to them about redfish and they barely knew what I was talking about18:45
JayFso nice to hear they've come around18:45
JayFsambetts: all onmetal servers are open compute :)18:45
JayFsambetts: so I've worked with them in production :D18:46
jrollsince when is 'opencompute in production' a :D18:46
jroll:P18:46
jroll(I kid, they've been pretty good to us)18:46
JayFjroll: destroy-proprietary-control (of the datacenteR)18:46
JayFlol18:46
jroll++18:46
sambettsoh cool, like the ones with all the harddrives laying down in them?18:46
JayFthere are multiple different ocp designs18:47
anishso many18:47
JayFWe run 3 different brands18:47
sambettsoh :/ I have no idea which ones I saw then18:49
sambettssomethink like this https://mguerri.web.cern.ch/mguerri/2015-01-21-OCP/img/wiwynn-jbod.png18:49
sambettssomething*18:49
*** portdirect is now known as portdirect_away18:51
*** yuanying has quit IRC19:03
*** milan has quit IRC19:10
*** bfournie has quit IRC19:10
*** Goneri has quit IRC19:10
*** anush has quit IRC19:11
*** moshele has joined #openstack-ironic19:13
*** bfournie has joined #openstack-ironic19:13
*** bfournie has quit IRC19:18
*** jerrygb has quit IRC19:21
*** jerrygb has joined #openstack-ironic19:22
*** ChubYann has joined #openstack-ironic19:22
*** sabeen has quit IRC19:24
*** jerrygb has quit IRC19:26
*** mmedvede has quit IRC19:28
*** moshele has quit IRC19:30
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Test discovery job  https://review.openstack.org/39294319:36
*** jerrygb has joined #openstack-ironic19:42
*** wajdi_ has quit IRC19:43
*** anush has joined #openstack-ironic19:44
*** bcornec1 has quit IRC19:45
*** rcernin has joined #openstack-ironic19:45
*** bcornec1 has joined #openstack-ironic19:45
TheJuliabcornec: FYI, I replied to the redfish spec again19:47
*** sabeen has joined #openstack-ironic19:47
*** bfournie has joined #openstack-ironic19:52
openstackgerritMerged openstack/ironic: API: lookup() ignore malformed MAC addresses  https://review.openstack.org/39211419:55
openstackgerritSukhdev Kapur proposed openstack/ironic: Add support for Security Groups for baremetal servers  https://review.openstack.org/36145119:57
*** openstack has joined #openstack-ironic19:59
*** yolanda has quit IRC19:59
*** shobhar_ has quit IRC20:01
*** mmedvede has joined #openstack-ironic20:04
*** Goneri has joined #openstack-ironic20:04
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Test discovery job  https://review.openstack.org/39294320:04
*** anush has quit IRC20:05
*** dprince has quit IRC20:08
*** devananda has quit IRC20:12
rama_yHello, I have a question about /opt/stack/data/ironic-inspector/ramdisk-logs20:15
JayFGo ahead and ask20:15
*** e0ne has joined #openstack-ironic20:16
rama_yJayF: when does a new entry get created for a particular node in ramdisk-logs?20:16
JayFWhat do you mean by a new entry?20:16
rama_yI have updated a node with new ramdisk and kernel images and I expected to see a new log entry20:16
*** jtomasek has quit IRC20:17
rama_yI see several tar.gz files per node uuid20:17
*** openstackgerrit has quit IRC20:18
rama_yI am trying to figure out if a new entry is created per node whenever its images are updated or when node is inspected.20:18
JayFhttps://github.com/openstack/ironic-inspector/blob/de04f7c5a0d768f54d2b20e78005638d7dba5781/ironic_inspector/process.py#L46 is the code in ironic-inspector that writes those logs out20:18
JayFit looks like they're named based on node uuid20:18
JayFand it looks like old logs are getting truncated20:18
*** openstackgerrit has joined #openstack-ironic20:18
JayFso it would be the same logfile with different contents, unless I'm misreading this20:18
rama_yI have several entries per node uuid with different time stamps. Oh I see, these are just continued in different files, basically.20:19
*** e0ne has quit IRC20:20
JayFyeah; I mean, it's pretty straightforward how it's handled in the code20:20
JayFI'm not 100% sure what you're asking but I think it may be answered?20:20
*** e0ne has joined #openstack-ironic20:20
rama_yJayF: I am actually interested in the var/log/ironic-python-agent.log20:21
*** jtomasek has joined #openstack-ironic20:21
*** e0ne has quit IRC20:21
*** e0ne has joined #openstack-ironic20:21
rama_yI do not see new date stamp after I updated the node with new images and did a node inspection in the ironic-python-agent.log20:21
*** e0ne has quit IRC20:21
rama_yWhat am I missing?20:21
JayFSo you had a node20:22
rama_yyes20:22
JayFyou updated the ramdisk and kernel to include (I'm assuming) some changes you made20:22
rama_yyes20:22
JayFre-inspected, and you didn't see a new set of logs get added in the ramdisk-logs dir?20:22
rama_yyes20:22
JayFDid you check the ironic-bm-logs/ logs for that node? The actual console output?20:22
JayFand read the logs to see if anything unexpected or different happened compared to other inspections you've done20:23
rama_yI checked the console output; it showed node going into introspection20:23
JayFCan you paste those logs?20:25
JayFinto a pastebin or something and link them here20:25
rama_ysure20:25
rama_ypaste.openstack.org/show/587690/20:29
*** jkilpatr has quit IRC20:30
JayFSo that pretty clearly looks like inspection didn't work on the node20:30
JayFbecause there are no logs there20:30
rama_yThis shows no bootable device;20:30
JayFdoes the node in ironic show the same way?20:30
JayFlike does it show as inspection failed?20:30
*** rbudden has quit IRC20:30
rama_yIt shows inspect failed20:30
JayFokay; and the only change you made was building your own ramdisk and putting the informatino about it in your node20:31
JayFso most likely that somehow got messed up20:31
*** devananda has joined #openstack-ironic20:31
JayFdo you wanna verify that those ramdisk and kernel options are mapped to the proper glance images?20:31
rama_yYes; prior to that, I set the node's maintenance mode to false.20:31
rama_yI will stop and start ironic-inspector and see if it helps.20:32
JayFthat will not likely make any difference20:32
JayFlets just do a basic verificatino of the environment -- on one of the nodes you haven't changed anything about the ramdisk at all on, lets try to inspect it and see if it succeeds20:33
rama_yOk, I am inspecting another node that is in manageable state.20:34
*** sabeen has quit IRC20:35
*** moshele has joined #openstack-ironic20:36
rama_yJayF: all nodes are failing inspection; in node-cache there is database integrity error.20:38
JayFrama_y: I've never heard of a database integrity error in node-cache? Where are you seeing that error? Can you paste the logs?20:38
openstackgerritSam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation  https://review.openstack.org/39295920:38
rama_ysure20:39
openstackgerritMario Villaplana proposed openstack/ironic: Add ability to time out on individual clean steps  https://review.openstack.org/39155420:42
rama_ypaste.openstack.org/show/587692/20:42
*** moshele has quit IRC20:42
*** aNuposic has quit IRC20:43
JayFrama_y: interesting; I'm not sure if that's to be expected or not20:44
JayFrama_y: it looks at a glance like it's just trying to re-insert the same info twice20:44
JayFin the log it says bmc_address-127.0.0.1 rather than bmc_address=127.0.0.120:44
JayFidk if that's supposed to be that way or indicates some kind of problem though20:45
*** srobert has quit IRC20:45
rama_yI don't know either; I may have sent several nodes into inspection in succession20:47
JayFthat should be a fine thing to do20:47
JayFit seems like somehow your environment has gotten messed up; I'd unstack/restack and then make sure you can inspect a node without making changes20:47
rama_ySure, thanks JayF.20:48
*** jkilpatr has joined #openstack-ironic20:50
openstackgerritMerged openstack/python-ironic-inspector-client: Fix functional tests after recent API changes  https://review.openstack.org/39001820:52
*** sabeen has joined #openstack-ironic20:53
*** shobhar_ has joined #openstack-ironic20:53
*** krtaylor has quit IRC20:55
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Allow to configure cache clean up  https://review.openstack.org/39294320:55
openstackgerritAnton Arefiev proposed openstack/ironic-inspector: Allow to configure cache clean up period  https://review.openstack.org/39294320:56
*** Sukhdev has joined #openstack-ironic20:57
*** shobhar_ has quit IRC21:02
JayFSo question about inspection21:03
JayFapparently setting deploy_ramdisk and deploy_kernel only impacts cleaning/deployment and not inspection, at least from some testing aslezil and I did21:03
JayFAre there separate settings fro inspection ramdisk? Where/what are they? I can't find anything about it in documentation21:04
sambettsJayF: inspection has its own files in the tftp root21:04
JayFsambetts: so if I want to modify the ramdisk a node uses in inspection21:04
*** jtomasek has quit IRC21:04
JayFsambetts: what's the right setting to modify on the node?21:04
JayFsambetts: and where does whatever the default is come from?21:04
sambettsJayF: there isn't a way at the moment we're working on making it work like that, currently the same ramdisk is used for all nodes and its stored in the tftp root, and the dhcp/kernel pxe params are all set in the inspector specfic dnsmasq/pxe settings21:06
JayFsambetts: so basically you don't even have to upload ramdisks for inspection to glance at all?21:06
JayFyou just yolo copy them into the tftproot?21:06
sambettsyep21:06
JayFthat's ... surprising21:06
JayFaslezil: rama_y ^ that's relevant to what both of you are working on21:06
JayFsambetts: Man, that's almost impossible to figure out as someone with ironic but not inspector knowledge :( I couldn't find it documented anywhere either21:07
aslezilthanks JayF and sambetts :)21:07
openstackgerritMichael Turek proposed openstack/ironic-specs: Add spec for OpenBMC power module, management module, and driver.  https://review.openstack.org/38821821:07
sambettsyeah we're working on making it possible for the inspection interface to use the prepare_ramdisk function on the boot/deploy interfaces21:07
JayFperfect21:07
JayFaslezil: good hunch on proving your code was actually running21:07
JayFsambetts: you should meet aslezil -- annie is working on adding some inspector collectors, and right now is figuring out how it all works before writing her first spec21:08
sambettsJayF: I think all we have is here: http://docs.openstack.org/developer/ironic-inspector/install.html#configuring-pxe21:09
aslezilsambetts that would be great if you could help me.21:09
JayFyeah, reading that made me come in here and ask21:09
JayFSam is probably going to help you the most by reviewing your specs and code :)21:09
JayFso bribe him at all opportunities :P21:09
JayF(the best way to bribe openstack contributors is with quality contributions and code review, lol)21:10
aslezilhe he21:10
*** e0ne has joined #openstack-ironic21:10
*** e0ne has quit IRC21:10
sambettsHi aslezil, whats your timezone?, its like 21:00 for me right now so I'm not normally on this late, I just happened to still have IRC open ;) I'm in the UK so tend to be around in UTC time21:10
*** e0ne has joined #openstack-ironic21:11
rama_yThanks JayF, sambetts. I was looking at the tftpboot as well21:11
*** e0ne has quit IRC21:11
*** e0ne has joined #openstack-ironic21:11
*** e0ne has quit IRC21:12
*** e0ne has joined #openstack-ironic21:12
*** e0ne has quit IRC21:13
*** e0ne has joined #openstack-ironic21:13
aslezilsambetts: I am in CDT(Texas). I am trying to debug and understand the hankshake between IPA and inspector to write my spec for adding a collector in Ironic Inspector.21:13
*** e0ne has quit IRC21:14
*** e0ne has joined #openstack-ironic21:14
*** e0ne has quit IRC21:14
rama_ysambetts: will tinyipa kernel, ramdisk images work in the tftproot?21:16
sambettsyup :)21:16
rama_yThanks.21:16
rama_yJayF: quick question about placing 'logs' collector at the end of all the collectors.21:17
JayFrama_y: go ahead21:17
rama_yDoes this need to be checked in ironic-inspector/devstack/plugin.sh?21:18
rama_yAnd/Or ironic.conf [pxe] pxe_append_params= as well?21:19
*** bfournie has quit IRC21:19
sambettsaslezil: so right now there isn't really a "handshake" as such, the way inspection works is that we have a default DHCP rule that PXE boots IPA onto anything not blocked by the inspector firewall rules, we pass the inspectors IP address into IPA via the kernel parameters, IPA runs its collectors and then sends all the collected data to the inspector data collection endpoint, at that point the21:19
sambettsramdisk is finished and inspector doesn't call back to it, all data processing then happens on the inspector server side21:19
sambettsthis is unlike the ironic use case for the ramdisk where there is handshake via the lookup and heartbeat endpoints21:20
JayFrama_y: I think those collectors are set in the ironic-inspector devstack plugin, if you wanted to change the order of those collectors you'd do it there and restack21:21
rama_yJayF: Thanks!21:21
sambettsthe collectors to run are configured via the PXE kernel parameters which are set by the devstack plugin21:21
aslezilsambetts:thank you for detailed explaination :)21:22
rama_ysambetts: the devstack plugin of Ironic Inspector, right?21:22
sambettsrama_y: correct21:22
sambettsaslezil: No problem :D if you have any other questions feel free to message me on here async and I'll pick them up in the morning21:23
* sambetts -> sleep 21:23
*** sambetts is now known as sambetts|afk21:23
sambetts|afknight aslezil, rama_y, JayF21:23
JayFnn21:23
rama_yThanks sambetts}afk21:23
aslezilgood night sambetts21:23
mrdaMorning Ironic21:27
mariojvmorning mrda21:28
*** jrcloud has joined #openstack-ironic21:35
Sukhdevvsaienk0 : Ping21:35
*** zackf has quit IRC21:37
Sukhdevvsaienk0 : when you get on-line, please ping me21:38
*** jcoufal has quit IRC21:38
openstackgerritOleksandr Berezovskyi proposed openstack/ironic-inspector: Added support of new CPU data format  https://review.openstack.org/39297621:41
mrdamariojv: o/21:43
*** jrcloud has quit IRC21:45
*** jheroux has quit IRC21:46
*** jrcloud has joined #openstack-ironic21:50
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/39273021:53
*** e0ne has joined #openstack-ironic21:54
openstackgerritOpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements  https://review.openstack.org/38993822:00
*** harlowja has quit IRC22:02
*** rbrud has quit IRC22:14
rama_yHi JayF, aslezil: so, to inspect a node with a specific ramdisk image, we need to have that image copied to tftpboot/master_images?22:36
JayFrama_y: it's not possible to set the inspection image per ndoe, you have to replace it globally22:36
JayFand I believe that's where it is22:36
JayFaslezil should be able to confirm as I think she's testing it right now22:36
rama_yOk22:38
aslezilJayF rama_y I am trying to find the path to add the images but not able to find it22:39
JayFso lets backtrack it then, right?22:40
JayFaslezil: you remember where the ipxe config is for ironic-inspector? That should have a url to the ramdisk/kernel you're downloading22:40
rama_yaslezil, JayF: in plugin.sh, there are two parameters: IRONIC_AGENT_KERNEL_URL and IRONIC_AGENT_RAMDISK_URL22:40
JayFrama_y: that's the URL that devstack will download the agent from if you don't build it. It's not relevant here since you're building your own agent.22:41
rama_yOh I see22:41
JayFso basically I think the images are going to be in $IRONIC_HTTP_DIR/ironic-inspector.[kernel|initramfs]22:41
aslezilthat's what rama_y mentioned /opt/stack/data/ironic/tftpboot/master_image22:42
*** Goneri has quit IRC22:42
JayFare there no ironic-inspector.* files in there?22:42
JayFhttps://github.com/openstack/ironic-inspector/blob/master/devstack/plugin.sh#L12822:42
JayFpoints to IRONIC_HTTP_SERVER22:42
JayFlet me see how that's configured; what dir it's using for webroot22:42
*** maratoid_ has joined #openstack-ironic22:42
aslezilJayF: I didn't find any22:42
JayFhttps://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L479 looks like here is where it configures apache22:43
JayFusing a template from the devstack files dir22:43
JayFso going to trace backwards from there22:43
JayFhttps://github.com/openstack/ironic/blob/master/devstack/files/apache-ironic.template is the template, but HTTPROOT is a variable, so what is it replaced with22:44
JayFlooks like $IRONIC_HTTP_DIR from the previous link22:44
JayFhttps://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L247 says IRONIC_HTTP_DIR=${IRONIC_HTTP_DIR:-$IRONIC_DATA_DIR/httpboot}22:44
*** syed_ has quit IRC22:45
JayFso instead of /opt/stack/data/ironic/tftpboot, try /opt/stack/data/ironic/httpboot/22:45
JayFaslezil: rama_y ^ follow how I got there? And I bet in there you'll see the ironic-inspector.* files22:45
JayFas a warning, something I saw when reading this devstack code is that on unstack/restack that directory gets cleared, so make sure any ramdisks you put in there are backed up so you can have them even if you unstack/restack22:45
rama_yJayF, aslezil: yes22:46
aslezilJayF: these files are found ther22:46
JayFso that's the ramdisk and kernel you have to replace with the custom ones you built22:46
JayFto get new code testing in inspection22:46
JayFI'd backup the originals just so you can flip it back to 'stock' in case you need to see if a problem is caused by your code or by something else22:46
aslezilJayF you just copy the new images ther right?22:46
JayFThe file you would've uploaded to glance as the aki (kernel image) would be copied to ironic-inspector.kernel22:47
JayFthe ari (ramdisk image) would be copied to ironic-inspector.initramfs22:47
JayFthen if you inspect you should get the new code22:47
rama_ySo, we need to first delete the .initramfs and .kernel from httpboot first and then create new glance images?22:48
JayFso ironic-inspector ramdisk images are not managed by glance22:48
JayFthat's what we learned today22:48
aslezilJayF sorry I am wrong the tiny IPA images have te kernal as .vimlinux and ramdisk as .gz files right?22:48
JayFso all you ahve to do is put your ramdisk and kernel over those files22:48
JayFaslezil: basically; ignore the extensions; just do something like mv ironic-inspector.kernel ironic-inspector.kernel.orig && mv /path/to/my/tinyipa.vmlinuz ironic-inspector.kernel22:49
JayFand then the same for ironic-inspector.ramdisk + tinyipa.gz22:49
*** ccamacho|out has quit IRC22:50
aslezilJayF the ramdisk and kernel files should be as .initramfs and .kernal22:50
JayFaslezil: yes; because otherwise you break those urls in the .ipxe config file22:51
aslezilJayF I got it22:51
rama_yThanks JayF22:51
openstackgerritBruno Cornec proposed openstack/ironic-specs: Propose Redfish support into Ironic  https://review.openstack.org/18465322:51
JayFnp :) happy to help22:51
aslezilJayF Thank you :)22:51
JayFI am calling it a day in about 10 minutes though, so if there's anything else speak soon ;)22:51
openstackgerritOleksandr Berezovskyi proposed openstack/ironic-inspector: Added support of new CPU data format  https://review.openstack.org/39297622:52
aslezilJayF good night zzzzzzzzz22:52
JayFhah; idk about sleep, but getting chores done and then watching game 7 of the world series THEN maybe sleep :P22:52
aslezilhe he22:53
rama_yGood night, aslezil; thanks very much, JayF!22:53
JayFno problem22:53
JayFI'm getting to learn how inspector works which was a big gap in my knowledge22:53
JayFit's always fun to learn new stuff22:53
aslezilGood night rama_y22:53
*** shobhar has joined #openstack-ironic22:54
*** g3ek has quit IRC22:57
*** g3ek has joined #openstack-ironic22:58
*** david-lyle has quit IRC23:04
*** harlowja has joined #openstack-ironic23:08
*** absubram has quit IRC23:09
*** sabeen has quit IRC23:11
*** ppiela has quit IRC23:12
openstackgerritOleksandr Berezovskyi proposed openstack/ironic-python-agent: Add possibility to inspect systems with multiple CPUs  https://review.openstack.org/39129023:16
openstackgerritOleksandr Berezovskyi proposed openstack/ironic-inspector: Added support of multiple CPU inspection data  https://review.openstack.org/39297623:18
*** mtanino has quit IRC23:20
*** e0ne has quit IRC23:24
*** david-lyle has joined #openstack-ironic23:30
*** rcernin has quit IRC23:31
*** pmannidi_ has joined #openstack-ironic23:36
*** awiddersheim has quit IRC23:38
*** pmannidi has quit IRC23:38
*** rpioso has quit IRC23:43
*** awiddersheim has joined #openstack-ironic23:51

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