Skip to content

Commit

Permalink
PMM-13232: Remove debug commands
Browse files Browse the repository at this point in the history
  • Loading branch information
peterSirotnak committed Jul 12, 2024
1 parent e4cc444 commit e19a324
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 12 deletions.
2 changes: 1 addition & 1 deletion tests/pages/adminPage.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
const { I } = inject();
const assert = require('assert');

const pathToPmmQaRepo = process.env.PATH_TO_PMM_QA || '/home/runner/work/pmm-qa/pmm-qa/qa-integration/pmm_qa';
const pathToPmmQaRepo = process.env.PATH_TO_PMM_QA || '/srv/pmm-qa';

module.exports = {
url: 'graph/d/pmm-home/home-dashboard?orgId=1',
Expand Down
2 changes: 0 additions & 2 deletions tests/pages/api/inventoryAPI.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,6 @@ module.exports = {
for (let i = 0; i < 60; i++) {
const services = await this.apiGetServices(service.serviceType);

console.log(services.data);

responseService = services.data[service.service].find((obj) => obj.service_name === serviceName);
if (responseService !== undefined) break;

Expand Down
9 changes: 0 additions & 9 deletions tests/verifyTLSPostgresRemoteInstance_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,6 @@ Data(instances).Scenario(
let details;
const remoteServiceName = `remote_${serviceName}`;

console.log(`Remote service name is: ${remoteServiceName}`);

if (serviceType === 'postgres_ssl') {
details = {
serviceName: remoteServiceName,
Expand Down Expand Up @@ -91,8 +89,6 @@ Data(instances).Scenario(
container,
} = current;

console.log(`Current container name is: ${container}`);

// Verify user is able to add service with --tls-skip-verify option
const responseMessage = 'PostgreSQL Service added.';
const command = `docker exec ${container} pmm-admin add postgresql --username=pmm --password=pmm --query-source="pgstatements" --tls --tls-skip-verify ${noSslCheckServiceName}`;
Expand All @@ -119,13 +115,8 @@ Data(instances).Scenario(
// verify metric for client container node instance
const localServiceName = await I.verifyCommand(`docker exec ${container} pmm-admin list | grep "PostgreSQL" | grep "ssl_service" | awk -F " " '{print $2}'`);

console.log(`Local Service name is: ${localServiceName}`);
console.log(await I.verifyCommand(`docker exec ${container} pmm-admin list`));

response = await grafanaAPI.checkMetricExist(metric, { type: 'service_name', value: localServiceName });

console.log(response.data);

result = JSON.stringify(response.data.data.result);

assert.ok(response.data.data.result.length !== 0, `Metrics ${metric} from ${serviceName} should be available but got empty ${result}`);
Expand Down

0 comments on commit e19a324

Please sign in to comment.