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

docker: Add centos7 to dockerstatic dockerfiles #3402

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

Conversation

Haroon-Khel
Copy link
Contributor

@Haroon-Khel Haroon-Khel commented Feb 19, 2024

  • commit message has one of the standard prefixes
  • faq.md updated if appropriate
  • other documentation is changed or added (if applicable)
  • playbook changes run through VPC or QPC (if you have access)
  • VPC/QPC not applicable for this PR
  • for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

Security support for centos7 ends in June of this year, so its worth having a few containers on our machines until then
https://endoflife.date/centos

Copy link
Member

@sxa sxa left a comment

Choose a reason for hiding this comment

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

I didn't realise we didn't already have one of these so yeah if it is suitable for passing the tests I'm ok with adding it for a few months :-)

@Haroon-Khel
Copy link
Contributor Author

@Haroon-Khel
Copy link
Contributor Author

Its git version is quite old

[root@664a8be7aebb ~]# git --version
git version 1.8.3.1

Causing problems with tests https://ci.adoptium.net/job/Test_openjdk17_hs_sanity.functional_x86-64_linux/405/console

12:28:26  get functional test material...
12:28:26  git clone --depth 1 --reference-if-able /home/jenkins/openjdk_cache -b master https://github.com/eclipse-openj9/openj9.git
12:28:26  error: unknown option `reference-if-able'
12:28:26  usage: git clone [options] [--] <repo> [<dir>]
12:28:26  
12:28:26      -v, --verbose         be more verbose
12:28:26      -q, --quiet           be more quiet
12:28:26      --progress            force progress reporting

karianna
karianna previously approved these changes Feb 19, 2024
@karianna karianna dismissed their stale review February 19, 2024 20:50

Failing tests

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

Successfully merging this pull request may close these issues.

None yet

3 participants