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.

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

Added jobs into yaml tests #980

Merged
merged 13 commits into from
Nov 22, 2019
Merged

Added jobs into yaml tests #980

merged 13 commits into from
Nov 22, 2019

Conversation

miguelgfierro
Copy link
Collaborator

Description

Added jobs in the yaml files

Also fixed a bug in the test pipeline of gpu https://dev.azure.com/best-practices/recommenders/_build/results?buildId=14699 due to a wrong placement of the time out input

Also added durations 0 to list computation times for the tests

Related Issues

Checklist:

  • I have followed the contribution guidelines and code style for this project.
  • I have added tests covering my contributions.
  • I have updated the documentation accordingly.

@miguelgfierro
Copy link
Collaborator Author

miguelgfierro commented Nov 15, 2019

this is the error I'm getting:

/tests/ci/azure_pipeline_test/dsvm_unit_win_cpu.yml: (Line: 27, Col: 5, Idx: 539) - (Line: 27, Col: 5, Idx: 539): While scanning a simple key, could not find expected ':'.


/tests/ci/azure_pipeline_test/dsvm_unit_linux_cpu.yml: (Line: 36, Col: 5, Idx: 877) - (Line: 36, Col: 5, Idx: 877): While scanning a simple key, could not find expected ':'.

I need to investigage

@yueguoguo
Copy link
Collaborator

Was there the same error before change? @miguelgfierro

@miguelgfierro
Copy link
Collaborator Author

the env might not be installed correctly, there error is:

[command]/bin/bash --noprofile --norc /data/home/recocat/agent/_work/_temp/eeef47ef-e675-4509-b0c2-f0eb93808f38.sh
Could not find conda environment: reco_gpu
You can list all discoverable environments with `conda info --envs`.

@miguelgfierro
Copy link
Collaborator Author

added the conda env to the machine

@yueguoguo
Copy link
Collaborator

LGTM! Thanks @miguelgfierro

@miguelgfierro miguelgfierro merged commit 239acc9 into staging Nov 22, 2019
@miguelgfierro miguelgfierro deleted the miguel/test_jobs branch November 22, 2019 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants