| *** RockKuo has joined #openstack-sahara | 00:42 | |
| *** IlyaE has quit IRC | 01:01 | |
| *** elmiko has quit IRC | 02:03 | |
| *** bradd1 has quit IRC | 02:34 | |
| *** bradd1 has joined #openstack-sahara | 02:35 | |
| *** akuznetsov has joined #openstack-sahara | 03:39 | |
| *** bradd1 has quit IRC | 03:51 | |
| *** akuznetsov has quit IRC | 03:53 | |
| *** akuznetsov has joined #openstack-sahara | 04:53 | |
| *** sballe_ has joined #openstack-sahara | 04:56 | |
| *** sballe has quit IRC | 04:59 | |
| *** RockKuo has quit IRC | 04:59 | |
| *** RockKuo has joined #openstack-sahara | 05:00 | |
| *** saju_m has joined #openstack-sahara | 05:16 | |
| *** IlyaE has joined #openstack-sahara | 05:32 | |
| *** akuznetsov has quit IRC | 06:00 | |
| *** akuznetsov has joined #openstack-sahara | 06:02 | |
| *** RockKuo has quit IRC | 06:05 | |
| openstackgerrit | Jenkins proposed a change to openstack/sahara: Imported Translations from Transifex https://review.openstack.org/86507 | 06:06 |
|---|---|---|
| *** IlyaE has quit IRC | 06:58 | |
| *** IlyaE has joined #openstack-sahara | 07:10 | |
| *** skolekonov has joined #openstack-sahara | 07:16 | |
| *** IlyaE has quit IRC | 07:18 | |
| *** skolekonov has quit IRC | 07:22 | |
| *** skolekonov has joined #openstack-sahara | 07:26 | |
| *** witlessb has joined #openstack-sahara | 07:27 | |
| *** witlessb has quit IRC | 07:45 | |
| *** witlessb has joined #openstack-sahara | 07:47 | |
| openstackgerrit | Daniele Venzano proposed a change to openstack/sahara-image-elements: Update hadoop-cdh element https://review.openstack.org/86820 | 08:05 |
| *** RockKuo has joined #openstack-sahara | 08:58 | |
| *** tosky has joined #openstack-sahara | 09:38 | |
| *** saju_m has quit IRC | 09:45 | |
| openstackgerrit | Dmitry Mescheryakov proposed a change to openstack/sahara: Fix storing binaries in Swift https://review.openstack.org/86732 | 09:53 |
| *** saju_m has joined #openstack-sahara | 10:01 | |
| *** saju_m has quit IRC | 10:25 | |
| openstackgerrit | Dmitry Mescheryakov proposed a change to openstack/sahara: Fix storing binaries in Swift https://review.openstack.org/86732 | 10:28 |
| tosky | dmitryme: does https://bugs.launchpad.net/sahara/+bug/1306505 mean that, even with the swift patch, the job is not going to work? | 10:33 |
| tosky | (btw, maybe https://bugs.launchpad.net/sahara/+bug/1306506 could be marked as duplicat of 1306505, not just invalid) | 10:34 |
| dmitryme | tosky: there are two ways to point to libs located in swift | 10:43 |
| dmitryme | 1. individual, like swift://test-container.sahara/edp-lib.jar | 10:43 |
| dmitryme | 2. a batch of libs located in container root, like swift://test-container.sahara | 10:44 |
| dmitryme | after my fix the 1st option works, the 2nd - still does not | 10:44 |
| tosky | oh, I didn't know about the 2nd | 10:44 |
| tosky | I used only the 1st one | 10:44 |
| dmitryme | it is not documented | 10:44 |
| dmitryme | I've just found it in the code | 10:45 |
| tosky | ah | 10:45 |
| dmitryme | seems like it is completely untested either | 10:45 |
| dmitryme | regarding marking bug as duplicate, for some reason I don't see 'duplicate' state among available | 10:46 |
| tosky | uhm, maybe that usage should be prevented while it's not complete (if it's easier than fix it) | 10:46 |
| dmitryme | tosky: I think it is pretty safe just not mention it | 10:46 |
| tosky | dmitryme: yes, duplicated is not a state it seems, but there is a " Mark as duplicate " link on the right pane | 10:46 |
| tosky | (I had to search the ubuntu bug triaging documentation '-_-) | 10:47 |
| dmitryme | :-) lol | 10:47 |
| dmitryme | ok, marked it as duplicate | 10:49 |
| tosky | thanks! | 10:49 |
| *** RockKuo has quit IRC | 11:19 | |
| openstackgerrit | Vadim Rovachev proposed a change to openstack/sahara: Add tenant_id getting in integration tests https://review.openstack.org/84781 | 11:45 |
| *** _crobertsrh is now known as crobertsrh | 12:46 | |
| *** tmckay has joined #openstack-sahara | 13:03 | |
| *** elmiko has joined #openstack-sahara | 13:08 | |
| *** witlessb has quit IRC | 13:25 | |
| *** witlessb has joined #openstack-sahara | 13:29 | |
| aignatov | tmckay: hi | 13:49 |
| aignatov | could you please take a look https://review.openstack.org/#/c/86732/ ? | 13:50 |
| *** elmiko has quit IRC | 13:56 | |
| *** elmiko has joined #openstack-sahara | 13:56 | |
| *** skolekonov has quit IRC | 13:58 | |
| *** IlyaE has joined #openstack-sahara | 14:00 | |
| SergeyLukjanov | crobertsrh, what do you think about adding "Co-Authored-By: ..." for your patches of merging sahara-dashboard to horizon for the initial authors? | 14:06 |
| crobertsrh | sure.....do we have a list? | 14:06 |
| tosky | I guess it's not possible because of project policies to inject a set of git commits (with the history) in that repository... | 14:06 |
| SergeyLukjanov | tosky, it's just a "Co-Authored-By" line in commit message | 14:08 |
| tosky | yes, I was thinking about saving the history (I've seen it in other projects) | 14:08 |
| SergeyLukjanov | crobertsrh, we could generate a list for each CR or add top X contributors | 14:09 |
| SergeyLukjanov | tosky, different file structure, some nit changes, it'll be hell to apply ~200 patches | 14:09 |
| tosky | SergeyLukjanov: trust me, I've seen worst scenarios :) | 14:09 |
| SergeyLukjanov | tosky ;) | 14:10 |
| SergeyLukjanov | crobertsrh, regarding to the https://github.com/openstack/sahara-dashboard/graphs/contributors | 14:11 |
| SergeyLukjanov | crobertsrh, I think that if you add Nikita and Dmitry in addition to you as author it'll cover 90% | 14:11 |
| tmckay | aignatov, sure | 14:12 |
| tosky | SergeyLukjanov: maybe also a pointer to the original repository too? | 14:13 |
| SergeyLukjanov | tosky, yup, pointer is good too | 14:14 |
| SergeyLukjanov | just to keep history a bit | 14:14 |
| SergeyLukjanov | and to understand were to search for reasoning :) | 14:14 |
| crobertsrh | Ok, I can add Nikita and Dmitry. and a pointer to https://github.com/openstack/sahara-dashboard | 14:14 |
| *** bradd1 has joined #openstack-sahara | 14:14 | |
| SergeyLukjanov | crobertsrh, it's based on top 4 contribs - Nikita, you, me and Dmitry | 14:17 |
| crobertsrh | Ok, I'll add all 4. | 14:17 |
| SergeyLukjanov | crobertsrh, me participated mostly in infra stuff, so -me | 14:17 |
| crobertsrh | ok 3 :) | 14:17 |
| SergeyLukjanov | crobertsrh, yup, if you ok with than it'll be good I think | 14:17 |
| openstackgerrit | Sergey Reshetnyak proposed a change to openstack/sahara: Remove IDH plugin from sahara https://review.openstack.org/86913 | 14:18 |
| *** bogdando has quit IRC | 14:28 | |
| *** bogdando has joined #openstack-sahara | 14:28 | |
| crobertsrh | SergeyLukjanov: If https://review.openstack.org/#/c/86902/ looks good, I'll change the others. | 14:29 |
| SergeyLukjanov | crobertsrh, yup, it looks good, thank you! | 14:43 |
| *** elmiko has quit IRC | 15:40 | |
| *** elmiko has joined #openstack-sahara | 15:41 | |
| tosky | dmitryme: around? I was testing the swift fix (v2) and I see this, is it because I put wrong values in the parameters? http://www.fpaste.org/93570/31072139/ | 15:51 |
| tosky | tmckay: ^ | 15:51 |
| tmckay | not sure | 15:52 |
| dmitryme | tosky: https://github.com/openstack/sahara/blob/master/sahara/service/edp/binary_retrievers/internal_swift.py#L28 | 15:58 |
| tosky | (anyway, the exception is not trapped and the job stays in pending state) | 15:58 |
| dmitryme | we take username and password from data source | 15:59 |
| dmitryme | but we take tenant from OS_ADMIN_TENANT parameter from sahara.conf | 15:59 |
| tosky | uhm, but the tenant I used is "demo", not "admin" | 15:59 |
| tosky | in fact it tries to access with "service", but it fails it seems | 16:00 |
| dmitryme | tosky: check that user from DataSource has access to OS_ADMIN_TENANT | 16:01 |
| dmitryme | from your words I assume OS_ADMIN_TENANT=service | 16:01 |
| tosky | "User abab3eb5c92b449d972c2b582c05add6 is unauthorized for tenant d9c50311b94c47b8bed593e422e2b1a3" means user admin and tenant service | 16:02 |
| dmitryme | ok, change OS_ADMIN_TENANT from 'service' to 'admin' or 'demo' and try again | 16:02 |
| dmitryme | ou, actually it should be os_admin_tenant_name | 16:03 |
| tosky | uh, admin is not in the service project/tenant | 16:03 |
| tosky | but if I change os_admin_tenant_name to something else, could it break other parts? | 16:03 |
| dmitryme | tosky: right, seems like that is a limitation of current design we overlooked | 16:03 |
| dmitryme | os_admin_tenant_name has only one other restriction: os_admin_username _must_ have 'admin' role in that tenant | 16:04 |
| dmitryme | looks like we have to allow user specify tenant_name in DataSource as well | 16:05 |
| tosky | should I file a bug/blueprint/other or wait? | 16:06 |
| dmitryme | tosky: yep, please file a bug | 16:11 |
| tosky | roger :) | 16:14 |
| *** ylobankov_ has quit IRC | 16:30 | |
| tosky | dmitryme: done, if something is missing please tell me https://bugs.launchpad.net/sahara/+bug/1306713 | 16:36 |
| dmitryme | tosky: It seems to be complete | 16:50 |
| tosky | dmitryme: I filed another one about the untrapped exceptions | 16:50 |
| dmitryme | agree, we should fix that one too | 16:51 |
| openstackgerrit | A change was merged to openstack/sahara: Fix storing binaries in Swift https://review.openstack.org/86732 | 16:56 |
| *** IlyaE has quit IRC | 17:09 | |
| *** IlyaE has joined #openstack-sahara | 17:27 | |
| *** elmiko is now known as elmiko_afk | 17:32 | |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Add RHEL as an expected platform https://review.openstack.org/86977 | 17:53 |
| SergeyLukjanov | _mattf, yay! rhel is coming! | 17:54 |
| crobertsrh | :) | 18:11 |
| *** tosky has quit IRC | 18:12 | |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Reformat the usage message https://review.openstack.org/86986 | 18:30 |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Refactor logic around selinux bug 1204824 https://review.openstack.org/86987 | 18:30 |
| *** _mattf is now known as mattf | 18:31 | |
| mattf | SergeyLukjanov, aye. it came w/ centos, it's just now that i feel comfortable that the image generation is working to fix the check | 18:32 |
| mattf | SergeyLukjanov, is andrew filing a "remove idh" bp for juno? | 18:32 |
| SergeyLukjanov | mattf, sreshetnyak already created it and pushed the CR | 18:33 |
| SergeyLukjanov | mattf, cool re rhel | 18:33 |
| * mattf needs to pop out of summit hell | 18:33 | |
| *** elmiko_afk is now known as elmiko | 18:34 | |
| * mattf finds himself partially procrastinating by cleaning elements | 18:34 | |
| SergeyLukjanov | heh :) | 18:35 |
| mattf | sreshetnyak, i'm in the elements code right now, mind if i nuke idh while i'm here? | 18:35 |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Remove IDH support https://review.openstack.org/86991 | 18:39 |
| * mattf takes the forgiveness path | 18:39 | |
| * mattf has sadly discovered debug mode is hdp only | 18:41 | |
| *** IlyaE has quit IRC | 18:42 | |
| *** IlyaE has joined #openstack-sahara | 18:47 | |
| elmiko | mattf: so happy to see your rhel patch for image-elements :) | 18:48 |
| mattf | i just couldnt wait! | 18:51 |
| * mattf tempts his fate w/ bob | 18:52 | |
| elmiko | lol | 18:52 |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Idiomatic -d (debug) flag https://review.openstack.org/86997 | 18:53 |
| mattf | ^^ | 18:53 |
| elmiko | fyi, running `diskimage-create.sh -p hdp -v 2 -d true` will fail with that password issue on rhel | 18:55 |
| mattf | good call, need an selinux test | 18:57 |
| elmiko | yea | 18:57 |
| mattf | worse than fail, it'll false positive | 18:58 |
| elmiko | uh-oh... | 18:58 |
| elmiko | i didn't notice the false positive, i just saw error spam and threw up my hands | 18:59 |
| mattf | i'm wrong, it won'y | 19:00 |
| mattf | it would in my hacked version (added || :) | 19:00 |
| elmiko | lol | 19:01 |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Refactor platform detection code https://review.openstack.org/87000 | 19:14 |
| openstackgerrit | Sergey Reshetnyak proposed a change to openstack/sahara: Remove IDH plugin from sahara https://review.openstack.org/86913 | 19:14 |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Detect bug 1292614 https://review.openstack.org/87003 | 19:22 |
| mattf | elmiko, for you ^^ | 19:22 |
| elmiko | very nice, +1 | 19:23 |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Add -d option to vanilla images https://review.openstack.org/87011 | 19:50 |
| *** mattf is now known as _mattf | 19:53 | |
| *** _mattf is now known as mattf | 19:53 | |
| *** sballe_ has quit IRC | 20:07 | |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Remove IDH support https://review.openstack.org/86991 | 20:09 |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Detect bug 1292614 https://review.openstack.org/87003 | 20:09 |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Refactor platform detection code https://review.openstack.org/87000 | 20:09 |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Idiomatic -d (debug) flag https://review.openstack.org/86997 | 20:09 |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Add -d option to vanilla images https://review.openstack.org/87011 | 20:09 |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Detect bug 1292614 https://review.openstack.org/87003 | 20:42 |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Refactor platform detection code https://review.openstack.org/87000 | 20:42 |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Add -d option to vanilla images https://review.openstack.org/87011 | 20:42 |
| *** crobertsrh is now known as _crobertsrh | 21:02 | |
| openstackgerrit | Michael McCune proposed a change to openstack/sahara: Implementing constants for the job types used by EDP https://review.openstack.org/80352 | 21:25 |
| *** elmiko is now known as elmiko_afk | 21:33 | |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Detect bug 1292614 https://review.openstack.org/87003 | 21:47 |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Refactor platform detection code https://review.openstack.org/87000 | 21:47 |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Idiomatic -d (debug) flag https://review.openstack.org/86997 | 21:47 |
| openstackgerrit | Matthew Farrellee proposed a change to openstack/sahara-image-elements: Add -d option to vanilla images https://review.openstack.org/87011 | 21:48 |
| *** tellesnobrega has quit IRC | 21:49 | |
| *** IlyaE has quit IRC | 22:45 | |
| *** witlessb has quit IRC | 23:03 | |
| openstackgerrit | A change was merged to openstack/sahara-image-elements: Add inject hadoop native libs to vanilla hadoop 2 https://review.openstack.org/83036 | 23:28 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!