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

Kibana public/private access using IAM #4240

Merged
merged 8 commits into from
Jan 31, 2022
Merged

Kibana public/private access using IAM #4240

merged 8 commits into from
Jan 31, 2022

Conversation

azhavoro
Copy link
Contributor

@azhavoro azhavoro commented Jan 26, 2022

Removed django-revproxy, configured traefik to route kibana requests

Motivation and context

resolve #4085

How has this been tested?

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) 2022 Intel Corporation
#
# SPDX-License-Identifier: MIT

@azhavoro azhavoro changed the title [WIP] Az/fix kibana Az/fix kibana Jan 27, 2022
@azhavoro azhavoro marked this pull request as ready for review January 27, 2022 10:59
@nmanovic nmanovic changed the title Az/fix kibana Kibana public/private access using IAM Jan 27, 2022
components/analytics/kibana_conf.yml Outdated Show resolved Hide resolved
cvat/apps/iam/permissions.py Outdated Show resolved Hide resolved
cvat/apps/iam/rules/analytics.csv Outdated Show resolved Hide resolved
cvat/apps/log_viewer/views.py Outdated Show resolved Hide resolved
@nmanovic
Copy link
Contributor

@azhavoro , it looks great. Need to add one more rule for admin into csv and I will approve and merge.

@nmanovic nmanovic merged commit 976e403 into develop Jan 31, 2022
@nmanovic nmanovic deleted the az/fix_kibana branch January 31, 2022 21:19
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.

Django-revproxy is incompatible with Django 3.2
2 participants