-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FIX] Load scheduling DSL TypeScript libraries in scheduler-server
#424
Conversation
Can you automate it instead just for good measure? We already have a set of scheduler e2e tests that should do the busy work of uploading a model and creating goals. |
@JoelCourtney Sure! I'll give it a shot. |
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.
Tested this branch against the UI e2e tests as well and all pass!
Not sure why this is failing:
The Edit: I think I've fixed this. The above error is strange as it makes it look like |
c58dc24
to
acbe765
Compare
acbe765
to
41ac03b
Compare
41ac03b
to
1c272a0
Compare
1c272a0
to
f29b554
Compare
f29b554
to
7b85970
Compare
7b85970
to
f9d5a87
Compare
@JoelCourtney Done :) |
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.
Did a quick manual check locally - intellisense is working in the scheduling goal editor again 🙏 Thanks for the quick turnaround, Paul 🎉
Description
Since splitting
scheduler-server
intoscheduler-worker
– and thereby relocating the scheduling DSL Node project – the scheduler server has not had access to static DSL libraries. This PR loads scheduling DSL TypeScript libraries from the scheduler server JAR instead of requiring Docker to be configured to copy in the directory and set an environment variable.Verification
.ts
files from thescheduler-server
JAR.Documentation
None.
Future work
None.