Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

oci: Fix running of OCI hooks #817

Merged
merged 1 commit into from
Aug 27, 2020

Conversation

bpradipt
Copy link
Contributor

@bpradipt bpradipt commented Aug 21, 2020

OCI hooks fails to run since the code was writing the config.json
to the read-only path. This patch fixes it

Fixes: #818

Signed-off-by: Pradipta Kr. Banerjee [email protected]

@codecov
Copy link

codecov bot commented Aug 21, 2020

Codecov Report

Merging #817 into stable-1.11 will decrease coverage by 0.00%.
The diff coverage is 50.00%.

@@               Coverage Diff               @@
##           stable-1.11     #817      +/-   ##
===============================================
- Coverage        60.61%   60.61%   -0.01%     
===============================================
  Files               17       17              
  Lines             2613     2618       +5     
===============================================
+ Hits              1584     1587       +3     
- Misses             875      877       +2     
  Partials           154      154              

@jodh-intel
Copy link
Contributor

Thanks @bpradipt. But that "Fixes" issue doesn't look right?

OCI hooks fails to run since the code was writing the config.json
to the read-only path. This patch fixes it

Fixes: kata-containers#818

Signed-off-by: Pradipta Kr. Banerjee <[email protected]>
@bpradipt
Copy link
Contributor Author

Thanks @bpradipt. But that "Fixes" issue doesn't look right?

@jodh-intel you are right. Looks like for the original PR also the issue reference was incorrect. I created a new issue in agent repo and linked this PR to the same. Thanks.

@bpradipt bpradipt closed this Aug 24, 2020
@bpradipt bpradipt reopened this Aug 24, 2020
@bpradipt
Copy link
Contributor Author

/test-ubuntu

Copy link
Contributor

@jodh-intel jodh-intel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @bpradipt.

@jodh-intel
Copy link
Contributor

/test

Copy link
Member

@fidencio fidencio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@egernst egernst merged commit 466db3f into kata-containers:stable-1.11 Aug 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants