-
Notifications
You must be signed in to change notification settings - Fork 116
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
RPM packages with maven #6
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Kalinga
pushed a commit
to Kalinga/kafka-topology-builder
that referenced
this pull request
Aug 24, 2020
* initial setup for building rpm when running mvn assembly:assembly * remove icon * clean up lefover soft link * update delivery jar file * clean up system out println statement in tests * detach the rpm plugin from the goals automatic execution * ammend packages names
sverrehu
referenced
this pull request
in sverrehu/julie
Sep 7, 2021
purbon
pushed a commit
that referenced
this pull request
Dec 10, 2021
* Refactor SyncTopicAction Split SyncTopicAction into separate actions for the different parts: create topic, update config and update schema reg * Only change topic configs that are actually changed Create a topic config update plan for the UpdateTopicConfigAction. And let the adminClient only update according to the plan. * run -> buildAndExecutePlan * applyCreate -> updatePlanWithPrincipalsCreation * apply -> updatePlan * applyDelete -> updatePlanWithPrincipalsDeletion * Auto-format * Spelling * bindings -> aclBindings * Make methods private. Add @OverRide annotations. Implement ManagerOfThings * ManagerOfThings -> ExecutionPlanUpdater * Make PrincipalManager an ExecutionPlanUpdater and simplify the plan update process * Revert "bindings -> aclBindings" This reverts commit 00bc474. * Explicit method calls per manager * buildAndExecutePlan -> run, as per review request. * Split PrincipalManager in two, as per review request. * Make method private * Remove unnecessary throws clause * dryRun: Only log actual topic config changes Log details on the actual topic config changes that will be done. Also fixed logic for detecting new configs added. * Renamed and fixed test for partition count update. * Remove comments * Test TopicConfigUpdatePlanBuilder and fix a bug (#12) Test logic for detecting new, updated and removed topic configs. * Rewrite deprecated code (#11) * Refactor bindings building (#6) * Merge upstream master and fix issues * Add both localhost and host.docker.internal to the cert, to make it usable on dind on Windows * Code style reformatting * Improve RegisterSchemaAction output * Print output for both dryrun and apply phases * Make log output format consistent for create and delete topic actions Co-authored-by: Sverre H. Huseby <shh@thathost.com>
purbon
pushed a commit
that referenced
this pull request
Sep 12, 2023
* fix: Apply formatting, using a formatter that supports Java 17 * fix: Don't throw exception if JulieOps is not allowed to delete topics (#3) * Don't throw exception if JuleOps is not allowed to delete topics. * Update dependencies (#2) * Update README.md * fix: Apply formatting, using a formatter that supports Java 17 * chore: Update dependencies * chore: Update dependencies * Update container versions * fix: Replace deprecated AssertJ method * chore: Upgrade Confluent version in workflow * fix: Replace deprecated methods * fix: Replace deprecated methods * chore: Update dependencies * fix: Extend connector path for new version of connect * fix: Revert ksqldb-server to a version that makes the test pass * fix: Correct return value from exception handler * fix: Hack around bug KAFKA-14325 * feat: Support finding topology files recursively (#4) * fix: Clean up zookeeper (#5) The Zookeeper dependency is required for a single test method only. This PR moves all ZK* classes to src/test, and marks the zookeeper dependency as test-coped. Unfortunately, recent versions of zookeeper drag in a logback dependency, that appears to turn on global DEBUG logging. So this PR also exludes those logback dependencies. Fixes #548 * fix: Allow logging of INFO+ from AdminClient (#6) * Excludes all Log4j2Plugins.dat from the shaded jar, to avoid Log4j2 startup problems (ERROR StatusLogger Unrecognized conversion specifier...) * Adds a log4j.properties that will be picked up by the Kafka dependencies * Create dependabot.yml * chore: Update plug-in versions * chore: Update jackson and redis to remove vulnerable json and snakeyaml packages * chore: Update plugins * chore: Update some deps * chore: Upgrade testcontainers * Updates (#8) * chore: Update some deps * chore: Update some deps * chore: Upgrade Confluent Platform * docs: Fetch changelog from v4.4.1 tag to get all history * chore: Remove unused imports and trailing whitespace (#9) * chore: Remove unused imports * docs: Remove trailing spaces * Update src/test/java/com/purbon/kafka/topology/integration/MDSBaseTest.java Co-authored-by: piotrsmolinski <piotr.smolinski.77@gmail.com> --------- Co-authored-by: piotrsmolinski <piotr.smolinski.77@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds an initial packages using the maven rpm:rpm command. This will generate a no-arch .rpm useful to be installed in any Redhat based environment.
Fixes #4