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

ensure fernet-keys folder permissions #170

Merged
merged 5 commits into from
Dec 16, 2020

Conversation

AlvaroVega
Copy link
Member

@AlvaroVega AlvaroVega commented Dec 16, 2020

Just for case of docker is using a volume for store fernet-keys (to enable distribute fernet keys abroad keystone nodes)

iot-keystone:
  container_name: iot-keystone
  hostname: iot-keystone
  image: telefonicaiot/fiware-keystone-spassword:latest
  links:
    - iot-mysql
  ports:
    - "5001:5001"
  environment:
    - LOG_LEVEL=DEBUG
  volumes:
    - ${PATH_LOG}/keystone:/var/log/keystone
    - ${PATH_KEYSTONE_KEYS)/keystone-keys:/etc/keystone/fernet-keys
  command: -dbhost iot-mysql -default_pwd XXX -mysql_pwd iotonpremise
  log_driver: json-file
  log_opt:
    max-size: "250m"

includes doc about issue #163

@@ -171,6 +171,21 @@ installed), the RPM package can be built invoking the following command:
sh ./package-keystone-spassword.sh
Copy link
Member

Choose a reason for hiding this comment

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

Do we have CNR in thi repo? Should something be added to it about the change done in this PR?

Copy link
Member

@fgalan fgalan left a comment

Choose a reason for hiding this comment

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

LGTM

@fgalan fgalan merged commit 4991233 into master Dec 16, 2020
@fgalan fgalan deleted the task/ensure_fernet_keys_folder_permissions branch December 16, 2020 15:40
@AlvaroVega AlvaroVega mentioned this pull request Dec 17, 2020
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.

2 participants