JayF | 000000 UTC gerrit outage strikes agani | 00:02 |
---|---|---|
JayF | grargh | 00:02 |
*** ijw has quit IRC | 00:03 | |
jlvillal | JayF: Me too! :( | 00:04 |
*** rama_y has quit IRC | 00:04 | |
jlvillal | Okay, looks like short lived pause | 00:04 |
openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Implement rescue mode https://review.openstack.org/171878 | 00:04 |
JayF | they do backups daily | 00:04 |
JayF | that's why it dies like that | 00:04 |
JayF | just because of the hours I keep I'm almost always in gerrit when it happens | 00:05 |
*** Sukhdev has quit IRC | 00:07 | |
jlvillal | JayF: Yep. Happens to me | 00:07 |
jlvillal | JayF: devananda https://review.openstack.org/#/c/313187/ +2 and +A I left some comments if want to have a follow-up patch, if appropriate. | 00:07 |
*** Sukhdev has joined #openstack-ironic | 00:08 | |
*** ijw has joined #openstack-ironic | 00:09 | |
*** ijw has quit IRC | 00:11 | |
*** ijw has joined #openstack-ironic | 00:11 | |
*** rbrud has quit IRC | 00:11 | |
JayF | mikal: would love you to land these stable backports: - Nova stable backports: https://review.openstack.org/#/c/323477/ https://review.openstack.org/#/c/323196/ when you get a sec | 00:11 |
JayF | mikal: you had asked me on one of hte original patches to backport, so I went ahead and got mine + the one it depended on. Matt R has already +2'd both | 00:11 |
*** ppiela has quit IRC | 00:13 | |
JayF | jroll: [async] https://review.openstack.org/#/c/321246/ appears to have pretty good consensus, you might want to land it before it needs a rebase | 00:15 |
*** PollyZ has joined #openstack-ironic | 00:15 | |
openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Implement rescue mode https://review.openstack.org/171878 | 00:16 |
*** clenimar has quit IRC | 00:17 | |
openstackgerrit | Jay Faulkner proposed openstack/ironic-specs: Implement rescue mode https://review.openstack.org/171878 | 00:18 |
devananda | JayF: omg - I just hid configdrive from the output of node-show | 00:19 |
devananda | via policy settings | 00:19 |
* JayF applies that patch downstream immediately | 00:19 | |
JayF | lol | 00:19 |
devananda | this makes me so happy | 00:19 |
JayF | when you do that | 00:19 |
JayF | does it hide both key and value | 00:19 |
JayF | or is it like configdrive: **** | 00:20 |
devananda | no - it masks the value, just like password | 00:20 |
devananda | because i'm usiing the same oslo method | 00:20 |
JayF | perfecto | 00:20 |
*** PollyZ has quit IRC | 00:20 | |
openstackgerrit | Merged openstack/ironic: Update the api-ref documentation for Ports https://review.openstack.org/313187 | 00:20 |
devananda | oslo_utils.strutils.mask_password() already checks for a property named "configdrive" | 00:20 |
devananda | we just never applied it to instance_info -- or created a separate policy line for it | 00:20 |
devananda | soooo easy | 00:20 |
JayF | you're making all this seem to easy | 00:21 |
devananda | yah | 00:21 |
devananda | do you think that should be a separate patch? or rolled in? | 00:21 |
JayF | here's the $1,000,000 question: | 00:21 |
devananda | ..... | 00:21 |
JayF | would our backwards compat block-password stuff trigger it | 00:21 |
devananda | huh? | 00:21 |
JayF | i.e. existing behavior would change or not | 00:21 |
JayF | grr not finding words | 00:22 |
JayF | https://github.com/openstack/ironic/blob/master/etc/ironic/policy.json#L3 | 00:22 |
JayF | is that going to be the same rule that blocks configdrive? | 00:22 |
JayF | or would it be something different; i.e. show_configdrive | 00:22 |
devananda | i added this as a new rule for now | 00:23 |
devananda | so | 00:23 |
JayF | as long as it's a new rule it's backward compat and awesome and I don't care if it's a different patc | 00:24 |
JayF | *patch | 00:24 |
JayF | or not | 00:24 |
devananda | someone updating to this ironic code will NEED to update their policy.json | 00:24 |
devananda | I want to ensure that they don't also need to update the roles in their cloud | 00:24 |
devananda | which I think we do by preserving the is_admin rule | 00:24 |
devananda | and allowing that override to still override everything else | 00:24 |
JayF | I'm not sure I understand why you'd be required to update your policy.json | 00:24 |
devananda | hmm | 00:25 |
devananda | maybe you're not. lemme check | 00:25 |
JayF | shouldn't our default policies be defined in code, re: the oslo spec anyway? Even if not it doesn't seem like you'd need to update it | 00:25 |
devananda | I may be mistaken about that | 00:26 |
devananda | I think policy.enforce defaults to pass when there is no such rule defined | 00:26 |
devananda | or defaults to the default rule (which is "is_admin") | 00:26 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: [WIP] Mask configdrive contents in API responses https://review.openstack.org/326768 | 00:26 |
devananda | JayF: there's the change ^ | 00:26 |
JayF | awesome; first thing on my list tomorrow is to read all that code and write the spec | 00:27 |
JayF | I'm not sure I should do it in that order, but that's what I'm doing | 00:27 |
JayF | lol | 00:27 |
devananda | hah | 00:27 |
JayF | which came first, the spec or the code | 00:27 |
JayF | doesn't matter; has policy | 00:27 |
devananda | :-D | 00:27 |
*** mtanino has quit IRC | 00:27 | |
devananda | in this case, I think the spec is going to be more about documenting the impact | 00:27 |
devananda | and then turning that into actual docs | 00:28 |
JayF | all the patches are labelled WIP, but do you want review on them yet at all? | 00:28 |
devananda | go for it | 00:28 |
JayF | wondering how much you expect them to mutate; should I read for review or read for knowledge | 00:28 |
devananda | they're ready for reviews | 00:28 |
JayF | a-ok will review them | 00:28 |
devananda | knowledge | 00:28 |
devananda | I WIP'd them because I don't want them to land until we have grenade | 00:28 |
JayF | That's very sensible. | 00:28 |
JayF | That's close, isn't it? | 00:28 |
devananda | yah | 00:28 |
devananda | I think jlvillal said there's like 3 patches outstanding at this point | 00:28 |
devananda | I'm happy to wait a week or three for these to land | 00:29 |
*** PollyZ has joined #openstack-ironic | 00:29 | |
devananda | if it means better confidence that it won't break the world | 00:29 |
JayF | That works. I feel like this cycle has been crazy productive so far. | 00:29 |
JayF | Maybe because all the attention on the gate has made things go more smoothly | 00:29 |
devananda | maybe that is the stuff we've done ;) | 00:29 |
*** clenimar has joined #openstack-ironic | 00:30 | |
JayF | Alright I'm getting off IRC for the day. Awesome stuff, seriously thanks for all this. | 00:31 |
JayF | o/ | 00:31 |
*** amotoki has joined #openstack-ironic | 00:31 | |
devananda | ciao! | 00:31 |
*** PollyZ has quit IRC | 00:33 | |
*** amotoki has quit IRC | 00:36 | |
*** Sukhdev has quit IRC | 00:37 | |
*** PollyZ has joined #openstack-ironic | 00:42 | |
*** Goneri has joined #openstack-ironic | 00:42 | |
*** hoangcx has joined #openstack-ironic | 00:45 | |
*** PollyZ has quit IRC | 00:47 | |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: [WIP] Mask configdrive contents in API responses https://review.openstack.org/326768 | 00:49 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: [WIP] Mask configdrive contents in API responses https://review.openstack.org/326768 | 00:50 |
*** Jabadia has joined #openstack-ironic | 00:50 | |
*** PollyZ has joined #openstack-ironic | 00:56 | |
*** baoli has joined #openstack-ironic | 00:57 | |
*** PollyZ has quit IRC | 01:00 | |
*** jaybeale_ has quit IRC | 01:07 | |
*** Goneri has quit IRC | 01:07 | |
*** PollyZ has joined #openstack-ironic | 01:09 | |
openstackgerrit | Merged openstack/ironic: Update the api-ref documentation for Drivers https://review.openstack.org/313708 | 01:12 |
*** PollyZ has quit IRC | 01:14 | |
*** PollyZ has joined #openstack-ironic | 01:23 | |
*** PollyZ has quit IRC | 01:28 | |
*** phuongnh has joined #openstack-ironic | 01:29 | |
*** frontrunner has quit IRC | 01:31 | |
*** amotoki has joined #openstack-ironic | 01:32 | |
*** ijw has quit IRC | 01:32 | |
*** amotoki has quit IRC | 01:36 | |
*** PollyZ has joined #openstack-ironic | 01:37 | |
*** ChrisAusten has joined #openstack-ironic | 01:38 | |
*** PollyZ has quit IRC | 01:41 | |
openstackgerrit | Merged openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/325451 | 01:46 |
*** sacharya has joined #openstack-ironic | 01:59 | |
*** rloo has quit IRC | 01:59 | |
*** PollyZ has joined #openstack-ironic | 02:09 | |
devananda | JayF: traced down why all the tests are failing for my patches: http://logs.openstack.org/99/325599/4/check/gate-tempest-dsvm-ironic-pxe_ipa/c9c8a11/logs/screen-ir-api.txt.gz#_2016-06-07_23_17_51_520 | 02:14 |
devananda | no response needed, just interesting for you | 02:14 |
*** Jabadia has quit IRC | 02:16 | |
*** PollyZ has quit IRC | 02:24 | |
*** PollyZ has joined #openstack-ironic | 02:28 | |
*** sdake has joined #openstack-ironic | 02:32 | |
*** amotoki has joined #openstack-ironic | 02:33 | |
*** PollyZ has quit IRC | 02:33 | |
*** moshele has joined #openstack-ironic | 02:34 | |
*** mtanino has joined #openstack-ironic | 02:34 | |
*** sdake has quit IRC | 02:34 | |
*** amotoki has quit IRC | 02:37 | |
*** openstackgerrit has quit IRC | 02:47 | |
*** openstackgerrit has joined #openstack-ironic | 02:48 | |
*** gcb has joined #openstack-ironic | 02:52 | |
*** rama_y has joined #openstack-ironic | 02:59 | |
*** moshele has quit IRC | 02:59 | |
*** kalpase has joined #openstack-ironic | 03:07 | |
*** mtanino has quit IRC | 03:08 | |
*** PollyZ has joined #openstack-ironic | 03:09 | |
*** PollyZ has quit IRC | 03:14 | |
*** fragatina has quit IRC | 03:19 | |
*** PollyZ has joined #openstack-ironic | 03:23 | |
*** PollyZ has quit IRC | 03:28 | |
*** sdake has joined #openstack-ironic | 03:28 | |
*** spandhe has quit IRC | 03:32 | |
*** amotoki has joined #openstack-ironic | 03:34 | |
*** links has joined #openstack-ironic | 03:35 | |
*** PollyZ has joined #openstack-ironic | 03:36 | |
*** amotoki has quit IRC | 03:38 | |
*** PollyZ has quit IRC | 03:41 | |
*** absubram has joined #openstack-ironic | 03:41 | |
*** PollyZ has joined #openstack-ironic | 03:50 | |
*** clenimar has quit IRC | 03:51 | |
*** sdake has quit IRC | 03:53 | |
*** PollyZ has quit IRC | 03:54 | |
*** vmud213 has joined #openstack-ironic | 04:01 | |
*** devananda has quit IRC | 04:11 | |
*** fragatina has joined #openstack-ironic | 04:17 | |
*** fragatina has quit IRC | 04:22 | |
*** devananda has joined #openstack-ironic | 04:23 | |
*** SiRiuS__ has joined #openstack-ironic | 04:26 | |
openstackgerrit | Merged openstack/proliantutils: Add RIS support for power operations https://review.openstack.org/289800 | 04:28 |
*** PollyZ has joined #openstack-ironic | 04:31 | |
*** Sukhdev has joined #openstack-ironic | 04:33 | |
*** baoli has quit IRC | 04:34 | |
*** amotoki has joined #openstack-ironic | 04:34 | |
*** PollyZ has quit IRC | 04:35 | |
*** fragatina has joined #openstack-ironic | 04:38 | |
*** amotoki has quit IRC | 04:39 | |
*** fragatina has quit IRC | 04:40 | |
*** fragatina has joined #openstack-ironic | 04:40 | |
*** fragatin_ has joined #openstack-ironic | 04:41 | |
*** garthb has joined #openstack-ironic | 04:42 | |
*** fragatina has quit IRC | 04:44 | |
*** vishwanathj has joined #openstack-ironic | 04:51 | |
*** garthb has quit IRC | 04:51 | |
*** spandhe has joined #openstack-ironic | 04:53 | |
*** spandhe_ has joined #openstack-ironic | 04:54 | |
*** PollyZ has joined #openstack-ironic | 04:58 | |
*** spandhe has quit IRC | 04:58 | |
*** spandhe_ is now known as spandhe | 04:58 | |
*** ChubYann has quit IRC | 04:59 | |
*** sdake has joined #openstack-ironic | 05:02 | |
*** PollyZ has quit IRC | 05:02 | |
*** PollyZ has joined #openstack-ironic | 05:11 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: Enhance Power Interface for Soft Power Off and Inject NMI https://review.openstack.org/186700 | 05:12 |
*** PollyZ has quit IRC | 05:16 | |
*** moshele has joined #openstack-ironic | 05:20 | |
*** sacharya has quit IRC | 05:22 | |
openstackgerrit | Aparna proposed openstack/proliantutils: Passing 'process_input' to the running hpssacli process https://review.openstack.org/325730 | 05:22 |
*** PollyZ has joined #openstack-ironic | 05:25 | |
*** PollyZ has quit IRC | 05:30 | |
*** sdake has quit IRC | 05:32 | |
*** amotoki has joined #openstack-ironic | 05:35 | |
*** fragatin_ has quit IRC | 05:36 | |
*** spandhe has quit IRC | 05:37 | |
*** PollyZ has joined #openstack-ironic | 05:39 | |
*** amotoki has quit IRC | 05:39 | |
*** PollyZ has quit IRC | 05:43 | |
*** rama_y has quit IRC | 05:44 | |
*** PollyZ has joined #openstack-ironic | 05:52 | |
*** mtanino has joined #openstack-ironic | 05:56 | |
*** PollyZ has quit IRC | 05:57 | |
*** stendulker has joined #openstack-ironic | 05:59 | |
*** jtomasek has quit IRC | 06:01 | |
*** ChrisAusten has quit IRC | 06:05 | |
*** PollyZ has joined #openstack-ironic | 06:06 | |
*** PollyZ has quit IRC | 06:11 | |
*** lifeless has quit IRC | 06:13 | |
*** rcernin has joined #openstack-ironic | 06:13 | |
*** PollyZ has joined #openstack-ironic | 06:19 | |
*** e0ne has joined #openstack-ironic | 06:21 | |
*** sacharya has joined #openstack-ironic | 06:22 | |
*** PollyZ has quit IRC | 06:25 | |
*** sacharya has quit IRC | 06:27 | |
*** purp has quit IRC | 06:30 | |
*** PollyZ has joined #openstack-ironic | 06:33 | |
*** Sukhdev has quit IRC | 06:33 | |
*** purp has joined #openstack-ironic | 06:34 | |
*** vishwanathj has quit IRC | 06:36 | |
*** vishwanathj has joined #openstack-ironic | 06:36 | |
*** PollyZ has quit IRC | 06:37 | |
*** loki_ has joined #openstack-ironic | 06:44 | |
*** openstackgerrit has quit IRC | 06:48 | |
*** gcb has quit IRC | 06:48 | |
*** openstackgerrit has joined #openstack-ironic | 06:48 | |
*** rbartal has joined #openstack-ironic | 06:53 | |
*** sdake has joined #openstack-ironic | 06:54 | |
*** itamarl has joined #openstack-ironic | 06:57 | |
*** priteau has joined #openstack-ironic | 06:59 | |
*** tesseract has joined #openstack-ironic | 07:00 | |
*** e0ne has quit IRC | 07:06 | |
*** sdake has quit IRC | 07:10 | |
*** rbartal has quit IRC | 07:13 | |
*** gcb has joined #openstack-ironic | 07:13 | |
*** PollyZ has joined #openstack-ironic | 07:14 | |
*** rbartal has joined #openstack-ironic | 07:14 | |
*** PollyZ has quit IRC | 07:19 | |
*** mtanino has quit IRC | 07:19 | |
*** ifarkas has joined #openstack-ironic | 07:20 | |
*** watanabe_isao has joined #openstack-ironic | 07:20 | |
*** sacharya has joined #openstack-ironic | 07:24 | |
*** mjura has joined #openstack-ironic | 07:25 | |
*** sacharya has quit IRC | 07:28 | |
*** watanabe_isao has quit IRC | 07:31 | |
openstackgerrit | vinay kumar muddu proposed openstack/ironic-python-agent: Wait for at least one interface before node lookup https://review.openstack.org/303503 | 07:32 |
*** amotoki has joined #openstack-ironic | 07:36 | |
*** amotoki has quit IRC | 07:41 | |
*** dtantsur|afk is now known as dtantsur | 07:44 | |
dtantsur | Morning Ironic | 07:44 |
*** openstackgerrit has quit IRC | 07:48 | |
*** openstackgerrit has joined #openstack-ironic | 07:48 | |
openstackgerrit | vinay kumar muddu proposed openstack/ironic-python-agent: Wait for at least one interface before node lookup https://review.openstack.org/303503 | 07:53 |
openstackgerrit | Zhenguo Niu proposed openstack/ironic-lib: Include wipefs --force option https://review.openstack.org/319918 | 07:53 |
*** PollyZ has joined #openstack-ironic | 07:55 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Return HTTP 500 from /v1/continue on unexpected exceptions https://review.openstack.org/326891 | 07:56 |
*** PollyZ has quit IRC | 08:00 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-ironic | 08:02 | |
*** jtomasek has joined #openstack-ironic | 08:05 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 08:10 |
*** lucas-dinner is now known as lucasagomes | 08:10 | |
lucasagomes | morning all | 08:10 |
dtantsur | hey morning lucasagomes | 08:11 |
*** bcornec1 has joined #openstack-ironic | 08:12 | |
*** vishwanathj has quit IRC | 08:14 | |
vmud213 | morning lucasagomes,dtantsur and all | 08:19 |
*** rcarrillocruz has joined #openstack-ironic | 08:20 | |
bcornec1 | Hello again ! | 08:22 |
*** aarefiev_ is now known as aarefiev | 08:26 | |
aarefiev | morning! | 08:26 |
*** ohamada has joined #openstack-ironic | 08:27 | |
*** kromanenko__ has joined #openstack-ironic | 08:28 | |
*** Romanenko_K has joined #openstack-ironic | 08:28 | |
*** kromanenko_ has joined #openstack-ironic | 08:28 | |
pas-ha | morning Ironic | 08:28 |
rcarrillocruz | morning folks | 08:33 |
rcarrillocruz | question | 08:33 |
rcarrillocruz | can i add arbitrary to ironic nodes metadata (or properties) ? | 08:33 |
rcarrillocruz | or only a set of keys that is validated in the backend? | 08:33 |
alineb | morning all | 08:34 |
*** PollyZ has joined #openstack-ironic | 08:36 | |
*** dmk0202 has joined #openstack-ironic | 08:37 | |
*** amotoki has joined #openstack-ironic | 08:37 | |
*** kalpase1 has joined #openstack-ironic | 08:38 | |
*** kalpase1 has left #openstack-ironic | 08:38 | |
*** kalpase has quit IRC | 08:39 | |
*** PollyZ has quit IRC | 08:41 | |
*** amotoki has quit IRC | 08:42 | |
*** derekh has joined #openstack-ironic | 08:42 | |
*** kalpase has joined #openstack-ironic | 08:44 | |
*** openstackgerrit has quit IRC | 08:48 | |
*** openstackgerrit has joined #openstack-ironic | 08:48 | |
*** PollyZ has joined #openstack-ironic | 08:49 | |
*** e0ne has joined #openstack-ironic | 08:50 | |
*** PollyZ has quit IRC | 08:54 | |
*** vishwanathj has joined #openstack-ironic | 08:55 | |
*** dtantsur is now known as dtantsur|bbl | 08:57 | |
*** _milan_ has joined #openstack-ironic | 08:59 | |
_milan_ | morning Ironic! :) | 09:00 |
rcarrillocruz | wow, i wrote it really badly | 09:03 |
rcarrillocruz | can i add arbitrary *metadata/properties* to ironic nodes? | 09:03 |
*** jtomasek has quit IRC | 09:04 | |
*** electrofelix has joined #openstack-ironic | 09:13 | |
lucasagomes | rcarrillocruz, yup, it's a json field so you basically can add whatever you want | 09:14 |
lucasagomes | same for other json fields (extra, driver_info, etc...) | 09:14 |
rcarrillocruz | excellent, thx lucasagomes | 09:14 |
*** sdake has joined #openstack-ironic | 09:16 | |
*** PollyZ has joined #openstack-ironic | 09:16 | |
lucasagomes | yw | 09:17 |
*** chenke has joined #openstack-ironic | 09:17 | |
*** jtomasek has joined #openstack-ironic | 09:18 | |
*** karthi has joined #openstack-ironic | 09:18 | |
karthi | Hi Ironics | 09:18 |
karthi | guys, i need help in applying the fix https://bugs.launchpad.net/glance/+bug/1561947 | 09:20 |
openstack | Launchpad bug 1561947 in Glance "Glance Store fails to authenticate to Swift with Keystone v3 API" [High,In progress] - Assigned to Kairat Kushaev (kkushaev) | 09:20 |
*** PollyZ has quit IRC | 09:21 | |
*** chenke has quit IRC | 09:21 | |
lucasagomes | karthi, I think you may want to check the #openstack-glance channel | 09:24 |
*** sacharya has joined #openstack-ironic | 09:24 | |
*** sacharya has quit IRC | 09:29 | |
*** PollyZ has joined #openstack-ironic | 09:30 | |
*** PollyZ has quit IRC | 09:35 | |
*** amotoki has joined #openstack-ironic | 09:38 | |
sambetts|afk | Morning all o/ | 09:41 |
*** sambetts|afk is now known as sambetts | 09:41 | |
*** mgould|afk is now known as mgould | 09:41 | |
mgould | morning sambetts | 09:41 |
mgould | morning all :-) | 09:41 |
sambetts | hey mgould | 09:41 |
*** amotoki has quit IRC | 09:42 | |
*** PollyZ has joined #openstack-ironic | 09:44 | |
lucasagomes | mgould, sambetts morning | 09:44 |
mgould | morning lucasagomes | 09:44 |
vdrok | morning ironic, sambetts mgould lucasagomes | 09:44 |
mgould | morning vdrok | 09:45 |
*** PollyZ has quit IRC | 09:49 | |
*** sdake has quit IRC | 09:49 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Gracefully degrade start_iscsi_target for Mitaka ramdisk https://review.openstack.org/319183 | 09:55 |
*** amotoki has joined #openstack-ironic | 09:56 | |
*** PollyZ has joined #openstack-ironic | 09:57 | |
*** PollyZ has quit IRC | 10:02 | |
*** vishwanathj has quit IRC | 10:08 | |
*** dtantsur|bbl is now known as dtantsur | 10:11 | |
dtantsur | morning vmud213, aarefiev, pas-ha, sambetts, vdrok | 10:11 |
*** PollyZ has joined #openstack-ironic | 10:11 | |
vdrok | good morning dtantsur ! | 10:11 |
sambetts | Morning vdrok dtantsur | 10:11 |
*** mjura has quit IRC | 10:12 | |
aarefiev | morning dtantsur, sambetts, vdrok | 10:13 |
vdrok | o/ aarefiev | 10:14 |
*** mjura has joined #openstack-ironic | 10:15 | |
*** PollyZ has quit IRC | 10:16 | |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Tests for testing node-create command https://review.openstack.org/262055 | 10:18 |
*** hoangcx has quit IRC | 10:22 | |
mgould | morning dtantsur | 10:23 |
dtantsur | morning mgould | 10:24 |
*** PollyZ has joined #openstack-ironic | 10:25 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Fix nits related to Ports api-ref https://review.openstack.org/326978 | 10:25 |
*** PollyZ has quit IRC | 10:29 | |
*** mbound has joined #openstack-ironic | 10:29 | |
*** lifeless has joined #openstack-ironic | 10:35 | |
vmud213 | morning sambetts,vdrok,mgould,aarefiev | 10:37 |
mgould | morning vmud213 | 10:38 |
*** PollyZ has joined #openstack-ironic | 10:38 | |
vdrok | hi vmud213 | 10:39 |
*** loki_ has quit IRC | 10:40 | |
*** PollyZ has quit IRC | 10:43 | |
sambetts | Hi aarefiev, vmud213 | 10:46 |
*** lifeless has quit IRC | 10:47 | |
*** lifeless_ has joined #openstack-ironic | 10:47 | |
*** tesseract has quit IRC | 10:50 | |
*** karthi has quit IRC | 10:51 | |
*** tesseract has joined #openstack-ironic | 10:51 | |
*** PollyZ has joined #openstack-ironic | 10:52 | |
*** PollyZ has quit IRC | 10:56 | |
*** d0ugal has quit IRC | 10:59 | |
*** d0ugal has joined #openstack-ironic | 11:00 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Document packing and unpacking the deploy ramdisk https://review.openstack.org/326995 | 11:01 |
dtantsur | lucasagomes, ^^^ | 11:01 |
dtantsur | I've found an easy way to not forget it any more :) | 11:01 |
lucasagomes | dtantsur, hah good, will take a look | 11:01 |
mgould | dtantsur++ | 11:02 |
sambetts | dtantsur, lucasagomes: I'm tempted to make the pack part of the tinyipa build a separate make alias, so you could modify the tinyipa final dir, then just run make pack or something | 11:03 |
sambetts | so then the manual workflow would be make build, make finalise, make pack | 11:03 |
lucasagomes | sambetts, yeah that's good too | 11:03 |
sambetts | or something | 11:03 |
lucasagomes | tho the commands that dtantsur uses apply to all ramdisks | 11:04 |
lucasagomes | I mean, most of them... you may still need to use unsquashfs for some | 11:04 |
sambetts | there are a couple of parts of the ramdisk build that kinda apply to all ramdisks, like the make iso part | 11:04 |
lucasagomes | and I don't think it works with dracut microcode | 11:04 |
mgould | sambetts: have `pack` depend on `finalise` depend on `build`, so you can just run `make pack` | 11:04 |
mgould | use the dependency-tracking system for what it's good at :-) | 11:05 |
*** PollyZ has joined #openstack-ironic | 11:05 | |
sambetts | mgould: I have it so that the default, "make" does build, finalise right now and "make all" does build, finalise, iso | 11:06 |
* mgould nods | 11:06 | |
*** stendulker has quit IRC | 11:06 | |
* sambetts hasn't much experience with complex make files | 11:07 | |
* mgould has some experience with them and is happy to help | 11:07 | |
dtantsur | I think this documentation update is at least useful for people who already have the ramdisks now | 11:08 |
sambetts | ++ | 11:08 |
dtantsur | e.g. I get this question downstream pretty often :) | 11:08 |
mgould | so "pack" means "make a cpio.gz file" | 11:08 |
mgould | and that's required by `make iso`? | 11:08 |
sambetts | mgould: yup | 11:08 |
*** d0ugal has quit IRC | 11:09 | |
sambetts | tbh it should really be "make tinyipa.gz" to take advantage of makes existing file check | 11:09 |
*** bcornec1 has quit IRC | 11:09 | |
mgould | yep | 11:09 |
*** phuongnh has quit IRC | 11:10 | |
mgould | but we could easily add a `pack` target which depends on tinyipa.gz, if it would make users' lives easier | 11:10 |
*** PollyZ has quit IRC | 11:10 | |
*** e0ne_ has joined #openstack-ironic | 11:11 | |
sambetts | I guess its a more readable targe t | 11:11 |
sambetts | mgould: tbh I thought about break all the environment variables out of the scripts for the tinyipa build, and having them sourced/configured in the make file or something, because several of them are shared/defined in both scripts | 11:13 |
*** e0ne has quit IRC | 11:13 | |
mgould | right, yep | 11:15 |
mgould | does the build process depend on the user's environment? That's not good | 11:15 |
sambetts | mgould: there are defaults for all the variables | 11:17 |
sambetts | but they can be overridden by setting env varaibles | 11:18 |
mgould | right, cool | 11:18 |
sambetts | for example export BUILD_AND_INSTALL_TINYIPA=false will override the default true | 11:18 |
dtantsur | lucasagomes, what do you mean by "this also doesn't work with ramdisks built with dracut microcode"? this is known to work with our downstream ramdisks based on dracut.. | 11:19 |
*** PollyZ has joined #openstack-ironic | 11:19 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Document packing and unpacking the deploy ramdisk https://review.openstack.org/326995 | 11:23 |
dtantsur | added a note about coreos ^^^ | 11:23 |
* dtantsur <= coffee | 11:23 | |
dtantsur | #pixiesay time for coffee! | 11:23 |
PixieBoots | ʕ•͡ᴥ•ʔ: time for coffee! | 11:23 |
*** PollyZ has quit IRC | 11:24 | |
lucasagomes | dtantsur, https://access.redhat.com/solutions/901053 | 11:24 |
lucasagomes | dtantsur, depends on how it's built | 11:25 |
lucasagomes | if it does use microcode or not | 11:25 |
*** sacharya has joined #openstack-ironic | 11:25 | |
lucasagomes | I wouldn't -1 for that, it's not a common practice to build the deploy ramdisks with microcode, but possible | 11:25 |
jroll | JayF: doh, thought I landed that :/ thanks | 11:28 |
jroll | morning everyone | 11:28 |
*** sacharya has quit IRC | 11:30 | |
sambetts | Morning jroll | 11:30 |
openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Negative tests for testing chassis-create command https://review.openstack.org/322788 | 11:30 |
lucasagomes | jroll, morning | 11:31 |
*** d0ugal has joined #openstack-ironic | 11:32 | |
* lucasagomes goes grab some lunch | 11:35 | |
*** lucasagomes is now known as lucas-afk | 11:35 | |
*** baoli has joined #openstack-ironic | 11:40 | |
vmud213 | dtantsur: For the packing and unpacking od deploy image, should not it be done with root? | 11:41 |
*** baoli_ has joined #openstack-ironic | 11:41 | |
vmud213 | dtantsur:Or may be with sudo? | 11:41 |
*** Goneri has joined #openstack-ironic | 11:42 | |
*** sdake has joined #openstack-ironic | 11:44 | |
*** baoli has quit IRC | 11:45 | |
*** gcb has quit IRC | 11:46 | |
*** PollyZ has joined #openstack-ironic | 11:46 | |
*** kalpase has left #openstack-ironic | 11:47 | |
*** yolanda has joined #openstack-ironic | 11:49 | |
openstackgerrit | Merged openstack/proliantutils: Passing 'process_input' to the running hpssacli process https://review.openstack.org/325730 | 11:50 |
*** PollyZ has quit IRC | 11:51 | |
openstackgerrit | Merged openstack/ironic: Add 'How to get a decision on something' to FAQ https://review.openstack.org/321246 | 11:52 |
dtantsur | vmud213, no | 11:56 |
dtantsur | lucas-afk, thanks! strangely enough, our ramdisks are not like that. I've packed and unpacked them numerous times | 11:57 |
dtantsur | morning jroll! do you remember why we didn't release IPA mitaka 1.2.2? | 11:58 |
vsaienko1 | Morning lucasagomes, dtantusr, vmud213, sambetts, vdrok and all Ironic'ers | 11:58 |
dtantsur | hey vsaienko1 | 11:58 |
*** ifarkas has quit IRC | 11:58 | |
*** PollyZ has joined #openstack-ironic | 12:00 | |
openstackgerrit | Sam Betts proposed openstack/ironic: [WIP] Add Network Interface Attach and Detach APIs https://review.openstack.org/327046 | 12:00 |
jroll | dtantsur: https://review.openstack.org/#/c/324884/ | 12:00 |
dtantsur | ah awesome :) | 12:00 |
jroll | in fact, https://review.openstack.org/#/q/project:openstack/releases+owner:%22Jim+Rollenhagen+%253Cjim%2540jimrollenhagen.com%253E%22+status:open | 12:01 |
*** bcornec1 has joined #openstack-ironic | 12:01 | |
jroll | matt just got to them yesterday so they should go out today I think | 12:01 |
dtantsur | cool, thanks, jroll | 12:02 |
*** vmud213 has quit IRC | 12:02 | |
dtantsur | heh, inspector release features only requirements :) I guess we still have to do it, right? | 12:02 |
*** PollyZ has quit IRC | 12:05 | |
jroll | dtantsur: yeah, it doesn't hurt | 12:05 |
jroll | I just did everything stablel because why not | 12:05 |
dtantsur | anyway, release often :) | 12:06 |
*** ifarkas has joined #openstack-ironic | 12:06 | |
jroll | ++ | 12:06 |
jroll | I'm releasing ironic 6.0 as soon as grenade is done | 12:06 |
dtantsur | woohoo | 12:06 |
jroll | s/done/working/ | 12:06 |
sambetts | \o/ | 12:06 |
dtantsur | jroll, do you want to wait for us to drop the old ramdisk? initially we planned it for 6.0 | 12:06 |
*** ayoung has quit IRC | 12:07 | |
jroll | dtantsur: oh yeah ++ | 12:08 |
jroll | are you working on that? | 12:08 |
dtantsur | jroll, no, but if we want it soon, I can start right now | 12:09 |
dtantsur | I was wondering if we want grenade first.. | 12:09 |
jroll | dtantsur: idk, I think it's fine without, we're only removing things and we don't test the bash ramdisk | 12:11 |
jroll | might be useful though, could base it on that patch | 12:11 |
*** clenimar has joined #openstack-ironic | 12:11 | |
dtantsur | jroll, ok, I'll start hacking, and then we decide if we want grenade first or not before landing it | 12:12 |
dtantsur | will start today, just let me enjoy my bit of tripleo-heat-templates debugging >_< | 12:12 |
jroll | dtantsur: "enjoy" :) | 12:12 |
jroll | thanks | 12:12 |
dtantsur | oh, I do :D | 12:12 |
mgould | sounds like Type 2 fun: http://www.urbandictionary.com/define.php?term=type%202%20fun | 12:13 |
dtantsur | jroll, actually, I'm working on making tripleo install ironic in overcloud. so it's a bit interesting as well | 12:13 |
jroll | oh, cool | 12:13 |
*** sdake has quit IRC | 12:13 | |
dtantsur | also expect me to have much more questions about serving bare metals to tenants :D | 12:13 |
jroll | hehe | 12:13 |
dtantsur | TIL type 2 fun, looks very related, thanks mgould | 12:13 |
*** frontrunner has joined #openstack-ironic | 12:14 | |
*** dprince has joined #openstack-ironic | 12:15 | |
rbartal | dtantsur, lucas-afk jroll Hi all, I open a bug about the failed tear down after cleaning we talked about yesterday | 12:16 |
rbartal | #1590408 Node cleaning fails with 'Failed to tear down from cleaning for node UUID' | 12:16 |
*** itamarl has quit IRC | 12:16 | |
dtantsur | thnx! | 12:16 |
jroll | ty rbartal | 12:17 |
rbartal | jroll, BTW, it is happens every time, I tested it on BM and virt | 12:18 |
rbartal | *it happens | 12:18 |
jroll | O_o | 12:18 |
vdrok | morning jroll, a question, have you had a chance to look at https://review.openstack.org/#/q/topic:ipa-partition-images ? | 12:19 |
jroll | I wonder if it's a combination of the user in the original context not having access to list networks in neutron, and the bug sambetts mentioned | 12:19 |
jroll | vdrok: I have not, sorry | 12:19 |
vdrok | jroll: it's pretty small, the only thing I'm wondering is - is it ok to use ephemeral size to pick the image for tempest tests, eg if ephemeral size == 0, use whole disk, otherwise use partition image | 12:20 |
jroll | vdrok: hmm, probably? not sure really | 12:21 |
vdrok | or better to introduce another parameter to request either whole disk or not. but our builder in project-config really becoming huge :) | 12:22 |
jroll | yeah, I agree, I think I'm fine with it | 12:22 |
jroll | dtantsur: ^ wdyt | 12:22 |
dtantsur | jroll, we definitely have to do something about our project-config, but otherwise I'm fine with it | 12:23 |
jroll | yeah | 12:23 |
dtantsur | we already have pxe vs ipxe chosen implicitly | 12:23 |
*** jcoufal has joined #openstack-ironic | 12:23 | |
jroll | I have on my todo list to write a plan for that | 12:23 |
vdrok | dtantsur: jroll OK, will unwip these then | 12:23 |
jroll | and for splitting api tests to a separate job | 12:23 |
jroll | and maybe most jobs testing deploys without nova? many of these options shouldn't affect nova | 12:24 |
dtantsur | jroll, my next question was if we plan to use simple testing (bifrost based?) for most of the jobs :) | 12:26 |
*** PollyZ has joined #openstack-ironic | 12:27 | |
jroll | dtantsur: yeah, I'd like to, probably just build a deploy() method into our tempest client | 12:27 |
dtantsur | cool | 12:27 |
vdrok | jroll: dtantsur btw I have this old patch - https://review.openstack.org/#/c/293434/1/ironic_tempest_plugin/tests/scenario/test_baremetal_standalone_ops.py@209, maybe need to fix it properly and move it to base baremetal_manager | 12:29 |
jroll | vdrok: ++ | 12:30 |
*** trown|outtypewww is now known as trown | 12:31 | |
*** PollyZ has quit IRC | 12:32 | |
*** itamarl has joined #openstack-ironic | 12:33 | |
mat128 | vdrok: looks pretty much like what we need :) | 12:33 |
vdrok | morning mat128 | 12:34 |
mat128 | good morning :) | 12:34 |
mgould | morning mat128 | 12:35 |
*** jjohnson2 has joined #openstack-ironic | 12:39 | |
*** vishwanathj has joined #openstack-ironic | 12:41 | |
*** PollyZ has joined #openstack-ironic | 12:41 | |
TheJulia | good morning | 12:43 |
*** lucas-afk is now known as lucasagomes | 12:44 | |
xavierr | good morning Ironic | 12:44 |
xavierr | morning dtantsur vdrok mat128 mgould TheJulia | 12:45 |
lucasagomes | rbartal, thanks | 12:45 |
xavierr | good morning lucasagomes o/ | 12:45 |
lucasagomes | xavierr, hi there, good morning | 12:45 |
*** PollyZ has quit IRC | 12:45 | |
jroll | sambetts: neutron still doesn't have wild-card dhcp, right? | 12:47 |
sambetts | jroll: no, and annoying the reviews on my specs for it have stalled | 12:48 |
sambetts | annoyingly* | 12:48 |
*** thrash has quit IRC | 12:48 | |
jroll | sambetts: naturally | 12:48 |
*** thrash has joined #openstack-ironic | 12:50 | |
*** thrash has quit IRC | 12:50 | |
*** thrash has joined #openstack-ironic | 12:50 | |
*** PollyZ has joined #openstack-ironic | 12:54 | |
*** PollyZ has quit IRC | 12:59 | |
openstackgerrit | Chuck Carmack proposed openstack/bifrost: Add redeploy-dynamic playbook https://review.openstack.org/325379 | 12:59 |
*** ccarmack has joined #openstack-ironic | 13:00 | |
*** nico-afk is now known as nicodemos | 13:01 | |
nicodemos | good morning | 13:01 |
*** links has quit IRC | 13:03 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs: Collect system logs from IPA https://review.openstack.org/323511 | 13:09 |
lucasagomes | dtantsur, ^ updated to cover the always retrieve case ^ | 13:09 |
*** rloo has joined #openstack-ironic | 13:10 | |
TheJulia | good morning rloo | 13:10 |
lucasagomes | nicodemos, TheJulia morning | 13:10 |
rloo | hi and good morning TheJulia, lucasagomes | 13:11 |
*** alexpilotti has joined #openstack-ironic | 13:11 | |
lucasagomes | rloo, hello there, good morning | 13:12 |
dtantsur | lucasagomes, cool! | 13:12 |
dtantsur | morning TheJulia, rloo, xavierr, nicodemos | 13:12 |
rloo | hi dtantsur | 13:13 |
*** priteau has quit IRC | 13:14 | |
nicodemos | hey, dtantsur o/ | 13:14 |
*** priteau has joined #openstack-ironic | 13:15 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Remove support for the old ramdisk (DIB ironic-deploy element) https://review.openstack.org/327083 | 13:15 |
dtantsur | here it starts ^^^ | 13:15 |
dtantsur | (now fix/remove all the unit tests) | 13:15 |
sambetts | :D | 13:15 |
dtantsur | reviews are still welcome, I'm sure I missed something | 13:15 |
*** links has joined #openstack-ironic | 13:18 | |
*** ppiela has joined #openstack-ironic | 13:18 | |
*** watanabe_isao has joined #openstack-ironic | 13:20 | |
*** rbudden has joined #openstack-ironic | 13:23 | |
*** ayoung has joined #openstack-ironic | 13:26 | |
*** alexpilo_ has joined #openstack-ironic | 13:29 | |
*** alexpilotti has quit IRC | 13:32 | |
*** PollyZ has joined #openstack-ironic | 13:35 | |
openstackgerrit | Chris Krelle proposed openstack/ironic-python-agent: Add basic troubleshooting doc https://review.openstack.org/326743 | 13:39 |
NobodyCam | Good morning Ironic | 13:39 |
*** PollyZ has quit IRC | 13:41 | |
dtantsur | morning NobodyCam | 13:41 |
lucasagomes | NobodyCam, hi there, morning | 13:43 |
*** mbound has quit IRC | 13:43 | |
NobodyCam | :) good morning dtantsur and lucasagomes :) | 13:43 |
NobodyCam | happy mid week :) | 13:43 |
mgould | morning NobodyCam | 13:44 |
NobodyCam | goodmorning mgould :) | 13:44 |
* TheJulia needs lots of coffee this morning | 13:48 | |
mgould | morning TheJulia | 13:49 |
NobodyCam | +++ I was woken up at like 5:20 am this morning :p .... | 13:49 |
* mgould needs coffee too, and it's mid-afternoon here :-( | 13:49 | |
mgould | I can beat that | 13:49 |
NobodyCam | morning TheJulia | 13:49 |
mgould | 0330 | 13:49 |
NobodyCam | mgould: ugh | 13:49 |
mgould | <- not very together today | 13:49 |
NobodyCam | mgould: ++++ | 13:50 |
NobodyCam | also nap's help ! :) | 13:50 |
mgould | I did get a bit more sleep after that, but the morning was a cascade of fail | 13:50 |
mgould | yeah, I think a nap may be the way forward | 13:50 |
*** PollyZ has joined #openstack-ironic | 13:50 | |
NobodyCam | :) | 13:50 |
*** watanabe_isao has quit IRC | 13:54 | |
rloo | morning NobodyCam, hi mgould | 13:56 |
mgould | hi rloo | 13:56 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Remove support for the old ramdisk (DIB ironic-deploy element) https://review.openstack.org/327083 | 13:59 |
dtantsur | please review carefully ^^^ | 13:59 |
dtantsur | jroll, ^^^ | 13:59 |
*** clenimar has quit IRC | 13:59 | |
NobodyCam | morning rloo :) | 14:01 |
*** ametts has joined #openstack-ironic | 14:02 | |
*** clenimar has joined #openstack-ironic | 14:04 | |
vdrok | morning TheJulia xavierr rloo and NobodyCam | 14:05 |
NobodyCam | morning vdrok :) | 14:06 |
jroll | dtantsur: ++ | 14:06 |
NobodyCam | morning jroll :) | 14:06 |
jroll | hey NobodyCam :) | 14:06 |
*** watanabe_isao has joined #openstack-ironic | 14:06 | |
NobodyCam | :) | 14:07 |
rloo | hi vdrok! | 14:08 |
*** yolanda_ has joined #openstack-ironic | 14:08 | |
*** yolanda has quit IRC | 14:09 | |
*** yolanda_ has quit IRC | 14:09 | |
*** yolanda has joined #openstack-ironic | 14:09 | |
*** vsaienko1 has quit IRC | 14:12 | |
*** vsaienko has joined #openstack-ironic | 14:13 | |
rbartal | TheJulia, HI there, I have a question about " | 14:13 |
TheJulia | about " ? | 14:13 |
NobodyCam | hehehehe | 14:13 |
NobodyCam | TheJulia: would that be ' " '? | 14:13 |
* TheJulia does not know the the historical context to speak in regards to ' " ' ;) | 14:14 | |
TheJulia | rbartal: Whats up? | 14:14 |
NobodyCam | it's one of those mornings. | 14:14 |
rbartal | TheJulia, Sorry, Thick fingers | 14:14 |
TheJulia | Indeed! The caffeine is just hitting my system now | 14:14 |
TheJulia | rbartal: no worries! | 14:15 |
rbartal | TheJulia, I'm testing clean steps and fails with Security Freeze Lock | 14:15 |
*** notq_ has joined #openstack-ironic | 14:15 | |
* TheJulia has sudden urge for music of a land she can never really visit. | 14:15 | |
TheJulia | rbartal: when trying to do ?erase_drives? ? | 14:17 |
rbartal | TheJulia, I did try the SSD with intel toolbox and re try to run the erase_device again and got the same error | 14:17 |
TheJulia | rbartal: can you collect the hdparm output from the device and share it with us? | 14:18 |
rbartal | TheJulia, sure, but it will take some time.. | 14:18 |
TheJulia | Also, what kind of hardware are you using? what I guess your using a recent IPA :) | 14:19 |
TheJulia | rbartal: s/what// | 14:19 |
TheJulia | at least, the second what | 14:19 |
* TheJulia may need more coffee | 14:19 | |
TheJulia | rbartal: I understand it is not the easiest to get that info from a machine, but i would suspect your bios is locking the device upon boot and the password is not NULL :\ | 14:21 |
rloo | hi, this has two +2s, does anyone want to take a look? otherwise, I'll +A it: https://review.openstack.org/#/c/319183/ | 14:23 |
TheJulia | looking | 14:23 |
*** jcoufal has quit IRC | 14:25 | |
*** rama_y has joined #openstack-ironic | 14:26 | |
rbartal | TheJulia, yes the bois password is null, the SSD is intel s3510 series 120GB, and the osp is 9 | 14:27 |
TheJulia | rbartal: what is the exact error your getting in the last_error field? | 14:28 |
dtantsur | (me notes that OSP 9 is Mitaka) | 14:28 |
TheJulia | dtantsur: thank you for the context | 14:28 |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Add config fixture to functional tests https://review.openstack.org/327130 | 14:29 |
*** ijw has joined #openstack-ironic | 14:29 | |
*** links has quit IRC | 14:29 | |
*** ChrisAusten has joined #openstack-ironic | 14:30 | |
TheJulia | dtantsur: so I take it that IPA patch that was backported while I was on leave is likely in rbartal's build? | 14:30 |
*** notq_ has quit IRC | 14:30 | |
dtantsur | TheJulia, no, it definitely is not. it wasn't even released yet | 14:31 |
TheJulia | ok, that is good to know | 14:31 |
dtantsur | ofc we'll rebase as soon as possible | 14:31 |
*** lintan has quit IRC | 14:31 | |
TheJulia | naturally :) | 14:31 |
*** alexpilo_ has quit IRC | 14:32 | |
*** lintan has joined #openstack-ironic | 14:32 | |
openstackgerrit | Yosef Hoffman proposed openstack/ironic-python-agent: [WIP] Add configuration options for DISK_WAIT https://review.openstack.org/324909 | 14:32 |
jroll | rloo: +A'd | 14:34 |
TheJulia | heh, Well, I just +2'ed it as well :) | 14:34 |
dtantsur | \o/ was it the last ironic patch before we can proceed with grenade? | 14:34 |
*** jaybeale has joined #openstack-ironic | 14:34 | |
jroll | dtantsur: yep! | 14:35 |
jroll | then it's just this one https://review.openstack.org/#/c/316662/ | 14:35 |
rloo | thx jroll and TheJulia! | 14:35 |
TheJulia | Has anyone asked infra regarding that one? | 14:35 |
jroll | I have not | 14:36 |
TheJulia | I'll ask | 14:36 |
rloo | TheJulia: dunno. I didn't. Seems like a straightforward one. | 14:36 |
vsaienko | morning jroll, rloo, TheJulia | 14:36 |
*** joprovost has joined #openstack-ironic | 14:37 | |
jroll | TheJulia: oops, too late | 14:37 |
rloo | morning vsaienko | 14:37 |
*** piet has joined #openstack-ironic | 14:37 | |
TheJulia | jroll: no worries, my manager pinged me in another window as I was typing the message in the infra room, thank you | 14:37 |
jroll | heh, cool | 14:37 |
rbartal | TheJulia, this is the node-show output http://paste.openstack.org/show/508947/ | 14:38 |
jroll | dtantsur: TheJulia: JayF: you know if we have plans to use this? https://review.openstack.org/#/c/326203/1 | 14:39 |
dtantsur | jroll, it's the first time I see it Oo | 14:40 |
jroll | dtantsur: looks like it's meant for bifrost | 14:40 |
jroll | 6 months ago | 14:40 |
jroll | but was never used | 14:40 |
* jroll needs TheJulia input on this one | 14:41 | |
*** jaybeale has quit IRC | 14:41 | |
*** clenimar has quit IRC | 14:41 | |
rbartal | TheJulia, dtantsur I also have the ironic-conductor.log, how can I share a file? or it's only copy-paste? | 14:43 |
*** clenimar has joined #openstack-ironic | 14:45 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Remove support for the old ramdisk (DIB deploy-ironic element) https://review.openstack.org/327083 | 14:45 |
dtantsur | vdrok, fixed the element name ^^ | 14:45 |
vdrok | dtantsur: thanks, otherwise I haven't found anything else, will wait for the gate :) | 14:46 |
*** bcornec1 has quit IRC | 14:46 | |
*** mtanino has joined #openstack-ironic | 14:49 | |
*** watanabe_isao has quit IRC | 14:51 | |
rbartal | TheJulia, which command line argument should I use for the hdparm? | 14:54 |
TheJulia | rbartal: as root, or via sudo: hdparm -I /dev/sda | 14:55 |
*** jcoufal has joined #openstack-ironic | 14:55 | |
*** clenimar has quit IRC | 14:56 | |
vsaienko | dtantsur, vdrok, lucasagomes please review https://review.openstack.org/#/c/299229 https://review.openstack.org/#/c/325952 | 14:56 |
rbartal | TheJulia, http://paste.openstack.org/show/508952/ | 14:56 |
lucasagomes | vsaienko, sure thing! 1 sec | 14:57 |
*** links has joined #openstack-ironic | 14:57 | |
*** thiagop has joined #openstack-ironic | 14:59 | |
thiagop | Good morning, Ironicers | 14:59 |
NobodyCam | morning thiagop | 14:59 |
TheJulia | jroll: all for removing it for many reasons() | 14:59 |
*** clenimar has joined #openstack-ironic | 15:00 | |
vsaienko | morning thiagop | 15:00 |
jroll | TheJulia: cool, thanks | 15:01 |
*** clenimar has quit IRC | 15:01 | |
mgould | morning thiagop | 15:01 |
thiagop | o/ NobodyCam vsaienko mgould | 15:01 |
rbartal | 15:02 | |
TheJulia | rbartal: checking a few things, seems, one moment | 15:02 |
rbartal | TheJulia, thanks | 15:02 |
rbartal | uit | 15:02 |
*** ijw has quit IRC | 15:02 | |
*** ijw has joined #openstack-ironic | 15:03 | |
*** itamarl has quit IRC | 15:06 | |
*** e0ne_ is now known as e0ne | 15:11 | |
TheJulia | rbartal: https://wiki.archlinux.org/index.php/SSD_memory_cell_clearing may be worth reading, it has refreshed my memory quite well. It is likely the bios asserting it, and it is a state that IPA is unable to recover from, unless you have two revisions in your build, one which has yet to land... and even then that only allows you to fallback to disk shredding/wiping out with zeros which ultimate | 15:12 |
TheJulia | ly is really bad for an SSD | 15:12 |
JayF | jroll: I hope someone has plans to do a DIB image in a job, but I'm not aware of anyone actively working on it. | 15:12 |
JayF | jroll: did someone look at the commit that added it? is ther person around still? | 15:13 |
*** dprince has quit IRC | 15:13 | |
TheJulia | JayF: I started down that path, but all the random dib failures due to mirrors/repos, single distro specific changes that broke other distros just honestly frustated me beyond belief. | 15:13 |
*** piet has quit IRC | 15:14 | |
JayF | TheJulia: yeah; I just think at some point we're going to have to document supported vs unsupported | 15:14 |
TheJulia | JayF: on a plus side, bifrost has a non voting job for building it with debian so we can see if it is broken and try and provide some visibility | 15:14 |
JayF | and if the dib image isn't gated; it's not supported | 15:14 |
TheJulia | ++ | 15:14 |
JayF | and even if it's supported, I'd be curious how we'd handle things like new deps as it's not in the repo | 15:14 |
TheJulia | JayF: I think part of the conundrum is some operators are going to need specific distros because of specific hardware drivers, so we're going to have to kind of treat it as best effort | 15:16 |
JayF | I mean, you're talking to someone who runs tools designed for old as hell centos on coreos | 15:16 |
TheJulia | but, we truly can't check all the distros unless it is a job in the dib repo that gates that element and tooling | 15:16 |
JayF | so you can tell which problem I chose, and DIB seemed more intimidating | 15:16 |
JayF | lol | 15:16 |
TheJulia | lol | 15:16 |
TheJulia | cinerama: ^^^ hint hint | 15:16 |
TheJulia | w/r/t gating/jobs | 15:17 |
*** rpioso has joined #openstack-ironic | 15:18 | |
*** joprovost has quit IRC | 15:19 | |
*** rama_y has quit IRC | 15:19 | |
NobodyCam | morning JayF | 15:19 |
JayF | gm | 15:19 |
*** davidlenwell has quit IRC | 15:20 | |
rbartal | TheJulia, OK, so the SSD is locked and IPA can not secure delete it? | 15:20 |
TheJulia | truthfully I feel really embarassed that I forgot about the postbuild job, but I think there was a solid two month span where the post job would hit a transient failure somewhere along the way of retrieving a package. | 15:21 |
TheJulia | rbartal: that is basically the gist of your issue. It might be a bios setting in your hardware that you can control, I know with one of my lab desktops it is actually a setting I can toggle | 15:21 |
rbartal | TheJulia, how can we tell its not the IPA that trigger the lock? | 15:22 |
TheJulia | s/embarassed/embarassd and ashamed/ | 15:22 |
JayF | rbartal: we have code in IPA that attempts to unlock it | 15:22 |
TheJulia | JayF: rbartal does not have the unlock code yet | 15:23 |
rbartal | TheJulia, I run this procedure on the disk and it worked http://www.intel.com/content/www/us/en/support/software/000006094.html | 15:23 |
TheJulia | and it is not locked http://paste.openstack.org/show/508952/ :\ | 15:23 |
*** yohoffman has joined #openstack-ironic | 15:23 | |
JayF | Just making it clear; if IPA locked the disk, it will recover on it's own: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/hardware.py#L747 | 15:24 |
JayF | you added that code and it's muy bueno | 15:24 |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Add config fixture to functional tests https://review.openstack.org/327130 | 15:24 |
rbartal | TheJulia, OK, so best is to try and run manual lock / unlock procedure and wait for the patch | 15:25 |
TheJulia | rbartal: try to unfreeze, reboot to another os, and inspect the drive parameters after power cycling. I suspect you'll see it frozen again | 15:25 |
NobodyCam | lucasagomes: would you happen to have any time to take a quick look at: https://bugs.launchpad.net/ironic/+bug/1590146 | 15:25 |
openstack | Launchpad bug 1590146 in Ironic "A timed out manual cleaning cannot be retried successfuly" [Undecided,New] | 15:25 |
*** sukhdev has joined #openstack-ironic | 15:25 | |
lucasagomes | NobodyCam, yup, 1 sec | 15:25 |
NobodyCam | lucasagomes: you awesome .... Thank you :) | 15:26 |
NobodyCam | s/you/your/ | 15:26 |
alineb | JayF: regarding your comment on the metric patch, does that mean the get_logger method should go in a separate module file too? | 15:26 |
JayF | alineb: I mean, it's just a nit. | 15:26 |
JayF | alineb: I'm just thinking if I was adding some new metrics logger, I'd like to have the already implemented one in a separate module | 15:27 |
JayF | alineb: but it's seriously just a nit, and if you disagree, ignore it | 15:27 |
JayF | lol | 15:27 |
alineb | JayF: it's not that I disagree, more that i'm not sure how to do it :) | 15:28 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Update terribly outdated installation instructions https://review.openstack.org/327187 | 15:28 |
JayF | I mean, I was just thinking put the Statsdmetricslogger class in it's own module | 15:28 |
dtantsur | mgould, could you please take a look ^^^ | 15:28 |
JayF | basically if I wanted to make a jaymetricslogger, I could copy the file and start there as a template | 15:28 |
rbartal | TheJulia, Thanks, and have a good day (I'm on UTC +3) | 15:29 |
mgould | dtantsur: looking now | 15:29 |
*** dmk0202 has quit IRC | 15:29 | |
alineb | JayF: yes, i think i understand what you meant, my problem is that the Statsdmetricslogger would need to import the base class, and that getLogger needs to import Statsdmetricslogger | 15:31 |
alineb | which explains my questions about getlogger in its own module file (or in utils.py) | 15:31 |
JayF | that sort of problem is why it's maybe nice to split it out now | 15:31 |
JayF | so you can make the next one easier before having to worry about backwards compat | 15:32 |
lucasagomes | NobodyCam, TheJulia interesting, it sounds like it affects automatic cleainng as well? | 15:32 |
lucasagomes | NobodyCam, TheJulia I think the proposed solution looks sane too, upon moving it to fail state we should clean up the fields so the operation can be retried | 15:32 |
alineb | JayF: ok, i will do that then | 15:33 |
*** rbartal has quit IRC | 15:33 | |
NobodyCam | ++ | 15:33 |
NobodyCam | I did not test with automated cleaning the stuff I looked at was all manual | 15:34 |
NobodyCam | however I assume that would be the same | 15:34 |
JayF | lucasagomes: NobodyCam: context? Do we have a bug up around cleaning? | 15:35 |
lucasagomes | JayF, yeah, NobodyCam just pointed me out to https://bugs.launchpad.net/ironic/+bug/1590146 | 15:35 |
openstack | Launchpad bug 1590146 in Ironic "A timed out manual cleaning cannot be retried successfuly" [Undecided,New] | 15:35 |
lucasagomes | NobodyCam, the periodic task that catches the time out is the same, so I assume that too | 15:35 |
JayF | oh my | 15:35 |
JayF | that explains downstream behavior | 15:35 |
JayF | jcook: ^ bug 1590146 is probably responsbile for some of our stuck cleanings | 15:36 |
NobodyCam | :) | 15:36 |
JayF | That has an even scarier side effect I think, would it ever be possible for that bug to cause a clean step to be skipped? | 15:37 |
NobodyCam | JayF: the bug or the fix? | 15:39 |
JayF | The bug. | 15:39 |
JayF | I guess at worse it'd cause them to repeat | 15:39 |
NobodyCam | yea, we had a node wedged to such a point the only fix was edit the DB | 15:40 |
JayF | NobodyCam: even manage/provide won't clear the clean step either | 15:41 |
JayF | man this is insideous | 15:41 |
NobodyCam | yes | 15:41 |
JayF | the only reason we haven't gotten wedged this hard downstream is generally our clean steps are idempotent | 15:41 |
TheJulia | lucasagomes: NobodyCam: JayF: Yeah, I think it could impact automated cleaning if you have something that could get it wedged into a timeout situation | 15:43 |
JayF | absolutely it impacts automatd cleaning | 15:43 |
JayF | I already updated the bug to reflect that | 15:43 |
TheJulia | less likely if it was just doing erase_drives, but you guys have a lot more going on than most | 15:43 |
lucasagomes | yeah it seems to do that | 15:43 |
JayF | Yep. Like I said the only reason we haven't been crushed by this downstream is b/c our steps are all idempotent | 15:44 |
TheJulia | JayF: awesome | 15:44 |
JayF | but I've def seen this behavior before, and couldn't figure it out and it sorta "solved itself" (because our steps are a-ok with rerunning) | 15:44 |
* NobodyCam looks at flash bios step and shutters just bit. | 15:45 | |
TheJulia | yeah, in this case we found it with with a dib based raid utility build that OOM'ed a machine and we're like "what?" "huh?" "Why is this not working?" | 15:45 |
*** sacharya has joined #openstack-ironic | 15:47 | |
JayF | What's the rules with bug triage? Like can I JFDI and assign an importance to this bug and mark it confirmed? | 15:52 |
JayF | or is that dtantsur's domain | 15:52 |
dtantsur | JayF, you can (and should :) | 15:52 |
JayF | marking it as high and confirming it | 15:54 |
JayF | is someone going to work on this today and fix? | 15:54 |
*** davidlenwell has joined #openstack-ironic | 15:56 | |
*** sabeen has joined #openstack-ironic | 15:58 | |
*** chlong has quit IRC | 15:58 | |
TheJulia | JayF: I will | 16:01 |
jroll | JayF: dtantsur gets bored, let him do it | 16:01 |
jroll | :D | 16:01 |
JayF | TheJulia loses the game of chicken :) | 16:02 |
dtantsur | LOL | 16:02 |
JayF | if you had waited 5 more minutes I might have volunteered ;) | 16:02 |
jroll | heh | 16:03 |
rloo | TheJulia has already thought about the fix, so I'm sure she really wants to fix it! ;) | 16:04 |
JayF | I think TheJulia has like, 2 weeks of boredom still left to unload | 16:05 |
JayF | point it at the bug and gogogo! | 16:05 |
TheJulia | heh, yeah, should be fairly quick since I've been looking at it, just writing tests | 16:05 |
TheJulia | lol | 16:05 |
rloo | thx JayF for volunteering to review TheJulia's patch :) | 16:05 |
TheJulia | lol | 16:06 |
JayF | A dirty secret: I review almost everything linked to me unless it's very deep in a vendor driver :) | 16:06 |
JayF | and I review everything in IPA when I'm working | 16:06 |
JayF | lol | 16:06 |
* rloo has to remember to link everything to JayF | 16:07 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Update terribly outdated installation instructions https://review.openstack.org/327187 | 16:07 |
dtantsur | rloo++ | 16:07 |
TheJulia | Is this where we can laugh evilly? | 16:08 |
*** Sukhdev_ has joined #openstack-ironic | 16:08 | |
*** joprovost has joined #openstack-ironic | 16:09 | |
rloo | TheJulia: hey, I'm ecstatic! | 16:12 |
ccarmack | Hi… can anyone give advice on this error when setting the provision state to active: | last_error | Timeout reached while waiting for callback for node | | 16:13 |
ccarmack | | | 30490c21-bfc8-4b40-9f43-e09894380db8 | 16:13 |
ccarmack | The node stays in wait for callback state for several minutes, then goes to deploy failed | 16:14 |
*** Sukhdev_ has quit IRC | 16:14 | |
thiagop | ccarmack: your deploy took too much time or your instance is unable to communicate with Ironic conductor | 16:14 |
*** links has quit IRC | 16:15 | |
thiagop | ccarmack: normally it is one of these two, but can be something more "black magic" (it always can...) | 16:15 |
ccarmack | ok, thanks… how does the node know where to contact the conductor? | 16:15 |
dtantsur | TheJulia, wdyt about making this change in IPA: https://bugzilla.redhat.com/show_bug.cgi?id=1283436#c29 (hopefully this link is public) | 16:15 |
openstack | bugzilla.redhat.com bug 1283436 in rhel-osp-director "diskless boot via iSCSI & iBFT not working with osp-director" [Urgent,On_dev] - Assigned to bnemec | 16:15 |
*** alexpilotti has joined #openstack-ironic | 16:15 | |
*** piet has joined #openstack-ironic | 16:16 | |
JayF | dtantsur: I'd be -1 to doing it unconditionally | 16:16 |
*** rpioso has quit IRC | 16:16 | |
thiagop | ccarmack: when deploying, Ironic creates a port on neutron with it's IP and PXE info. When the node powers on, it would get it on the DHCP request and contact PXE to download its image | 16:16 |
JayF | dtantsur: as long as that doesn't run in non-iscsi scenarios and all the ramdisks have that, then sure | 16:16 |
JayF | dtantsur: with the caveat I don't know anything about icscistart to know if -b is safe :) | 16:16 |
dtantsur | me neither | 16:17 |
dtantsur | well, we can create one more configuration option :) | 16:17 |
thiagop | ccarmack: then, if the BM node can do PXE and download the deploy images, normally it should contact conductor | 16:17 |
JayF | I mean, given that's hardware specific | 16:17 |
TheJulia | dtantsur: Kind of with JayF on that, Jay and I talked briefly at the summit about maybe creating a secondary hardware manager or something that could identify such things and run appropriate commands | 16:17 |
JayF | I must ask if it's something ... ^ yep | 16:17 |
JayF | bingo | 16:17 |
JayF | way to steal my thunder Julia :P | 16:18 |
dtantsur | how far are you from "maybe creating a secondary hardware manager"? :) | 16:18 |
dtantsur | I get bugged pretty often by people trying things like that (see the whole bug for just one example) | 16:18 |
JayF | I mean, you can do it. It's trival. You just add a new hardware manager class to IPA that implements the $whatever that the specific hardware needs | 16:18 |
JayF | and that gives us the benefit of being able to use entrypoints in setup.cfg to determine what runs or not and/or finding the broken hardware in evaluate_hardware_support() so the workarounds are only enabled as needed | 16:19 |
TheJulia | me, haven't gotten a chance to look at doing it, but as JayF has said, there are other hardware manager examples out there that can be built in with it | 16:19 |
ccarmack | thiagop: I'm actually using bifrost with stand-alone ironic | 16:19 |
dtantsur | well, this particular change looks pretty generic. do you think something like that can go to IPA upstream? | 16:19 |
dtantsur | otherwise it means I'll have to develop and support a downstream hw manager for these people | 16:20 |
TheJulia | dtantsur: I think we need like an "advanced storage hardware manager" | 16:20 |
thiagop | ccarmack: then I think TheJulia can answer better then me | 16:20 |
*** jtomasek has quit IRC | 16:20 | |
TheJulia | and we can put all the important things for advanced storage tech into that | 16:20 |
* thiagop needs to give a try to bifrost someday | 16:20 | |
cinerama | ccarmack: thanks for putting in that change. i haven't had a chance to look at your most recent patchset yet | 16:20 |
dtantsur | TheJulia, I still struggle to understand how this potential hw manager will detect that it should kick in.. | 16:21 |
dtantsur | (maybe it's because JayF didn't do his tech talk on hw managers yet ;) | 16:21 |
ccarmack | cinerama: no problem… the other thing I'm looking at is scoping bifrost_inventory to a subset of the nodes | 16:21 |
TheJulia | it would have to be built in with ipa, and it would likely have to, upon start-up, detect if there is "supported" hardware maybe and take particular actions? | 16:21 |
ccarmack | cinerama: we want to deploy to a subset of our nodes at a time | 16:22 |
* TheJulia is just tossing the last idea out there | 16:22 | |
dtantsur | TheJulia, yeah, I understand, I just don't know how to define "supported" in this case. do we need to figure out what this "iscsistart" thing supports? | 16:22 |
*** rama_y has joined #openstack-ironic | 16:22 | |
TheJulia | ccarmack: so the "where to go talk to" is on the kernel command line that gets passed in ipxe. Generally when people are stuck where your at, it is something to do with networking, or maybe the deployment ramdisk doesn't support your network cards or something frustratingly silly like that | 16:22 |
* NobodyCam is totally out of bagels an muffin's this morning | 16:22 | |
JayF | dtantsur: tl;dr: evaluate_hardware_support() on each loaded manager runs on IPA start, if it returns 0, it's not a valid manager and will never be called to run things | 16:23 |
ccarmack | TheJulia: this just started happening.. so I'll need to contact the admin | 16:23 |
JayF | dtantsur: it's hard to see upstream because we implement that method with a static return vs detecting if a given piece of hardware is there (and/or explicitly enabling it if needed) | 16:23 |
cinerama | ccarmack, that sounds potentially interesting | 16:24 |
JayF | dtantsur: think about it this way; that crazy hardware is what causes you to need to run that iscsistart command | 16:24 |
TheJulia | dtantsur: I think so yes :\ Perhaps a midcycle topic or maybe a much higher bandwidth discussion is required. I also need to completely read that bug and try and gain full context | 16:24 |
JayF | dtantsur: so start with detecting if that crazy hardware exists | 16:24 |
ccarmack | cinerama: I have new env var (BIFROST_IRONIC_SOURCE) that points to a json file with the node list to call ironic for | 16:24 |
cinerama | (re your subset-of-nodes idea) | 16:24 |
dtantsur | JayF, not really hardware, rather kernel arguments (that's how I understand this command) | 16:25 |
dtantsur | but maybe I'm just wrong, not sure | 16:25 |
JayF | hmm. That'd be interesting. | 16:25 |
JayF | You could totes trigger it based on kernel command line | 16:25 |
TheJulia | dtantsur: Yeah, I'm kind of reading this as they are trying to implement iscsi targets | 16:25 |
TheJulia | software iscsi targets at that | 16:25 |
*** ijw has quit IRC | 16:26 | |
dtantsur | yeah.. then it's not hardware thing | 16:26 |
TheJulia | but, I think it would fall into advanced storage, but yeah :\ | 16:26 |
*** Sukhdev_ has joined #openstack-ironic | 16:26 | |
TheJulia | I think we need a higher bandwidth discussion | 16:26 |
*** alexpilotti has quit IRC | 16:27 | |
dtantsur | I get it. I don't want to postpone it for too long though, as this people have eaten my brain since Kilo :) | 16:27 |
*** daemontool has joined #openstack-ironic | 16:27 | |
TheJulia | dtantsur: let me try and read through the entire bug and see what they are trying to do since there seems ot be some heavy customization taking place | 16:27 |
dtantsur | cool, thanks million times! | 16:28 |
*** sabeen has quit IRC | 16:28 | |
TheJulia | no problem | 16:28 |
dtantsur | I have to run now, will get back to it tomorrow. will also try to figure out what they actually achieve by doing that | 16:28 |
*** sabeen has joined #openstack-ironic | 16:28 | |
*** Sukhdev_ has quit IRC | 16:29 | |
NobodyCam | brb | 16:29 |
*** e0ne has quit IRC | 16:30 | |
dtantsur | see you tomorrow then | 16:30 |
TheJulia | have a good evening | 16:31 |
*** trown is now known as trown|lunch | 16:31 | |
*** tesseract has quit IRC | 16:31 | |
*** mbound has joined #openstack-ironic | 16:32 | |
*** mjura has quit IRC | 16:32 | |
*** dtantsur is now known as dtantsur|afk | 16:32 | |
*** d0ugal has quit IRC | 16:33 | |
*** sabeen has quit IRC | 16:36 | |
*** frontrunner has quit IRC | 16:36 | |
*** mbound has quit IRC | 16:36 | |
openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: Fix the logic for fetching the gpu device count https://review.openstack.org/315402 | 16:37 |
*** yolanda has quit IRC | 16:37 | |
*** kromanenko_ has quit IRC | 16:37 | |
*** kromanenko__ has quit IRC | 16:37 | |
*** Romanenko_K has quit IRC | 16:38 | |
*** yolanda has joined #openstack-ironic | 16:40 | |
*** dprince has joined #openstack-ironic | 16:41 | |
*** rcernin has quit IRC | 16:42 | |
openstackgerrit | Bruno Cornec proposed openstack/ironic: Fix JSON error in documentation https://review.openstack.org/326079 | 16:42 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add node provsioning methods to the tempest plugin https://review.openstack.org/327238 | 16:42 |
*** yohoffman has quit IRC | 16:44 | |
*** jaybeale has joined #openstack-ironic | 16:46 | |
lucasagomes | folks, calling it a day too | 16:47 |
lucasagomes | have a great evening all, talk to you all tomorrow | 16:47 |
*** frontrunner has joined #openstack-ironic | 16:47 | |
sambetts | o/ lucasagomes | 16:48 |
*** moshele has quit IRC | 16:48 | |
lucasagomes | sambetts, :D night! | 16:48 |
*** lucasagomes is now known as lucas-dinner | 16:48 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP: Patch to test Grenade. https://review.openstack.org/311109 | 16:48 |
openstackgerrit | Aline Bousquet proposed openstack/ironic-lib: Add support for metrics https://review.openstack.org/301526 | 16:48 |
devananda | morning, all | 16:48 |
*** d0ugal has joined #openstack-ironic | 16:49 | |
jlvillal | lucas-dinner: Ciao! | 16:49 |
jlvillal | devananda: Good morning | 16:49 |
vdrok | morning devananda and jlvillal | 16:50 |
* jlvillal does a little happy dance as only one patch left for Grenade to work :) | 16:50 | |
sambetts | \o/ | 16:50 |
jlvillal | Big thanks to vdrok and vsaienko and everyone else for the help!! | 16:50 |
devananda | jlvillal: nice! the one you just posted above? | 16:50 |
jlvillal | devananda: That just tests it. But if you look in the commit message you can see the one patch. | 16:50 |
devananda | k | 16:50 |
devananda | I'll take a look this morning | 16:51 |
jlvillal | warning, it will cause your browser to crawl to a halt for a few moments... | 16:51 |
devananda | oh? | 16:51 |
* jlvillal glances at sambetts ;) | 16:51 | |
jlvillal | LOL | 16:51 |
jlvillal | devananda: A lot of messages in that patch set | 16:51 |
*** ChubYann has joined #openstack-ironic | 16:52 | |
sambetts | heh | 16:52 |
JayF | good lord why did cisco ci explode on that patch | 16:52 |
JayF | lol | 16:52 |
jlvillal | I think I suffer more because I had Toggle CI = On | 16:52 |
sambetts | Not really sure... I think it had something to do with my recheck regex, cauing it to recheck itself or something ;-P | 16:52 |
JayF | that's a hilarious bug | 16:54 |
JayF | I don't mind things being broken sometimes if they break in funny ways :P | 16:54 |
sambetts | my Zuul version also had a bug which caused infinite loops, which didn't get fixed because I was on the stable release of Zuul, and infra expects everyone to CD zuul in there CI, so didn't push a new stable :/ | 16:55 |
* jlvillal investigates changing grenade jobs from experimental to non-voting... | 16:55 | |
sambetts | (I didn't realise they expected a CD'd Zuul at the time :)) | 16:55 |
JayF | ah | 16:56 |
JayF | jlvillal: want me to jfdi? | 16:56 |
jlvillal | One +2 for the final patch | 16:56 |
*** PollyZ has quit IRC | 16:56 | |
JayF | jlvillal: I know exactly what knobs to turn to make that happen | 16:56 |
* jlvillal wonders if he should go beg in #openstack-infra | 16:56 | |
JayF | I just did | 16:56 |
JayF | that's why you got a +2 | 16:56 |
sambetts | the same reason broke my CIs ability to process depends on tags when infra upgraded gerrit and the gerrit API changed | 16:56 |
JayF | lol | 16:57 |
*** daemontool has quit IRC | 16:57 | |
jlvillal | JayF: Thanks. Well I begged in #openstack-qa with a ping to sdague | 16:57 |
JayF | cool | 16:57 |
JayF | jlvillal: want me to handle the move of the grenade job? | 16:57 |
jlvillal | JayF: That would be great :) | 16:57 |
jlvillal | Can you have it depend on the final patch? | 16:57 |
JayF | project-config doesn't work like that | 16:58 |
JayF | depends-on isn't effective | 16:58 |
jlvillal | JayF: I think it is effective for getting code merged. | 16:58 |
jlvillal | JayF: Or blocking it from getting merged | 16:58 |
*** cdearborn has joined #openstack-ironic | 16:58 | |
JayF | ah, got what you mean | 16:58 |
jlvillal | JayF: But yes, not effective for testing the code. | 16:58 |
JayF | you wanna make sure it doesn't move before the fix merges | 16:58 |
jlvillal | Yep | 16:58 |
jlvillal | I seriously doubt it would happen that fast, but I want be safe. | 16:59 |
* jlvillal knows his wife thinks he is too cautious while she is learning to drive. He keeps saying, slow down, stop, foot over the brake, look, etc... :) | 16:59 | |
*** rpioso has joined #openstack-ironic | 16:59 | |
JayF | I suspect the dsg change will be merging in minutes | 16:59 |
*** PollyZ has joined #openstack-ironic | 16:59 | |
JayF | if it's not core reviewed and landed in 15m, going to ping a couple of folks directly | 16:59 |
jlvillal | Oh did we get a +A too | 16:59 |
JayF | one line changes should not block things | 16:59 |
JayF | No, you just have the shameless review hound on the case now, I'll make sure it gets merged :P | 17:00 |
*** sabeen has joined #openstack-ironic | 17:00 | |
jlvillal | JayF: Agreed and thank you. | 17:00 |
jlvillal | Woot! | 17:00 |
*** ijw has joined #openstack-ironic | 17:00 | |
jlvillal | Oh meeting time | 17:00 |
jroll | yay meetings | 17:00 |
krtaylor | :) | 17:00 |
*** ayoung has quit IRC | 17:01 | |
*** sacharya_ has joined #openstack-ironic | 17:02 | |
*** sacharya has quit IRC | 17:02 | |
*** fragatina has joined #openstack-ironic | 17:03 | |
*** fragatin_ has joined #openstack-ironic | 17:04 | |
*** PollyZ has quit IRC | 17:04 | |
*** rbartal has joined #openstack-ironic | 17:07 | |
*** fragatina has quit IRC | 17:08 | |
*** derekh has quit IRC | 17:08 | |
*** _milan_ has quit IRC | 17:09 | |
*** sdake has joined #openstack-ironic | 17:10 | |
*** vishwanathj has quit IRC | 17:11 | |
*** trown|lunch is now known as trown | 17:20 | |
*** dprince has quit IRC | 17:22 | |
*** dprince has joined #openstack-ironic | 17:23 | |
*** electrofelix has quit IRC | 17:24 | |
*** PollyZ has joined #openstack-ironic | 17:26 | |
*** alexpilotti has joined #openstack-ironic | 17:28 | |
*** PollyZ has quit IRC | 17:31 | |
*** e0ne has joined #openstack-ironic | 17:32 | |
*** moshele has joined #openstack-ironic | 17:32 | |
*** sukhdev has quit IRC | 17:33 | |
*** ohamada has quit IRC | 17:34 | |
jlvillal | JayF: Feel free to tell me I'm wrong on the idea for a second patch for the ironic grenade job as n-v for grenade. | 17:38 |
JayF | I think you are :) | 17:38 |
jlvillal | JayF: I get worried about needing more people to agree. So I lean towards more granularity. | 17:38 |
JayF | if you look at the list of grenade jobs this seems pretty typical | 17:38 |
JayF | and when that job votes for us, it'll have to vote for grenade to or else our gate will get wedged | 17:39 |
JayF | so it's just putting it off | 17:39 |
JayF | we'll have to do it, so there's no need to wait | 17:39 |
jlvillal | JayF: Okay. You convinced me. | 17:39 |
jlvillal | JayF: Not sure if you want to respin vsaienko's patch? Unsure if he is still online | 17:40 |
jlvillal | Though vsaienko seems to always be working... | 17:40 |
openstackgerrit | Yosef Hoffman proposed openstack/ironic-python-agent: Add configuration options for DISK_WAIT https://review.openstack.org/324909 | 17:40 |
JayF | I'll look at that if we get to the point where the other stuff merges | 17:40 |
JayF | now I gotta try and get the ds-g patch merged | 17:40 |
jlvillal | JayF: Thanks for all your help. | 17:41 |
JayF | np | 17:41 |
*** sabeen has quit IRC | 17:41 | |
*** mgould is now known as mgould|afk | 17:42 | |
JayF | Just wanted to confirm my suspicion: There's no case where we'd need to run grenade against IPA, right? | 17:42 |
*** e0ne has quit IRC | 17:43 | |
* jlvillal is unsure and needs to leave desk area... | 17:43 | |
jroll | JayF: I don't think so | 17:43 |
JayF | but there's a rabbithole there... we do release stable/* of IPA now | 17:43 |
jroll | an ipa-grenade job might be interesting | 17:43 |
JayF | do we have to make sure new IPA works with old ironic or old IPA works with new ironic? | 17:43 |
jroll | in theory we should do both eventually | 17:44 |
jroll | grenade on ironic takes care of the latter | 17:44 |
*** ijw has quit IRC | 17:49 | |
*** e0ne has joined #openstack-ironic | 17:50 | |
*** sdake has quit IRC | 17:51 | |
*** rbartal has quit IRC | 17:51 | |
* sambetts calling it for today | 17:52 | |
sambetts | Night all | 17:52 |
sambetts | o/ | 17:52 |
TheJulia | goodnight sambetts | 17:53 |
*** sambetts is now known as sambetts|afk | 17:53 | |
NobodyCam | night sambetts | 17:53 |
sambetts|afk | night TheJulia, NobodyCam | 17:53 |
NobodyCam | :) | 17:53 |
*** e0ne has quit IRC | 17:53 | |
*** PollyZ has joined #openstack-ironic | 17:54 | |
JayF | jroll: so really we don't need an IPA grenade job | 17:54 |
JayF | jroll: as much as we need to test new IPA against latest stable, it sounds like | 17:54 |
*** sabeen has joined #openstack-ironic | 17:54 | |
jroll | JayF: yeah, probably | 17:56 |
jroll | JayF: btw, looking at this e-r bug, this is nasty | 17:56 |
JayF | jroll: the gate breakage | 17:56 |
jroll | JayF: though this fixes it in a much simpler way, if you want to rebase on that https://review.openstack.org/#/c/320994/ | 17:56 |
JayF | jroll: and/or the bug breaking e-r unit tests? | 17:56 |
jroll | the latter | 17:56 |
JayF | well if you read from late yesterday | 17:57 |
JayF | it appears my query is AFU as well | 17:57 |
jroll | ah, I did but forgot :( | 17:57 |
JayF | so my patch is basically a steaming pile all the way around | 17:57 |
JayF | lol | 17:57 |
*** PollyZ has quit IRC | 17:59 | |
*** sabeen has quit IRC | 18:01 | |
*** ijw has joined #openstack-ironic | 18:01 | |
*** sabeen has joined #openstack-ironic | 18:01 | |
*** priteau has quit IRC | 18:05 | |
*** priteau has joined #openstack-ironic | 18:05 | |
*** frontrunner has quit IRC | 18:10 | |
-openstackstatus- NOTICE: Zuul has been restarted to correct an error condition. Events since 17:30 may have been missed; please 'recheck' your changes if they were uploaded since then, or have "NOT_REGISTERED" errors. | 18:10 | |
*** fragatin_ has quit IRC | 18:14 | |
*** Sukhdev has joined #openstack-ironic | 18:16 | |
*** fragatina has joined #openstack-ironic | 18:19 | |
*** PollyZ has joined #openstack-ironic | 18:21 | |
*** jaybeale has quit IRC | 18:23 | |
*** ijw has quit IRC | 18:24 | |
*** ayoung has joined #openstack-ironic | 18:28 | |
*** alexpilotti has quit IRC | 18:32 | |
*** alexpilo_ has joined #openstack-ironic | 18:32 | |
*** links has joined #openstack-ironic | 18:32 | |
*** ijw has joined #openstack-ironic | 18:36 | |
*** rbrud has joined #openstack-ironic | 18:36 | |
*** frontrunner has joined #openstack-ironic | 18:36 | |
*** sdake has joined #openstack-ironic | 18:37 | |
openstackgerrit | Merged openstack/bifrost: Make ansible installation directory configurable https://review.openstack.org/324789 | 18:38 |
*** piet has quit IRC | 18:47 | |
*** fragatina has quit IRC | 18:48 | |
*** yolanda has quit IRC | 18:48 | |
*** krtaylor has quit IRC | 18:52 | |
*** Sukhdev has quit IRC | 18:53 | |
*** priteau has quit IRC | 18:57 | |
*** piet has joined #openstack-ironic | 18:59 | |
*** jtomasek has joined #openstack-ironic | 19:01 | |
*** Sukhdev has joined #openstack-ironic | 19:01 | |
*** alex_xu has quit IRC | 19:02 | |
*** alexpilotti has joined #openstack-ironic | 19:04 | |
*** alexpilo_ has quit IRC | 19:05 | |
*** krtaylor has joined #openstack-ironic | 19:05 | |
*** alex_xu has joined #openstack-ironic | 19:06 | |
*** harlowja has quit IRC | 19:12 | |
*** milan has joined #openstack-ironic | 19:12 | |
*** e0ne has joined #openstack-ironic | 19:13 | |
*** harlowja has joined #openstack-ironic | 19:15 | |
*** frontrunner has quit IRC | 19:21 | |
*** piet has quit IRC | 19:24 | |
*** rcernin has joined #openstack-ironic | 19:25 | |
*** flwang has quit IRC | 19:29 | |
*** ifarkas has quit IRC | 19:31 | |
*** MattMan has quit IRC | 19:31 | |
*** MattMan has joined #openstack-ironic | 19:31 | |
*** links has quit IRC | 19:31 | |
*** alexpilotti has quit IRC | 19:34 | |
*** flwang has joined #openstack-ironic | 19:34 | |
*** sdake_ has joined #openstack-ironic | 19:47 | |
*** sdake has quit IRC | 19:48 | |
openstackgerrit | Chris Krelle proposed openstack/ironic-python-agent: Add basic troubleshooting doc https://review.openstack.org/326743 | 19:49 |
*** yolanda has joined #openstack-ironic | 19:50 | |
*** e0ne has quit IRC | 20:04 | |
*** Sukhdev has quit IRC | 20:08 | |
*** rbrud has quit IRC | 20:16 | |
*** rbrud has joined #openstack-ironic | 20:17 | |
*** piet has joined #openstack-ironic | 20:31 | |
*** joprovost has quit IRC | 20:33 | |
*** rbrud has quit IRC | 20:34 | |
*** devananda has quit IRC | 20:35 | |
*** devananda has joined #openstack-ironic | 20:36 | |
*** mbound has joined #openstack-ironic | 20:39 | |
rloo | hi NobodyCam, wrt https://review.openstack.org/#/c/326066/, we had agreed in our weekly meeting, on that change. I don't think it requires waiting for others to review. | 20:46 |
rloo | NobodyCam: I'm going to +A it if you don't want to. | 20:46 |
*** openstackgerrit has quit IRC | 20:48 | |
NobodyCam | I can it hit the button | 20:48 |
NobodyCam | :) | 20:48 |
*** openstackgerrit has joined #openstack-ironic | 20:48 | |
NobodyCam | Done | 20:49 |
openstackgerrit | Merged openstack/python-ironicclient: Add test for 'port-show --field' command https://review.openstack.org/304008 | 20:49 |
*** devananda has quit IRC | 20:51 | |
*** devananda has joined #openstack-ironic | 20:53 | |
*** SiRiuS__ has quit IRC | 20:53 | |
*** PollyZ has quit IRC | 20:54 | |
*** SiRiuS__ has joined #openstack-ironic | 20:56 | |
*** rama_y has quit IRC | 20:59 | |
*** PollyZ has joined #openstack-ironic | 21:01 | |
*** fragatina has joined #openstack-ironic | 21:01 | |
*** sdake_ has quit IRC | 21:01 | |
openstackgerrit | Merged openstack/ironic-specs: Update active node creation spec based on feedback https://review.openstack.org/326066 | 21:05 |
*** chlong has joined #openstack-ironic | 21:06 | |
rloo | thx NobodyCam :) | 21:07 |
*** jaybeale has joined #openstack-ironic | 21:09 | |
NobodyCam | :) | 21:12 |
*** SiRiuS__ has quit IRC | 21:14 | |
*** ametts has quit IRC | 21:14 | |
*** trown is now known as trown|outtypewww | 21:18 | |
JayF | Anyone else getting randomly signed out of gerrit? | 21:18 |
JayF | or is my session AFU :/ | 21:18 |
*** jjohnson2 has quit IRC | 21:19 | |
*** alexpilotti has joined #openstack-ironic | 21:19 | |
NobodyCam | Gerrit has been supper slow for me today | 21:20 |
*** lifeless_ is now known as lifeless | 21:30 | |
jcook | JayF bug 1590146 ack | 21:36 |
openstack | bug 1590146 in Ironic "A timed out cleaning cannot be retried successfully" [High,Confirmed] https://launchpad.net/bugs/1590146 - Assigned to Julia Kreger (juliaashleykreger) | 21:36 |
*** baoli_ has quit IRC | 21:38 | |
*** piet has quit IRC | 21:38 | |
*** sdake has joined #openstack-ironic | 21:49 | |
*** jcoufal has quit IRC | 21:51 | |
*** moshele has quit IRC | 21:51 | |
mrda | Morning Ironic | 21:51 |
*** nicodemos is now known as nico-afk | 21:52 | |
xavierr | morning mrda :) | 21:52 |
mrda | hi xavierr | 21:53 |
*** rcernin is now known as rcernin|off | 21:54 | |
*** xavierr is now known as xavierr_afk | 21:54 | |
*** sdake has quit IRC | 21:54 | |
*** r-daneel has joined #openstack-ironic | 21:55 | |
*** sdake has joined #openstack-ironic | 21:55 | |
*** rpioso has quit IRC | 21:56 | |
NobodyCam | morning mrda | 21:59 |
*** dprince has quit IRC | 22:01 | |
mrda | o/ | 22:07 |
*** rama_y has joined #openstack-ironic | 22:08 | |
*** ChrisAusten has quit IRC | 22:08 | |
*** cdearborn has quit IRC | 22:11 | |
*** thiagop has quit IRC | 22:17 | |
*** rcernin|off has quit IRC | 22:21 | |
*** gumbyblake has joined #openstack-ironic | 22:36 | |
*** rbudden has quit IRC | 22:37 | |
JayF | devananda: fyi https://review.openstack.org/#/c/327399/ is to make your api-ref changes effective by linking to them | 22:44 |
JayF | devananda: thanks very much for helping take that on | 22:44 |
devananda | JayF: ah! thanks for that | 22:47 |
devananda | JayF: have all my api-ref changes landed? | 22:47 |
JayF | all of them on that topic did | 22:47 |
JayF | it looked complete to me but that question makes me worry | 22:48 |
JayF | lol | 22:48 |
JayF | There are definately no more api-ref patchesowned by you | 22:48 |
JayF | https://review.openstack.org/#/q/topic:api-ref+and+project:openstack/ironic are all merged | 22:49 |
mikal | JayF: hey, I am expecting to batch review backports this week | 22:49 |
mikal | JayF: I expect tonyb to say sarastic things aobut that ot me this morning | 22:49 |
JayF | mikal: I already bribed mrda to get your attention | 22:49 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add cleanwait timeout cleanup process https://review.openstack.org/327403 | 22:49 |
JayF | mikal: so I'm going to have to get a refund from him then! | 22:49 |
TheJulia | JayF: ^^^ | 22:49 |
JayF | mikal: thanks, just wanted to make sure it's on someone's radar | 22:49 |
* JayF has had patches sit for long-ass times in nova | 22:50 | |
mikal | JayF: lies. Nova is perfect! Perfect I say! | 22:51 |
mikal | JayF: also, yes ask mrda for a refund | 22:51 |
mikal | :P | 22:51 |
TheJulia | lol | 22:51 |
JayF | I'm a little sore about patches being ignored, I started tracking my outstanding patches more carefully | 22:51 |
JayF | and that seems to make it worse | 22:52 |
JayF | lol | 22:52 |
JayF | well, more like any patches I care about, not just my outstanding ones | 22:52 |
*** piet has joined #openstack-ironic | 22:53 | |
*** jaybeale has quit IRC | 22:54 | |
mrda | mikal: why do you hang out here and not our team channel? Do you hate us? | 22:54 |
mrda | mikal: Also, I asked tonyb, because he is less likely to ignore me. HINT HINT :-P | 22:54 |
*** yolanda has quit IRC | 22:55 | |
mrda | mikal: Also, I have a Pixie Boots sticker for you as a bribe. So +2 those reviews. | 22:55 |
*** amotoki has quit IRC | 22:56 | |
JayF | Actually, that's more of a proxy bribe | 22:57 |
JayF | since I bought and shipped those out to you | 22:57 |
JayF | lol | 22:57 |
mrda | JayF: sssh! | 22:57 |
mrda | :) | 22:58 |
*** Jabadia has joined #openstack-ironic | 23:00 | |
*** amotoki has joined #openstack-ironic | 23:02 | |
*** PollyZ has quit IRC | 23:02 | |
*** sabeen has quit IRC | 23:07 | |
*** bnemec has quit IRC | 23:11 | |
*** snizar has joined #openstack-ironic | 23:13 | |
openstackgerrit | Julia Kreger proposed openstack/ironic: Add cleanwait timeout cleanup process https://review.openstack.org/327403 | 23:14 |
TheJulia | JayF: revised ^ based on feedback | 23:14 |
*** sacharya_ has quit IRC | 23:17 | |
*** bnemec has joined #openstack-ironic | 23:19 | |
*** snizar has quit IRC | 23:30 | |
*** ppiela has quit IRC | 23:38 | |
*** mbound has quit IRC | 23:39 | |
*** alexpilotti has quit IRC | 23:40 | |
*** mbound has joined #openstack-ironic | 23:40 | |
*** Goneri has quit IRC | 23:40 | |
*** mbound has quit IRC | 23:44 | |
*** Sukhdev has joined #openstack-ironic | 23:47 | |
* tonyb would take a pixie boot sticker | 23:49 | |
* tonyb +W's patches | 23:49 | |
JayF | tonyb: you're in Australia, right? I shipped quite a few that way to mrda | 23:58 |
JayF | tonyb: if he's all out lmk and I'll ship ya one | 23:58 |
JayF | FWIW that goes for any Ironic contributor; if you want a pixie boots sticker and don't have one just ask me nicely (And land my patches) | 23:58 |
tonyb | JayF: Thanks. I'll grab one from mrda the next time I see him. | 23:58 |
tonyb | JayF: I feel like it's appropriate that i get a proxy sticker for being a proxy contributor ;P | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!