Skip to content

Conversation

@ehelms
Copy link
Member

@ehelms ehelms commented Sep 30, 2025

No description provided.

@ehelms
Copy link
Member Author

ehelms commented Sep 30, 2025

Related is southalc/podman#119 to ensure true idempotency.


in_sync = current_checksum == desired_checksum
Puppet.debug("Current checksum: #{current_checksum}, Desired checksum: #{desired_checksum}. In sync: #{in_sync}")
File.exist?(File.join(resource[:destination], '.iop_frontend_checksum'))
Copy link
Member

Choose a reason for hiding this comment

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

I've been starring at this, thinking "but this breaks updates", but it doesn't because you define a insync? below, that ensures the checksum again?

Copy link
Member

@evgeni evgeni left a comment

Choose a reason for hiding this comment

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

There are now three occurrences of (metadata_file = )?File.join(resource[:destination], '.iop_frontend_checksum'), I guess it's time to make it a method?

Otherwise LGTM once tests pass

@ehelms
Copy link
Member Author

ehelms commented Oct 1, 2025

The tests will pass once southalc/podman#119 is merged. This change, and behaviour unconvered that failure in puppet-podman design.

@evgeni evgeni added the Bug Something isn't working label Oct 1, 2025
@evgeni evgeni merged commit 048a3fd into master Oct 1, 2025
9 checks passed
@evgeni evgeni deleted the fix-iop-frontend branch October 1, 2025 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants