Skip to content
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

PMM-9610 DbaaS Upgrade Test pipelines #1590

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

PMM-9610 DbaaS Upgrade Test pipelines #1590

wants to merge 16 commits into from

Conversation

pmm/pmm2-dbaas-upgrade-tests.groovy Outdated Show resolved Hide resolved
pmm/pmm2-dbaas-upgrade-tests.groovy Outdated Show resolved Hide resolved
pmm/pmm2-dbaas-upgrade-tests.groovy Outdated Show resolved Hide resolved
pmm/pmm2-dbaas-upgrade-tests.groovy Outdated Show resolved Hide resolved
sh '''
docker-compose --version
sudo yum -y update --security
sudo yum -y install php php-mysqlnd php-pdo jq svn bats mysql
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have all of this on agent except mysql

pmm/pmm2-dbaas-upgrade-tests.groovy Outdated Show resolved Hide resolved
pmm/pmm2-dbaas-upgrade-tests.groovy Outdated Show resolved Hide resolved
steps {
setupNodejs()
sh """
sudo yum install -y gettext
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have gettext and can remove it

pmm/pmm2-dbaas-upgrade-tests.groovy Outdated Show resolved Hide resolved
nikita-b
nikita-b previously approved these changes Jun 17, 2022

pipeline {
agent {
label 'large-amazon'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
label 'large-amazon'
label 'cli'

Should be enough.

sh '''
docker-compose --version
sudo yum -y install mysql
sudo amazon-linux-extras install epel -y
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we also don't need it :)

@nikita-b
Copy link
Contributor

@puneet0191 LGTM only 2 small changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants