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
mgr/dashboard: Allow sourcing run-backend-api-tests.sh
#20874
mgr/dashboard: Allow sourcing run-backend-api-tests.sh
#20874
Conversation
aa32cc9
to
f5e8e34
Compare
Jenkins failed with
|
Jenkins retest please |
4ff5d37
to
94917d5
Compare
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.
LGTM - I like this approach.
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.
This is really really useful, many thanks!
Besides the comment I left, could you also update the HACKING documentation with instructions on how to use this script?
|
||
setup_teuthology |
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.
can we move this up to the sourced section? I think usually everyone wants to run this at the start, so it would be one less step to run.
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.
done
94917d5
to
1087f8c
Compare
sleep 5 | ||
|
||
cd $CURR_DIR | ||
rm -rf $TEMP_DIR |
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.
Consider removing the defined functions in the cleanup routine using unset -f <function_name>
. If this isn't done, the functions will stay available after clean_teuthology
has been called.
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.
fixed. Although, I'll now need to retest my latest changes again.
@rjfd: I just updated the documentation |
1087f8c
to
c9288ba
Compare
run-backend-api-tests.sh
run-backend-api-tests.sh
This allows us to re-run tests faster. Also updated HACKING.rst with the latest changes. Signed-off-by: Sebastian Wagner <sebastian.wagner@suse.com>
c9288ba
to
f4d2f01
Compare
This allows us to re-run tests faster.
Signed-off-by: Sebastian Wagner sebastian.wagner@suse.com