The package version may not include the "-nightly" suffix. Otherwise,
`pip install .` fails with:
setuptools.extern.packaging.version.InvalidVersion: Invalid version: '...-nightly'
When running `do cairn-createuser`, the created db did not have the
right name. If the user had been created earlier with SSO, then the
Clickhouse DB did not exist.
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!
Here, we make it possible to add a "-nightly" suffix to the package
version. This suffix will find its way to the Docker image tags. Thus,
the nightly branch will have different image tags. This will resolve
some confusion, as image tags are currently identical in nightly and
master.
CI was failing during `apt update` with the following error:
E: The repository 'https://repo.clickhouse.com/deb/stable main/ Release' does not have a Release file.
Note that this error was occurring in the UK and Germany, but not in
France.
Clickhouse client command was triggering a permission denied error:
$ tutor local run cairn-clickhouse cairn-client
Code: 586. DB::ErrnoException: Cannot create file:
//.clickhouse-client-history, errno: 13, strerror: Permission denied.
(CANNOT_CREATE_FILE)
Traceback (most recent call last):
File "/scripts/cairn", line 161, in <module>
main()
File "/scripts/cairn", line 62, in main
args.func(args)
File "/scripts/cairn", line 68, in command_client
subprocess.check_call(get_client_command())
File "/usr/lib/python3.8/subprocess.py", line 364, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['clickhouse', 'client',
'--host', 'cairn-clickhouse', '--port', '9000', '--user', 'openedx',
'--password', 'xrIDmynTAiDuJeVZc5ar', '--multiline', '--multiquery',
'--database', 'openedx']' returned non-zero exit status 74.
This was because the home of the cairn client was set to "/".
"Course completion" was causing browser crashes when it attempted to display
10k course items (which is the maximum allowed). Displaying 10000 bars does not
make sense anyway, so we threshold this number to 100.
Superset was ignoring the cairn-redis container, so we decided to remove it.
But the superset celery worker was writing to the /0 database, which conflicts
with the default celery database from the lms/cms. We fix this issue by running
in a different database.
The CELERY_RESULT_BACKEND seemed to be completely useless, so we got rid of it.
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.
When creating a new user, but with an email address already associated
to a different user, the `cairn createuser` script crashes with an
obscure error. This slightly improves the error messsage.
See: https://discuss.openedx.org/t/cairn-password-reset/8226
This is necessary for some customers who are not using the
common.djangoapps.track.shim.LegacyFieldMappingProcessor in
settings.EVENT_TRACKING_BACKENDS.
