Skip to content

Add ci tests#26

Merged
gliargovas merged 27 commits intomainfrom
add-ci-tests
Apr 3, 2023
Merged

Add ci tests#26
gliargovas merged 27 commits intomainfrom
add-ci-tests

Conversation

@gliargovas
Copy link
Copy Markdown
Collaborator

@gliargovas gliargovas commented Mar 31, 2023 •
edited
Loading

Addresses issue #25

@gliargovas gliargovas added work in progress priority Try to resolve this issue before others labels Mar 31, 2023
@gliargovas gliargovas self-assigned this Mar 31, 2023
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 31, 2023

OS:ubuntu-22.04
Fri Mar 31 03:21:18 UTC 2023
Summary: 0/6 tests passed.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 31, 2023

OS:ubuntu-22.04
Fri Mar 31 15:58:05 UTC 2023
Summary: 0/6 tests passed.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 31, 2023

OS:ubuntu-22.04
Fri Mar 31 16:14:14 UTC 2023
Summary: 4/6 tests passed.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 31, 2023

OS:ubuntu-22.04
Fri Mar 31 17:25:55 UTC 2023
Summary: 4/5 tests passed.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 31, 2023

OS:ubuntu-22.04
Fri Mar 31 17:42:21 UTC 2023
Summary: 6/6 tests passed.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 2, 2023

OS:ubuntu-22.04
Sun Apr 2 13:55:00 UTC 2023
Summary: 3/6 tests passed.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 2, 2023

OS:ubuntu-22.04
Sun Apr 2 15:32:55 UTC 2023
Summary: 6/6 tests passed.

@gliargovas gliargovas linked an issue Apr 2, 2023 that may be closed by this pull request
Add tests in CI #25
Closed
@gliargovas gliargovas requested a review from angelhof April 2, 2023 15:34
@gliargovas
Copy link
Copy Markdown
Collaborator Author

gliargovas commented Apr 2, 2023

  • Added basic CI config for scheduler tests.
  • Changed test script to automatically generate bash and orch scritps from a single template.
  • Changed test script to clear Riker cache before running each test as this may have led to false positives.
  • Tests can now be run from any directory.
  • Discovered a tracing issue on the way, fixed it.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 2, 2023

OS:ubuntu-22.04
Sun Apr 2 19:43:56 UTC 2023
Summary: 6/6 tests passed.

@gliargovas
Copy link
Copy Markdown
Collaborator Author

gliargovas commented Apr 2, 2023 •
edited
Loading

  • Fixed directory path resolution for mkdirs. Now every dependency has an absolute path.

@gliargovas gliargovas added ready for review This pull request is ready for review and removed work in progress labels Apr 2, 2023
angelhof
angelhof approved these changes Apr 3, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@angelhof angelhof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!

@gliargovas gliargovas merged commit 00fc237 into main Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@angelhof angelhof angelhof approved these changes

Assignees

@gliargovas gliargovas

Labels

priority Try to resolve this issue before others ready for review This pull request is ready for review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add tests in CI

2 participants

@gliargovas @angelhof