Skip to content

Utilization of Yum or Apt according Unix version#1100

Open
helviojunior wants to merge 1 commit into
jenkinsci:masterfrom
helviojunior:master
Open

Utilization of Yum or Apt according Unix version#1100
helviojunior wants to merge 1 commit into
jenkinsci:masterfrom
helviojunior:master

Conversation

@helviojunior
Copy link
Copy Markdown

Utilization of Yum or Apt according Unix version

computer,
clientSession,
javaPath + " -fullversion",
"apt install -y --no-install-recommends fontconfig openjdk-11-jdk-headless",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Java 17 is the minimum required (21 recommended), so there is no way this could work. Which leads me to think that the Yum variant has not actually worked for a long time either, and nobody noticed because nobody was using this system. Nor should you. Installing Java at runtime forces every agent launch to take much longer, and abuses the bandwidth offered for free by Linux distro servers. 45bae19 was a bad idea, #245 made it worse, #246 compounded the bad idea, and #777 perpetuated it. Since the existing code does not work, it would be better to just delete it and not pretend.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

(If you want to play with the plugin without going to the bother of creating your own AMI, you can already install a Java package using the init script. But you should never do this in a production system or at scale.)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants