Michael Edgar

Results 220 comments of Michael Edgar

Fair enough, I'll push a fix for this to ignore duplicates when the duplicate is an override.

> This looks pretty messy to be honest. Beauty is in the eye of the beholder :rofl: Anything in particular or all of it?

> The assembly stuff is IMHO required for the build process. So not wure what sense does it make to disable it. The idea is that it's not really useful...

Job failure seem unrelated: ``` tar: c7a4a6b4c0744e558980828618275b4d_archive.tar: Cannot write: No space left on device tar: Error is not recoverable: exiting now ApplicationInsightsTelemetrySender correlated 1 events with X-TFS-Session c8e8b170-2700-4668-9cb4-4e70bf2050d4 ##[error]Process returned...

How do we proceed here? As far as discussion of the issues, the ones addressed with this PR are: 1. Use the maven-resources-plugin to place files from the project root...

> I don't think we should not be strict on what maintainers are using. People can move across companies and can get restrictions on IDEs to use, so what I...

Is there any reason `kafka-versions.yaml` must be in the project root as opposed to `cluster-operator/src/main/resources`? Of course it's convenient to see and edit it in the root, but ultimately it's...

When I say that it's a cluster operator resource, I mean that it's bundled with and used by that module exclusively at runtime. In other words, multiple modules don't use...

My next suggestion is to simply document in the development guide how a user could make the POM changes locally to work around the Eclipse bug. Would that be acceptable,...

@elmuerte, if you have some time, would you mind trying this change with the test setup you had in #1752 ?