Skip to content

Adds an integration workflow that exercises the formula within a watchmaker build - #642

Merged
lorengordon merged 2 commits into
masterfrom
ci/integration-workflow
Jul 9, 2026
Merged

Adds an integration workflow that exercises the formula within a watchmaker build#642
lorengordon merged 2 commits into
masterfrom
ci/integration-workflow

Conversation

@lorengordon

@lorengordon lorengordon commented Jul 8, 2026

Copy link
Copy Markdown
Member

No description provided.

lorengordon

This comment was marked as outdated.

Sign in to view

lorengordon

This comment was marked as outdated.

Sign in to view

@lorengordon
lorengordon force-pushed the ci/integration-workflow branch from a3b516e to 0a9496e Compare July 8, 2026 21:51
lorengordon

This comment was marked as outdated.

Sign in to view

@lorengordon
lorengordon force-pushed the ci/integration-workflow branch from 0a9496e to bacaaab Compare July 8, 2026 21:53
lorengordon

This comment was marked as outdated.

Sign in to view

@lorengordon
lorengordon force-pushed the ci/integration-workflow branch from bacaaab to 5443521 Compare July 8, 2026 21:55
lorengordon

This comment was marked as outdated.

Sign in to view

@lorengordon
lorengordon force-pushed the ci/integration-workflow branch from 5443521 to 9f95910 Compare July 8, 2026 22:07
lorengordon

This comment was marked as outdated.

Sign in to view

@lorengordon
lorengordon force-pushed the ci/integration-workflow branch from 9f95910 to b684190 Compare July 8, 2026 22:24
lorengordon

This comment was marked as outdated.

Sign in to view

@lorengordon
lorengordon force-pushed the ci/integration-workflow branch from b684190 to 21e96ca Compare July 8, 2026 22:36
lorengordon

This comment was marked as outdated.

Sign in to view

@lorengordon

lorengordon commented Jul 8, 2026

Copy link
Copy Markdown
Member Author

Reviewing the watchmaker build logs, can see watchmaker retrieves the source from the PR branch archive...

2026-07-08 22:43:50,795 [watchmaker.workers.base.SaltLinux][DEBUG][16757]: Downloading: https://github.com/plus3it/ash-linux-formula/archive/refs/heads/ci/integration-workflow.zip
2026-07-08 22:43:50,795 [watchmaker.workers.base.SaltLinux][DEBUG][16757]: Destination: /usr/tmp/watchmaker/workingfiles/salt-zsneug6o/integration-workflow.zip
2026-07-08 22:43:50,795 [watchmaker.workers.base.SaltLinux][DEBUG][16757]: Establishing connection to the host, https://github.com/plus3it/ash-linux-formula/archive/refs/heads/ci/integration-workflow.zip
2026-07-08 22:43:50,981 [watchmaker.workers.base.SaltLinux][DEBUG][16757]: Opening the file handle, /usr/tmp/watchmaker/workingfiles/salt-zsneug6o/integration-workflow.zip
2026-07-08 22:43:50,982 [watchmaker.workers.base.SaltLinux][DEBUG][16757]: Saving file to local filesystem...
2026-07-08 22:43:50,994 [watchmaker.workers.base.SaltLinux][INFO ][16757]: Retrieved the file successfully. url=https://github.com/plus3it/ash-linux-formula/archive/refs/heads/ci/integration-workflow.zip. filename=/usr/tmp/watchmaker/workingfiles/salt-zsneug6o/integration-workflow.zip
2026-07-08 22:43:50,994 [watchmaker.workers.base.SaltLinux][INFO ][16757]: Creating a working directory.
2026-07-08 22:43:50,994 [watchmaker.workers.base.SaltLinux][DEBUG][16757]: Created working directory: /usr/tmp/watchmaker/workingfiles/salt-zsneug6o/integration-workflow.zip-u1uzbo37
2026-07-08 22:43:50,995 [watchmaker.workers.base.SaltLinux][DEBUG][16757]: File Type: zip
2026-07-08 22:43:51,021 [watchmaker.workers.base.SaltLinux][INFO ][16757]: Extracted file. source=/usr/tmp/watchmaker/workingfiles/salt-zsneug6o/integration-workflow.zip, dest=/usr/tmp/watchmaker/workingfiles/salt-zsneug6o/integration-workflow.zip-u1uzbo37
2026-07-08 22:43:51,021 [watchmaker.workers.base.SaltLinux][DEBUG][16757]: Placing user formula in salt file roots. formula_url=https://github.com/plus3it/ash-linux-formula/archive/refs/heads/ci/integration-workflow.zip, formula_loc=/srv/watchmaker/salt/formulas/ash-linux-formula

@lorengordon
lorengordon marked this pull request as ready for review July 8, 2026 22:56
@lorengordon
lorengordon requested a review from a team July 8, 2026 22:56
@lorengordon
lorengordon force-pushed the ci/integration-workflow branch from 21e96ca to bfa13b0 Compare July 8, 2026 23:07
lorengordon

This comment was marked as outdated.

Sign in to view

@lorengordon
lorengordon force-pushed the ci/integration-workflow branch from bfa13b0 to 61f662a Compare July 9, 2026 00:01
lorengordon

This comment was marked as outdated.

Sign in to view

ferricoxide
ferricoxide previously approved these changes Jul 9, 2026
View reviewed changes

@ferricoxide ferricoxide left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved (though I did have questions about some "path forward" types of things

Comment thread .github/workflows/integration.yml
Comment thread .github/workflows/integration.yml Outdated
@mergify

mergify Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

lorengordon
lorengordon commented Jul 9, 2026
View reviewed changes

@lorengordon lorengordon left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

/build

@lorengordon
lorengordon force-pushed the ci/integration-workflow branch from 04b6074 to b344a5d Compare July 9, 2026 14:20
lorengordon
lorengordon commented Jul 9, 2026
View reviewed changes

@lorengordon lorengordon left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

/build

@ferricoxide
ferricoxide self-requested a review July 9, 2026 14:54
@lorengordon
lorengordon merged commit aa07979 into master Jul 9, 2026
21 checks passed
@lorengordon
lorengordon deleted the ci/integration-workflow branch July 9, 2026 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ferricoxide ferricoxide ferricoxide approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lorengordon @ferricoxide