Skip to content

Commit

Permalink
Merge pull request #157 from galaxyproject/add_github_action
Browse files Browse the repository at this point in the history
Added github action for tests and use cloudbridge full package
  • Loading branch information
nuwang authored Sep 15, 2022
2 parents d32b0f8 + c6c86e0 commit 4b445a2
Show file tree
Hide file tree
Showing 9 changed files with 47 additions and 51 deletions.
27 changes: 27 additions & 0 deletions .github/workflows/tests.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
name: Run tests
on:
push:
branches:
- master
pull_request:
branches:
- master
jobs:
build:
name: tests
runs-on: ubuntu-latest
strategy:
fail-fast: true
steps:
- name: 'Set up Python'
uses: actions/setup-python@v2
with:
python-version: '3.8'
- uses: actions/checkout@v2
- run: pip install tox
- run: pip install tox-docker>=2.0.0a3
- run: pip install coveralls
- run: tox -e py38-integration
- run: coveralls -v --service=github
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
32 changes: 0 additions & 32 deletions .travis.yml

This file was deleted.

8 changes: 4 additions & 4 deletions README.rst
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
CloudMan is a cloud infrastructure and application manager, primarily for Galaxy.

.. image:: https://github.com/galaxyproject/cloudman/actions/workflows/tests.yaml/badge.svg
:target: https://github.com/galaxyproject/cloudman/actions/workflows/tests.yaml
:alt: Github Build Status

.. image:: https://coveralls.io/repos/github/galaxyproject/cloudman/badge.svg?branch=master
:target: https://coveralls.io/github/galaxyproject/cloudman?branch=master
:alt: Test Coverage Report

.. image:: https://travis-ci.org/galaxyproject/cloudman.svg?branch=master
:target: https://travis-ci.org/galaxyproject/cloudman
:alt: Travis Build Status

Installation
------------
CloudMan is intended to be installed via the `CloudMan Helm chart`_.
Expand Down
2 changes: 1 addition & 1 deletion cloudman/cloudman/urls.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
1. Add an import: from other_app.views import Home
2. Add a URL to urlpatterns: url(r'^$', Home.as_view(), name='home')
Including another URLconf
1. Import the include() function: from django.conf.urls import url, include
1. Import the include() function: from django.urls import re_path, include
2. Add a URL to urlpatterns: url(r'^blog/', include('blog.urls'))
"""
from django.conf import settings
Expand Down
2 changes: 1 addition & 1 deletion cloudman/clusterman/tests/test_cluster_api.py
Original file line number Diff line number Diff line change
Expand Up @@ -1301,7 +1301,7 @@ def test_scale_up_unschedulable(self):
# Ensure that the created node has the correct size
vm_types = self._get_cluster_node_vm_types(cluster_id)
self.assertEqual(len(vm_types), 1)
self.assertTrue("i3en.24xlarge" in vm_types)
self.assertTrue("i3en.24xlarge" in vm_types or 'dl1.24xlarge' in vm_types)

def test_scale_up_allowed_vm_types(self):
# create the parent cluster
Expand Down
8 changes: 4 additions & 4 deletions cloudman/clusterman/urls.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
"""CloudMan Create URL configuration."""

from django.conf.urls import include
from django.conf.urls import url
from django.urls import include
from django.urls import re_path

from . import views
from djcloudbridge.drf_routers import HybridDefaultRouter
Expand All @@ -28,6 +28,6 @@

cluster_regex_pattern = r'^'
urlpatterns = [
url(r'^', include(router.urls)),
url(cluster_regex_pattern, include(cluster_router.urls))
re_path(r'^', include(router.urls)),
re_path(cluster_regex_pattern, include(cluster_router.urls))
]
6 changes: 3 additions & 3 deletions cloudman/helmsman/urls.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
"""CloudMan Create URL configuration."""

from django.conf.urls import include
from django.conf.urls import url
from django.urls import include
from django.urls import re_path

from . import views
from djcloudbridge.drf_routers import HybridDefaultRouter
Expand All @@ -20,5 +20,5 @@
app_name = "helmsman"

urlpatterns = [
url(r'^', include(router.urls)),
re_path(r'^', include(router.urls)),
]
8 changes: 4 additions & 4 deletions cloudman/projman/urls.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
"""CloudMan Create URL configuration."""

from django.conf.urls import include
from django.conf.urls import url
from django.urls import include
from django.urls import re_path

from . import views

Expand All @@ -22,6 +22,6 @@

cluster_regex_pattern = r'^'
urlpatterns = [
url(r'^', include(router.urls)),
url(cluster_regex_pattern, include(project_router.urls))
re_path(r'^', include(router.urls)),
re_path(cluster_regex_pattern, include(project_router.urls))
]
5 changes: 3 additions & 2 deletions requirements_test.txt
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,12 @@ paramiko
sqlalchemy
# required by moto
sshpubkeys
git+https://github.com/CloudVE/moto@fix_unknown_instance_type
git+https://github.com/CloudVE/cloudbridge
git+https://github.com/CloudVE/cloudbridge#egg=cloudbridge[dev]
git+https://github.com/CloudVE/djcloudbridge
# Leave cloudlaunch-cli before cloudlaunch-server due to coreapi version mismatch
git+https://github.com/CloudVE/cloudlaunch-cli
git+https://github.com/galaxyproject/cloudlaunch
pytz>=2022.2.1
tzdata>=2022.2
-r requirements.txt
-e ".[test]"

0 comments on commit 4b445a2

Please sign in to comment.