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

Scary warning when running two aktualizr at the same time #1217

Merged
merged 2 commits into from
May 23, 2019

Conversation

lbonn
Copy link
Contributor

@lbonn lbonn commented May 23, 2019

No description provided.

From some code in Bootloader

Signed-off-by: Laurent Bonnans <laurent.bonnans@here.com>
To guide users about the dangers of running several aktualizr instances
based on the same storage location.

Signed-off-by: Laurent Bonnans <laurent.bonnans@here.com>
@lbonn lbonn force-pushed the feat/OTA-2582/log-simultaneous-aktualizr branch from d5642df to 4cdc56a Compare May 23, 2019 14:06
Copy link
Collaborator

@pattivacek pattivacek left a comment

Choose a reason for hiding this comment

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

Tested and confirmed the lock warning.

@codecov-io
Copy link

Codecov Report

Merging #1217 into master will increase coverage by 0.07%.
The diff coverage is 87.87%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1217      +/-   ##
==========================================
+ Coverage   79.37%   79.44%   +0.07%     
==========================================
  Files         168      168              
  Lines       10005    10029      +24     
==========================================
+ Hits         7941     7968      +27     
+ Misses       2064     2061       -3
Impacted Files Coverage Δ
src/libaktualizr/utilities/utils.h 90.9% <ø> (ø) ⬆️
src/libaktualizr/storage/sqlstorage_base.h 100% <100%> (ø) ⬆️
src/libaktualizr/utilities/utils.cc 83.03% <100%> (+0.26%) ⬆️
src/libaktualizr/bootloader/bootloader.cc 62.22% <25%> (-0.94%) ⬇️
src/libaktualizr/storage/sqlstorage_base.cc 78.23% <94.73%> (+2.45%) ⬆️
src/libaktualizr/uptane/directorrepository.cc 95.71% <0%> (-1.43%) ⬇️
src/libaktualizr/uptane/imagesrepository.cc 92.59% <0%> (-0.62%) ⬇️
src/libaktualizr/storage/sqlstorage.cc 75.8% <0%> (-0.39%) ⬇️
src/libaktualizr/primary/sotauptaneclient.cc 92.51% <0%> (-0.16%) ⬇️
src/libaktualizr/crypto/crypto.cc 86.8% <0%> (+0.79%) ⬆️
... and 1 more

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 d69f4f6...4cdc56a. Read the comment docs.

@lbonn lbonn merged commit 6662fda into master May 23, 2019
@lbonn lbonn deleted the feat/OTA-2582/log-simultaneous-aktualizr branch May 23, 2019 15:56
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.

None yet

3 participants