This repository has been archived by the owner on Sep 5, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 13
lti changes #644
Open
rupeshparab
wants to merge
61
commits into
main
Choose a base branch
from
lti_changes
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
lti changes #644
Changes from all commits
Commits
Show all changes
61 commits
Select commit
Hold shift + click to select a range
7ec5396
lti changes
rupeshparab ee0dfd7
added illumidesk secret option
Abhi94N d07623c
Dockerfile changes
rupeshparab f30091c
add env variable
rupeshparab 32db3a5
initialization of secrets manager
Abhi94N 070ca4f
updated graderservice to add restart grader functionality
Abhi94N 4554626
updated route to restart grader
Abhi94N 2ad280f
added response code for route
Abhi94N 938df6a
updated formgrader secretsmanager version to a valid one
Abhi94N b8944e5
graderservice restart grader timeout 10 seconds
Abhi94N 48a6467
grader service sleep changed
Abhi94N 7ac98ac
updated to jupyterhub ltiauthenticator
Abhi94N abe0606
updated jinja2 and markupsafe version
Abhi94N 47bf382
remove markup package
Abhi94N 089e1c3
set jinja version to 3.0.3
Abhi94N 508d95d
removed markup safe
Abhi94N 28d73e9
updated jinja2 and packages dependent on jinja2
Abhi94N 96390e1
fixed jinja version
Abhi94N 6a7c7a1
update jedi version
Abhi94N 5bfdada
jinja 3.1.1
Abhi94N 0216198
oauthlib 3.1.1
Abhi94N 40f79ca
revert jinja2 to 3.0.3
Abhi94N 980b972
updated flask to version 2.1.0
Abhi94N cee4464
revert flask version
Abhi94N e2aceec
flask version 1.1.1
Abhi94N db33df4
update requirements
Abhi94N 784dd48
update requirements
Abhi94N 5d3a3f4
update test location in makefile
Abhi94N 8a4c151
removed asyncnbgrader test
Abhi94N 972352e
add async_test back
Abhi94N aa9d7da
uses secretmanager package to fetch secret values
Abhi94N e1fdbb5
fix role addition issue
rupeshparab 98d951e
update markup safe
Abhi94N 29cb562
update jinja2 version
Abhi94N fb09de6
updated flask to version 2.1.0
Abhi94N 0405223
updated jinja2
Abhi94N e2fb9a7
update jinja2 for async_nbgrader
Abhi94N 92db213
updated jinja2 to semver 2.10>=jinja2<3.1.0
Abhi94N 56d5cbb
updated nbconvert version
Abhi94N 9e6533a
jinja2 version downgrade to 3.1.0
Abhi94N 7ba436a
updated flask to 2.1.0
Abhi94N 13d4ad2
revert nbconvert
Abhi94N c365961
update werkzeug
Abhi94N a4691ea
update jinja2 to the latest version
Abhi94N a533e56
jinja 3.1
Abhi94N 08463b3
nbconvert version 6.4.3
Abhi94N 9da1fb6
reverted requirements txt
Abhi94N 7c53d06
updates ltiauthenticator to use pypi version
Abhi94N 83856ec
matched kubernetes version
Abhi94N b336d3f
update requirements.txt
Abhi94N a39211c
fix build
rupeshparab 819dbb7
updated graderservice.py
Abhi94N 6f05a41
updated secrets manager with restart grader route
Abhi94N 839a1a8
tests for secrets and rollow restart route
Abhi94N d29cbd7
merge secrets pr to include secretsarn
Abhi94N 17ea6c3
add default course hook for auth0
rupeshparab 1d8a951
Merge branch 'lti_changes' of github.com:IllumiDesk/illumidesk into l…
rupeshparab 9fbb70e
fix requirements
rupeshparab 1351cc1
update default course name
rupeshparab 6c0ce3f
add user to default course
rupeshparab 3abcc7f
deployment fixes
rupeshparab File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -15,4 +15,4 @@ sphinx_rtd_theme | |
sphinx-autodoc-typehints | ||
nbval | ||
requests-mock | ||
wheel | ||
wheel |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Has the removal of
GRADER_EXCHANGE_SHARED_PVC
been tested? I was still having issues after I removed the deployment. That being said I was testing it without efs access points as we didn't need that in the Oregon cluster. I can try again with a new image and approve this afterward. Let me know if you have a new image, if not I'll fetch this PR and create the image and once tested, approve this.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have created a new image of this with pr-71 tag