Default deploy-changes does not use graph execution

Bug #1620620 reported by Vladimir Kuklin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
High
Bulat Gaifullin
Mitaka
Fix Released
High
Stanislaw Bogatkin
Newton
Fix Committed
High
Bulat Gaifullin

Bug Description

With graph concept extension feature we allow a user to run a sequence of graphs as a 'scenario' and thus unhardcode things in Nailgun such as 'deploy-changes'. But this functionality is not actually used by our current code - it seems we forgot to switch to this mechanism for new 9.1 releases.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

Fix proposed to branch: master
Review: https://review.openstack.org/366121

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-fuelclient (master)

Fix proposed to branch: master
Review: https://review.openstack.org/366688

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/366121
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=78d04a5ead60a51a964839a777f344ce882f5d6f
Submitter: Jenkins
Branch: master

commit 78d04a5ead60a51a964839a777f344ce882f5d6f
Author: Bulat Gaifullin <email address hidden>
Date: Mon Sep 5 15:08:20 2016 +0300

    Itroduced deployment sequences

    Deployment sequence is a mechanism for saving
    recently used sequences of graphs in database
    for further execution.
    Sequences are related to release object.

    Partial-Bug: 1620620
    Change-Id: I23ce74e8edbab579d6704a947d89e8d6e9c58fb7

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

Fix proposed to branch: master
Review: https://review.openstack.org/367322

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/367322
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=a022fd74ac5a6d420ded06499b2ebf77ee686f4f
Submitter: Jenkins
Branch: master

commit a022fd74ac5a6d420ded06499b2ebf77ee686f4f
Author: Bulat Gaifullin <email address hidden>
Date: Thu Sep 8 14:19:10 2016 +0300

    Filter sequences output by release or cluster

    It will be more usefull to get sequences for release or for cluster

    Change-Id: Ia2efc30050283da5653a1bbc23aeaf9f4de6a13b
    Partial-Bug: 1620620

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/367594

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-fuelclient (master)

Reviewed: https://review.openstack.org/366688
Committed: https://git.openstack.org/cgit/openstack/python-fuelclient/commit/?id=bac19f24b365ced6a0f39f4c4b4ead48c79969c1
Submitter: Jenkins
Branch: master

commit bac19f24b365ced6a0f39f4c4b4ead48c79969c1
Author: Bulat Gaifullin <email address hidden>
Date: Wed Sep 7 14:24:52 2016 +0300

    Added commands to manage deployment sequences

    Commands:
      - fuel2 sequence create -r <release> -n <name> -t <graph_type1> [graph_typeN]
      - fuel2 sequence upload -r <release> --file <file_path>
      - fuel2 sequence download <id> [--file <file path>]
      - fuel2 sequence delete <id>
      - fuel2 sequence update <id> [-name <name>] [-t <graph_type1> [graph_typeN]]
      - fuel2 sequence list -r <release_id> | -e <env_id>
      - fuel2 sequence show <id>
      - fuel2 sequence execute <id> -e <env_id> [--force] [--dry-run] [--noop]

    DocImpact
    Change-Id: I6eb688c5cc91b2b3dbaa2fe5c52a69fe062da664
    Partial-Bug: 1620620

tags: added: in-stable-mitaka
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (stable/mitaka)

Reviewed: https://review.openstack.org/367594
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=f6593c8d4210505b422d0b4425eff5994ad3ef4d
Submitter: Jenkins
Branch: stable/mitaka

commit f6593c8d4210505b422d0b4425eff5994ad3ef4d
Author: Bulat Gaifullin <email address hidden>
Date: Mon Sep 5 15:08:20 2016 +0300

    Itroduced deployment sequences

    Deployment sequence is a mechanism for saving
    recently used sequences of graphs in database
    for further execution.
    Sequences are related to release object.

    Partial-Bug: 1620620
    Change-Id: I23ce74e8edbab579d6704a947d89e8d6e9c58fb7

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/367908

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-fuelclient (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/367911

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-fuelclient (stable/mitaka)

Reviewed: https://review.openstack.org/367911
Committed: https://git.openstack.org/cgit/openstack/python-fuelclient/commit/?id=e6640344d4eac03c4f4b21e8b3a2c791a616d10d
Submitter: Jenkins
Branch: stable/mitaka

commit e6640344d4eac03c4f4b21e8b3a2c791a616d10d
Author: Bulat Gaifullin <email address hidden>
Date: Wed Sep 7 14:24:52 2016 +0300

    Added commands to manage deployment sequences

    Commands:
      - fuel2 sequence create -r <release> -n <name> -t <graph_type1> [graph_typeN]
      - fuel2 sequence upload -r <release> --file <file_path>
      - fuel2 sequence download <id> [--file <file path>]
      - fuel2 sequence delete <id>
      - fuel2 sequence update <id> [-name <name>] [-t <graph_type1> [graph_typeN]]
      - fuel2 sequence list -r <release_id> | -e <env_id>
      - fuel2 sequence show <id>
      - fuel2 sequence execute <id> -e <env_id> [--force] [--dry-run] [--noop]

    DocImpact
    Change-Id: I6eb688c5cc91b2b3dbaa2fe5c52a69fe062da664
    Partial-Bug: 1620620

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (stable/mitaka)

Reviewed: https://review.openstack.org/367908
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=eec5d43728b5517afb5d92b3cabf7f660d1368c4
Submitter: Jenkins
Branch: stable/mitaka

commit eec5d43728b5517afb5d92b3cabf7f660d1368c4
Author: Bulat Gaifullin <email address hidden>
Date: Thu Sep 8 14:19:10 2016 +0300

    Filter sequences output by release or cluster

    It will be more usefull to get sequences for release or for cluster

    Change-Id: Ia2efc30050283da5653a1bbc23aeaf9f4de6a13b
    Partial-Bug: 1620620
    (cherry picked from commit a022fd74ac5a6d420ded06499b2ebf77ee686f4f)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

Fix proposed to branch: master
Review: https://review.openstack.org/368781

Changed in fuel:
assignee: Bulat Gaifullin (bgaifullin) → Stanislaw Bogatkin (sbogatkin)
Changed in fuel:
assignee: Stanislaw Bogatkin (sbogatkin) → Bulat Gaifullin (bgaifullin)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/369902

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (stable/mitaka)

Reviewed: https://review.openstack.org/369902
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=142bd5b413897f17461344b4f605dcade3a32272
Submitter: Jenkins
Branch: stable/mitaka

commit 142bd5b413897f17461344b4f605dcade3a32272
Author: Bulat Gaifullin <email address hidden>
Date: Mon Sep 12 14:58:00 2016 +0300

    Enable using new transaction manager for old handlers

    DeployChanges will use the sequence 'deploy-changes' if it is available.
    DeployNodes and ProvisionNodes will be use apropriate graphs if it is available.

    Change-Id: Icb66826ce96c41f77d85d97990b4cc600aa5fe04
    Closes-Bug: 1620620

Revision history for this message
Bulat Gaifullin (bulat.gaifullin) wrote :

sequences and graphs are needed for this fix was not committed to library.
for testing this feature need to upload them manually.
sequences: "deploy_changes"
graphs: "deletion", "net-verification", "provision", "default"

tags: added: on-verification
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/368781
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=17ca94d09fe7e41ad40e791ca2c868e8ea344990
Submitter: Jenkins
Branch: master

commit 17ca94d09fe7e41ad40e791ca2c868e8ea344990
Author: Bulat Gaifullin <email address hidden>
Date: Mon Sep 12 14:58:00 2016 +0300

    Enable using new transaction manager for old handlers

    DeployChanges will use the sequence 'deploy-changes' if it is available.
    DeployNodes and ProvisionNodes will be use apropriate graphs if it is available.

    Change-Id: Icb66826ce96c41f77d85d97990b4cc600aa5fe04
    Closes-Bug: 1620620

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
TatyanaGladysheva (tgladysheva) wrote :

Verified on 9.1 snapshot #270.

tags: removed: on-verification
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-web 10.0.0rc1

This issue was fixed in the openstack/fuel-web 10.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-web 10.0.0

This issue was fixed in the openstack/fuel-web 10.0.0 release.

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.