-
Notifications
You must be signed in to change notification settings - Fork 567
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
[Medium] Patch ceph to fix CVE-2014-5461 #12123
[Medium] Patch ceph to fix CVE-2014-5461 #12123
Conversation
@kevin-b-lockwood can you combine this and these two other PRs? we will hit merge conflicts if we try to take any of these changes individually. |
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
@sameluch How do I go about fixing these toolchain issues? It builds fine on my machine so I'm looking for a way to check this stuff. Maybe the toolchain jsons have changed in |
yes, we recently checked in a change from Bala which updated python so the manifests changed. Updating the base branch and restarting the build should resolve the issue. |
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
@kevin-b-lockwood can you close the other PRs for ceph CVEs in favor of just using this one for the moment? |
e3900ec
to
3fd0b1e
Compare
My apologies, I rebased after seeing your message but did not see that you merged first. Hopefully that is not to much of an issue. I will mark those other PRs as closed. |
No worries, I will kick off another build for this PR with the update. |
/AzurePipelines run |
Azure Pipelines successfully started running 1 pipeline(s). |
@sameluch I hate to bother you, but I can't actually see the test results due to permissions issues. I have no such test failures when I build it on my machine, so I'm struggling to fix these issues. Is there something I can do to get those permissions? |
@kevin-b-lockwood, this build looks good. The PR check build does some additional packages (generally whatever is in flight on the fasttrack branch already and any packages that depend on the updated package). The failures seen are all known failures, so we are going to take this change. |
Glad to hear it, I can worry about other PRs now. |
(cherry picked from commit d419417)
Auto cherry-pick results: Auto cherry-pick pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=726030&view=results |
Merge Checklist
All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)
*-static
subpackages, etc.) have had theirRelease
tag incremented../cgmanifest.json
,./toolkit/scripts/toolchain/cgmanifest.json
,.github/workflows/cgmanifest.json
)./LICENSES-AND-NOTICES/SPECS/data/licenses.json
,./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md
,./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON
)*.signatures.json
filessudo make go-tidy-all
andsudo make go-test-coverage
passSummary
Fix CVE-2014-5461
Change Log
Does this affect the toolchain?
NO
Links to CVEs
Test Methodology