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
Use fixed alpine version for upgrade/downgrade tests #5043
Conversation
417cabb
to
64353e3
Compare
Codecov Report
@@ Coverage Diff @@
## main #5043 +/- ##
==========================================
- Coverage 89.64% 89.59% -0.05%
==========================================
Files 227 227
Lines 51616 51593 -23
==========================================
- Hits 46270 46225 -45
- Misses 5346 5368 +22
Continue to review full report at Codecov.
|
8ad5b81
to
838f2fb
Compare
70b69c3
to
3ee8a0a
Compare
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.
Overall, it looks good to me. I just had a small note on the comments.
3ee8a0a
to
e775a13
Compare
a60565d
to
0e37fd9
Compare
1b1c9e2
to
778b478
Compare
778b478
to
3095124
Compare
Not specifying alpine version causes libssl version to change, which in turn cause error in downgrade tests as well as ABI tests. This commit also fixes shellcheck failures. Some failing windows tests are addd to ignore list. Co-authored-by: Lakshmi Narayanan Sreethar <lakshmi@timescale.com> Co-authored-by: Alexander Kuzmenkov <akuzmenkov@timescale.com> Signed-off-by: Sachin <sachin@timescale.com>
3095124
to
7c1929d
Compare
Updating alpine version causes libssl version to change, which in turn cause error in downgrade tests.