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
OCI: Fix bundle reload and activation unconditionally #4658
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good -- just a test would be really nice. Also @ashutosh-narkar knows this code much better than I do, so I'll wait for him to have a look.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding tests! I might have just missed it -- are we testing that we no longer unconditionally reload and activate?
t.Fatalf("expected true but got: %s", response) | ||
} | ||
|
||
expContains := "Bundle loaded and activated successfully" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm perhaps we should also check that the second pull did not do this? 🤔 Maybe I'm missing something... 🔍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rebased and added a verification that the bundle is skipped also in the e2e OCI test if it's the same on the pooling mechanism. I had to set the log level to debug in the test as the skip message is shown only on debug mode.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
|
||
time.Sleep(3 * time.Second) // wait for the downloader pooling mechanism to kick in | ||
expContains := "Bundle loaded and activated successfully" | ||
skipContains := "Bundle load skipped, server replied with not modified." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm. not exactly on-point, but fair enough 🤔
Could you please rebase and squash the commits? 😃 |
…d not change Signed-off-by: carabasdaniel <dani@aserto.com>
Using the etag in the OCI downloader to store the SHA fixes issue #4637