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

[DPE-2704] Integrate Kafka built from source #144

Merged
merged 2 commits into from
Oct 25, 2023

Conversation

deusebio
Copy link
Contributor

@deusebio deusebio commented Oct 11, 2023

This PR is to test that the charm work well with a new snap revision that was built from source. That revision is currently sitting on 3/edge/dpe-2704-kafka-from-source

Copy link
Contributor

@zmraul zmraul left a comment

Choose a reason for hiding this comment

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

Does it need approval, or is it just a test to check that it works?

@deusebio
Copy link
Contributor Author

Well, I'd say both. I open the PR to make sure it works, but also we want to merge this such that we pack SNAPs using canonical provided artifacts. So, please do make also sure that updating the litearls is all is needed and - if so -give me thumbs up

However, to merge this PR, we need to first merge this, release a new SNAP to 3/edge, and use that revision here.

@deusebio deusebio force-pushed the dpe-2704-kafka-from-source branch from d3988d7 to 19d7b8a Compare October 20, 2023 08:13
@deusebio
Copy link
Contributor Author

deusebio commented Oct 20, 2023

The snap has now been published to 3/edge with revision 21 here, so I have updated the PR which is now ready to be merged

@deusebio deusebio force-pushed the dpe-2704-kafka-from-source branch from 19d7b8a to 48c5f63 Compare October 24, 2023 11:17
@deusebio deusebio merged commit 1426c8d into main Oct 25, 2023
15 checks passed
@deusebio deusebio deleted the dpe-2704-kafka-from-source branch October 25, 2023 20:01
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