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

test: update integration tests script #25

Merged

Conversation

Unix4ever
Copy link
Member

Omni integration tests executable was changed, change the integration tests script to make it work again.
Also enable ConfigPatching tests.

@Unix4ever Unix4ever force-pushed the update-integration-tests-config branch from 48b89c4 to 73d13d0 Compare January 24, 2025 15:02
Copy link
Member

@utkuozdemir utkuozdemir left a comment

Choose a reason for hiding this comment

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

🆒

@Unix4ever Unix4ever force-pushed the update-integration-tests-config branch from 73d13d0 to 218f799 Compare January 24, 2025 15:35
Omni integration tests executable was changed, change the integration
tests script to make it work again.
Also enable `ConfigPatching` tests.

Co-authored-by: Utku Ozdemir <[email protected]>
Signed-off-by: Artem Chernyshev <[email protected]>
@Unix4ever Unix4ever force-pushed the update-integration-tests-config branch from 218f799 to 4581ace Compare January 24, 2025 16:39
@Unix4ever
Copy link
Member Author

I've merged the PR #22 into this one, as I guess the tests won't pass without it

@utkuozdemir
Copy link
Member

I've merged the PR #22 into this one, as I guess the tests won't pass without it

Good, I suspect there's another problem though. Let's see.

@Unix4ever
Copy link
Member Author

Seemed to work

@Unix4ever
Copy link
Member Author

/m

@talos-bot talos-bot merged commit 4581ace into siderolabs:main Jan 24, 2025
13 checks passed
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.

5 participants