Skip to content
Snippets Groups Projects

manager: tooling fixes & ci integration

Merged Vaclav Sraier requested to merge manager-tooling-fixes into manager
All threads resolved!

The tooling of manager is broken due to changes in paths after integration with kresd's repository. This should mainly fix the linting step in CI and all other tools as well.

Edited by Vaclav Sraier

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Tomas Krizek marked this merge request as draft

    marked this merge request as draft

  • Tomas Krizek added 2 commits

    added 2 commits

    • 526ab0f7 - ci/images: install KnotDNS with apkg as regular package
    • fe5f2687 - WIP ci

    Compare with previous version

  • Tomas Krizek added 1 commit

    added 1 commit

    Compare with previous version

  • Tomas Krizek added 1 commit

    added 1 commit

    Compare with previous version

  • Tomas Krizek added 5 commits

    added 5 commits

    • eb90aca4 - ci: execute manager unit tests
    • 9d9c9461 - manager: remove pyright in favor of mypy
    • 69cc42a7 - manager: remove obsolete integration test framework
    • 24e03c5b - manager: remove ununsed containers
    • 8c4d12b5 - ci: make pytests behavior consistent with remaining jobs

    Compare with previous version

  • Tomas Krizek resolved all threads

    resolved all threads

  • Tomas Krizek marked this merge request as ready

    marked this merge request as ready

  • Tomas Krizek changed title from Draft: manager: tooling fixes to manager: tooling fixes & ci integration

    changed title from Draft: manager: tooling fixes to manager: tooling fixes & ci integration

  • Tomas Krizek added 4 commits

    added 4 commits

    • ea78fe2b - manager: remove pyright in favor of mypy
    • a465e8a6 - manager: remove obsolete integration test framework
    • e136e59c - manager: remove ununsed containers
    • e2e6efc8 - ci: make pytests behavior consistent with remaining jobs

    Compare with previous version

  • This is good to go, summary of changes:

    • remove pre-loading of poetry dependencies - it was too much extra complexity not worth the negligable speedup
    • add Fedora container because it has easily installable python interpreters
    • run unit tests on all supported python versions
    • remove deprecated manager's containers
    • remove the remainders of integration test runner - will be replaced by packaging tests
    • remove pyright in favor of mypy
    Edited by Tomas Krizek
  • Tomas Krizek requested review from @vsraier

    requested review from @vsraier

  • Tomas Krizek added 2 commits

    added 2 commits

    • 2dc29927 - manager: remove ununsed containers
    • 44b7afc9 - ci: make pytests behavior consistent with remaining jobs

    Compare with previous version

  • Vaclav Sraier
  • Vaclav Sraier
  • Tomas Krizek added 2 commits

    added 2 commits

    • 4d4aa5cd - fixup! manager: scripts: fixed after changes in repository structure, added missing dependencies
    • 400dcf95 - DROP

    Compare with previous version

  • Tomas Krizek marked this merge request as draft from 4d4aa5cd

    marked this merge request as draft from 4d4aa5cd

  • Tomas Krizek added 1 commit

    added 1 commit

    • 19496244 - ci/manager: use junit test report

    Compare with previous version

  • Tomas Krizek added 1 commit

    added 1 commit

    • 9e60e8bc - fixup! ci/manager: use junit test report

    Compare with previous version

  • Vaclav Sraier mentioned in merge request !1252 (merged)

    mentioned in merge request !1252 (merged)

  • Vaclav Sraier added 1 commit

    added 1 commit

    • b2b7e5f3 - fixup! manager: scripts: fixed after changes in repository structure, added missing dependencies

    Compare with previous version

  • Vaclav Sraier added 7 commits

    added 7 commits

    • b845f9f5 - manager: scripts: fixed after changes in repository structure, added missing dependencies
    • 77d2035b - manager: ci container: update URL for installing poetry due to upstream change
    • a054fe43 - ci: execute manager unit tests
    • a66422ff - manager: remove pyright in favor of mypy
    • 5d32438e - manager: remove ununsed containers
    • de9b6f3a - ci: make pytests behavior consistent with remaining jobs
    • 230ddfb9 - ci/manager: use junit test report

    Compare with previous version

  • Vaclav Sraier resolved all threads

    resolved all threads

  • Vaclav Sraier marked this merge request as ready

    marked this merge request as ready

  • Vaclav Sraier enabled an automatic merge when the pipeline for 230ddfb9 succeeds

    enabled an automatic merge when the pipeline for 230ddfb9 succeeds

  • Author Maintainer

    Merging even though some things are not perfect, because this branch is a blocker for other tasks. CI can be improved later.

  • Vaclav Sraier mentioned in commit e70ca7ac

    mentioned in commit e70ca7ac

  • merged

  • mentioned in issue #717 (closed)

  • Vaclav Sraier mentioned in commit 0d2d0afd

    mentioned in commit 0d2d0afd

  • Vaclav Sraier mentioned in commit 4857b543

    mentioned in commit 4857b543

  • Please register or sign in to reply
    Loading