Skip to content

Commit

Permalink
Merge branch 'main' into v3
Browse files Browse the repository at this point in the history
  • Loading branch information
ademidoff committed Dec 1, 2023
2 parents 9b44bbb + 9db1746 commit 5aa6e6b
Show file tree
Hide file tree
Showing 103 changed files with 13,049 additions and 816 deletions.
156 changes: 156 additions & 0 deletions .github/workflows/api-tests.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,156 @@
name: 'API'

on:
# NOTE: no action on `push` and `pull_request` since the API tests need to be run
# against a container that has the latest changes from your code. Therefore, the
# PMM_SERVER_IMAGE should either be the one from your feature build or the local
# devcontainer. One more scenario is when we want to run the API tests on dev-latest
# to see if the tests are in a good shape.
# That said, this workflow is mostly a convenience if you prefer Github to Jenkins.
# https://github.com/Percona-Lab/jenkins-pipelines/blob/master/pmm/pmm2-api-tests.groovy

workflow_dispatch:
inputs:
MYSQL_IMAGE:
description: "MYSQL image version"
default: "percona:5.7"
required: true
type: string
POSTGRESQL_IMAGE:
description: "POSTGRESQL image version"
default: "postgres:12"
required: true
type: string
MONGODB_IMAGE:
description: "MONGODB image version"
default: "percona/percona-server-mongodb:4.4"
required: true
type: string
PMM_SERVER_IMAGE:
description: "PMM Server image version"
default: "perconalab/pmm-server:dev-latest"
required: true
type: string

workflow_call:
inputs:
BRANCH:
description: "The branch to source API tests from"
default: "main"
required: true
type: string
MYSQL_IMAGE:
description: "MYSQL image version"
default: "percona:5.7"
required: true
type: string
POSTGRESQL_IMAGE:
description: "POSTGRESQL image version"
default: "postgres:12"
required: true
type: string
MONGODB_IMAGE:
description: "MONGODB image version"
default: "percona/percona-server-mongodb:4.4"
required: true
type: string
PMM_SERVER_IMAGE:
description: "PMM Server image version"
default: "perconalab/pmm-server:dev-latest"
required: true
type: string

jobs:
test:
name: Tests
runs-on: ubuntu-22.04
env:
PMM_URL: http://admin:[email protected]
BRANCH: ${{ github.event.inputs.BRANCH || 'main' }}
MYSQL_IMAGE: ${{ github.event.inputs.MYSQL_IMAGE || 'percona:5.7' }}
POSTGRESQL_IMAGE: ${{ github.event.inputs.POSTGRESQL_IMAGE || 'postgres:12' }}
MONGODB_IMAGE: ${{ github.event.inputs.MONGODB_IMAGE || 'percona/percona-server-mongodb:4.4' }}
PMM_SERVER_IMAGE: ${{ github.event.inputs.PMM_SERVER_IMAGE || 'perconalab/pmm-server:dev-latest' }}

steps:
- name: Check out code
uses: actions/checkout@v4
with:
ref: ${{ env.BRANCH }}

- name: Login to docker.io registry
uses: docker/login-action@v3
with:
registry: docker.io
username: ${{ secrets.DOCKERHUB_USERNAME }}
password: ${{ secrets.DOCKERHUB_PASSWORD }}

- name: Run PMM Server
run: |
cat <<-EOF > compose.pmm-server-test.yml
services:
pmm-server:
image: ${{ env.PMM_SERVER_IMAGE }}
container_name: pmm-agent_pmm-server
environment:
- PMM_DEBUG=1
- PERCONA_TEST_CHECKS_INTERVAL=10s
- PERCONA_TEST_PLATFORM_ADDRESS=https://check-dev.percona.com
- PERCONA_TEST_PLATFORM_PUBLIC_KEY=RWTg+ZmCCjt7O8eWeAmTLAqW+1ozUbpRSKSwNTmO+exlS5KEIPYWuYdX
ports:
- 80:80
- 443:443
volumes:
- ./managed/testdata/checks:/srv/checks
EOF
# Run it and wait until it's healthy
docker compose -f compose.pmm-server-test.yml up -d --wait --wait-timeout=100
- name: Build the test image
shell: bash
run: |
docker build -t percona/pmm-api-tests .
- name: Run DB containers
shell: bash
run: |
pushd api-tests
docker compose up test_db # no daemon mode
# TODO: review the provisions below (copied from the Jenkins pipeline)
# MYSQL_IMAGE=${{env.MYSQL_IMAGE}} docker compose up -d mysql
# MONGO_IMAGE=${{env.MONGODB_IMAGE}} docker compose up -d mongo
# POSTGRES_IMAGE=${{env.POSTGRESQL_IMAGE}} docker compose up -d postgres
# docker compose up -d sysbench
popd
- name: Check connectivity to PMM Server
shell: bash
run: curl -f ${{env.PMM_URL}}/ping

- name: Run API tests
shell: bash
run: |
docker run -e PMM_SERVER_URL=${{env.PMM_URL}} \
-e PMM_RUN_UPDATE_TEST=0 \
-e PMM_RUN_STT_TESTS=0 \
--name pmm-api-tests \
--network host \
percona/pmm-api-tests
- name: Get PMM logs
if: ${{ failure() }}
run: curl --insecure ${{env.PMM_URL}}/logs.zip --output ${{ github.workspace }}/logs.zip || true

- name: Upload the logs on failure
if: ${{ failure() }}
uses: actions/upload-artifact@v3
with:
name: "logs.zip"
path: ${{ github.workspace }}/logs.zip

- name: Run debug commands on failure
if: ${{ failure() }}
run: |
echo "----- ENVIRONMENT VARIABLES -----"
env | sort
5 changes: 3 additions & 2 deletions Makefile.include
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ gen-api: ## Generate PMM API

bin/buf generate -v api

for API in api/agentlocalpb api/serverpb api/inventorypb api/managementpb api/managementpb/alerting api/managementpb/backup api/managementpb/azure api/managementpb/role api/qanpb api/managementpb/agent api/managementpb/node api/managementpb/service api/platformpb api/userpb; do \
for API in api/agentlocalpb api/serverpb api/inventorypb api/managementpb api/managementpb/alerting api/managementpb/backup api/managementpb/dump api/managementpb/azure api/managementpb/role api/qanpb api/managementpb/agent api/managementpb/node api/managementpb/service api/platformpb api/userpb; do \
set -x ; \
bin/swagger mixin $$API/json/header.json $$API/*.swagger.json --output=$$API/json/$$(basename $$API).json --keep-spec-order; \
bin/swagger flatten --with-flatten=expand --with-flatten=remove-unused $$API/json/$$(basename $$API).json --output=$$API/json/$$(basename $$API).json ; \
Expand Down Expand Up @@ -91,6 +91,7 @@ gen-api: ## Generate PMM API
api/managementpb/json/managementpb.json \
api/managementpb/alerting/json/alerting.json \
api/managementpb/backup/json/backup.json \
api/managementpb/dump/json/dump.json \
api/managementpb/azure/json/azure.json \
api/managementpb/role/json/role.json \
api/managementpb/agent/json/agent.json \
Expand Down Expand Up @@ -127,7 +128,7 @@ clean: clean_swagger ## Remove generated files
find api -name '*.pb.gw.go' -print -delete
find api -name '*.validate.go' -print -delete

for API in api/agentlocalpb api/serverpb api/inventorypb api/managementpb api/managementpb/alerting api/managementpb/backup api/managementpb/role api/managementpb/agent api/managementpb/node api/managementpb/service api/qanpb api/platformpb ; do \
for API in api/agentlocalpb api/serverpb api/inventorypb api/managementpb api/managementpb/alerting api/managementpb/backup api/management/dump api/managementpb/role api/managementpb/agent api/managementpb/node api/managementpb/service api/qanpb api/platformpb ; do \
rm -fr $$API/json/client $$API/json/models $$API/json/$$(basename $$API).json ; \
done
rm -f api/swagger/swagger.json api/swagger/swagger-dev.json
Expand Down
4 changes: 2 additions & 2 deletions agent/cmd/pmm-agent-entrypoint/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -73,15 +73,15 @@ var (
var pmmAgentProcessID = 0

func runPmmAgent(ctx context.Context, commandLineArgs []string, restartPolicy restartPolicy, l *logrus.Entry, pmmAgentSidecarSleep int) int {
pmmAgentFullCommand := "pmm-admin " + strings.Join(commandLineArgs, " ")
pmmAgentFullCommand := "pmm-agent " + strings.Join(commandLineArgs, " ")
for {
select {
case <-ctx.Done():
return 1
default:
}
var exitCode int
l.Infof("Starting 'pmm-admin %s'...", strings.Join(commandLineArgs, " "))
l.Infof("Starting 'pmm-agent %s'...", strings.Join(commandLineArgs, " "))
cmd := commandPmmAgent(commandLineArgs)
if err := cmd.Start(); err != nil {
l.Errorf("Can't run: '%s', Error: %s", commandLineArgs, err)
Expand Down
Loading

0 comments on commit 5aa6e6b

Please sign in to comment.