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

kdb rm /..: removes spec key #2761

Closed
markus2330 opened this issue Jun 9, 2019 · 8 comments · Fixed by #3794
Closed

kdb rm /..: removes spec key #2761

markus2330 opened this issue Jun 9, 2019 · 8 comments · Fixed by #3794
Assignees

Comments

@markus2330
Copy link
Contributor

markus2330 commented Jun 9, 2019

Steps to Reproduce the Problem

Assume that spec.ini is in pwd and contains:

[]
mountpoint = mymountpoint.ini

[mykey]
type = long
sudo kdb mount `pwd`/spec.ini spec/tests/hello ni
sudo kdb spec-mount /tests/hello

kdb set /tests/hello/mykey 123
kdb rm /tests/hello/mykey

Expected Result

That I removed what I've set just before.

Actual Result

The spec key is removed, spec.ini then contains:

;Ni1
; Generated by Nickel Plugin using Elektra (see libelektra.org).

 = 

[]
 mountpoint = mymountpoint.ini

Interestingly, the validation then still fails and kdb meta-get /tests/hello/mykey type still returns long. But this might be a different mmap-related bug (#2760).

System Information

  • Elektra Version: master
@kodebach
Copy link
Member

kodebach commented Jun 9, 2019

kdb getmeta /tests/hello/mykey type still returns long

That may be because of #2700. It may also be related to #2456.

@markus2330
Copy link
Contributor Author

Thank you for making the connections.

@stale
Copy link

stale bot commented Jun 8, 2020

I mark this issue stale as it did not have any activity for one year. I'll close it in two weeks if no further activity occurs. If you want it to be alive again, ping the issue by writing a message here or create a new issue with the remainder of this issue.
Thank you for your contributions 💖

@stale stale bot added stale and removed stale labels Jun 8, 2020
@markus2330
Copy link
Contributor Author

As this bug might lead to accidental removal of wrong data, we should fix it. I updated the initial description to use meta-get.

Also the to-be-believed-mmap-related bug still can be reproduced with Elektra 0.9.2.

@stale
Copy link

stale bot commented Jun 9, 2021

I mark this issue stale as it did not have any activity for one year. I'll close it in two weeks if no further activity occurs. If you want it to be alive again, ping the issue by writing a message here or create a new issue with the remainder of this issue.
Thank you for your contributions 💖

@stale stale bot added the stale label Jun 9, 2021
@stale
Copy link

stale bot commented Jun 23, 2021

I closed this issue now because it has been inactive for more than one year. If I closed it by mistake, please do not hesitate to reopen it or create a new issue with the remainder of this issue.
Thank you for your contributions 💖

@stale stale bot closed this as completed Jun 23, 2021
@kodebach
Copy link
Member

@markus2330 Does this bug still exist? Should we reopen the issue?

@markus2330 markus2330 linked a pull request Jun 23, 2021 that will close this issue
20 tasks
@markus2330
Copy link
Contributor Author

#3742 should fix it.

@markus2330 markus2330 reopened this Jun 23, 2021
@stale stale bot removed the stale label Jun 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants