Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix devstack trove deploying failed #432

Merged
merged 1 commit into from
Feb 20, 2019
Merged

Fix devstack trove deploying failed #432

merged 1 commit into from
Feb 20, 2019

Conversation

kiwik
Copy link
Contributor

@kiwik kiwik commented Feb 20, 2019

  • OpenStack upstream have used tox to deploy trove in python3,
    OpenLab keep to use same way to do it.
  • OpenStack CI use 4 vCPUs and 8G RAM instances to run trove integration
    tests, OpenLab should be able to use same falvor, so remove nodepool
    label ubuntu-xenial-otc.

Closes: theopenlab/openlab#183

- OpenStack upstream have used tox to deploy trove in python3,
  OpenLab keep to use same way to do it.
- OpenStack CI use 4 vCPUs and 8G RAM instances to run trove integration
  tests, OpenLab should be able to use same falvor, so remove nodepool
  label `ubuntu-xenial-otc`.

Closes: theopenlab/openlab#183
@theopenlab-ci
Copy link

theopenlab-ci bot commented Feb 20, 2019

Build succeeded.

Copy link
Contributor

@liusheng liusheng left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@huangtianhua huangtianhua left a comment

Choose a reason for hiding this comment

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

thanks

@huangtianhua huangtianhua merged commit c56efb6 into master Feb 20, 2019
@huangtianhua huangtianhua deleted the fix-trove branch February 20, 2019 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants