Skip to content

Conversation

staszczuk
Copy link
Collaborator

@staszczuk staszczuk commented Sep 1, 2025

  • Mark all dual host tests with dual marker
  • Move build steps to composite action
  • Update runner labels
  • Minor improvements

@staszczuk staszczuk force-pushed the pstaszczuk/dual_host_tests_workflow branch 3 times, most recently from 8e2d687 to c31dd07 Compare September 10, 2025 14:02
@staszczuk staszczuk force-pushed the pstaszczuk/dual_host_tests_workflow branch from c31dd07 to 1ae5143 Compare September 11, 2025 15:30
@staszczuk staszczuk changed the title Ci: Add dual host tests to workflows Ci: Workflow improvements Sep 12, 2025
@staszczuk staszczuk marked this pull request as ready for review September 12, 2025 09:40
Copy link
Collaborator

@DawidWesierski4 DawidWesierski4 left a comment

Choose a reason for hiding this comment

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

I love that we are streamlining the building
But i belive if we wan't to really integrate all of those elements lets settle on scripts

Eventually, in distant future we will switch to ansible
But for now lets make the bash work for us instead of repeating steps one after the other

Copy link
Collaborator

@DawidWesierski4 DawidWesierski4 left a comment

Choose a reason for hiding this comment

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

LGTM

@staszczuk staszczuk merged commit 18cdb7f into main Sep 16, 2025
17 of 18 checks passed
@staszczuk staszczuk deleted the pstaszczuk/dual_host_tests_workflow branch September 16, 2025 09:51
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.

2 participants