Skip to content

Chronus#119

Merged
theyoyojo merged 3 commits into
masterfrom
chronus
Jun 3, 2024
Merged

Chronus#119
theyoyojo merged 3 commits into
masterfrom
chronus

Conversation

@charliemirabile

@charliemirabile charliemirabile commented May 26, 2024

Copy link
Copy Markdown
Contributor

This is an initial version of a new 'chronus' container that manages the list of assignments and their due dates and generates peer review assignments.

Currently email is still not added to the journal automatically. The expectation for making this work in production is that a cronjob should be setup that releases all email every 5m (or something like that) by executing podman-compose exec pop /usr/local/bin/init_journal /var/lib/email/journal/journal /var/lib/email/journal/temp /var/lib/email/mail.

This system merely regulates restricting access to the latest emails for specific students based on whether or not they submitted.

@theyoyojo

Copy link
Copy Markdown
Contributor

looks good except my test user can't see the peer review when triggering intitial.py after running email-test.sh on email-test-4

@theyoyojo

Copy link
Copy Markdown
Contributor

the peer review email is present in the volumes though

@theyoyojo theyoyojo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

charliemirabile and others added 3 commits June 2, 2024 20:56
This container manages the list of assignments and their due dates.
On startup and when reloaded, its main program reads the list of all
assignments, and spawns children proccesses to wait for the passage
of each of their initial and final submission due dates and run a
script then they occur.

It skips due dates that are already in the past, so this container can be
restarted safely without needing to make the scripts it runs idempotent.

Currently, the scripts are just stubs that print their arguments.

There is a `configure.sh` program similar to warpdrive that runs
`configure.py` within the container to manage the assignments and reload
the daemon after succesfully changing them.
Based on the list of students who submitted, we can form a cycle
of peers to review each other's patches.
@theyoyojo theyoyojo merged commit a022245 into master Jun 3, 2024
@theyoyojo theyoyojo deleted the chronus branch June 3, 2024 00:56
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