| *** lifeless_ has joined #refstack | 03:03 | |
| *** lifeless has quit IRC | 03:03 | |
| openstackgerrit | Chandan Kumar proposed openstack/python-tempestconf master: [DNM] patch to test backward compatibilty/ container check https://review.openstack.org/570524 | 05:54 |
|---|---|---|
| *** kopecmartin has joined #refstack | 07:04 | |
| chandankumar | arxcruz|ruck: kopecmartin please have a look at these two patches https://review.openstack.org/568869 https://review.openstack.org/572775 | 07:15 |
| *** tosky has joined #refstack | 07:26 | |
| chandankumar | tosky: please have a look at these two patches https://review.openstack.org/568869 https://review.openstack.org/572775 | 07:53 |
| tosky | chandankumar: why the "# Note(chkumar246): It will be used in all services to check" ? | 07:57 |
| tosky | I mean, just define the method; other methods defined there don't have a TODO even if they are not used by all service classes yet | 07:58 |
| chandankumar | tosky: I will remove that one , in one go, Any more comments | 07:59 |
| tosky | chandankumar: sure, only if there are more comments | 07:59 |
| tosky | one thing I was not sure about the change is the removal of the option object-storage-feature-enabled.discoverability, my question is: | 08:00 |
| tosky | that option was added (I think by you) so that users can stop the detection of swift when ceph is used (because some API are missing) | 08:00 |
| tosky | with this new code, will swift be simply set to unavailable? | 08:01 |
| tosky | I mean, even without the discoverability option, will the final effect be the same? | 08:01 |
| chandankumar | tosky: yup, with if healthcheck api will work, swift is unavailable discoverabilty option is related with swift so it will auto get disabled | 08:02 |
| tosky | chandankumar: and right now what do we do when ceph is used instead of swift? Don't we set it to true but without discovery? | 08:04 |
| tosky | added a question to the other review too | 08:11 |
| chandankumar | tosky: http://git.openstack.org/cgit/openstack/tripleo-quickstart/tree/config/general_config/featureset008.yml#n52 | 08:22 |
| chandankumar | tosky: http://git.openstack.org/cgit/openstack/tripleo-quickstart/tree/config/general_config/featureset019.yml#n117 | 08:22 |
| tosky | does that mean that tripleo never tests ceph+radosgw as swift replacement with tempest? | 08:25 |
| chandankumar | tosky: yup, i think so, I need to check with Emilienm | 08:29 |
| tosky | chandankumar: ok, please recheck, and also check if it something that should be done in the future | 08:30 |
| tosky | chandankumar: I know that we can always override the service availability, but just to be sure | 08:30 |
| chandankumar | tosky: fs8 it is enabled https://github.com/openstack/tripleo-heat-templates/blob/master/ci/environments/scenario004-multinode-containers.yaml#L8 | 08:32 |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Remove admin_* from identity section https://review.openstack.org/573022 | 08:44 |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Generate accounts.yaml https://review.openstack.org/570781 | 08:45 |
| kopecmartin | chandankumar, tosky in order to finish --create-accounts-file patch, i needed to clean some things which were super confusing, starting by this patch | 08:47 |
| kopecmartin | https://review.openstack.org/#/c/573017/ | 08:47 |
| tosky | chandankumar: do you mean that ceph is enabled, but the detection is disabled and the service is marked as non-available, and that the new code will not change this because it will set the service as non-available even without discoverability? | 08:47 |
| tosky | kopecmartin: oh, I was hoping that it's a change for later | 08:47 |
| kopecmartin | tosky, me too, but we really need to know, where and what credentials are stored | 08:48 |
| chandankumar | tosky: ceph is enabled, yup the detection is disabled, as it might not support swift functionality | 08:49 |
| tosky | kopecmartin: so why the current code that creates accounts.yaml does not work when using the variables with tenant? | 08:49 |
| chandankumar | tosky: object-storage-featured-enabled.discoverabilty is dependent on swift service, if it is disabled it will get all auto disabled | 08:50 |
| tosky | chandankumar: but I thought that there are some places where ceph is deployed and the swift is marked as available, even if without detection | 08:50 |
| chandankumar | tosky: I will wait for Emilienm to come and clarify | 08:50 |
| tosky | my point is: if anyone wants to test ceph/radosgw as swift, is the process going to be much different from today? | 08:51 |
| kopecmartin | tosky, it might work, i just didn't want to do if else statements .. and it was super confusing to figure out how it is going with credentials if i needed to keep in mind project/tenant difference | 08:51 |
| tosky | chandankumar: right now you need to set autodetection to false (with discoverability) and then set the service as available explicitly | 08:51 |
| tosky | chandankumar: will it be much different, apart from the lack of the discoverability key? | 08:51 |
| chandankumar | tosky: nope, I will make the changes | 08:52 |
| chandankumar | tosky: let me push a new patchset | 08:52 |
| tosky | kopecmartin: and all the versions that we need to support accepts the configuration key with project in their name? | 08:52 |
| tosky | chandankumar: ehm, which changes? | 08:52 |
| tosky | do you mean that a) yes, the process will be much different and some changes are needed b) no, the process won't be different, just need to document it c) something else? | 08:53 |
| kopecmartin | tosky, they should, it was removed in Tempest 12 and currently we have Tempest 16 on Ocata .. | 08:53 |
| kopecmartin | tosky, https://storyboard.openstack.org/#!/story/2002116 | 08:53 |
| tosky | kopecmartin: ok then | 08:56 |
| *** lifeless_ has quit IRC | 09:17 | |
| *** lifeless has joined #refstack | 09:18 | |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: [DNM] Test --create-accounts-file argument https://review.openstack.org/572695 | 09:20 |
| chandankumar | tosky: the healthcheck api patch | 09:24 |
| chandankumar | kopecmartin:https://review.openstack.org/#/q/topic:remove-support-of-cinder-v1-api+(status:open+OR+status:merged) | 09:24 |
| tosky | chandankumar: irrelevant for now | 09:26 |
| tosky | "<gmann> we cannot remove tempest/lib/services/volume/v1 as they were never got deprecated formally so we have keep them as 1 more cycle with formal deprecation period" | 09:27 |
| chandankumar | arxcruz|ruck: tosky kopecmartin https://etherpad.openstack.org/p/YVR18-forum-qa-ops-user-feedback | 09:39 |
| chandankumar | tempest.conf creation from keystone endpoint and overrides -> already taken care by tempestconf | 09:39 |
| arxcruz|ruck | tosky: do you watch nanatsu no taizai ? | 09:40 |
| tosky | arxcruz|ruck: oh, the seven deadly sins, yes; are new episodes landing? | 09:45 |
| * chandankumar searches | 09:45 | |
| arxcruz|ruck | tosky: not sure if you are aware, but there's a second season ongoing | 09:45 |
| tosky | oh, missed that; I will wait for netflix :) | 09:48 |
| chandankumar | arxcruz|ruck: where is freely available | 09:50 |
| arxcruz|ruck | chandankumar: well, i know where there is in japanese with portuguese subtitles | 09:51 |
| arxcruz|ruck | :D | 09:51 |
| chandankumar | Ah i india it is not available | 09:51 |
| chandankumar | on netflix | 09:51 |
| arxcruz|ruck | chandankumar: vpn | 09:56 |
| arxcruz|ruck | ;) | 09:56 |
| arxcruz|ruck | one punch man and atack on titans will get new seasons this year also | 09:57 |
| arxcruz|ruck | can't wait! | 09:57 |
| arxcruz|ruck | :D | 09:57 |
| chandankumar | tosky: arxcruz|ruck Have you watched Troll hunters Tales of Arcadia | 10:13 |
| chandankumar | ? | 10:13 |
| arxcruz|ruck | yup | 10:14 |
| chandankumar | arxcruz|ruck: How is it? | 10:15 |
| arxcruz|ruck | awesome | 10:18 |
| arxcruz|ruck | unfortunately it's over | 10:18 |
| arxcruz|ruck | they will make 2 more spin offs | 10:18 |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Remove admin_* from identity section https://review.openstack.org/573022 | 10:21 |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Generate accounts.yaml https://review.openstack.org/570781 | 10:22 |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: [DNM] Test --create-accounts-file argument https://review.openstack.org/572695 | 10:23 |
| openstackgerrit | Chandan Kumar proposed openstack/python-tempestconf master: Remove image.http_image from playbook https://review.openstack.org/572775 | 11:40 |
| openstackgerrit | Merged openstack/python-tempestconf master: Fix broken link in readme https://review.openstack.org/572438 | 11:41 |
| openstackgerrit | Merged openstack/python-tempestconf master: fix url https://review.openstack.org/572439 | 11:42 |
| chandankumar | tosky: regarding healthcheck api, you mean to explicility set object-storage-feature-enabled.discoverabilyt to false irrespctive of swift is available or not? | 11:54 |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: Generate accounts.yaml https://review.openstack.org/570781 | 12:08 |
| openstackgerrit | Martin Kopec proposed openstack/python-tempestconf master: [DNM] Test --create-accounts-file argument https://review.openstack.org/572695 | 12:09 |
| openstackgerrit | Chandan Kumar proposed openstack/python-tempestconf master: Use healthcheck api to determine swift service https://review.openstack.org/568869 | 12:09 |
| chandankumar | tosky: ^^ please have a look | 12:09 |
| openstackgerrit | Chandan Kumar proposed openstack/python-tempestconf master: Use healthcheck api to determine swift service https://review.openstack.org/568869 | 12:11 |
| tosky | always after the gates (in fact you just updated it :) | 12:18 |
| openstackgerrit | Chandan Kumar proposed openstack/refstack-client master: Generate tempest.conf automatically using refstack-client https://review.openstack.org/541273 | 13:06 |
| tosky | chandankumar: Task, not ask | 13:08 |
| *** openstackgerrit has quit IRC | 13:34 | |
| *** lifeless_ has joined #refstack | 14:53 | |
| *** lifeless has quit IRC | 14:53 | |
| kopecmartin | tosky, chandankumar arxcruz|ruck the chain of patches starting by this https://review.openstack.org/#/c/573017 is ready to be reviewed | 15:21 |
| arxcruz|ruck | kopecmartin: i'll not do that today, it's almost 6pm here already ;) | 15:28 |
| arxcruz|ruck | man need to play video games | 15:29 |
| -openstackstatus- NOTICE: Zuul update for Ansible 2.5 in progress. Scheduler crashed as unexpected side effect of pip upgrade. Will be back and running shortly. | 15:32 | |
| -openstackstatus- NOTICE: The zuul upgrade to ansible 2.5 is complete and zuul is running again. Changes uploaded or approved between 15:25 and 15:45 will need to be rechecked. Please report any problems in #openstack-infra | 16:11 | |
| *** lifeless has joined #refstack | 16:16 | |
| *** lifeless_ has quit IRC | 16:16 | |
| chandankumar | arxcruz|ruck: tosky https://review.openstack.org/568869 please have a look at healthcheck one | 16:24 |
| *** lifeless_ has joined #refstack | 16:35 | |
| *** lifeless has quit IRC | 16:36 | |
| *** kopecmartin has quit IRC | 17:07 | |
| *** openstackgerrit has joined #refstack | 18:36 | |
| openstackgerrit | Chris Hoge proposed openstack/refstack master: WIP Migrating from bower to yarn https://review.openstack.org/573404 | 18:36 |
| openstackgerrit | Merged openstack/python-tempestconf master: List and create swift roles https://review.openstack.org/571981 | 18:59 |
| *** beisner-sick is now known as beisner | 19:00 | |
| openstackgerrit | Chris Hoge proposed openstack/refstack master: WIP Migrating from bower to yarn https://review.openstack.org/573404 | 19:38 |
| *** lifeless_ has quit IRC | 22:03 | |
| *** lifeless has joined #refstack | 22:04 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!