*** tosky has quit IRC | 01:06 | |
*** dave-mccowan has quit IRC | 01:23 | |
*** dave-mccowan has joined #openstack-sahara | 02:50 | |
*** dave-mccowan has quit IRC | 04:56 | |
*** rcernin has quit IRC | 07:09 | |
*** tosky has joined #openstack-sahara | 08:27 | |
*** tellesnobrega has joined #openstack-sahara | 10:14 | |
openstackgerrit | Telles Mota Vidal Nóbrega proposed openstack/sahara master: APIv2 - api-ref documentation for APIv2 https://review.openstack.org/624365 | 11:25 |
---|---|---|
*** tellesnobrega has quit IRC | 12:22 | |
*** tellesnobrega has joined #openstack-sahara | 13:39 | |
*** jeremyfreudberg has joined #openstack-sahara | 13:55 | |
tellesnobrega | so, tosky any plans on how to solve the issue? | 15:00 |
tosky | about https://review.openstack.org/#/c/626643/ and what happens after it (sahara-eng stuck on ssh) | 15:01 |
tosky | it's something about pickle, and I suspect something related with the compatibility of pickle between python 3 (on the host, I tried on bionic) and the guest (centos7 with python 2) | 15:01 |
tosky | I tried to force a specific version of the pickle protocol, no success | 15:02 |
tosky | and then I decided to postpone the investigation after the plugin split and the APIv2 stabilization | 15:02 |
tellesnobrega | sounds like a good prioritization plan | 15:02 |
tosky | we will have enough time between M2 and M3 - unless I have another eureka moment | 15:02 |
tellesnobrega | awesome | 15:03 |
tosky | maybe the next step is to try a vanilla image based on xenial, which should ship with python3, just to see if the issue is about mixed python versions | 15:04 |
tosky | but even that, we will need to support guest images with only python2 for a while | 15:04 |
tellesnobrega | yes | 15:07 |
tellesnobrega | we need a fix that can make it work with py2 and py3 interchanging | 15:07 |
tellesnobrega | at least for now | 15:07 |
*** Bhujay has joined #openstack-sahara | 15:15 | |
*** Bhujay has quit IRC | 15:16 | |
openstackgerrit | Jeremy Freudberg proposed openstack/sahara master: APIv2 - api-ref documentation for APIv2 https://review.openstack.org/624365 | 15:31 |
*** jeremyfreudberg has quit IRC | 15:41 | |
*** tellesnobrega has quit IRC | 15:42 | |
*** tellesnobrega has joined #openstack-sahara | 16:36 | |
tosky | tellesnobrega: this is the document about creating projects, which also explain how to add new repositories: https://docs.openstack.org/infra/manual/creators.html | 16:59 |
tosky | I sent the initial reviews, please check if they contain all the requested information | 17:00 |
tellesnobrega | will do | 17:00 |
tosky | https://review.openstack.org/#/c/628209/ and https://review.openstack.org/#/c/628210/, for the logs | 17:00 |
tellesnobrega | tosky, both patches look really good | 17:30 |
tosky | I have follow-up patches ready for project-config | 17:32 |
tosky | tellesnobrega: please check AJeager's comment, you probably need to cleanup the old branches | 17:35 |
tosky | I also suggest to run the following command in each repository before repushing: | 17:35 |
tosky | git gc --aggressive --prune=now | 17:35 |
tellesnobrega | checking now | 17:45 |
tellesnobrega | will clean up now | 17:46 |
tosky | there is another comment about i18n | 17:46 |
tosky | is it something that you think it's better to add to each repository before the import? | 17:47 |
tellesnobrega | I saw the comment, I can double check what we need for translation, but I guess we can follow ajeager's suggestion to do it without translation and fix it later | 17:50 |
tellesnobrega | what do you think? | 17:53 |
tellesnobrega | tosky, ran git gc --agressive --prune=now on all repos, and deleted the branches | 17:56 |
tellesnobrega | now it is all masters | 17:56 |
tosky | fine with adding/fixing translation later | 17:56 |
tosky | higher contribution stats :) | 17:56 |
tosky | tellesnobrega: oh, each split plugin has the enable_lazy call in its tests/__init__.py | 17:58 |
tosky | do you want to remove it once and for all after the split, or wait for https://review.openstack.org/#/c/626643/ to land | 17:58 |
tellesnobrega | I thought about this once I saw your patch | 18:01 |
tellesnobrega | I can remove it and amend on the current patch | 18:01 |
tellesnobrega | but I guess the best way is to rebase it after it merges | 18:01 |
tosky | then let's hope that 626643 lands before :) | 18:02 |
tosky | before the import | 18:02 |
openstackgerrit | Telles Mota Vidal Nóbrega proposed openstack/sahara master: APIv2 - api-ref documentation for APIv2 https://review.openstack.org/624365 | 18:03 |
tellesnobrega | yes | 18:03 |
tellesnobrega | if it lands after I will send a patch fixing it | 18:03 |
tellesnobrega | tosky, I'm taking a look at the inconsistencies on policy enforcement | 18:58 |
tellesnobrega | I believe that I can move the v2 stuff out of jobs and into job-executions, with v2/jobs/ url | 18:59 |
tosky | uhm, uhm | 19:01 |
tosky | I will probably need to see it | 19:01 |
tosky | as long as it does not change APIv1.1 behavior, I guess I'm fine with that | 19:02 |
tellesnobrega | it shouldn't | 19:02 |
tellesnobrega | I write up a patch | 19:02 |
openstackgerrit | Telles Mota Vidal Nóbrega proposed openstack/sahara master: Fixing policies inconsistencies https://review.openstack.org/628289 | 20:14 |
*** tellesnobrega has quit IRC | 20:19 | |
*** rcernin has joined #openstack-sahara | 22:42 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!