Skip to content

Conversation

@Haroon-Khel
Copy link
Contributor

@Haroon-Khel Haroon-Khel commented Jan 8, 2026

  • 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

Changes made:

  • Return instructions to install X11, not sure why they were removed in the first place
  • Installs XLC16 fixpack 10 alongside XLC16
  • A role has been created to install OpenXL 17, but the role is not yet in use
  • Can modify the path to the vendor files if using local vendor files
  • Keep fontconfig-devel to version 2.11.95, based on Ansible setup of p10-aix-adopt05.osuosl.org #4179 (comment)

Testing to do:

  • Check if vendor path changes work on AWX
  • Check if Jenkins key file name change works on AWX
  • Test playbook run

@Haroon-Khel Haroon-Khel changed the title Aix.playbook.updates aixPb: OpenXL 17 role and other improvements Jan 8, 2026
@github-actions github-actions bot added the os:aix label Jan 8, 2026
@Haroon-Khel
Copy link
Contributor Author

This pr no longer installs the XLC17 compiler, but the runtime only. Thats enough to run JDK25 tests. ref #4179 (comment)

@sxa
Copy link
Member

sxa commented Jan 21, 2026

While it may be ok for a test machine, surely we need the v17 compiler to be able to set up machines suitable for building the recent versions of Temurin?

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

A block has been put on this Pull Request as this repository is temporarily under a code freeze due to an ongoing release cycle.

If this pull request needs to be merged during the release cycle then please comment /merge and a PMC member will be able to remove the block.

If the code freeze is over you can remove this block by commenting /thaw.

@Haroon-Khel
Copy link
Contributor Author

Haroon-Khel commented Jan 21, 2026

While it may be ok for a test machine, surely we need the v17 compiler to be able to set up machines suitable for building the recent versions of Temurin?

I agree. I just dont have access to the compiler files at the moment. The files in #3142 (comment) are the for the runtime only. I may have to speak to the IBM team to get them

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.

2 participants