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

Fix integrated service tests #3612

Merged
merged 3 commits into from
Apr 6, 2022
Merged

Fix integrated service tests #3612

merged 3 commits into from
Apr 6, 2022

Conversation

akijakya
Copy link
Contributor

@akijakya akijakya commented Mar 31, 2022

Q A
Bug fix? yes
New feature? no
API breaks? no
Deprecations? no
Related tickets -
License Apache 2.0

What's in this PR?

  • Fixed hashFiles function in is-test workflows.
  • Updated versions of services in test environment to be in sync with what is being used in the dev env.
  • Kubernetes version 1.21.1 has been specified in the Makefile to be used by kind when creating a cluster to run tests against.

Why?

In a recent change hashfiles function throws an error instead of failing silently causing these is-test workflows to fail.
Also the latest Ubuntu images that are used in the Github actions use the latest Kind version which starts kubernetes v1.23.4 version clusters by default, causing the integrated services v2 test to fail.

Checklist

  • Implementation tested (with at least one cloud provider)
  • Code meets the Developer Guide
  • OpenAPI and Postman files updated (if needed)
  • User guide and development docs updated (if needed)
  • Related Helm chart(s) updated (if needed)

@akijakya akijakya force-pushed the fix/is-test branch 3 times, most recently from 01af690 to 6aeff39 Compare April 5, 2022 11:10
The latest ubuntu images that are used in the Github actions use the latest Kind version which starts kubernetes v1.23.4 version clusters by default, causing the integrated services v2 test to fail.
@akijakya akijakya changed the title Fix hashfiles function in is-test workflows Fix integrated service tests Apr 6, 2022
@akijakya akijakya merged commit 73d5a14 into master Apr 6, 2022
@akijakya akijakya deleted the fix/is-test branch April 6, 2022 07:33
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