fix(bootstrap): consider 18 ent pgpro_version changes#220
Conversation
Kmortyk
force-pushed
the
MMNS-45-bootstrap-fix
branch
5 times, most recently
from
bfe9c85 to
5fe127b
Compare
Kmortyk
force-pushed
the
MMNS-45-bootstrap-fix
branch
from
5fe127b to
27211a3
Compare
|
|
||
| # install and set up components missing in docker image (sudo, wget, bc, unzip, lsb-release, gnupg, tzdata) | ||
| apt-get clean && apt-get update && apt-get install -y sudo | ||
| rm -rf /var/lib/apt/lists/* && apt-get update && apt-get install -y sudo |
There was a problem hiding this comment.
Got some problem with cache here, so changed mirrors and removed cached packages.
There was a problem hiding this comment.
probably apt-get clean all would suffice but this will do too
| MAMONSU_PATH: '/home/runner/work/mamonsu/mamonsu/' | ||
| strategy: | ||
| matrix: | ||
| docker_os: ['centos:7', 'centos:8'] |
There was a problem hiding this comment.
Centos 7 tries to check PostgreSQL 12 package and fails, so removed for now.
There was a problem hiding this comment.
yep, I guess as a temporary change just for the sake of this PR we can merge it. I'm going to add gh actions fix soon
| MAMONSU_PATH: '/home/runner/work/mamonsu/mamonsu/' | ||
| strategy: | ||
| matrix: | ||
| docker_os: ['centos:7', 'centos:8'] |
There was a problem hiding this comment.
yep, I guess as a temporary change just for the sake of this PR we can merge it. I'm going to add gh actions fix soon
|
|
||
| # install and set up components missing in docker image (sudo, wget, bc, unzip, lsb-release, gnupg, tzdata) | ||
| apt-get clean && apt-get update && apt-get install -y sudo | ||
| rm -rf /var/lib/apt/lists/* && apt-get update && apt-get install -y sudo |
There was a problem hiding this comment.
probably apt-get clean all would suffice but this will do too
2 participants
Fixed edition check for 18 ent during bootstrap.