Fix SSL connection from Memgraph Docker container and packaging issues#3793
Merged
mattkjames7 merged 9 commits intomasterfrom Feb 17, 2026
Merged
Fix SSL connection from Memgraph Docker container and packaging issues#3793mattkjames7 merged 9 commits intomasterfrom
mattkjames7 merged 9 commits intomasterfrom
Conversation
mattkjames7
added
bug
Contributor
Author
Tracking
Standard development
CI Testing Labels
Documentation checklist
|
Fixed ISSU test
mattkjames7
added
Docs - changelog only
|
mattkjames7
changed the title
DavIvek
reviewed
Feb 17, 2026
DavIvek
approved these changes
Feb 17, 2026
mattkjames7
added a commit
that referenced
this pull request
Feb 17, 2026
#3793) - Memgraph's Docker container is missing `ca-certificates` package - meaning that pulling data using LOAD CSV fails (see #3791) - this package has been added to the dockerfile and a smoke test has been added to check that this feature still works in the future. - The License files were incorrectly being copied into the `query_modules` directory when building MAGE with the `setup` Python script - now only the grappolo license is copied, as was the behaviour before MAGE was merged into Memgraph (fixes this issue: #3790). - Licenses were missing from final Docker container image under `/usr/share/doc/memgraph/`, but directory tree existed and the files existed within the DEB package -> the reason for this was that Ubuntu's `dpkg` exclusion configuration has changed, so now an exception must be made for our license files. - ISSU tests were failing since the updates to https://github.com/memgraph/helm-charts since the `3.8.0` release of Memgraph, due to increased automation within the helm charts -> removing `setup.cypherl` and updating logic to check for pods fixed this in #3796). - Fixed source of flakiness in checking for `mgconsole` after packaging - if `grep` exited before `echo` had finished, then there would be a broken pipe. - Upgraded `crypotography` version to address a couple of high severity CVEs.
This was referenced Feb 17, 2026
github-merge-queue bot
pushed a commit
that referenced
this pull request
Feb 17, 2026
Added a check at the end of the `package_memgraph` function to confirm that the package contains the primary set of licenses. Also added a smoke test to check for license files inside the Docker containers - while working on #3793 I found that `dpkg` has started ignoring files under `/usr/share/docs` when installing DEB packages, meaning that our license files would not be installed during the image build, even though they exist in the DEB package itself.
github-merge-queue bot
pushed a commit
that referenced
this pull request
Feb 18, 2026
Added a check at the end of the `package_memgraph` function to confirm that the package contains the primary set of licenses. Also added a smoke test to check for license files inside the Docker containers - while working on #3793 I found that `dpkg` has started ignoring files under `/usr/share/docs` when installing DEB packages, meaning that our license files would not be installed during the image build, even though they exist in the DEB package itself.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ca-certificatespackage - meaning that pulling data using LOAD CSV fails (see Problem with the SSL CA cert when usingLOAD CSV#3791) - this package has been added to the dockerfile and a smoke test has been added to check that this feature still works in the future.query_modulesdirectory when building MAGE with thesetupPython script - now only the grappolo license is copied, as was the behaviour before MAGE was merged into Memgraph (fixes this issue: License files inside the wrong directory #3790)./usr/share/doc/memgraph/, but directory tree existed and the files existed within the DEB package -> the reason for this was that Ubuntu'sdpkgexclusion configuration has changed, so now an exception must be made for our license files.3.8.0release of Memgraph, due to increased automation within the helm charts -> removingsetup.cypherland updating logic to check for pods fixed this in ISSU fix #3796).mgconsoleafter packaging - ifgrepexited beforeechohad finished, then there would be a broken pipe.crypotographyversion to address a couple of high severity CVEs.