[devops] move time sync and upload manifests to dos.py

Bug #1318641 reported by Bogdan Dobrelya
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Opinion
Medium
Kirill Omelchenko

Bug Description

In order to improve manual testing speed, such methods as
sync_node_time https://github.com/stackforge/fuel-main/blob/master/fuelweb_test/models/environment.py#L361
upload_manifests https://github.com/stackforge/fuel-main/blob/master/fuelweb_test/helpers/decorators.py#L71
should be moved to the devops in order to make it possible to call them from dos.py as well.

Fuelweb tests then would have to just call them from devops w/o any other changes in its logic, but CLI would be greatly enhaced to allow faster manual testing workflow for non-typical environments (which are not covered by the existing deployment cases), e.g.:
1) deploy non-typical env (combined roles, node numbers etc.)
2) provision it with Fuel CLI
3) dos.py suspend & snapshot & resume it
4) deploy it with Fuel CLI (or test the deployment process as well)
5) ... test something unstable or broken ...
6) dos.py revert & resume it
7) dos.py sync_node & upload_manifests it (*currently is missing, and should be added as a subj of the issue*)
8) repeat (4)-(7)

Changed in fuel:
importance: Undecided → High
tags: removed: low-hanging-fruit
Dmitry Ilyin (idv1985)
summary: - move time sync and upload manifests to devops
+ [devops] move time sync and upload manifests to devops
Changed in fuel:
milestone: 5.1 → 6.0
importance: High → Medium
Changed in fuel:
assignee: Fuel QA Team (fuel-qa) → Kirill Omelchenko (komelchenko)
Revision history for this message
Igor Shishkin (teran) wrote : Re: [devops] move time sync and upload manifests to devops

Guys, basically I don't think that functionality should be moved to fuel-devops.
fuel-devops primary role is to control sets of virtual machines. Not matter what OS there.
In fact you could install enviroment with 10xWindows nodes or DOS nodes and that's what fuel-devops should do.

tags: added: fuel-devops
summary: - [devops] move time sync and upload manifests to devops
+ [devops] move time sync and upload manifests to dos.py
Revision history for this message
Igor Shishkin (teran) wrote :

Nastya dos.py - simple wrapper. What is done in related bug doesn't relates to dos.py directly.

Revision history for this message
Bogdan Dobrelya (bogdando) wrote :

>fuel-devops primary role is to control sets of virtual machines. Not matter what OS there.
>In fact you could install enviroment with 10xWindows nodes or DOS nodes and that's what fuel-devops should do.

I believe, time sync and upload manifests is *very* nice to have at this devops layer as well, and devops could provide some 'drivers' for guests specific layer (at least for Centos and Ubuntu :-) ) to make it happen...

Revision history for this message
Nastya Urlapova (aurlapova) wrote :

Yep, dos.py is exactly simple wrapper and there is we extend it functionality.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-devops (master)

Reviewed: https://review.openstack.org/118372
Committed: https://git.openstack.org/cgit/stackforge/fuel-devops/commit/?id=5caad8e8353c274a7be715ee48e19df672624f00
Submitter: Jenkins
Branch: master

commit 5caad8e8353c274a7be715ee48e19df672624f00
Author: Kirill Omelchenko <email address hidden>
Date: Tue Sep 2 17:35:48 2014 +0300

    Add time-sync functionality to dos.py

    Add functionality to sync time on
    specific env nodes on demand using dos.py

    Related-Bug: #1318641

    Change-Id: If26abc316a80e31fb575fec0ebffa1685025d556

Changed in fuel:
status: Triaged → Fix Committed
Revision history for this message
Bogdan Dobrelya (bogdando) wrote :

Please update the docs before to consider the bug closed

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

Other bug subscribers

Remote bug watches

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