-
Notifications
You must be signed in to change notification settings - Fork 1
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
Allow running several CI steps with the view #1
Comments
It looks like having "proper" steps would require to split up the action into several smaller actions, e.g. a A slightly different approach, that could work with the current action already would be to explicitly introduce grouping of log lines in the workflows directly, essentially: run: |
echo "::group::Build"
# build commands
echo "::endgroup::"
echo "::group::Run tests"
# test commands
echo "::endgroup::" This results in collapsed log lines for the groups, e.g. all build outputs will be collapsed under a To group the setup parts of the current action it would also be rather straight forward to introduce some logging groups in the setup scripts, that would at least make the payloads easier to see. |
FYI, eic/run-cvmfs-osg-eic-shell#6 implements this in our EIC workflow environments (we only use singularity, though). Requesting the same container will reuse the container instance. Requesting a different container will start a new one. |
Ah very nice. In principle that should work very similar with docker (apart from the slightly different syntax for getting running instances and getting a shell in them) From what it looks like the EIC solution uses images that already have the complete environment setup inside of them? So for the use case here, we would essentially have to inject a |
Yes, our containers include the whole kitchen sink. |
It's arguably nicer to have the different phases of a CI job - which are usually configure, compile, test, install - in their own steps, as this makes the logs more accessible in github and shows at a glance where an error occured.
The text was updated successfully, but these errors were encountered: