*** sgotliv_ has joined #openstack-sahara | 00:11 | |
*** sgotliv has quit IRC | 00:14 | |
*** openstackgerrit has quit IRC | 00:16 | |
*** openstackgerrit has joined #openstack-sahara | 00:16 | |
*** witlessb has quit IRC | 00:22 | |
*** sgotliv_ has quit IRC | 00:22 | |
*** tmckay has joined #openstack-sahara | 00:25 | |
*** jinxing has joined #openstack-sahara | 00:40 | |
*** tmckay has quit IRC | 00:55 | |
*** ViswaV has joined #openstack-sahara | 01:03 | |
*** tellesnobrega_af is now known as tellesnobrega | 01:03 | |
*** ViswaV_ has joined #openstack-sahara | 01:07 | |
*** ViswaV has quit IRC | 01:07 | |
*** hdd has joined #openstack-sahara | 01:26 | |
openstackgerrit | Jaxon Wang proposed openstack/sahara: Add unit test cases for cdh plugin utils https://review.openstack.org/248068 | 02:05 |
---|---|---|
openstackgerrit | Merged openstack/sahara: Option for disabling wait condition feature https://review.openstack.org/245716 | 02:12 |
*** david-ly_ has quit IRC | 02:36 | |
*** tellesnobrega is now known as tellesnobrega_af | 02:36 | |
*** houming has joined #openstack-sahara | 02:39 | |
*** tmckay has joined #openstack-sahara | 02:42 | |
*** tmckay has left #openstack-sahara | 02:43 | |
openstackgerrit | lu huichun proposed openstack/sahara: Update scenario test readme file https://review.openstack.org/248841 | 03:03 |
*** pcaruana has quit IRC | 03:27 | |
*** jinxing has quit IRC | 03:35 | |
*** jinxing has joined #openstack-sahara | 03:36 | |
*** david-lyle has joined #openstack-sahara | 04:08 | |
*** hdd has quit IRC | 04:10 | |
*** akuznetsov has joined #openstack-sahara | 04:47 | |
*** akuznetsov has quit IRC | 04:52 | |
*** ashishb has joined #openstack-sahara | 05:14 | |
*** rcernin has joined #openstack-sahara | 05:15 | |
*** rcernin has quit IRC | 05:34 | |
*** akuznetsov has joined #openstack-sahara | 05:45 | |
*** pcaruana has joined #openstack-sahara | 05:49 | |
*** Poornima has joined #openstack-sahara | 05:50 | |
*** rcernin has joined #openstack-sahara | 06:18 | |
*** sgotliv has joined #openstack-sahara | 06:19 | |
*** pcaruana has quit IRC | 06:28 | |
*** akuznetsov has quit IRC | 06:56 | |
*** akuznetsov has joined #openstack-sahara | 07:02 | |
*** akuznetsov has quit IRC | 07:06 | |
*** nkrinner has joined #openstack-sahara | 07:10 | |
*** akuznetsov has joined #openstack-sahara | 07:35 | |
*** akuznetsov has quit IRC | 08:26 | |
openstackgerrit | lu huichun proposed openstack/sahara: Check when cluster is None https://review.openstack.org/249068 | 08:36 |
*** _degorenko|afk is now known as degorenko | 08:42 | |
*** witlessb has joined #openstack-sahara | 08:54 | |
*** vgridnev has joined #openstack-sahara | 09:15 | |
*** openstackgerrit has quit IRC | 09:16 | |
*** openstackgerrit has joined #openstack-sahara | 09:16 | |
*** jinxing has quit IRC | 09:32 | |
*** ashishb has quit IRC | 09:35 | |
*** ashishb has joined #openstack-sahara | 09:41 | |
*** jinxing has joined #openstack-sahara | 09:42 | |
openstackgerrit | lu huichun proposed openstack/sahara: Check when cluster is None https://review.openstack.org/249068 | 10:06 |
*** jinxing has quit IRC | 10:20 | |
*** houming has quit IRC | 10:21 | |
*** tosky has joined #openstack-sahara | 10:22 | |
openstackgerrit | Vitaly Gridnev proposed openstack/sahara: testing commit https://review.openstack.org/249112 | 10:26 |
*** jinxing has joined #openstack-sahara | 10:28 | |
*** jinxing has quit IRC | 10:32 | |
*** tellesnobrega_af is now known as tellesnobrega | 10:34 | |
*** ViswaV_ has quit IRC | 11:05 | |
*** houming has joined #openstack-sahara | 11:08 | |
*** ViswaV has joined #openstack-sahara | 11:08 | |
*** rcernin is now known as rcernin|lunch | 11:13 | |
*** jinxing has joined #openstack-sahara | 11:55 | |
openstackgerrit | Jaxon Wang proposed openstack/sahara: Add unit test cases for cdh plugin utils https://review.openstack.org/248068 | 12:12 |
openstackgerrit | Vitaly Gridnev proposed openstack/sahara: testing commit https://review.openstack.org/249112 | 12:17 |
*** tellesnobrega is now known as tellesnobrega_af | 12:19 | |
*** tellesnobrega_af is now known as tellesnobrega | 12:25 | |
*** rcernin|lunch is now known as rcernin | 12:36 | |
*** ashishb has quit IRC | 12:38 | |
*** tmckay has joined #openstack-sahara | 12:47 | |
*** tmckay has quit IRC | 12:52 | |
*** houming has quit IRC | 13:04 | |
openstackgerrit | Vitaly Gridnev proposed openstack/sahara: Enable heat_enable_wait_condition by default https://review.openstack.org/249112 | 13:16 |
*** _crobertsrh is now known as crobertsrh | 13:48 | |
*** tmckay has joined #openstack-sahara | 13:53 | |
*** tmckay has left #openstack-sahara | 13:57 | |
*** houming has joined #openstack-sahara | 13:57 | |
*** tosky_ has joined #openstack-sahara | 14:01 | |
*** tosky has quit IRC | 14:03 | |
*** rcernin has quit IRC | 14:03 | |
*** Poornima has quit IRC | 14:11 | |
*** egafford has joined #openstack-sahara | 14:29 | |
openstackgerrit | Merged openstack/python-saharaclient: Replacing hard coded cluster status using utils in sahara-client https://review.openstack.org/247444 | 14:40 |
*** goodygum_ is now known as noqa_v_qoovnie | 14:44 | |
*** tosky_ is now known as tosky | 14:48 | |
*** tmckay has joined #openstack-sahara | 14:51 | |
openstackgerrit | Michael Ionkin proposed openstack/sahara: Fix bashate warnings https://review.openstack.org/248822 | 14:54 |
openstackgerrit | Michael McCune proposed openstack/sahara: [WIP] Initial key manager implementation https://review.openstack.org/220680 | 14:57 |
*** houming has quit IRC | 14:58 | |
*** houming has joined #openstack-sahara | 15:05 | |
*** jinxing has quit IRC | 15:13 | |
*** vgridnev has quit IRC | 15:19 | |
*** houming has quit IRC | 15:44 | |
*** houming has joined #openstack-sahara | 15:45 | |
openstackgerrit | Evgeny Sikachev proposed openstack/sahara: [do not review]test commit https://review.openstack.org/249281 | 15:49 |
*** hdd has joined #openstack-sahara | 16:00 | |
openstackgerrit | Evgeny Sikachev proposed openstack/sahara-ci-config: Fix section name for tempest tests https://review.openstack.org/249292 | 16:03 |
openstackgerrit | Merged openstack/sahara-ci-config: Fix section name for tempest tests https://review.openstack.org/249292 | 16:09 |
*** vgridnev has joined #openstack-sahara | 16:11 | |
*** nkrinner has quit IRC | 16:12 | |
*** rcernin has joined #openstack-sahara | 16:16 | |
openstackgerrit | Evgeny Sikachev proposed openstack/sahara: [do not review]test commit tempest https://review.openstack.org/249281 | 16:17 |
openstackgerrit | Evgeny Sikachev proposed openstack/sahara-ci-config: Migrate to heat-engine https://review.openstack.org/249319 | 16:44 |
*** ashishb has joined #openstack-sahara | 16:51 | |
*** tosky has quit IRC | 16:53 | |
*** tosky has joined #openstack-sahara | 16:54 | |
*** degorenko is now known as _degorenko|afk | 16:58 | |
*** david-ly_ has joined #openstack-sahara | 17:19 | |
*** david-l__ has joined #openstack-sahara | 17:23 | |
*** david-ly_ has quit IRC | 17:23 | |
*** david-lyle has quit IRC | 17:23 | |
*** david-l__ has quit IRC | 17:30 | |
*** houming has quit IRC | 17:34 | |
*** david-lyle has joined #openstack-sahara | 17:44 | |
openstackgerrit | Ethan Gafford proposed openstack/sahara-specs: Move Image Generation Scripts into Plugins https://review.openstack.org/248842 | 17:56 |
*** david-lyle has quit IRC | 18:03 | |
*** david-lyle has joined #openstack-sahara | 18:11 | |
*** ViswaV has quit IRC | 18:12 | |
*** ViswaV has joined #openstack-sahara | 18:13 | |
*** tosky has quit IRC | 18:26 | |
openstackgerrit | lu huichun proposed openstack/sahara: No need to check if edp_engine is None when run job https://review.openstack.org/249068 | 18:46 |
*** tellesnobrega is now known as tellesnobrega_af | 19:03 | |
*** ashishb has quit IRC | 19:36 | |
*** ashishb has joined #openstack-sahara | 20:04 | |
*** hdd has quit IRC | 20:17 | |
*** ashishb has quit IRC | 20:34 | |
*** crobertsrh1 has joined #openstack-sahara | 20:53 | |
*** raildo is now known as raildo-afk | 21:01 | |
elmiko | crobertsrh1, tmckay, i've come across a situation that i'm having trouble with for the secret store. i could use a sounding board for advice =) | 21:08 |
crobertsrh1 | Shout away. Here or bluejeans? | 21:08 |
elmiko | i think here works, but bluejeans might be faster. i could do either | 21:08 |
crobertsrh1 | Ok | 21:09 |
elmiko | so, i've got the storage and retrieval working just fine | 21:09 |
tmckay | same here | 21:09 |
elmiko | but i've run into a sticky spot with updates | 21:09 |
elmiko | and to a lesser extent deletes | 21:09 |
elmiko | an example, | 21:09 |
elmiko | i create a job binary, that is stored in swift, and uses my credentials | 21:10 |
elmiko | i then choose to update that job binary | 21:10 |
elmiko | now, at this point, if i am using barbican, i have created a stored secret in the barbican db | 21:10 |
elmiko | so, if i update the creds for my job binary, i will need to destroy the old entry in barbican and create a new one | 21:11 |
elmiko | but, this will require me to load the job binary, check the values, then do something | 21:11 |
elmiko | looking at the conductor, i'm not seeing other interactions like this. does it make sense to load the job binary before the update, are there any gotchas i should worry about? | 21:11 |
elmiko | (a similar situation occurs on delete) | 21:12 |
crobertsrh1 | I think it makes sense....I had to do something similar for updating shares | 21:12 |
elmiko | oh yea, where you check the ones that already there? | 21:12 |
crobertsrh1 | on an active cluster share update, I first get the original object so I can compare it to the update | 21:12 |
elmiko | yea, i vaguely remember that review | 21:12 |
crobertsrh1 | I think I did it in the manager | 21:12 |
elmiko | ok, i'll look at that example again | 21:13 |
elmiko | i just wasn't sure about loading the jb before updating, i suppose there is no problem there though? | 21:13 |
crobertsrh1 | conductor was my first thought as well | 21:13 |
tmckay | elmiko, I don't see a problem | 21:13 |
elmiko | k, thanks guys | 21:13 |
tmckay | why do you have to check values? | 21:13 |
crobertsrh1 | I don't think so...slightly less efficient, but you gotta do something, right? | 21:13 |
elmiko | for update, i just need to make sure we aren't updating the creds. which i think i can do before having to load the object | 21:14 |
tmckay | elmiko, could you assume that if someone put creds in the data section for the update that they are in fact being updated? | 21:14 |
elmiko | for delete, i need to load the object to get the key id so i can remove it from barbican, to ensure we don't junk up the barbican db | 21:14 |
elmiko | tmckay: yea, i think i will have to do that | 21:15 |
tmckay | it would be a little inefficient if someone is setting creds to the current value, but ... | 21:15 |
elmiko | yea | 21:15 |
tmckay | elmiko, either way. a query on the current object should not be a problem | 21:16 |
crobertsrh1 | I wouldn't -1 a change for that | 21:16 |
elmiko | ok, cool | 21:16 |
tmckay | elmiko, we sorta do checks like that here and there when we look to see if objects are referenced by other objects | 21:16 |
elmiko | i had been so focused on creating the secrets that i totally forgot about cleaning up after my mess ;) | 21:16 |
crobertsrh1 | heh....updates are always an afterthought in sahara :) | 21:17 |
elmiko | tmckay: yea, i thought we did. i just wanted to make sure i wasn't missing some detail. | 21:17 |
tmckay | I think job binary updates for example will be relatively rare compared to the other things we do, so an extra query here or there should be fine | 21:17 |
elmiko | agreed | 21:18 |
elmiko | and it would be rare to update credentials | 21:18 |
openstackgerrit | Ethan Gafford proposed openstack/sahara-specs: Move Image Generation Scripts into Plugins https://review.openstack.org/248842 | 21:21 |
crobertsrh1 | doh....I am reviewing the old version :) | 21:46 |
*** egafford has quit IRC | 21:47 | |
*** tmckay has left #openstack-sahara | 21:54 | |
*** sgotliv has quit IRC | 22:02 | |
elmiko | is something funky happening with the ci? | 22:07 |
crobertsrh1 | Might be...I've seen a lot of fails | 22:09 |
elmiko | just getting really weird errors from my last patch | 22:14 |
crobertsrh1 | which patch? | 22:14 |
elmiko | https://review.openstack.org/#/c/220680/ | 22:15 |
crobertsrh1 | Strange, it's like swift ain't home | 22:16 |
crobertsrh1 | Nothing surprises me when it comes to our ci | 22:16 |
crobertsrh1 | we need a ci for our ci | 22:16 |
elmiko | lol | 22:16 |
elmiko | yea, i'm just trying to sort through these errors to figure out what i've introduced and what is normal background noise | 22:16 |
crobertsrh1 | ci-ci aas | 22:17 |
crobertsrh1 | project "cici" | 22:17 |
elmiko | nice | 22:17 |
crobertsrh1 | Ok. Since everything is broken, it must be time to go curling. Have a good one :) | 22:18 |
elmiko | likewise | 22:18 |
*** crobertsrh1 has quit IRC | 22:22 | |
*** vgridnev has quit IRC | 22:24 | |
*** rcernin has quit IRC | 23:42 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!