Linked Epic: https://github.com/overhangio/tutor/issues/1251
Some new rules that have been added are:
I: sort imports
N: check for pep8-naming standards
W292: check for missing extra line at EOF
Some of our lines, particularly in config, break the 88 character line length limit. For that, we add a # noqa: E501 to let ruff know to ignore the rule for that line
* fix: don't add cairn-postgresql as a dependency when CAIRN_RUN_POSTGRESQL is false
* fix: don't manage cairn-clickhouse permissions CAIRN_RUN_CLICKHOUSE is false
* fix: add missing CAIRN_POSTGRESQL_HOST and CAIRN_POSTGRESQL_PORT settings to allow usage with external postgresql DBs
pkg_resources is a package that is unavailable in python 3.12, unless
setuptools is explicitely installed. Turns out, there are replacement
functions coming from importlib_resources, which can be obtained from
the importlib-resources pypi package. This package will be installed
with tutor starting from 17.0.2.
This upgrade includes SSO login. Finally! We can now login to Superset
via the LMS. Thanks to OARS for providing the base security manager
implementation!
Vector is spamming us with messages concerning absent containers. Such messages include:
tutor_local-cairn-vector-1 | 2022-10-07T13:09:27.053010Z INFO source{component_kind="source" component_id=docker_logs component_type=docker_logs component_name=docker_logs}: vector::internal_events::docker_logs: Started watching for container logs. container_id=680c2bf183676caadee2c165b26493f90fb5d02cb15b9f2579a106fdd436f827
Or:
{“log”:“2022-10-07T07:19:17.839042Z INFO vector::internal_events::docker_logs: Started watching for container logs. container_id=a932d5916f9a1d1848224b71070591d16c813e83796d6850fa64b0a5f956d5ed\n”,“stream”:“stderr”,“time”:“2022-10-07T07:19:17.83961548Z”}
{“log”:“2022-10-07T07:19:17.839311Z INFO vector::internal_events::docker_logs: Stopped watching for container logs. container_id=cb9a4330dc154e02f02f6f6b49d9ddb04b5357083becdf6e9c5cbf49f4d01f68\n”,“stream”:“stderr”,“time”:“2022-10-07T07:19:17.83967086Z”}
See also: https://github.com/vectordotdev/vector/issues/8881
This issue has affected multiple users in the past. To address it, we first
tried to upgrade Vector, which did not help. Thus, we decided to reduce the
verbosity of Vector.
