Skip to content

HYRAX-1964: Dockerize bes build #1213

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.

Sign up for GitHub

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

Jump to bottom
Open
hannahilea wants to merge 44 commits into
base: master
Choose a base branch
from
Open

HYRAX-1964: Dockerize bes build #1213

hannahilea wants to merge 44 commits into from

Conversation

@hannahilea
Copy link
Contributor

@hannahilea hannahilea commented Dec 16, 2025 •
edited
Loading

Description

See HYRAX-1964 for details.

@hannahilea
fail fast wip: comment out all existing travis jobs
17b7abd
@hannahilea
add basic docker build step by copying rpm step; not yet tweaked for …
6d62494 
…docker
@hannahilea
initial commit of docker build step, autoreconf only
f3ed8bf
@hannahilea
wip configure succeeds
c6e303c
@hannahilea
wip libdap deps
b9f924b
@hannahilea
fix for travis
3febcf2
@hannahilea
add docker service
89adab8
@hannahilea
turns out travis dislikes slashes
c82d14c
@hannahilea
fix build version tag
98336e0
@hannahilea
travis tweak
9234c0f
@hannahilea
download_dir
b45a09c
@hannahilea
fix download dir
e89242e
@hannahilea
run multistage with install
c487300
@hannahilea
coppy docker deploy script from hyrax-docker
f55fef7
@hannahilea
Move docker build to script
97129ff
@hannahilea
whoops
ccba43a
@hannahilea
fix password env var
dfadb0f
@hannahilea
fix debug commentout
4ca9406
@hannahilea
fail fast
9ec1f18
@hannahilea
tweak
0442a54
@hannahilea
travis is weird about empty strings
64430c0
@hannahilea
plz bash
d911903
@hannahilea
update docker repo name now that it exists
18390a5
@hannahilea
update build script
55300c8
@hannahilea
undo travis comment-outs
7f549b5
@hannahilea hannahilea marked this pull request as ready for review December 18, 2025 21:31
@hannahilea
Copy link
Contributor Author

hannahilea commented Dec 18, 2025

Opening for review, but not going to merge until the accompanying PR in docker-hyrax is looking good.

Note that we are not removing anything in this PR, only adding a new docker image build. We can remove the old rpm builds in a later PR, once we're sure we no longer need them.

hannahilea
hannahilea commented Dec 18, 2025
View reviewed changes
.travis.yml

git:
depth: false
depth: 1
Copy link
Contributor Author

@hannahilea hannahilea Dec 18, 2025

Choose a reason for hiding this comment

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

...sneaking in a nicer Travis build! :)

hannahilea
hannahilea commented Dec 18, 2025
View reviewed changes
.travis.yml
- export SNAPSHOT_IMAGE_TAG="opendap/${DOCKER_NAME}:snapshot"
- export BUILD_VERSION_TAG="opendap/${DOCKER_NAME}:${BES_BUILD_NUMBER}"
- bash ${TRAVIS_BUILD_DIR}/travis/build-rhel-docker.sh
# TODO: remove this before merging! Using for debugging
Copy link
Contributor Author

@hannahilea hannahilea Dec 18, 2025

Choose a reason for hiding this comment

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

  • remove before merging, once otherwise ready to go

@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 18, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 18, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 18, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

hannahilea
hannahilea commented Dec 18, 2025
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@hannahilea