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

Do not detect concurrent storage in readonly mode #1229

Merged
merged 1 commit into from
Jun 13, 2019

Conversation

lbonn
Copy link
Contributor

@lbonn lbonn commented Jun 7, 2019

No description provided.

@lbonn lbonn requested review from pattivacek and eu-siemann June 7, 2019 16:38
@codecov-io
Copy link

Codecov Report

Merging #1229 into master will decrease coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1229      +/-   ##
==========================================
- Coverage   79.38%   79.37%   -0.01%     
==========================================
  Files         170      170              
  Lines       10122    10123       +1     
==========================================
  Hits         8035     8035              
- Misses       2087     2088       +1
Impacted Files Coverage Δ
src/libaktualizr/storage/sqlstorage_base.cc 78.37% <100%> (+0.14%) ⬆️
src/libaktualizr/storage/sqlstorage.cc 75.41% <0%> (-0.78%) ⬇️
src/libaktualizr/uptane/imagesrepository.cc 92.59% <0%> (-0.62%) ⬇️
src/libaktualizr/primary/sotauptaneclient.cc 92.62% <0%> (-0.31%) ⬇️
src/libaktualizr/uptane/directorrepository.cc 97.14% <0%> (ø) ⬆️
src/aktualizr_primary/secondary.cc 87.67% <0%> (+1.36%) ⬆️
src/libaktualizr/package_manager/ostreemanager.cc 77.11% <0%> (+1.69%) ⬆️
src/libaktualizr-posix/ipuptanesecondary.cc 86.66% <0%> (+4%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3c26e78...68b08db. Read the comment docs.

@pattivacek pattivacek merged commit ba0e4b2 into master Jun 13, 2019
@pattivacek pattivacek deleted the fix/aktualizr-info-scary branch June 13, 2019 09:05
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.

3 participants