Skip to content

Switch el9-builds to build from bes_core docker image instead of RPMs#120

Merged
hannahilea merged 18 commits intohr/HYRAX-1964-use-dockerized-besfrom
hr/use-dockerized-bes-el9
Mar 6, 2026
Merged

Switch el9-builds to build from bes_core docker image instead of RPMs#120
hannahilea merged 18 commits intohr/HYRAX-1964-use-dockerized-besfrom
hr/use-dockerized-bes-el9

Conversation

@hannahilea
Copy link
Copy Markdown
Contributor

@hannahilea hannahilea commented Mar 4, 2026 •
edited
Loading

Based on changes made in #107; I applied the same changes to the el9-builds as were made for the el8-builds in #107.

This branch will then get merged into PR #107, so that when #107 merges down both el8 and el9 will be good.

@hannahilea hannahilea requested a review from ndp-opendap March 4, 2026 17:49
@hannahilea hannahilea mentioned this pull request Mar 4, 2026
Merged
5 tasks
@hannahilea
Copy link
Copy Markdown
Contributor Author

hannahilea commented Mar 4, 2026 •
edited
Loading

These Travis builds will fail until OPENDAP/bes#1238 is merged.

EDIT: merged 3/5

hannahilea
hannahilea commented Mar 4, 2026
View reviewed changes
.travis.yml
- master
- main
- /^(.*-test-deploy)$/
- hr/HYRAX-1964-use-dockerized-bes
Copy link
Copy Markdown
Contributor Author

@hannahilea hannahilea Mar 4, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

  • TODO before merge: remove this addition (+ 2 more lower down)

Copy link
Copy Markdown
Contributor Author

@hannahilea hannahilea Mar 6, 2026

Choose a reason for hiding this comment

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

...am going to wait and remove them in #107, so we can continue to see them pass for this branch!

hannahilea and others added 11 commits March 4, 2026 18:00
@hannahilea
i suppose equals signs are important
28b497a
@travis-ci-opendap
OLFS: Triggering hyrax-docker image production for el8.
7f5ed78 
Build Version Matrix:
TARGET_OS: el8
libdap4-3.21.1-390 2026-03-04T03:14:01+0000
bes-3.21.1-1096 2026-03-04T23:14:34+0000
olfs-1.18.15-726 2026-03-04T23:29:26+0000
hyrax-1.17.1-726 2026-03-04T23:29:26+0000
@travis-ci-opendap
OLFS: Triggering hyrax-docker image production for el9.
38f050c 
Build Version Matrix:
TARGET_OS: el9
TOMCAT_MAJOR_VERSION: 11
libdap4-3.21.1-390 2026-03-04T03:14:01+0000
bes-3.21.1-1096 2026-03-04T23:14:34+0000
olfs-1.18.15-727 2026-03-05T00:05:12+0000
hyrax-1.17.1-727 2026-03-05T00:05:12+0000
@travis-ci-opendap
OLFS: Triggering hyrax-docker image production for el8.
a35a572 
Build Version Matrix:
TARGET_OS: el8
libdap4-3.21.1-390 2026-03-04T03:14:01+0000
bes-3.21.1-1096 2026-03-04T23:14:34+0000
olfs-1.18.15-730 2026-03-05T17:35:18+0000
hyrax-1.17.1-730 2026-03-05T17:35:18+0000
@travis-ci-opendap
OLFS: Triggering hyrax-docker image production for el8.
9741756 
Build Version Matrix:
TARGET_OS: el8
libdap4-3.21.1-390 2026-03-04T03:14:01+0000
bes-3.21.1-1101 2026-03-05T17:25:18+0000
olfs-1.18.15-731 2026-03-05T17:37:19+0000
hyrax-1.17.1-731 2026-03-05T17:37:19+0000
@travis-ci-opendap
OLFS: Triggering hyrax-docker image production for el9.
ef38fb9 
Build Version Matrix:
TARGET_OS: el9
TOMCAT_MAJOR_VERSION: 11
libdap4-3.21.1-390 2026-03-04T03:14:01+0000
bes-3.21.1-1101 2026-03-05T17:25:18+0000
olfs-1.18.15-732 2026-03-05T18:13:42+0000
hyrax-1.17.1-732 2026-03-05T18:13:42+0000
@hannahilea
Merge branch 'master' into hr/use-dockerized-bes-el9
475e5d1
@hannahilea
install chkconfig for el9
b18beec
@hannahilea
fix spelling bork
16b2176
@hannahilea
rearrange
0b40ca4
@hannahilea
update chkconfig install
e80f3e2
@hannahilea hannahilea changed the title Switch el9-builds off of RPMs too Switch el9-builds to build from bes_core docker image instead of RPMs Mar 5, 2026
ndp-opendap
ndp-opendap approved these changes Mar 5, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@ndp-opendap ndp-opendap left a comment

Choose a reason for hiding this comment

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

Wee change suggestions, looks like it's Great!

el9-builds/hyrax/entrypoint.sh
bes_uid="$(id -u bes)"
bes_username="$BES_USER"
bes_uid="$(id -u ${bes_username})"
echo "bes_uid: $bes_uid"
Copy link
Copy Markdown
Collaborator

@ndp-opendap ndp-opendap Mar 5, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

Note:

No change needed just an observation:
The output of these echo statements are used by other software to determine the bes user's uid:gid so that the permissions on mounted volumes can be altered to match what the docker image expects.

So it's good that these bits remain unchanged...

Copy link
Copy Markdown
Contributor Author

@hannahilea hannahilea Mar 6, 2026

Choose a reason for hiding this comment

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

😅 PHEW

@hannahilea hannahilea merged commit 8d1525d into hr/HYRAX-1964-use-dockerized-bes Mar 6, 2026
2 checks passed
@hannahilea hannahilea deleted the hr/use-dockerized-bes-el9 branch March 6, 2026 15:15
hannahilea added a commit that referenced this pull request Mar 10, 2026
@hannahilea @ndp-opendap @travis-ci-opendap
HYRAX-1964: Build images from bes base docker image (bes_core) inst…
37c4038 
…ead of bes and libdap rpms (#107)

* fixes made on other branch

* use bes_core docker image as hyrax docker base image

* comment out travis stages not yet updated

* fix travis for main build

* Update besd image to use bes_core

* enable besd in travis build

* enable hyrax_ncwms building

* build ngap image from bes_core base image

* undo unneeded hyrax docker change

* add missing var

* clean up use of bes_core_image_tag

* do not fail regression test when no log provided

* move bes_core_image_tag definition to build-el8

* force to el8

* remove whitespace poke travis

* remove besd rpm fetchin gfrom el8

* export core_image_tag

* comment out to fail slower

* Apply suggestion from @hannahilea

* Apply suggestion from @hannahilea

* Trigger cicd

* Trigger cicd

* Refining image name test

* Refining image name test

* remove cruft

* change approach to hardcoding build recipe

* java11 to java17 in prep for bump on master

* revert build recipe override

* use different approach to forcing el8 build

* add back vestigial .pem file substitutions

* rename USER to BES_USER

* revert path to ngap_netrc file

* Switch el9-builds to build from `bes_core` docker image instead of RPMs (#120)

* Update el-9 shared and besd build

* update el9 hyrax

* udpate el9 ngap

* clean up

* update helper functions

* update .travis.yml to run CI on branch

* i suppose equals signs are important

* OLFS: Triggering hyrax-docker image production for el8.
Build Version Matrix:
TARGET_OS: el8
libdap4-3.21.1-390 2026-03-04T03:14:01+0000
bes-3.21.1-1096 2026-03-04T23:14:34+0000
olfs-1.18.15-726 2026-03-04T23:29:26+0000
hyrax-1.17.1-726 2026-03-04T23:29:26+0000

* OLFS: Triggering hyrax-docker image production for el9.
Build Version Matrix:
TARGET_OS: el9
TOMCAT_MAJOR_VERSION: 11
libdap4-3.21.1-390 2026-03-04T03:14:01+0000
bes-3.21.1-1096 2026-03-04T23:14:34+0000
olfs-1.18.15-727 2026-03-05T00:05:12+0000
hyrax-1.17.1-727 2026-03-05T00:05:12+0000

* OLFS: Triggering hyrax-docker image production for el8.
Build Version Matrix:
TARGET_OS: el8
libdap4-3.21.1-390 2026-03-04T03:14:01+0000
bes-3.21.1-1096 2026-03-04T23:14:34+0000
olfs-1.18.15-730 2026-03-05T17:35:18+0000
hyrax-1.17.1-730 2026-03-05T17:35:18+0000

* OLFS: Triggering hyrax-docker image production for el8.
Build Version Matrix:
TARGET_OS: el8
libdap4-3.21.1-390 2026-03-04T03:14:01+0000
bes-3.21.1-1101 2026-03-05T17:25:18+0000
olfs-1.18.15-731 2026-03-05T17:37:19+0000
hyrax-1.17.1-731 2026-03-05T17:37:19+0000

* OLFS: Triggering hyrax-docker image production for el9.
Build Version Matrix:
TARGET_OS: el9
TOMCAT_MAJOR_VERSION: 11
libdap4-3.21.1-390 2026-03-04T03:14:01+0000
bes-3.21.1-1101 2026-03-05T17:25:18+0000
olfs-1.18.15-732 2026-03-05T18:13:42+0000
hyrax-1.17.1-732 2026-03-05T18:13:42+0000

* install chkconfig for el9

* fix spelling bork

* rearrange

* update chkconfig install

* tweaks from code review

---------

Co-authored-by: Hannah Robertson <1076701+hannahilea@users.noreply.github.com>
Co-authored-by: The-Robot-Travis <npotter@opendap.org>

* remove travis testing cruft

* force el8 build

* for testing always push images

* revert travis change

* remove more rpm cruft

* fix path in entrypoints

---------

Co-authored-by: Hannah Robertson <1076701+hannahilea@users.noreply.github.com>
Co-authored-by: ndp-opendap <ndp@opendap.org>
Co-authored-by: The-Robot-Travis <npotter@opendap.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ndp-opendap ndp-opendap ndp-opendap approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@hannahilea @ndp-opendap @travis-ci-opendap