-
Notifications
You must be signed in to change notification settings - Fork 20
add pulp services #34
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
Changes from 16 commits
8577568
266dfc6
1d08e23
9f99f69
06378a7
e9541cc
94328d4
a36e0ac
c0a052c
ced9c95
992a6f8
675a2ee
846263c
84a5a67
f87525f
78ca43a
3a92635
3d1f220
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,4 @@ | ||
| httpd_ssl_dir: /etc/pki/httpd | ||
| httpd_pulp_api_backend: http://localhost:8080 | ||
| httpd_pulp_content_backend: http://localhost:8080 | ||
| httpd_pulp_api_backend: http://localhost:24817 | ||
| httpd_pulp_content_backend: http://localhost:24816 | ||
| httpd_foreman_backend: http://localhost:3000 |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,10 +1,18 @@ | ||
| --- | ||
| pulp_image: quay.io/pulp/pulp:stable | ||
| pulp_ports: | ||
| - "8080:80" | ||
| pulp_image: quay.io/pulp/pulp-minimal:stable | ||
| pulp_api_image: "{{ pulp_image }}" | ||
| pulp_content_image: "{{ pulp_image }}" | ||
| pulp_worker_image: "{{ pulp_image }}" | ||
|
|
||
| pulp_api_ports: | ||
| - "24817:80" | ||
| pulp_content_ports: | ||
| - "24816:80" | ||
ehelms marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| pulp_worker_count: 2 | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is also unused, but I think this could still be needed to tune the worker count. |
||
|
|
||
| pulp_volumes: | ||
| - /var/lib/pulp/settings:/etc/pulp:Z | ||
| - /var/lib/pulp/pulp_storage:/var/lib/pulp:Z | ||
| - /var/lib/pulp/pgsql:/var/lib/pgsql:Z | ||
| - /var/lib/pulp/containers:/var/lib/containers:Z | ||
| pulp_container_name: pulp | ||
| - /var/lib/pulp:/var/lib/pulp | ||
|
|
||
| pulp_api_container_name: pulp-api | ||
| pulp_content_container_name: pulp-content | ||
| pulp_worker_container_name: pulp-worker | ||
| Original file line number | Diff line number | Diff line change | ||
|---|---|---|---|---|
| @@ -1,7 +1,20 @@ | ||||
| CONTENT_ORIGIN="http://{{ ansible_hostname }}:8080" | ||||
| CONTENT_ORIGIN="http://{{ ansible_fqdn }}:24816" | ||||
| CACHE_ENABLED=True | ||||
| REDIS_HOST="localhost" | ||||
| REDIS_PORT=6379 | ||||
ekohl marked this conversation as resolved.
Show resolved
Hide resolved
|
||||
| REDIS_DB=8 | ||||
|
|
||||
| DATABASES = { | ||||
| 'default': { | ||||
| 'ENGINE': 'django.db.backends.postgresql', | ||||
| 'NAME': 'pulp', | ||||
| 'USER': 'pulp', | ||||
| 'PASSWORD': '{{ pulp_db_password }}', | ||||
| 'HOST': 'localhost', | ||||
evgeni marked this conversation as resolved.
Show resolved
Hide resolved
|
||||
| 'PORT': '5432', | ||||
|
||||
| 'PORT': '5432', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm a bit unsure why my suggestion wasn't followed. IMHO the config should be as short as possible and not copy all the defaults.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no, i did updated it 3d1f220
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My suggestion was to remove the line.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah, i understand now! sorry, updating it in another PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,38 +1,45 @@ | ||
| import json | ||
|
|
||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This empty line is common in Python. It separates the built in modules ( |
||
| import pytest | ||
|
|
||
|
|
||
| PULP_HOST = 'localhost' | ||
| PULP_PORT = 8080 | ||
|
|
||
| PULP_API_PORT = 24817 | ||
| PULP_CONTENT_PORT = 24816 | ||
|
|
||
| @pytest.fixture(scope="module") | ||
| def pulp_status_curl(host): | ||
| return host.run(f"curl -k -s -w '%{{stderr}}%{{http_code}}' http://{PULP_HOST}:{PULP_PORT}/pulp/api/v3/status/") | ||
|
|
||
| return host.run(f"curl -k -s -w '%{{stderr}}%{{http_code}}' http://{PULP_HOST}:{PULP_API_PORT}/pulp/api/v3/status/") | ||
|
|
||
| @pytest.fixture(scope="module") | ||
| def pulp_status(pulp_status_curl): | ||
| return json.loads(pulp_status_curl.stdout) | ||
|
|
||
| def test_pulp_api_service(host): | ||
| pulp_api = host.service("pulp-api") | ||
| assert pulp_api.is_running | ||
| assert pulp_api.is_enabled | ||
|
|
||
| def test_pulp_service(host): | ||
| pulp = host.service("pulp") | ||
| assert pulp.is_running | ||
| assert pulp.is_enabled | ||
| def test_pulp_content_service(host): | ||
| pulp_content = host.service("pulp-content") | ||
| assert pulp_content.is_running | ||
| assert pulp_content.is_enabled | ||
|
|
||
| def test_pulp_worker_service(host): | ||
| pulp_worker = host.service("pulp-worker") | ||
| assert pulp_worker.is_running | ||
| assert pulp_worker.is_enabled | ||
|
|
||
| def test_pulp_port(host): | ||
| pulp = host.addr(PULP_HOST) | ||
| assert pulp.port(PULP_PORT).is_reachable | ||
| def test_pulp_api_port(host): | ||
| pulp_api = host.addr(PULP_HOST) | ||
| assert pulp_api.port(PULP_API_PORT).is_reachable | ||
|
|
||
| def test_pulp_content_port(host): | ||
| pulp_content = host.addr(PULP_HOST) | ||
| assert pulp_content.port(PULP_CONTENT_PORT).is_reachable | ||
|
|
||
| def test_pulp_status(pulp_status_curl): | ||
| assert pulp_status_curl.succeeded | ||
| assert pulp_status_curl.stderr == '200' | ||
|
|
||
|
|
||
| def test_pulp_status_database_connection(pulp_status): | ||
| assert pulp_status['database_connection']['connected'] | ||
|
|
||
|
|
@@ -54,6 +61,9 @@ def test_pulp_status_workers(pulp_status): | |
|
|
||
| @pytest.mark.xfail(reason='password auth is deactivated when we use cert auth') | ||
| def test_pulp_admin_auth(host): | ||
| cmd = host.run(f"curl --silent --write-out '%{{stderr}}%{{http_code}}' --user admin:CHANGEME http://{PULP_HOST}:{PULP_PORT}/pulp/api/v3/users/") | ||
| cmd = host.run(f"curl --silent --write-out '%{{stderr}}%{{http_code}}' --user admin:CHANGEME http://{PULP_HOST}:{PULP_API_PORT}/pulp/api/v3/users/") | ||
| assert cmd.succeeded | ||
| assert cmd.stderr == '200' | ||
|
|
||
| def test_pulp_volumes(host): | ||
| assert host.file("/var/lib/pulp").is_directory | ||
Uh oh!
There was an error while loading. Please reload this page.