Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixed cvat startup process #2293

Merged
merged 6 commits into from Oct 12, 2020
Merged

fixed cvat startup process #2293

merged 6 commits into from Oct 12, 2020

Conversation

azhavoro
Copy link
Contributor

@azhavoro azhavoro commented Oct 9, 2020

Slightly improved ssh keys import

Fixed CVAT startup process if the user has umask 077 in .bashrc file.
This issue is actual only for CVAT images based on ubuntu 16.04 and not reproducible for 20.04.
The root cause is difference behavior of the ssh-add tool:

ls -la .ssh/README.md 
-rw------- 1 django django 851 Oct  8 09:54 .ssh/README.md
  • for ubuntu 16.04 ssh-add asks for a passphrase, so the startup process waits for input
   ssh-add .ssh/README.md 
   Enter passphrase for .ssh/README.md:
  • ubuntu 20.04 ssh-add just throws the exception:
   ssh-add .ssh/README.md 
   Error loading key ".ssh/README.md": invalid format

Motivation and context

resolve #1794

How has this been tested?

Manual test

Checklist

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.
  • I have updated the license header for each file (see an example below)
# Copyright (C) 2020 Intel Corporation
#
# SPDX-License-Identifier: MIT

@nmanovic
Copy link
Contributor

nmanovic commented Oct 9, 2020

@azhavoro , please add CHANGELOG

@coveralls
Copy link

coveralls commented Oct 9, 2020

Coverage Status

Coverage remained the same at 66.695% when pulling eade417 on az/fix_ssh_keys_import into d4129f2 on develop.

bsekachev
bsekachev previously approved these changes Oct 12, 2020
@andyDoucette
Copy link

Thank you! 😁🙏

@nmanovic nmanovic deleted the az/fix_ssh_keys_import branch October 13, 2020 08:23
@azhavoro
Copy link
Contributor Author

Thank you! 😁🙏

Thank you too! It was possible to reproduce this issue only with your hint.

@andyDoucette
Copy link

andyDoucette commented Oct 14, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error when creating superuser in fresh install
5 participants