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

Add CI and dependabot GitHub actions #2021

Merged
merged 2 commits into from
Dec 13, 2024
Merged

Add CI and dependabot GitHub actions #2021

merged 2 commits into from
Dec 13, 2024

Conversation

LuiggiTenorioK
Copy link
Member

Now that we are on GitHub, we need our CI pipeline ASAP 🚀

@LuiggiTenorioK LuiggiTenorioK self-assigned this Dec 13, 2024
@LuiggiTenorioK LuiggiTenorioK marked this pull request as draft December 13, 2024 07:32
@LuiggiTenorioK LuiggiTenorioK marked this pull request as ready for review December 13, 2024 07:36
@codecov-commenter
Copy link

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@LuiggiTenorioK LuiggiTenorioK requested a review from kinow December 13, 2024 07:38
Copy link
Member

@kinow kinow left a comment

Choose a reason for hiding this comment

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

Thank you!


# - name: Lint code
# run: |
# ruff check .
Copy link
Member

Choose a reason for hiding this comment

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

💥 already foreseeing our next step here! Thanks!

@LuiggiTenorioK LuiggiTenorioK merged commit 8a1b2df into master Dec 13, 2024
5 checks passed
@LuiggiTenorioK LuiggiTenorioK deleted the gh-action branch December 13, 2024 08:03
dbeltrankyl pushed a commit that referenced this pull request Dec 13, 2024
* add CI and dependabot gh actions

* disable linting temporarily

(cherry picked from commit 8a1b2df)
dbeltrankyl added a commit that referenced this pull request Dec 16, 2024
* Fixed DB test

* Now the db test also checks the stat and job_names in the filesystem

* feedback

* Changed test name,
Added generic functions in the test to expand it with other run options

* Added few todos

* disabled the check exit code as it is not working on the pipeline and not important right now

* Adding more chunks to the wrapper success

* More detailed test

* Add CI and dependabot GitHub actions (#2021)

* add CI and dependabot gh actions

* disable linting temporarily

(cherry picked from commit 8a1b2df)

* Improved test output in case of failure

* more info

* more info

* Test adding +x

* added chmod after sending

* chmod added

---------

Co-authored-by: dbeltran <daniel.beltran@bsc.es>
Co-authored-by: Luiggi Tenorio <luiggibit@gmail.com>
dbeltrankyl added a commit that referenced this pull request Dec 16, 2024
…n scenearios

* Update VERSION

* added process_jobs_to_submit fixes runtime issue

* Added few todos that requires to change critical stuff

* Fix runtime issue, recovery now shows a better output

* Improved package test

* Improved package test

* Closing file descriptors on reconnect.

* Fixes an issue with logs with this sequence of commands: `create + run + create + recovery + run`

* Intregation test added, fixed few bugs

* Fixed DB test

* Now the db test also checks the stat and job_names in the filesystem

* feedback

* Changed test name,
Added generic functions in the test to expand it with other run options

* Added few todos

* disabled the check exit code as it is not working on the pipeline and not important right now

* Adding more chunks to the wrapper success

* More detailed test

* Add CI and dependabot GitHub actions (#2021)

* add CI and dependabot gh actions

* disable linting temporarily

(cherry picked from commit 8a1b2df)

* Improved test output in case of failure

* more info

* more info

* Test adding +x

* added chmod after sending

* chmod added

---------

Co-authored-by: dbeltran <daniel.beltran@bsc.es>
Co-authored-by: Luiggi Tenorio <luiggibit@gmail.com>

---------

Co-authored-by: dbeltran <daniel.beltran@bsc.es>
Co-authored-by: Luiggi Tenorio <luiggibit@gmail.com>
dbeltrankyl added a commit that referenced this pull request Dec 16, 2024
…n scenearios

* Update VERSION

* added process_jobs_to_submit fixes runtime issue

* Added few todos that requires to change critical stuff

* Fix runtime issue, recovery now shows a better output

* Improved package test

* Improved package test

* Closing file descriptors on reconnect.

* Fixes an issue with logs with this sequence of commands: `create + run + create + recovery + run`

* Intregation test added, fixed few bugs

* Fixed DB test

* Now the db test also checks the stat and job_names in the filesystem

* feedback

* Changed test name,
Added generic functions in the test to expand it with other run options

* Added few todos

* disabled the check exit code as it is not working on the pipeline and not important right now

* Adding more chunks to the wrapper success

* More detailed test

* Add CI and dependabot GitHub actions (#2021)

* add CI and dependabot gh actions

* disable linting temporarily

(cherry picked from commit 8a1b2df)

* Improved test output in case of failure

* more info

* more info

* Test adding +x

* added chmod after sending

* chmod added

---------

Co-authored-by: dbeltran <daniel.beltran@bsc.es>
Co-authored-by: Luiggi Tenorio <luiggibit@gmail.com>

---------

Co-authored-by: dbeltran <daniel.beltran@bsc.es>
Co-authored-by: Luiggi Tenorio <luiggibit@gmail.com>
dbeltrankyl added a commit that referenced this pull request Dec 16, 2024
* Autosubmit config version compatibility fix ( rebasing )

Reverting a conflict in the rebase to the other option II ( tests are only failing in the CI/CD )

Reverting a conflict in the rebase to the other option

updated typing

added a missing return

Changed err_code function to return the err_code

Rebase fix

Squashed zombies/orphan and database fixes.

PIPELINE

Fix doc test

Fix pipeline

Added comments and typing (II )

Added comments and typing

Test pipeline ( now with output)

Test pipeline

Update as version

Fix tests, added pytest-xdist

!475 vertical regression test

[Single job] Test working, retrials working

Changes to stat, db and retrials (WIP)

Added database test, fixed some issues

reduced time.sleeps

Mock exists() so it works in the pipeline

Fix events queue being reset in a platform failure

clean import

added more tests

Changed the timeout

removed debug message

Added tests for signals

cleanup code

Added cleanup signal

changes to signals and waitpid

A new orphan/zombies code approach

WIP: Looking good, some more fixes neccesaries and ready to be pushed

WIP: Zombies/orphan processors logs fix

* Pipeline failing, debug (I)

* Fix Stat_0 file being deleted when it shouldn't

* Fixed issues with the db and logs not being recovered in different run scenearios 

* Update VERSION

* added process_jobs_to_submit fixes runtime issue

* Added few todos that requires to change critical stuff

* Fix runtime issue, recovery now shows a better output

* Improved package test

* Improved package test

* Closing file descriptors on reconnect.

* Fixes an issue with logs with this sequence of commands: `create + run + create + recovery + run`

* Intregation test added, fixed few bugs

* Fixed DB test

* Now the db test also checks the stat and job_names in the filesystem

* feedback

* Changed test name,
Added generic functions in the test to expand it with other run options

* Added few todos

* disabled the check exit code as it is not working on the pipeline and not important right now

* Adding more chunks to the wrapper success

* More detailed test

* Add CI and dependabot GitHub actions (#2021)

* add CI and dependabot gh actions

* disable linting temporarily

(cherry picked from commit 8a1b2df)

* Improved test output in case of failure

* more info

* more info

* Test adding +x

* added chmod after sending

* chmod added

---------

Co-authored-by: dbeltran <daniel.beltran@bsc.es>
Co-authored-by: Luiggi Tenorio <luiggibit@gmail.com>

---------

Co-authored-by: dbeltran <daniel.beltran@bsc.es>
Co-authored-by: Luiggi Tenorio <luiggibit@gmail.com>

---------

Co-authored-by: dbeltran <daniel.beltran@bsc.es>
Co-authored-by: Luiggi Tenorio <luiggibit@gmail.com>
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.

3 participants