Closed evanyeyeye closed 4 months ago
This update introduces a feature allowing the disabling of network access for autograding containers on an assessment level in an educational platform. It includes backend and frontend changes to support this new option, including a database migration to store the setting, modifications to the model to handle the new attribute, and updates to the user interface to allow users to toggle this setting. Additionally, the integration with the autograding system, Tango, has been updated to accept this new parameter.
File Path | Change Summary |
---|---|
app/helpers/assessment_autograde_core.rb |
Added disable_network parameter to job properties for TangoClient. |
app/models/assessment.rb |
Included disable_network in serialized attributes. |
app/views/assessments/_edit_advanced.html.erb |
Added checkbox for disabling network access in UI. |
db/migrate/20240219175942_add_disable_network_to_assessments.rb |
Migration to add disable_network boolean column to assessments table. |
db/schema.rb |
Updated ActiveRecord schema version and added disable_network boolean column to courses table. |
Objective | Addressed | Explanation |
---|---|---|
Add checkbox in Autolab assessment settings to disable internet access of docker containers run for that assessment. (#2069) | ✅ | |
Add Tango endpoint support for this option using --network none . (#2069) |
❓ | The code changes mention adding a disable_network parameter, but the specific implementation detail of using --network none in Tango endpoint support is not explicitly confirmed. |
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
@coderabbitai review
Description
Added checkbox under the advanced tab for edit assessments to disable network access. Corresponding Tango PR #249.
Motivation and Context
Closes #2069.
How Has This Been Tested?
disable_network
field is included in the outgoing request to Tango.disable_network
field is false by default and modified by the checkbox.disable_network
field works in YAML config (false when undefined).Types of changes