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

refactor function mesh ci integration tests #148

Merged
merged 48 commits into from
May 7, 2021

Conversation

freeznet
Copy link
Member

@freeznet freeznet commented May 1, 2021

  • make function mesh integration tests actually process messages from topic, to test the whole workflow.
  • moves all function tests into one integration test action to save CI resources.
  • add function sample images for Java, Python and Go.

@freeznet freeznet self-assigned this May 1, 2021
@freeznet freeznet requested a review from nlu90 as a code owner May 1, 2021 02:05
@freeznet freeznet force-pushed the freeznet/fix-integration-tests branch from cdcdabb to 9e0eebe Compare May 2, 2021 08:25
@freeznet freeznet force-pushed the freeznet/fix-integration-tests branch from 56f9fc4 to 60dd349 Compare May 3, 2021 04:55
@freeznet freeznet requested a review from tuteng May 3, 2021 07:16
@freeznet freeznet changed the title fix function mesh ci integration tests refactor function mesh ci integration tests May 3, 2021
@freeznet
Copy link
Member Author

freeznet commented May 6, 2021

@nlu90 @tuteng please help to review this PR if you have time, thanks.

images:
zookeeper:
repository: streamnative/pulsar-all
tag: 2.7.1
Copy link
Member

Choose a reason for hiding this comment

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

Set this tag to a variable?

Copy link
Member Author

@freeznet freeznet May 7, 2021

Choose a reason for hiding this comment

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

@tuteng I don't think helm values.yaml support variable in this way, since the values.yaml is a variable collection pass to helm template. But the values can be overwritten by using helm install --set images.zookeeper.tag=2.7.2.

ref: helm/helm#2133

@freeznet freeznet merged commit 21047b0 into master May 7, 2021
@freeznet freeznet deleted the freeznet/fix-integration-tests branch May 7, 2021 13:34
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.

3 participants