Thursday, 2014-10-23

openstackgerritZhongyue Luo proposed a change to openstack/sahara: Fix dict iteration usage  https://review.openstack.org/12982400:31
*** crobertsrh is now known as _crobertsrh01:11
*** Longgeek has joined #openstack-sahara01:28
*** Longgeek has quit IRC01:33
*** witlessb has joined #openstack-sahara01:56
*** tellesnobrega has quit IRC02:00
*** witlessb has quit IRC02:01
*** Longgeek has joined #openstack-sahara03:00
*** tellesnobrega has joined #openstack-sahara03:02
*** Longgeek has quit IRC03:04
*** Networkn3rd has joined #openstack-sahara03:41
*** Longgeek has joined #openstack-sahara04:30
*** Longgeek has quit IRC04:35
*** Longgeek has joined #openstack-sahara05:46
*** Networkn3rd has quit IRC05:49
*** Longgeek has quit IRC05:51
openstackgerritZhongyue Luo proposed a change to openstack/sahara: Fix dict iteration usage  https://review.openstack.org/12982405:53
*** openstack has joined #openstack-sahara06:12
*** openstackstatus has quit IRC06:12
*** clds_ has joined #openstack-sahara06:15
*** clds has quit IRC06:15
*** Longgeek has joined #openstack-sahara06:28
*** tellesnobrega has quit IRC06:46
*** tellesnobrega has joined #openstack-sahara06:47
*** coolsvap has quit IRC06:53
*** openstack has quit IRC06:59
*** openstack has joined #openstack-sahara14:17
*** openstackstatus has joined #openstack-sahara14:18
*** ChanServ sets mode: +v openstackstatus14:18
openstackgerritDenis Egorenko proposed a change to stackforge/sahara-ci-config: Add distributed Sahara to CI  https://review.openstack.org/13055614:32
*** mattf is now known as _mattf14:42
*** openstackgerrit has quit IRC15:07
*** tmckay has quit IRC15:10
elmikocrobertsrh, did you ever get to the bottom of the errors that were coming up from the swift_configs 8!=7 stuff?15:13
crobertsrhhmm...I dont think so15:14
*** _mattf is now known as mattf15:30
crobertsrhelmiko:  refresh my memory on that.  I'm not sure I recall a swift_configs problem.  I thought I did, but I don't.15:41
crobertsrh8 != 7 is very true except for the very small values of 815:41
elmikolol15:42
elmikosec15:42
elmikohttp://paste.fedoraproject.org/144628/40790401/15:44
crobertsrhI don't recall ever seeing that one.  Are you sure it was me that was looking at it?15:45
elmikomaybe it was tmckay15:45
crobertsrhMust be15:46
*** openstack has joined #openstack-sahara16:01
*** ghenriks has quit IRC16:01
*** tmckay has joined #openstack-sahara16:02
*** ghenriks has joined #openstack-sahara16:02
*** DenisEgorenko has left #openstack-sahara16:08
tmckayanybody know how log calls work from unit tests when running tox -e py26, for example?  I want to add output to tests, but unless I run them outside of tox I can't seem to get the output16:15
*** IvanBerezovskiy has left #openstack-sahara16:16
tmckayI probably can just open a file for append and do it that way16:16
*** abonilla has joined #openstack-sahara16:20
abonillaHi - anyone related on how the Sahara images were created?16:21
abonillahttp://lists.openstack.org/pipermail/openstack-dev/2014-September/046893.html16:21
crobertsrhabonilla:  I believe that sreshetnyak took care of that.16:21
crobertsrhHe may be out for the day though.  Do you have a specific question about them?16:22
abonillaYes, it looks like.16:22
crobertsrhThey were generated using the sahara-image-elements repo16:22
abonillacrobertsrh: not really, I want to create my own but need to know how he did them16:22
*** Longgeek has quit IRC16:23
crobertsrhabonilla:  https://github.com/openstack/sahara-image-elements16:23
abonillacrobertsrh: is that documented? I see them from CentOS and Ubuntu but I use a different distro.16:23
crobertsrhThat repo has the scripts and such that are used to generate the images.16:23
crobertsrhWhich distro are you looking to use?16:23
abonillaopenSUSE/SUSE16:24
crobertsrhAh, ok.  I don't think that distro is currently supported in there, but you can probably make some changes to make it a supported distro.16:25
crobertsrhRight now, it's ubuntu/CentOS/fedora16:25
crobertsrhThe project that sahara-image-elements uses is called diskimage-builder.  People in the #tripleo channel may know more about that.16:26
abonillaOK16:27
abonillaThanks.16:28
crobertsrhYou don't necessarily have to use the diskimage builder though.  I suppose you could use anything to generate an image that has your desired packages on it and then test to see that it works with our plugins.16:28
crobertsrhGood luck!16:28
abonillacrobertsrh: we have other methods, but I need to know what is the goal od sahara-image-elements16:30
crobertsrhAh16:30
abonillaI guess I just need to look into it.16:30
crobertsrhEssentially, it installs the required packages (hadoop, java, etc)16:30
abonillaCloudera and HW have repos for SUSE and putting that there would be easy, I wonder if there is any "metadata" handler or any ala cloud-init tool that needs to be put in there for Sahara to inject its magic into the image.16:31
crobertsrhOur images do have cloud-init "magic" so they will work with openstack16:31
crobertsrhThe diskimage-create.sh script is fairly straightforward.  Hopefully, you can see all of what is done by just looking at it.16:32
crobertsrhIf not, just ping someone around here.16:32
tmckaycrobertsrh, slow going, but I have reproduced the intermittent test error touching os_region_name, and determined that it happens (at least but not limited to) when test_get_swift_configs and test_url_for_regions run in the same worker process16:35
tmckayI bet it can also happen with some cinder tests, which also touch os_region_name16:35
tmckaysomething about the successive config override and cleanup is not right, and it shows up with multiple tests in the same worker process16:35
tmckayIf I figure this out I want a gold star ;-)16:36
tmckaythis kind of thing might be at the root of some of our intermittent CI erorrs.  dueling config overrides16:36
* tmckay gets a snack16:37
crobertsrhwow, strange stuff16:39
crobertsrhYou deserve a snack :)16:39
*** mattf is now known as _mattf16:39
tmckayonly if I figure it out.  I'm on snack credit.16:44
tmckayactually, read that wrong.  They were in separate processes.  More looking16:48
*** openstackgerrit has joined #openstack-sahara16:49
*** _mattf is now known as mattf17:00
*** tosky has quit IRC17:08
tmckayMismatchError: None != 'RegionOne'17:11
tmckaysmoking gone17:11
tmckayright after a call to set_override17:11
tmckayvery intermittent17:11
elmikodo we have any examples of adding a swift data source to a java job as an argument?17:14
elmikoor do i just set the arg as "swift://container/object" ?17:14
tmckayI think the java wordcount example does that17:14
tmckayyes17:14
tmckayjava jobs don't take data sources17:14
elmikook, i'm finally getting to dig in on that swift job binaries bug17:14
elmikohow do i add an input then?17:15
tmckayIt would be nice if you could specify a data source as an arg, with the understanding that the path would be extracted.  We've talked about it, maybe kilo17:15
elmikothat's odd, the readme totally references adding an argument. but it is quite nebulous17:15
tmckayso for wordcount, it takes 2 args, input and output, which are just paths17:15
elmikowhat are the paths relative to though?17:16
tmckayswift://tmckay/input, swift://tmckay/output, or you can use hdfs:// on a long running cluster17:16
*** tellesnobrega has joined #openstack-sahara17:17
elmikook, cool.17:17
elmikoand for wordcount, first arg is input, second is output?17:17
tmckayyeah17:17
elmikohopefully this works =)17:17
tmckaythe example dir under sahara/etc should have a whole readme file that shows how to do it17:18
elmikoit does, but it's thin on some details17:18
tmckay:)17:18
elmikolol17:18
tmckaysorry17:18
tellesnobregahi, i was trying to create a node group and cluster template with a name like: cluster-0.9.2 it showed an error saying that the name is invalid. My code is kinda old, is this suppose to be like this? is it a bug?17:18
*** mattf is now known as _mattf17:18
elmikotellesnobrega: good question, i'm not sure about the validation on cluster names17:19
tmckayhmm, must be the dash or the ".", if anything17:19
tmckayyou could try "_"17:19
tmckayI'm not sure what rules it's applying17:20
tellesnobregai think its "."17:20
tellesnobregai used with dash before17:20
tellesnobregajust wanted to make sure that it is a bug17:21
tellesnobregaor it was by design17:21
tmckaytellesnobrega, not sure.  take a look at the validation routine17:29
crobertsrhI wonder if it's specifically excluding names with dots17:39
crobertsrhdoh...I needed to scroll down a bit :)17:39
crobertsrhI know that we had some rather heavy-handed validations at one point, like for job binary names17:40
tellesnobregaok, i will take a look17:41
*** openstackgerrit has quit IRC17:48
*** openstackgerrit has joined #openstack-sahara17:49
*** tosky has joined #openstack-sahara17:53
SergeyLukjanovmeeting in 5 mins17:55
SergeyLukjanovelmiko, thank you for woring with ossg team!17:57
elmikoSergeyLukjanov: glad to help =)17:57
SergeyLukjanovelmiko, I've been planning to ask you to join the secgroup for sahara17:58
elmikoSergeyLukjanov: good timing17:58
SergeyLukjanov:)17:59
*** sreshetnyak has quit IRC17:59
SergeyLukjanovjinx, buy me a sode :) lol17:59
SergeyLukjanovtmckay, ^^17:59
*** sreshetnyak has joined #openstack-sahara17:59
*** sreshetnyak has quit IRC17:59
*** sreshetnyak has joined #openstack-sahara17:59
tmckayjo18:00
elmikoLOL18:00
elmikoSergeyLukjanov: well played, first soda on me ;)18:00
SergeyLukjanov:)18:00
SergeyLukjanovtmckay, time to join meeting :)18:01
tmckayyep, typing wrong18:01
*** _mattf is now known as mattf18:13
*** tnovacik has quit IRC18:15
*** witlessb has quit IRC18:27
*** stannie has quit IRC18:29
* tmckay gets that coffee18:58
*** mattf is now known as _mattf19:02
tmckayhey folks, looking for an opinion -- should we use [UT] as a label for unit test bugs, or put them under [IT] with the integration tests?19:08
* tmckay uses [IT] for now19:09
elmikotmckay: i take it the integration tests use [IT] currently?19:10
tmckayyeah19:10
elmiko+1 to [UT] then19:10
elmikotmckay: does a java job get a workflow created for oozie?19:16
tmckayyes19:16
elmikoi'm hoping this sub-folder issue doesn't create a huge problem19:17
openstackgerritChad Roberts proposed a change to openstack/sahara: Fixing validation exception for valid security group  https://review.openstack.org/13057619:29
*** sreshetnyak has quit IRC19:30
openstackgerritTrevor McKay proposed a change to openstack/sahara: Remove explicit set of CONF.os_region_name in mapr plugin tests  https://review.openstack.org/13062819:39
tmckayaignatov, alazarev, SergeyLukjanov, please review/+2/approve https://review.openstack.org/#/c/130628/ quickly, it fixes some intermittent unit test errors in the gate19:44
alazarevtmckay, done19:45
tmckaythank you19:45
*** witlessb has joined #openstack-sahara20:21
openstackgerritChad Roberts proposed a change to openstack/sahara: Fixing validation exception for valid security group  https://review.openstack.org/13057620:29
tmckaycrobertsrh, heh, your test got hit by the os_region_name issue on patch set 220:32
crobertsrhI'll call it the tmckay bug from henceforth20:32
tmckayooo, mine passed.  SergeyLukjanov, aignatov, care to +2 and approve? https://review.openstack.org/#/c/130628/120:33
tmckayI'll push it through myself if it's not done by 5, it's messing stuff up20:33
tmckaybrb20:36
*** Networkn3rd has quit IRC20:46
*** crobertsrh is now known as _crobertsrh21:03
tellesnobregaelmiko, i took a look into the regex that matches valid_name_hostname, from what i saw, dash is allowes21:04
tellesnobregaallowed21:04
tellesnobrega. should be if im not mistaken, but i used the regex in some online regex tester and they failed for . and in another if failed for dashes and points and number which is very weird21:05
elmikotellesnobrega: sounds like a bug maybe21:06
elmikoi wonder if we can improve the unit tests surrounding that stuff?21:06
tellesnobregai think so. I need to check properly the regex21:06
tellesnobregamaybe so21:06
elmikothat would be my first approach, see if you can mock up a test that fails with the current code. then we'll know for sure21:06
tellesnobregai will try to take a look into this later today. If i can confirm the bug i will report it and work on it, and improve tests as well21:07
tellesnobregasure21:07
elmikoawesome!21:07
tellesnobregahow are the preparations for the summit?21:07
elmikohectic =)21:07
elmikoespecially now that i've been nominated to chair one of our sessions21:07
tellesnobregacool21:08
elmikoyea, i'm excited21:08
tellesnobregai wish i was going21:08
elmiko=(21:08
tellesnobregai'm back working with storm21:08
tellesnobregahopefully i will have it ready soon21:09
elmikothat's cool21:09
tellesnobregai had a little change of context, so i stopped for a while21:09
*** _mattf is now known as mattf21:09
elmikoi noticed you were quiet for a bit, glad you can dig back in =)21:09
tellesnobregai hope to have something to show for next week21:09
tellesnobregatanks21:09
tellesnobregathanks21:09
elmikonice21:09
tellesnobregai would like to have it ready for sure before summit21:10
elmikocool21:10
tellesnobregai think stream processing is a great contribution for sahara, is a new area and can be used by lots of people. we actually have some people interested already here at the university.21:10
elmikooh yea, using it with sahara?21:11
*** witlessb has quit IRC21:11
tellesnobregaonce i can get it work21:11
tellesnobregalol21:11
elmikolol, right21:11
*** witlessb has joined #openstack-sahara21:11
tellesnobregabut its looking good. storm improved a bit, so maybe i will work with no major problems.21:12
* elmiko fingers crossed21:12
tellesnobrega:)21:13
elmikothen you can start working on all the fun bits like storm/swift integration!21:13
tellesnobregasounds like a very fun bit21:14
tellesnobregabut sure, gotta make it happen21:14
elmikoyea, i don't know much about storm's storage options, so i'm just throwing it out there21:14
* tmckay _tmckay21:15
tellesnobregait can be used to storage outputs if there is some. The main idea of storm is to process many and keep few21:16
tellesnobregai have to go now, there is star wars movie session now21:18
tellesnobregai will take a look at the name problem today and get back to you with a result21:18
elmikotellesnobrega: take care, and may the force be with you ;)21:20
openstackgerritA change was merged to openstack/sahara: Remove explicit set of CONF.os_region_name in mapr plugin tests  https://review.openstack.org/13062822:07
tmckayelmiko, heh, your neutron CR got hit by the os_region_name test bug too22:26
elmikotmckay: yea, i didn't recheck because i thought you had mentioned the bug before22:39
tmckayyeah, I pushed the fix through and just re-launched a half dozen or so22:39
tmckayI think that's all of them22:39
elmikotmckay: i had actually talked with _crobertsrh about it this morning, because i thought he had mentioned it22:39
elmikotmckay: awesome, thanks22:39
tmckaynp.  I would love a really, really, stable CI.  I wonder if there are things we can make better.22:40
elmikoi noticed a few design sessions about CI, might be worth investigating?22:40
*** witlessb has quit IRC22:40
*** tosky has quit IRC23:21

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!