| *** salmon_ has joined #openstack-solar | 07:49 | |
| pigmej | hello everyone :) | 08:00 |
|---|---|---|
| *** dshulyak has joined #openstack-solar | 08:20 | |
| salmon_ | hi | 08:53 |
| pigmej | sup salmon_ ? | 08:54 |
| pigmej | just don't tell: "friday" :) | 08:54 |
| salmon_ | :D | 08:54 |
| salmon_ | dshulyak: btw. We are now official https://github.com/Mirantis/f2s | 09:14 |
| salmon_ | I added there requirements.txt | 09:14 |
| pigmej | dshulyak: I have idea how to do that "arbitrary commands" | 10:10 |
| pigmej | since we have "repository" we could easily make it parsed on repository creation | 10:10 |
| pigmej | and repository would create files with arbitrary commands | 10:10 |
| pigmej | "shortcut" | 10:10 |
| pigmej | :) | 10:10 |
| pigmej | OR we can make our handlers aware of another keyword instead, so instead of "run" you would need to make "run_cmd" | 10:11 |
| dshulyak | can we add shell handler? | 10:11 |
| dshulyak | this handler wont look for any files | 10:11 |
| dshulyak | ssh.run(action) | 10:11 |
| salmon_ | whe not resource with cmd input? | 10:12 |
| salmon_ | why | 10:12 |
| pigmej | salmon_: because you will need files still | 10:12 |
| dshulyak | salmon_: parsing becomes more complex :) | 10:12 |
| pigmej | dshulyak: well, you could also BUT let's not name then it "run" just "run_cmd" | 10:12 |
| salmon_ | dshulyak: why? | 10:12 |
| pigmej | because it will confuse a lot of things. And a lot of things would need to be adjusted | 10:12 |
| dshulyak | salmon_: because of how parsing is done :) | 10:13 |
| dshulyak | i dont know if it makes sense exlaining it, you should look at parser | 10:14 |
| pigmej | dshulyak: I'm +1 for separate handler, I could write it (i will ahve time after i will fix stupid fabric) | 10:14 |
| salmon_ | dshulyak: I'm lost, but I'm not against doing handler | 10:14 |
| pigmej | just -1 to reuse of "run" in meta.yaml | 10:14 |
| dshulyak | why is that? | 10:15 |
| pigmej | because that "run" kw is used in many places | 10:16 |
| pigmej | we discover these actions by listing files, appending there stuff | 10:17 |
| pigmej | it will be fucking mess, or a lot of ifs | 10:17 |
| pigmej | basically, we assume that whatever is in 'run' is a file, if it's not scan actions and add files there | 10:17 |
| pigmej | while adding run_cmd doesn't need any logic changes in current "run" things (composer, etc) | 10:18 |
| pigmej | not that it's super hardcore, but for now we everywhere assume that run == file | 10:18 |
| salmon_ | RequirementParseError: Expected version spec in enum34;python_version=='2.7' or python_version=='2.6' or python_version=='3.3' at ;python_version=='2.7' or python_version=='2.6' or python_version=='3.3' | 10:20 |
| salmon_ | ideas? | 10:21 |
| pigmej | eeeeeeeeeee ? | 10:21 |
| dshulyak | pigmej: i dont understand, from resource point of view it is exactly run/update/whatever - it will make no sense to make it run_cmd/update_cmd | 10:26 |
| salmon_ | ideas https://bpaste.net/show/055fd7e6e180 ? | 10:28 |
| dshulyak | salmon_: is it the same error you shared on monday? | 10:29 |
| pigmej | dshulyak: I know, BUT the problem is how we work with 'run' stuff in composer logic | 10:29 |
| pigmej | salmon_: zerorpc.... | 10:29 |
| salmon_ | dshulyak: no, this is different. After Maciek changed versions and pigmej fixed zeromq | 10:29 |
| pigmej | tbh I think that zerorpc is a shitty lib (at least python one) | 10:29 |
| pigmej | salmon_: memoryview... are you sure that it's python 2.x ? | 10:30 |
| salmon_ | (venv) [root@fuel solar]# python --version | 10:31 |
| salmon_ | Python 2.7.5 | 10:31 |
| pigmej | it says so.. | 10:31 |
| pigmej | msgpack version? | 10:31 |
| pigmej | msgpack-python==0.4.7 | 10:31 |
| salmon_ | msgpack-python==0.4.7 | 10:31 |
| dshulyak | salmon_: i think this is it - https://github.com/0rpc/zerorpc-python/issues/133 | 10:32 |
| dshulyak | pigmej: why it is shitty? | 10:33 |
| pigmej | there are a lot of weird bugs there | 10:33 |
| dshulyak | it seems this one related to msgpack :) | 10:34 |
| pigmej | and well for our use case, it's "big" | 10:34 |
| pigmej | sure, but for gevent, I hit a lot of issues + see my ugly hack | 10:34 |
| pigmej | https://review.openstack.org/#/c/284262/ | 10:34 |
| pigmej | <3 | 10:35 |
| pigmej | and zerorpc supports a LOT of things taht we don't need :) (not that it's negative) | 10:36 |
| salmon_ | dshulyak: thx, reinstalling msgpack helped :D | 10:36 |
| dshulyak | i guess we need g++ in deps | 10:37 |
| salmon_ | so, g++ compiler is required | 10:37 |
| salmon_ | yup | 10:37 |
| salmon_ | with packages it should be solved | 10:37 |
| pigmej | anyway guys please review: https://review.openstack.org/#/c/284262/ | 10:39 |
| pigmej | :P | 10:39 |
| salmon_ | (venv) [root@fuel f2s]# solar orch report | 10:56 |
| salmon_ | tools.run -> SUCCESS | 10:56 |
| salmon_ | :) | 10:56 |
| salmon_ | progress! | 10:56 |
| dshulyak | salmon_: cool, can you upload your changes? | 10:57 |
| salmon_ | dshulyak: yup, I will update README also | 10:58 |
| salmon_ | dshulyak: 20 minutes | 10:58 |
| salmon_ | dshulyak: in each COmposer file you have a loop. For example here https://github.com/dshulyak/f2s/blob/more_tasks/vrs/vr_apache.yaml#L3 It should be f2s/apache | 10:59 |
| salmon_ | inifnite loop | 11:00 |
| dshulyak | salmon_: you mean from ? | 11:00 |
| salmon_ | dshulyak: yes | 11:01 |
| salmon_ | dshulyak: now you are pointing to itself | 11:01 |
| dshulyak | salmon_: https://github.com/Mirantis/f2s/pull/1/files | 11:11 |
| dshulyak | what relative path we will use? | 11:11 |
| dshulyak | should i change it to f2s/resources/ f2s/vrs ? | 11:11 |
| salmon_ | dshulyak: if names are diifferent f2s alone is ok | 11:12 |
| salmon_ | if you want f2s/resources and f2s/vrs is olso ok | 11:12 |
| salmon_ | just choose one | 11:12 |
| salmon_ | dshulyak: leave it as in this patch | 11:13 |
| salmon_ | dshulyak: merged, here is mine https://github.com/Mirantis/f2s/pull/2 | 11:20 |
| salmon_ | dshulyak: and in each VR/resource we should set puppet_modules=/etc/puppet/modules | 11:21 |
| salmon_ | I think in Resource definition | 11:22 |
| salmon_ | dshulyak: and what I did: With fuel I deployed primary_controller on one node. On second node i deployed base-os. From now I'm using solar and i deployed tools resource | 11:23 |
| salmon_ | If your stage/shell resources are working we still need a resources for this hooks https://github.com/openstack/fuel-astute/blob/master/lib/astute/deploy_actions.rb#L146-L148 | 11:25 |
| salmon_ | those are creating astute.yaml | 11:26 |
| dshulyak | salmon_: upload_facts and link should be solved by role_data resource, no? | 11:35 |
| salmon_ | dshulyak: yup | 11:35 |
| dshulyak | salmon_: and it looks like this one we dont need https://github.com/openstack/fuel-astute/blob/master/mcagents/puppetd.rb#L215 , because this one removes created by astute locks | 11:37 |
| pigmej | salmon_: sorry I'm unable to reproduce your error | 11:37 |
| pigmej | I need more info about resources involved | 11:37 |
| salmon_ | dshulyak: yup, it looks like | 11:37 |
| salmon_ | pigmej: which one? | 11:37 |
| salmon_ | I reported many :P | 11:37 |
| pigmej | fabric | 11:37 |
| pigmej | I mean it works for me ;/ | 11:38 |
| pigmej | ah | 11:38 |
| pigmej | ok, I was able | 11:38 |
| salmon_ | :) | 11:38 |
| salmon_ | dshulyak: I will revert role_data and adjust it. Or are you alredy doing it? | 11:41 |
| dshulyak | salmon_: nope, i am doing fixes to nailgun patch | 11:50 |
| salmon_ | dshulyak: ok | 11:51 |
| *** dshulyak has quit IRC | 11:51 | |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Fabric sync transport does not exit anymore https://review.openstack.org/285268 | 12:26 |
| pigmej | salmon_: ^ | 12:26 |
| openstackgerrit | Bogdan Dobrelya proposed openstack/solar: (WIP) Add centos7 docker support https://review.openstack.org/284782 | 12:36 |
| *** dshulyak has joined #openstack-solar | 12:40 | |
| pigmej | bogdando: I have misc problems with that docker support that you added | 12:43 |
| pigmej | could yuo please chcek everything again on "fresh" machine? | 12:43 |
| pigmej | I was able to build docker image, but sadly, it seems to be not fully provisioned (solar is not installed, repositories are not created etc) | 12:44 |
| salmon_ | hmm, I'm close to implement something like configfb... | 12:56 |
| openstackgerrit | Bogdan Dobrelya proposed openstack/solar: (WIP) Add centos7 docker support https://review.openstack.org/284782 | 13:26 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Improve version parsing in Repository module https://review.openstack.org/285336 | 14:51 |
| dshulyak | salmon_: have you tried to run tasks on bootstrap? | 15:33 |
| salmon_ | dshulyak: nope, only on node with base-os role | 15:34 |
| salmon_ | It looks like I generated astue.yaml! Time to test it with something :) | 15:58 |
| pigmej | dshulyak: I had to destroy your scipts :/ | 16:27 |
| dshulyak | pigmej: what scripts? | 16:27 |
| pigmej | dshulyak: https://github.com/Mirantis/f2s/tree/setupify | 16:28 |
| pigmej | sorry :( | 16:28 |
| dshulyak | i dont understand what the problem :) you just moved them to another directory | 16:29 |
| pigmej | but you can now pip install f2s | 16:29 |
| pigmej | yup | 16:29 |
| pigmej | https://github.com/Mirantis/f2s/pull/3 | 16:29 |
| dshulyak | so why syou are orry? | 16:29 |
| pigmej | salmon_: scrared me :P | 16:30 |
| salmon_ | pigmej: :P | 16:30 |
| pigmej | so after it will be merged, artem can start making package :) | 16:31 |
| dshulyak | pigmej: why this one is commented +# solar>=2.0.1 ? | 16:32 |
| pigmej | fuck | 16:33 |
| pigmej | requirements shouldn't be hanged | 16:33 |
| pigmej | changed* | 16:33 |
| pigmej | pushed with force :D | 16:34 |
| salmon_ | what do uou need to change? | 16:35 |
| salmon_ | *you | 16:35 |
| pigmej | ? | 16:36 |
| pigmej | I did not | 16:36 |
| pigmej | I pushed my temp requirements | 16:37 |
| dshulyak | hm | 16:44 |
| dshulyak | i have transports import errors - ImportError: No module named libtorrent and solar-agent.client | 16:45 |
| dshulyak | after pip instal | 16:45 |
| dshulyak | pip install solar | 16:45 |
| pigmej | yuo have old code ? | 16:45 |
| dshulyak | pip install ? | 16:46 |
| dshulyak | solar==0.2.1 | 16:46 |
| pigmej | https://review.openstack.org/#/c/281163/ | 16:46 |
| dshulyak | do we have it in 0.2.1 ? | 16:47 |
| pigmej | nope | 16:47 |
| pigmej | you shouldn't use 0.2.1 | 16:47 |
| pigmej | hmm | 16:48 |
| pigmej | why guys in f2s you require solar 2.0.1 ? :D | 16:48 |
| pigmej | salmon_: :> | 16:48 |
| salmon_ | pigmej: >= :P | 16:48 |
| pigmej | 2.0.1!! | 16:49 |
| salmon_ | dshulyak: install it from git | 16:49 |
| pigmej | salmon_: 2.0.1 vs 0.2.1 | 16:49 |
| dshulyak | yeah, i installed it from master already | 16:49 |
| salmon_ | pigmej: ah :D | 16:50 |
| pigmej | salmon_: ;D | 16:50 |
| dshulyak | it seems we forgot to add semantic_version to reqs | 16:50 |
| pigmej | dshulyak: https://github.com/openstack/solar/blob/master/requirements.txt#L25 | 16:51 |
| pigmej | it is there | 16:51 |
| dshulyak | wtf | 16:51 |
| *** angdraug has joined #openstack-solar | 16:56 | |
| salmon_ | globals.yaml is also generating | 17:04 |
| salmon_ | pigmej: is it ready https://github.com/Mirantis/f2s/pull/3 ? | 17:08 |
| pigmej | salmon_: sure | 17:10 |
| *** dshulyak has quit IRC | 17:10 | |
| salmon_ | And Dima quit :( | 17:10 |
| salmon_ | ok, merging | 17:10 |
| *** alexz__ has joined #openstack-solar | 17:13 | |
| *** azvyagintsev has quit IRC | 17:20 | |
| openstackgerrit | Bogdan Dobrelya proposed openstack/solar: d(WIP) Add centos7 docker support https://review.openstack.org/284782 | 17:21 |
| openstackgerrit | Lukasz Oles proposed openstack/solar: Pass all args to resource and allow dumping as a yaml https://review.openstack.org/285449 | 17:26 |
| *** dshulyak has joined #openstack-solar | 17:27 | |
| pigmej | salmon_: where are tests for it? | 17:27 |
| pigmej | and please store result of args... | 17:28 |
| pigmej | like args = resource.args etc | 17:28 |
| salmon_ | https://github.com/Mirantis/f2s/pull/4 | 17:29 |
| salmon_ | pigmej: ok, on Monday :) | 17:29 |
| salmon_ | going off for now, maybe I will be later | 17:29 |
| pigmej | k | 17:29 |
| pigmej | take care salmon_ ! | 17:29 |
| openstackgerrit | Bogdan Dobrelya proposed openstack/solar: Add centos7 docker support https://review.openstack.org/284782 | 17:33 |
| *** dshulyak has quit IRC | 17:46 | |
| *** dshulyak has joined #openstack-solar | 18:35 | |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Replace Bunch with Munch package https://review.openstack.org/285512 | 19:35 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Replace python-pyzmq by python-zmq in solar.spec https://review.openstack.org/285513 | 19:36 |
| *** angdraug has quit IRC | 20:30 | |
| *** dshulyak has quit IRC | 22:24 | |
| openstackgerrit | Merged openstack/solar: Replace python-pyzmq by python-zmq in solar.spec https://review.openstack.org/285513 | 22:52 |
| openstackgerrit | Merged openstack/solar: Workaround for TypeError in zerorpc Events.__del__ https://review.openstack.org/284262 | 23:23 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Replace Bunch with Munch package https://review.openstack.org/285512 | 23:29 |
| *** angdraug has joined #openstack-solar | 23:29 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!