Skip to content

Make pivot and repivot separate in feature test#639

Merged
fmuyassarov merged 1 commit into
metal3-io:masterfrom
Nordix:fetch-repivot-out-of-pivot
Apr 29, 2021
Merged

Make pivot and repivot separate in feature test#639
fmuyassarov merged 1 commit into
metal3-io:masterfrom
Nordix:fetch-repivot-out-of-pivot

Conversation

@kashifest
Copy link
Copy Markdown
Member

No description provided.

@metal3-io-bot metal3-io-bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Apr 23, 2021
@kashifest
Copy link
Copy Markdown
Member Author

/test-integration
/test-centos-integration
/test-features
/test-upgrade-features

@kashifest
Copy link
Copy Markdown
Member Author

/test-centos-integration

@kashifest kashifest force-pushed the fetch-repivot-out-of-pivot branch from 3f87919 to c85c3e7 Compare April 23, 2021 15:23
@kashifest
Copy link
Copy Markdown
Member Author

/test-integration
/test-centos-integration
/test-features
/test-upgrade-features

@kashifest
Copy link
Copy Markdown
Member Author

/test-features

Copy link
Copy Markdown
Member

@furkatgofurov7 furkatgofurov7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

In feature tests as it is now, we re-pivot after pivoting, which is not desirable as we need a way to test scale in & node reuse features right after pivoting. For that, this patch gets the re-pivot action out of pivoting, and so that re-pivoting will be called after node reuse tests.

@furkatgofurov7
Copy link
Copy Markdown
Member

/lgtm

@furkatgofurov7
Copy link
Copy Markdown
Member

/approve
/lgtm

@smoshiur1237
Copy link
Copy Markdown
Member

/lgtm

@fmuyassarov fmuyassarov merged commit fc641d6 into metal3-io:master Apr 29, 2021
@kashifest kashifest deleted the fetch-repivot-out-of-pivot branch September 29, 2021 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants