Fuel for OpenStack

[prod-ci] LDAP jobs does not destroy envs after finish

Bug #1648715 reported by Vladimir Khlyunev
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Medium
Nikita Karpin
Fuel for OpenStack 9.2

Bug Description

UPD: https://github.com/fuel-infra/jenkins-jobs/blob/2bfdf50781e333c47e5a09ab0cd063a09009d6bc/common/macros.yaml#L19

https://github.com/fuel-infra/jenkins-jobs/blob/master/servers/product-ci/9.x/ldap/builders/run_plugin_ldap_deploy_test.sh#L69-L72

LDAP jobs leaves running VMs after finish which leads to lack of resources

(venv-nailgun-tests-2.9)vkhlyunev@srv71-bud:~$ virsh list
 Id Name State
----------------------------------------------------
 2773 9.x.ldap_deploy_test.ubuntu.plugin-ldap.mld_proxy.87.87_admin running
 2774 9.x.ldap_deploy_test.ubuntu.plugin-ldap.mld_proxy.87.87_slave-01 running
 2775 9.x.ldap_deploy_test.ubuntu.plugin-ldap.mld_proxy.87.87_slave-02 running
 2776 9.x.ldap_deploy_test.ubuntu.plugin-ldap.mld_proxy.87.87_slave-03 running
 2777 9.x.ldap_deploy_test.ubuntu.plugin-ldap.mld_proxy.87.87_slave-04 running
 2881 9.x.ldap_deploy_test.ubuntu.plugin-ldap.mld_proxy.88.88_admin running
 2882 9.x.ldap_deploy_test.ubuntu.plugin-ldap.mld_proxy.88.88_slave-01 running
 2883 9.x.ldap_deploy_test.ubuntu.plugin-ldap.mld_proxy.88.88_slave-02 running
 2884 9.x.ldap_deploy_test.ubuntu.plugin-ldap.mld_proxy.88.88_slave-03 running
 2885 9.x.ldap_deploy_test.ubuntu.plugin-ldap.mld_proxy.88.88_slave-04 running

even if 88 build is running 87 should be destroyed

See original description
Vladimir Khlyunev (vkhlyunev)
description: updated
Vladimir Khlyunev (vkhlyunev)
description: updated
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to fuel-infra/jenkins-jobs (master)

Reviewed: https://review.fuel-infra.org/29268
Submitter: Ivan Remizov <email address hidden>
Branch: master

Commit: 3c12627a6ccb1a1d10758d59d383fc82572268bc
Author: Ivan Remizov <email address hidden>
Date: Fri Dec 9 07:33:23 2016

Move Ldap systest to separate label

LDAP systests works incorrectly and pollutes envs.
We need to move it to separate label until it will
be fixed.

Otherwise product swarms are failing

Partial-Bug: #1648715
Change-Id: I28e557c4c7efcdd0d9fb8c6671091dddd5a2055d

Revision history for this message
Roman Vyalov (r0mikiam) wrote :

move priority to medium because a workaround was applied

Changed in fuel:
importance: Critical → Medium
Roman Vyalov (r0mikiam)
Changed in fuel:
status: New → Fix Committed
Dmitry Kaigarodеsev (dkaiharodsev)
Changed in fuel:
status: Fix Committed → Confirmed
assignee: Fuel CI (fuel-ci) → Dmitry Kaigarodеsev (dkaiharodsev)
Tatyana Borukhovich (tatyanaborukhovich)
tags: added: swarm-blocker
Revision history for this message
Dmitry Kaigarodеsev (dkaiharodsev) wrote :

after discussion with Nastya Urlapova we decide to disable related LDAP swarm treads:
https://product-ci.infra.mirantis.net/job/9.x.plugin-ldap.runner/

Changed in fuel:
status: Confirmed → Fix Committed
Nikita Karpin (mkarpin)
Changed in fuel:
status: Fix Committed → Confirmed
assignee: Dmitry Kaigarodеsev (dkaiharodsev) → Nikita Karpin (mkarpin)
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to fuel-infra/jenkins-jobs (master)

Fix proposed to branch: master
Change author: Mykyta Karpin <email address hidden>
Review: https://review.fuel-infra.org/29631

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to fuel-infra/jenkins-jobs (master)

Reviewed: https://review.fuel-infra.org/29631
Submitter: Ivan Remizov <email address hidden>
Branch: master

Commit: 1447a0f1bfd53f6596ef14ebd17a8a3b5af7f947
Author: Mykyta Karpin <email address hidden>
Date: Fri Dec 30 13:56:54 2016

Fix post-destroy-vms macro

post-destroy-vms(always) macro uses publisher post-tasks which
generates post build action with condition containing empty log-text
field. The result of that condition is always False and thus destroying
of vms is skipped.

This patch fixes this by checking log for 'Started' string presence.

Change-Id: I6cd7139ed9d731bec247c85b2ccb3eaa37615b7e
Closes-Bug: #1648715

Changed in fuel:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.