*** rama_y has quit IRC | 00:01 | |
*** natorious has joined #openstack-ironic | 00:01 | |
*** dprince has joined #openstack-ironic | 00:01 | |
*** natorious has quit IRC | 00:05 | |
*** hamzy has joined #openstack-ironic | 00:08 | |
*** natorious has joined #openstack-ironic | 00:17 | |
*** sdake_ is now known as sdake | 00:22 | |
*** tdurakov has quit IRC | 00:23 | |
*** tdurakov has joined #openstack-ironic | 00:23 | |
*** JayF has quit IRC | 00:24 | |
*** fungi has quit IRC | 00:26 | |
*** aNuposic has joined #openstack-ironic | 00:30 | |
*** fungi has joined #openstack-ironic | 00:30 | |
*** JayF has joined #openstack-ironic | 00:37 | |
*** rbrud has quit IRC | 00:38 | |
*** mtanino has quit IRC | 00:40 | |
*** dprince has quit IRC | 00:40 | |
*** mgagne has quit IRC | 00:40 | |
*** mgagne has joined #openstack-ironic | 00:45 | |
*** hoangcx has joined #openstack-ironic | 00:45 | |
*** sdake has quit IRC | 00:46 | |
*** baoli has quit IRC | 00:49 | |
*** amotoki has joined #openstack-ironic | 00:49 | |
*** baoli has joined #openstack-ironic | 00:50 | |
*** amotoki has quit IRC | 00:54 | |
*** mgoddard has joined #openstack-ironic | 00:54 | |
*** baoli has quit IRC | 00:59 | |
*** tuanluong has joined #openstack-ironic | 01:00 | |
*** sdake has joined #openstack-ironic | 01:01 | |
*** baoli has joined #openstack-ironic | 01:02 | |
*** phuongnh has joined #openstack-ironic | 01:03 | |
*** syed_ has quit IRC | 01:04 | |
*** xuao has joined #openstack-ironic | 01:09 | |
*** aNuposic has quit IRC | 01:11 | |
tuanluong | Hello ironic | 01:16 |
---|---|---|
*** mgoddard has quit IRC | 01:17 | |
*** baoli has quit IRC | 01:20 | |
*** baoli has joined #openstack-ironic | 01:21 | |
*** phuongnh has quit IRC | 01:25 | |
*** tuanluong has quit IRC | 01:25 | |
*** hoangcx has quit IRC | 01:25 | |
*** phuongnh has joined #openstack-ironic | 01:26 | |
*** [1]cdearborn has quit IRC | 01:27 | |
*** tuanluong has joined #openstack-ironic | 01:29 | |
*** rloo has quit IRC | 01:36 | |
*** rloo has joined #openstack-ironic | 01:37 | |
*** rloo has quit IRC | 01:58 | |
*** Guest50630 has quit IRC | 02:03 | |
*** baoli has quit IRC | 02:05 | |
*** zigo has joined #openstack-ironic | 02:05 | |
*** zigo is now known as Guest59427 | 02:06 | |
*** zhangjl has joined #openstack-ironic | 02:13 | |
*** mtanino has joined #openstack-ironic | 02:16 | |
*** gcb has joined #openstack-ironic | 02:22 | |
*** yuanying has quit IRC | 02:48 | |
*** amit213 has quit IRC | 02:48 | |
*** amit213 has joined #openstack-ironic | 02:49 | |
*** amotoki has joined #openstack-ironic | 02:51 | |
*** amotoki has quit IRC | 02:51 | |
*** amotoki has joined #openstack-ironic | 02:51 | |
*** MattMan has quit IRC | 02:53 | |
*** MattMan has joined #openstack-ironic | 02:53 | |
*** rama_y has joined #openstack-ironic | 02:53 | |
*** thrash is now known as thrash|g0ne | 02:54 | |
*** sdake has quit IRC | 02:57 | |
*** yuanying has joined #openstack-ironic | 02:58 | |
*** zhangjl1 has joined #openstack-ironic | 02:58 | |
*** amotoki has quit IRC | 02:59 | |
*** zhangjl has quit IRC | 02:59 | |
*** baoli has joined #openstack-ironic | 02:59 | |
*** yuanying has quit IRC | 03:02 | |
*** baoli has quit IRC | 03:05 | |
*** odyssey4me has quit IRC | 03:14 | |
*** odyssey4me has joined #openstack-ironic | 03:15 | |
*** sdake has joined #openstack-ironic | 03:18 | |
*** hoangcx has joined #openstack-ironic | 03:21 | |
*** amotoki has joined #openstack-ironic | 03:24 | |
*** vikrant has joined #openstack-ironic | 03:26 | |
*** yuanying has joined #openstack-ironic | 03:30 | |
*** yuanying has quit IRC | 03:33 | |
*** yuanying has joined #openstack-ironic | 03:36 | |
*** baoli has joined #openstack-ironic | 03:38 | |
*** amotoki has quit IRC | 03:40 | |
*** yuanying has quit IRC | 03:45 | |
*** anush_ has joined #openstack-ironic | 03:45 | |
*** yuanying has joined #openstack-ironic | 03:49 | |
*** amotoki has joined #openstack-ironic | 03:53 | |
*** baoli has quit IRC | 03:53 | |
*** baoli has joined #openstack-ironic | 03:54 | |
*** baoli has quit IRC | 03:54 | |
*** anush_ has quit IRC | 03:55 | |
*** baoli has joined #openstack-ironic | 03:55 | |
*** amotoki has quit IRC | 03:59 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Fix setting persistent boot device does not work https://review.openstack.org/376322 | 04:02 |
*** links has joined #openstack-ironic | 04:10 | |
*** jistr has quit IRC | 04:15 | |
*** jistr has joined #openstack-ironic | 04:16 | |
*** alex_xu has quit IRC | 04:18 | |
*** alex_xu has joined #openstack-ironic | 04:20 | |
*** baoli has quit IRC | 04:20 | |
*** rama_y has quit IRC | 04:21 | |
*** rama_y has joined #openstack-ironic | 04:22 | |
*** natorious has quit IRC | 04:24 | |
openstackgerrit | Xu Ao proposed openstack/ironic: Make the configuration item "api_url" have better adaptability https://review.openstack.org/385261 | 04:26 |
*** natorious has joined #openstack-ironic | 04:27 | |
*** vishwanathj has quit IRC | 04:32 | |
*** mtanino has quit IRC | 04:39 | |
*** rama_y has quit IRC | 04:45 | |
*** karthiks has joined #openstack-ironic | 04:54 | |
*** jaosorior has joined #openstack-ironic | 05:02 | |
*** rama_y has joined #openstack-ironic | 05:05 | |
*** alex_xu has quit IRC | 05:09 | |
*** alex_xu has joined #openstack-ironic | 05:11 | |
*** appu103_ has joined #openstack-ironic | 05:13 | |
*** appu103_ is now known as aparnav | 05:16 | |
*** rama_y has quit IRC | 05:18 | |
*** amotoki has joined #openstack-ironic | 05:19 | |
*** ChubYann has quit IRC | 05:25 | |
*** e0ne has joined #openstack-ironic | 05:27 | |
*** e0ne has quit IRC | 05:34 | |
*** sabeen has quit IRC | 05:37 | |
*** sabeen has joined #openstack-ironic | 05:37 | |
*** e0ne has joined #openstack-ironic | 05:38 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/pyghmi: Remove xrange for run both Python 2 and Python 3 https://review.openstack.org/385281 | 05:40 |
*** sabeen has quit IRC | 05:42 | |
*** sabeen has joined #openstack-ironic | 05:42 | |
*** e0ne has quit IRC | 05:44 | |
*** amotoki has quit IRC | 05:48 | |
*** e0ne has joined #openstack-ironic | 05:48 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Ipmitool power driver for soft reboot and soft power off https://review.openstack.org/216738 | 05:52 |
*** links has quit IRC | 05:54 | |
*** e0ne has quit IRC | 06:01 | |
*** e0ne has joined #openstack-ironic | 06:05 | |
*** jtomasek_ has joined #openstack-ironic | 06:05 | |
*** links has joined #openstack-ironic | 06:10 | |
*** mjura has joined #openstack-ironic | 06:11 | |
*** e0ne has quit IRC | 06:14 | |
*** makowals has joined #openstack-ironic | 06:15 | |
*** e0ne has joined #openstack-ironic | 06:18 | |
*** jtomasek|bbl is now known as jtomasek | 06:20 | |
*** jtomasek_ has quit IRC | 06:24 | |
*** e0ne has quit IRC | 06:27 | |
*** e0ne has joined #openstack-ironic | 06:32 | |
*** amotoki has joined #openstack-ironic | 06:34 | |
*** rcernin has joined #openstack-ironic | 06:36 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic-python-agent: Use ironic-lib to create configdrive https://review.openstack.org/296466 | 06:37 |
*** hshiina has joined #openstack-ironic | 06:41 | |
*** skazi has quit IRC | 06:41 | |
*** e0ne has quit IRC | 06:43 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 06:48 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: iRMC power driver for soft reboot and soft power off https://review.openstack.org/216743 | 06:50 |
*** e0ne has joined #openstack-ironic | 06:52 | |
*** e0ne has quit IRC | 06:54 | |
*** skazi has joined #openstack-ironic | 06:54 | |
*** e0ne has joined #openstack-ironic | 06:58 | |
*** e0ne has quit IRC | 07:00 | |
*** e0ne has joined #openstack-ironic | 07:04 | |
*** tesseract has joined #openstack-ironic | 07:06 | |
*** tesseract is now known as Guest24890 | 07:06 | |
*** e0ne has quit IRC | 07:06 | |
*** _vsaienko has joined #openstack-ironic | 07:11 | |
*** _vsaienko has quit IRC | 07:12 | |
*** vsaienko has joined #openstack-ironic | 07:12 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 07:16 |
*** sdake has quit IRC | 07:22 | |
*** Guest59427 is now known as zigo | 07:33 | |
*** skazi has quit IRC | 07:36 | |
*** dtantsur|afk is now known as dtantsur | 07:38 | |
dtantsur | Morning Ironic | 07:38 |
pas-ha | morning dtantsur and all other Ironicers :) | 07:42 |
vsaienk0 | morning dtantsur, sambetts, lucasagomes, pas-ha, vdrok, milan and all Ironic'ers! | 07:43 |
*** vsaienko has quit IRC | 07:43 | |
*** rcernin has quit IRC | 07:45 | |
*** skazi has joined #openstack-ironic | 07:47 | |
*** rcernin has joined #openstack-ironic | 07:49 | |
*** milan|afk has joined #openstack-ironic | 07:53 | |
aarefiev | morning | 07:53 |
milan|afk | morning aarefiev! :) | 07:54 |
*** milan|afk is now known as milan | 07:54 | |
aarefiev | morning milan | 07:54 |
milan | and morning Ironic! :) | 07:54 |
milan | aarefiev, w/r the API WG patch I pinged cdent yesterday and he promised to have a resolution by Thursday EOB | 07:56 |
milan | https://review.openstack.org/#/c/383862 | 07:56 |
patchbot | patch 383862 - api-wg - Specify time intervals based filtering queries | 07:56 |
*** stendulker has joined #openstack-ironic | 07:57 | |
aarefiev | milan: nice, I'm looking https://review.openstack.org/#/c/344921/ now | 07:59 |
patchbot | patch 344921 - ironic-inspector - Add API for listing all introspection statuses | 07:59 |
*** zzzeek has quit IRC | 08:00 | |
milan | aarefiev, cool, thx | 08:00 |
milan | I'm a bit afraid we'll have to rework the api_tools.py once that WG patch lands | 08:00 |
*** zzzeek has joined #openstack-ironic | 08:00 | |
milan | atm it supports just equality operation | 08:00 |
milan | but will need to implement more operators | 08:01 |
*** sabeen has quit IRC | 08:02 | |
aarefiev | milan: you mean in separate patch with filters, intervals | 08:03 |
milan | aarefiev, yeah | 08:03 |
milan | but I'm afraid the api between node_cache.py and api_tools.py will have to change | 08:03 |
* milan pastes | 08:03 | |
milan | get_node_list(ironic=None, marker=None, limit=CONF.api.max_limit) | 08:04 |
milan | get_node_list(ironic=lambda x: None, marker=lambda x: None, limit=lambda x: CONF.api.max_limit) | 08:05 |
milan | to be able to process operations | 08:05 |
aarefiev | ok, I open only doc, thought we talk about only limit here | 08:05 |
milan | yeah, I think this patch can land | 08:05 |
milan | but that change is inevitable later | 08:05 |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 08:05 |
yuriyz | morning ironic | 08:05 |
milan | we need to pass the operation somehow | 08:06 |
milan | and I think it's better parsed in the api_tools than in node_cache | 08:06 |
milan | thus the functions | 08:06 |
milan | morning yuriyz | 08:06 |
milan | :) | 08:06 |
aarefiev | morning yuriyz | 08:06 |
xuao | morning ~~~ | 08:07 |
aarefiev | milan: it's internal staff, we can easy change it | 08:07 |
milan | morning xuao! :) | 08:07 |
milan | aarefiev, ack | 08:07 |
milan | it's just FYI ;) | 08:07 |
milan | and my morning brain dump | 08:08 |
milan | :D | 08:08 |
aarefiev | :) | 08:08 |
*** jpich has joined #openstack-ironic | 08:10 | |
*** e0ne has joined #openstack-ironic | 08:10 | |
milan | actually should have been get_node_list(ironic=lambda: None, marker=lambda: None, limit=lambda: CONF.api.max_limit) as those are nullary ops, then finished_at=lambda x: db.Node.finished_at <OP> x for the "unary" operations but that's a further patch story ;) | 08:12 |
*** athomas has joined #openstack-ironic | 08:16 | |
*** vsaienko has joined #openstack-ironic | 08:21 | |
openstackgerrit | Xu Ao proposed openstack/ironic: Make the configuration item "api_url" have better adaptability https://review.openstack.org/385261 | 08:25 |
*** _vsaienko has joined #openstack-ironic | 08:26 | |
*** vsaienko has quit IRC | 08:26 | |
aarefiev | milan: why do you need lambda here | 08:27 |
aarefiev | where they are used as callable | 08:28 |
milan | aarefiev, I'm not sure how else one should transport the parsed operation | 08:30 |
aarefiev | as usual values, I don't see how you will use them, so hard to say | 08:31 |
milan | aarefiev, maybe the lambdas won't need a parameter: finished_at=lambda: db.Node.finished_at >= 15:30 | 08:32 |
milan | as an example ^ | 08:32 |
milan | so if there was a query, the parser would compile it into such a lambda | 08:32 |
milan | and the node_list would use it in the filter as an expression | 08:33 |
*** amotoki has quit IRC | 08:33 | |
milan | cause calling it would return the SQLAlchemy expression stemming in the fact that there's a comparison of a column with a value | 08:33 |
*** Romanenko_K has joined #openstack-ironic | 08:34 | |
milan | SQLAlchemy then gives part of a string like: "...SELECT .... WHERE ... finished_at >= 15:30" | 08:34 |
milan | * mind 15:30 is seconds since epoch but that's also up to the parser to figure out IMO | 08:35 |
milan | * "?finished_at=gt:15:30" ~> finished_at=lambda: db.Node.finished_at >= <secs since Epoch> ~> DB filter param | 08:37 |
* milan just realised that only filtering and sorting queries need these lambdas | 08:38 | |
milan | marker and limit is OK as it is | 08:38 |
aarefiev | milan: ok, it's for use case when you compare things , I was confused by ironic, marker, limit | 08:39 |
aarefiev | aha ok | 08:39 |
openstackgerrit | haris proposed openstack/python-ironicclient: Enables ironic list commands to use labels https://review.openstack.org/380671 | 08:47 |
*** derekh has joined #openstack-ironic | 08:47 | |
*** natorious has quit IRC | 08:47 | |
vdrok | morning all, dtantsur pas-ha vsaienk0 milan aarefiev yuriyz xuao ! | 08:49 |
aarefiev | morning vdrok | 08:49 |
xuao | morning vdrok~ | 08:49 |
milan | morning vdrok! :) | 08:49 |
*** natorious has joined #openstack-ironic | 08:50 | |
*** dtantsur is now known as dtantsur|bbl | 08:55 | |
*** hshiina has quit IRC | 09:03 | |
xuao | hello ironic, I have a question~ | 09:07 |
xuao | while start the console, what's the different between 'usesolkeepalive' and 'nokeepalive'? | 09:08 |
openstackgerrit | Merged openstack/ironic-ui: Imported Translations from Zanata https://review.openstack.org/384057 | 09:08 |
*** _vsaienko has quit IRC | 09:17 | |
*** ohamada has joined #openstack-ironic | 09:19 | |
openstackgerrit | Merged openstack/ironic-inspector: Stop adding ServiceAvailable group option https://review.openstack.org/384346 | 09:22 |
sambetts|afk | Morning all | 09:23 |
*** sambetts|afk is now known as sambetts | 09:23 | |
*** vsaienko has joined #openstack-ironic | 09:25 | |
milan | morning sambetts! :) | 09:26 |
sambetts | hi milan | 09:27 |
tangchen | morning all | 09:32 |
*** mgould|afk is now known as mgould | 09:39 | |
*** jaosorior has quit IRC | 09:39 | |
mgould | morning aarefiev vdrok milan sambetts tangchen xuao and all other Ironicers! | 09:39 |
milan | mgould, morning! :) | 09:40 |
aarefiev | morning sambetts, mgould | 09:40 |
*** jaosorior has joined #openstack-ironic | 09:40 | |
*** zhangyufei has joined #openstack-ironic | 09:40 | |
xuao | morning~~sambetts mgould : ) | 09:41 |
aarefiev | sambetts: hey, could you revisit spec when convenient for you https://review.openstack.org/#/c/305864/ | 09:42 |
patchbot | patch 305864 - ironic-specs - Boot management for in-band inspection | 09:42 |
aarefiev | thanks | 09:42 |
sambetts | aarefiev: will take a look when I get a moment :) | 09:43 |
vdrok | morning sambetts tangchen and mgould | 09:44 |
*** ralonsoh has joined #openstack-ironic | 09:46 | |
openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Add API for listing all introspection statuses https://review.openstack.org/344921 | 09:55 |
*** dtantsur|bbl is now known as dtantsur | 09:56 | |
milan | aarefiev, I think we could drop the if statement if the test case name was zzz_test_list... but it's a hack O:-) wdyt? ^ | 09:56 |
dtantsur | morning vdrok, aarefiev, sambetts, mgould, milan | 09:57 |
milan | morning dtantsur! :) | 09:57 |
milan | and morning tangchen! :) | 09:57 |
openstackgerrit | Derek Higgins proposed openstack/ironic-python-agent: Install grub in the EFI "Fallback path" https://review.openstack.org/384996 | 09:57 |
*** natorious has quit IRC | 10:02 | |
*** natorious has joined #openstack-ironic | 10:02 | |
aarefiev | milan: not blocker, it's just heart cry :) | 10:02 |
aarefiev | morning dtantsur | 10:02 |
mgould | milan: that sounds like a *horrible* hack | 10:03 |
milan | aarefiev, OK :) would be a risk replaced with a risky hack :D | 10:03 |
milan | mgould, yeah :D | 10:04 |
*** hoangcx has quit IRC | 10:08 | |
stendulker | dtantsur: Hi | 10:13 |
mgould | morning stendulker | 10:13 |
stendulker | dtantsur: morning :) | 10:14 |
stendulker | dtantsur: A query on a jenkins failure for IPA config drive fix. https://review.openstack.org/#/c/296466/ | 10:14 |
patchbot | patch 296466 - ironic-python-agent - Use ironic-lib to create configdrive | 10:14 |
stendulker | dtantsur: This jenkinshas started failing recently http://logs.openstack.org/66/296466/8/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ssh-tinyipa-src/5eba208/logs/screen-ir-cond.txt.gz#_2016-10-12_07_05_05_692 | 10:14 |
*** vsaienko has quit IRC | 10:14 | |
stendulker | Error reported is "Unexpected error while running command.\nCommand: dd if=/tmp/configdrivenL2FFg of=/dev/vda2 bs=1M oflag=direct" | 10:15 |
stendulker | Any suggestion? | 10:15 |
*** mgoddard has joined #openstack-ironic | 10:16 | |
stendulker | dtantsur: Error given by dd is '"Stderr: dd: failed to open \'/dev/vda2\': Invalid argument' | 10:18 |
dtantsur | stendulker, "started failing recently" probably because we were not testing config drive before that... | 10:19 |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Help a user to enable console redirection https://review.openstack.org/384578 | 10:19 |
dtantsur | stendulker, also this patch passed the job yesterday: https://review.openstack.org/#/c/384996/ | 10:20 |
patchbot | patch 384996 - ironic-python-agent - Install grub in the EFI "Fallback path" | 10:20 |
dtantsur | stendulker, I suspect the problem is with your patch | 10:20 |
dtantsur | (after looking at the patch) and/or with ironic-lib | 10:21 |
stendulker | dtantsur: there is no instrusive change in IPA side, it consumes ironic-lib | 10:21 |
dtantsur | well, something is broken in either place, as you see | 10:23 |
stendulker | dtantsur: Let me have a one more look at IPA and ironic lib to compare it with script | 10:24 |
stendulker | dtantsur: thank you | 10:24 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Set default baremetal api version to 1.9 for OSC plugin https://review.openstack.org/385386 | 10:31 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for node list and show with specific fields https://review.openstack.org/384923 | 10:42 |
*** xuao has quit IRC | 10:43 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 10:48 |
*** dprince has joined #openstack-ironic | 10:55 | |
*** natorious has quit IRC | 10:58 | |
*** zhangjl1 has left #openstack-ironic | 10:59 | |
*** natorious has joined #openstack-ironic | 11:01 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic-specs: Torrent-based provisioning https://review.openstack.org/311091 | 11:12 |
*** amotoki has joined #openstack-ironic | 11:14 | |
*** tuanluong has quit IRC | 11:32 | |
*** phuongnh has quit IRC | 11:33 | |
*** stendulker has quit IRC | 11:46 | |
openstackgerrit | Vadim Hmyrov proposed openstack/ironic-python-agent: Add bash scripts style checking for IPA https://review.openstack.org/374147 | 11:47 |
*** thrash|g0ne is now known as thrash | 11:48 | |
*** ohamada has quit IRC | 11:53 | |
*** ohamada has joined #openstack-ironic | 11:53 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 11:58 |
zhangyufei | hi,could anyone help tell me that can i configure raid with pxe_ipmi driver? I have take several hours to read code and documents, I still can’t make sure. | 11:58 |
zhangyufei | Documents say that I can configure raid with IPA,but i don’t find where the code is in the ironic-python-client. | 11:58 |
dtantsur | zhangyufei, hi, there is not in-tree support for raid with pxe_ipmitool (and other purely ipmi) drivers | 11:59 |
*** bfournie has quit IRC | 11:59 | |
zhangyufei | dtantsur, thanks a lot, so the support for raid is only in drac and ilo driver? | 12:00 |
mat128 | zhangyufei: currently, yes | 12:00 |
mat128 | zhangyufei: we are adding support for MD raid, which is the software raid used by linux | 12:01 |
mat128 | zhangyufei: do you have a specific controller you are using? | 12:01 |
zhangyufei | no, and I see that bp just now | 12:02 |
zhangyufei | I’m reading it | 12:02 |
zhangyufei | thanks again | 12:02 |
openstackgerrit | Sofiia Andriichenko proposed openstack/python-ironicclient: Add testcases for osc portgroup comands https://review.openstack.org/385427 | 12:03 |
*** zhangyufei has quit IRC | 12:05 | |
*** zhangyufei has joined #openstack-ironic | 12:06 | |
*** ralonsoh has quit IRC | 12:06 | |
*** ralonsoh_ has joined #openstack-ironic | 12:06 | |
*** awidders_ has quit IRC | 12:07 | |
*** awiddersheim has joined #openstack-ironic | 12:07 | |
*** trown|outtypewww is now known as trown|workfrombu | 12:09 | |
*** awiddersheim has quit IRC | 12:11 | |
*** awiddersheim has joined #openstack-ironic | 12:11 | |
*** awidders_ has joined #openstack-ironic | 12:15 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-lib: DO NOT MERGE: testing new jobs https://review.openstack.org/383687 | 12:15 |
*** awiddersheim has quit IRC | 12:15 | |
*** sdake has joined #openstack-ironic | 12:21 | |
*** sdake has quit IRC | 12:22 | |
*** sdake has joined #openstack-ironic | 12:23 | |
*** root has joined #openstack-ironic | 12:24 | |
*** root is now known as Guest47899 | 12:24 | |
Guest47899 | ping | 12:26 |
*** trown|workfrombu is now known as trown|outtypewww | 12:26 | |
*** ohamada has quit IRC | 12:26 | |
*** ohamada has joined #openstack-ironic | 12:27 | |
mat128 | Guest47899: pong | 12:29 |
*** Guest47899 has quit IRC | 12:30 | |
*** aparnav has quit IRC | 12:30 | |
*** mgoddard has quit IRC | 12:31 | |
TheJulia | Good morning | 12:36 |
*** root____4 has joined #openstack-ironic | 12:37 | |
*** root____4 has quit IRC | 12:37 | |
*** bfournie has joined #openstack-ironic | 12:38 | |
*** root___10 has joined #openstack-ironic | 12:40 | |
sambetts | Hi TheJulia | 12:40 |
*** links has quit IRC | 12:42 | |
root___10 | 12:42 | |
*** root___10 has quit IRC | 12:43 | |
mat128 | morning TheJulia, sambetts | 12:43 |
*** trown|outtypewww is now known as trown | 12:46 | |
*** jheroux has joined #openstack-ironic | 12:46 | |
*** vsaienko has joined #openstack-ironic | 12:47 | |
*** Goneri has joined #openstack-ironic | 12:47 | |
*** hshiina has joined #openstack-ironic | 12:48 | |
*** jcoufal has joined #openstack-ironic | 12:50 | |
*** octopusZhang has joined #openstack-ironic | 12:52 | |
xavierr | good morning Ironic :) | 12:53 |
*** vikrant has quit IRC | 12:53 | |
*** octopusZhang has joined #openstack-ironic | 12:54 | |
*** octopusZ1ang has joined #openstack-ironic | 12:57 | |
*** zhangyufei has quit IRC | 12:58 | |
mgould | morning mat128 TheJulia xavierr | 12:58 |
xavierr | mgould: hi there | 13:00 |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Fix stable states validation in FSM https://review.openstack.org/385459 | 13:00 |
*** octopusZhang has quit IRC | 13:00 | |
*** e0ne has quit IRC | 13:02 | |
*** baoli has joined #openstack-ironic | 13:03 | |
*** e0ne has joined #openstack-ironic | 13:03 | |
*** baoli_ has joined #openstack-ironic | 13:04 | |
openstackgerrit | haris proposed openstack/python-ironicclient: Enables ironic list commands to use labels https://review.openstack.org/380671 | 13:07 |
dtantsur | morning mat128, TheJulia, xavierr | 13:07 |
*** jcoufal has quit IRC | 13:07 | |
*** mtanino has joined #openstack-ironic | 13:08 | |
*** jcoufal has joined #openstack-ironic | 13:08 | |
*** baoli has quit IRC | 13:08 | |
*** jcoufal_ has joined #openstack-ironic | 13:12 | |
*** [1]cdearborn has joined #openstack-ironic | 13:13 | |
*** jcoufal has quit IRC | 13:13 | |
*** rcernin has quit IRC | 13:13 | |
*** rcernin has joined #openstack-ironic | 13:13 | |
*** vishwanathj has joined #openstack-ironic | 13:15 | |
*** mtanino has quit IRC | 13:17 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 13:18 |
*** rloo has joined #openstack-ironic | 13:25 | |
mariojv | good morning ironic o/ | 13:29 |
*** phschwartz has quit IRC | 13:30 | |
jroll | morning y'all | 13:32 |
dtantsur | morning jroll, mariojv | 13:32 |
mariojv | morning jroll dtantsur everybody else | 13:32 |
openstackgerrit | Hironori Shiina proposed openstack/ironic: A few fixes in Multitenancy document https://review.openstack.org/385476 | 13:34 |
*** phschwartz has joined #openstack-ironic | 13:36 | |
rloo | morning all, mariojv, jroll, dtantsur | 13:40 |
jroll | \o | 13:40 |
rloo | jroll: question about the summit sessions. the priorities. what's the plan wrt the etherpad (which doesn't have any entries except a title) | 13:41 |
jroll | rloo: I was going to brain dump some things in there this week, others are welcome to as well | 13:41 |
jroll | if that's what you mean? | 13:41 |
jroll | in general, I made etherpads so folks can start populating them with info, and we'll use them during the sessino | 13:42 |
rloo | jroll: and then what? summarize that into a spec next week? | 13:42 |
rloo | oh, during the session. ok, is that fine with JayF? | 13:42 |
jroll | rloo: use it as notes | 13:42 |
rloo | i mean, I'm fine with it. I'll wait for you to populate, then add/comment :) | 13:42 |
jroll | I mean... this is how we always do it | 13:42 |
jroll | and then we put it in the specs repo post-summit | 13:42 |
rloo | jroll: I know. i'll ping jay about that to see if he is ok with it. although if he was at the meeting on mon presumably he is. | 13:43 |
rloo | jroll: i'm just reading the notes/comments in our summit etherpad :) | 13:43 |
jroll | rloo: I'm confused, okay with what? deciding priorities at the summit? | 13:44 |
jroll | in the comments, jay wanted a way to give input pre-summit | 13:45 |
rloo | jroll: (JayF) Can we be sure to garner input for this before the summit session so those of us aren't coming can at least have some input? Maybe a draft spec up a week before the summit? | 13:45 |
jroll | and I hope that etherpad will suffice, idk a better way to do it | 13:45 |
jroll | I don't want to go between spec drafts and etherpads for the summit | 13:45 |
rloo | jroll: right, he suggested a draft spec. wanted to check that he is ok with the etherpad. | 13:45 |
dtantsur | hey rloo | 13:45 |
rloo | hi dtantsur | 13:45 |
rloo | jroll: i suspect as long as the etherpad has some way to reflect the order of priorities or something, so jay/others have an idea of what the likely candidates are or something. i'm not sure. | 13:46 |
jroll | rloo: honestly, I don't expect any cores to be surprised by anything about our priorities | 13:47 |
jroll | we've lots of work on the backlog | 13:47 |
rloo | jroll: i agree but i just want to make sure we can make those folks not attending the summit, happy... :) | 13:47 |
dtantsur | on the bright side, we can assign anything we want to JayF | 13:47 |
jroll | ++ | 13:47 |
rloo | dtantsur: :D | 13:48 |
jroll | rloo: sure, jay is a reasonable person, as long as he can voice opinions I think he'll be okay :) | 13:48 |
openstackgerrit | haris proposed openstack/python-ironicclient: Enables ironic list commands to use labels https://review.openstack.org/380671 | 13:48 |
vdrok | morning mat128 TheJulia xavierr mariojv jroll and rloo :) | 13:49 |
rloo | hi vdrok | 13:52 |
*** jaosorior has quit IRC | 13:53 | |
*** joprovost has joined #openstack-ironic | 13:54 | |
*** pcaruana has joined #openstack-ironic | 13:54 | |
openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Help a user to enable console redirection https://review.openstack.org/384578 | 13:56 |
xavierr | morning vdrok rloo | 13:56 |
*** mgoddard has joined #openstack-ironic | 13:56 | |
vsaienk0 | dtantsur: few new features coming like security groups, portgroups, boot from volume. They will require configuration changes as well | 13:58 |
dtantsur | vsaienk0, yeah, this is a separate thing | 13:58 |
*** e0ne has quit IRC | 14:00 | |
*** anush_ has joined #openstack-ironic | 14:00 | |
*** e0ne has joined #openstack-ironic | 14:00 | |
rloo | hi xavierr | 14:01 |
*** joprovost1 has joined #openstack-ironic | 14:01 | |
*** joprovost has quit IRC | 14:03 | |
*** joprovost1 is now known as joprovost | 14:03 | |
*** yufei has joined #openstack-ironic | 14:04 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Set default baremetal api version to 1.9 for OSC plugin https://review.openstack.org/385386 | 14:05 |
xavierr | rloo, could you give me a hand with this patch? I made the changes you asked for, but yuriy is saying that there are may details in the relesenote. :) https://review.openstack.org/#/c/374989/ | 14:08 |
patchbot | patch 374989 - ironic - Avoid name errors in oneview periodics | 14:08 |
jroll | xavierr: I don't think that needs a release note at all, to be honest | 14:08 |
rloo | xavierr: it is too detailed. | 14:09 |
jroll | it's a small bug fix for an exceptional (ha!) case | 14:09 |
rloo | xavierr: what happens if this isn't fixed? | 14:09 |
xavierr | rloo: periodics execution brake | 14:09 |
rloo | xavierr: how often does this happen | 14:09 |
rloo | xavierr: so i'm an operator. so the periodic execution breaks. so what? | 14:09 |
jroll | I think about it more as "what do operators need to know" | 14:09 |
rloo | xavierr: why do i care? yeah, like jroll said | 14:10 |
rloo | jroll: maybe we should update the doc about release notes :) | 14:10 |
*** anush_ has quit IRC | 14:10 | |
xavierr | rloo, jroll: yes, please update :) | 14:10 |
jroll | "Fixes a bug in the oneview driver where periodic tasks may stop running before completion" would be my suggestion | 14:10 |
*** mjturek has quit IRC | 14:11 | |
xavierr | jroll: but I don't understand how could be bad add more context to the operator in the note :/ | 14:11 |
rloo | jroll: it isn't any/all periodic tasks. i think it should mention something about checking /not checking if notes were taken over by oneview users. | 14:11 |
rloo | jroll, xavierr: to be honest, i don't actually know what is going on, which is why i didn't offer/suggest anything | 14:12 |
jroll | the docs seems fine to me... it isn't bad to add a release note here, but isn't big enough to warrant one | 14:12 |
rloo | jroll: i mean, to mention in our docs, that the release note should indicate how it affects operators, not a description for developers | 14:12 |
jroll | xavierr: if you assume that the operator is not a developer, this release note is confusing | 14:13 |
vsaienk0 | dtantsur: what do you think if we add the same number of ironic nodes as number of use-cases we want to test. Of course use-cases that requires configuration change is separate thing. Also for each use-case we create a separate tempest test and make sure each test uses specific ironic node it may be a flavor or availability zone or something else. Next we launch some number of tests in parallel, this value might be less than number of ironic no | 14:13 |
jroll | rloo: our docs say that http://docs.openstack.org/developer/ironic/dev/faq.html#know-if-a-release-note-is-needed-for-my-change | 14:13 |
jroll | "it is intended to contain info for devs" | 14:13 |
jroll | er s/devs/deployers/ | 14:13 |
dtantsur | vsaienk0, like, a combined approach? some things in parallel (like I've proposed), some sequentially (like you've proposed)? | 14:14 |
rloo | jroll: oh, so we already documented it! You mean, intended to contain info for deployers :) | 14:14 |
jroll | yes | 14:14 |
dtantsur | vsaienk0, that makes sense. I just wonder how many times we can run tests sequentially before hitting timeouts | 14:14 |
rloo | xavierr: did that ^^ help? | 14:15 |
* rloo wonders if xavierr fainted from our comments | 14:15 | |
jroll | How about "Fixes a bug in the oneview driver where the periodic task to check if a node is in use by oneview may end prematurely" | 14:16 |
jroll | that's the best I have | 14:16 |
xavierr | rloo, jroll: yes, it helps | 14:16 |
jlvillal | Good morning Ironic :) | 14:16 |
jlvillal | vsaienk0: Are you still around? Curious if you had the script you use to run the multi-node job locally? | 14:16 |
rloo | xavierr: :) | 14:16 |
rloo | morning jlvillal | 14:16 |
jlvillal | rloo: Hi o/ | 14:16 |
xavierr | another question, when do I need to create a release note? docs patches don't need them... | 14:17 |
xavierr | morning jlvillal | 14:17 |
*** pester has quit IRC | 14:17 | |
* jlvillal is trying to wake up earlier before the summit. Maybe he will be on east coast time by the time he leaves for the summit :) | 14:17 | |
jroll | xavierr: did you read our faq about that? | 14:17 |
jroll | xavierr: http://docs.openstack.org/developer/ironic/dev/faq.html#know-if-a-release-note-is-needed-for-my-change | 14:17 |
jlvillal | xavierr: hi \o | 14:18 |
*** mjturek has joined #openstack-ironic | 14:18 | |
*** anush_ has joined #openstack-ironic | 14:18 | |
xavierr | jroll: I didn't jroll. thanks! :) | 14:18 |
jroll | np :) | 14:20 |
vsaienk0 | dtantsur: it took near 50min to launch agent_ssh-wholedisk-localboot, agent_ssh-partitioned-localboot, pxe_ssh-wholedisk-localboot, pxe_ssh-partitioned-localboot from scratch. | 14:22 |
dtantsur | vsaienk0, our typical job timeout is 80 mins, so this at a dangerous border | 14:23 |
vsaienk0 | jlvillal: I'm suing two separate VMs, create vxlan between them manually. Copy local.conf from primary and subnode to each of VM, update ips and run stack.sh first on primary node than on subnode | 14:24 |
jlvillal | vsaienk0: Do you run devstack-gate? | 14:24 |
vsaienk0 | dtantsur: the maximum timeout on infra is 3.5 hours | 14:24 |
dtantsur | vsaienk0, yeah, but we definitely don't want it to be so much (neither does infra) | 14:25 |
vsaienk0 | jlvillal: nope, I skip this step and do all things manually | 14:25 |
jroll | yeah, 3 hours waiting for results is far too long | 14:25 |
jlvillal | vsaienk0: Ah. So any idea how long that takes you if you start with two brand new VMs? | 14:25 |
dtantsur | to be honest, the scenarious you've mentioned can be run in parallel, but if something can't run in parallel in one devstack, maybe we should jsut have different jobs... | 14:25 |
*** pcaruana has quit IRC | 14:27 | |
jroll | grenade does 3 instances in parallel afaik | 14:27 |
vsaienk0 | dtantsur: yes, I agree that it would be much faster if we using parallel runs, but even with sequential run the overall time is not highly increased. Current each gate job takes near 30-40 minutes to test 1 configuration, with multitest job it took 50 minutes to check 4 configurations | 14:27 |
dtantsur | vsaienk0, yeah, maybe it's not so bad, let's see | 14:28 |
jroll | I lean toward parallel testing, because reconfiguring things is pretty different than how openstack tests normally work | 14:28 |
vsaienk0 | jlvillal: I don't know :) | 14:28 |
jlvillal | vsaienk0: okay :) | 14:29 |
*** fxpester has joined #openstack-ironic | 14:30 | |
vsaienk0 | dtantsur: also we may try to do not use nova in all tests, we may write our own ironic tests to check specific scenario but without nova like in stand alone mode. It will give ability to control that we using right node for specific test. And of course we still need to have some job that tests nova integration | 14:31 |
jroll | I agree with that, but let's take baby steps | 14:32 |
jroll | the real goal here is to reduce the number of jobs | 14:32 |
vgadiraj | vgadiraj | 14:33 |
vsaienk0 | jroll: yeah, we need to reduce the number of existed jobs, and also we need to cover new functionality by CI tests | 14:35 |
*** baoli_ has quit IRC | 14:36 | |
*** e0ne has quit IRC | 14:36 | |
jroll | vsaienk0: we need to reduce jobs first | 14:36 |
*** hj-hpe has joined #openstack-ironic | 14:40 | |
dtantsur | unfortunately, yes.. so whatever we do should fit into roughly the same number of jobs | 14:41 |
dtantsur | give or take temporary duplicates. we can't create more, we actually have to create less | 14:41 |
*** e0ne has joined #openstack-ironic | 14:45 | |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Fix webclient viability after error https://review.openstack.org/385506 | 14:48 |
*** athomas has quit IRC | 14:50 | |
openstackgerrit | Xavier proposed openstack/ironic: Avoid name errors in oneview periodics https://review.openstack.org/374989 | 14:52 |
vsaienk0 | jroll: I will try to modify portgroups tests so we can launch them in parallel with multitenancy tests | 14:53 |
*** syed_ has joined #openstack-ironic | 14:53 | |
rloo | hi dtantsur, i have been meaning to ask you about https://bugs.launchpad.net/ironic/+bug/1427923. Your comment #2. | 14:54 |
openstack | Launchpad bug 1427923 in Ironic "boot device API blocks while waiting on the BMC" [Medium,Confirmed] - Assigned to bin Yu (froyo-bin) | 14:54 |
rloo | dtantsur: if we make the get/set_boot_device async, that will break inspector? | 14:54 |
rloo | dtantsur: what if the API has a --wait option. | 14:54 |
dtantsur | rloo, that will break anyone relying on this behavior, so I guess it should go with an API version.. | 14:55 |
rloo | dtantsur: oh, DEFINITELY. we love our microversions! | 14:55 |
dtantsur | in the meanwhile, I've just got struck by folks' love to do "except Exception" again >_< | 14:56 |
rloo | dtantsur: we discussed that in one of our meetings, you were on pto i think: http://eavesdrop.openstack.org/meetings/ironic/2016/ironic.2016-09-19-17.00.log.html | 14:57 |
jlvillal | dtantsur: Oh I hate that! It gets me riled up | 14:57 |
sambetts | dtantsur: oh no :( | 14:57 |
rloo | dtantsur: and i offered to bring to ML, but it seemed to me that we were all for async, except your comment. so you're good with async? | 14:57 |
dtantsur | rloo, yes, I guess we'll figure it out somehow | 14:57 |
rloo | dtantsur: ok, i'll just comment in that bug then, so that whoever works on it has some direction :) | 14:58 |
rloo | dtantsur: thx | 14:58 |
rloo | dtantsur: what's wrong with except Exception? | 14:59 |
octopusZ1ang | 、、 | 15:00 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Do not hide unexpected exceptions in inspection code https://review.openstack.org/385510 | 15:00 |
dtantsur | something like ^^^ | 15:00 |
dtantsur | rloo, ^^^ | 15:00 |
dtantsur | I don't know how to debug 'NoneType' object has no attribute 'strip' without any tracebacks | 15:00 |
rloo | dtantsur: oh, that except Exception. | 15:01 |
rloo | dtantsur: i think we have that pattern more than once. you already have a patch to fix one. maybe we should open a bug, have someone go through the code... | 15:01 |
openstackgerrit | haris proposed openstack/python-ironicclient: Enables ironic list commands to use labels https://review.openstack.org/380671 | 15:02 |
*** mjura has quit IRC | 15:02 | |
dtantsur | rloo, yeah.. this is what I'm suffering with right now: dell oob inspection fails, and I've no clues why | 15:02 |
openstackgerrit | Marc Aubry proposed openstack/python-ironicclient: Fix python3 compatibility when HTTP Error are returned https://review.openstack.org/378050 | 15:03 |
rloo | dtantsur: :( | 15:03 |
*** sdake has quit IRC | 15:05 | |
jlvillal | rloo: Not sure about the specific case but the problem with "except Exception" is that it catches everything. SyntaxError in the code, sure we will catch that for you no problem and maybe we will not even tell you about it :) | 15:05 |
openstackgerrit | Merged openstack/pyghmi: Remove xrange for run both Python 2 and Python 3 https://review.openstack.org/385281 | 15:06 |
*** sdake has joined #openstack-ironic | 15:06 | |
jlvillal | rloo: But looks like in the case the patch is doing, it does get re-raised. So that is good. | 15:06 |
*** krtaylor has joined #openstack-ironic | 15:06 | |
openstackgerrit | Merged openstack/pyghmi: Check for buffer and use memoryview if missing https://review.openstack.org/385051 | 15:06 |
rloo | jlvillal: if i understand it correctly, is about not logging exceptions for unknown exceptions. | 15:07 |
jlvillal | rloo: Huh. I'm wondering into the middle of the conversation but that sounds strange. Code gets an unknown exception, it shouldn't be logged? So any unknown exception that occurs there is no notification? | 15:08 |
rloo | jlvillal: so yeah. 1. we should be specific and handle known/'expected' exceptions. for all other exceptions. if they occur, how do we debug, so getting the stack trace helps. | 15:08 |
jlvillal | s/wondering/wandering/ | 15:08 |
dtantsur | mgould, https://bugs.launchpad.net/ironic/+bug/1632747 | 15:08 |
openstack | Launchpad bug 1632747 in Ironic "Drac OOB inspection fails on Dell PowerEdge R320 with 'NoneType' object has no attribute 'strip'" [Medium,Triaged] | 15:08 |
rloo | jlvillal: it should be logged, at exception level though | 15:09 |
jlvillal | rloo: Okay good. I was worried for a moment :) | 15:09 |
rloo | jlvillal: so it is ok for expected exceptions not to be logged at that level, but not ok for the unexpected exceptions. | 15:09 |
dtantsur | ++ | 15:10 |
mgould | dtantsur: thanks | 15:10 |
*** joprovost1 has joined #openstack-ironic | 15:11 | |
*** joprovost has quit IRC | 15:12 | |
*** joprovost1 is now known as joprovost | 15:12 | |
jlvillal | rloo: makes sense. Thanks. | 15:12 |
*** joprovost has quit IRC | 15:16 | |
*** joprovost has joined #openstack-ironic | 15:19 | |
*** rcernin has quit IRC | 15:19 | |
*** mtanino has joined #openstack-ironic | 15:22 | |
*** ccamacho has joined #openstack-ironic | 15:25 | |
JayF | I'm seeing unit test failures in the gate that I think can be attributed to ironic-lib release | 15:26 |
openstackgerrit | Merged openstack/pyghmi: Fix webclient viability after error https://review.openstack.org/385506 | 15:26 |
JayF | http://logs.openstack.org/69/370469/11/gate/gate-ironic-python27-db-ubuntu-xenial/4a69263/console.html#_2016-10-11_22_27_02_273951 | 15:26 |
JayF | aslezil: ^ | 15:26 |
*** rama_y has joined #openstack-ironic | 15:27 | |
JayF | anyone been digging on this today? If not I'll begin... | 15:28 |
dtantsur | JayF, missing upper-constraints. where is it? | 15:28 |
JayF | I don't know? This was a gate job | 15:29 |
JayF | and all 3 unit test jobs failed in the gate | 15:29 |
JayF | and aslezil found other patches failing similarly | 15:29 |
dtantsur | we fixed something like that for mitaka and liberty IPA (?) yesterday | 15:29 |
JayF | this is not just ipa | 15:30 |
JayF | this is ironic | 15:30 |
*** aNuposic has joined #openstack-ironic | 15:30 | |
dtantsur | maybe we like some backport (or maybe it was ironic, I don't remember) | 15:30 |
dtantsur | s/like/miss/ | 15:30 |
dtantsur | what am I writing? | 15:30 |
JayF | It's just the make-ipa-build-use-upper-constraints patch backported, right? | 15:31 |
JayF | I think this is unrelated | 15:31 |
dtantsur | right, this was ironic: https://github.com/openstack/ironic/commit/850376338f0319000abd3baf64acc9d18704651d | 15:32 |
dtantsur | and https://github.com/openstack/ironic/commit/b375ca2c078df0ca5580517025586a82d85665df | 15:32 |
rloo | looks like failures started yesterday: http://ci-watch.tintri.com/project?project=ironic&time=7+days | 15:32 |
*** baoli has joined #openstack-ironic | 15:33 | |
dtantsur | it's not impossible that the remove of mkfs from ironic-lib was incorrect | 15:34 |
JayF | the failures themselves appear related to ironic-lib | 15:34 |
dtantsur | yes, it was: https://github.com/openstack/ironic/blob/5603a215b726ca465eed2867d830821bb24b1a72/ironic/common/images.py#L112 | 15:34 |
JayF | so I think that's right | 15:34 |
dtantsur | we need to revert that patch and re-release | 15:34 |
dtantsur | jroll, ^^^ | 15:34 |
* dtantsur facepalms | 15:34 | |
JayF | dtantsur: you wanna put up the revert? I'll vote onit | 15:34 |
dtantsur | I will | 15:35 |
dtantsur | saying hello to jlvillal for https://github.com/openstack/ironic-lib/commit/c10121bc26e3cc019a7d7395f81d9e34ddc9680e :) | 15:35 |
*** jcoufal has joined #openstack-ironic | 15:35 | |
*** jcoufal_ has quit IRC | 15:35 | |
dtantsur | or wait, maybe instead of a revert we should just stop using this one-line wrapper | 15:35 |
* dtantsur is confused again | 15:36 | |
openstackgerrit | haris proposed openstack/python-ironicclient: Enables ironic list commands to use labels https://review.openstack.org/380671 | 15:36 |
JayF | IMO best route is to revert and re-release | 15:36 |
jlvillal | dtantsur: Looking | 15:36 |
JayF | dtantsur: filing a bug on it though, so we have something to track | 15:36 |
dtantsur | JayF, re-releasing is longer, and we need to ensure we understand it right | 15:36 |
rloo | dtantsur: shouldn't we change that could to use oslo_concurrency.processutils.mkfs()? | 15:36 |
rloo | s/could/code/ | 15:37 |
jroll | that's... odd, that ironic code is using the right import | 15:37 |
dtantsur | it seems to be in tests | 15:37 |
jroll | heh, bad mock I guess? | 15:37 |
dtantsur | I suspect https://github.com/openstack/ironic/blob/659f951d72e96f39bb967455a6855682e517ca43/ironic/tests/unit/drivers/modules/test_deploy_utils.py#L362-L364 | 15:37 |
dtantsur | lemme test something locally | 15:38 |
JayF | another question I have ... isn't ironic-lib tested against ironic master/ | 15:38 |
JayF | I'm guessing just tempest though, not unit tests/ | 15:38 |
jroll | dtantsur: that sounds right | 15:38 |
jroll | JayF: right | 15:38 |
jlvillal | dtantsur: I'm confused. The first link is using 'utils.mkfs' | 15:38 |
jroll | it's just a unit test thing, going by the traceback | 15:38 |
xavierr | I'm getting with mock http://logs.openstack.org/35/372735/8/check/gate-ironic-python27-db-ubuntu-xenial/d6fe3ce/console.html#_2016-10-12_15_21_39_208075 | 15:38 |
jlvillal | dtantsur: I had removed 'disk_utils.mkfs' | 15:38 |
*** jtomasek has quit IRC | 15:38 | |
jroll | jlvillal: https://github.com/openstack/ironic/blob/659f951d72e96f39bb967455a6855682e517ca43/ironic/tests/unit/drivers/modules/test_deploy_utils.py#L362-L364 | 15:39 |
JayF | https://bugs.launchpad.net/ironic/+bug/1632762 filed | 15:40 |
openstack | Launchpad bug 1632762 in Ironic "Unit tests fail with ironic-lib 2.2.0" [Critical,New] | 15:40 |
*** links has joined #openstack-ironic | 15:40 | |
jlvillal | jroll: Oh. | 15:40 |
dtantsur | jlvillal, wanna fix it or should I? | 15:40 |
jlvillal | dtantsur: I can do it | 15:40 |
dtantsur | thanks! | 15:40 |
jlvillal | dtantsur: Don't want you having to work through the evening :) | 15:40 |
dtantsur | much appreciated :) | 15:41 |
jlvillal | dtantsur: Plus my patch is the cause :( | 15:41 |
*** davidlenwell_ has joined #openstack-ironic | 15:41 | |
dtantsur | that was hard to predict, IMO, don't blame yourself | 15:41 |
jlvillal | dtantsur: Thanks | 15:41 |
* jroll wonders if we should run our unit tests on u-c updates | 15:41 | |
dtantsur | this test looks.. awful.. | 15:41 |
dtantsur | jroll, we should, if folks agree to it | 15:42 |
jroll | ++ no blame here, things happen, that would be hard to find | 15:42 |
jlvillal | thanks | 15:43 |
JayF | jroll: I was wondering more if we should run ironic unit tests on ironic-lib patches | 15:44 |
JayF | jroll: or at a minimum put it in the ironic-lib experimental pipeline so we can run them at least once in gate prior to release | 15:44 |
jroll | JayF: well, do you want to solve this for ironic-lib releases or all dependencies? :) | 15:45 |
JayF | I mean, in some ways ironic-lib is a special case | 15:45 |
*** adam_g` has joined #openstack-ironic | 15:45 | |
JayF | I don't have any data on how often u-c updates are | 15:45 |
jroll | often | 15:45 |
*** davidlenwell_ has quit IRC | 15:45 | |
JayF | or how disruptive it'd be to make them be +2A'd after jobs | 15:45 |
jroll | every time a g-r thing releases | 15:45 |
jroll | they already run many jobs | 15:45 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add tests for OSC baremetal chassis create command https://review.openstack.org/382496 | 15:45 |
*** david-lyle has joined #openstack-ironic | 15:46 | |
JayF | oh? They're run against requirements project then, I'd presume | 15:46 |
JayF | there's no commit made to ironic with the new u-c, right? | 15:46 |
jroll | JayF: https://review.openstack.org/#/c/384501/ for example | 15:46 |
patchbot | patch 384501 - requirements - update constraint for keystonemiddleware to new re... (MERGED) | 15:46 |
jroll | right | 15:46 |
JayF | hmm. nobody else has to run their unit tests. | 15:46 |
jroll | they do | 15:46 |
jroll | gate-cross-$project-python27-ubuntu-xenial | 15:46 |
JayF | oh, I see, yeah | 15:46 |
jroll | "the big 6" are there | 15:46 |
JayF | time to make it 7 up then? | 15:47 |
jroll | yeah, we'll have to get requirements team permission | 15:48 |
* jroll runs off for lunch, thanks all for digging into that | 15:49 | |
*** baoli has quit IRC | 15:50 | |
jlvillal | dtantsur: I'm starting to agree with you on that unit test... :) | 15:50 |
*** david-lyle has quit IRC | 15:52 | |
*** Sukhdev has joined #openstack-ironic | 15:52 | |
*** narasimha_SV_ has joined #openstack-ironic | 15:52 | |
narasimha_SV_ | http://paste.openstack.org/show/585482/ getting this error in ironic-conductor log while deploying | 15:52 |
narasimha_SV_ | i didn't understand what exactly is missing here | 15:52 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Do not hide unexpected exceptions in inspection code https://review.openstack.org/385510 | 15:54 |
*** yufei has quit IRC | 15:57 | |
JayF | So while on the topic of the gate | 15:58 |
JayF | I know we fixed the iscsi xenial issue, is there a project-config followup patch to migrate that to xenial we should run? | 15:58 |
JayF | It looks like the issue itself is resolved, I'm just unsure (and it's not clear from the bug) the last step needed to complete the xenial migration | 15:59 |
*** jcoufal_ has joined #openstack-ironic | 15:59 | |
dtantsur | JayF, I'm tracking that with vsaienk0 | 16:00 |
dtantsur | JayF, the current patch is https://review.openstack.org/385433 | 16:00 |
patchbot | patch 385433 - openstack-infra/project-config - Revert "Temporary move ironic-lib back to Trusty" | 16:00 |
JayF | perfect, mind if I go try to get you a +2A on it? | 16:00 |
dtantsur | I definitely don't mind it :) | 16:00 |
*** athomas has joined #openstack-ironic | 16:01 | |
*** jcoufal has quit IRC | 16:02 | |
*** trown is now known as trown|lunch | 16:02 | |
dtantsur | that was quick :) | 16:04 |
jlvillal | JayF: We might want to wait until we fix the unit test breakage. I'm working on it now. | 16:04 |
JayF | I do good spam. lol. | 16:04 |
JayF | jlvillal: that patch won't even run ironic tests, so it doesn't matter too much :D | 16:04 |
dtantsur | jlvillal, I don't quite see how unit tests are related here, this is about ironic-lib dsvm CI | 16:05 |
jlvillal | Jay, ah okay | 16:05 |
jlvillal | dtantsur: Ah duh :) | 16:05 |
*** david-lyle has joined #openstack-ironic | 16:06 | |
*** davidlenwell_ has joined #openstack-ironic | 16:07 | |
*** sabeen has joined #openstack-ironic | 16:12 | |
*** rama_y has quit IRC | 16:12 | |
jlvillal | Took me awhile to figure out in Ubuntu 16.04 I need to install vim-nox-py2 in order to have support for using python in a vim script | 16:15 |
*** aNuposic- has quit IRC | 16:15 | |
*** aNuposic- has joined #openstack-ironic | 16:16 | |
*** yibo has quit IRC | 16:18 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal node commands https://review.openstack.org/301687 | 16:18 |
*** yibo has joined #openstack-ironic | 16:18 | |
NobodyCam | Good Morning Ironic'ers :) | 16:18 |
mgould | morning NobodyCam | 16:20 |
dtantsur | morning NobodyCam | 16:20 |
NobodyCam | morning mgould, dtantsur :) | 16:20 |
jlvillal | dtantsur: It seems like the unit tests are testing that the ironic_lib code works. https://github.com/openstack/ironic/blob/659f951d72e96f39bb967455a6855682e517ca43/ironic/tests/unit/drivers/modules/test_deploy_utils.py#L482-L560 | 16:25 |
*** rama_y has joined #openstack-ironic | 16:25 | |
*** vsaienko has quit IRC | 16:25 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Set OSC default baremetal api version as in ironicclient https://review.openstack.org/385386 | 16:25 |
jlvillal | dtantsur: I'm thinking about changing it so that it just mocks ironic_lib.disk_utils.work_on_disk() | 16:26 |
dtantsur | jlvillal, ugh.. I think we even have a bug for it. yes, +1. just make sure that ironic-lib actually has the relevant test | 16:26 |
jlvillal | dtantsur: Ah good call. /me assumed that ironic-lib would be testing that. | 16:26 |
*** vsaienko has joined #openstack-ironic | 16:27 | |
dtantsur | see you tomorrow | 16:28 |
*** dtantsur is now known as dtantsur|afk | 16:28 | |
*** davidlenwell_ has quit IRC | 16:29 | |
jlvillal | dtantsur|afk: Good night | 16:30 |
*** baoli has joined #openstack-ironic | 16:31 | |
rloo | jlvillal: would it make sense to just fix that mkfs call in the unit test. then clean it up in a followup patch? | 16:33 |
jlvillal | rloo: That is what I'm doing actually :) | 16:33 |
rloo | jlvillal: great minds... ;) | 16:33 |
jlvillal | rloo: I was going to add some fixme notes and get a relatively quick fix. | 16:33 |
jlvillal | rloo: heh | 16:34 |
*** jpich has quit IRC | 16:34 | |
*** baoli_ has joined #openstack-ironic | 16:34 | |
*** baoli_ has quit IRC | 16:35 | |
*** baoli_ has joined #openstack-ironic | 16:35 | |
*** baoli has quit IRC | 16:36 | |
*** milan has quit IRC | 16:36 | |
*** rama_y has quit IRC | 16:37 | |
*** milan has joined #openstack-ironic | 16:37 | |
openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Have range become xrange where available https://review.openstack.org/385558 | 16:39 |
*** rama_y has joined #openstack-ironic | 16:39 | |
*** vsaienko has quit IRC | 16:40 | |
*** jcoufal_ has quit IRC | 16:41 | |
*** vsaienko has joined #openstack-ironic | 16:41 | |
*** jcoufal has joined #openstack-ironic | 16:41 | |
* mgould -> home; good night! | 16:43 | |
*** mgould is now known as mgould|afk | 16:43 | |
*** watanabe_isao has joined #openstack-ironic | 16:44 | |
*** natorious has quit IRC | 16:44 | |
jlvillal | mgould|afk: Bye | 16:44 |
openstackgerrit | Merged openstack/pyghmi: Add script to help build rpms https://review.openstack.org/382044 | 16:45 |
*** links has quit IRC | 16:45 | |
*** e0ne has quit IRC | 16:45 | |
*** natorious has joined #openstack-ironic | 16:47 | |
*** Sukhdev has quit IRC | 16:48 | |
*** derekh has quit IRC | 16:52 | |
jlvillal | Anyone else notice that if run the tox tests: py35 and then py27, that py27 fails with "No module named gdbm" | 16:56 |
*** Romanenko_K has quit IRC | 16:57 | |
jlvillal | But if I: rm -rf .testrepository/ then the py27 test passes | 16:57 |
jlvillal | :( | 16:57 |
*** rcernin has joined #openstack-ironic | 16:57 | |
jroll | yeah, longstanding bug in testr | 16:57 |
jroll | although I thought it was the reverse | 16:57 |
jroll | maybe os x only iirc? | 16:57 |
JayF | Basically it creates the testr db in a newer format if you run py3 tests first | 16:58 |
JayF | so then the py2 testr can't read it | 16:58 |
JayF | something like that? | 16:58 |
jroll | yeah | 16:58 |
rloo | jlvillal: that's my understanding. which is why we modified the tox.ini files to have py35 first. are you seeing it the other way around? py35 first, py27 second and it fails? | 16:59 |
jlvillal | jroll: No different bug. I am running py35 first and then py27. | 17:00 |
jroll | jlvillal: os x? | 17:00 |
jlvillal | jroll: If I run py27 and then py35 then I get the db error | 17:00 |
jlvillal | jroll: Fedora | 17:00 |
jroll | mmm | 17:00 |
*** Guest24890 has quit IRC | 17:00 | |
jlvillal | jroll: Actually. Ubuntu 16.04. I forgot I was on my VM | 17:01 |
jroll | yeah, not what I'm thinking of | 17:01 |
jroll | I feel like I've seen that on os x, but not sure | 17:01 |
openstackgerrit | Julia Kreger proposed openstack/ironic-specs: Boot from Volume - Reference Drivers https://review.openstack.org/294995 | 17:01 |
jroll | jlvillal: qa meeting? | 17:01 |
jlvillal | jroll: Thanks! | 17:02 |
*** rbrud has joined #openstack-ironic | 17:02 | |
*** ralonsoh_ has quit IRC | 17:02 | |
*** jtomasek has joined #openstack-ironic | 17:04 | |
*** trown|lunch is now known as trown | 17:08 | |
*** baoli_ has quit IRC | 17:08 | |
*** jtomasek has quit IRC | 17:08 | |
*** jtomasek has joined #openstack-ironic | 17:08 | |
*** baoli has joined #openstack-ironic | 17:12 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Fix unit tests failing with ironic-lib 2.1.1 https://review.openstack.org/385569 | 17:16 |
jlvillal | rloo: JayF jroll sambetts ^^^ Fix for gate | 17:16 |
* jroll waits for tests | 17:16 | |
rloo | jlvillal: looking... | 17:16 |
jroll | 82 lines? | 17:16 |
openstackgerrit | Merged openstack/pyghmi: Have range become xrange where available https://review.openstack.org/385558 | 17:17 |
jlvillal | jroll: sorry :( | 17:17 |
*** baoli has quit IRC | 17:19 | |
*** Sukhdev has joined #openstack-ironic | 17:19 | |
*** ChubYann has joined #openstack-ironic | 17:22 | |
*** lekha has quit IRC | 17:24 | |
*** thrash is now known as thrash|biab | 17:25 | |
jlvillal | rloo: JayF jroll sambetts: Tests passed for gate fix: https://review.openstack.org/385569 | 17:25 |
patchbot | patch 385569 - ironic - Fix unit tests failing with ironic-lib 2.1.1 | 17:25 |
jroll | cool | 17:26 |
*** lekha has joined #openstack-ironic | 17:26 | |
*** jcoufal has quit IRC | 17:26 | |
*** jcoufal has joined #openstack-ironic | 17:27 | |
rloo | jlvillal: +A | 17:27 |
*** ohamada has quit IRC | 17:28 | |
jlvillal | Thanks :) | 17:28 |
openstackgerrit | Stephanie Miller proposed openstack/ironic: Add volume_connector table to save connector information https://review.openstack.org/200983 | 17:28 |
* sambetts -> home | 17:31 | |
*** sambetts is now known as sambetts|afk | 17:32 | |
*** baoli has joined #openstack-ironic | 17:32 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-specs: Promote iPXE to separate boot interface https://review.openstack.org/385572 | 17:33 |
*** hshiina has quit IRC | 17:33 | |
openstackgerrit | Merged openstack/ironic: Fix unit tests failing with ironic-lib 2.1.1 https://review.openstack.org/385569 | 17:34 |
*** jcoufal has quit IRC | 17:35 | |
*** jcoufal has joined #openstack-ironic | 17:36 | |
*** baoli has quit IRC | 17:37 | |
jlvillal | FYI: Whiteboard updated that gate is now golden. Moved issue to history page. | 17:39 |
openstackgerrit | Stephanie Miller proposed openstack/bifrost: Update playbooks to support os_client_config https://review.openstack.org/380224 | 17:39 |
*** mag009 has quit IRC | 17:39 | |
rloo | thx jlvillal! | 17:39 |
*** rloo_ has joined #openstack-ironic | 17:43 | |
*** rloo has quit IRC | 17:43 | |
JayF | aslezil: you should be able to recheck your patch now ^ jlvillal just merged in a fix for the gate bug | 17:45 |
JayF | I will play recheck fairy in a little bit if nobody else has by then | 17:45 |
jlvillal | Anyone know what happened to thiagop? Does he still hang out here? | 17:46 |
aslezil | Thank you JayF, rloo just pinged me about it | 17:46 |
narasimha_SV_ | http://paste.openstack.org/show/585499/ http://paste.openstack.org/show/585482/ while doing pxe boot using pxe_ipmitool | 17:46 |
narasimha_SV_ | getting these issues in iscsi and ironic conductor logs | 17:47 |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 17:50 |
mariojv | rloo_: dtantsur|afk: i brought up another point about LOG.exception vs. LOG.error for those exception types here https://review.openstack.org/#/c/321865/41/ironic/conductor/notification_utils.py | 17:50 |
patchbot | patch 321865 - ironic - Add power state change notifications | 17:50 |
watanabe_isao | jroll, sir, the re-send mail you said a long time before, it is still not arrived, I'm thinking, my questions in that mail are not important now, and, maybe I should use a gmail next time. Sorry for token your time. | 17:51 |
mariojv | i personally would find it useful to log most of those exceptions during development (which is where most of those will occur). maybe MessageDeliveryFailure should just be LOG.error though | 17:52 |
rloo_ | mariojv: these logs are going to be used in production so we want to make sure that we don't spew out too much stuff but spew out enough to help troubleshoot if need be. | 17:53 |
mariojv | perhaps a warning would even be appropriate for MessageDeliveryFailure, but if the same queue is being used for both RPC and notifications it'd be good to do LOG.error | 17:54 |
JayF | I mean, if it's the same queue as RPC, won't the RPC code complain about it itself? | 17:54 |
mariojv | rloo_: right; i think my point was that the exceptions except MessageDeliveryFailure will only occur during development unless there's a severe coding bug | 17:54 |
mariojv | the notification might potentially do it first | 17:54 |
JayF | We should think what level the log should be for notifications, and let RPC connection failures log at the right level for RPC imo | 17:54 |
mariojv | sure | 17:54 |
JayF | but I'd still say, if you have it configured and it's failing, it's WARN for sure | 17:54 |
rloo_ | JayF: it is currently coded as log.exception | 17:55 |
*** Sukhdev has quit IRC | 17:55 | |
jroll | watanabe_isao: no problem | 17:55 |
mariojv | i suppose on the other hand, a developer could change it to LOG.exception for debugging if they need to | 17:55 |
rloo_ | JayF: the question is whether to lower it and if so, for which 'class' of exceptions | 17:55 |
mariojv | so, how about i change it to LOG.warning? | 17:55 |
mariojv | for everything | 17:55 |
rloo_ | mariojv: i think for the exceptions that we think are development ones, LOG.error | 17:56 |
rloo_ | mariojv: i pointed it out cuz dtantsur|afk was indicating that for 'unknown' exceptions, it is useful to do log.exception to get the trace | 17:56 |
JayF | I don't really understand where LOG.exception fits in, tbh. It's not a traditional log level. Does it just print an exception at ERROR level? | 17:56 |
mariojv | JayF: yep | 17:56 |
JayF | I'd just log them at log.error then | 17:56 |
rloo_ | JayF: yes AND a trace | 17:56 |
rloo_ | JayF: so very verbose | 17:56 |
*** rcernin has quit IRC | 17:56 | |
jroll | log.exception == log.error + traceback | 17:56 |
JayF | please do not put tracebacks in production logs :D | 17:57 |
JayF | even if we don't think it's likely to happen | 17:57 |
jroll | ? | 17:57 |
JayF | a dev can get their own traceback when debugging if needed | 17:57 |
jroll | how'd you expect to debug without... hm | 17:57 |
rloo_ | ok, maybe we need to discuss this and come up with guidelines... | 17:57 |
mariojv | there are guidelines | 17:57 |
jroll | I've always appreciated a traceback when things blow up | 17:57 |
mariojv | http://specs.openstack.org/openstack/openstack-specs/specs/log-guidelines.html | 17:57 |
mariojv | http://specs.openstack.org/openstack/openstack-specs/specs/log-guidelines.html#stacktraces-in-logs | 17:57 |
* watanabe_isao said Good night to all and left. .zZZ | 17:58 | |
*** watanabe_isao has quit IRC | 17:58 | |
mariojv | i think that fits in with LOG.error (or even LOG.warning) everything except unknown exceptions | 17:58 |
JayF | yes | 17:58 |
mariojv | http://specs.openstack.org/openstack/openstack-specs/specs/log-guidelines.html#definition-of-log-levels would you want to notify on-call for a notification failure? | 17:58 |
mariojv | that would be LOG.error | 17:59 |
mariojv | if not, LOG.warning | 17:59 |
JayF | LOG.warning imo | 18:00 |
rloo_ | i like the "Rationale: The current behavior of OpenStack is extremely stack trace happy" | 18:00 |
vdrok | good night everyone! | 18:00 |
rloo_ | mariojv: that is what you would have done :) | 18:00 |
rloo_ | night vdrok | 18:00 |
narasimha_SV_ | does agent_ipmitool driver require swift ?? | 18:02 |
narasimha_SV_ | can't we use this driver without swift ??? | 18:02 |
openstackgerrit | haris proposed openstack/python-ironicclient: Enables ironic list commands to use labels https://review.openstack.org/380671 | 18:03 |
jroll | narasimha_SV_: with nova, it does - without nova, you can use any http url | 18:04 |
jroll | I think it may have ceph support too | 18:04 |
narasimha_SV_ | what is the importance of this swift_temp_url_key here with this driver ? | 18:05 |
narasimha_SV_ | if I am not wrong the images must need to be stored in swift if i want to use this driver ? | 18:06 |
jroll | we use swift temp urls to give the deploy agent a way to download the image: http://docs.openstack.org/kilo/config-reference/content/object-storage-tempurl.html | 18:06 |
jroll | so that it doesn't need to auth to glance | 18:06 |
jroll | yes, swift must be the backend for glance | 18:06 |
narasimha_SV_ | is there a way I can avoid this ?? | 18:07 |
mariojv | switching it to LOG.warning for the known exceptions then | 18:07 |
jroll | narasimha_SV_: well, you can use ceph instead | 18:07 |
jroll | narasimha_SV_: but in general, no | 18:07 |
*** milan has quit IRC | 18:08 | |
narasimha_SV_ | ok | 18:08 |
*** vsaienko has quit IRC | 18:10 | |
*** jcoufal has quit IRC | 18:16 | |
*** jcoufal__ has joined #openstack-ironic | 18:16 | |
*** vsaienko has joined #openstack-ironic | 18:17 | |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 18:21 |
mariojv | ^ that has the logging changes we discussed | 18:21 |
*** rama_y has quit IRC | 18:22 | |
*** jcoufal has joined #openstack-ironic | 18:29 | |
*** jcoufal__ has quit IRC | 18:31 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 18:34 |
*** e0ne has joined #openstack-ironic | 18:37 | |
*** narasimha_SV_ has quit IRC | 18:38 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add initial keystone use documentation https://review.openstack.org/385041 | 18:39 |
*** rama_y has joined #openstack-ironic | 18:43 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Add ability to write clouds.yaml https://review.openstack.org/380227 | 18:43 |
openstackgerrit | Merged openstack/ironic: Remove unneeded exception handling from conductor https://review.openstack.org/381530 | 18:49 |
*** thrash|biab is now known as thrash | 18:50 | |
*** penick has joined #openstack-ironic | 18:54 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Test timeout extension for inspection https://review.openstack.org/375050 | 18:58 |
*** mmedvede has quit IRC | 19:02 | |
*** rloo_ has quit IRC | 19:03 | |
*** rloo has joined #openstack-ironic | 19:04 | |
*** baoli_ has joined #openstack-ironic | 19:07 | |
*** penick has quit IRC | 19:11 | |
*** jcoufal_ has joined #openstack-ironic | 19:14 | |
*** penick has joined #openstack-ironic | 19:16 | |
*** jcoufal has quit IRC | 19:16 | |
*** e0ne has quit IRC | 19:24 | |
*** e0ne has joined #openstack-ironic | 19:28 | |
*** sinh_ is now known as sinh | 19:28 | |
*** e0ne has quit IRC | 19:29 | |
rloo | oh, hey JayF, wrt ocata priorities, are you OK with an etherpad for collecting priorities, and the discussion/decision happening at the summit | 19:35 |
rloo | JayF: you had suggested a draft spec | 19:35 |
JayF | Honestly, the boiled-down-to-basics of my request was mainly to see a preliminary list | 19:35 |
JayF | so I can make sure if something is missing that I expected to be on there, I can point it out | 19:35 |
rloo | JayF: ok, will try to keep that in mind wrt the etherpad :) | 19:36 |
JayF | or conversely that there's no dependencies that I'd know about that was missed | 19:36 |
rloo | JayF: am waiting for jroll to start that pad | 19:36 |
rloo | JayF: i mean start adding stuff to it :) | 19:36 |
JayF | yeah, I mean, I'm not terribly worried about it either way tbh | 19:36 |
JayF | I just think we have to be careful about requiring a $$$$ trip and summit pass to have any input to priorities | 19:36 |
rloo | JayF: ok, just wanted to make sure you were good with it | 19:36 |
JayF | in general, not just for me :D | 19:36 |
rloo | JayF: agree. we should just NOT have summits where people meet physically. do it all virtually. that'd solve that problem ;) | 19:37 |
JayF | I mean, I can't make that big of a change, but I can ask us to have a list of priorities up before summit for comment before they're finalized in real life | 19:38 |
rloo | JayF: i would hope that even after they are 'finalized' at the summit, if there is something missing, that we would be willing to change it. | 19:39 |
jroll | ^^ | 19:39 |
jroll | I was just typing that | 19:39 |
jroll | the 'final' thing is a review in the specs repo | 19:39 |
rloo | jroll: for once, i was faster than you. That goes with any decision we make. | 19:39 |
*** penick has quit IRC | 19:39 | |
*** _vsaienko has joined #openstack-ironic | 19:44 | |
*** vsaienko has quit IRC | 19:44 | |
*** e0ne has joined #openstack-ironic | 19:44 | |
*** penick has joined #openstack-ironic | 19:48 | |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 19:50 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Set OSC default baremetal api version as in ironicclient https://review.openstack.org/385386 | 19:51 |
*** zackf has quit IRC | 19:59 | |
openstackgerrit | Merged openstack/ironic: Add description for vendor passthru methods https://review.openstack.org/370469 | 20:02 |
*** davidlenwell_ has joined #openstack-ironic | 20:07 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 20:09 |
*** e0ne has quit IRC | 20:10 | |
*** ccamacho has quit IRC | 20:11 | |
mariojv | has anyone using IPA with devstack modified kernel params before? rama_y and i are trying to figure out how to enable inspection with some specific collectors in devstack | 20:13 |
JayF | mariojv: assuming you already found the var in devstack, to change the kernel params passed? | 20:17 |
JayF | mariojv: that maps directly to an ironic.conf var, so you can change ironic.conf directly and restart conductors, or you'll have to change the devstack var and unstack/restack | 20:17 |
JayF | I can confirm it works because you have to use it if you onmetal+devstack or else the vm-nodes fail to boot | 20:17 |
mariojv | JayF: i didn't see a var for the kernel params in devstack | 20:17 |
* mariojv looks in ironic.conf | 20:18 | |
JayF | mariojv: $IRONIC_EXTRA_PXE_PARAMS | 20:18 |
*** davidlenwell_ has quit IRC | 20:18 | |
mariojv | \o/ thanks | 20:18 |
JayF | mariojv: which maps to CONF.pxe.pxe_append_params (but devstack adds some too, that are needed) | 20:18 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal node commands https://review.openstack.org/301687 | 20:20 |
*** zackf has joined #openstack-ironic | 20:23 | |
*** zackf has joined #openstack-ironic | 20:24 | |
*** dprince has quit IRC | 20:24 | |
*** joprovost has quit IRC | 20:25 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Set OSC default baremetal api version as in ironicclient https://review.openstack.org/385386 | 20:26 |
*** mtreinish has quit IRC | 20:31 | |
*** mtreinish has joined #openstack-ironic | 20:32 | |
openstackgerrit | Annie Lezil proposed openstack/ironic: Adding Timing metrics for DRAC drivers. https://review.openstack.org/377957 | 20:33 |
*** davidlenwell_ has joined #openstack-ironic | 20:34 | |
*** fxpester has quit IRC | 20:40 | |
*** fxpester has joined #openstack-ironic | 20:40 | |
mrda | Morning Ironic | 20:41 |
JayF | o/ | 20:41 |
jroll | have I ever complained that we require a full environment to run tox -e docs? | 20:42 |
jroll | which requires things like mysql and postgres headers? | 20:43 |
jroll | because we do and it's awful :( | 20:43 |
jroll | morning mrda | 20:43 |
JayF | I mean, how else do we fix that? | 20:44 |
JayF | if we have something like a docs-requirements.txt it'll be somewhat nonstandard | 20:44 |
mrda | hey jroll | 20:44 |
jroll | JayF: yeah idk | 20:45 |
mrda | jroll: it is a little bit frustrating, but as JayF syas, standardisation | 20:45 |
jroll | I AM IRRATIONALLY ANGRY ABOUT THIS | 20:45 |
jroll | :D | 20:45 |
mrda | (also makes doing docs work on a non-linux difficult, but I digress...) | 20:45 |
jroll | indeed | 20:45 |
jroll | like, I know how to deal with it fine, I have a VM set up and stuff | 20:46 |
jroll | but isn't friendly to newer contributors | 20:46 |
JayF | if a new contrib gave me that complaint; I'd proabbly suggest they use the gate to render their docs | 20:46 |
JayF | the jobs don't take long b/c we don't run tempest | 20:46 |
*** pester has joined #openstack-ironic | 20:47 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add basic tests for OSC plugin baremetal node commands https://review.openstack.org/301687 | 20:47 |
*** zackf has quit IRC | 20:47 | |
*** baoli_ has quit IRC | 20:48 | |
*** zackf has joined #openstack-ironic | 20:48 | |
*** baoli has joined #openstack-ironic | 20:49 | |
mrda | JayF: well, if you're going to do that, if insubtree(ironc/doc ironic/releasenotes) { # Only run docs jobs } | 20:49 |
JayF | mrda: we already (effectively) do that in the gate | 20:49 |
*** fxpester has quit IRC | 20:50 | |
openstackgerrit | Michael Turek proposed openstack/ironic: Enable PXE for systems using petitboot https://review.openstack.org/185987 | 20:50 |
mrda | JayF: It's true, I just looked again at my current docs patch (382108) and it's only the Dell Ironic CI that's running functional tests on a docs only patch | 20:52 |
*** Goneri has quit IRC | 20:53 | |
*** aNuposic has quit IRC | 20:55 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add docs about releasing ironic projects https://review.openstack.org/385616 | 20:55 |
jroll | rloo: you might have fun with that one :) | 20:55 |
rloo | jroll: oh, torture! | 20:55 |
openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 20:55 |
jroll | rloo: it's short! | 20:56 |
*** penick has quit IRC | 20:56 | |
rloo | jroll: short and sweet? :) | 20:57 |
jroll | yep :) | 20:57 |
*** pester has quit IRC | 21:00 | |
*** pester has joined #openstack-ironic | 21:01 | |
*** penick has joined #openstack-ironic | 21:01 | |
JayF | jroll: +1 with a question inline about something that should be doc'd in there | 21:02 |
jroll | JayF: yeah I probably missed something | 21:02 |
jroll | oh, good one | 21:02 |
*** mtreinish has quit IRC | 21:05 | |
*** anush_ has quit IRC | 21:08 | |
*** trown is now known as trown|outtypewww | 21:09 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add docs about releasing ironic projects https://review.openstack.org/385616 | 21:09 |
jroll | done | 21:09 |
*** vsaienko has joined #openstack-ironic | 21:10 | |
*** lekha_ has joined #openstack-ironic | 21:11 | |
*** makowals_ has joined #openstack-ironic | 21:11 | |
*** mtreinish has joined #openstack-ironic | 21:12 | |
*** alex_xu_ has joined #openstack-ironic | 21:16 | |
*** natorious_ has joined #openstack-ironic | 21:16 | |
*** rm_work| has joined #openstack-ironic | 21:16 | |
*** zigo_ has joined #openstack-ironic | 21:17 | |
*** aNuposic has joined #openstack-ironic | 21:17 | |
*** odyssey4me_ has joined #openstack-ironic | 21:17 | |
*** mgagne_ has joined #openstack-ironic | 21:17 | |
*** odyssey4me has quit IRC | 21:18 | |
*** odyssey4me_ is now known as odyssey4me | 21:18 | |
*** _vsaienko has quit IRC | 21:19 | |
*** lekha has quit IRC | 21:19 | |
*** natorious has quit IRC | 21:19 | |
*** athomas has quit IRC | 21:19 | |
*** krtaylor has quit IRC | 21:19 | |
*** jheroux has quit IRC | 21:19 | |
*** makowals has quit IRC | 21:19 | |
*** alex_xu has quit IRC | 21:19 | |
*** zigo has quit IRC | 21:19 | |
*** mgagne has quit IRC | 21:19 | |
*** gabriel-bezerra has quit IRC | 21:19 | |
*** anush has quit IRC | 21:19 | |
*** rm_work has quit IRC | 21:19 | |
*** natorious_ is now known as natorious | 21:19 | |
*** rm_work| is now known as rm_work | 21:19 | |
*** rm_work has quit IRC | 21:19 | |
*** rm_work has joined #openstack-ironic | 21:19 | |
*** lekha_ is now known as lekha | 21:20 | |
*** vsaienko has quit IRC | 21:20 | |
*** gabriel-bezerra has joined #openstack-ironic | 21:21 | |
*** anush_ has joined #openstack-ironic | 21:21 | |
*** clif_h has quit IRC | 21:23 | |
*** jmccrory has quit IRC | 21:23 | |
*** adreznec has quit IRC | 21:23 | |
*** dtantsur|afk has quit IRC | 21:23 | |
*** anush_ has quit IRC | 21:24 | |
*** mjturek has quit IRC | 21:24 | |
*** phschwartz has quit IRC | 21:24 | |
*** athomas has joined #openstack-ironic | 21:25 | |
*** krtaylor has joined #openstack-ironic | 21:25 | |
*** phschwartz has joined #openstack-ironic | 21:26 | |
*** clif_h has joined #openstack-ironic | 21:28 | |
*** jmccrory has joined #openstack-ironic | 21:28 | |
*** adreznec has joined #openstack-ironic | 21:28 | |
*** dtantsur|afk has joined #openstack-ironic | 21:28 | |
*** zigo_ is now known as zigo | 21:28 | |
*** davidlenwell_ has quit IRC | 21:31 | |
*** zackf has quit IRC | 21:32 | |
*** vsaienko has joined #openstack-ironic | 21:33 | |
jroll | TheJulia: do you need any of these blueprints? I want to disable blueprints everywhere: https://blueprints.launchpad.net/bifrost | 21:34 |
TheJulia | jroll: I forgot those were even there. Nukem :) | 21:35 |
jroll | k, thanks | 21:35 |
*** mtreinish has quit IRC | 21:36 | |
*** vsaienko has quit IRC | 21:37 | |
*** portdirect has joined #openstack-ironic | 21:40 | |
*** ChanServ sets mode: +o jroll | 21:40 | |
jroll | freenode services are going down for a bit, putting this hat on just in case | 21:40 |
*** rbrud has quit IRC | 21:41 | |
*** mtreinish has joined #openstack-ironic | 21:42 | |
*** davidlenwell_ has joined #openstack-ironic | 21:44 | |
*** ppiela has joined #openstack-ironic | 21:55 | |
*** ppiela has quit IRC | 21:56 | |
*** jralph__ has quit IRC | 21:56 | |
*** jralph_ is now known as jralph | 21:57 | |
*** vsaienko has joined #openstack-ironic | 21:57 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Add docs about releasing ironic projects https://review.openstack.org/385616 | 21:58 |
*** vsaienko has quit IRC | 22:03 | |
*** vsaienko has joined #openstack-ironic | 22:05 | |
*** mtreinish has quit IRC | 22:08 | |
*** Sukhdev has joined #openstack-ironic | 22:10 | |
*** [1]cdearborn has quit IRC | 22:17 | |
*** mtreinish has joined #openstack-ironic | 22:19 | |
*** jcoufal_ has quit IRC | 22:22 | |
*** jcoufal has joined #openstack-ironic | 22:24 | |
*** mgoddard has quit IRC | 22:25 | |
*** vsaienko has quit IRC | 22:28 | |
*** vsaienko has joined #openstack-ironic | 22:28 | |
openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Ironic multitest initial commit https://review.openstack.org/369021 | 22:29 |
jroll | rloo: JayF: I've added my thoughts to https://etherpad.openstack.org/p/ironic-ocata-summit-priorities | 22:30 |
jroll | I've probably missed obvious things | 22:30 |
jroll | and it may be more than we can do, idk | 22:30 |
*** penick has quit IRC | 22:31 | |
*** jcoufal has quit IRC | 22:33 | |
*** joprovost has joined #openstack-ironic | 22:33 | |
JayF | jroll: I'd like to keep install guide on the list for smaller things | 22:33 |
JayF | jroll: I think there's improvement still to be done and it should be considered a priority | 22:33 |
jroll | JayF: add it then | 22:34 |
jroll | :) | 22:34 |
jroll | this is an etherpad, not a ... dang what's the word | 22:34 |
jroll | edict. | 22:34 |
*** jcoufal has joined #openstack-ironic | 22:34 | |
JayF | stone table | 22:34 |
JayF | *tablet | 22:34 |
jroll | heh | 22:34 |
jroll | JayF: please add to name/color map | 22:35 |
jroll | else I'll forget | 22:35 |
JayF | I just did | 22:35 |
jroll | thanks | 22:35 |
JayF | I meant the one actualyl built into etherpad | 22:35 |
jroll | etherpad is slow, it showed up after you said "I just did" | 22:35 |
JayF | but I added to that one too | 22:35 |
jroll | oh | 22:36 |
jroll | that only helps when you have it open :P | 22:36 |
JayF | I always try to be black background/white text regardless of the etherpad instance | 22:36 |
JayF | and will proactively change it to make it that | 22:36 |
jroll | I will never remember that :P | 22:36 |
JayF | see, everyone should have a signature etherpad color | 22:37 |
jroll | I try to use pink | 22:37 |
rloo | thx jroll. the colour stuff doesn't work for me. | 22:39 |
jroll | rloo: gear icon -> authorship colors | 22:39 |
rloo | jroll: i'll look/add to that etherpad later. | 22:39 |
jroll | rloo: thanks, no rush | 22:40 |
rloo | jroll: does the color stuff stick afterwards, eg: https://etherpad.openstack.org/p/ironic-newton-summit-gate. i have no idea who wrote what | 22:41 |
rloo | jroll: you just appeared in pink heh, but otherwise, w/o people's nics... | 22:41 |
jroll | rloo: yes, the colors stay, though I think clearing cookies sometimes will change one's color | 22:41 |
*** mtreinish has quit IRC | 22:41 | |
jroll | right | 22:41 |
jroll | which is why I like putting the color map thing there | 22:42 |
*** vsaienko has quit IRC | 22:44 | |
*** vsaienko has joined #openstack-ironic | 22:44 | |
rloo | jroll: so maybe i don't know what i'm looking at. in that etherpad ^, when you showed up, i saw you as pink on the RHS/map. BUT in the actual page, i see pink, green, blue, and don't know who those people are. although it looks like dtantsur is/was also pink. | 22:44 |
jroll | rloo: right... I wasn't pink there because I've since changed my color or whatever (which can happen with a new browser session or cookies cleared) | 22:45 |
jroll | rloo: that's why I like putting a color map in the text itself, because that will stay as it was at the time of writing | 22:45 |
jroll | and match the actual text | 22:45 |
jroll | make sense? | 22:46 |
JayF | /win 23 | 22:46 |
JayF | whoops | 22:46 |
rloo | jroll: OH... ok, we should ask everyone to do that then so it becomes 'normal' to do so. | 22:46 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/385180 | 22:46 |
JayF | aiui keeping the consistant color across a single etherpad is a cookie-thing. | 22:47 |
jroll | rloo: yeah, it's usually helpful | 22:47 |
JayF | I try to be aware if mine changes, and explicitly change it back to black background. Apparently I'm the only person in the world who ever noticed I did this. | 22:47 |
JayF | lol | 22:47 |
JayF | also I always put my name in the built-in etherpad map | 22:47 |
*** mtreinish has joined #openstack-ironic | 22:47 | |
*** jcoufal has quit IRC | 22:47 | |
* jroll is done for the day, I will see you all tomorrow | 22:48 | |
* rloo wonders how many colours we can use before we can't tell the diff between the people | 22:48 | |
JayF | For me, probably like 8. Maybe someone more color-literate could go up to 16 or 32 :P | 22:49 |
rloo | ciao jroll. i need to take off too. | 22:49 |
rloo | JayF: so as long as there are < 10 eg folks commenting, we're good :) | 22:49 |
JayF | lol. yep. | 22:49 |
*** anush_ has joined #openstack-ironic | 22:49 | |
*** mjturek has joined #openstack-ironic | 22:49 | |
JayF | but I'll always be the Johnny Cash of etherpad. (i.e.; man in black) | 22:50 |
openstackgerrit | Merged openstack/ironic: Fix docstrings to match with method arguments https://review.openstack.org/370068 | 22:50 |
rloo | JayF: :) /me thinks of men in black... | 22:51 |
*** vsaienko has quit IRC | 22:51 | |
JayF | I don't own nearly a nice enough suit for that reference | 22:52 |
*** vsaienko has joined #openstack-ironic | 22:52 | |
*** athomas has quit IRC | 22:55 | |
*** yibo has quit IRC | 22:57 | |
*** yibo has joined #openstack-ironic | 22:57 | |
*** vsaienko has quit IRC | 22:58 | |
*** anush_ has quit IRC | 23:00 | |
*** mjturek has quit IRC | 23:00 | |
*** vsaienko has joined #openstack-ironic | 23:02 | |
*** zackf has joined #openstack-ironic | 23:12 | |
anteaya | its all in the sunglasses | 23:12 |
*** vsaienko has quit IRC | 23:13 | |
*** anush_ has joined #openstack-ironic | 23:15 | |
*** mjturek has joined #openstack-ironic | 23:15 | |
*** zackf has quit IRC | 23:16 | |
*** vsaienko has joined #openstack-ironic | 23:16 | |
*** yibo has quit IRC | 23:20 | |
*** yibo has joined #openstack-ironic | 23:21 | |
*** vsaienko has quit IRC | 23:21 | |
*** vsaienko has joined #openstack-ironic | 23:23 | |
*** bfournie has quit IRC | 23:30 | |
JayF | If anyone knows any UEFI experts, or is one, would love a more educated comment on https://review.openstack.org/#/c/384996/ than the one I provided. | 23:30 |
patchbot | patch 384996 - ironic-python-agent - Install grub in the EFI "Fallback path" | 23:30 |
*** yibo has quit IRC | 23:30 | |
*** yibo has joined #openstack-ironic | 23:31 | |
*** vsaienko has quit IRC | 23:32 | |
*** vsaienko has joined #openstack-ironic | 23:33 | |
*** rama_y has quit IRC | 23:36 | |
*** vsaienko has quit IRC | 23:38 | |
JayF | lucas-afk: https://review.openstack.org/#/c/322632/ has 2x+2, but I figured since you ushered through the design and had previous votes you might want to look before it lands. | 23:38 |
patchbot | patch 322632 - ironic - Support multi arch deployment | 23:38 |
*** vsaienko has joined #openstack-ironic | 23:38 | |
*** bcornec1 has quit IRC | 23:45 | |
*** bcornec1 has joined #openstack-ironic | 23:45 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!