| *** Longgeek has joined #openstack-sahara | 01:37 | |
| *** Longgeek has quit IRC | 01:42 | |
| *** witlessb has joined #openstack-sahara | 01:56 | |
| *** witlessb has quit IRC | 02:01 | |
| *** chandankumar has joined #openstack-sahara | 02:02 | |
| *** Longgeek has joined #openstack-sahara | 03:22 | |
| *** chandankumar has quit IRC | 04:05 | |
| *** chandankumar has joined #openstack-sahara | 04:10 | |
| *** chandan_kumar has joined #openstack-sahara | 05:02 | |
| *** chandankumar has quit IRC | 05:06 | |
| *** chandan_kumar has quit IRC | 05:28 | |
| *** chandankumar has joined #openstack-sahara | 05:28 | |
| *** chandan_kumar has joined #openstack-sahara | 05:41 | |
| *** chandankumar has quit IRC | 05:42 | |
| *** chandan_kumar has quit IRC | 05:47 | |
| openstackgerrit | Andrew Lazarev proposed a change to openstack/sahara-specs: Added spec for indirect VMs access https://review.openstack.org/128475 | 06:05 |
|---|---|---|
| *** k4n0 has joined #openstack-sahara | 06:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/sahara: Imported Translations from Transifex https://review.openstack.org/129142 | 06:14 |
| *** tellesnobrega has quit IRC | 06:24 | |
| *** chandankumar has joined #openstack-sahara | 06:44 | |
| *** stannie has joined #openstack-sahara | 07:50 | |
| *** skolekonov has joined #openstack-sahara | 08:02 | |
| *** witlessb has joined #openstack-sahara | 08:09 | |
| *** IvanBerezovskiy has joined #openstack-sahara | 08:20 | |
| *** dmitryme has quit IRC | 08:37 | |
| *** Longgeek has quit IRC | 08:40 | |
| *** Longgeek has joined #openstack-sahara | 08:41 | |
| *** dmitryme has joined #openstack-sahara | 08:42 | |
| *** Weiting has joined #openstack-sahara | 08:52 | |
| openstackgerrit | A change was merged to stackforge/sahara-ci-config: Fix typo in floating_ip_pool config name https://review.openstack.org/129044 | 08:55 |
| openstackgerrit | Denis Egorenko proposed a change to stackforge/sahara-ci-config: Uncomment section for UI image https://review.openstack.org/129175 | 09:06 |
| *** amarouni has joined #openstack-sahara | 09:17 | |
| *** Weiting has quit IRC | 09:18 | |
| *** tosky has joined #openstack-sahara | 09:26 | |
| openstackgerrit | A change was merged to stackforge/sahara-ci-config: Uncomment section for UI image https://review.openstack.org/129175 | 09:43 |
| openstackgerrit | Sergey Reshetnyak proposed a change to openstack/sahara: Fix parallel testing EDP jobs for Fedora and CentOS images https://review.openstack.org/129194 | 10:02 |
| *** tmckay has left #openstack-sahara | 10:02 | |
| openstackgerrit | Andrey Pavlov proposed a change to openstack/python-saharaclient: Add volume_type support to node group templates https://review.openstack.org/127963 | 11:45 |
| *** tnovacik has joined #openstack-sahara | 12:06 | |
| openstackgerrit | Andrey Pavlov proposed a change to openstack/sahara: Added cancel before deleting job execution https://review.openstack.org/120119 | 12:09 |
| *** _crobertsrh is now known as crobertsrh | 12:15 | |
| openstackgerrit | Denis Egorenko proposed a change to stackforge/sahara-ci-config: Mark as non-voting jobs with Vanilla 2.4 https://review.openstack.org/129218 | 12:16 |
| openstackgerrit | Kazuki OIKAWA proposed a change to openstack/sahara-specs: Enable Swift resident Hive tables for EDP with the vanilla plugin https://review.openstack.org/115259 | 12:25 |
| *** chandankumar has quit IRC | 12:32 | |
| *** tnovacik has quit IRC | 12:41 | |
| *** k4n0 has quit IRC | 12:55 | |
| *** tellesnobrega has joined #openstack-sahara | 13:13 | |
| tellesnobrega | elmiko, crobertsrh i just watched the juno release promo video yesterday. pretty cool that its main feature is sahara | 13:14 |
| tellesnobrega | nice work :) | 13:15 |
| elmiko | tellesnobrega: nice, got a link? | 13:15 |
| crobertsrh | it was all elmiko! | 13:15 |
| * elmiko blushes | 13:15 | |
| tellesnobrega | http://www.youtube.com/v/TgPTjrf1y0A?autoplay=1 | 13:16 |
| elmiko | awesome, thanks! | 13:16 |
| tellesnobrega | sahara got like 20s out of 1:44m | 13:17 |
| elmiko | crobertsrh: i don't think they used any of my footage | 13:18 |
| crobertsrh | ah | 13:18 |
| crobertsrh | yeah...I was thinking of your video | 13:18 |
| crobertsrh | yours was better! | 13:19 |
| elmiko | still, cool to see prominent sahara placement | 13:19 |
| *** openstackgerrit has quit IRC | 13:19 | |
| elmiko | lol, thanks | 13:19 |
| *** openstackgerrit has joined #openstack-sahara | 13:19 | |
| openstackgerrit | Sergey Reshetnyak proposed a change to openstack/sahara: Add Hive support to CDH plugin https://review.openstack.org/128015 | 13:21 |
| tellesnobrega | congrats to all of you | 13:28 |
| tellesnobrega | hopefully i will have some time to work on storm again to have it in the next release | 13:28 |
| crobertsrh | great! | 13:30 |
| *** chandankumar has joined #openstack-sahara | 13:37 | |
| *** chandankumar has quit IRC | 13:44 | |
| *** tmckay has joined #openstack-sahara | 13:53 | |
| openstackgerrit | Vitaly Gridnev proposed a change to openstack/sahara: Drop obsolete wsgi and xmlutils modules https://review.openstack.org/129260 | 13:58 |
| openstackgerrit | Sergey Reshetnyak proposed a change to openstack/sahara: Add Hive support to CDH plugin https://review.openstack.org/128015 | 14:01 |
| *** skolekonov has quit IRC | 14:11 | |
| *** bradd1 has joined #openstack-sahara | 14:21 | |
| *** tellesnobrega has quit IRC | 14:27 | |
| *** tellesnobrega has joined #openstack-sahara | 14:28 | |
| openstackgerrit | Sergey Reshetnyak proposed a change to openstack/sahara: Add Hue support to Cloudera plugin https://review.openstack.org/129286 | 14:44 |
| *** tellesnobrega has quit IRC | 15:14 | |
| *** amarouni has left #openstack-sahara | 15:24 | |
| tmckay | Hi Sahara folks. This change is failing with an error on security groups, not sure why https://review.openstack.org/#/c/128737/ | 15:39 |
| tmckay | any insight? alazarev, SergeyLukjanov, sreshetnyak I'm guessing it is something else, but the test code looks okay to me. Is it possible there is something left over from another test? | 15:41 |
| tmckay | The error message is coming from validation code in Sahara | 15:41 |
| elmiko | weird error | 15:43 |
| tmckay | elmiko, yeah, it thinks the sec group is already defined in nova. | 15:43 |
| elmiko | definitely what it looks like | 15:43 |
| tmckay | from what I can see, it means that a cluster must already have been built | 15:43 |
| openstackgerrit | Sergey Reshetnyak proposed a change to openstack/sahara: Fix 'Clock Offset' error in Cloudera Manager https://review.openstack.org/129304 | 15:43 |
| elmiko | it also looked like one of the node groups was getting created with a floating pool id | 15:44 |
| elmiko | yea, is it possible to have leftovers from a previous test? (i wouldn't think so) | 15:44 |
| tmckay | depends where the openstack instance is, I guess | 15:44 |
| elmiko | good point | 15:44 |
| tmckay | whether the instance is reused | 15:44 |
| tmckay | I'm wondering if just a fake commit on master would have the same error | 15:45 |
| elmiko | is anything else failing currently? | 15:45 |
| tmckay | add an "if false" somewhere in the dashboard stuff and commit | 15:45 |
| tmckay | not sure where else gate-ui-tests run | 15:45 |
| tmckay | crobertsrh, ^^ any insight on this? All I was trying to do was fix a typo :) | 15:46 |
| elmiko | tmckay: makes me wonder if the typo was hiding something | 15:53 |
| tmckay | yeah, could be. | 15:54 |
| elmiko | although, looking at the change you made i'm having trouble imagining how | 15:54 |
| tmckay | well, the security group stuff is relatively recent. And there is another change on master in sahara-dashboard that touches sec groups https://review.openstack.org/#/c/128216/1/saharadashboard/tests/base.py | 15:55 |
| tmckay | maybe the two interact? I | 15:56 |
| tmckay | I'll try a fake commit and see what happens | 15:56 |
| elmiko | is that test stack even using sahara-dashboard, or the integrated one? | 15:57 |
| openstackgerrit | Trevor McKay proposed a change to openstack/sahara-dashboard: Try a fake change to see if gate ui passes -- DONT MERGE https://review.openstack.org/129308 | 15:59 |
| elmiko | what a faker... | 15:59 |
| tmckay | heh | 15:59 |
| elmiko | this is one of those situations where it would be cool to have access to another test setup | 16:00 |
| tmckay | yeah. I'm baffled. | 16:00 |
| elmiko | it totally looks like maybe the keystone db isn't reset and the selenium sec group is still there. but i would think in that case that we should just accept that it's there and move on. | 16:01 |
| tmckay | well, if you reused the same security group then you're going to be mixing stuff between clusters potentially, right? That seems bad | 16:02 |
| tmckay | we could fix the autocreate to add a random uuid | 16:02 |
| tmckay | it's named for cluster and node group | 16:02 |
| tmckay | seems like if you reuse a cluster and node group name, you get a fail | 16:02 |
| *** chandankumar has joined #openstack-sahara | 16:02 | |
| elmiko | yea, could be an issue if you had meant to have 2 different groups | 16:02 |
| tmckay | hmm, that should be testable | 16:03 |
| tmckay | create a cluster with auto sec group, delete cluster, create it again. Maybe it's supposed to get deleted on cluster removal, but doesn't in some cases? | 16:03 |
| elmiko | could be | 16:11 |
| openstackgerrit | Trevor McKay proposed a change to openstack/sahara-dashboard: Try a fake change to see if gate ui passes -- DONT MERGE https://review.openstack.org/129308 | 16:21 |
| *** tellesnobrega has joined #openstack-sahara | 16:25 | |
| tmckay | elmiko, you know what I think the problem is? | 16:28 |
| tmckay | (rhetorical) | 16:28 |
| crobertsrh | Any progress on that problem? I just got back from lunch. | 16:28 |
| tmckay | auto_security_groups (default true) was added in https://review.openstack.org/#/c/128216/1. The change has selenium set the option on screen accordingly | 16:29 |
| tmckay | that didn't used to be there | 16:29 |
| tmckay | It didn't used to do that. crobertsrh, is the auto_security_groups checkbox in horizon on or off by default? | 16:30 |
| crobertsrh | I think off, but I'll double check now | 16:30 |
| tmckay | if it's off, and the ci config does not have auto_security_groups = False, then we have changed the test | 16:30 |
| tmckay | and possibly uncovered a problem with ci openstack security group memory :) | 16:31 |
| crobertsrh | Looks like it's unchecked by default | 16:31 |
| tmckay | okay, I have the ci git checked out, let me check the config .... | 16:32 |
| tmckay | hmm, no auto_security_group setting in slave-scripts/gate-ui-tests.sh, which means we've changed the behavior by defaulting to true | 16:34 |
| tmckay | I think | 16:34 |
| *** tellesnobrega has quit IRC | 16:36 | |
| *** tellesnobrega has joined #openstack-sahara | 16:39 | |
| *** chandankumar has quit IRC | 16:39 | |
| openstackgerrit | Trevor McKay proposed a change to openstack/sahara-dashboard: Try a fake change to see if gate ui passes -- DONT MERGE https://review.openstack.org/129308 | 16:43 |
| *** Longgeek has quit IRC | 16:44 | |
| *** chandankumar has joined #openstack-sahara | 16:54 | |
| elmiko | tmckay: nice find | 17:20 |
| tmckay | elmiko, still unsure. The trick may actually be to leave auto sec = True and add a uuid to the cluster name create | 17:20 |
| tmckay | or at the very least a pid | 17:20 |
| tmckay | pid is probably sufficient, at least to see if it gets over the hump | 17:21 |
| elmiko | i think you're correct about investigating the naming uniqueness, wonder if we can make a test to exercise this? | 17:21 |
| tmckay | doing a recheck now, cause I turned off auto sec but got weird selenium errors (and I don't know much (anything) about selenium) | 17:21 |
| tmckay | elmiko, what kind of test? | 17:22 |
| elmiko | tmckay: like something to auto-generate a sec group, then another something to try and create the same group. i dunno, i'd probably have to look closer at the code. | 17:24 |
| tmckay | elmiko, looks like the sec group is supposed to be deleted when the cluster is deleted. But, if something went wrong ... seems like lots of ways it could be left behind | 17:31 |
| *** tellesnobrega has quit IRC | 17:34 | |
| *** tellesnobrega has joined #openstack-sahara | 17:39 | |
| elmiko | tmckay: interesting, seems like some room for more testing | 17:42 |
| openstackgerrit | Jonathan Halterman proposed a change to openstack/sahara: Fix command line arguments for image-register and image-add-tag commands in quickstart guide https://review.openstack.org/129339 | 17:43 |
| openstackgerrit | Trevor McKay proposed a change to openstack/sahara-dashboard: Try a fake change to see if gate ui passes -- DONT MERGE https://review.openstack.org/129308 | 17:56 |
| openstackgerrit | Trevor McKay proposed a change to openstack/sahara-dashboard: Try a fake change to see if gate ui passes -- DONT MERGE https://review.openstack.org/129308 | 18:41 |
| openstackgerrit | Jonathan Halterman proposed a change to openstack/sahara: Fix quickstart guide https://review.openstack.org/129339 | 18:50 |
| *** stannie has quit IRC | 18:59 | |
| *** tellesnobrega has quit IRC | 19:05 | |
| *** hogepodge has quit IRC | 19:10 | |
| tmckay | crobertsrh, ping | 19:14 |
| crobertsrh | yes? | 19:14 |
| tmckay | I need your expertise, I'm wondering out of my experience | 19:14 |
| crobertsrh | uh oh | 19:14 |
| tmckay | https://review.openstack.org/#/c/128153/1/openstack_dashboard/dashboards/project/data_processing/nodegroup_templates/templates/data_processing.nodegroup_templates/_fields_help.html | 19:14 |
| tmckay | see that change, made on 10/13 (just a few days ago) | 19:15 |
| tmckay | "Show full configuration" changed from a link type to a button type, it looks like to me | 19:15 |
| crobertsrh | yep | 19:15 |
| crobertsrh | You are correct | 19:15 |
| tmckay | crobertsrh, my fake ui test change is failing because it can't find "Show full configuration" | 19:15 |
| tmckay | but it's looking for a link | 19:15 |
| tmckay | not a button | 19:15 |
| tmckay | (theory) | 19:15 |
| crobertsrh | That seems like it should fail | 19:16 |
| crobertsrh | button certainly != link | 19:16 |
| tmckay | NoSuchElementException: Message: u'Unable to locate element: {"method":"link text","selector":"Show full configuration"}' ; Stacktrace: | 19:16 |
| crobertsrh | probably need a new selector in the test | 19:16 |
| tmckay | so I'm assuming this is supposed to be button text, or some such | 19:16 |
| *** tosky has quit IRC | 19:17 | |
| crobertsrh | do you know where the source for the test is off hand? | 19:17 |
| tmckay | It's in saharadashboard/tests/base.py, config_helper() where the show_param stuff is | 19:17 |
| tmckay | I wandered down this road trying to figure out why my simple typo fix didn't work | 19:18 |
| tmckay | one reason is apparently related to sec groups | 19:18 |
| crobertsrh | heh...fun for a Friday | 19:18 |
| tmckay | but fix that, and boom, this stuff | 19:18 |
| tmckay | basically, gate-ui-tests is horked currently. | 19:18 |
| tmckay | I think we have multiple overlapping errors | 19:18 |
| tmckay | crobertsrh, if we come up with fixes, I can try them out on my fake_change until we have a consistent CR | 19:20 |
| crobertsrh | I think button_text should work | 19:20 |
| tmckay | by.BY.BUTTON_TEXT you mean? | 19:21 |
| tmckay | I'll try it | 19:21 |
| crobertsrh | I'll check the docs | 19:21 |
| crobertsrh | hmm, that might not quite work | 19:22 |
| crobertsrh | ok....tired of vi already....firing up pycharm | 19:23 |
| elmiko | are you just starting the path of vi? | 19:24 |
| crobertsrh | No, I've used it for years.....sparingly | 19:24 |
| crobertsrh | I'm a spoiled IDE boy | 19:24 |
| elmiko | totally | 19:24 |
| elmiko | pycharm is really nice too | 19:25 |
| crobertsrh | Ok, looks like you can use ID, XPATH, LINK_TEXT, NAME, TAG_NAME, CLASS_NAME or CSS_SELECTOR | 19:25 |
| crobertsrh | Yeah, I'm loving pycharm | 19:25 |
| crobertsrh | tmckay: it looks like XPATH could be a winner | 19:26 |
| crobertsrh | tmckay: a few lines below the LINK_TEXT attempt there is an XPATH example | 19:26 |
| tmckay | yeah, I'm hanging here. I can't find anything that says "link_text" doesn't work for buttons, but it doesn't look good. http://selenium-python.readthedocs.org/en/latest/locating-elements.html | 19:26 |
| crobertsrh | you can use the "full-config-show/hide" classes | 19:27 |
| tmckay | crobertsrh, is that thing an input? | 19:28 |
| crobertsrh | or even By.XPATH, '//button[text()="Show full configuration"]' might also work | 19:28 |
| crobertsrh | is what thing an input? | 19:28 |
| tmckay | the button "Show full configuration" | 19:29 |
| crobertsrh | Not strictly speaking. a button is just a button | 19:29 |
| tmckay | from the above page, example: clear_button = driver.find_element_by_xpath("//input[@name='continue'][@type='button']") | 19:29 |
| tmckay | driver.find_element(By.XPATH, '//button[text()="Some text"]') | 19:30 |
| tmckay | They also have that | 19:30 |
| crobertsrh | self.waiting_element_in_visible_state( | 19:31 |
| crobertsrh | by.By.XPATH, //button[text()="Show full configuration"]) | 19:31 |
| crobertsrh | That should work | 19:31 |
| tmckay | yeah, that jives with the above | 19:31 |
| tmckay | k. The only way I have to test this is in the CI :) | 19:31 |
| crobertsrh | Are you familiar with XPATH stuff? | 19:31 |
| tmckay | nope | 19:31 |
| crobertsrh | ah | 19:31 |
| tmckay | making it up as I go along | 19:31 |
| tmckay | so far I reckoned it right, though, I think, as far as the error :) | 19:32 |
| crobertsrh | yeah...the "//" is shorthand for "whatever", then "give me the button that has the text I ask for" | 19:32 |
| tmckay | cool, I'll try it | 19:32 |
| crobertsrh | You could give a full xpath (by not using the // shortcut) that would start at the top of the DOM tree and hit every div on the way down...it would be ugly. | 19:33 |
| tmckay | this is one of the problems with horizon integration | 19:33 |
| tmckay | someone changed this over in horizon, and the tests broke | 19:33 |
| *** bradd1 has quit IRC | 19:33 | |
| crobertsrh | Yes...there are some tests being built in to the horizon side of things now for data_processing | 19:33 |
| crobertsrh | That is a problem...we will never really know when this situation is going to happen again. | 19:34 |
| crobertsrh | It's not always "sahara people" making changes over there | 19:34 |
| openstackgerrit | Trevor McKay proposed a change to openstack/sahara-dashboard: Try a fake change to see if gate ui passes -- DONT MERGE https://review.openstack.org/129308 | 19:40 |
| tmckay | alright, we'll see if it gets a little further | 19:41 |
| tmckay | at the very least, I can make some bugs for this and someone better at this stuff can fix it | 19:41 |
| crobertsrh | Yeah, you're at least finding the problems :) | 19:42 |
| crobertsrh | I had totally forgot that that repo was even in play for anything | 19:42 |
| tmckay | I should have left the typo I found. Thought it would be easy pickins | 19:43 |
| elmiko | at the least it needs to be around until Icehouse goes away | 19:43 |
| elmiko | the repo not the typo ;) | 19:43 |
| tmckay | heh | 19:43 |
| tmckay | I could do all this locally, but the thought of installing selenium and setting up an environment to run local tests is daunting | 19:44 |
| elmiko | agreed | 19:44 |
| crobertsrh | Yeah, no need to bother with that. Hopefully, this change is a small one and that is all you need. | 19:44 |
| elmiko | you're already hip deep in unexpected quicksand | 19:44 |
| tmckay | true :) I didn't fix all the errors, just enough so it should get a little further if it's legit | 19:45 |
| tmckay | I think it will break on the "Hide" now | 19:45 |
| openstackgerrit | Trevor McKay proposed a change to openstack/sahara-dashboard: Try a fake change to see if gate ui passes -- DONT MERGE https://review.openstack.org/129308 | 20:04 |
| tmckay | doh, left off a quote | 20:04 |
| tmckay | one more time | 20:04 |
| *** witlessb has quit IRC | 20:10 | |
| crobertsrh | heh | 20:11 |
| *** witlessb has joined #openstack-sahara | 20:15 | |
| *** chandankumar has quit IRC | 20:16 | |
| *** crobertsrh is now known as _crobertsrh | 20:28 | |
| -openstackstatus- NOTICE: Gerrit will be offline from 2100-2130 for project renames | 20:34 | |
| *** ChanServ changes topic to "Gerrit will be offline from 2100-2130 for project renames" | 20:34 | |
| *** bradd1 has joined #openstack-sahara | 20:45 | |
| *** miqui has quit IRC | 20:50 | |
| -openstackstatus- NOTICE: Gerrit is offline from 2100-2130 for project renames | 21:03 | |
| *** ChanServ changes topic to "Gerrit is offline from 2100-2130 for project renames" | 21:03 | |
| *** hogepodge has joined #openstack-sahara | 21:22 | |
| -openstackstatus- NOTICE: Gerrit is back online | 21:26 | |
| *** ChanServ changes topic to "launchpad openid is down. login to openstack services will fail until launchpad openid is happy again" | 21:26 | |
| *** tmckay has left #openstack-sahara | 21:27 | |
| *** openstackgerrit has quit IRC | 22:03 | |
| *** openstackgerrit has joined #openstack-sahara | 22:04 | |
| *** tellesnobrega has joined #openstack-sahara | 22:14 | |
| *** tellesnobrega has quit IRC | 23:11 | |
| *** tellesnobrega has joined #openstack-sahara | 23:17 | |
| *** bradd1 has quit IRC | 23:54 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!