| *** shu-mutou-AFK is now known as shu-mutou | 00:00 | |
| *** Drago has quit IRC | 00:03 | |
| *** Qiming has joined #senlin | 01:02 | |
| *** Qiming_ has joined #senlin | 01:32 | |
| *** Qiming__ has joined #senlin | 01:33 | |
| *** Qiming__ has quit IRC | 01:33 | |
| *** yanyanhu has joined #senlin | 01:37 | |
| *** Qiming- has joined #senlin | 01:38 | |
| *** Qiming- has quit IRC | 01:44 | |
| openstackgerrit | Merged openstack/senlin: Remove duplicated logic about oslo notifier https://review.openstack.org/324271 | 01:44 |
|---|---|---|
| *** Qiming_ has quit IRC | 01:46 | |
| *** Qiming_ has joined #senlin | 01:46 | |
| *** Qiming has quit IRC | 01:49 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Minor tweak versioned object implementation https://review.openstack.org/324298 | 01:56 |
| *** elynn has joined #senlin | 01:57 | |
| *** elynn_ has joined #senlin | 02:02 | |
| *** elynn has quit IRC | 02:02 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Fix nova_v2 driver https://review.openstack.org/324341 | 02:09 |
| *** Qiming_ is now known as Qiming | 02:10 | |
| *** zzxwill has joined #senlin | 02:18 | |
| openstackgerrit | zzxwill proposed openstack/senlin: enlin Basics in senlin typo in remove node section https://review.openstack.org/325002 | 02:26 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Fix typo in tutorial about removing node https://review.openstack.org/325002 | 02:28 |
| Qiming | zzxwill, thanks for the patch | 02:29 |
| Qiming | just published an edit to the commit message | 02:29 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Unify the naming of tempest API test cases https://review.openstack.org/325006 | 02:29 |
| zzxwill | Thans:) | 02:29 |
| zzxwill | Thanks:) | 02:29 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Fix nova_v2 driver https://review.openstack.org/324341 | 02:36 |
| elynn_ | Hi yanyanhu about this patch ^^ | 02:43 |
| yanyanhu | hi, elynn_ | 02:43 |
| elynn_ | should we support old openstacksdk functions? | 02:43 |
| yanyanhu | you mean nova_v2 driver? | 02:44 |
| elynn_ | yes | 02:44 |
| elynn_ | Since in mitaka the openstacksdk requirements is >=0.8.1 | 02:44 |
| yanyanhu | oh, that is a problem | 02:44 |
| yanyanhu | either we add cap to sdk requirement | 02:44 |
| elynn_ | Can we change the requirements in stable release? | 02:44 |
| yanyanhu | or back port the change | 02:44 |
| yanyanhu | not sure | 02:45 |
| yanyanhu | but I guess adding cap is better than back port? | 02:45 |
| elynn_ | That would be better if we can change requirements for openstacksdk in mitaka... | 02:45 |
| yanyanhu | yes... | 02:45 |
| elynn_ | I think so... | 02:45 |
| yanyanhu | Qiming, have any experience about this? | 02:45 |
| yanyanhu | another way is back port this change to mitaka stable | 02:46 |
| Qiming | is mitaka broken? | 02:46 |
| elynn_ | bump openstacksdk version to >=0.8.2 would be enough. | 02:46 |
| elynn_ | Qiming, yes. | 02:46 |
| Qiming | I think in mitaka, there is a openstacksdk version cap | 02:46 |
| yanyanhu | Qiming, if user install senlin/mitaka/stable newly | 02:46 |
| yanyanhu | it will break | 02:46 |
| yanyanhu | oh, if cap is there | 02:46 |
| yanyanhu | there is no problem | 02:46 |
| Qiming | okay, then propose a cap there | 02:46 |
| Qiming | cap the vesion to 0.8.1 would be okay? | 02:47 |
| yanyanhu | I think so | 02:47 |
| yanyanhu | let double check it | 02:48 |
| elynn_ | Is it that you are talking about? https://github.com/openstack/requirements/blob/stable/mitaka/upper-constraints.txt#L199 | 02:48 |
| Qiming | yes | 02:49 |
| yanyanhu | nice | 02:49 |
| yanyanhu | == | 02:49 |
| elynn_ | Then it is correct. | 02:49 |
| Qiming | so, if you are using mitaka, you are not supposed to use openstacksdk>=0.8.2 | 02:49 |
| yanyanhu | so version of all packages will be fixed when releasing | 02:49 |
| yanyanhu | cool | 02:49 |
| Qiming | capped | 02:49 |
| elynn_ | okay, then we only need to fix master. | 02:50 |
| *** yuanying has quit IRC | 02:50 | |
| elynn_ | Just curious why functional tests can't catch this bug? | 02:50 |
| Qiming | maybe your functional tests venv is not refreshed | 02:50 |
| elynn_ | It won't go to openstacksdk? | 02:51 |
| Qiming | openstacksdk will be installed to .venv/functional | 02:51 |
| Qiming | if you do 'tox -e functional' instead of 'tox -r -e functional', you will be always using the version you installed the first time you do 'tox -e functional' | 02:52 |
| elynn_ | If using openstack_test backend, will it use openstacksdk? | 02:52 |
| Qiming | in other words, it becomes your responsibility to make sure your venv is synced with the latest packages | 02:52 |
| Qiming | no, it will use a fake backend | 02:53 |
| Qiming | that is a problem we need to fix | 02:53 |
| elynn_ | okay... we need to setup integration tests... | 02:54 |
| yanyanhu | elynn_, that's why we need integration test I think | 02:54 |
| yanyanhu | which will be run with all TRUE backend service enabled | 02:54 |
| Qiming | in addition to 'functional' test which are using the fake backend, we need to add 'integration' tests that are using real drivers | 02:54 |
| yanyanhu | yep | 02:54 |
| Qiming | btw, I don't like the place where the fake drivers are placed | 02:54 |
| Qiming | last time I tried to find the fake drivers, it took me quite some time | 02:54 |
| Qiming | oh, guys, today is the deadline for creating newton-1 release | 02:57 |
| Qiming | please help check the review pipeline and help check if there are release notes we want to add | 02:58 |
| openstackgerrit | Merged openstack/python-senlinclient: Updated from global requirements https://review.openstack.org/324864 | 03:03 |
| openstackgerrit | Merged openstack/senlin: Fix typo in tutorial about removing node https://review.openstack.org/325002 | 03:04 |
| openstackgerrit | Merged openstack/senlin: Unify the naming of tempest API test cases https://review.openstack.org/325006 | 03:39 |
| openstackgerrit | Merged openstack/senlin: Minor tweak versioned object implementation https://review.openstack.org/324298 | 03:40 |
| *** zzxwill has quit IRC | 03:46 | |
| *** yuanying has joined #senlin | 03:49 | |
| openstackgerrit | Merged openstack/senlin: Optimize unit tests https://review.openstack.org/324224 | 04:12 |
| *** elynn_ has quit IRC | 04:37 | |
| *** zzxwill has joined #senlin | 04:55 | |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Initial framework for health manager listener https://review.openstack.org/325045 | 05:01 |
| *** elynn_ has joined #senlin | 05:04 | |
| *** elynn__ has joined #senlin | 05:09 | |
| *** elynn_ has quit IRC | 05:09 | |
| *** yuanying has quit IRC | 05:17 | |
| *** eldon_ has joined #senlin | 05:18 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Fix nova_v2 driver https://review.openstack.org/324341 | 05:40 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Initial framework for health manager listener https://review.openstack.org/325045 | 05:50 |
| *** yuanying has joined #senlin | 06:05 | |
| *** openstackgerrit has quit IRC | 06:17 | |
| *** openstackgerrit has joined #senlin | 06:18 | |
| *** elynn has joined #senlin | 06:18 | |
| *** elynn__ has quit IRC | 06:19 | |
| *** eldon_ has quit IRC | 06:19 | |
| *** yuanying has quit IRC | 06:28 | |
| *** yuanying has joined #senlin | 06:29 | |
| *** yuanying has quit IRC | 06:31 | |
| *** yuanying has joined #senlin | 06:31 | |
| *** yuanying has quit IRC | 06:33 | |
| *** yuanying has joined #senlin | 06:34 | |
| *** eldon_ has joined #senlin | 06:50 | |
| *** openstackgerrit has quit IRC | 07:03 | |
| *** openstackgerrit has joined #senlin | 07:04 | |
| *** elynn has quit IRC | 07:27 | |
| openstackgerrit | Merged openstack/senlin: Fix nova_v2 driver https://review.openstack.org/324341 | 07:28 |
| *** elynn has joined #senlin | 07:31 | |
| *** elynn_ has joined #senlin | 07:36 | |
| *** elynn has quit IRC | 07:37 | |
| *** yuanying has quit IRC | 07:50 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Add negative tempest API test for delete notfound https://review.openstack.org/325112 | 08:10 |
| elynn_ | Hi Qiming | 08:19 |
| elynn_ | Do you know what's the difference between 'check' and 'gate' in gate layout.yaml? https://github.com/openstack-infra/project-config/blob/master/zuul/layout.yaml#L6702-L6707 | 08:20 |
| yanyanhu | elynn_, just as it is described in layout.yam I think :) | 08:24 |
| *** shu-mutou is now known as shu-mutou-AFK | 08:24 | |
| yanyanhu | check is for any new patchset and gate is for verify workflow | 08:24 |
| yanyanhu | verify and merge | 08:24 |
| elynn_ | So if I want to enable tempest job, I shall add it both to check and gate section, right? | 08:25 |
| yanyanhu | yes, then it will vote for both patch check and verify | 08:27 |
| yanyanhu | oh, you mean just enable it without voting? | 08:27 |
| yanyanhu | if so, just adding it to check and gate list is ok | 08:27 |
| elynn_ | yanyanhu, Saw you in higgins project, are you working on that ? | 08:33 |
| yanyanhu | yes, hope can spend some time on it | 08:33 |
| yanyanhu | watching and reviewing I think | 08:33 |
| elynn_ | What's the status of this project? | 08:34 |
| yanyanhu | just initialized about a month ago | 08:34 |
| yanyanhu | now the team has reached consensus on some basic but important work items | 08:35 |
| yanyanhu | but still lots of things need to discuss and clarify | 08:35 |
| elynn_ | good to know | 08:36 |
| yanyanhu | sheel and yuanying will start working on API design I think | 08:36 |
| yanyanhu | yes, how it can grow fast :) | 08:36 |
| elynn_ | Hope this project going well! | 08:37 |
| yanyanhu | then we will have a backend for container cluster management | 08:37 |
| yanyanhu | yes :) | 08:37 |
| *** senk_ has joined #senlin | 08:37 | |
| *** senk has quit IRC | 08:39 | |
| elynn_ | seems the jenkins are not stable right now. | 08:39 |
| yanyanhu | yes... | 08:39 |
| *** yuanying has joined #senlin | 08:39 | |
| *** zzxwill has quit IRC | 09:02 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Add negative tempest API test for node create https://review.openstack.org/325153 | 09:26 |
| Qiming | yanyan, https://review.openstack.org/#/c/321382/ | 09:28 |
| Qiming | this patch is incomplete | 09:28 |
| Qiming | the patch is about cluster deletion, but the modification was extended to policies and profiles | 09:29 |
| Qiming | okay, I see, the second part is only about doc correction, the code is correct | 09:30 |
| openstackgerrit | Qiming Teng proposed openstack/senlin: Release notes for newton-1 https://review.openstack.org/325162 | 09:34 |
| -openstackstatus- NOTICE: CI is experiencing issues with test logs, all jobs are currently UNSTABLE as a result. No need to recheck until this is fixed! Thanks for your patience. | 09:38 | |
| yanyanhu | Qiming, yes :) | 09:39 |
| *** elynn_ has quit IRC | 09:42 | |
| openstackgerrit | Qiming Teng proposed openstack/python-senlinclient: Added release notes for a new release https://review.openstack.org/325172 | 09:42 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Add negative tempest API test for cluster create https://review.openstack.org/325176 | 09:44 |
| *** eldon_ has quit IRC | 09:48 | |
| yanyanhu | hi, Qiming have you finished the release? | 09:48 |
| yanyanhu | I think there could be a issue in api doc | 09:49 |
| yanyanhu | there is no chance that resource get api returns 400? | 09:49 |
| yanyanhu | bad request | 09:49 |
| Qiming | not yet | 09:50 |
| yanyanhu | should remove it from api doc? | 09:51 |
| yanyanhu | now 400 is listed in possible resp status code of resource get API | 09:52 |
| Qiming | checking | 09:53 |
| Qiming | yes, there is a possibility | 09:53 |
| Qiming | because we use resource identity instead of resource ID for retrieval | 09:53 |
| Qiming | the identity can be a name, short-id or UUID of a resource | 09:53 |
| yanyanhu | yes | 09:54 |
| Qiming | there is a possibility that a user specified a short-id, but it is matching more than one resources | 09:54 |
| Qiming | We raise MultipleChoices from internal | 09:54 |
| Qiming | then translate that into a BadRequest (400) | 09:54 |
| yanyanhu | ok, I see | 09:54 |
| yanyanhu | right, this is possible | 09:54 |
| Qiming | it is users fault that they have to be more specific | 09:54 |
| yanyanhu | understand | 09:55 |
| Qiming | python-senlinclient release 0.4.2 will be based on 685ab4d317745811254b7e1eb347cffaafc26f8b | 09:57 |
| Qiming | senlin release 2.0.0.0b1 will be based on 23 from senlin.api.common import serializers | 09:58 |
| Qiming | 2 | 09:58 |
| Qiming | 8f9ddb9185851b4353201e913db3a5ecbecf4286 | 09:58 |
| Qiming | sorry | 09:58 |
| Qiming | waiting for gate to stabilize and have those two commits in | 09:58 |
| Qiming | then will propose the release patch | 09:58 |
| yanyanhu | ok | 10:01 |
| -openstackstatus- NOTICE: CI is experiencing issues with test logs, all jobs are currently UNSTABLE as a result. No need to recheck until this is fixed! Thanks for your patience. | 10:10 | |
| *** ChanServ changes topic to "CI is experiencing issues with test logs, all jobs are currently UNSTABLE as a result. No need to recheck until this is fixed! Thanks for your patience." | 10:10 | |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Add negative tempest API test for resource get https://review.openstack.org/325207 | 10:22 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Add negative tempest API test for node create https://review.openstack.org/325153 | 10:28 |
| openstackgerrit | Yanyan Hu proposed openstack/senlin: Add negative tempest API test for cluster create https://review.openstack.org/325176 | 10:31 |
| *** yanyanhu has quit IRC | 10:32 | |
| *** zzxwill has joined #senlin | 11:35 | |
| -openstackstatus- NOTICE: CI is experiencing issues with test logs, all jobs are currently UNSTABLE as a result. No need to recheck until this is fixed! Thanks for your patience. | 11:44 | |
| openstackgerrit | Merged openstack/python-senlinclient: Added release notes for a new release https://review.openstack.org/325172 | 13:43 |
| *** ChanServ changes topic to "IRCLog: http://eavesdrop.openstack.org/irclogs/%23senlin/ | Bugs: bugs.launchpad.net/senlin | Review: https://review.openstack.org/#/q/project:openstack/senlin,n,z" | 14:02 | |
| -openstackstatus- NOTICE: Cleanup from earlier block storage disruption on static.openstack.org has been repaired, and any jobs which reported an "UNSTABLE" result or linked to missing logs between 08:00-14:00 UTC can be retriggered by leaving a "recheck" comment. | 14:02 | |
| openstackgerrit | Merged openstack/senlin: Release notes for newton-1 https://review.openstack.org/325162 | 14:03 |
| *** zzxwill has quit IRC | 14:34 | |
| *** Drago has joined #senlin | 15:01 | |
| *** Drago has quit IRC | 15:01 | |
| *** Drago has joined #senlin | 15:02 | |
| -openstackstatus- NOTICE: The infrastructure team is taking Gerrit offline for maintenance this afternoon, beginning shortly after 20:00 UTC. We aim to have it back online around 00:00 UTC. | 20:00 | |
| -openstackstatus- NOTICE: Gerrit is offline for maintenance until 00:00 UTC | 20:12 | |
| *** ChanServ changes topic to "Gerrit is offline for maintenance until 00:00 UTC" | 20:12 | |
| *** Drago has quit IRC | 20:52 | |
| *** Drago has joined #senlin | 21:07 | |
| *** Drago has quit IRC | 23:32 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!