Skip to content

Commit 70715da

Browse files
authored
Updates fix pgsm support (#109)
1 parent 136d172 commit 70715da

File tree

3 files changed

+8
-10
lines changed

3 files changed

+8
-10
lines changed

pmm_qa/pdpgsql_pgsm_setup.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
pgstat_monitor_branch: "{{ lookup('vars', 'extra_pgstat_monitor_branch', default=lookup('env','PGSTAT_MONITOR_BRANCH') | default('2.1.0', true) ) }}"
1616
pgstat_monitor_repo: "{{ lookup('vars', 'extra_pgstat_monitor_repo', default=lookup('env','PGSTAT_MONITOR_REPO') | default('percona/pg_stat_monitor', true) ) }}"
1717
use_socket: "{{ lookup('vars', 'extra_pdpgsql_version', default=lookup('env','USE_SOCKET') | default('', true) ) }}"
18-
port: "{{ lookup('vars', 'extra_pdpgsql_port', default=lookup('env','PORT') | default(5447, true) ) }}"
18+
pdpgsql_pgsm_port: "{{ lookup('vars', 'extra_pdpgsql_port', default=lookup('env','PDPGSQL_PGSM_PORT') | default(5447, true) ) }}"
1919
distribution: "{{ lookup('vars', 'extra_pdpgsql_distribution', default=lookup('env','DISTRIBUTION') | default('PPG', true) ) }}"
2020

2121
tasks:
@@ -33,7 +33,7 @@
3333
- name: Prepare Container for PostgreSQL
3434
shell: >
3535
sudo docker run -d --name={{ pdpgsql_pgsm_container }}
36-
-p {{ port }}:5432
36+
-p {{ pdpgsql_pgsm_port }}:5432
3737
phusion/baseimage:jammy-1.0.1
3838
3939
- name: Copy all required Artifacts to the docker pdpgsql_pgsm_container

pmm_qa/pgsql_pgss_setup.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
vars:
99
pgsql_version: "{{ lookup('vars', 'extra_pgsql_version', default=lookup('env','PGSQL_VERSION') | default('14', true) ) }}"
1010
pgsql_pgss_container: "{{ lookup('vars', 'extra_pgsql_pgss_container', default=lookup('env','PGSQL_PGSS_CONTAINER') | default('pgsql_pgss', true) ) }}"
11-
pgsql_pgss_port: "{{ lookup('vars', 'extra_pgsql_port', default=lookup('env','PGSQL_PGSS_PORT') | default('5438', true) ) }}"
11+
pgsql_pgss_port: "{{ lookup('vars', 'extra_pgsql_port', default=lookup('env','PGSQL_PGSS_PORT') | default('5448', true) ) }}"
1212
pmm_server_ip: "{{ lookup('vars', 'extra_pmm_server_ip', default=lookup('env','PMM_SERVER_IP') | default('127.0.0.1', true) ) }}"
1313
client_version: "{{ lookup('vars', 'extra_client_version', default=lookup('env','CLIENT_VERSION') | default('dev-latest', true) ) }}"
1414
admin_password: "{{ lookup('vars', 'extra_admin_password', default=lookup('env','ADMIN_PASSWORD') | default('admin', true) ) }}"

pmm_qa/pmm-framework.py

+5-7
Original file line numberDiff line numberDiff line change
@@ -284,7 +284,7 @@ def setup_pdpgsql(db_type, db_version=None, db_config=None, args=None):
284284
'CLIENT_VERSION': get_value('CLIENT_VERSION', db_type, args, db_config),
285285
'USE_SOCKET': get_value('USE_SOCKET', db_type, args, db_config),
286286
'ADMIN_PASSWORD': os.getenv('ADMIN_PASSWORD') or args.pmm_server_password or 'admin',
287-
'PORT': 5447,
287+
'PDPGSQL_PGSM_PORT': 5447,
288288
'DISTRIBUTION': '',
289289
'PMM_QA_GIT_BRANCH': os.getenv('PMM_QA_GIT_BRANCH') or 'v3'
290290
}
@@ -337,20 +337,18 @@ def setup_pgsql(db_type, db_version=None, db_config=None, args=None):
337337

338338
# Define environment variables for playbook
339339
env_vars = {
340-
'PDPGSQL_VERSION': pgsql_version,
341-
'PGSTAT_MONITOR_BRANCH': 'main',
340+
'PGSQL_VERSION': pgsql_version,
342341
'PMM_SERVER_IP': args.pmm_server_ip or container_name or '127.0.0.1',
343-
'PDPGSQL_PGSM_CONTAINER': 'pgsql_pgsm_pmm_' + str(pgsql_version),
342+
'PGSQL_PGSM_CONTAINER': 'pgsql_pgss_pmm_' + str(pgsql_version),
344343
'CLIENT_VERSION': get_value('CLIENT_VERSION', db_type, args, db_config),
345344
'USE_SOCKET': get_value('USE_SOCKET', db_type, args, db_config),
346345
'ADMIN_PASSWORD': os.getenv('ADMIN_PASSWORD') or args.pmm_server_password or 'admin',
347-
'PORT': 5448,
348-
'DISTRIBUTION': 'PGDG',
346+
'PGSQL_PGSS_PORT': 5448,
349347
'PMM_QA_GIT_BRANCH': os.getenv('PMM_QA_GIT_BRANCH') or 'v3'
350348
}
351349

352350
# Ansible playbook filename
353-
playbook_filename = 'pdpgsql_pgsm_setup.yml'
351+
playbook_filename = 'pgsql_pgss_setup.yml'
354352

355353
# Call the function to run the Ansible playbook
356354
run_ansible_playbook(playbook_filename, env_vars, args)

0 commit comments

Comments
 (0)