Monday, 2016-12-19

*** yuanying has quit IRC00:08
*** baoli has quit IRC00:09
*** fragatina has joined #openstack-ironic00:40
*** SerenaFeng has joined #openstack-ironic00:45
*** hoangcx has joined #openstack-ironic00:59
*** phuongnh has joined #openstack-ironic01:00
*** baoli has joined #openstack-ironic01:00
*** yuanying has joined #openstack-ironic01:04
*** baoli has quit IRC01:05
*** baoli has joined #openstack-ironic01:07
*** zackf has quit IRC01:07
*** zhangjl has joined #openstack-ironic01:22
*** rbudden has joined #openstack-ironic01:27
*** yufei has joined #openstack-ironic01:41
*** godp1301 has joined #openstack-ironic01:43
*** tuanluong has joined #openstack-ironic01:46
*** trananhkma has joined #openstack-ironic01:47
*** zhs_ has joined #openstack-ironic02:01
*** hw_wutianwei has joined #openstack-ironic02:16
*** godp1301 has quit IRC02:16
*** fragatina has quit IRC02:17
*** SerenaFeng has quit IRC02:23
*** bfournie has quit IRC02:27
*** baoli has quit IRC02:55
*** yuanying_ has joined #openstack-ironic02:57
*** baoli has joined #openstack-ironic02:57
*** godp1301 has joined #openstack-ironic02:58
*** yuanying has quit IRC03:00
*** clif_h has quit IRC03:01
*** clif_h has joined #openstack-ironic03:01
*** Nakato has quit IRC03:10
*** Nakato has joined #openstack-ironic03:10
*** yuanying_ has quit IRC03:13
*** yuanying has joined #openstack-ironic03:13
*** rbudden has quit IRC03:14
*** yuanying has quit IRC03:17
*** baoli has quit IRC03:30
*** baoli has joined #openstack-ironic03:30
*** baoli has quit IRC03:30
*** godp1301 has quit IRC03:36
*** vikrant has joined #openstack-ironic03:42
*** SerenaFeng has joined #openstack-ironic03:57
*** Nisha_Agarwal has joined #openstack-ironic04:14
*** Goneri has quit IRC04:20
*** SerenaFeng has quit IRC04:21
*** baoli has joined #openstack-ironic04:32
*** baoli has quit IRC04:53
*** bks has joined #openstack-ironic05:24
*** SerenaFeng has joined #openstack-ironic05:40
*** gcb has joined #openstack-ironic05:49
*** amotoki has joined #openstack-ironic05:55
*** amotoki has quit IRC05:58
*** jaosorior has joined #openstack-ironic06:06
*** SerenaFeng has quit IRC06:14
*** yuanying has joined #openstack-ironic06:14
*** yufei has quit IRC06:17
*** amotoki has joined #openstack-ironic06:18
*** yuanying has quit IRC06:19
*** yufei has joined #openstack-ironic06:21
*** SerenaFeng has joined #openstack-ironic06:24
*** e0ne has joined #openstack-ironic06:31
*** enginy has joined #openstack-ironic06:31
openstackgerritNaohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off  https://review.openstack.org/21673006:31
*** pmannidi has quit IRC06:32
openstackgerritNaohiro Tamura proposed openstack/ironic: Ipmitool power driver for soft reboot and soft power off  https://review.openstack.org/21673806:33
*** e0ne has quit IRC06:33
*** enginy has quit IRC06:34
openstackgerritNaohiro Tamura proposed openstack/ironic: iRMC power driver for soft reboot and soft power off  https://review.openstack.org/21674306:34
*** dsneddon has quit IRC06:34
*** dsneddon has joined #openstack-ironic06:35
*** enginy has joined #openstack-ironic06:35
*** enginy has quit IRC06:36
*** qwertyco has joined #openstack-ironic06:36
*** e0ne has joined #openstack-ironic06:36
*** e0ne has quit IRC06:38
*** dsneddon has quit IRC06:39
openstackgerritNaohiro Tamura proposed openstack/ironic: Update the existing APIs due to adding get_supported_power_states  https://review.openstack.org/38219406:39
*** dsneddon has joined #openstack-ironic06:39
*** pmannidi has joined #openstack-ironic06:49
*** mjura has joined #openstack-ironic07:03
*** pmannidi has quit IRC07:03
*** tesseract has joined #openstack-ironic07:04
*** fxpester has joined #openstack-ironic07:04
*** tesseract is now known as Guest3325407:05
*** pmannidi has joined #openstack-ironic07:05
openstackgerritNaohiro Tamura proposed openstack/ironic: Generic management I/F for Inject NMI  https://review.openstack.org/34819107:06
openstackgerritNaohiro Tamura proposed openstack/ironic: Ipmitool management driver for Inject NMI  https://review.openstack.org/34872107:08
*** mjura has quit IRC07:09
openstackgerritNaohiro Tamura proposed openstack/ironic: iRMC management driver for Inject NMI  https://review.openstack.org/34873207:09
*** mjura has joined #openstack-ironic07:10
*** amotoki has quit IRC07:10
*** Nisha_ has joined #openstack-ironic07:12
*** Nisha_Agarwal has quit IRC07:12
*** SerenaFeng has quit IRC07:15
*** fragatina has joined #openstack-ironic07:18
pas-hamorning ironic :)07:19
*** SerenaFeng has joined #openstack-ironic07:20
*** baoli has joined #openstack-ironic07:21
*** moshele has joined #openstack-ironic07:22
*** baoli has quit IRC07:26
*** amotoki has joined #openstack-ironic07:32
*** pcaruana has joined #openstack-ironic07:33
openstackgerritHao Li proposed openstack/ironic: Decorator METRICS.timer parameter is inconsistent with function  https://review.openstack.org/41233907:51
openstackgerritTuan Luong-Anh proposed openstack/ironic: Fix setting persistent boot device does not work  https://review.openstack.org/37632207:53
*** swatson_ has quit IRC07:54
*** jaosorior has quit IRC07:58
*** tuanluong has quit IRC08:00
*** amotoki has quit IRC08:06
openstackgerritVladyslav Drok proposed openstack/ironic: Fix policy dict checkers  https://review.openstack.org/41176408:21
openstackgerritVladyslav Drok proposed openstack/ironic: Add test to ensure policy is always authorized  https://review.openstack.org/35017708:21
vdrokmorning all!08:21
*** baoli has joined #openstack-ironic08:22
*** jaosorior has joined #openstack-ironic08:24
*** qwertyco has quit IRC08:24
*** amotoki has joined #openstack-ironic08:25
*** rbartal has joined #openstack-ironic08:27
*** e0ne has joined #openstack-ironic08:30
*** qwertyco has joined #openstack-ironic08:36
*** amotoki has quit IRC08:40
*** dsneddon has quit IRC08:44
*** baoli has quit IRC08:45
*** dsneddon has joined #openstack-ironic08:45
*** jaosorior has quit IRC08:45
*** jaosorior has joined #openstack-ironic08:46
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Migrate to session for glance client  https://review.openstack.org/41076708:47
*** jpich has joined #openstack-ironic08:50
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Keystone session for swift-temp-url generation  https://review.openstack.org/41076408:56
*** jtomasek has joined #openstack-ironic08:57
*** zzzeek has quit IRC09:00
*** tuanluong has joined #openstack-ironic09:01
*** zzzeek has joined #openstack-ironic09:01
*** qwertyco has quit IRC09:06
*** ohamada has joined #openstack-ironic09:06
*** tuanluong_ has joined #openstack-ironic09:11
*** sborkows has joined #openstack-ironic09:16
sborkowsmorning, all09:16
tuanluong_mornig sborkows09:17
*** jtomasek has quit IRC09:19
*** lucas-afk is now known as lucasagomes09:26
*** Nisha_ has quit IRC09:27
aarefievmorning!09:27
sborkowsmorning aarefiev09:27
lucasagomesmorning all09:27
aarefievmorning sborkows, lucasagomes09:27
sborkowsdoes anybody know whether gates are working now without timeouts?09:29
*** SerenaFeng has quit IRC09:33
lucasagomessborkows, looking at the latest patches, it seems to be working fine (the ironic gate at least)09:36
*** athomas has joined #openstack-ironic09:46
*** sborkows_ has joined #openstack-ironic09:47
sborkows_lucasagomes, ok thank you, already requested a recheck09:47
lucasagomescool :-)09:47
*** sborkows has quit IRC09:48
*** amotoki has joined #openstack-ironic09:56
openstackgerritMerged openstack/ironic: Updated from global requirements  https://review.openstack.org/41213309:59
vdrokmorning sborkows_ tuanluong_ aarefiev and lucasagomes !10:02
aarefievmorning vdrok10:02
tuanluong_mroning vdrok aarefiev lucasagomes10:02
lucasagomeso/10:02
sergekMorning Ironicers!10:03
*** milan has joined #openstack-ironic10:03
*** amotoki has quit IRC10:06
sborkows_morning, sergek, vdrok10:07
vdrokmorning sergek10:08
milanmorning Ironic! :)10:08
sergeko/10:08
milanmorning sborkows_ sergek vdrok! :)10:08
vdrokmorning milan :)10:08
*** amotoki has joined #openstack-ironic10:09
vdroklucasagomes: https://review.openstack.org/#/c/216730/31/ironic/api/controllers/v1/node.py I was looking here at the problem with wsme.IntegerType not doing the validation. it seems like even @validate decorator does not actually call validate method of the custom type. So all types that need to have validate called need to do it in frombasetype method. Do10:14
vdrokyou think it's worth fixing in wsme, or maybe redefining in this type in ironic, or leaving as-is with custom validation code inside controller?10:14
patchbotpatch 216730 - ironic - Generic power interface for soft reboot and soft p...10:14
aarefievmorning tuanluong_, sergek, milan10:14
milanmorning aarefiev, tuanluong! :)10:15
*** yuanying has joined #openstack-ironic10:15
*** tuanluong has quit IRC10:16
lucasagomesvdrok, def fixing in wsme. We can workaround in ironic until we get a release10:16
lucasagomesjust leave a FIXME in the code so we don't forget10:16
vdroklucasagomes: aha, OK10:16
lucasagomesvdrok, can you put a patch up for wsme ? I will review it10:16
vdroklucasagomes: will try to today or tomorrow10:17
lucasagomesRight on! Thanks a lot10:17
vdroknp :)10:18
*** Romanenko_K has joined #openstack-ironic10:20
*** trananhkma has quit IRC10:20
pas-hamorning lucasagomes10:23
lucasagomeshi there10:23
milanmorning lucasagomes, pas-ha! :)10:23
*** amotoki has quit IRC10:23
pas-halucasagomes: could you please review the fix to unit tests in staging drivers? apparently I've forgot to remove all relevant AMT parts, and now unit tests are broken :( https://review.openstack.org/#/c/412363/10:24
patchbotpatch 412363 - ironic-staging-drivers - Fix AMT unit tests10:24
pas-hamorning milan10:24
lucasagomespas-ha, ++ sure thing10:24
*** phuongnh has quit IRC10:38
*** hoangcx has quit IRC10:38
*** amotoki has joined #openstack-ironic10:38
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add RPC and object version pinning  https://review.openstack.org/40749110:43
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add version column  https://review.openstack.org/41239710:43
openstackgerritMerged openstack/ironic-inspector: Use the device hints matching mechanism from ironic-lib  https://review.openstack.org/40855210:43
*** derekh has joined #openstack-ironic10:43
*** bks has quit IRC10:43
*** amotoki has quit IRC10:45
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add new dbsync command online_data_migration  https://review.openstack.org/40855610:54
*** tuanluong_ has quit IRC10:55
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Allow diskless nodes thru a configuration option  https://review.openstack.org/32662010:56
*** amoralej is now known as amoralej|brb11:05
openstackgerritZhenguo Niu proposed openstack/ironic: Clean nodes stuck in CLEANING state when ir-cond restarts  https://review.openstack.org/34997111:05
openstackgerritZhenguo Niu proposed openstack/ironic: Clean nodes stuck in DELETING state when ir-cond restarts  https://review.openstack.org/35043911:07
*** Nisha_ has joined #openstack-ironic11:07
*** pester has joined #openstack-ironic11:08
*** fxpester has quit IRC11:10
openstackgerritMerged openstack/ironic: Remove agent vendor passthru completely  https://review.openstack.org/39784811:14
openstackgerritMerged openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/41213411:16
*** yufei has quit IRC11:17
*** jkilpatr has quit IRC11:17
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Keystone session for swift-temp-url generation  https://review.openstack.org/41076411:23
mrtenioGood morning Ironic :)11:30
*** hecliunyx has joined #openstack-ironic11:31
milanmorning mrtenio! :)11:32
vdrokmorning mrtenio11:34
*** rbartal has quit IRC11:37
openstackgerritZhenguo Niu proposed openstack/ironic: Remove unneeded 'reserved' filter in cleaning stuck nodes  https://review.openstack.org/35037711:37
*** rbartal has joined #openstack-ironic11:39
openstackgerritVladyslav Drok proposed openstack/ironic: Fix policy dict checkers  https://review.openstack.org/41176411:40
openstackgerritVladyslav Drok proposed openstack/ironic: Add test to ensure policy is always authorized  https://review.openstack.org/35017711:40
*** baoli has joined #openstack-ironic11:44
*** baoli has quit IRC11:48
*** e0ne has quit IRC11:50
*** lucasagomes is now known as lucas-hungry11:55
*** yolanda has quit IRC11:59
*** yolanda has joined #openstack-ironic12:00
*** jkilpatr has joined #openstack-ironic12:04
*** SerenaFeng has joined #openstack-ironic12:06
*** zhangjl has left #openstack-ironic12:11
*** jkilpatr has quit IRC12:11
sborkows_Can somebody confirm, that gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-coreos-src-ubuntu-xenial is timeouting?12:13
*** amoralej|brb is now known as amoralej12:15
vdroksborkows_: sometimes it wholedisk, sometimes partition, seems just the gates slowed down a bit. maybe we should increase job timeouts12:18
vdroksborkows_: or you mean this one - bash: /opt/stack/new/devstack-gate/devstack-vm-gate.sh: No such file or directory12:19
vdrokthat's something new12:20
*** SerenaFeng has quit IRC12:20
sborkows_vdrok, nope, the first one. I got the same issue as you12:20
*** rbartal has quit IRC12:20
vdrokyeah, that started happening more frequently12:21
*** dprince has joined #openstack-ironic12:22
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add version column  https://review.openstack.org/41239712:22
*** jkilpatr has joined #openstack-ironic12:25
*** rbartal has joined #openstack-ironic12:27
milanand inspector is hosed w/ https://github.com/openstack/tempest/commit/d3685ca17e52e7b51128b57bbc4630b89797e2e712:34
*** bfournie has joined #openstack-ironic12:34
milanhttp://logs.openstack.org/20/326620/6/check/gate-tempest-dsvm-ironic-inspector-discovery-ubuntu-xenial/35696f9/console.html#_2016-12-19_11_33_12_48345312:34
milanaarefiev ^ :-/12:34
aarefievmilan: looking12:35
milanaarefiev, I think we should remove 'baremetal' test tag https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/test/inspector_tempest_plugin/tests/test_basic.py#L6012:40
* milan found https://github.com/openstack/ironic/commit/d0e1d5e1f84367f4f868a62776212e6296a2dee212:41
milanaarefiev, wdyt?12:41
* milan files a bug meanwhile12:41
aarefievmilan: yes, and looks like not only 'baremetal', I see more  redundant tags12:42
milanyeah12:42
milanlet's just keep whatever ironic has12:42
milan?12:42
*** baoli has joined #openstack-ironic12:44
aarefievmilan: we need to delete all 'baremetal' tags as if ironic doesn't exist we will skip test anyway12:45
milanaarefiev, ack12:45
aarefievalso I see redundant  'compute' tag12:46
aarefievwe don't boot instance in some tests12:46
milanaarefiev, Ironic seems to keep the compute https://github.com/openstack/ironic/commit/d0e1d5e1f84367f4f868a62776212e6296a2dee212:46
milanbut OK, I don't follow completely12:47
aarefievmilan: yeah, we need it only for basic test afair12:48
xhkumorning o/12:48
milanmorning xhku! :)12:48
openstackgerritZhenguo Niu proposed openstack/ironic: Add nodes tagging support - objs  https://review.openstack.org/23335712:48
*** baoli has quit IRC12:49
milanaarefiev, https://bugs.launchpad.net/ironic-inspector/+bug/165112312:49
openstackLaunchpad bug 1651123 in Ironic Inspector "Tempest hosed because of removed test tags" [Undecided,New]12:49
*** hw_wutianwei has quit IRC12:50
milanaarefiev, you submitting patch?12:50
* milan can do if aarefiev busy12:51
aarefievmilan: I could if you don't :)12:51
aarefievmilan: ok, I'm assigning to you :)12:52
openstackgerritVasyl Saienko proposed openstack/python-ironicclient: Add interface attach/detach support  https://review.openstack.org/36442012:55
openstackgerritAparna proposed openstack/ironic: Add hpsum firmware update as a clean step  https://review.openstack.org/40927112:55
*** jtomasek has joined #openstack-ironic12:56
milanaarefiev, OK cool :)12:56
* milan hacks12:56
*** skramaja_ has joined #openstack-ironic13:00
*** skramaja has quit IRC13:00
*** moshele has quit IRC13:01
*** lucas-hungry is now known as lucasagomes13:01
*** moshele has joined #openstack-ironic13:01
*** jcoufal has joined #openstack-ironic13:04
xavierrgood morning ironic :)13:07
jrollmorning y'all13:07
*** tuan_luong has joined #openstack-ironic13:09
*** vikrant has quit IRC13:10
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Tempest test tag baremetal doesn't exist  https://review.openstack.org/41244813:11
milanmorning jroll, xavierr! :)13:11
milanaarefiev, ^^ me curious what Zuul has to say about it ;)13:12
aarefievmilan: let's see :)13:13
aarefievmorning jroll, xavierr13:13
* milan -> lunch 13:13
*** gcb has quit IRC13:14
*** milan is now known as milan|afk13:14
*** trown|outtypewww is now known as trown13:14
*** pester has quit IRC13:15
*** bfournie has quit IRC13:17
*** vikrant has joined #openstack-ironic13:19
*** vikrant has quit IRC13:19
*** lindycoder has joined #openstack-ironic13:19
openstackgerritZhenguo Niu proposed openstack/ironic: Add DB layer support for node tags filter  https://review.openstack.org/27262413:21
TheJuliaGood morning everyone13:25
vdrokmorning xavierr jroll and TheJulia13:25
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Create port with specific port group UUID in OSC  https://review.openstack.org/41184013:26
*** watanabe_isao has joined #openstack-ironic13:27
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Create port with specific port group UUID  https://review.openstack.org/41182613:27
watanabe_isaojroll, hello, sir.13:27
watanabe_isaojroll, I have sent you a mail on 12/15 0:00(UTC). Could you confirm if it has arrived or not, please?13:29
jrollwatanabe_isao: hi, yes, I have it13:31
openstackgerritMoshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support  https://review.openstack.org/26425713:31
watanabe_isaojroll, thank you very much.13:32
jrollwatanabe_isao: you're welcome :)13:32
openstackgerritMerged openstack/ironic: Remove grenade config workaround  https://review.openstack.org/41178113:33
watanabe_isaojroll, I know you are in busy. If you have time to reply the mail. It would be a big help. :)13:35
*** gcb has joined #openstack-ironic13:36
jrollwatanabe_isao: yes, I will13:36
*** skramaja has joined #openstack-ironic13:36
jrollzomg, the tempest thing landed https://review.openstack.org/#/c/355586/13:36
patchbotpatch 355586 - tempest - Removing baremetal tests from tempest tree (MERGED)13:36
jrolland didn't break our gate13:36
*** skramaja_ has quit IRC13:36
* jroll \o/ \o/ \o/ \o/ \o/ \o/13:36
milan|afkjroll, that one broke inspector instead :D13:37
jrollwait really? :/13:38
milan|afkjroll, https://bugs.launchpad.net/ironic-inspector/+bug/165112313:38
openstackLaunchpad bug 1651123 in Ironic Inspector "Tempest hosed because of removed test tags" [Critical,In progress] - Assigned to milan k (vetrisko)13:38
jrollyeah I see it now13:38
jrollhow can I help?13:38
milan|afkI think so ;)13:38
* jroll looks at patch13:38
milan|afkjroll, once Zuul is OK we'd need your vote I guess13:38
milan|afkcause I submitted but there's just aarefiev from the cores atm13:39
jrollhopefully this is the last time ironic breaks inspector for a while13:39
jrollyeah13:39
*** jheroux has joined #openstack-ironic13:39
milan|afkyeah :)13:39
* jroll +213:39
milan|afkjroll, thx! :)13:39
jroll:)13:39
* milan|afk really lunch now :D13:39
jrollwe should make inspector vote on ironic, methinks13:40
*** baoli has joined #openstack-ironic13:45
openstackgerritJulia Kreger proposed openstack/ironic: Add storage_interface DB field  https://review.openstack.org/34800513:46
openstackgerritJulia Kreger proposed openstack/ironic: Add storage_interface to base driver class  https://review.openstack.org/34800613:46
openstackgerritJulia Kreger proposed openstack/ironic: Add base cinder common interface  https://review.openstack.org/35562513:46
*** clenimar has joined #openstack-ironic13:46
openstackgerritJulia Kreger proposed openstack/ironic: Add Cinder storage driver  https://review.openstack.org/36619713:46
jrollwatanabe_isao: replied, let me know if it doesn't come through, since we've been having issues with that13:47
mrtenioHi lucasagomes JayF vdrok. Since our CI is back, could you rereview https://review.openstack.org/#/c/377073/? :)13:48
patchbotpatch 377073 - ironic - Adds another validation step when using dynamic al...13:48
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient: Functional tests for port groups in ironicclient  https://review.openstack.org/29772713:49
*** tuan_luong has quit IRC13:49
watanabe_isaojroll, I received it this time. Thank you very much. :D13:50
jrollwoot13:50
jrollthe system works :)13:50
*** baoli has quit IRC13:50
*** nicodemos has joined #openstack-ironic13:51
*** lindycoder has quit IRC13:51
lucasagomesmrtenio, o/ adding to the list13:51
nicodemosmorning, ironic.13:51
lucasagomesjroll, morning! When you get some time (no rush, really) mind taking a quick look at https://review.openstack.org/#/c/375095/ ? This is a patch for devstack changing the default image in ironic to a UEFI capable one13:55
patchbotpatch 375095 - openstack-dev/devstack - Ironic: Set default images for UEFI deployment13:55
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add new dbsync command online_data_migration  https://review.openstack.org/40855613:56
mrteniolucasagomes, thanks :)13:57
*** bfournie has joined #openstack-ironic13:57
mrteniomorning nicodemos13:57
jrolllucasagomes: nice, I don't see any issues with that, +113:57
lucasagomesjroll, thanks!13:58
*** mike_ has joined #openstack-ironic14:00
*** mike_ is now known as Guest682614:01
mrtenio#pixiesay Testing the bot --mood happy14:01
* mrtenio hoped it would work.14:02
*** e0ne has joined #openstack-ironic14:02
*** rloo has joined #openstack-ironic14:05
jrollpixieboots ain't here :(14:05
*** Goneri has joined #openstack-ironic14:06
*** lindycoder has joined #openstack-ironic14:09
*** amoralej is now known as amoralej|lunch14:10
*** melee has joined #openstack-ironic14:12
*** ppiela has joined #openstack-ironic14:12
*** Guest6826 has quit IRC14:14
*** chlong has joined #openstack-ironic14:14
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Make _send_sensors_data concurrent  https://review.openstack.org/40742914:15
*** moshele has quit IRC14:17
*** baoli has joined #openstack-ironic14:19
*** lindycoder has quit IRC14:20
TheJuliaWhere did pixieboots go?14:20
*** sasha21 has joined #openstack-ironic14:21
*** cvstealth has joined #openstack-ironic14:23
*** mkrai has joined #openstack-ironic14:24
*** dyasny has joined #openstack-ironic14:24
*** dyasny has left #openstack-ironic14:24
*** dyasny has joined #openstack-ironic14:24
*** baoli has quit IRC14:25
*** ohamada has quit IRC14:26
*** mkrai_ has joined #openstack-ironic14:26
*** ohamada has joined #openstack-ironic14:26
openstackgerritDaniel Mellado proposed openstack/ironic: Grenade: configure tempest with Ironic plugin  https://review.openstack.org/40068714:27
openstackgerritDaniel Mellado proposed openstack/ironic: Skip tempest tests if ironic isn't available  https://review.openstack.org/37925614:27
*** lindycoder has joined #openstack-ironic14:29
*** baoli has joined #openstack-ironic14:29
*** rbudden has joined #openstack-ironic14:32
*** watanabe_isao has quit IRC14:34
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add new dbsync command online_data_migration  https://review.openstack.org/40855614:42
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Make _send_sensors_data concurrent  https://review.openstack.org/40742914:44
*** ohamada has quit IRC14:44
*** wajdi has joined #openstack-ironic14:44
*** ohamada has joined #openstack-ironic14:44
*** zackf has joined #openstack-ironic14:44
openstackgerritJulia Kreger proposed openstack/ironic: Wire in storage interface attach/detach operations  https://review.openstack.org/40629014:44
dmelladojroll: ping o/14:45
jrolldmellado: hey14:45
dmelladore: the patch about service_available14:45
jrollmhm14:45
dmelladowhen I first started the patch, there wasn't such option in tempest14:46
dmelladoso the tests weren't skipped even if ironic was set to false in service_available14:46
dmelladoif that has been already implemented while this was on hold I can just abandon it :D14:46
*** mkrai has left #openstack-ironic14:47
jrolldmellado: I think it was there before, but we moved it to our tree14:47
* jroll gets another link or two14:47
aarefievjroll: grenade fails here https://review.openstack.org/#/c/412448/1 as we are using plugin from branches14:48
patchbotpatch 412448 - ironic-inspector - Tempest test tag baremetal doesn't exist14:48
jrollyeah, was here before https://github.com/openstack/tempest/commit/d3685ca17e52e7b51128b57bbc4630b89797e2e7#diff-3b51f1b0290f0dc264ed3132913afd36L97814:48
jrollaarefiev: ugh, we shouldn't be doing that, I thought someone was going to fix it?14:48
dmelladojroll: I see, ack then, I'll just abandon the patch14:49
aarefievjroll: https://review.openstack.org/#/c/412477/114:49
patchbotpatch 412477 - openstack-infra/project-config - Use Ironic Inspector tempest plugin from master14:49
dmelladothanks!14:49
jrolldmellado: awesome14:49
jrollaarefiev: awesome14:49
jroll+114:49
jrollthanks for that :)14:49
aarefievit's only copy-paste :)14:50
*** jtomasek has quit IRC14:50
dmelladojroll: btw, now that I got your attention I was checking some ironic stuff with the tempest plugin and got into this while installint ironic with setup.py develop14:50
dmelladohttps://bugs.launchpad.net/ironic/+bug/165114114:50
openstackLaunchpad bug 1651141 in Ironic "Python setup.py develop fails with: "TypeError: dist must be a Distribution instance"" [Undecided,Confirmed]14:50
jrolldmellado: I don't believe we intend to support setup.py commands14:51
*** cdearborn has joined #openstack-ironic14:51
TheJuliasetup.py is managed by openstack requirements14:51
TheJuliaand what jroll said14:51
jrolland anyway looks like a futurist bug, if we decide it's a bug14:51
*** milan|afk is now known as milan14:51
dmelladojroll: TheJulia just out of curiosity, how would you intend then to install ironic14:52
TheJuliapip14:52
dmelladotox would just call setup.py develop under the hood14:52
dmelladopip install -e14:52
dmelladogot it14:52
jrollyep14:52
dmelladojroll: TheJulia thanks for the clarification!14:52
jrollhuh, tox uses setup.py develop? why doesn't tox break then.14:52
dmelladowell, it uses develop mode14:52
jrollso does this bug, it seems?14:53
dmelladohttps://github.com/openstack/ironic/blob/master/tox.ini#L714:53
*** sborkows_ has quit IRC14:53
dmelladobut it doesn't replicate over tox14:53
jrolloh, we override install_command, though14:53
dmelladohttps://github.com/openstack/ironic/blob/master/tox.ini#L814:53
dmelladoyep, just saw that14:53
dmelladothanks for clarifying this in any case, I was just creating a venv with tempest+ironic and ran into some issues14:54
jrollyeah, no worries14:54
*** melee has quit IRC14:55
*** amoralej|lunch is now known as amoralej14:56
*** jtomasek has joined #openstack-ironic14:59
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Catch unknown exceptions in validate driver ifaces  https://review.openstack.org/40615714:59
*** vinsh has joined #openstack-ironic15:01
*** Nisha_ has quit IRC15:04
*** dyasny has quit IRC15:06
milanaarefiev, w/r the https://review.openstack.org/#/c/412448/ failing in grenade, what is the best thing to do? Tempest not being versioned... Should we just back port to stable?15:08
patchbotpatch 412448 - ironic-inspector - Tempest test tag baremetal doesn't exist15:08
milanmorning dmellado, TheJulia! :)15:08
dmelladomorning milan ;)15:09
aarefievmilan: the problem is our plugin is versioned now15:09
aarefievwhat are you going to backport?15:09
milanaarefiev, just asking what should be done15:10
dmelladoaarefiev:tempest is not versioned but we do use tags, maybe you could use that before splittiong out the plugin15:10
aarefievI've proposed patch to always use master as ironic does15:11
dmelladoaarefiev: +115:11
dmelladobut I've one question, at some point what would happen if you release a branch15:12
dmelladoto stable and so on15:12
dmelladowould you intend for that branch to use master as well..?15:12
* dmellado checking the patch15:12
*** jtomasek has quit IRC15:13
aarefievyeah, we doesn't  guarantee that branch plugin will work with master tempest15:15
milanaarefiev, +1 on the patch15:18
* milan just hopes it lands soon15:18
aarefievthx15:19
*** wajdi has quit IRC15:19
*** cdearborn has quit IRC15:20
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Make _send_sensors_data concurrent  https://review.openstack.org/40742915:21
NobodyCamGood Morning Ironic'er... Happy Monday15:21
vdrokmorning NobodyCam :)15:21
dmelladoaarefiev: I mean, at some point it might be useful to have ironic work with the 'stable tempest' tag15:21
dmelladowhen you move it there15:21
dmelladoin any case I really do hope that the proposal for moving the plugins out of tree gets this solved15:21
dmelladoas they'll be branchless as well15:21
*** dyasny has joined #openstack-ironic15:21
milanNobodyCam, good morning! :)15:21
NobodyCamGood Morning vdrok15:21
NobodyCammorning milan15:22
NobodyCam:)15:22
aarefievdmellado: I hope so15:22
aarefievmorning NobodyCam :)15:22
NobodyCammorning aarefiev :)15:22
dmelladoaarefiev: not official yet but most probably we'll try to organize a session again at the PTG ;)15:22
aarefievit would be interesting :)15:23
*** dyasny has quit IRC15:24
TheJuliagood morning NobodyCam15:25
TheJuliagood morning milan15:25
NobodyCamgood Morning TheJulia :)15:25
milan:)15:25
Romanenko_KHello!15:29
vdrokmorning Romanenko_K15:29
vdrokor rather evening :)15:30
*** mkrai_ has quit IRC15:32
*** cdearborn has joined #openstack-ironic15:37
milanmorning Romanenko_K! :)15:38
* milan -> errands15:38
*** milan is now known as milan|afk15:38
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Catch unknown exceptions in validate driver ifaces  https://review.openstack.org/40615715:39
*** aarefiev is now known as aarefiev_afk15:41
*** Syed__ has joined #openstack-ironic15:42
jrolldmellado: aarefiev_afk: fwiw, I did start moving the plugins out but got busy with other things, I hope to pick that up in january or so15:43
*** baoli has quit IRC15:44
dmelladojroll: pls let me know if you'd like me to help on that ;)15:45
jrolldmellado: will do :)15:45
*** baoli has joined #openstack-ironic15:45
openstackgerritGalyna Zholtkevych proposed openstack/ironic: ETAGs maintenance for all api node's requests  https://review.openstack.org/39221315:47
*** cdearborn has quit IRC15:48
*** mtanino has joined #openstack-ironic15:50
*** cdearborn has joined #openstack-ironic15:51
openstackgerritDerek Higgins proposed openstack/ironic-python-agent: Advertise the correct address when using IPv6  https://review.openstack.org/41181715:52
openstackgerritDerek Higgins proposed openstack/ironic-python-agent: lio: Listen for connections on both IPv4 and IPv6 ports  https://review.openstack.org/41181915:52
openstackgerritDerek Higgins proposed openstack/ironic-python-agent: Agent: Listen for connections on both IPv4 and IPv6 ports  https://review.openstack.org/41181815:52
*** Sukhdev has joined #openstack-ironic15:52
*** Sukhdev has quit IRC15:53
*** godp1301 has joined #openstack-ironic15:54
*** cdearborn has quit IRC15:55
*** Sukhdev has joined #openstack-ironic15:57
*** Guest33254 has quit IRC16:02
*** Sukhdev_ has joined #openstack-ironic16:04
rloohi and good monday morning, ironic'ers, including jroll, TheJulia, vdrok, NobodyCam, Romanenko_K, milan|afk, dmellado16:06
vdrokmorning rloo16:06
NobodyCammorning rloo :)16:06
TheJuliagood morning rloo16:07
jrollmorning rloo :)16:07
rloovdrok, vsaienk0: I am totally confused by https://review.openstack.org/#/c/327046/31..33/ironic/drivers/modules/network/common.py, L14616:07
patchbotpatch 327046 - ironic - Add Virtual Network Interface Driver APIs16:07
vdrokrloo: by what exactly? :)16:08
rloovdrok: why port.internal_info[tenant_vif_port_id) doesn't exist16:09
patchbotError: Missing "]".  You may want to quote your arguments with double quotes in order to prevent extra brackets from being evaluated as nested commands.16:09
*** keekz has quit IRC16:09
vdrokrloo: I think because we don't put it there yet16:09
vdrokerr16:09
vdroknot port, portgroup.internal_info.tenant_vif_port_idf16:10
rloowe don't put portgroup.internal_info[tenant..]?16:10
vdroknope16:10
vdrokwe do it only for ports for now16:10
vdroksam's intent was to add portgroups later16:11
*** mjura has quit IRC16:11
vdrokiirc16:11
rloovdrok: but L146 is checking for port vif, not portgroup vif16:11
*** keekz has joined #openstack-ironic16:11
rloovdrok:  and also, if we don't put portgroup.internal_info[tenan...], why do we have L122 which checks there?16:11
*** jtomasek has joined #openstack-ironic16:12
vdrokrloo: ok, lemme look through this once again16:12
rloovdrok: thx, very possible i misunderstand something...16:12
vdrokugh, gerrit is super-slow for me16:12
*** pcaruana has quit IRC16:15
openstackgerritMerged openstack/ironic-python-agent: Add a new Hardware Manager for CNA network card  https://review.openstack.org/39183616:16
rloomariojv, alineb, jroll: how familiar are you with the metrics stuff? Is it OK to change the name of the metric? (wrt backwards compat?) https://review.openstack.org/#/c/412339/1/ironic/drivers/modules/pxe.py16:16
patchbotpatch 412339 - ironic - Decorator METRICS.timer parameter is inconsistent ...16:16
*** jtomasek has quit IRC16:16
jrollyikes16:16
jrollwe never defined what back compat means for these...16:16
mariojvyeah, i don't think there's a real policy around that16:16
jrollI imagine it would be annoying to deal with in the consumer, but I'm not sure how annoying16:17
rlooi was thinking last week, that there seems to be a lot of stuff to remember, when coding/reviewing... :-(16:17
jrollrloo: probably something we need to discuss on the list or something16:17
jrollyes, we need to write more things down like that16:17
mariojvdepends... if you're using it with graphite/grafana i think you'd need to find + replace it for any graphs that use it16:17
rloojroll: write things down *after* we know what to remember to ask :)16:18
*** cdearborn has joined #openstack-ironic16:18
jrollrloo: well, this is a good opportunity :)16:18
rloomariojv: so at least a deprecation period?16:18
rloomariojv: have two metric decorators?16:18
*** bcornec1 has quit IRC16:18
mariojvthere's no list of metrics emitted by ironic that's maintained, right?16:18
rloojroll: true16:19
*** rbartal has quit IRC16:19
mariojvi'm not sure that we should really guarantee much about the metrics16:19
openstackgerritZhenguo Niu proposed openstack/ironic: Add nodes tagging support - API  https://review.openstack.org/25047816:19
mariojvwhat if there's some refactor that removes a method decorated with a metric? block that patch for a cycle?16:20
rloojroll: i just looked at our spec template, and it talks about upgrades/backwards compatibility but to existing ironic. not how this new feature will/can be backwards compatible with itself...16:20
jrollrloo: good point16:20
rloomariojv: let's delete the metrics feature so we don't have to deal with these questions!16:20
mariojv:D16:20
TheJulialol16:20
rloomariojv: are metrics being used in openstack, outside ironic? seems like a xproject thing.16:21
jrollswift has something similar16:21
rloojroll: what do you suggest. -2 on that patch and some action item to figure out our policy on that?16:23
jrollrloo: yeah, either meeting or mailing list16:24
mariojvi think nova collects some metrics for computes, but nothing for api/conductor/cells/etc16:24
* jroll suspects that's another typo patch anyway16:24
mariojvnothing like ours16:24
rloomariojv: not picking on you but jay isn't here, and i can't remember who (but someone in rackspace?) wanted the metrics. Are you available/willing to look into this, what operators might want, what other openstack projects are doing?16:25
mariojvrloo: i'm pretty busy the next 2 days and out until the end of the year after that - but i can do some research early next year16:26
vdrokmariojv: hm, is it really breaking? If it's just written to db, and then you collect statistics?16:27
vdrokI mean renaming16:27
rloomariojv: ok thx. (i'm here til thurs, out til Jan 9, so don't want to start something)16:27
jrollvdrok: yeah, like I said, it's annoying but not sure how annoying. dashboards and such would need to be fixed16:28
jrollit's probably fine but I'd like to have the discussion around it16:28
vdrokaha, ok16:28
rloovdrok: seems like there should at least be a deprecation period to give folks a chance to change over?16:28
jrollit's easy to do a transition period, just decorate it twice16:28
rloojroll: yeah, that's what i'm thinking16:28
mariojvyeah, it could break a consumer16:28
vdrokmhm, decorating twice should be good16:29
*** MattMan has quit IRC16:29
*** MattMan has joined #openstack-ironic16:29
mariojvthere are cases not covered by that though - i.e. if a patch needs to remove one of those decorated methods16:29
rloomariojv: right, so we need to come up with a policy for this stuff, and that policy should be similar across openstack16:29
jrollif the method is removed I suspect you don't need to watch the metrics for it :)16:30
mariojvyeah16:30
mariojvmaybe it'd be good to maintain a list of metrics emitted by ironic, have some code to generate it somehow16:30
mariojvnot sure how hard that'd be16:30
vdrokrloo: so yup, that place need to look into internal_info too16:30
rloomariojv: I'm not going to -2 that patch cuz we all hate -2 but i'm going to comment that you're going to look into a policy for it and until that time ...16:31
rloovdrok: yes! I'm not totally confused then :)16:31
mariojvsounds fine16:31
jrollgrep metrics\.(.*?)\('(.*?)'\)16:31
jroll:P16:31
vdrokrloo: as we're changing standalone_ports_supported to False, and checking that none of the ports have vifs. we sort ports by pxe_enabled value but don't exclude ports that are member of a portgroup16:32
vdrokso, good catch :)16:32
rloovdrok: that's my job! ;)16:32
*** moshele has joined #openstack-ironic16:33
jrollrloo: any rfe review to add to ironic's meeting agenda? :)16:34
*** ElCoyote_ has quit IRC16:35
rloojroll: nope, i was going to do one just for fun, but i ended up just asking for a spec. I saw TheJulia had a few items so didn't feel the need. Wait til next year, i've got some other ideas...16:35
jrollperfect :)16:35
*** snecklifter has joined #openstack-ironic16:37
* TheJulia wonders if it is a good thing, or bad thing when she pre-types long statements for irc meeting.16:38
TheJulias/./s./16:38
*** Sukhdev_ has quit IRC16:39
openstackgerrityolanda.robla proposed openstack/ironic-specs: Ironic deployment steps  https://review.openstack.org/41252316:40
* jroll steps away for a few before the meeting16:41
yolandadtantsur|afk, or other cores... as per our talk with Dmitry, i created a new spec to just reflect the deployment steps. Could you provide feedback on that? i just tried to isolate the content from https://review.openstack.org/#/c/38209116:41
patchbotpatch 382091 - ironic-specs - RAID configuration during deployment16:41
*** cinerama has joined #openstack-ironic16:43
TheJuliayolanda: Thank you!16:43
*** cinerama has quit IRC16:44
*** cinerama has joined #openstack-ironic16:44
*** jaosorior has quit IRC16:48
*** jaosorior has joined #openstack-ironic16:49
*** ElCoyote_ has joined #openstack-ironic16:51
*** melee has joined #openstack-ironic16:52
rloolucasagomes: do you know what the status is wrt the SIGHUP rfe: https://bugs.launchpad.net/ironic/+bug/158559516:53
openstackLaunchpad bug 1585595 in Ironic "[RFE] Handling SIGHUP on Ironic services" [Wishlist,In progress] - Assigned to Lucas Alvares Gomes (lucasagomes)16:53
*** e0ne has quit IRC16:59
*** melee has quit IRC17:01
*** melee has joined #openstack-ironic17:02
lucasagomesrloo, sorry for the delay, I was in a meeting... Still not completed, I had a patch for it at some point but it requires me to create a spec, that I still haven't done17:02
*** ElCoyote_1 has joined #openstack-ironic17:02
lucasagomesrloo, I will unassign it but will leave it in my list to reiterate when I get some time17:02
rloolucasagomes: ok thx. it would be good to have for rolling upgrades, but not a show stopper17:03
lucasagomesindeed17:03
* jroll wonders if that truly needs a spec17:03
lucasagomesit's neat, and oslo.config already supports it17:03
lucasagomesso it should in theory be easy to do17:04
lucasagomesif I have time, I will take a look at it this week17:04
*** ElCoyote_ has quit IRC17:04
lucasagomesin any case, it's unassigned so people can jump in17:04
*** jpich has quit IRC17:04
openstackgerritDerek Higgins proposed openstack/ironic: Wrap iscsi portal in []'s if IPv6  https://review.openstack.org/41180917:06
rloolucasagomes: thx. no big hurry on it; i'll check/look into it in jan. i don't recall why we wanted a spec either.17:07
* lucasagomes joins upstream meeting17:09
*** Romanenko_K has quit IRC17:10
openstackgerrityolanda.robla proposed openstack/ironic-specs: Ironic deployment steps  https://review.openstack.org/41252317:11
*** moshele has quit IRC17:17
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add new dbsync command online_data_migration  https://review.openstack.org/40855617:22
*** milan|afk is now known as milan17:23
milanmorning rloo! :)17:23
*** aNuposic has joined #openstack-ironic17:24
rloohi milan!17:25
openstackgerritVladyslav Drok proposed openstack/python-ironicclient: Add mode and properties to portgroup  https://review.openstack.org/41254517:29
*** rloo has quit IRC17:39
*** rloo has joined #openstack-ironic17:39
openstackgerritGalyna Zholtkevych proposed openstack/python-ironicclient: [WIP] POC: Full-featured objects with ETAGs supporting  https://review.openstack.org/40033517:41
*** derekh has quit IRC17:44
*** moshele has joined #openstack-ironic17:45
*** trown is now known as trown|lunch17:48
vdrokrloo: just in case you're interested :) -- https://review.openstack.org/412545, I'll upload OSC one tomorrow17:48
patchbotpatch 412545 - python-ironicclient - Add mode and properties to portgroup17:48
rloovdrok: thx!17:49
* vdrok heads home17:49
vdrokgood night everyone!17:49
NobodyCamhave a good night vdrok17:49
lucasagomesvdrok, night17:49
lucasagomesI'm heading too, have a great evening all17:49
lucasagomestalk tomorrow!17:49
jroll\o17:50
NobodyCamnight lucasagomes :)17:50
* jroll grabs lunch17:50
lucasagomesNobodyCam, morning :D17:50
NobodyCam:)17:50
*** lucasagomes is now known as lucas-afk17:50
* TheJulia has a nice warm soup for lunch... considering it is 26F outside right now :(17:50
NobodyCam++++!!!!!!++++17:50
milannight lucas-afk vdrok! :)17:51
*** soliosg has joined #openstack-ironic17:52
meleewho can tell me how to create a whole disk image?17:54
TheJuliamelee: diskimage-builder?17:54
meleecan you give me an example?17:56
TheJuliadisk-image-create -o ubuntu.qcow2 ubuntu vm17:56
meleewhat is the difference with Partition images?17:57
meleedoes the command just create a qcow2 file?17:58
TheJuliahttp://docs.openstack.org/developer/diskimage-builder/user_guide/building_an_image.html17:58
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Catch unknown exceptions in validate driver ifaces  https://review.openstack.org/40615717:58
TheJuliamelee: The difference is that a whole disk image has a boot loader and a partition table.  A partition image is the contents of a single partition, and is generally made bootable by installing a boot loader and partition table which can be problematic sometimes, net booting, or via special preparation to the beginning of the partition17:59
meleethanks, TheJulia!18:02
* TheJulia steps away for a little bit18:02
openstackgerritGalyna Zholtkevych proposed openstack/ironic: ETAGs maintenance for all api node's requests  https://review.openstack.org/39221318:04
*** melee has quit IRC18:09
*** ohamada has quit IRC18:10
jlvillaljroll, vsaienk0, vdrok : Do you think we should get rid of tempest config option 'shared_physical_network' and use 'create_isolated_networks' instead?18:16
jlvillalTo me it is confusing to have both of those options as they contradict each other.18:16
jlvillalBut maybe I'm missing something.18:16
*** athomas has quit IRC18:17
*** pc_m has quit IRC18:37
*** pc_m has joined #openstack-ironic18:39
*** moshele has quit IRC18:41
*** srart has quit IRC18:44
rlooTheJulia: when you have a minute. I added 'ironic-ui' to the subteam reports on our IronicWhiteBoard. Would you do me a favour and add the contact person(s) and let them know about the subteam reports?18:48
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Fix take over of ACTIVE nodes in AgentDeploy  https://review.openstack.org/40866718:49
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Fix take over for ACTIVE nodes in PXEBoot  https://review.openstack.org/40485818:49
TheJuliarloo: sure!18:50
*** srart has joined #openstack-ironic18:54
rlooTheJulia: thx :)18:58
*** baoli has quit IRC18:59
*** baoli has joined #openstack-ironic19:00
*** jkilpatr_ has joined #openstack-ironic19:06
rloojroll: wrt the dbsync online-data-migration cmd, i'm good with it (even though i have a question). https://bugs.launchpad.net/ironic/+bug/158514119:08
openstackLaunchpad bug 1585141 in Ironic "[RFE] Add new ironic-dbsync command online-data-migration" [Wishlist,In progress] - Assigned to Grzegorz Grasza (xek)19:08
*** fragatina has quit IRC19:13
*** david-lyle_ has joined #openstack-ironic19:13
*** Sukhdev has quit IRC19:13
*** openstackstatus has quit IRC19:13
*** jkilpatr has quit IRC19:13
*** david-lyle has quit IRC19:13
*** openstack has joined #openstack-ironic19:15
openstackgerritAnnie Lezil proposed openstack/ironic-python-agent: [WIP]Reboot and Poweroff fails with coreos IPA image  https://review.openstack.org/41259419:20
*** e0ne has joined #openstack-ironic19:23
*** trown|lunch is now known as trown19:25
*** e0ne has quit IRC19:27
*** e0ne has joined #openstack-ironic19:31
*** baoli has quit IRC19:32
*** baoli has joined #openstack-ironic19:32
*** chlong has quit IRC19:34
*** chlong has joined #openstack-ironic19:36
* TheJulia wonders if she should worry about her internet connectivity when Mail.app is hung sending a single email.19:36
openstackgerritMerged openstack/ironic: Fix policy dict checkers  https://review.openstack.org/41176419:40
*** nicodemos has quit IRC19:41
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Serve boot configs from Ironic API  https://review.openstack.org/36943819:44
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Implement Driver API for dynamic boot config  https://review.openstack.org/39439819:44
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Add RPC API to get the boot config  https://review.openstack.org/39439919:44
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Allow not writing pxe configs to disk  https://review.openstack.org/39439719:44
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Convert iPXE boot config to proper Jinja template  https://review.openstack.org/39346219:44
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: DNM test enabled dynamic iPXE  https://review.openstack.org/39194319:45
openstackgerritJohn L. Villalovos proposed openstack/ironic: Configure tempest for multitenancy/flat network  https://review.openstack.org/40637619:46
openstackgerritMerged openstack/ironic: Adds another validation step when using dynamic allocation  https://review.openstack.org/37707319:49
openstackgerritJohn L. Villalovos proposed openstack/ironic: Configure tempest for multitenancy/flat network  https://review.openstack.org/40841719:50
*** jaosorior has quit IRC19:52
*** rcernin has joined #openstack-ironic19:56
*** e0ne has quit IRC19:56
TheJuliajroll: https://review.openstack.org/41260419:58
patchbotpatch 412604 - openstack-infra/project-config - Removing ironic-webclient gating for retirement19:58
*** e0ne has joined #openstack-ironic20:00
jlvillalTheJulia: Your connectivity seems fine. Still seeing your IRC messages ;)20:02
TheJuliaI had to nuke one of my uplinks20:03
jlvillalTheJulia: So where are you at now that it is 26F? Are you still cruising the motorways?20:04
*** charlledaniel has quit IRC20:04
*** rpioso has joined #openstack-ironic20:05
TheJuliajlvillal: We've been parked in new mexico spending time with my father. Rewarding, although... it sure is cold outside. :)20:05
jlvillalTheJulia: And I always thought New Mexico would be hot :)20:06
jlvillalTheJulia: Enjoy!20:06
TheJuliaWell, the day star is hot. :)20:06
*** cdaniel has joined #openstack-ironic20:06
* TheJulia can feel it through the wall :(20:06
rpiosojroll: Hi20:06
jlvillalrpioso: https://blogs.gnome.org/markmc/2014/02/20/naked-pings/  :)20:07
* jlvillal annoys his coworkers with that link :D20:07
jrolljlvillal: I don't think those are necessarily the same thing, you can have a not-shared physical networks but still not wish to create isolated networks, I think? idk. I added the former because the QA folks didn't want to use the latter for everything ¯\_(ツ)_/¯20:07
jrollrloo: I'll look, one moment20:08
jrollTheJulia: ditto20:08
jrollrpioso: hello20:08
TheJuliajroll: do desk fans help at all?20:08
rpiosojroll: I wish to update you on my attempt to get manual cleaning with RAID cleaning steps to work.20:08
jlvillaljroll: Hmm. They seem to conflict with each other. Let me think.20:09
jrollTheJulia: heh, I meant "I'll look, one moment" to you as well :P I like ceiling fans though20:09
*** jkilpatr_ has quit IRC20:09
jlvillalTheJulia: "day star", does that mean the "sun"?20:09
TheJuliajroll: Ahh.... :)20:09
TheJuliajlvillal: That it does :)20:09
jrolljlvillal: dunno, ask tempest cores (see the ironic removal patch)20:09
jlvillal:)20:09
rpiosojroll: After you departed, TheJulia, dtantsur, mgould, and others assisted me.20:10
*** aNuposic has quit IRC20:10
jrollTheJulia: hrm, CI failed on that project-config change, suspect you may need to add noop-jobs in projects.yaml20:11
TheJuliaWeird...20:11
TheJuliaIt passed tox locally20:12
jrollTheJulia: or like, have it exist in projects.yaml somehow20:12
jrolloh odd20:12
TheJuliaerr, no, it didn't20:12
jrollrpioso: yay, we do have a helpful community here20:12
jroll:)20:12
rpiosojroll: TheJulia suggested that I ensure that introspection was run before manual cleaning.  Introspection creates the ports on the provisioning/cleaning network.  Recall that that network had no ports, which caused manual cleaning's prep for inband cleaning to fail.20:13
TheJuliajroll: Looks like there is a orphaned job regex20:14
jrollrpioso: yeah, that makes sense20:14
rpiosojroll: Doing that solved the issue.  The manual out-of-band cleaning with RAID cleaning steps worked.20:14
jrollrloo: attempted to answer your questions the way I see it working20:14
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: iPXE allow skip caching in standalone mode  https://review.openstack.org/36732120:14
rpiosojroll: So, no additonal bug needed to be opened.20:15
jrollrpioso: hooray20:15
jrollrloo: and another comment that I'm +1 on approving that20:16
*** amoralej is now known as amoralej|off20:17
rpiosojroll: Thanks, again, for your assistance with that.20:17
rloojroll: ok. so the contraction script is part of the dbsync migration command.20:17
jrollrpioso: no problem20:17
rpiosojlvillal: Noted.  ty for the pointer.20:17
jrollrloo: yeah, it's just a typical 'alter table' thing20:17
rloojroll: we just need to make sure the messaging is for the user to run the previous migration script/cmd.20:18
rloojroll: so we can approve or should we see if others are ok with it?20:18
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic: Fix bashate warning in devstack plugin  https://review.openstack.org/41260620:18
jrollrloo: wouldn't hurt to get one more, but if not we can approve it20:19
jrollrloo: we can start reviewing that code either way while we wait :P20:19
rloojroll: ok, will wait til eg wed to see if anyone else looks at it.20:19
jrollyeah, sounds good20:19
rloojroll: yeah, i already reviewed it a bit. i think. hard to keep track of them all...20:19
jrollheh, cool20:19
*** jkilpatr_ has joined #openstack-ironic20:22
*** ccamacho has quit IRC20:30
*** e0ne has quit IRC20:30
*** Sukhdev has quit IRC20:33
*** e0ne has joined #openstack-ironic20:34
rloovdrok: wrt https://review.openstack.org/#/c/411764/. i want to make sure there was no security issue. w/o that fix, the user had to be admin, right?20:34
patchbotpatch 411764 - ironic - Fix policy dict checkers (MERGED)20:34
vdrokrloo: correct20:35
rloovdrok: thx (i didn't think you were still around today!)20:36
vdrokjlvillal:  I will take a look at the conf option tomorrow, but yeah, I'd prefer having one option to enable-disable things20:36
vdrokGone now :)20:37
rloobye vdrok :)20:37
*** e0ne has quit IRC20:38
*** ccamacho has joined #openstack-ironic20:46
*** baoli has quit IRC20:48
*** jkilpatr_ has quit IRC20:48
*** jkilpatr has joined #openstack-ironic20:51
jrollTIL we don't use objects for drivers21:04
* jroll debates jumping into this rabbit hole21:04
*** baoli has joined #openstack-ironic21:04
*** godp1301 has quit IRC21:05
*** e0ne has joined #openstack-ironic21:07
*** e0ne has quit IRC21:10
*** e0ne has joined #openstack-ironic21:14
*** ccamacho has quit IRC21:15
*** e0ne has quit IRC21:18
openstackgerritXavier proposed openstack/ironic: Shutdown server before change boot order for agent drivers  https://review.openstack.org/40829821:19
openstackgerritJohn L. Villalovos proposed openstack/ironic: See if stable/newton is broken for grenade  https://review.openstack.org/41261921:24
jlvillaljroll: ^^^21:25
jlvillaljroll: Is http://logs.openstack.org/17/408417/2/check/gate-tempest-dsvm-ironic-inspector-ubuntu-xenial-nv/805d268/console.html#_2016-12-19_20_22_14_58072721:25
jlvillalThe ironic-inspector thing you mentioned in the meeting?21:25
jlvillalOh, never mind that is a non-voting job.21:27
jlvillalBut might be the thing that was mentioned. But not the error I was trying to track down21:28
jrolljlvillal: that is the same problem yes21:29
jrolljlvillal: https://bugs.launchpad.net/ironic-inspector/+bug/1651123 is what I mentioned21:30
openstackLaunchpad bug 1651123 in Ironic Inspector "Tempest hosed because of removed test tags" [Critical,In progress] - Assigned to milan k (vetrisko)21:30
jlvillaljroll: Thanks. I'm looking at stable/newton grenade job failure. Not sure if it is my patch or something else.21:30
jrolljlvillal: oh, we might need to backport a couple things to mitaka, sigh21:30
jlvillaljroll: FYI: http://logs.openstack.org/17/408417/2/check/gate-grenade-dsvm-ironic-ubuntu-trusty/b180336/logs/  But I haven't dug in yet.21:31
jlvillalStarting to look now21:31
jrollhttps://review.openstack.org/#/c/397824/ https://review.openstack.org/#/c/403615/ https://review.openstack.org/#/q/status:merged+project:openstack/ironic+branch:stable/newton+topic:tempest-thing-backports21:32
patchbotpatch 397824 - ironic (stable/newton) - Bring configurations from tempest to ironic_tempes... (MERGED)21:32
patchbotpatch 403615 - ironic (stable/newton) - Drop bad skip check in tempest plugin (MERGED)21:32
jrollprobably all need to be backported21:32
jlvillaljroll: Ah, okay. I can work on testing those out today. Unless you want to.21:32
jlvillalI'm happy to do it.21:32
jlvillalBlocking my stuff :)21:32
jrolljlvillal: that would be great, I'm trying to work on driver composition a bit21:33
jlvillaljroll: Sounds like a plan.21:33
jrollthanks21:35
openstackgerritJohn L. Villalovos proposed openstack/ironic: Turn off tempest's multitenant network tests  https://review.openstack.org/41262121:37
openstackgerritJohn L. Villalovos proposed openstack/ironic: Move CONF.service_available.ironic to our plugin  https://review.openstack.org/41262221:45
*** jcoufal has quit IRC21:47
*** fragatina has joined #openstack-ironic21:48
mrdaMorning Ironic21:49
jlvillalmrda: You can just shorten that to "Moronic" ;)21:50
jlvillalmrda: Okay don't do that, but I found it mildly amusing21:51
mrdajlvillal: And I thought you liked me :)21:51
jlvillalmrda: I do! :D21:51
mrda:)21:52
*** fragatina has quit IRC21:54
*** fragatina has joined #openstack-ironic21:55
*** fragatina has quit IRC21:58
*** fragatina has joined #openstack-ironic21:59
*** Goneri has quit IRC21:59
*** trown is now known as trown|outtypewww21:59
*** dprince has quit IRC22:02
*** milan has quit IRC22:05
*** fragatina has quit IRC22:06
*** aNuposic has joined #openstack-ironic22:07
*** Sukhdev has joined #openstack-ironic22:10
openstackgerritJim Rollenhagen proposed openstack/ironic: WIP: Register/unregister hardware interfaces for conductors  https://review.openstack.org/41263122:15
*** lindycoder has quit IRC22:19
openstackgerritAnnie Lezil proposed openstack/ironic-python-agent: [WIP]Reboot and Poweroff fails with coreos IPA image  https://review.openstack.org/41259422:26
openstackgerritJohn L. Villalovos proposed openstack/ironic: See if stable/newton is broken for grenade  https://review.openstack.org/41261922:27
*** baoli has quit IRC22:29
*** jkilpatr has quit IRC22:36
openstackgerritJim Rollenhagen proposed openstack/ironic: [WIP] Create a table to track loaded interfaces  https://review.openstack.org/40981222:38
openstackgerritJim Rollenhagen proposed openstack/ironic: WIP: Register/unregister hardware interfaces for conductors  https://review.openstack.org/41263122:38
*** jheroux has quit IRC22:39
jrollblahhhhhh22:39
* jroll off for the night, see you all tomorrow22:39
*** cdearborn has quit IRC22:46
openstackgerritkobig proposed openstack/ironic-specs: Code review changes  https://review.openstack.org/41263922:51
*** hjensas has joined #openstack-ironic22:55
*** baoli has joined #openstack-ironic22:56
*** baoli has quit IRC23:01
*** bfournie has quit IRC23:03
*** jkilpatr has joined #openstack-ironic23:16
*** aNuposic has quit IRC23:32
*** Sukhdev has quit IRC23:34
*** ElCoyote_1 has left #openstack-ironic23:42
*** ElCoyote_ has joined #openstack-ironic23:45
*** bfournie has joined #openstack-ironic23:47
*** Sukhdev has joined #openstack-ironic23:59

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