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

ci: diff back-to-back built images #250

Closed
wants to merge 2 commits into from
Closed

ci: diff back-to-back built images #250

wants to merge 2 commits into from

Conversation

zeha
Copy link
Member

@zeha zeha commented Nov 18, 2023

Only known differences are accepted. Useful for finding new, unexpected differences.

The current list of accepted differences is a start; IMO we should shrink it.

@adrelanos
Copy link
Contributor

This is probably considered blocked by:

But I think it should not be considered blocked because of that. I think this PR would be good to be merged rather sooner than later to avoid merge conflicts. It seems to be a nice feature towards reproducible builds.

While the actual test is still broken on CI, it's still adding informative output to grml-debootstrap. I might even send a PR to improve that debug output. Also any scripts to diff two different VM images are nice for local use.

As long as #252 is pending, simply keep the test disabled on CI so CI does not break until this gets fixed in the future?

@zeha
Copy link
Member Author

zeha commented Dec 8, 2023

simply keep the test disabled on CI

we can just keep the PR open, no?

Only known differences are accepted. Useful for finding new, unexpected
differences.

The current list of accepted differences is a start; IMO we should
shrink it.
@zeha zeha force-pushed the b2b branch 2 times, most recently from e0c41f8 to 54de541 Compare February 7, 2024 10:25
@zeha zeha closed this by deleting the head repository Aug 17, 2024
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