Skip to content

Fixed helm unit tests and database docker build #653

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
Merged
lsierant merged 3 commits into from
Dec 18, 2025
Merged

Fixed helm unit tests and database docker build #653

lsierant merged 3 commits into from
Dec 18, 2025

Conversation

@lsierant
Copy link
Contributor

@lsierant lsierant commented Dec 17, 2025 •
edited
Loading

Summary

  • Bumped helm version as recent update of helm's unittest plugin stopped working with helm 3.17.
  • Fixed building of mongodb-enterprise-database image. net-snmp and net-snmp-agent-libs caused problems. Upon a closer look those are unnecessary according to docs.

Proof of Work

Passing helm tests
Passing mongodb-kubernetes-database build

@lsierant lsierant self-assigned this Dec 17, 2025
@github-actions
Copy link

github-actions bot commented Dec 17, 2025

⚠️ (this preview might not be accurate if the PR is not rebased on current master branch)

MCK 1.6.2 Release Notes

@lsierant lsierant changed the title Fixed unittest helm plugin Fixed helm unit tests Dec 17, 2025
@lsierant lsierant added the skip-changelog Use this label in Pull Request to not require new changelog entry file label Dec 17, 2025
@lsierant lsierant marked this pull request as ready for review December 17, 2025 17:16
@lsierant lsierant requested a review from a team as a code owner December 17, 2025 17:16
@lsierant lsierant mentioned this pull request Dec 18, 2025
Open
3 tasks
@lsierant lsierant force-pushed the lsierant/fix-helm-tests branch 2 times, most recently from a1679e6 to 9c0c27f Compare December 18, 2025 08:03
@lsierant lsierant changed the base branch from master to lsierant/fix-database-docker December 18, 2025 08:04
@lsierant lsierant force-pushed the lsierant/fix-helm-tests branch from 9c0c27f to 16a9e0a Compare December 18, 2025 08:14
@lsierant lsierant changed the title Fixed helm unit tests Fixed helm unit tests and database docker build Dec 18, 2025
@lsierant lsierant changed the base branch from lsierant/fix-database-docker to master December 18, 2025 08:17
@lsierant lsierant mentioned this pull request Dec 18, 2025
Draft
@lsierant lsierant merged commit 13cf357 into master Dec 18, 2025
32 of 34 checks passed
@lsierant lsierant deleted the lsierant/fix-helm-tests branch December 18, 2025 14:18
@lsierant lsierant mentioned this pull request Dec 18, 2025
Merged
lsierant added a commit that referenced this pull request Dec 19, 2025
@lsierant
Revert fix for missing net-snmp packages in ubi8 (#659)
2352446 
# Summary

Reverted missing package from #653

It has been fixed in https://issues.redhat.com/browse/RHEL-136705

## Proof of Work

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

Reviewers

@m1kola m1kola m1kola approved these changes

@nammn nammn nammn approved these changes

@lucian-tosa lucian-tosa lucian-tosa approved these changes

@fealebenpae fealebenpae Awaiting requested review from fealebenpae fealebenpae is a code owner automatically assigned from mongodb/kubernetes-hosted

@mircea-cosbuc mircea-cosbuc Awaiting requested review from mircea-cosbuc

Assignees

@lsierant lsierant

Labels

skip-changelog Use this label in Pull Request to not require new changelog entry file

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@lsierant @m1kola @nammn @lucian-tosa