-
Notifications
You must be signed in to change notification settings - Fork 185
Upgrade to Karaf 4.4.8 #6928
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
Upgrade to Karaf 4.4.8 #6928
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly just nitpicks. I'll also build this and do some manual testing.
...s/test-itests-kernel/src/test/java/org/codice/ddf/itests/kernel/ITSynchronizedInstaller.java
Outdated
Show resolved
Hide resolved
libs/test-common/src/main/java/org/codice/ddf/test/common/configurators/KarafOptions.java
Outdated
Show resolved
Hide resolved
I was able to install, ingest, and search. |
These packages are also exported by the bouncycastle bundles installed by Karaf, which can cause import resolution problems for bundles that use them.
build now |
Internal build has been started, your results will be available at build completion. |
Build SUCCESS See the job results in legacy Jenkins UI or in Blue Ocean UI. |
build now |
Internal build has been started, your results will be available at build completion. |
Build SUCCESS See the job results in legacy Jenkins UI or in Blue Ocean UI. |
build now |
Internal build has been started, your results will be available at build completion. |
Build SUCCESS See the job results in legacy Jenkins UI or in Blue Ocean UI. |
What does this PR do?
Upgrades to Karaf 4.4.8
Who is reviewing it?
@derekwilhelm
@brendan-hofmann
How should this be tested?
Generally use the system and verify it works as expected.
Checklist:
Notes on Review Process
Please see Notes on Review Process for further guidance on requirements for merging and abbreviated reviews.
Review Comment Legend: