Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Manually install some packages from conda-forge #729

Merged
merged 5 commits into from
Mar 18, 2020
Merged

Manually install some packages from conda-forge #729

merged 5 commits into from
Mar 18, 2020

Conversation

shankari
Copy link
Contributor

@AmplabJenkins
Copy link

Merged build finished. Test FAILed.

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/e-mission-server-prb/1173/
Test FAILed.

@AmplabJenkins
Copy link

Merged build finished. Test FAILed.

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/e-mission-server-prb/1174/
Test FAILed.

@shankari
Remove the prefix creation for every run
e66fb38 
Since we are moving to a model where we run the tests in separate containers anyway
@AmplabJenkins
Copy link

Merged build finished. Test FAILed.

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/e-mission-server-prb/1175/
Test FAILed.

shankari added 2 commits March 17, 2020 17:08
@shankari
Replace hardcodeded localhost URL for some of the tests
057e5c9
@shankari
Fix the teardown tests to match the setup tests
2a373f6 
In particular,
- change the environment name to match e66fb38
- remove the lines that clear out config files since they are cleared out
  automatically in the tests
@AmplabJenkins
Copy link

Merged build finished. Test FAILed.

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/e-mission-server-prb/1176/
Test FAILed.

@shankari
Copy link
Contributor Author

Ran tests in a docker container after mounting code checked out from this branch.

$ source setup/setup_tests.sh
$ ./runAllTests.sh
$ source setup/teardown_tests.sh

All tests passed. The jenkins tests are just completely broken due to lack of conda so I am not going to rely on them.

web-server_1  | ----------------------------------------------------------------------
web-server_1  | Ran 211 tests in 1091.963s
web-server_1  |
web-server_1  | OK
web-server_1  |
...
em-server_web-server_1 exited with code 0

@shankari shankari merged commit 62c56fd into e-mission:master Mar 18, 2020
@shankari shankari deleted the stabilize_tests branch October 26, 2020 18:03
jf87 pushed a commit to jf87/e-mission-server that referenced this pull request Jun 21, 2021
@shankari
Merge pull request e-mission#729 from shankari/stabilize_tests
a0df328 
Manually install some packages from conda-forge
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@shankari @AmplabJenkins