apluslms / gitmanager

Git manager service for A+. Separated from the MOOC-Grader.
0 stars 6 forks source link

Add config caching and transferring + minor fixes and changes #11

Closed lainets closed 2 years ago

lainets commented 2 years ago

Description

What?

Adds config caching for the different build stages and transferring the configs between them to speed up the build process. Also means that the separate processes do not need to load the configs separately.

Fixes a bug with HOME variable missing in git calls.

Also contains some smaller fixes and architectural changes.

Why?

Speed up the build process.

HOME variable is needed for git to find the config files.

How?

Store the configs in a Django cache where each build stage is cached separately. Transfer the configs in the cache when the configs are transferred in the fiesystem. This eliminates the need to reload the config after each build stage.

Fixes #1. Fixes #2.

Testing

What type of test did you run?

Tested that there are no difference between the config loaded from the filesystem and the config fetched from the cache on a couple courses. Should be no problems.

Did you test the changes in

Programming style

Have you updated the README or other relevant documentation?

Is it Done?