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

Added Dockerfiles for Murfey services and build context for Murfey frontend #450

Merged
merged 39 commits into from
Feb 10, 2025

Conversation

tieneupin
Copy link
Contributor

@tieneupin tieneupin commented Jan 15, 2025

This PR adds the Dockerfiles and Helm charts needed in order to build and deploy images of the different components needed in order to run a Murfey data transfer environment on a Kubernetes cluster.

The .pre-commit-config.yaml file was updated to exclude checking deployment.yaml files, which contain template expression syntax and are thus not purely YAML.

Additionally, the .bumpversion.toml file was updated so that the Helm chart versions are also updated in tandem with the rest of the repo.

Merge this after PR #448 .

@tieneupin tieneupin added server Relates to the server component client Relates to the client component web ui labels Jan 15, 2025
@tieneupin tieneupin self-assigned this Jan 15, 2025
Copy link

codecov bot commented Jan 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 26.90%. Comparing base (3f11363) to head (4bf988a).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #450      +/-   ##
==========================================
- Coverage   27.40%   26.90%   -0.51%     
==========================================
  Files          78       78              
  Lines       10140    10509     +369     
  Branches     1332     1490     +158     
==========================================
+ Hits         2779     2827      +48     
- Misses       7266     7569     +303     
- Partials       95      113      +18     

github-advanced-security[bot]

This comment was marked as outdated.

@tieneupin tieneupin marked this pull request as ready for review January 15, 2025 17:56
@tieneupin tieneupin requested a review from d-j-hatton January 15, 2025 17:56
d-j-hatton
d-j-hatton previously approved these changes Jan 31, 2025
@tieneupin tieneupin dismissed d-j-hatton’s stale review February 10, 2025 10:25

Quite a few additional files have been added since this approval was given

@tieneupin tieneupin merged commit 42b31d2 into main Feb 10, 2025
17 checks passed
@tieneupin tieneupin deleted the murfey-containers branch February 10, 2025 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client Relates to the client component server Relates to the server component web ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants