Tuesday, 2014-07-29

« Monday, 2014-07-28 Index Wednesday, 2014-07-30 »
openstackgerritSteve McLellan proposed a change to stackforge/murano-dashboard: Add keypair '+' button  https://review.openstack.org/11017200:39
*** bharath has joined #murano00:41
*** sjmc7 has quit IRC00:44
openstackgerritGeorgy Okrokvertskhov proposed a change to stackforge/murano: Fix the concurrency issue agent queue creation and VM agent  https://review.openstack.org/11003001:37
openstackgerritRuslan Kamaldinov proposed a change to stackforge/murano-apps: don't look at me  https://review.openstack.org/11018201:53
openstackgerritRuslan Kamaldinov proposed a change to stackforge/murano: Added Murano config checks  https://review.openstack.org/10172301:55
*** bharath has quit IRC02:31
*** chandankumar has joined #murano04:15
*** murali has joined #murano04:42
*** murali has quit IRC05:47
*** chandankumar has quit IRC06:35
katyaferventbharath hi!07:47
katyaferventthe app name is taken from UI, but we have a blueprint that suppose to change that behaviour https://blueprints.launchpad.net/murano/+spec/dynamic-ui-specify-no-explicit-name-field07:53
*** killer_prince is now known as lazy_prince08:19
*** lazy_prince is now known as killer_prince08:22
openstackgerritA change was merged to stackforge/murano: Fix the issue with multiple agent call  https://review.openstack.org/11015608:31
openstackgerritEkaterina Chernova proposed a change to stackforge/murano-deployment: Update due to test reorganization  https://review.openstack.org/10999008:32
*** sergmelikyan has quit IRC08:39
*** sergmelikyan has joined #murano09:02
openstackgerritRuslan Kamaldinov proposed a change to stackforge/murano: Two fixes to API unit tests  https://review.openstack.org/10915109:26
openstackgerritRuslan Kamaldinov proposed a change to stackforge/murano: [WIP] Default is_public to false  https://review.openstack.org/11024209:26
*** stan_lagun has joined #murano10:42
*** slagun has quit IRC10:45
openstackgerritRuslan Kamaldinov proposed a change to stackforge/murano: [WIP] Default is_public to false  https://review.openstack.org/11024211:25
*** IgorYozhikov is now known as iyozhikov11:41
*** lecalcot has joined #murano12:07
*** lecalcot has quit IRC12:36
*** drupalmonkey has joined #murano12:50
*** lecalcot has joined #murano12:53
*** lecalcot has quit IRC13:25
*** lecalcot has joined #murano13:44
*** lecalcot has quit IRC13:45
*** lecalcot has joined #murano13:45
*** chandankumar has joined #murano14:11
openstackgerritGeorgy Okrokvertskhov proposed a change to stackforge/murano: Fix the concurrency issue agent queue creation and VM agent  https://review.openstack.org/11003014:18
*** sjmc7 has joined #murano14:20
openstackgerritRuslan Kamaldinov proposed a change to stackforge/murano: [WIP] Default is_public to false  https://review.openstack.org/11024214:28
openstackgerritSergey Reshetnyak proposed a change to stackforge/murano: Use with_variant method for dialects db types  https://review.openstack.org/11030514:29
openstackgerritEkaterina Chernova proposed a change to stackforge/murano-dashboard: Provide horizon tag during settings update  https://review.openstack.org/11030614:32
*** Murali has joined #murano14:33
openstackgerritSteve McLellan proposed a change to stackforge/murano: Fixes to unit tests around policies  https://review.openstack.org/10915114:36
*** chandankumar has quit IRC14:43
Muralisjmc7:  after applying patch set 2 and patch set 3, i notice some of the pop up windows --> Mark Image , Upload package windows are not loading properly14:47
Muraliis this anything to woth code change ?14:47
sjmc7which patchset 2 and 3?14:47
Muralihttps://review.openstack.org/#/c/110125/14:48
Muralihttps://review.openstack.org/10915114:48
Muralimorning i applied this and restarted the murano service.14:49
sjmc7the second one should only affect deployment. i don't think the first one should affect the UI but i'll check now14:51
sjmc7when you say they are 'not loading properly', do you get an error?14:51
Muralino i m not getting any error. then screen goes blank that it14:51
Muralican u pass me the ur l to paste logs14:52
*** sjmc7_ has joined #murano14:54
*** tsufiev has quit IRC14:55
*** sjmc7 has quit IRC14:58
*** chandankumar has joined #murano14:58
*** tsufiev_ has joined #murano15:01
openstackgerritEkaterina Chernova proposed a change to stackforge/murano: Move and rename functional tests  https://review.openstack.org/10998715:03
openstackgerritGeorgy Okrokvertskhov proposed a change to stackforge/murano: Add caller information to Begin execute log entry  https://review.openstack.org/11031715:04
*** tsufiev_ has quit IRC15:05
Muralisjmc7_ : please take a look https://bugs.launchpad.net/murano/+bug/134989415:05
*** tsufiev_ has joined #murano15:06
sjmc7_murali - those changes are not yet merged so it should not be considered a bug15:06
sjmc7_instead you give the change that you think is causing problems a negative review15:06
sjmc7_and provide details there15:06
*** tsufiev_ has quit IRC15:07
Muraliokey15:07
sjmc7_if you can isolate it to one of the changesets, that would be very helpful15:08
*** chandankumar has quit IRC15:08
*** tsufiev has joined #murano15:09
openstackgerritEkaterina Chernova proposed a change to stackforge/murano-deployment: Update dasboard script due to test reorganization  https://review.openstack.org/10999015:11
*** iyozhikov is now known as IgorYozhikov15:13
*** IgorYozhikov is now known as iyozhikov15:13
openstackgerritEkaterina Chernova proposed a change to stackforge/murano-deployment: Update murano engine ci script  https://review.openstack.org/11032715:22
openstackgerritEkaterina Chernova proposed a change to stackforge/murano-dashboard: Move functional tests under muranodashboard/tests  https://review.openstack.org/10995415:28
ruhesjmc7_: (re setting package.is_public to False by default) do you want to modify existing migration script?15:30
sjmc7_i think just remove the migration. sorry, stuck trying to get the dashboard to get past the auth page at the moment15:31
ruhei've seen people hurt by alembic too many times (including myself)15:33
*** iyozhikov is now known as IgorYozhikov15:41
sjmc7_yeah, it seems to have wrinkles. this dashboard problem is making me tear my hair out15:41
ruhewhich one is it?15:43
sjmc7_occasionally it refuses to log in successfully15:44
sjmc7_and then, as if by magic, fixes itself half an hour later. think it's something in horizon or my settings file15:45
sjmc7_anyway, regarding the alembic migration - it should just be removed i think15:45
sjmc7_new records will behave properly15:46
sjmc7_i'll do it in a second, testing something else15:46
sjmc7_Murali, sorry for the delay. i can't reproduce the problem you're having with either of those changes15:49
openstackgerritSteve McLellan proposed a change to stackforge/murano: Default is_public to false  https://review.openstack.org/10915315:51
ruhesjmc7_: ok. got it. good luck with dashboard!15:51
sjmc7_ruhe ^15:51
sjmc7_dashboard's workign again too15:52
ruhesjmc7_: maybe you should also update http://git.openstack.org/cgit/stackforge/murano/tree/murano/db/migration/alembic_migrations/versions/001_inital_version.py#n151 ?15:53
ruhei'm not sure about this one. it has pros and cons15:53
sjmc7_what's the con?15:53
sjmc7_the default should be False anyway15:54
sjmc7_i mean, as things stand15:54
sjmc7_the initial migration doesn't set a default15:54
sjmc7_so actually the current state is inconsistent, and this patch makes everything consistent again15:54
sjmc7_without changing 00115:55
ruheit could introduce inconsistency between database schema produced by sqlalchemy metadata and by alembic migrations15:55
sjmc7_right - it doesn't, i don't think15:56
sjmc7_the alembic migration doesn't set a default15:56
sjmc7_sqla was setting it to True15:56
ruheyou convinced me :)15:56
sjmc7_so i think things are all square15:56
sjmc7_:)15:56
ruhemeeting will start in less than an hour. everyone is welcome to update the agenda https://wiki.openstack.org/wiki/Meetings/MuranoAgenda#Agenda16:11
sjmc7_tnurlygayanov - did you reproduce 134989416:13
sjmc7_?16:13
sjmc7_i tested with both those in-flight reviews, and everything worked; i can't see any reason they'd break UI pages16:14
tnurlygayanovsjmc7_  no, I just reviewed this bug and commits16:15
tnurlygayanovso, if it works on your environment could you please mark this bug as 'invalid' and add some comment?16:16
tnurlygayanovsjmc7_ screenshots in this bug looks strange, probably the reason of this in browser configuration16:17
sjmc7_yes16:17
tnurlygayanovsjmc7_ thank you!16:17
ruhefolks, it seems like i will not be able to chair the meeting today (i'll be mostly in read-only mode). i'll pass the chair to sergmelikyan16:19
ruhecore team, please review agenda (and proposed blueprints) in advance16:22
*** ruhe_ has joined #murano16:30
ruhesjmc7_: are there any other BPs you and your team (beside of keystone things and agentless deployments) would like to target for j3?16:32
sjmc7_will look16:32
sjmc7_focusing on stability, ui16:33
*** serg_melikyan has joined #murano16:39
Muralisjmc7, no problem.  I m setting up another clean environment with Murano again. Let me see whether i m able to reproduce that issue again. Meanwhile i ll keep mu fingers crossed16:41
sjmc7_ok16:41
*** ruhe_ has quit IRC16:46
*** ruhe_ has joined #murano16:50
*** ruhe_ has quit IRC16:54
*** btully has joined #murano16:56
serg_melikyanPlease, I would like to remind about our meeting in #openstack-meeting-alt that starts in a few minutes16:58
serg_melikyanagenda: https://wiki.openstack.org/wiki/Meetings/MuranoAgenda16:58
*** gokrokve has joined #murano17:00
*** sjmc7_ is now known as sjmc717:01
*** bharath has joined #murano17:04
*** ruhe_ has joined #murano17:04
bharathhi, how can we send details to scripts in Resources from muranopl?17:05
serg_melikyanbharath: what details?17:06
bharathserg_melikyan, if I have created two instances, I have to send ip address to scripts17:06
bharathserg_melikyan, I mean those instances are created  using the same package17:12
*** chandankumar has joined #murano17:41
bharathserg_melikyan, any idea?17:45
*** Murali has quit IRC17:46
serg_melikyanbharath: sure, few minutes, we have weekly meeting in #openstack-meeting-alt17:47
bharathserg_melikyan, oh okay I am not aware of that17:47
*** ruhe_ has quit IRC17:48
serg_melikyanOur team has weekly meeting in #openstack-meeting-alt that starting on 17:00 UTC17:49
serg_melikyanJoin us :)17:49
*** ruhe_ has joined #murano17:53
serg_melikyanbharath: https://github.com/murano-project/murano-app-incubator/blob/master/io.murano.apps.PostgreSql/Classes/PostgreSql.yaml#L5917:59
serg_melikyanthis is line that responsible to sending data to VM17:59
sjmc7serg_melikyan - related to the package retrieval thing, i'm seeing really high loads from murano-engine18:01
serg_melikyanThis line do: 1) Open file XXX.template as string 2) Parse as yaml 3) Replace text in opened file18:01
serg_melikyansjmc7: yeah, I will redo this part as soon as will have time. I expect that this issue will be closed in a week or so18:02
bharathserg_melikyan, I didn't get third stp18:02
bharath*step18:02
serg_melikyanbharath: function bind takes dictionary, and replace each entrance of the key by it's value18:03
bharathserg_melikyan, I think those values are sent to https://github.com/murano-project/murano-app-incubator/blob/master/io.murano.apps.PostgreSql/Resources/ConfigurePostgreSqlDatabase.template18:03
serg_melikyanbharath: yes18:03
bharathserg_melikyan, okay , here this method https://github.com/murano-project/murano-app-incubator/blob/master/io.murano.apps.PostgreSql/Resources/ConfigurePostgreSqlDatabase.template#L11 refers to  the method below right?18:04
*** ruhe_ has quit IRC18:04
serg_melikyanlet's assume that you have file that contains this string: "foo=$valueForFoo" than line.bind(dict(valueForFoo => 1)) will return "foo=1"18:04
stan_lagunfoo=$valueForFoo is not a valid YAML18:05
stan_lagunbind replaces values in datra structure, not strings in text file18:06
bharathserg_melikyan, okay, in that example how we are getting those details18:06
stan_lagunhttps://github.com/stackforge/murano/blob/master/murano/tests/dsl/test_engine_yaql_functions.py#L17518:07
serg_melikyanbharath: https://wiki.openstack.org/wiki/Murano/_UnifiedAgent#Execution_Plans this one may help too18:08
serg_melikyanbharath: https://github.com/murano-project/murano-app-incubator/blob/master/io.murano.apps.PostgreSql/Resources/ConfigurePostgreSqlDatabase.template#L5-9 this are mappings Agent will receive template where $name will be replaced by actuall value18:11
bharathserg_melikyan, that body  line "return configure('{0} {1} {2}'.format(args.database, args.username, args.password)).stdout" refers to scripts->configure right?18:13
stan_lagunyes18:13
bharathokay18:13
bharathbut we are getting those details , I mean $database value in https://github.com/murano-project/murano-app-incubator/blob/master/io.murano.apps.PostgreSql/Classes/PostgreSql.yaml#L60 ?18:20
stan_lagunit is where user input is put to execution plan18:20
bharathstan_lagun, sorry I didn't get you18:22
bharathstan_lagun, you mean user will give those details?18:22
stan_lagunThe line you pointed out is the place where function parameter ($database) is placed into execution plan that is sent to VM18:23
stan_lagunI guess $database originaly comes from user input18:23
bharathstan_lagun, but it is not there in ui.yaml18:24
stan_lagunit is in another UI.yaml18:27
stan_lagunio.murano.apps.java.HelloWorld18:27
stan_lagunbut this is a bad code style - HelloWorld assumes that database class must have createDatabase method although its contract says "any Application"18:28
bharathstan_lagun, yeah I just saw that18:29
openstackgerritEkaterina Chernova proposed a change to stackforge/murano: Move and rename functional tests  https://review.openstack.org/10998718:29
bharathcreateDatabase method is useful only when we are using helloworld package18:29
stan_lagunyes. I think the idea was to make each DBMS class to have a method that created database on that DBMS but the author forget to make common parent for DBMS applications with such method declared18:31
bharathstan_lagun, yeah18:33
bharathstan_lagun, return configure('{0} {1} {2}'.format(args.database, args.username, args.password)).stdout this line will pass those args as command line args to shell script right?18:36
stan_lagunyes18:36
bharathstan_lagun, okay18:37
*** bharath has quit IRC18:40
*** christopheraedo has quit IRC18:49
*** christopheraedo has joined #murano18:52
*** lecalcot has quit IRC19:00
*** lecalcot has joined #murano19:03
*** chandankumar has quit IRC19:24
*** murali_ has joined #murano19:28
murali_sjmc7 : https://bugs.launchpad.net/murano/+bug/134989419:28
murali_the issue was due to css changes. not because of code change19:29
murali_i updated bug with details19:29
sjmc7ok, thanks. can you add which CSS file has the issue?19:29
sjmc7and can you try clearing your browser cache?19:30
murali_yes i tried all the options, and also in other machines too19:30
murali_same problem.19:30
sjmc7is your murano-dashboard up to date?19:30
sjmc7and are you using horizon trunk, or the icehouse release?19:30
murali_yes i clone from here https://github.com/stackforge/murano-dashboard/tree/master/muranodashboard19:31
murali_but not applied any patch19:31
murali_icehouse release19:31
sjmc7are you able to update to master to test if that's it?19:31
sjmc7we moved to master a while back so something may have broken against icehouse19:32
murali_okey. i didnt update to master. i just clone it thats it19:33
sjmc7you must have horizon installed to run the dashboard19:33
murali_yes i have horizon installed19:34
murali_i m running murano dashboard on openstack19:35
sjmc7yes. murano-dashboard relies on functiontliy in horizon, so if you are running the icehouse release of horizon, it is possible you're seeing problems that we are not19:36
sjmc7if you are able to install horizon from the current master, it would rule that out19:36
murali_ok i see.. can you give me the url please19:39
murali_i want to make sure19:39
sjmc7http://tarballs.openstack.org/horizon/horizon-master.tar.gz will get you the nightly build19:40
murali_ok19:40
*** openstackgerrit has quit IRC19:48
murali_sjmc7 : is there any instruction to install horizon from current master19:49
sjmc7pip install horizon19:49
murali_oh thts it19:49
sjmc7or pip install http://tarballs.openstack.org/horizon/horizon-master.tar.gz19:49
murali_ok19:49
sjmc7you should use virtualenv if you're not able to install globally19:49
murali_okey19:50
*** lecalcot has quit IRC20:03
*** lecalcot has joined #murano20:13
murali_sjmc7 : i have updated the horizon20:34
murali_still seeing the same issue20:35
sjmc7ok. can you add whichy css file is causing the problem? someone will try and verify it20:35
murali_yes f7f9b38adf90.css line 667820:37
murali_updated the bug20:38
*** asalkeld has joined #murano20:54
ankurrrstan_lagun: I had a question regarding contracts of arrays of MuranoPL objects, when using namespaces. I've noticed the following work for contracts:   "[]" ,  "[$.class(MyClass).notNull()]", but I get runtime exceptions if I try to do either of the following: "[$.class(io.murano.resources.Instance).notNull()]" or "[$.class(res:Instance).notNull()]" with a namespace declaration of "res : io.murano.resources".20:55
stan_lagunThats because io.murano.... is expression (access of property 'murano' on string object 'io'(20:56
stan_lagunput quoutes araound class name20:56
stan_lagunsingle quotes20:56
ankurrrstan_lagun: ah ha!  I was thinkng quotes might be needed.  Ok, thanks.20:56
stan_lagunregarding the second case the problem is with ':'20:57
stan_lagunif you use YAML syntax instead of JSON it will work20:57
stan_lagun- instead of []20:57
stan_lagunthis is PyYAML issue20:57
ankurrrstan_lagun: so, what would that look like, if I wanted to use JSON?20:58
*** openstackgerrit has joined #murano20:58
stan_laguneither "- $.class(res:Name)" or "[!yaql "$.class(res:Name)"]"20:58
stan_lagunotherwise in JSON-compatible mode PyYAML considers ":" to be a key-value dictionary separator20:59
ankurrrstan_lagun: ok21:01
sjmc7stan_lagun - on a related note, is there a way to force it NOT to interpret as yaql? e.g. a key name with a dash gets interpreted as a() - b()21:06
sjmc7single quote enough?21:07
stan_lagunyes21:07
stan_lagunoe even more explicitly !!str "something"21:07
stan_lagunsingle or double21:07
sjmc7great, thanks21:08
*** lecalcot has quit IRC21:08
*** lecalcot has joined #murano21:08
*** drupalmonkey has quit IRC21:51
*** murali_ has quit IRC22:15
*** btully has quit IRC22:29
*** lecalcot has quit IRC23:29
« Monday, 2014-07-28 Index Wednesday, 2014-07-30 »

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