*** vsaienko has quit IRC | 00:02 | |
mrda | mariojv: JayF: Question answered on review 382108 | 00:05 |
---|---|---|
*** vsaienko has joined #openstack-ironic | 00:05 | |
JayF | ty for the reference | 00:06 |
JayF | <3 references | 00:06 |
mrda | \o/ | 00:06 |
*** davidlenwell_ has quit IRC | 00:07 | |
*** sdake_ has joined #openstack-ironic | 00:11 | |
openstackgerrit | Merged openstack/python-ironicclient: Move duplicated info to new v1/utils.py https://review.openstack.org/364038 | 00:11 |
*** vsaienko has quit IRC | 00:11 | |
openstackgerrit | Merged openstack/python-ironicclient: If no resource, don't call Resource.to_dict() https://review.openstack.org/379567 | 00:12 |
openstackgerrit | Merged openstack/python-ironicclient: Add docs for create command https://review.openstack.org/358025 | 00:12 |
*** sdake has quit IRC | 00:14 | |
openstackgerrit | Merged openstack/ironic: Remove unused argument in Tempest Plugin https://review.openstack.org/370094 | 00:15 |
openstackgerrit | Merged openstack/ironic: Update ironic config docs for keystone v3 https://review.openstack.org/382108 | 00:16 |
*** aNuposic has quit IRC | 00:16 | |
*** hj-hpe has quit IRC | 00:16 | |
*** vsaienko has joined #openstack-ironic | 00:16 | |
*** vsaienko has quit IRC | 00:24 | |
*** bfournie has joined #openstack-ironic | 00:28 | |
*** hoangcx has joined #openstack-ironic | 00:56 | |
*** phuongnh has joined #openstack-ironic | 00:59 | |
*** Sukhdev has quit IRC | 01:00 | |
openstackgerrit | gecong proposed openstack/ironic-inspector: Fix a typo in base.py https://review.openstack.org/385705 | 01:06 |
*** xuao has joined #openstack-ironic | 01:08 | |
*** zackf has joined #openstack-ironic | 01:09 | |
*** zackf has quit IRC | 01:14 | |
*** tuanluong has joined #openstack-ironic | 01:19 | |
openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Support i18n for baremetal chassis cmds https://review.openstack.org/375972 | 01:27 |
openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Support i18n for baremetal port cmds https://review.openstack.org/375971 | 01:27 |
openstackgerrit | Tang Chen proposed openstack/python-ironicclient: Support i18n for baremetal node cmds https://review.openstack.org/357639 | 01:27 |
*** thrash is now known as thrash|g0ne | 01:42 | |
openstackgerrit | Xu Ao proposed openstack/ironic: Make the configuration item "api_url" have better adaptability https://review.openstack.org/385261 | 01:47 |
*** phuongnh has quit IRC | 02:02 | |
*** phuongnh has joined #openstack-ironic | 02:02 | |
*** joprovost has quit IRC | 02:03 | |
*** mtanino has quit IRC | 02:12 | |
*** rloo has quit IRC | 02:16 | |
*** Nakato has quit IRC | 02:32 | |
*** zhangyufei has joined #openstack-ironic | 02:33 | |
*** Nakato has joined #openstack-ironic | 02:33 | |
*** gcb has quit IRC | 02:39 | |
*** yuanying_ has joined #openstack-ironic | 02:49 | |
*** yuanying has quit IRC | 02:49 | |
*** gcb has joined #openstack-ironic | 02:51 | |
*** MattMan has quit IRC | 02:53 | |
*** MattMan has joined #openstack-ironic | 02:53 | |
*** yuanying_ has quit IRC | 02:59 | |
*** stendulker has joined #openstack-ironic | 03:01 | |
*** yuanying has joined #openstack-ironic | 03:06 | |
*** yuanying has quit IRC | 03:08 | |
openstackgerrit | gecong proposed openstack/ironic: Fix some typo https://review.openstack.org/385720 | 03:13 |
*** baoli has quit IRC | 03:14 | |
openstackgerrit | gecong proposed openstack/ironic: Fix some typo https://review.openstack.org/385720 | 03:15 |
*** sabeen has quit IRC | 03:16 | |
*** sabeen has joined #openstack-ironic | 03:19 | |
*** zackf has joined #openstack-ironic | 03:20 | |
*** phuongnh has quit IRC | 03:23 | |
*** phuongnh has joined #openstack-ironic | 03:24 | |
*** absubram has quit IRC | 03:27 | |
*** mtanino has joined #openstack-ironic | 03:31 | |
*** syed_ has quit IRC | 03:34 | |
*** vikrant has joined #openstack-ironic | 03:36 | |
*** absubram has joined #openstack-ironic | 03:38 | |
openstackgerrit | gecong proposed openstack/ironic: Fix some typos https://review.openstack.org/385720 | 03:44 |
*** links has joined #openstack-ironic | 03:45 | |
*** yuanying has joined #openstack-ironic | 03:47 | |
*** mtanino has quit IRC | 03:48 | |
*** jaosorior has joined #openstack-ironic | 03:51 | |
*** zackf has quit IRC | 03:59 | |
*** zackf has joined #openstack-ironic | 04:05 | |
*** zackf has quit IRC | 04:10 | |
*** rama_y has joined #openstack-ironic | 04:12 | |
*** jaosorior has quit IRC | 04:21 | |
*** jaosorior has joined #openstack-ironic | 04:21 | |
*** rjrooke has joined #openstack-ironic | 04:50 | |
*** rjrooke has quit IRC | 04:58 | |
*** yuanying has quit IRC | 05:00 | |
*** rbartal has joined #openstack-ironic | 05:07 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/385180 | 05:21 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/385746 | 05:27 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/385747 | 05:27 |
*** ChubYann has quit IRC | 05:34 | |
*** rama_y has quit IRC | 05:41 | |
*** e0ne has joined #openstack-ironic | 05:46 | |
*** e0ne has quit IRC | 05:47 | |
*** e0ne has joined #openstack-ironic | 05:52 | |
*** vmud213 has joined #openstack-ironic | 05:56 | |
*** yuanying has joined #openstack-ironic | 06:10 | |
*** vsaienko has joined #openstack-ironic | 06:12 | |
*** Sukhdev has joined #openstack-ironic | 06:15 | |
*** vishwanathj is now known as vishwanathj_zzz | 06:16 | |
*** mjura has joined #openstack-ironic | 06:21 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic-specs: fix a typo https://review.openstack.org/385780 | 06:22 |
*** yuanying has quit IRC | 06:23 | |
*** sabeen has quit IRC | 06:23 | |
*** mjura has quit IRC | 06:27 | |
*** vsaienko has quit IRC | 06:28 | |
*** mjura has joined #openstack-ironic | 06:28 | |
*** yuanying has joined #openstack-ironic | 06:29 | |
*** yuanying_ has joined #openstack-ironic | 06:37 | |
openstackgerrit | haris proposed openstack/python-ironicclient: Enables ironic list commands to use labels https://review.openstack.org/380671 | 06:38 |
*** yuanying has quit IRC | 06:39 | |
*** e0ne has quit IRC | 06:45 | |
*** e0ne has joined #openstack-ironic | 06:49 | |
*** e0ne has quit IRC | 06:50 | |
*** e0ne has joined #openstack-ironic | 06:53 | |
*** e0ne has quit IRC | 06:54 | |
*** rcernin has joined #openstack-ironic | 06:56 | |
*** Sukhdev has quit IRC | 07:01 | |
*** absubram has quit IRC | 07:03 | |
*** absubram has joined #openstack-ironic | 07:03 | |
*** tesseract- has joined #openstack-ironic | 07:04 | |
*** crinkle has quit IRC | 07:18 | |
*** crinkle has joined #openstack-ironic | 07:19 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 07:26 |
*** jpich has joined #openstack-ironic | 07:30 | |
*** ccamacho has joined #openstack-ironic | 07:31 | |
*** ohamada has joined #openstack-ironic | 07:35 | |
*** yuanying_ has quit IRC | 07:36 | |
*** yuanying has joined #openstack-ironic | 07:36 | |
*** watanabe_isao has joined #openstack-ironic | 07:39 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add tests for cleaning https://review.openstack.org/304469 | 07:58 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 07:58 |
*** hoangcx has quit IRC | 07:59 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-ironic | 08:00 | |
*** heyongli has joined #openstack-ironic | 08:07 | |
*** watanabe_isao has quit IRC | 08:09 | |
*** dtantsur|afk is now known as dtantsur | 08:12 | |
dtantsur | Morning Ironic | 08:12 |
*** Vinsh has quit IRC | 08:17 | |
tuanluong | morning ironic | 08:18 |
*** vsaienko has joined #openstack-ironic | 08:21 | |
*** e0ne has joined #openstack-ironic | 08:23 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Fix stable states validation in FSM https://review.openstack.org/385459 | 08:26 |
*** Vinsh has joined #openstack-ironic | 08:28 | |
*** Romanenko_K has joined #openstack-ironic | 08:32 | |
openstackgerrit | Merged openstack/ironic: Remove wrong check from conductor periodic task https://review.openstack.org/384899 | 08:33 |
*** athomas has joined #openstack-ironic | 08:38 | |
*** vsaienko has quit IRC | 08:42 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui: Imported Translations from Zanata https://review.openstack.org/385832 | 08:47 |
vdrok | good morning ironic, dtantsur tuanluong | 08:51 |
tuanluong | morning vdrok | 08:52 |
*** milan has joined #openstack-ironic | 08:53 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Fix setting persistent boot device does not work https://review.openstack.org/376322 | 08:53 |
milan | morning Ironic! :) | 08:53 |
*** kragniz has quit IRC | 08:53 | |
dtantsur | hey milan | 08:53 |
*** kragniz has joined #openstack-ironic | 08:54 | |
vdrok | morning milan :) | 08:54 |
milan | dtantsur, hi :) mind a review? ;) | 08:54 |
milan | morning vdrok! :) | 08:54 |
dtantsur | -2 :) | 08:54 |
dtantsur | morning vdrok | 08:54 |
milan | dtantsur, :P | 08:54 |
dtantsur | milan, I'm one call away from getting to reviewing a few essentially patches, yours including. stay tuned | 08:54 |
milan | dtantsur, nice thanks! :) | 08:55 |
milan | aarefiev, morning! :) | 08:55 |
*** makowals has joined #openstack-ironic | 08:56 | |
*** yuanying has quit IRC | 08:57 | |
*** makowals_ has quit IRC | 08:58 | |
openstackgerrit | Merged openstack/ironic: Support multi arch deployment https://review.openstack.org/322632 | 08:58 |
*** derekh has joined #openstack-ironic | 09:01 | |
sambetts|afk | Morning all | 09:14 |
*** sambetts|afk is now known as sambetts | 09:14 | |
*** jaosorior is now known as jaosorior_lunch | 09:16 | |
*** heyongli has quit IRC | 09:16 | |
*** anush_ has quit IRC | 09:17 | |
*** mjturek has quit IRC | 09:17 | |
*** clif_h has quit IRC | 09:17 | |
*** jmccrory has quit IRC | 09:17 | |
*** adreznec has quit IRC | 09:17 | |
*** dtantsur has quit IRC | 09:17 | |
*** kragniz has quit IRC | 09:17 | |
*** zzzeek has quit IRC | 09:17 | |
*** zhangyufei has quit IRC | 09:17 | |
*** portdirect has quit IRC | 09:17 | |
*** derekh has quit IRC | 09:17 | |
*** ccamacho has quit IRC | 09:17 | |
*** crinkle has quit IRC | 09:17 | |
*** vmud213 has quit IRC | 09:17 | |
*** MattMan has quit IRC | 09:17 | |
*** sdake_ has quit IRC | 09:17 | |
*** jtomasek has quit IRC | 09:17 | |
*** skazi has quit IRC | 09:17 | |
*** ricardoas has quit IRC | 09:17 | |
*** openstackgerrit has quit IRC | 09:17 | |
*** vgadiraj has quit IRC | 09:17 | |
*** larsks has quit IRC | 09:17 | |
*** HenryG has quit IRC | 09:17 | |
*** cinerama has quit IRC | 09:17 | |
*** patchbot has quit IRC | 09:17 | |
*** mat128 has quit IRC | 09:17 | |
*** Nakato has quit IRC | 09:17 | |
*** xuao has quit IRC | 09:17 | |
*** bfournie has quit IRC | 09:17 | |
*** gabriel-bezerra has quit IRC | 09:17 | |
*** mgagne_ has quit IRC | 09:17 | |
*** zigo has quit IRC | 09:17 | |
*** rm_work has quit IRC | 09:17 | |
*** lekha has quit IRC | 09:17 | |
*** adam_g` has quit IRC | 09:17 | |
*** vishwanathj_zzz has quit IRC | 09:17 | |
*** amit213 has quit IRC | 09:17 | |
*** early has quit IRC | 09:17 | |
*** AlexeyAbashkin has quit IRC | 09:17 | |
*** robcresswell has quit IRC | 09:17 | |
*** harlowja has quit IRC | 09:17 | |
*** bnemec has quit IRC | 09:17 | |
*** strigazi has quit IRC | 09:17 | |
*** xek_ has quit IRC | 09:17 | |
*** zhenguo has quit IRC | 09:17 | |
*** kozhukalov has quit IRC | 09:17 | |
*** agordeev has quit IRC | 09:17 | |
*** ildikov has quit IRC | 09:17 | |
*** mariojv has quit IRC | 09:17 | |
*** htruta has quit IRC | 09:17 | |
*** UForgotten has quit IRC | 09:17 | |
*** milan has quit IRC | 09:17 | |
*** e0ne has quit IRC | 09:17 | |
*** rcernin has quit IRC | 09:17 | |
*** jaosorior_lunch has quit IRC | 09:17 | |
*** bcornec1 has quit IRC | 09:17 | |
*** odyssey4me has quit IRC | 09:17 | |
*** natorious has quit IRC | 09:17 | |
*** alex_xu_ has quit IRC | 09:17 | |
*** octopusZ1ang has quit IRC | 09:17 | |
*** jistr has quit IRC | 09:17 | |
*** fungi has quit IRC | 09:17 | |
*** tdurakov has quit IRC | 09:17 | |
*** mrtenio has quit IRC | 09:17 | |
*** evgenyl has quit IRC | 09:17 | |
*** d0ugal has quit IRC | 09:17 | |
*** jcook has quit IRC | 09:17 | |
*** tangchen has quit IRC | 09:17 | |
*** marlinc has quit IRC | 09:17 | |
*** sergek has quit IRC | 09:17 | |
*** romcheg has quit IRC | 09:17 | |
*** serverascode has quit IRC | 09:17 | |
*** fyxim has quit IRC | 09:17 | |
*** cppforlife_ has quit IRC | 09:17 | |
*** _fortis has quit IRC | 09:17 | |
*** dstepanenko has quit IRC | 09:17 | |
*** rbartal has quit IRC | 09:17 | |
*** gcb has quit IRC | 09:17 | |
*** phschwartz has quit IRC | 09:17 | |
*** ionutbalutoiu has quit IRC | 09:17 | |
*** Guest66666 has quit IRC | 09:17 | |
*** EmilienM has quit IRC | 09:17 | |
*** Madasi has quit IRC | 09:17 | |
*** SpamapS has quit IRC | 09:17 | |
*** kencjohnston has quit IRC | 09:17 | |
*** wznoinsk has quit IRC | 09:17 | |
*** zer0c00l has quit IRC | 09:17 | |
*** NobodyCam has quit IRC | 09:17 | |
*** boris-42 has quit IRC | 09:17 | |
*** patrickeast has quit IRC | 09:17 | |
*** mgkwill has quit IRC | 09:17 | |
*** kun_huang has quit IRC | 09:17 | |
*** betherly has quit IRC | 09:17 | |
*** dims has quit IRC | 09:17 | |
*** greghaynes has quit IRC | 09:17 | |
*** Ng has quit IRC | 09:17 | |
*** TheJulia has quit IRC | 09:17 | |
*** jroll has quit IRC | 09:17 | |
*** johnthetubaguy has quit IRC | 09:17 | |
*** tonyb has quit IRC | 09:17 | |
*** jhesketh has quit IRC | 09:17 | |
vdrok | good morning sambetts | 09:18 |
*** jhesketh has joined #openstack-ironic | 09:19 | |
*** jpich has quit IRC | 09:19 | |
*** absubram has quit IRC | 09:20 | |
*** jpich has joined #openstack-ironic | 09:20 | |
sambetts | hey vdrok | 09:20 |
*** yuanying has joined #openstack-ironic | 09:22 | |
*** igordcard has quit IRC | 09:22 | |
*** igordcard has joined #openstack-ironic | 09:22 | |
*** evgenyl has joined #openstack-ironic | 09:39 | |
*** zackf has joined #openstack-ironic | 09:43 | |
*** zackf has quit IRC | 09:48 | |
*** yuanying has quit IRC | 09:49 | |
*** kragniz has joined #openstack-ironic | 10:00 | |
*** amotoki has quit IRC | 10:01 | |
*** derekh has joined #openstack-ironic | 10:06 | |
*** zzzeek has joined #openstack-ironic | 10:06 | |
*** zhangyufei has joined #openstack-ironic | 10:06 | |
*** portdirect has joined #openstack-ironic | 10:06 | |
*** anush_ has joined #openstack-ironic | 10:07 | |
*** mjturek has joined #openstack-ironic | 10:07 | |
*** serverascode has joined #openstack-ironic | 10:07 | |
*** rbartal has joined #openstack-ironic | 10:07 | |
*** gcb has joined #openstack-ironic | 10:07 | |
*** phschwartz has joined #openstack-ironic | 10:07 | |
*** ionutbalutoiu has joined #openstack-ironic | 10:07 | |
*** Guest66666 has joined #openstack-ironic | 10:07 | |
*** Madasi has joined #openstack-ironic | 10:07 | |
*** EmilienM has joined #openstack-ironic | 10:07 | |
*** wznoinsk has joined #openstack-ironic | 10:07 | |
*** SpamapS has joined #openstack-ironic | 10:07 | |
*** kencjohnston has joined #openstack-ironic | 10:07 | |
*** zer0c00l has joined #openstack-ironic | 10:07 | |
*** NobodyCam has joined #openstack-ironic | 10:07 | |
*** boris-42 has joined #openstack-ironic | 10:07 | |
*** patrickeast has joined #openstack-ironic | 10:07 | |
*** mgkwill has joined #openstack-ironic | 10:07 | |
*** kun_huang has joined #openstack-ironic | 10:07 | |
*** betherly has joined #openstack-ironic | 10:07 | |
*** greghaynes has joined #openstack-ironic | 10:07 | |
*** dims has joined #openstack-ironic | 10:07 | |
*** Ng has joined #openstack-ironic | 10:07 | |
*** TheJulia has joined #openstack-ironic | 10:07 | |
*** johnthetubaguy has joined #openstack-ironic | 10:07 | |
*** jroll has joined #openstack-ironic | 10:07 | |
*** tonyb has joined #openstack-ironic | 10:07 | |
*** tepper.freenode.net sets mode: +o jroll | 10:07 | |
*** heyongli has joined #openstack-ironic | 10:09 | |
*** jaosorior has joined #openstack-ironic | 10:09 | |
*** cppforlife_ has joined #openstack-ironic | 10:09 | |
*** milan has joined #openstack-ironic | 10:09 | |
*** e0ne has joined #openstack-ironic | 10:09 | |
*** rcernin has joined #openstack-ironic | 10:09 | |
*** bcornec1 has joined #openstack-ironic | 10:09 | |
*** odyssey4me has joined #openstack-ironic | 10:09 | |
*** natorious has joined #openstack-ironic | 10:09 | |
*** alex_xu_ has joined #openstack-ironic | 10:09 | |
*** octopusZ1ang has joined #openstack-ironic | 10:09 | |
*** jistr has joined #openstack-ironic | 10:09 | |
*** fungi has joined #openstack-ironic | 10:09 | |
*** tdurakov has joined #openstack-ironic | 10:09 | |
*** mrtenio has joined #openstack-ironic | 10:09 | |
*** d0ugal has joined #openstack-ironic | 10:09 | |
*** jcook has joined #openstack-ironic | 10:09 | |
*** tangchen has joined #openstack-ironic | 10:09 | |
*** marlinc has joined #openstack-ironic | 10:09 | |
*** sergek has joined #openstack-ironic | 10:09 | |
*** romcheg has joined #openstack-ironic | 10:09 | |
*** fyxim has joined #openstack-ironic | 10:09 | |
*** _fortis has joined #openstack-ironic | 10:09 | |
*** dstepanenko has joined #openstack-ironic | 10:09 | |
*** clif_h has joined #openstack-ironic | 10:10 | |
*** jmccrory has joined #openstack-ironic | 10:10 | |
*** adreznec has joined #openstack-ironic | 10:10 | |
*** dtantsur has joined #openstack-ironic | 10:10 | |
*** clif_h has quit IRC | 10:10 | |
*** jmccrory has quit IRC | 10:10 | |
*** adreznec has quit IRC | 10:10 | |
*** dtantsur has quit IRC | 10:10 | |
*** jaosorior has quit IRC | 10:10 | |
*** cppforlife_ has quit IRC | 10:10 | |
*** milan has quit IRC | 10:10 | |
*** e0ne has quit IRC | 10:10 | |
*** rcernin has quit IRC | 10:10 | |
*** bcornec1 has quit IRC | 10:10 | |
*** odyssey4me has quit IRC | 10:10 | |
*** natorious has quit IRC | 10:10 | |
*** alex_xu_ has quit IRC | 10:10 | |
*** octopusZ1ang has quit IRC | 10:10 | |
*** jistr has quit IRC | 10:10 | |
*** fungi has quit IRC | 10:10 | |
*** tdurakov has quit IRC | 10:10 | |
*** mrtenio has quit IRC | 10:10 | |
*** d0ugal has quit IRC | 10:10 | |
*** jcook has quit IRC | 10:10 | |
*** tangchen has quit IRC | 10:10 | |
*** marlinc has quit IRC | 10:10 | |
*** sergek has quit IRC | 10:10 | |
*** romcheg has quit IRC | 10:10 | |
*** fyxim has quit IRC | 10:10 | |
*** _fortis has quit IRC | 10:10 | |
*** dstepanenko has quit IRC | 10:10 | |
*** heyongli has quit IRC | 10:10 | |
*** serverascode has quit IRC | 10:10 | |
*** rbartal has quit IRC | 10:10 | |
*** gcb has quit IRC | 10:10 | |
*** phschwartz has quit IRC | 10:10 | |
*** ionutbalutoiu has quit IRC | 10:10 | |
*** Guest66666 has quit IRC | 10:10 | |
*** EmilienM has quit IRC | 10:10 | |
*** SpamapS has quit IRC | 10:10 | |
*** Madasi has quit IRC | 10:10 | |
*** wznoinsk has quit IRC | 10:10 | |
*** zer0c00l has quit IRC | 10:10 | |
*** kencjohnston has quit IRC | 10:10 | |
*** NobodyCam has quit IRC | 10:10 | |
*** boris-42 has quit IRC | 10:10 | |
*** patrickeast has quit IRC | 10:10 | |
*** mgkwill has quit IRC | 10:10 | |
*** kun_huang has quit IRC | 10:10 | |
*** betherly has quit IRC | 10:10 | |
*** greghaynes has quit IRC | 10:10 | |
*** dims has quit IRC | 10:10 | |
*** Ng has quit IRC | 10:10 | |
*** TheJulia has quit IRC | 10:10 | |
*** jroll has quit IRC | 10:10 | |
*** johnthetubaguy has quit IRC | 10:10 | |
*** tonyb has quit IRC | 10:10 | |
*** anush_ has quit IRC | 10:10 | |
*** mjturek has quit IRC | 10:10 | |
*** zzzeek has quit IRC | 10:10 | |
*** zhangyufei has quit IRC | 10:10 | |
*** portdirect has quit IRC | 10:10 | |
*** derekh has quit IRC | 10:10 | |
pas-ha | morning Ironic | 10:16 |
*** gzholtkevych has joined #openstack-ironic | 10:19 | |
*** tangchen has joined #openstack-ironic | 10:20 | |
*** heyongli has joined #openstack-ironic | 10:26 | |
*** derekh has joined #openstack-ironic | 10:26 | |
*** fyxim has joined #openstack-ironic | 10:26 | |
*** serverascode has joined #openstack-ironic | 10:26 | |
*** rbartal has joined #openstack-ironic | 10:26 | |
*** gcb has joined #openstack-ironic | 10:26 | |
*** phschwartz has joined #openstack-ironic | 10:26 | |
*** ionutbalutoiu has joined #openstack-ironic | 10:26 | |
*** Guest66666 has joined #openstack-ironic | 10:26 | |
*** Madasi has joined #openstack-ironic | 10:26 | |
*** EmilienM has joined #openstack-ironic | 10:26 | |
*** wznoinsk has joined #openstack-ironic | 10:26 | |
*** SpamapS has joined #openstack-ironic | 10:26 | |
*** kencjohnston has joined #openstack-ironic | 10:26 | |
*** zer0c00l has joined #openstack-ironic | 10:26 | |
*** NobodyCam has joined #openstack-ironic | 10:26 | |
*** boris-42 has joined #openstack-ironic | 10:26 | |
*** patrickeast has joined #openstack-ironic | 10:26 | |
*** mgkwill has joined #openstack-ironic | 10:26 | |
*** kun_huang has joined #openstack-ironic | 10:26 | |
*** betherly has joined #openstack-ironic | 10:26 | |
*** greghaynes has joined #openstack-ironic | 10:26 | |
*** dims has joined #openstack-ironic | 10:26 | |
*** Ng has joined #openstack-ironic | 10:26 | |
*** TheJulia has joined #openstack-ironic | 10:26 | |
*** johnthetubaguy has joined #openstack-ironic | 10:26 | |
*** jroll has joined #openstack-ironic | 10:26 | |
*** tonyb has joined #openstack-ironic | 10:26 | |
*** tepper.freenode.net sets mode: +o jroll | 10:26 | |
*** clif_h has joined #openstack-ironic | 10:27 | |
*** jmccrory has joined #openstack-ironic | 10:27 | |
*** adreznec has joined #openstack-ironic | 10:27 | |
*** dtantsur has joined #openstack-ironic | 10:27 | |
*** zzzeek has joined #openstack-ironic | 10:27 | |
*** portdirect has joined #openstack-ironic | 10:27 | |
*** 18WAAFSW8 has joined #openstack-ironic | 10:27 | |
*** jaosorior has joined #openstack-ironic | 10:27 | |
*** cppforlife_ has joined #openstack-ironic | 10:27 | |
*** milan has joined #openstack-ironic | 10:27 | |
*** e0ne has joined #openstack-ironic | 10:27 | |
*** rcernin has joined #openstack-ironic | 10:27 | |
*** bcornec1 has joined #openstack-ironic | 10:27 | |
*** odyssey4me has joined #openstack-ironic | 10:27 | |
*** natorious has joined #openstack-ironic | 10:27 | |
*** alex_xu_ has joined #openstack-ironic | 10:27 | |
*** jistr has joined #openstack-ironic | 10:27 | |
*** fungi has joined #openstack-ironic | 10:27 | |
*** tdurakov has joined #openstack-ironic | 10:27 | |
*** mrtenio has joined #openstack-ironic | 10:27 | |
*** d0ugal has joined #openstack-ironic | 10:27 | |
*** jcook has joined #openstack-ironic | 10:27 | |
*** marlinc has joined #openstack-ironic | 10:27 | |
*** sergek has joined #openstack-ironic | 10:27 | |
*** romcheg has joined #openstack-ironic | 10:27 | |
*** _fortis has joined #openstack-ironic | 10:27 | |
*** dstepanenko has joined #openstack-ironic | 10:27 | |
*** octopusZhang has joined #openstack-ironic | 10:27 | |
*** ccamacho has joined #openstack-ironic | 10:28 | |
*** crinkle has joined #openstack-ironic | 10:28 | |
*** vmud213 has joined #openstack-ironic | 10:28 | |
*** MattMan has joined #openstack-ironic | 10:28 | |
*** sdake_ has joined #openstack-ironic | 10:28 | |
*** jtomasek has joined #openstack-ironic | 10:28 | |
*** skazi has joined #openstack-ironic | 10:28 | |
*** ricardoas has joined #openstack-ironic | 10:28 | |
*** openstackgerrit has joined #openstack-ironic | 10:28 | |
*** vgadiraj has joined #openstack-ironic | 10:28 | |
*** larsks has joined #openstack-ironic | 10:28 | |
*** HenryG has joined #openstack-ironic | 10:28 | |
*** cinerama has joined #openstack-ironic | 10:28 | |
*** patchbot has joined #openstack-ironic | 10:28 | |
*** mat128 has joined #openstack-ironic | 10:28 | |
*** ralonsoh has joined #openstack-ironic | 10:28 | |
*** Nakato has joined #openstack-ironic | 10:28 | |
*** bfournie has joined #openstack-ironic | 10:28 | |
*** gabriel-bezerra has joined #openstack-ironic | 10:28 | |
*** mgagne_ has joined #openstack-ironic | 10:28 | |
*** zigo has joined #openstack-ironic | 10:28 | |
*** rm_work has joined #openstack-ironic | 10:28 | |
*** lekha has joined #openstack-ironic | 10:28 | |
*** adam_g` has joined #openstack-ironic | 10:28 | |
*** vishwanathj_zzz has joined #openstack-ironic | 10:28 | |
*** amit213 has joined #openstack-ironic | 10:28 | |
*** early has joined #openstack-ironic | 10:28 | |
*** AlexeyAbashkin has joined #openstack-ironic | 10:28 | |
*** robcresswell has joined #openstack-ironic | 10:28 | |
*** bnemec has joined #openstack-ironic | 10:28 | |
*** strigazi has joined #openstack-ironic | 10:28 | |
*** xek_ has joined #openstack-ironic | 10:28 | |
*** zhenguo has joined #openstack-ironic | 10:28 | |
*** kozhukalov has joined #openstack-ironic | 10:28 | |
*** agordeev has joined #openstack-ironic | 10:28 | |
*** ildikov has joined #openstack-ironic | 10:28 | |
*** mariojv has joined #openstack-ironic | 10:28 | |
*** htruta has joined #openstack-ironic | 10:28 | |
*** UForgotten has joined #openstack-ironic | 10:28 | |
*** anush_ has joined #openstack-ironic | 10:28 | |
*** mjturek has joined #openstack-ironic | 10:28 | |
*** marlinc has quit IRC | 10:28 | |
*** NobodyCam has quit IRC | 10:29 | |
*** serverascode has quit IRC | 10:29 | |
*** octopusZhang has quit IRC | 10:30 | |
*** zhangyufei_ has joined #openstack-ironic | 10:30 | |
*** 18WAAFSW8 has quit IRC | 10:30 | |
*** tdurakov has quit IRC | 10:31 | |
*** kozhukalov has quit IRC | 10:32 | |
*** fyxim is now known as Guest73033 | 10:32 | |
*** marlinc has joined #openstack-ironic | 10:34 | |
*** sdake_ has quit IRC | 10:35 | |
*** amotoki has joined #openstack-ironic | 10:35 | |
*** evgenyl has quit IRC | 10:35 | |
*** evgenyl has joined #openstack-ironic | 10:35 | |
*** gzholtkevych has quit IRC | 10:35 | |
*** gzholtkevych has joined #openstack-ironic | 10:35 | |
*** Guest73033 has quit IRC | 10:35 | |
*** Guest73033 has joined #openstack-ironic | 10:35 | |
*** cppforlife_ has quit IRC | 10:36 | |
*** cppforlife_ has joined #openstack-ironic | 10:36 | |
*** links has quit IRC | 10:36 | |
*** NobodyCam has joined #openstack-ironic | 10:38 | |
dtantsur | g'morning to whoever I missed during netsplits | 10:41 |
*** jaosorior has quit IRC | 10:44 | |
*** jaosorior has joined #openstack-ironic | 10:44 | |
*** yuanying has joined #openstack-ironic | 10:45 | |
*** gzholtkevych has quit IRC | 10:45 | |
*** gzholtkevych has joined #openstack-ironic | 10:45 | |
sambetts | hey dtantsur | 10:46 |
milan | morning sambetts! :) | 10:46 |
sambetts | Hi milan! | 10:46 |
*** kozhukalov has joined #openstack-ironic | 10:47 | |
dtantsur | so, judging by the time it takes to clean it, my Dell PowerEdge R320 doesn't support ATA secure erase >_< | 10:48 |
sambetts | :( | 10:48 |
gzholtkevych | hello, ironicers! I have a question related to virt ironic driver in nova. https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L755 | 10:48 |
sambetts | gzholtkevych: hi | 10:48 |
gzholtkevych | should it be fixed for ironic case when the ipa is working on node or it doest not make sense? | 10:48 |
gzholtkevych | hi, sambetts! | 10:49 |
gzholtkevych | I am asking about this, because do not know, how nova's instance settings are working for baremetal nodes. | 10:49 |
*** tdurakov has joined #openstack-ironic | 10:50 | |
*** links has joined #openstack-ironic | 10:50 | |
gzholtkevych | for example, this specific case as the partition for ephemeral drive.if it does not touch ironic behaviour, this will be another problem | 10:50 |
gzholtkevych | thank you in advance for any advice | 10:51 |
*** serverascode has joined #openstack-ironic | 10:51 | |
*** mgould|afk is now known as mgould | 10:51 | |
mgould | morning Ironic! | 10:51 |
milan | morning gzholtkevych, mgould! :) | 10:52 |
mgould | morning milan sambetts gzholtkevych dtantsur vdrok tuanluong | 10:52 |
dtantsur | hey mgould | 10:52 |
*** amotoki has quit IRC | 10:53 | |
*** Vinsh has quit IRC | 10:55 | |
*** tangchen_ has joined #openstack-ironic | 10:56 | |
gzholtkevych | morning, milan! | 10:57 |
vdrok | gzholtkevych: so the problem is, ephemeral partition is not mounted when the instance gets booted? | 10:57 |
vdrok | morning mgould and gzholtkevych :) | 10:58 |
gzholtkevych | vdrok, I think, yes | 10:58 |
dtantsur | sambetts, I've +2ed the LLDP CLI spec, wanna check it? | 10:59 |
*** awidders_ has quit IRC | 10:59 | |
gzholtkevych | but I am going to check this on devstack and see deeply to images | 10:59 |
*** mmedvede has joined #openstack-ironic | 10:59 | |
*** amotoki has joined #openstack-ironic | 11:00 | |
sambetts | dtantsur: will do | 11:01 |
vdrok | gzholtkevych: yep, please try to boot an instance with ephemeral partition, and then take a look at configdrive contents in a partition labeled config-2 | 11:02 |
vdrok | if this thing gets passed there, it needs to be fixed I think | 11:02 |
gzholtkevych | ok, thank you | 11:03 |
vdrok | this thing == default_ephemeral_device | 11:03 |
*** sdake has joined #openstack-ironic | 11:03 | |
*** izaakk has quit IRC | 11:06 | |
*** izaakk has joined #openstack-ironic | 11:06 | |
*** thrash|g0ne is now known as thrash | 11:08 | |
tangchen_ | Hi dtantsur, vdrok, morning. Would you please help to check some of my ironicclient patch ? The 3 i18n patches, I think they are very simple fixes. I have just rebased them on the latest master branch. https://review.openstack.org/#/c/357639/ | 11:08 |
patchbot | patch 357639 - python-ironicclient - Support i18n for baremetal node cmds | 11:08 |
tangchen_ | https://review.openstack.org/#/c/375971/ https://review.openstack.org/#/c/375972/ | 11:09 |
patchbot | patch 375971 - python-ironicclient - Support i18n for baremetal port cmds | 11:09 |
patchbot | patch 375972 - python-ironicclient - Support i18n for baremetal chassis cmds | 11:09 |
dtantsur | tangchen_, these are very low priority, so while we'll get to them one day, I don't think it's useful to ping folks roughly every day | 11:09 |
dtantsur | thanks for contributing anyway, just please bear in mind that we have a huge review backlog | 11:10 |
vdrok | tangchen_: I'll take a look if I have time today | 11:10 |
tangchen_ | dtantsur, vdrok: Sure, OK. Thanks. :) | 11:10 |
*** sdake_ has joined #openstack-ironic | 11:11 | |
*** tangchen_ has quit IRC | 11:12 | |
*** stendulker has quit IRC | 11:12 | |
*** sdake has quit IRC | 11:13 | |
*** Vinsh has joined #openstack-ironic | 11:13 | |
*** amotoki has quit IRC | 11:18 | |
*** hshiina has joined #openstack-ironic | 11:18 | |
*** sdake has joined #openstack-ironic | 11:27 | |
*** sdake_ has quit IRC | 11:28 | |
openstackgerrit | Merged openstack/ironic: Introduce default_boot_option configuration option https://review.openstack.org/375458 | 11:31 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: Trivial: fix warning message formatting https://review.openstack.org/385919 | 11:31 |
openstackgerrit | Merged openstack/ironic: Clarify when oneview node can be managed by ironic https://review.openstack.org/372735 | 11:32 |
*** ccamacho has quit IRC | 11:34 | |
dtantsur | shredding of a 500 Gi drive takes 2 hours \o/ | 11:37 |
*** dprince has joined #openstack-ironic | 11:37 | |
*** cmyster has joined #openstack-ironic | 11:38 | |
*** ccamacho has joined #openstack-ironic | 11:39 | |
vsaienk0 | dtantsur: do you know if partitioned image with localboot works at the moment? At the gates test is failing but I didn't have time to check locally: http://logs.openstack.org/21/369021/42/experimental/gate-ironic-dsvm-multitest-nv/7c4dde8/logs/ir-cond-agent_ssh-partition-local-tinyipa.txt.gz?level=ERROR | 11:46 |
dtantsur | vsaienk0, it should | 11:46 |
dtantsur | well, I've never tried agent_ssh drivers; the whole tripleo uses local boot with pxe_* drivers though | 11:47 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: DO NOT MERGE: testing local boot https://review.openstack.org/385936 | 11:51 |
dtantsur | vsaienk0, testing all drivers ^^^ | 11:51 |
*** trown|outtypewww is now known as trown | 11:52 | |
*** athomas_ has joined #openstack-ironic | 11:53 | |
vsaienk0 | dtantsur: nice, thanks lets wait for test results | 11:53 |
*** lucas-afk is now known as lucasagomes | 11:54 | |
lucasagomes | morning all | 11:54 |
pas-ha | morning lucasagomes | 11:55 |
dtantsur | morning lucasagomes | 11:55 |
sergek | morning lucasagomes | 11:55 |
*** athomas has quit IRC | 11:56 | |
mariojv | morning lucasagomes & all | 12:02 |
*** dprince has quit IRC | 12:02 | |
*** phuongnh has quit IRC | 12:03 | |
*** sdake_ has joined #openstack-ironic | 12:05 | |
*** tuanluong has quit IRC | 12:05 | |
*** sdake has quit IRC | 12:06 | |
TheJulia | Good morning | 12:11 |
jlvillal | dtantsur, So fixed the unit tests yesterday. Not pretty. | 12:13 |
jlvillal | dtantsur, You mentioned a bug about those unit tests? | 12:13 |
jlvillal | dtantsur, Or I could create a new one. Might be a good bug for someone new to the project to work on. | 12:13 |
dtantsur | jlvillal, I think there was a bug, but I don't have a link handy | 12:14 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 12:14 |
jlvillal | dtantsur, Okay. I looked a little yesterday and didn't find it. | 12:14 |
jlvillal | But there are a lot of bugs | 12:14 |
dtantsur | oh yeah, way too many | 12:15 |
*** ralonsoh_ has joined #openstack-ironic | 12:15 | |
*** marlinc has quit IRC | 12:15 | |
vsaienk0 | morning lucasagomes, TheJulia, jlvillal, sergek and everyone else :) | 12:16 |
jlvillal | vsaienk0, :) | 12:16 |
*** bfournie has quit IRC | 12:16 | |
* jlvillal struggles to understand: $ exec 1> >( $TOP_DIR/tools/outfilter.py -v -o "${LOGFILE}" ) 2>&1 | 12:16 | |
* jlvillal thinks it is redirecting stdout, stderr to the program outfilter.py | 12:17 | |
dtantsur | also morning mariojv, TheJulia, jlvillal | 12:17 |
jlvillal | For the rest of the script run. But not sure. | 12:17 |
jlvillal | Hi dtantsur mariojv TheJulia lucasagomes vdrok and everyone else :) | 12:18 |
*** baoli has joined #openstack-ironic | 12:18 | |
*** ralonsoh has quit IRC | 12:18 | |
*** baoli_ has joined #openstack-ironic | 12:20 | |
*** zhangyufei_ has quit IRC | 12:21 | |
vdrok | morning pas-ha lucasagomes sergek mariojv TheJulia and jlvillal | 12:21 |
sambetts | jlvillal: so the magic there is the >() its called a named pipe, and its wizardry, I only discovered it recently | 12:21 |
jlvillal | sambetts, Yeah, I'm learning about it in devstack :) | 12:21 |
lucasagomes | o/ | 12:21 |
*** dprince has joined #openstack-ironic | 12:21 | |
*** marlinc has joined #openstack-ironic | 12:22 | |
sambetts | jlvillal: http://tldp.org/LDP/abs/html/process-sub.html | 12:23 |
*** baoli has quit IRC | 12:23 | |
jlvillal | sambetts, Thanks! | 12:24 |
sambetts | jlvillal: it blew my mind when I found out what that did | 12:25 |
jlvillal | sambetts, heh. Yeah. I'm trying to figure out how to force it closed when finished with it. | 12:26 |
*** vsaienko has joined #openstack-ironic | 12:26 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic-python-agent: Fix config drive writing script https://review.openstack.org/385957 | 12:30 |
vdrok | dtantsur: seems to be yet another one with configdrive ^^ | 12:31 |
dtantsur | oh | 12:31 |
*** marlinc has quit IRC | 12:31 | |
*** marlinc has joined #openstack-ironic | 12:31 | |
sambetts | jlvillal: do you know what version of bash you'll be running against? | 12:32 |
jlvillal | sambetts, What is in the gate. I think I figured it out. | 12:32 |
jlvillal | sambetts, I hope. Though I was doing my testing on Fedora 24. | 12:33 |
jlvillal | GNU bash, version 4.3.42(1)-release (x86_64-redhat-linux-gnu) | 12:33 |
jlvillal | Not sure actually what is on the gate. | 12:33 |
* jlvillal goes to prepare to go to work | 12:34 | |
sambetts | jlvillal: I was just reading some interesting things to do with using coprocs inplace of the process sub so that you can store a ref to the process, but you need version 4 or greater | 12:34 |
jlvillal | sambetts, Probably the bash version in ubuntu 14.04 and 16.04 | 12:35 |
jlvillal | sambetts, This is the patch: https://review.openstack.org/383543 | 12:35 |
patchbot | patch 383543 - openstack-dev/devstack - Inform user that the logfile skips lines matching ... | 12:35 |
jlvillal | sambetts, My hack is to add a sleep at the very end of stack.sh. I'll see if that gives me the output I am expecting | 12:36 |
vsaienk0 | dtantus: indeed we have a bug in agent driver http://logs.openstack.org/36/385936/1/check/gate-tempest-dsvm-ironic-ipa-partition-agent_ssh-tinyipa-nv/4a41e67/logs/screen-ir-cond.txt.gz?level=ERROR | 12:36 |
jlvillal | sambetts, My theory is that the files get closed and the script exits so quickly that lose the output | 12:36 |
sambetts | jlvillal: oh interesting! yeah | 12:36 |
vsaienk0 | sorry, dtantsur ^ | 12:37 |
jlvillal | sambetts, Or worse theory. The files get closed and then we don't see the output anymore. | 12:37 |
*** mtanino has joined #openstack-ironic | 12:38 | |
jlvillal | vsaienk0, As they say in the Princess Bride, Inconceivable! :) | 12:38 |
lucasagomes | vdrok, oh wow. I wonder if that fix gpt code now works | 12:38 |
lucasagomes | since it was never used | 12:38 |
dtantsur | vsaienk0, ......... thanks for tracking this | 12:39 |
mgould | morning vsaienk0 jlvillal TheJulia sergek lucasagomes | 12:39 |
vdrok | lucasagomes: yup, my vms for some reason switched to gpt (don't even know why :D) and that fixed things for me | 12:39 |
*** vmud213 has quit IRC | 12:39 | |
dtantsur | lucasagomes, vdrok, I need to do my planned CI refactor asap I guess :D | 12:40 |
*** ricardoas has left #openstack-ironic | 12:41 | |
vdrok | ++, we have some flood of bugs here with different combinations of stuff | 12:41 |
*** ricardoas has joined #openstack-ironic | 12:41 | |
lucasagomes | dtantsur, heh yeah we need to get the UEFI stuff in too | 12:42 |
* lucasagomes will brag about a release of cirros again | 12:42 | |
*** mtanino has quit IRC | 12:42 | |
lucasagomes | vdrok, ok if you tested I feel more confident, thanks! | 12:42 |
dtantsur | vsaienk0, mind filing a bug? | 12:42 |
*** nicodemos has joined #openstack-ironic | 12:44 | |
jroll | morning y'all | 12:46 |
*** ChanServ sets mode: -o jroll | 12:46 | |
vdrok | morning jroll | 12:46 |
sambetts | o/ jroll, apprently you now have a @ | 12:46 |
jroll | jlvillal: this is the bug you were looking for re ironic-lib mocking https://bugs.launchpad.net/ironic/+bug/1552843 | 12:47 |
openstack | Launchpad bug 1552843 in Ironic "ironic-lib is not properly mocked" [Medium,In progress] - Assigned to Zhenguo Niu (niu-zglinux) | 12:47 |
jroll | sambetts: indeed, I put the op hat on because freenode services were under maintenance last night | 12:47 |
dtantsur | morning jroll. we continue detecting config-drive related issues, as you see... | 12:47 |
sambetts | jroll: oh interesting, so ChanServ was down? | 12:47 |
jroll | sambetts: yeah, for a short time | 12:48 |
jroll | dtantsur: indeed :( | 12:48 |
vsaienk0 | dtantsur: I can fill a bug if you have not started yet :) | 12:51 |
dtantsur | vsaienk0, please do, I'm a bit busy right this second | 12:51 |
lucasagomes | vdrok, quick question. That parted code will work for both GPT and MBR. It doesn't actually check the type | 12:51 |
*** clenimar has quit IRC | 12:53 | |
lucasagomes | vdrok, oh forget | 12:53 |
lucasagomes | vdrok, I didn't pay attention to the grep there | 12:54 |
vdrok | lucasagomes: sorry, out for lunch now | 12:55 |
lucasagomes | vdrok, no worries it was my bad | 12:55 |
mgould | morning jroll | 12:56 |
*** hoangcx has joined #openstack-ironic | 12:57 | |
*** hoangcx has quit IRC | 12:58 | |
*** bfournie has joined #openstack-ironic | 12:59 | |
*** vikrant has quit IRC | 12:59 | |
*** hoangcx has joined #openstack-ironic | 13:01 | |
dtantsur | so, local boot with partition images seems broken for all drivers: https://review.openstack.org/#/c/385936/ :( | 13:05 |
patchbot | patch 385936 - ironic - DO NOT MERGE: testing local boot | 13:05 |
*** tangchen has quit IRC | 13:06 | |
xavierr | good morning Ironic :) | 13:07 |
*** jcoufal has joined #openstack-ironic | 13:10 | |
Romanenko_K | Hi! | 13:11 |
*** Guest73033 is now known as fyxim | 13:11 | |
vsaienk0 | dtantsur: https://bugs.launchpad.net/ironic/+bug/1633072 | 13:12 |
openstack | Launchpad bug 1633072 in Ironic "localboot with partitioned image doesn't work in agent driver" [Undecided,New] | 13:12 |
dtantsur | thanks | 13:12 |
*** [1]cdearborn has joined #openstack-ironic | 13:13 | |
dtantsur | I'm putting importance High, as it's an essential scenario | 13:13 |
*** jheroux has joined #openstack-ironic | 13:14 | |
openstackgerrit | Fabio Dassan proposed openstack/pyghmi: Implement M4 and Kent support https://review.openstack.org/361457 | 13:15 |
dtantsur | ugh, pxe_ssh drivers are broken too. importance=critical then. objections? | 13:15 |
*** nicodemos has quit IRC | 13:16 | |
sambetts | dtantsur: if they are borked can we kill them ? | 13:17 |
sambetts | ;) | 13:17 |
jroll | lol | 13:17 |
dtantsur | sure, great solution :D | 13:17 |
jroll | so this is since ironic-lib release, I guess? | 13:17 |
lucasagomes | I'm about to +a https://review.openstack.org/#/c/382889/, someone wants to take a quick look before ? | 13:18 |
patchbot | patch 382889 - ironic - Cleanup unused (i)PXE kernel parameters | 13:18 |
dtantsur | maybe? I wonder why tripleo is not affected. there must be some slight difference | 13:18 |
*** zackf has joined #openstack-ironic | 13:18 | |
jroll | lucasagomes: go for it | 13:18 |
* lucasagomes done! | 13:18 | |
lucasagomes | dtantsur, urgh... are you looking at it ? If not I will do | 13:19 |
dtantsur | lucasagomes, I'm not right now, still busy with a lot of distractions | 13:19 |
vsaienk0 | jroll, sambetts: I'm wondered if we should allow to build instances in provisioning network | 13:20 |
jroll | vsaienk0: why not? | 13:21 |
jroll | vsaienk0: I'd like to get rid of 'flat' and have that covered by building instances in the provisioning network | 13:21 |
jroll | vsaienk0: deployers using multitenant should make sure tenants cannot use the provisioning network | 13:21 |
*** vsaienko has quit IRC | 13:21 | |
sambetts | ++ if the tenent owns the provisioning network then there is no reason for that not to work | 13:22 |
bfournie | morning ironic | 13:22 |
vsaienk0 | jroll, sambetts: provisioning network is owned by admin, because it is special network, and deployer should provide access to this from control plane | 13:23 |
mgould | morning xavierr Romanenko_K bfournie | 13:23 |
*** zackf has quit IRC | 13:23 | |
*** portdirect has quit IRC | 13:23 | |
vsaienk0 | jroll, sambetts: I think we shouldn't allow to build instance inside provisioning, otherwise we gate access to control plane | 13:23 |
*** portdirect_ has joined #openstack-ironic | 13:23 | |
vsaienk0 | *we gave | 13:23 |
*** vsaienko has joined #openstack-ironic | 13:23 | |
*** portdirect_ is now known as portdirect | 13:23 | |
jroll | vsaienk0: which is why deployers using multitenant should make sure tenants cannot use the provisioning network | 13:24 |
sambetts | vsaienk0: provisinging network is owned by the deployers tenant, therefore the person creating an instance in the provisoing network knows the risks# | 13:24 |
sambetts | vsaienk0: other tenant can't use that network | 13:24 |
sambetts | unless deployer (worryingly) give them access to it | 13:24 |
vsaienk0 | sambetts: at the moment ironic doesn't allow to build instance in provisioning network, so I don't know if we should fix it or generate proper error message | 13:25 |
jroll | it doesn't? | 13:25 |
jroll | what code blocks that? | 13:25 |
*** b00tcat has joined #openstack-ironic | 13:25 | |
sambetts | I've never see that blocked before :/ | 13:26 |
vsaienk0 | jroll: https://github.com/openstack/ironic/blob/master/ironic/common/neutron.py#L128 | 13:27 |
vsaienk0 | jroll: we expect that there is no port with the same mac under provisioning network | 13:27 |
vsaienk0 | but when user build instance in it, nova already pre-create a port | 13:27 |
jroll | aha | 13:27 |
jroll | nice find, that's interesting | 13:27 |
vsaienk0 | we can just skip portalreadyexist exception and continue | 13:28 |
vsaienk0 | but I'm wondered if we should do it :) | 13:28 |
*** rloo has joined #openstack-ironic | 13:28 | |
jroll | yeah, seems odd to skip it | 13:28 |
*** Goneri has joined #openstack-ironic | 13:28 | |
sambetts | we'd have to track it because our code would delete the port after provisioning | 13:28 |
jroll | vsaienk0: wanna file a bug and we can all think about it a bit? | 13:29 |
*** nicodemos has joined #openstack-ironic | 13:29 | |
vsaienk0 | jroll: ok will do | 13:29 |
jroll | thanks | 13:29 |
jlvillal | jroll: Thanks for the bug link | 13:32 |
jroll | np | 13:32 |
b00tcat | Hi everyone - I'm using the pxe_ssh driver and I would like the spawned VM to have 2 NICs so I set 2 MAC addresses in my nodes JSON file | 13:38 |
b00tcat | however the spawned machines didn't have 2 NICs, only one | 13:38 |
b00tcat | am I missing something? or should I use `ironic port-create` manually? | 13:38 |
*** makowals has quit IRC | 13:39 | |
dtantsur | b00tcat, Ironic does not create any NIC (it's impossible for bare metal) | 13:39 |
dtantsur | b00tcat, instead, the ironic port list just records existing NICs | 13:39 |
*** makowals has joined #openstack-ironic | 13:39 | |
jroll | what is a "nodes json file"? | 13:39 |
lucasagomes | b00tcat, the VMs should be previoustly created for ironic to use it (what you mean by json file btw ?) | 13:40 |
sambetts | I'm guessing http://docs.openstack.org/developer/tripleo-docs/basic_deployment/basic_deployment_cli.html#register-nodes | 13:40 |
sambetts | ? | 13:40 |
* jroll sees tripleo and nopes out | 13:40 | |
dtantsur | haha | 13:40 |
b00tcat | indeed, I thought the JSON thing was standard at least when using pxe_ssh | 13:41 |
dtantsur | jroll, brace yourself, I've implemented ironic support in tripleo overcloud in newton :D now we have 2 ironic's there, so double the issues | 13:42 |
b00tcat | dtantsur: basically, if ironic tells my hypervysor to grant X CPUs, Y GB's of RAM and 1 NIC, how can I tell ironic to create an additional NIC on the VM? | 13:42 |
sambetts | ironic doesn't tell a hypervisor anything | 13:42 |
jroll | dtantsur: ++ | 13:42 |
dtantsur | b00tcat, ironic never tells anything to a hypervisor, outside of "power on/power off" | 13:42 |
*** hamzy has quit IRC | 13:42 | |
dtantsur | b00tcat, ironic is a bare metal service, we only use VMs to simulate bare metals | 13:43 |
dtantsur | b00tcat, we never ask a hypervisor to do anything impossible on regular bare metals | 13:43 |
openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/385180 | 13:43 |
b00tcat | ah, now this is new - openstack is so big that I'm mixing what does what... sorry about that then :-) | 13:43 |
b00tcat | dtantsur: yeah that makes senes | 13:44 |
b00tcat | sense* | 13:44 |
mgould | b00tcat: if you want to spawn VMs, you need nova | 13:44 |
sambetts | jroll: can we kill that driver pleeeeeeese | 13:45 |
dtantsur | sambetts, I hope we will, this cycle | 13:45 |
mgould | b00tcat: https://www.openstack.org/software/project-navigator/ may be useful | 13:45 |
b00tcat | mgould: yeah I do have it, but afaik nova doesn't care about nics, only CPU, RAM & disk size | 13:45 |
b00tcat | that's why I assumed it was ironic ^^" | 13:45 |
jroll | sambetts: three month deprecation period :) | 13:45 |
sambetts | :( | 13:46 |
mgould | b00tcat: https://ask.openstack.org/en/question/28577/add-network-interface-to-instance/ | 13:46 |
sambetts | b00tcat: if you want to deploy VMs and manage their networking see Neutron | 13:46 |
* mgould gripes Yet Again about OpenStack's love of impenetrable project codenames | 13:47 | |
b00tcat | thanks! | 13:48 |
*** athomas_ has quit IRC | 13:48 | |
*** links has quit IRC | 13:48 | |
*** mgoddard has joined #openstack-ironic | 13:49 | |
*** Marx314 has joined #openstack-ironic | 13:49 | |
mgould | b00tcat: np, good luck | 13:50 |
* jroll keeps a close eye on https://review.openstack.org/#/c/385994/1 | 13:51 | |
patchbot | patch 385994 - tempest - [WIP] Remove Ironic | 13:51 |
dtantsur | I like the title :D I imagine a patch with `rm -rf .` | 13:52 |
jroll | pretty much | 13:52 |
jroll | thiagop had started on this, trying to get jordan to pick that work up | 13:52 |
jroll | if someone here wanted to pick that up as well, feel free | 13:53 |
jroll | http://lists.openstack.org/pipermail/openstack-dev/2016-October/105603.html | 13:53 |
jroll | every time a tempest person tries to remove it, they do it in a way that will break all of our CI | 13:53 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: DNM: testing tempest removal https://review.openstack.org/385997 | 13:54 |
* jroll proves it with a patch | 13:54 | |
rloo | hi and morning dtantsur, jroll, mgould, sambetts, b00tcat, lucasagomes, vsaienk0 | 13:55 |
dtantsur | morning rloo | 13:55 |
jroll | morning rloo :) | 13:55 |
mgould | morning rloo | 13:55 |
rloo | dtantsur: fyi, i approved before thinking that you might want to look at it, given that you're release liason: https://review.openstack.org/#/c/385616/ | 13:55 |
patchbot | patch 385616 - ironic - Add docs about releasing ironic projects | 13:55 |
jroll | rloo++ | 13:55 |
lucasagomes | rloo, hi there | 13:56 |
mgould | jroll: I think thiagop is in Patagonia until late Nov/early Dec | 13:56 |
dtantsur | rloo, looks correct, thanks | 13:56 |
* mgould is most envious | 13:56 | |
rloo | mgould: oh, that rings a bell, yes, he was going on a great adventure! | 13:57 |
mgould | yep | 13:57 |
mgould | three-month outdoor-leadership course, IIRC | 13:57 |
mgould | sounds amazing | 13:57 |
lucasagomes | dtantsur, vsaienk0 I think the local boot problem is an image problem (devstack is being setup, so I didn't confirm yet) | 13:57 |
lucasagomes | but the cirros uac from (https://github.com/openstack-dev/devstack/blob/master/stackrc#L669) seems to be an blank image | 13:57 |
jroll | mgould: ya, saw your email | 13:58 |
rloo | sorry to bring this up, jroll et al. i don't really want to encourage fixing typos in our specs: https://review.openstack.org/#/c/385780/ | 13:58 |
patchbot | patch 385780 - ironic-specs - fix a typo | 13:58 |
dtantsur | lucasagomes, WUT? if that's true, then ugh.. | 13:58 |
lucasagomes | dtantsur, yeah I mounted it in a loop locally | 13:58 |
lucasagomes | it doesn't contain /dev etc... I wonder even if it has grub2 installed | 13:59 |
dtantsur | rloo, well, there are way too many typo fixing patches nowadays | 13:59 |
mgould | lucasagomes: that seems *excessively* minimalist | 13:59 |
lucasagomes | I will confirm anyway... | 13:59 |
lucasagomes | mgould, yes | 13:59 |
dtantsur | lucasagomes, if so, we need to fix it asap.. we're planning on switching to local boot eventually | 13:59 |
rloo | dtantsur: exactly. i don't mind user-facing ones. but the specs -- i'm not even commenting on all the grammatical stuff there | 13:59 |
jroll | rloo: while I understand why, I think it takes more effort to talk about it and/or explain why we wouldn't accept it than to just +2A | 13:59 |
lucasagomes | dtantsur, right, lemme see the other partition images available for cirros | 14:00 |
* dtantsur hopes that these people don't just try to get a free ATC ticker or an ability to vote | 14:00 | |
rloo | jroll: i was going to suggest we bring it up in our meeting, vote on it, so we have a 'policy' wrt the specs. then i would feel fine -2ing. | 14:00 |
*** anush has joined #openstack-ironic | 14:00 | |
*** vsaienko has quit IRC | 14:01 | |
mgould | rloo: should I stop noting typos/etc on commits I'm reviewing? | 14:01 |
rloo | jroll: there are way too many grammar/typos in the specs, no one should waste their time on those. | 14:01 |
dtantsur | rloo, maybe worth asking other projects too? there was a thread already on pointless changes | 14:01 |
rloo | mgould: when reviewing, i think it is fine. after landing though? | 14:01 |
* mgould nods | 14:01 | |
dtantsur | mgould, IMO no. for user-visible strings it's even fine to -1 for them (also IMO) | 14:01 |
rloo | dtantsur: we had a thread years ago about that, but didn't discuss specs. | 14:01 |
mgould | I can see the point of a typo-fix as your first OpenStack patch, to get used to the process | 14:02 |
jroll | yeah, I don't think these harm anything | 14:02 |
mgould | but that's not the case here | 14:02 |
jroll | I don't see why we need a policy | 14:02 |
jroll | if you feel like it's wasting your time, then don't review it | 14:02 |
dtantsur | mgould, I'd prefer people starting picking low-hanging-fruit's tbh | 14:02 |
openstackgerrit | Stenio Araujo proposed openstack/ironic: Adds another validation step when using dynamic allocation https://review.openstack.org/377073 | 14:02 |
rloo | jroll: ok, moving on then | 14:02 |
rloo | jroll: sorry, so you're fine with it and you'd (or others) would review those, right? cuz if we all just ignore them, i think that's wrong. | 14:04 |
jroll | rloo: if I see them, I will, it takes 30 seconds of my time | 14:04 |
openstackgerrit | Merged openstack/ironic-specs: fix a typo https://review.openstack.org/385780 | 14:05 |
rloo | jroll: ok | 14:05 |
jroll | rloo: I think it's also important to remember that typos and grammar problems make things harder to read for non-english speakers | 14:05 |
rloo | jroll: i agree but in the specs? anyway, i've got your ok to ignore them, so. | 14:06 |
jroll | rloo: you don't think it's important for non-english people to comprehend specs as well as english people? | 14:07 |
pas-ha | lucasagomes: what is our deprecation policy re backward compat for old IPA + new Ironic? | 14:07 |
* pas-ha just looking at JayF comments in pxe params cleanup patch | 14:07 | |
rloo | jroll: i do think it is important. i don't think one-word typo patches is effective use of our time | 14:07 |
rloo | jroll: esp when it is s/a/an/ or whatever it was. | 14:08 |
lucasagomes | pas-ha, not sure it's well defined. But I think we should follow the same deprecation policy as ironic does for config options. 1 cycle | 14:08 |
JayF | To be blunt; I think one of the underlying unasked questions is; if stackalytics didn't exist, would these one-word-typo-patches exist? | 14:08 |
jroll | rloo: yeah, I guess, I just think +2A takes less time than -2 | 14:08 |
mariojv | imo it takes the same amount of time to +2 or -1 it, so it's a sunken cost | 14:08 |
JayF | I think that's part of the reason it sorta can grind the gears | 14:09 |
pas-ha | JayF: +100500 | 14:09 |
mariojv | JayF: probably not | 14:09 |
lucasagomes | #1. log a warning when old options are used #2. remove 'em after 1 cycle + release note to upgrade | 14:09 |
* mgould feels an ambiguity in a spec is definitely a problem, but not all grammar errors create ambiguity | 14:09 | |
mgould | JayF: I've submitted typo-fix patches to open source projects that weren't on Stackalytics :-) | 14:09 |
mgould | though possibly we wouldn't have so many of them | 14:09 |
*** sdake_ has quit IRC | 14:09 | |
jroll | pas-ha: we test that X version of IPA works with X+1 version of ironic, we also try to make sure X+1 IPA works with X ironic | 14:09 |
jroll | lucasagomes: ^^ | 14:10 |
lucasagomes | jroll, ++ | 14:10 |
rloo | jroll, mariojv. yes, it takes the same amount of time to +2 or -1. the question/issue i'm thinking of is the culture or whatever we are creating. i'd rather folks focus on reviewing/fixing more important issues, than firing off one-word/line typo patches. | 14:10 |
*** sdake has joined #openstack-ironic | 14:10 | |
jroll | rloo: I think if it becomes a problem to the point where we feel overwhelmed by them, we can re-evaluate | 14:10 |
rloo | the +2 condones that we are perfectly happy with commit msgs that say 'typo fix. trivial', with one/minimal update/changes. | 14:11 |
jroll | rloo: right, I personally am okay with that right now | 14:11 |
mariojv | ++ i think if experienced contributors are using it to just game their numbers, then we might re-evaluate | 14:11 |
jroll | rloo: as always, I am not making an edict here, if the rest of the community wants to start blocking them, I'm okay with that | 14:11 |
rloo | i don't even want to deal with the whole stackalytics thing. that makes me sad but we all have jobs for companies that seem to like numbers. | 14:11 |
*** joprovost has joined #openstack-ironic | 14:11 | |
pas-ha | jroll: ok, so x+2 / x-2 is not a concern.. I am thinking about removing ipa-driver-name and driver_name from IPA | 14:11 |
mariojv | if a couple new contributors are doing it, it doesn't seem to matter that much, and having a high barrier to entry might discourage them from future contributions | 14:11 |
jroll | pas-ha: yeah, not worried about that | 14:12 |
JayF | rloo: I think all the companies like numbers. Some are slightly more obsessed than others :x | 14:13 |
*** absubram has joined #openstack-ironic | 14:14 | |
rloo | jroll: i agree, for that one instance. when that one becomes many though... | 14:14 |
TheJulia | jroll: When you have a minute, if you could put your blessing on https://review.openstack.org/#/c/386002/ :) | 14:14 |
patchbot | patch 386002 - openstack-infra/project-config - Add label-Verified for bifrost | 14:14 |
mgould | JayF: https://www.deming.org/theman/theories/deadlydiseases lists "management by numbers" as one of the seven deadly diseases of Western management... | 14:16 |
rloo | mariojv: should i review your power notification patch or were you going to do another revision? | 14:16 |
jroll | TheJulia: ooo, are we getting third party CI or something? | 14:16 |
TheJulia | jroll: opnfv has been posting results from jobs for a few weeks :) | 14:17 |
jroll | oh nice | 14:17 |
jroll | \o/ | 14:17 |
TheJulia | Just, no visibility so we want to get that fixed | 14:17 |
jroll | TheJulia: don't you need to create bifrost-ci first? | 14:17 |
jroll | idk how those are made | 14:17 |
mariojv | rloo: please go ahead, i may change the except...except thing if someone -1s based on that, but i don't have any planned revisions at this time | 14:18 |
mariojv | rloo: thanks! | 14:18 |
TheJulia | my understanding is it gets auto-created upon the entry and infra needs to set ownership from there | 14:18 |
jroll | okay cool | 14:18 |
*** openstackgerrit has quit IRC | 14:18 | |
jroll | TheJulia: +1'd | 14:18 |
*** openstackgerrit has joined #openstack-ironic | 14:19 | |
TheJulia | \o/ | 14:19 |
rloo | mariojv: i'm a bit iffy on +2 if both yuriy and i are questioning the isinstance() vs try except except. i'll think about it in the background... | 14:24 |
mariojv | rloo: alright. i'm fine with changing it, just seems weird to either a) duplicate code in both except clauses or b) put that shared logic outside of some exception handling | 14:25 |
mariojv | maybe there's a better way than a) or b) that i'm not thinking of | 14:25 |
rloo | mariojv: we have cases where eg the error msg is generated before hand and may not be used. | 14:26 |
mariojv | hmm, ok, i'll try to find an example | 14:26 |
rloo | mariojv: i also don't like the i18n's _() requiring strings as arguments | 14:26 |
mariojv | rloo: same, it's kind of silly | 14:26 |
*** hshiina has quit IRC | 14:27 | |
rloo | mariojv: we don't get much xlated anyway so i'm not too concerned about that | 14:27 |
mariojv | true :| | 14:27 |
*** hamzy has joined #openstack-ironic | 14:30 | |
openstackgerrit | Xavier proposed openstack/ironic: Adds validation before delete profile in oneview https://review.openstack.org/386014 | 14:33 |
openstackgerrit | Ruby Loo proposed openstack/ironic-specs: 'iPXE to use Swift Temp URLs' done in 6.2 https://review.openstack.org/386015 | 14:35 |
jroll | oops, did I miss that oen? | 14:36 |
jroll | one* | 14:36 |
rloo | jroll: yeah. i can't recall why now, but i think maybe i didn't mention it cuz when i noticed it, it was still missing documentation :) | 14:37 |
rloo | jroll: after the doc landed, i forgot about it. am reviewing them for the ocata priorities. | 14:37 |
jroll | rloo: ah yeah, that'll do it | 14:37 |
rloo | jroll: the priorities etherpad isn't listing them in priority order, just two groups, large/small, right? | 14:38 |
jroll | rloo: right | 14:38 |
jroll | just dumping options right now | 14:38 |
rloo | jroll: i'm going to add a link to any related specs to those 'things'. | 14:38 |
jroll | rloo: cool, do whatever you want with that pad :) | 14:38 |
rloo | jroll: 'whatever you want', hmm... :D | 14:39 |
jroll | rloo: my delete key still works ;) | 14:39 |
rloo | darn! | 14:39 |
*** yuriyz|2 has joined #openstack-ironic | 14:41 | |
*** mtanino has joined #openstack-ironic | 14:41 | |
rloo | jroll: qa/testing changes in ocata. what big/small things? | 14:46 |
jroll | rloo: dmitry's refactor is the big one I'm thinking of | 14:46 |
jroll | also testing more things | 14:46 |
rloo | jroll: and multinode grenade | 14:46 |
rloo | are those one or three things? | 14:46 |
jroll | rloo: I think of multinode as the same as rolling upgrades | 14:47 |
rloo | jroll: ok, i put one thing for qa/testing | 14:47 |
jroll | thanks | 14:47 |
mariojv | is there anything wrong with current unit tests on master? they don't seem to be passing even after i completely blow away .tox | 14:49 |
mariojv | saw this from yesterday and it made me suspicious https://github.com/openstack/ironic/commit/3110fa8594a3e5d7e71376b935853f37db3b69e3 | 14:49 |
jroll | mariojv: there was yesterday, should be fixed in master | 14:49 |
jroll | yeah, that was the fix | 14:49 |
jroll | maybe tox -r, I wonder if old ironic-lib breaks unit tests now | 14:50 |
mariojv | seems like something small internal to ironic-lib, it's using positional args instead of kwargs in some call | 14:51 |
mariojv | ah, i don't think i actually destroyed my current tox env correctly | 14:51 |
jroll | probably | 14:53 |
mariojv | yup - works now. thanks jroll | 14:53 |
jroll | we mock ironic-lib things in ironic :/ | 14:53 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce VolumeConnector object https://review.openstack.org/214584 | 14:54 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Introduce new RPCs to support volume connector operation https://review.openstack.org/214585 | 14:55 |
mariojv | jlvillal: is there a bug up for the comments here (or are you working on it atm)? https://github.com/openstack/ironic/commit/3110fa8594a3e5d7e71376b935853f37db3b69e3#diff-ab28dc8ae209d2ad143dd17c5f7f6fccR339 | 14:55 |
mariojv | otherwise i might take a look tonight if i have some time | 14:55 |
jroll | mariojv: https://bugs.launchpad.net/ironic/+bug/1552843 | 14:55 |
openstack | Launchpad bug 1552843 in Ironic "ironic-lib is not properly mocked" [Medium,In progress] | 14:55 |
mariojv | thanks | 14:55 |
jroll | np | 14:55 |
rloo | jroll: ok, updated the priorities. i'm not sure about the color map thing though. maybe we need a designer to pick our colours for us. ha ha | 14:56 |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Make agent driver always PXE before cleaning https://review.openstack.org/263842 | 14:58 |
jroll | rloo: you can change your color :P | 14:58 |
jroll | good call on some of those | 14:58 |
jroll | I swear I had driver composition in there! | 14:59 |
rloo | lucasagomes: there are two specs by you: iPXE dynamic configuration, override PXE options via glance property. I don't know how important those features are. should they be ocata priorities? | 14:59 |
rloo | jroll: i refuse to change my colours! :) | 14:59 |
jroll | fine! | 14:59 |
rloo | jroll: wait, let me get a bible... | 14:59 |
rloo | jroll: the problem is your pink, not my blue! | 15:00 |
jroll | heh | 15:00 |
jroll | your links are super hard to read | 15:00 |
jroll | and I was here first :) | 15:00 |
rloo | jroll: i know. isn't that funny? let me think about it. i hate baby blue... | 15:00 |
*** rajinir has joined #openstack-ironic | 15:00 | |
*** sdake has quit IRC | 15:00 | |
lucasagomes | rloo, hmm I think they are nice things to have but I wouldn't call it a priority | 15:01 |
*** mjura has quit IRC | 15:01 | |
rloo | lucasagomes: ok, good to know. | 15:01 |
lucasagomes | rloo, well... depends, the glance one can be depending on people's usecase | 15:01 |
openstackgerrit | Merged openstack/ironic: Cleanup unused (i)PXE kernel parameters https://review.openstack.org/382889 | 15:02 |
openstackgerrit | Merged openstack/ironic: Add docs about releasing ironic projects https://review.openstack.org/385616 | 15:02 |
lucasagomes | e.g, if people wants to deploy an Fedora/CentOS/RHEL atomic partition image that will be needed | 15:02 |
rloo | lucasagomes: i could put it down as a small thing/priority. to see if folks think it is. these are suggestions for priorities... | 15:02 |
lucasagomes | rloo, yeah, leave it low | 15:03 |
rloo | lucasagomes: i'll put it down, but i won't fight for it :D | 15:03 |
lucasagomes | rloo, fair enough :-) | 15:03 |
jlvillal | mariojv: There was a proposed patch but it never got finished by the author. No idea if it was good or not. | 15:03 |
dtantsur | lucasagomes, rloo, I'd prefer we don't implement "override PXE options via glance" if we can't do the same for local boot (and we rejected this idea) | 15:03 |
*** rbrud has joined #openstack-ironic | 15:04 | |
rloo | dtantsur: not implement it at all? | 15:04 |
dtantsur | rloo, yes | 15:04 |
lucasagomes | dtantsur, what you mean? | 15:04 |
*** zackf has joined #openstack-ironic | 15:04 | |
rloo | dtantsur: spec was approved: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/override-pxe-options-via-glance-property.html | 15:04 |
lucasagomes | dtantsur, oh gotcha | 15:04 |
*** syed_ has joined #openstack-ironic | 15:04 | |
dtantsur | rloo, I know, but the situation has changed | 15:04 |
lucasagomes | dtantsur, yeah, we need to be able to set the kernel paraemters for local bootloader as well | 15:04 |
lucasagomes | probably that would be step #1 | 15:04 |
*** sdake has joined #openstack-ironic | 15:05 | |
rloo | dtantsur, lucasagomes: ok, i'll leave it for you two to work out/update the spec... | 15:05 |
lucasagomes | does we have any RFE for ^ ? | 15:05 |
jroll | can we remove the spec from 'approved' if we are de-approving it? | 15:05 |
jroll | (un-approving?) | 15:05 |
dtantsur | lucasagomes, we've rejected such once already... tl;dr was messing with local boot loader is too OS-specific and crosses the border of configuring an instance | 15:05 |
*** ElCoyote_ has left #openstack-ironic | 15:06 | |
dtantsur | I don't completely agree with the latter, but the former is an issue for sure | 15:06 |
lucasagomes | dtantsur, hmm well depending on the options you use to configure | 15:06 |
sambetts | and this is why /me loves the whole disk images ;) | 15:06 |
lucasagomes | nowadays you can isntall pretty much any linux distro on top of btrfs | 15:06 |
lucasagomes | and you will need it to boot from a btrfs subvolume | 15:06 |
dtantsur | sambetts, yeah, but you can't create a new image each time you need to change something | 15:07 |
lucasagomes | not os spefic at all | 15:07 |
lucasagomes | but anyway... it's not a prio for now I gues | 15:07 |
sambetts | dtantsur: change what though? boot loader options? | 15:07 |
sambetts | can't we use image metadata to store it?? | 15:08 |
dtantsur | sambetts, SR-IOV folks needs to change them more often than not IIRC.. enable something in kernel | 15:08 |
sambetts | lucasagomes: ^ | 15:08 |
sambetts | use image meta data to store the kernel params you want to use at boot time, then you can just update them | 15:08 |
dtantsur | but the problem is that the procedure of updating kernel parameters differs even between Linux distributions | 15:08 |
dtantsur | sambetts, the "just update" bit is really complex, as it seems | 15:09 |
lucasagomes | yeah, need to set to /etc/grub/default (assuming grub) or something | 15:09 |
sambetts | well ironic isn't going to update a running instance | 15:09 |
lucasagomes | to propagate on updates | 15:09 |
lucasagomes | dtantsur, for the lvm work (raid on deploy) you may need it too | 15:09 |
sambetts | so it would just grab the parameters from the img metadata on bootloader install and then deploy as normal | 15:10 |
dtantsur | lucasagomes, sigh... but note that lvm != raid on deploy | 15:10 |
dtantsur | sambetts, "on bootloader install" is OS-specific | 15:10 |
lucasagomes | unless we make sure we always use the same name for volume group and lvm parittion | 15:10 |
lucasagomes | dtantsur, right, yeah just as an example | 15:10 |
*** ElCoyote_ has joined #openstack-ironic | 15:10 | |
dtantsur | sambetts, ubuntu and RH system differs in it | 15:10 |
sambetts | right but there are standard tools for doing grub updates from the grub packages right? | 15:11 |
dtantsur | well, the way you configure it is different. I don't quite remember, seems like RH systems use grubby, while ubuntu/debian use a config file | 15:11 |
lucasagomes | dtantsur, ++ | 15:13 |
sambetts | kexec? | 15:13 |
sambetts | ;) | 15:13 |
lucasagomes | we need to check what grubby does, if it only goes and update /etc/default/grub we probably should use that | 15:13 |
lucasagomes | sambetts, noooes hah | 15:14 |
rloo | jroll: you know what is involved, is this a priority: https://bugs.launchpad.net/ironic/+bug/1594242 | 15:14 |
openstack | Launchpad bug 1594242 in Ironic "[RFE] Security Groups support for baremetal servers" [Wishlist,In progress] - Assigned to Sukhdev Kapur (sukhdev-8) | 15:14 |
*** joprovost has quit IRC | 15:14 | |
lucasagomes | sambetts, once I worked in a project (inside the university) that we had to bot windows from a linux ramdisk | 15:14 |
jroll | rloo: I would like to just finish that before summit | 15:14 |
sambetts | lucasagomes: oh god... | 15:14 |
lucasagomes | sambetts, we've used kexec to load grub4dos to boot windows | 15:14 |
dtantsur | lucasagomes, feel free to restart the conversation on the boot options.. I still find it rather useful | 15:14 |
rloo | jroll: doubtful but you can try :) | 15:15 |
lucasagomes | and we had a lot of hacks to reset the state of the video cards etc | 15:15 |
lucasagomes | it was hell! | 15:15 |
lucasagomes | dtantsur, sure | 15:15 |
jroll | rloo: it's a small patch | 15:15 |
*** rbartal has quit IRC | 15:16 | |
rloo | jroll: ok, doesn't sound like we need to put it as a priority then. it'll get done. | 15:16 |
jroll | rloo: yeah | 15:17 |
*** rcernin has quit IRC | 15:20 | |
dtantsur | jlvillal, fyi I've started putting some ideas to https://etherpad.openstack.org/p/ironic-ocata-summit-qa | 15:20 |
*** joprovost has joined #openstack-ironic | 15:20 | |
jlvillal | dtantsur: Sweet. Thanks :) | 15:24 |
*** lucasagomes is now known as lucas-hungry | 15:24 | |
*** aNuposic has joined #openstack-ironic | 15:31 | |
*** amotoki has joined #openstack-ironic | 15:32 | |
*** jcoufal_ has joined #openstack-ironic | 15:36 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-python-agent: Remove old lookup/heartbeat from IPA https://review.openstack.org/386052 | 15:38 |
*** jcoufal has quit IRC | 15:38 | |
*** amotoki has quit IRC | 15:40 | |
sambetts | jroll: I completely forgot to mention it the other day, do we want to include this in the priorities discussion? | 15:48 |
sambetts | jroll: https://review.openstack.org/#/c/341086 | 15:48 |
patchbot | patch 341086 - ironic-specs - Add spec for IPA API versioning support | 15:48 |
*** vishwanathj_zzz is now known as vishwanathj | 15:48 | |
*** b00tcat has quit IRC | 15:49 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-python-agent: Remove old lookup/heartbeat from IPA https://review.openstack.org/386052 | 15:50 |
*** amotoki has joined #openstack-ironic | 15:54 | |
rloo | sambetts: i put that down as a priority in the smaller things. it is small? | 15:55 |
*** krtaylor has quit IRC | 15:56 | |
sambetts | I don't thing that its a large amount of work, I just think that its something we need to address quite soon to put the framework in palce for handling issues like we have seen before as people upgraded from mitaka to newton | 15:56 |
rloo | sambetts: agree. that's why i put it down as a priority. a small but mighty feature :) | 15:57 |
sambetts | totally :) | 15:57 |
JayF | Where would a new RFE for an ironic-inspector collector go? The code goes in IPA, but I wasn't sure if we did RFEs against ipa project or not | 15:59 |
mariojv | or the ironic-inspector project maybe? https://bugs.launchpad.net/ironic-inspector | 16:00 |
mariojv | since this is kind of cross-sub-project (inspector uses it, most code goes in IPA), maybe just generic ironic project would be the best | 16:01 |
jroll | sambetts: ++ | 16:01 |
jroll | JayF: yeah, RFEs can go anywhere I thiiiink? | 16:01 |
sambetts | well an RFE is a bug in launchpad so you can tag multiple projects with it right? | 16:01 |
*** rama_y has joined #openstack-ironic | 16:01 | |
jroll | JayF: if it's inspector focused I'd put it in inspector, I think often there needs to be code in inspector | 16:01 |
jroll | heh. true that. | 16:01 |
JayF | there are rfes in ipa | 16:01 |
JayF | oh good point | 16:01 |
*** yuriyz|2 has quit IRC | 16:04 | |
*** tesseract- has quit IRC | 16:06 | |
*** derekh has quit IRC | 16:06 | |
*** aNuposic has quit IRC | 16:07 | |
*** ohamada has quit IRC | 16:12 | |
*** e0ne has quit IRC | 16:14 | |
*** mgould is now known as mgould|afk | 16:15 | |
*** vsaienko has joined #openstack-ironic | 16:17 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/ironic: Test ML2 Neutron Events feature https://review.openstack.org/370016 | 16:20 |
* sambetts -> home | 16:21 | |
*** sambetts is now known as sambetts|afk | 16:21 | |
sambetts|afk | night all | 16:21 |
*** ppiela has joined #openstack-ironic | 16:21 | |
milan | night sambetts|afk! | 16:21 |
*** lucas-hungry is now known as lucasagmes | 16:24 | |
lucasagmes | dtantsur, vsaienk0 a 2 good and 1 bad news. Good: #1. Local boot works #2. The default_boot_option config works. Bad: The cirros partition image doesn't have grub installed | 16:26 |
openstackgerrit | Merged openstack/ironic: Trivial: fix warning message formatting https://review.openstack.org/385919 | 16:26 |
dtantsur | >_< | 16:26 |
lucasagmes | I've created a ubuntu image with DIB and it deployed a partition image with agent driver + local boot | 16:26 |
dtantsur | lucasagmes, this sounds fixable though (also something is wrong with your nick ;) | 16:27 |
*** lucasagmes is now known as lucasagomes | 16:27 | |
lucasagomes | dtantsur, fixed hah | 16:27 |
lucasagomes | dtantsur, yeah, installing grub shouldn't be a problem. But we need a custom image | 16:27 |
dtantsur | oh, yeah, this is upstream cirros... damn | 16:28 |
lucasagomes | I wonder if we could create one and host somewhere to use in our gate testings | 16:28 |
lucasagomes | testing* | 16:28 |
NobodyCam | Good Morning Ironic folk :) | 16:28 |
lucasagomes | NobodyCam, hi there | 16:29 |
NobodyCam | Morning lucasagomes :) | 16:29 |
milan | morning NobodyCam! :) | 16:29 |
NobodyCam | morning milan :) | 16:29 |
*** harlowja_at_home has joined #openstack-ironic | 16:30 | |
*** vsaienko has quit IRC | 16:31 | |
dtantsur | morning NobodyCam | 16:32 |
NobodyCam | Morning dtantsur :) | 16:32 |
*** vsaienko has joined #openstack-ironic | 16:32 | |
dtantsur | ald g'night, see you tomorrow | 16:33 |
*** dtantsur is now known as dtantsur|afk | 16:33 | |
lucasagomes | dtantsur, night | 16:33 |
NobodyCam | night dtantsur|afk :) | 16:33 |
milan | good night dtantsur|afk! :) | 16:33 |
*** vsaienko has quit IRC | 16:37 | |
*** vsaienko has joined #openstack-ironic | 16:37 | |
jroll | lucasagomes: yeah, shouldn't be a problem to host one on tarballs.o.o | 16:39 |
jroll | then we can do uefi and bonding and such too | 16:39 |
jroll | maybe we make a quick build script for it in tools/ and publish it on any commit that changes it, idk | 16:40 |
lucasagomes | jroll, yeah that would be the best plan IMHO, I've commented in the bug: https://bugs.launchpad.net/ironic/+bug/1633072 | 16:40 |
openstack | Launchpad bug 1633072 in Ironic "Local bootloader with partition images doesn't work in CI" [Critical,Confirmed] - Assigned to Lucas Alvares Gomes (lucasagomes) | 16:40 |
jroll | cool, thanks! | 16:40 |
jroll | we should probably downgrade that to high since it's just CI | 16:40 |
lucasagomes | done | 16:40 |
lucasagomes | jroll, like a post-merge job in IPA right ? | 16:42 |
jroll | lucasagomes: yeah, something like that | 16:42 |
*** sdake_ has joined #openstack-ironic | 16:42 | |
JayF | Is it perhaps a better idea to do it in a separate project/repo? | 16:44 |
JayF | Then it's just a straight-up post job | 16:44 |
JayF | since really this cirros image would be used in ipa, ironic-inspector, and ironic jobs | 16:44 |
lucasagomes | jroll, sambetts|afk we use tinyipa in grenade right ? Cause of the resource limitation. It's important that the image being deployed also can boot with that specs | 16:45 |
*** jpich has quit IRC | 16:45 | |
lucasagomes | problem with customizing cirros is that they use buildroot, so there's no package management or anything in the image | 16:45 |
lucasagomes | JayF, yeah, there's some folks in neutron also talking about building their own cloud images | 16:45 |
*** hwoarang has quit IRC | 16:45 | |
jroll | lucasagomes: we use tinyipa almost everywhere | 16:45 |
lucasagomes | JayF, https://etherpad.openstack.org/p/cirros-respin | 16:45 |
*** sdake has quit IRC | 16:46 | |
jroll | oh neat | 16:46 |
JayF | the only jobs that should ever not run tinyipa | 16:46 |
JayF | are in ipa | 16:46 |
lucasagomes | right | 16:46 |
JayF | the ipa jobs that build from source all supported ramdisks enforce the api | 16:46 |
JayF | then we use tinyipa, not because it's the best, or good for production, but because it's teeny tiny | 16:46 |
jroll | so yeah, we have to be able to boot the thing in 512mb | 16:46 |
lucasagomes | yeah exactly, I believe it's the same for cirros | 16:46 |
JayF | in fact, I wonder if we should put something in docs warning about security issues with tinyipa ... like the fact there's no security whatsoever in the way it's built (no ssl, no hash verify, etc) | 16:47 |
*** hwoarang has joined #openstack-ironic | 16:47 | |
jroll | btw, https://review.openstack.org/#/c/329832/ | 16:47 |
patchbot | patch 329832 - openstack-infra/project-config - Add job which builds test images with watchdog sup... | 16:47 |
lucasagomes | jroll, JayF this would be a good thing to discuss on friday in the summit if we don't reach a conclusion until there | 16:48 |
lucasagomes | we could invite the neutron folks too | 16:48 |
JayF | lucasagomes: I'm not at the summit, so I disagree :P | 16:48 |
jroll | lucasagomes: ++ | 16:48 |
lucasagomes | JayF, oh | 16:48 |
JayF | yeah, that was pretty much how I felt about not getting the travel approved | 16:48 |
JayF | lol | 16:48 |
lucasagomes | you can join us on a hangout session :-) | 16:49 |
*** pshedimb has joined #openstack-ironic | 16:52 | |
*** krtaylor has joined #openstack-ironic | 16:52 | |
*** ChubYann has joined #openstack-ironic | 16:53 | |
*** penick has joined #openstack-ironic | 16:55 | |
*** ralonsoh_ has quit IRC | 16:55 | |
*** ralonsoh_ has joined #openstack-ironic | 16:55 | |
*** Romanenko_K has quit IRC | 16:56 | |
*** e0ne has joined #openstack-ironic | 16:58 | |
*** vishwana_ has joined #openstack-ironic | 16:59 | |
*** vsaienko has quit IRC | 16:59 | |
*** sdake_ is now known as sdake | 17:01 | |
*** baoli_ has quit IRC | 17:01 | |
*** baoli has joined #openstack-ironic | 17:02 | |
*** yibo has quit IRC | 17:02 | |
*** vishwanathj has quit IRC | 17:03 | |
*** ralonsoh_ has quit IRC | 17:03 | |
*** vsaienko has joined #openstack-ironic | 17:03 | |
*** bdeschenes has joined #openstack-ironic | 17:04 | |
*** Goneri has quit IRC | 17:05 | |
*** baoli has quit IRC | 17:07 | |
*** yibo has joined #openstack-ironic | 17:09 | |
*** vsaienko has quit IRC | 17:10 | |
*** vsaienko has joined #openstack-ironic | 17:10 | |
*** vsaienko has quit IRC | 17:12 | |
*** e0ne has quit IRC | 17:12 | |
*** Goneri has joined #openstack-ironic | 17:18 | |
*** pshedimb has quit IRC | 17:18 | |
*** mgoddard has quit IRC | 17:20 | |
* jroll steps out for a bit | 17:21 | |
*** e0ne has joined #openstack-ironic | 17:22 | |
*** dnikishov has quit IRC | 17:23 | |
*** e0ne has quit IRC | 17:23 | |
*** pshedimb has joined #openstack-ironic | 17:26 | |
*** trown is now known as trown|lunch | 17:29 | |
*** pshedimb has quit IRC | 17:31 | |
*** anush has quit IRC | 17:34 | |
*** hj-hpe has joined #openstack-ironic | 17:36 | |
*** ccamacho has quit IRC | 17:38 | |
*** jaosorior has quit IRC | 17:40 | |
*** jaosorior has joined #openstack-ironic | 17:40 | |
*** amotoki has quit IRC | 17:49 | |
*** Sukhdev has joined #openstack-ironic | 17:49 | |
*** adam_g` is now known as adam_g | 17:55 | |
*** adam_g has quit IRC | 17:55 | |
*** adam_g has joined #openstack-ironic | 17:55 | |
*** cfarquhar has joined #openstack-ironic | 17:56 | |
cfarquhar | hi ironic. I'm running into a strange issue in mitaka. If anyone can point me in the right troubleshooting direction I would appreciate it. https://bugs.launchpad.net/openstack-ansible/+bug/1632436/comments/8 | 17:59 |
openstack | Launchpad bug 1632436 in openstack-ansible "os_ironic uses wrong keystone_authtoken config keys" [Undecided,In progress] - Assigned to Charles Farquhar (cfarquhar) | 17:59 |
*** mgoddard has joined #openstack-ironic | 17:59 | |
JayF | cfarquhar: one thing I thought of when reading that is that ironic-api and ironic-conductor use keystone auth for different things; ironic-api usually to validate tokens (I don't think much else?), ironic-conductor to interact with neutron, glance, etc | 18:00 |
JayF | cfarquhar: I'm heading out to lunch, but there are some folks here who worked with that stuff frequently | 18:00 |
JayF | devananda: if you have a moment, that's very relevant to some of the keystone changes that merged in with your policy work ^ | 18:00 |
cfarquhar | cool thanks JayF ... I'm halfway in a meeting atm anyway | 18:01 |
*** sambetts|afk has quit IRC | 18:02 | |
JayF | oh, that's mitaka. heh. | 18:02 |
JayF | so none of the keystone v3/policy work in there | 18:02 |
*** pshedimb has joined #openstack-ironic | 18:04 | |
*** sambetts_ has joined #openstack-ironic | 18:05 | |
*** aNuposic has joined #openstack-ironic | 18:05 | |
*** rbrud has quit IRC | 18:05 | |
*** vsaienko has joined #openstack-ironic | 18:08 | |
*** pshedimb has quit IRC | 18:11 | |
*** Sukhdev has quit IRC | 18:11 | |
*** pshedimb has joined #openstack-ironic | 18:11 | |
*** baoli has joined #openstack-ironic | 18:12 | |
*** aNuposic has quit IRC | 18:12 | |
*** Sukhdev has joined #openstack-ironic | 18:13 | |
*** ekoorjr has joined #openstack-ironic | 18:17 | |
*** sdake_ has joined #openstack-ironic | 18:19 | |
*** harlowja_still_a has joined #openstack-ironic | 18:20 | |
*** sdake has quit IRC | 18:20 | |
*** harlowja_at_home has quit IRC | 18:23 | |
*** vishwana_ has quit IRC | 18:27 | |
*** vishwanathj has joined #openstack-ironic | 18:28 | |
*** aNuposic has joined #openstack-ironic | 18:28 | |
*** trown|lunch is now known as trown | 18:30 | |
*** harlowja_still_a has quit IRC | 18:30 | |
*** e0ne has joined #openstack-ironic | 18:30 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-inspector: Correct conf.py missing space https://review.openstack.org/386131 | 18:31 |
*** adu has joined #openstack-ironic | 18:32 | |
*** milan has quit IRC | 18:32 | |
*** e0ne has quit IRC | 18:37 | |
jroll | cfarquhar: can you run a pip freeze in that environment? | 18:39 |
*** rbrud has joined #openstack-ironic | 18:39 | |
*** milan has joined #openstack-ironic | 18:40 | |
*** e0ne has joined #openstack-ironic | 18:41 | |
jroll | cfarquhar: I'm suspecting some newer keystone library or e.g. neutron client that doesn't accept admin_user anymore, this is a config from mitaka gate that only has admin_user etc http://logs.openstack.org/35/354735/4/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa/eba987c/logs/etc/ironic/ironic.conf.txt.gz | 18:42 |
*** e0ne has quit IRC | 18:45 | |
*** milan has quit IRC | 18:47 | |
*** milan has joined #openstack-ironic | 18:48 | |
*** michauds has joined #openstack-ironic | 18:49 | |
*** vsaienko has quit IRC | 18:51 | |
*** narasimha_SV has joined #openstack-ironic | 18:52 | |
narasimha_SV | http://paste.openstack.org/show/585649/ | 18:53 |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Add API for listing all introspection statuses https://review.openstack.org/344921 | 18:53 |
narasimha_SV | while pxe booting deployment is failing at log permissions | 18:53 |
*** mjturek has quit IRC | 18:54 | |
*** baoli has quit IRC | 18:57 | |
*** baoli has joined #openstack-ironic | 18:58 | |
*** yuriyz|2 has joined #openstack-ironic | 19:00 | |
*** sdake has joined #openstack-ironic | 19:01 | |
*** adu has quit IRC | 19:01 | |
*** joprovost has quit IRC | 19:02 | |
*** sdake_ has quit IRC | 19:04 | |
*** nicodemos has quit IRC | 19:05 | |
*** nicodemos has joined #openstack-ironic | 19:05 | |
*** yuriyz|2 has quit IRC | 19:11 | |
*** mjturek has joined #openstack-ironic | 19:11 | |
*** dprince has quit IRC | 19:12 | |
jroll | fyi, took care of that openstack-ansible bug in their channel | 19:16 |
JayF | jroll: I'm not in that channel, do you have a tldr | 19:19 |
jroll | JayF: tl;dr keystone v3 isn't supported in mitaka ironic, but for logs, http://eavesdrop.openstack.org/irclogs/%23openstack-ansible/%23openstack-ansible.2016-10-13.log.html#t2016-10-13T18:53:41 | 19:20 |
JayF | ty | 19:22 |
*** dprince has joined #openstack-ironic | 19:26 | |
*** Goneri has quit IRC | 19:29 | |
narasimha_SV | jroll: http://paste.openstack.org/show/585650/ | 19:29 |
jroll | narasimha_SV: I've seen your error all week, if I had an answer for you I would tell you, sorry | 19:29 |
narasimha_SV | ok | 19:30 |
*** milan has quit IRC | 19:50 | |
*** milan has joined #openstack-ironic | 19:50 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Replace parse_root_device_hints with the ironic-lib version one https://review.openstack.org/386204 | 19:57 |
*** hamzy has quit IRC | 20:00 | |
*** penick has quit IRC | 20:00 | |
*** penick has joined #openstack-ironic | 20:04 | |
*** clenimar has joined #openstack-ironic | 20:05 | |
*** jcoufal_ has quit IRC | 20:06 | |
*** francoblanco has quit IRC | 20:18 | |
*** openstackgerrit has quit IRC | 20:19 | |
*** francoblanco has joined #openstack-ironic | 20:19 | |
*** openstackgerrit has joined #openstack-ironic | 20:19 | |
*** pshedimb has quit IRC | 20:24 | |
*** pshedimb has joined #openstack-ironic | 20:24 | |
*** nicodemos has quit IRC | 20:25 | |
*** nicodemos has joined #openstack-ironic | 20:25 | |
*** joprovost has joined #openstack-ironic | 20:26 | |
*** pshedimb has quit IRC | 20:29 | |
lucasagomes | I'm calling it a day! Have a great evening all | 20:29 |
lucasagomes | see ya tomorrow | 20:29 |
*** lucasagomes is now known as lucas-afk | 20:30 | |
JayF | o/ | 20:35 |
*** baoli has quit IRC | 20:36 | |
*** baoli has joined #openstack-ironic | 20:36 | |
*** mjturek has quit IRC | 20:39 | |
*** baoli has quit IRC | 20:39 | |
*** baoli has joined #openstack-ironic | 20:39 | |
*** narasimha_SV has quit IRC | 20:40 | |
*** sdake has quit IRC | 20:44 | |
openstackgerrit | Merged openstack/ironic: Avoid name errors in oneview periodics https://review.openstack.org/374989 | 20:44 |
*** baoli has quit IRC | 20:46 | |
*** baoli has joined #openstack-ironic | 20:46 | |
*** yuriyz|2 has joined #openstack-ironic | 20:51 | |
jroll | JayF: want an easy one? https://review.openstack.org/#/c/386015/ | 20:51 |
patchbot | patch 386015 - ironic-specs - 'iPXE to use Swift Temp URLs' done in 6.2 | 20:51 |
*** mjturek has joined #openstack-ironic | 20:51 | |
JayF | done | 20:52 |
*** baoli_ has joined #openstack-ironic | 20:54 | |
*** baoli has quit IRC | 20:54 | |
openstackgerrit | Merged openstack/ironic-specs: 'iPXE to use Swift Temp URLs' done in 6.2 https://review.openstack.org/386015 | 20:54 |
jroll | thanks! | 20:55 |
JayF | np | 20:56 |
*** penick has quit IRC | 20:58 | |
*** sdake has joined #openstack-ironic | 20:59 | |
*** trown is now known as trown|outtypewww | 21:02 | |
*** pshedimb has joined #openstack-ironic | 21:02 | |
*** yuriyz|2 has quit IRC | 21:04 | |
*** penick has joined #openstack-ironic | 21:05 | |
*** baoli_ has quit IRC | 21:12 | |
*** baoli has joined #openstack-ironic | 21:13 | |
*** absubram has quit IRC | 21:18 | |
*** pshedimb has quit IRC | 21:21 | |
*** dprince has quit IRC | 21:21 | |
*** krtaylor has quit IRC | 21:23 | |
*** baoli_ has joined #openstack-ironic | 21:24 | |
*** baoli has quit IRC | 21:25 | |
JayF | jroll: want an easy one? https://review.openstack.org/#/c/385476/ | 21:26 |
patchbot | patch 385476 - ironic - A few fixes in Multitenancy document | 21:26 |
*** baoli_ has quit IRC | 21:28 | |
*** baoli has joined #openstack-ironic | 21:29 | |
JayF | question around https://review.openstack.org/#/c/376978/ cc: mariojv | 21:30 |
*** Goneri has joined #openstack-ironic | 21:30 | |
patchbot | patch 376978 - ironic - Add "agent_last_heartbeat_datetime" field to agent... | 21:30 |
*** michauds has quit IRC | 21:30 | |
JayF | how do we deprecate a field that's not used programatically by anything upstream? | 21:31 |
rloo | JayF: what field? | 21:31 |
JayF | We can't know to print a log because there's no way to know when someone is using it | 21:31 |
JayF | rloo: agent_last_heartbeat (being replaced with agent_last_heartbeat_datetime) | 21:31 |
JayF | in driver_info{} for agent drivers | 21:31 |
jroll | JayF: done | 21:32 |
* jroll out for the night | 21:32 | |
rloo | JayF: oh, i see the link above your question. sec let me look | 21:32 |
mariojv | it's driver_internal_info which has even less of a guarantee imo | 21:32 |
*** baoli has quit IRC | 21:33 | |
jroll | we don't guarantee backward compat for keys in our json fields | 21:33 |
jroll | never have | 21:33 |
mariojv | we could just change it and not violate any "contract" with operators i think | 21:33 |
rloo | JayF, mariojv: yeah, what jroll said | 21:33 |
mariojv | i was just being nice, since i know that breaks downstream onmetal tooling :P | 21:33 |
*** baoli has joined #openstack-ironic | 21:33 | |
rloo | mariojv: as long as our code works | 21:33 |
mariojv | i'm fine with just switching it over to DT though | 21:33 |
JayF | mariojv: we don't get to be special flowers :P | 21:33 |
jroll | now, it'd be nice to do a release note and leave it for a cycle | 21:33 |
jroll | because it's easy and we can | 21:33 |
jroll | but don't worry about deprecation warnings | 21:33 |
JayF | in that case, mariojv's patch as it sits handles it that way8 | 21:34 |
mariojv | yeah, it doesn't log anything about deprecation | 21:34 |
* jroll really gone now | 21:34 | |
rloo | mariojv: if nothing is using it, why even bother converting it to datetime? | 21:34 |
*** jheroux has quit IRC | 21:34 | |
* mariojv shrugs | 21:34 | |
mariojv | i would use it in logs and tooling | 21:34 |
JayF | That's a bad answer :P | 21:34 |
mariojv | but if no one else will use it, we can just get rid of it i think | 21:34 |
JayF | I agree, it's a useful value to know as an operator | 21:35 |
JayF | and that Ironic /should/ use it for some stuff | 21:35 |
JayF | but we don't at all right now | 21:35 |
rloo | mariojv, JayF: umm... the agent heartbeats at some regular basis? isn't there supposed to be code on the conductor side to do something if it doesn't hear from the agent after some period of time? | 21:35 |
JayF | rloo: no such code exists | 21:35 |
JayF | rloo: although calling it a "heartbeat" would certainly make you think so... | 21:35 |
rloo | JayF: but should exist, right? | 21:35 |
JayF | rloo: I would assert so, yes | 21:36 |
JayF | I have a spec going down that rabbithole right now | 21:36 |
JayF | sec | 21:36 |
rloo | JayF, if it did exist, we'd need to keep track of the time of the most recent heartbeat | 21:36 |
JayF | https://review.openstack.org/#/c/351943/ needs an update though | 21:36 |
patchbot | patch 351943 - ironic-specs - Spec for Long Running Ramdisk Support | 21:36 |
openstackgerrit | Merged openstack/ironic: A few fixes in Multitenancy document https://review.openstack.org/385476 | 21:36 |
rloo | JayF, mariojv: I think we should delete it if none of our code uses it, and add it whenever it gets used | 21:37 |
aNuposic | JayF: rloo: I remember seeing "Too many heartbeats missed error" on missing many heartbeats | 21:38 |
JayF | aNuposic: that's a different heartbeat; conductors heartbeat to each other via the DB | 21:38 |
aNuposic | JayF: Ah okay | 21:38 |
JayF | rloo: I think you're right, and mariojv, the ops-use-case for it is solely for our downstream stuff that boots agents for long periods of time | 21:38 |
rloo | mariojv: my other question is whether our other fields with datetime info, are named with '*_datetime' | 21:38 |
mariojv | that's true JayF - rloo i don't think so, i just picked that kind of randomly | 21:39 |
JayF | I suspect that name was picked just to have a different name than the one that /isn't/ datetime | 21:39 |
rloo | mariojv: ok. i'll add a comment (wow, adding a comment by just reading the commit msg :)) | 21:39 |
JayF | mariojv: mind updating the patch to just kill the badly formatted info? Make sure the release note is noisy about it :D | 21:39 |
mariojv | i thought we didn't want to add release notes for internal fields | 21:40 |
mariojv | i don't mind re-adding it, and being nice, though :) | 21:40 |
JayF | mariojv: printing deprecation warnings is different than a heads up in the release notes imo | 21:40 |
mariojv | k | 21:41 |
* mariojv will update | 21:41 | |
*** baoli has quit IRC | 21:41 | |
*** rloo has quit IRC | 21:44 | |
*** baoli has joined #openstack-ironic | 21:44 | |
*** rloo has joined #openstack-ironic | 21:44 | |
JayF | another easy one https://review.openstack.org/#/c/385747/ (requirements update for client) | 21:49 |
patchbot | patch 385747 - python-ironicclient - Updated from global requirements | 21:49 |
*** baoli_ has joined #openstack-ironic | 21:50 | |
*** baoli has quit IRC | 21:51 | |
rloo | JayF: if you're looking for a fairly easy one to review: https://review.openstack.org/#/c/366275/ | 21:53 |
patchbot | patch 366275 - python-ironicclient - OSC plugin support microversions 1.21 & 1.22 | 21:53 |
JayF | rloo: I'm basicaly reviewing until I reach the bottom of meaningful things I can review. We just seem to have a huge backlog. | 21:53 |
*** baoli_ has quit IRC | 21:54 | |
rloo | JayF: yes, we have a huge backlog. I'm overwhelmed by it.. | 21:54 |
*** baoli has joined #openstack-ironic | 21:54 | |
JayF | I just take the pac man approach | 21:54 |
JayF | take lots of pills and march forward until you start seeing blue | 21:54 |
JayF | I mean, uh, one small bite at a time! | 21:54 |
rloo | JayF: my latest strategy was to work from new to old but i am not getting far that way | 21:54 |
JayF | I use deva's review dashboard, and generally work my way down... the only thing that discourages me there is how many of the useless-one-character-fixes I see | 21:55 |
openstackgerrit | Merged openstack/python-oneviewclient: Updated from global requirements https://review.openstack.org/377559 | 21:56 |
rloo | JayF: oh, you can just ignore those :) | 21:56 |
*** bdeschenes has quit IRC | 21:56 | |
JayF | I mean, by the time I get them open, see they are small, it's only |-| more effort to +2A if they don't conflict with anything | 21:56 |
JayF | it's just like ... we need help on meaningful things, please don't just try to check a metrics box and keep going | 21:57 |
*** baoli has quit IRC | 21:59 | |
*** bfournie has quit IRC | 22:00 | |
*** baoli has joined #openstack-ironic | 22:00 | |
*** penick has quit IRC | 22:01 | |
rloo | JayF: i hear you. Hence my discussion earlier today... | 22:01 |
rloo | JayF: the other thing is that the commit should have a bit more info than just 'typo'. sometimes they aren't even typos, they are grammatical corrections. anyway, moving along... | 22:02 |
*** penick has joined #openstack-ironic | 22:05 | |
openstackgerrit | Prudhvi Rao Shedimbi proposed openstack/ironic-lib: Changed message attribute to _msg_fmt https://review.openstack.org/386251 | 22:10 |
*** rajinir has quit IRC | 22:16 | |
*** baoli has quit IRC | 22:16 | |
openstackgerrit | Merged openstack/python-oneviewclient: Remove cpus validation due inconsistency with inspection cpus count https://review.openstack.org/375089 | 22:16 |
*** mgoddard has quit IRC | 22:18 | |
*** Goneri has quit IRC | 22:18 | |
openstackgerrit | Stephanie Miller proposed openstack/bifrost: Convert ironic.conf to template generated https://review.openstack.org/305634 | 22:18 |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Mock ironic-lib properly in test_deploy_utils https://review.openstack.org/386255 | 22:19 |
*** baoli has joined #openstack-ironic | 22:19 | |
*** [1]cdearborn has quit IRC | 22:20 | |
*** penick has quit IRC | 22:22 | |
*** baoli_ has joined #openstack-ironic | 22:22 | |
*** baoli has quit IRC | 22:24 | |
jlvillal | mariojv: That looks like it took a long time... | 22:25 |
mariojv | jlvillal: nah, maybe a bit over an hour? | 22:25 |
mariojv | most of it was just duplicate code | 22:25 |
openstackgerrit | Merged openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/377446 | 22:26 |
jlvillal | mariojv: You're fast. Nice :) | 22:28 |
*** joprovost has quit IRC | 22:28 | |
jlvillal | mariojv: I left a comment on how I think it is horrific. | 22:30 |
jlvillal | mariojv: Not what you did but the existing code :) | 22:30 |
mariojv | jlvillal: yeah, i agree tbh | 22:30 |
jlvillal | :) | 22:31 |
mariojv | with decorators that explodes to like 15 parameters though | 22:31 |
mariojv | which is also horrific | 22:31 |
mariojv | :( | 22:31 |
jlvillal | Does it really need to mock all of those? Sad that it does. | 22:31 |
openstackgerrit | Merged openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/385747 | 22:32 |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: No need to clear "target_provision_state" again from conductor https://review.openstack.org/380951 | 22:32 |
mariojv | yeah, a lot of these are calling out to external tools it seems | 22:33 |
mariojv | a lot of iscsiadm commands | 22:34 |
mariojv | get_dev might not need to be mocked | 22:34 |
mariojv | maybe... | 22:35 |
jlvillal | yuriyz: So late! | 22:35 |
mariojv | i'm fine with refactoring it, if 15 parameters is better than that weird mock trickery :P | 22:35 |
jlvillal | mariojv: dtantsur|afk probably wouldn't approve but maybe: def _test_deploy_partion_image(self, **kwargs) | 22:35 |
mariojv | ah, yeah, they all are kwargs | 22:36 |
jlvillal | Then make sure any kwarg passed in is in the allowed list. And any not set are set to the default. | 22:36 |
mariojv | yeah, i think that's a lot better | 22:36 |
* mariojv makes a note of it | 22:36 | |
jlvillal | default_values = {'foo': 1, 'spam': 2} | 22:36 |
jlvillal | not_allowed_args = set(kwargs) - set(default_values) | 22:37 |
jlvillal | if not_allowed_args: Bad bad bad! | 22:37 |
jlvillal | default_values.update(kwargs) | 22:37 |
jlvillal | or something like that. | 22:37 |
mariojv | ++ | 22:37 |
jlvillal | And then if the mocks get added maybe not so big | 22:37 |
openstackgerrit | Merged openstack/python-ironicclient: Set OSC default baremetal api version as in ironicclient https://review.openstack.org/385386 | 22:39 |
*** baoli_ has quit IRC | 22:47 | |
openstackgerrit | Merged openstack/ironic-inspector: Fix a typo in base.py https://review.openstack.org/385705 | 22:51 |
*** zackf has quit IRC | 22:53 | |
*** vishwanathj has quit IRC | 22:59 | |
*** ekoorjr has quit IRC | 23:00 | |
openstackgerrit | Merged openstack/ironic-specs: Collect Ironic agent ramdisk logs https://review.openstack.org/337223 | 23:12 |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Remove "agent_last_heartbeat" internal field from agent drivers https://review.openstack.org/376978 | 23:13 |
*** igordcard has quit IRC | 23:14 | |
*** igordcard has joined #openstack-ironic | 23:16 | |
*** awiddersheim has joined #openstack-ironic | 23:18 | |
*** rbrud has quit IRC | 23:23 | |
openstackgerrit | Merged openstack/ironic-inspector: Correct conf.py missing space https://review.openstack.org/386131 | 23:26 |
*** cfarquhar has quit IRC | 23:32 | |
*** [1]cdearborn has joined #openstack-ironic | 23:41 | |
*** bfournie has joined #openstack-ironic | 23:47 | |
*** sdake_ has joined #openstack-ironic | 23:51 | |
*** sdake has quit IRC | 23:52 | |
*** Sukhdev has quit IRC | 23:55 | |
*** hoangcx_ has joined #openstack-ironic | 23:56 | |
*** aNuposic has quit IRC | 23:58 | |
*** hoangcx has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!