Skip to content
This repository was archived by the owner on Aug 15, 2022. It is now read-only.

klient: improve opening boltdb / fix wrong team default#11441

Merged
rjeczalik merged 2 commits intomasterfrom
klient-fix
Oct 5, 2017
Merged

klient: improve opening boltdb / fix wrong team default#11441
rjeczalik merged 2 commits intomasterfrom
klient-fix

Conversation

@rjeczalik
Copy link
Member

This PR:

  • fixes "invalid team name or user does not belong to the team", wrong default
  • fixes "bad file descriptor" error, which is a regression; previously we ensured in kd's main that boltdb file exists; this is no longer true, that's why function which opens BoltDB now is responsible for ensuring the file exists, when database is being open in read mode.

@cihangir
Copy link
Contributor

cihangir commented Oct 4, 2017

@rjeczalik Could you please rebase and push again? tests are failing and restarting did not help :/

Fixes:

    invalid team name or user does not belong to the team
Fixes "bad file descriptor" error, which is a regression.

Previously we ensured in kd's main that boltdb file exists. This is no
longer true, that's why function which opens BoltDB now is responsible
for ensuring the file exists, when database is being open in read mode.
@rjeczalik
Copy link
Member Author

@cihangir Done.

@codecov-io
Copy link

Codecov Report

Merging #11441 into master will decrease coverage by <.1%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #11441     +/-   ##
=========================================
- Coverage    30.2%    30.2%   -0.1%     
=========================================
  Files         101      101             
  Lines       11420    11425      +5     
=========================================
  Hits         3456     3456             
- Misses       7567     7572      +5     
  Partials      397      397

@rjeczalik rjeczalik merged commit b5b2c25 into master Oct 5, 2017
@rjeczalik rjeczalik deleted the klient-fix branch October 5, 2017 14:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants