| *** angdraug has quit IRC | 02:32 | |
| *** dshulyak has joined #openstack-solar | 07:03 | |
| YorikSar_ | dshulyak: Openstack clients use Cliff for CLI. It's basically a wrapper around argparse. | 07:06 |
|---|---|---|
| *** YorikSar_ is now known as YorikSar | 07:06 | |
| dshulyak | YorikSar: but how they solved problem of nested sys.argv parsers? i am talking about parsing oslo.config cli opts (CONF(sys.argv)) at the same time | 07:10 |
| YorikSar | dshulyak: Oh, they most likely didn't :) | 07:11 |
| YorikSar | dshulyak: They don't seem to use oslo.config, at least not directly. | 07:11 |
| YorikSar | dshulyak: They call out to https://github.com/openstack/os-client-config/blob/master/os_client_config/config.py to get cloud config | 07:13 |
| dshulyak | thanks, i see that they are just using env variable for specifying additional config path, it will be enough | 07:18 |
| *** salmon_ has joined #openstack-solar | 08:31 | |
| pigmej | dshulyak: having ENV config is kinda required I think, because otherwise we could struggle a bit. | 09:13 |
| pigmej | salmon_: bash utils/jenkins/run_hosts_example.sh 2.78s user 0.31s system 3% cpu 1:22.17 total | 09:14 |
| pigmej | but around 10seconds is for auth popups | 09:15 |
| salmon_ | pigmej: now with a patch? :) | 09:15 |
| pigmej | salmon_: it's new image + overlay | 09:15 |
| dshulyak | pigmej: so you think it will be usefull to overwrite any config property with env? | 09:15 |
| salmon_ | nice | 09:15 |
| salmon_ | pigmej: now it makes sense to remove sleep 30 :) | 09:15 |
| pigmej | dshulyak: maybe not any, but I would expect to allow overwrite source file at least | 09:16 |
| pigmej | https://bpaste.net/show/75e6a8c48fec | 09:16 |
| pigmej | salmon_: ^ | 09:16 |
| salmon_ | docker... | 09:16 |
| pigmej | dshulyak: I would expect at least possibitity of set "override" file | 09:16 |
| pigmej | salmon_: dshulyak please review https://review.openstack.org/#/c/275325/ :) | 09:17 |
| pigmej | salmon_: I got this error 3 times in a row now | 09:18 |
| salmon_ | pigmej: is it fatal error? | 09:18 |
| pigmej | yup | 09:20 |
| pigmej | https://bpaste.net/show/e619c8e587cd | 09:20 |
| salmon_ | pigmej: https://github.com/docker/docker/issues/14461 :) | 09:23 |
| pigmej | but why it happens only for me ? | 09:24 |
| pigmej | salmon_: I have more fs related errors | 09:26 |
| pigmej | https://bpaste.net/show/8748dc3e412f | 09:27 |
| pigmej | this is non fatal though | 09:27 |
| salmon_ | this may be ok | 09:27 |
| pigmej | how so ? | 09:28 |
| salmon_ | I will check it :) | 09:29 |
| pigmej | ok, I cleared all image related stuff | 09:35 |
| pigmej | and it seems to be ok now. | 09:35 |
| pigmej | salmon_: is the overlay somehow "cached" ? | 09:35 |
| pigmej | or maybe reused ? | 09:36 |
| salmon_ | pigmej: I use the same names, but overlay image should be removed before new run | 09:36 |
| salmon_ | there may be bug somwhere there | 09:36 |
| pigmej | because, when I removed them by hand, everything seems to be fine | 09:37 |
| pigmej | no error from docker | 09:37 |
| pigmej | yeah, salmon, now it's ok | 09:41 |
| pigmej | :) | 09:41 |
| salmon_ | strange | 09:44 |
| pigmej | yup | 09:44 |
| pigmej | but now works ;) | 09:44 |
| *** tzn has joined #openstack-solar | 09:46 | |
| pigmej | dshulyak: https://review.openstack.org/#/c/275062/ check it please :) | 10:02 |
| openstackgerrit | Merged openstack/solar: Use overlay images, it speedups tests https://review.openstack.org/275062 | 10:16 |
| salmon_ | dshulyak: http://docs-draft.openstack.org/42/274642/3/check/gate-solar-docs/cb09027//doc/build/html/orchestration/entities.html | 10:25 |
| salmon_ | Executor Executor | 10:25 |
| salmon_ | Client Client | 10:25 |
| pigmej | ah it's format issue :) | 10:27 |
| dshulyak | thanks, i forgot new line there | 10:28 |
| pigmej | dshulyak: you could use definition list | 10:31 |
| pigmej | http://docutils.sourceforge.net/docs/user/rst/quickref.html#definition-lists | 10:31 |
| salmon_ | dshulyak: also http://docs-draft.openstack.org/42/274642/3/check/gate-solar-docs/cb09027//doc/build/html/orchestration/configuration.html "see *_driver options in config." | 10:32 |
| salmon_ | formating | 10:32 |
| pigmej | brb | 10:33 |
| openstackgerrit | Dmitry Shulyak proposed openstack/solar: Use oslo.config instead of custom parser https://review.openstack.org/275241 | 11:01 |
| pigmej | back | 11:04 |
| dshulyak | who knows what can be an issue http://docs-draft.openstack.org/42/274642/3/check/gate-solar-docs/cb09027//doc/build/html/orchestration/configuration.html#worker-driver-namespaces ? | 11:14 |
| dshulyak | indentation is identical to http://docs-draft.openstack.org/42/274642/3/check/gate-solar-docs/cb09027//doc/build/html/orchestration/configuration.html#constructor-hooks-namespaces | 11:14 |
| dshulyak | but for some reason new line is not preserved in 1st | 11:15 |
| pigmej | I don't get it | 11:15 |
| pigmej | what is the problem with new line ? | 11:16 |
| pigmej | btw dshulyak I think you should use definition list instead of this .. note: based stuff :) | 11:17 |
| dshulyak | idk, in this case it looks bad | 11:22 |
| pigmej | salmon_: does new ansible stuff work for you ? | 11:23 |
| openstackgerrit | Dmitry Shulyak proposed openstack/solar: Introduce documentation related to new worker https://review.openstack.org/274642 | 11:29 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Fix response handling in handlers/base.py https://review.openstack.org/275646 | 11:53 |
| pigmej | ^ "hotfix" | 11:53 |
| -openstackstatus- NOTICE: Infra running with lower capacity now, due to a temporary problem affecting one of our nodepool providers. Please expect some delays in your jobs. Apologies for any inconvenience caused. | 12:41 | |
| *** openstackgerrit has quit IRC | 13:02 | |
| *** openstackgerrit has joined #openstack-solar | 13:03 | |
| openstackgerrit | Merged openstack/solar: Introduce documentation related to new worker https://review.openstack.org/274642 | 13:09 |
| openstackgerrit | Merged openstack/solar: Fix response handling in handlers/base.py https://review.openstack.org/275646 | 13:15 |
| dshulyak | pigmej: do you know something about this error RiakError: '{error,locked}' ? | 13:24 |
| dshulyak | i started to see it on my local environment | 13:25 |
| pigmej | hmm, bitcask may sometimes return similar error | 13:27 |
| pigmej | but I don't remember it exactly | 13:27 |
| pigmej | is it on bitcask? | 13:27 |
| pigmej | but hmm, just {error, locked} sounds weird | 13:27 |
| salmon_ | any one interested https://bugs.launchpad.net/solar/+bug/1541394 ? | 13:28 |
| openstack | Launchpad bug 1541394 in Solar "Solar introduction documentation is missing" [Medium,Confirmed] | 13:28 |
| pigmej | salmon_: no, but if you mentioned it you should tackle it :) | 13:29 |
| pigmej | I believe in you! | 13:29 |
| salmon_ | :P | 13:29 |
| openstackgerrit | Dmitry Shulyak proposed openstack/solar: Use oslo.config instead of custom parser https://review.openstack.org/275241 | 14:33 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Torrent transport can use corret solar_torrent.py https://review.openstack.org/275745 | 15:11 |
| pigmej | salmon_: dshulyak ^ | 15:11 |
| pigmej | and this is unrelated fix: https://github.com/Mirantis/solar-resources/pull/15 but still :) | 15:12 |
| pigmej | dshulyak: https://review.openstack.org/#/c/275241/ this should have also documentation adjustments | 15:41 |
| pigmej | https://review.openstack.org/#/c/275325/ blocks 2 bugs | 15:54 |
| openstackgerrit | Lukasz Oles proposed openstack/solar: Ip addressess fixes https://review.openstack.org/275834 | 17:21 |
| openstackgerrit | Lukasz Oles proposed openstack/solar: Create tmp dir for overlay images https://review.openstack.org/275851 | 17:27 |
| pigmej | oh, why tmp is needed ? | 17:30 |
| salmon_ | pigmej: https://github.com/openstack/solar/blob/master/utils/jenkins/env.py#L12 | 17:31 |
| salmon_ | I don't want images to be in a root dir | 17:31 |
| pigmej | ah | 17:31 |
| salmon_ | we can test change before merge https://jenkins-sandbox.infra.mirantis.net/job/solar_custom_example_test/25/console ! | 17:33 |
| salmon_ | it's private but will be public after tests | 17:33 |
| pigmej | :) | 17:34 |
| pigmej | cool | 17:34 |
| salmon_ | just pass refs/changes/34/275834/1 and it will test it | 17:35 |
| pigmej | :) | 17:36 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Use extra-vars in all ansible handlers https://review.openstack.org/275866 | 17:54 |
| pigmej | job for me for tomorrow/friday in train, => unified iface for ansible handlers | 17:56 |
| pigmej | cu! | 18:02 |
| salmon_ | cu | 18:03 |
| pigmej | salmon_: jenkins is on vaccations, so... have a nice evening :) | 18:04 |
| salmon_ | u2 :) | 18:05 |
| openstackgerrit | Lukasz Oles proposed openstack/solar: Do not interfere with vagrant network https://review.openstack.org/275886 | 18:52 |
| openstackgerrit | Dmitry Shulyak proposed openstack/solar: Use oslo.config instead of custom parser https://review.openstack.org/275241 | 19:32 |
| *** dshulyak has quit IRC | 21:03 | |
| openstackgerrit | Merged openstack/solar: Create tmp dir for overlay images https://review.openstack.org/275851 | 21:34 |
| openstackgerrit | Lukasz Oles proposed openstack/solar: Do not interfere with vagrant network https://review.openstack.org/275886 | 21:41 |
| *** tzn has quit IRC | 22:30 | |
| *** dshulyak has joined #openstack-solar | 22:31 | |
| salmon_ | pigmej: we need to talk tomorrow with Dima about what should be included in next release :) | 22:31 |
| *** dshulyak has quit IRC | 22:31 | |
| pigmej | yup | 22:31 |
| pigmej | salmon_: just when you said it dima left :P | 22:31 |
| pigmej | but for sure we need to move it :) | 22:32 |
| salmon_ | :D | 22:32 |
| salmon_ | yeah | 22:32 |
| salmon_ | current date was just a guess | 22:32 |
| pigmej | yup, and it was not a bad guess, we could still make it on this date :) | 22:32 |
| salmon_ | well, we could. If we focussed only on docs | 22:34 |
| pigmej | yup | 22:34 |
| pigmej | this one https://bugs.launchpad.net/solar/+bug/1537736 makes me cry | 22:34 |
| openstack | Launchpad bug 1537736 in Solar "Distinguish Resource and Resource Definition " [Undecided,New] | 22:34 |
| salmon_ | :( | 22:38 |
| pigmej | it's kinda... wtf | 22:38 |
| pigmej | I would put info in documentation that we use Resource as different things | 22:39 |
| pigmej | I tried to separate them once and.. well, I get lost :D | 22:39 |
| salmon_ | hmm, we at lest need it in documentation | 22:44 |
| pigmej | yeah, we can put big red box | 22:45 |
| pigmej | "when we use resource name we could mean:. ... " (not exactly like this but you get the point probably) | 22:46 |
| salmon_ | yup | 22:52 |
| pigmej | and we need more described simple examples | 22:55 |
| pigmej | maybe step - by - step riak makes sense (especially that we already have riak resources in solar-resoruces), so repo creation steps would be not required | 22:56 |
| salmon_ | yup | 22:59 |
| *** angdraug has joined #openstack-solar | 23:40 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!