-
Notifications
You must be signed in to change notification settings - Fork 223
Open
Labels
examplesImprovements to the example projectsImprovements to the example projectsgood first issueDenotes an issue ready for a new contributor, according to the "help wanted" guidelines.Denotes an issue ready for a new contributor, according to the "help wanted" guidelines.help wantedDenotes an issue that needs help from a contributor. Must meet "help wanted" guidelines.Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines.
Description
The idea of the samples should be that I should be able to grab folders outside repo and they should work.
From a maintenance standpoint and verification that might be bad as repo possibly needs to depend on the compiled examples etc. but from the usage point of view there is some extra work need to get this working outside the repository.
This would be something that simple CLI/Maven Artifacts could fix.
Happy to build CLI for java operator if needed and add some capability to generate operator.
Metadata
Metadata
Assignees
Labels
examplesImprovements to the example projectsImprovements to the example projectsgood first issueDenotes an issue ready for a new contributor, according to the "help wanted" guidelines.Denotes an issue ready for a new contributor, according to the "help wanted" guidelines.help wantedDenotes an issue that needs help from a contributor. Must meet "help wanted" guidelines.Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines.
Type
Projects
Milestone
Relationships
Development
Select code repository
Activity
[-]Samples are not easy to consume due to extensive usage of parents. [/-][+]Samples are not easy to consume due to usage of parent in pom[/+]s-soroosh commentedon Dec 16, 2020
Thank you @wtrocki for bringing this topic up, indeed using the samples should be simpler than what it is.
I agree with building an initiator to setup the project.
It could be used to address #28 and the current issue.
I would suggest we make a new repo for it rather than a new module here to ensure it's developed in an isolated environment and its releases not being dependent on the SDK releases.
What do you think @adam-sandor @csviri @metacosm?
metacosm commentedon Dec 16, 2020
I'd argue that the samples should maybe be in a different repository. A generator to generate a skeleton operator would also help.
adam-sandor commentedon Dec 16, 2020
Separate repos for each sample or single samples repo? I'd vote for single samples repo, so when the framework changes we can upgrade the samples with one PR.
csviri commentedon Dec 17, 2020
The original idea was, when we started to extract the samples, that we have just a few simple samples, that supports only the development maybe some quick checking / testing, that sample works with the update.
And other samples (which are more complex) should be extracted to a separate repo, I would say it's own. And there should be not too much of them.
adam-sandor commentedon Dec 17, 2020
I think single repo for all (non-test) samples would be better. Easier to browse all the samples and easier to upgrade them on FW change.
csviri commentedon Dec 17, 2020
That is true that is easier to upgrade them. So maybe the examples which are kind of showcases we should have in one repo.
Question is what about the memcached, if the intention is to have it as a production ready operator, that should be a separate repo. Not sure if that is the intention.
adam-sandor commentedon Dec 17, 2020
I don't think memcached will be a production ready operator and it should actually go into the repo in the operator-sdk with the other memcached samples.
csviri commentedon Dec 17, 2020
@adam-sandor you mean a separate repo with the samples, or directly to the SDK?
adam-sandor commentedon Dec 17, 2020
here https://github.com/operator-framework/operator-sdk-samples
csviri commentedon Dec 17, 2020
Ok, now I'm completely lost how you mean this, let's discuss it maybe on next weekly.
wtrocki commentedon Dec 17, 2020
Is weekly open for community. Would love to join as listener?
s-soroosh commentedon Dec 17, 2020
+1 for a repo with samples and deprecate the tomcat-operator.
adam-sandor commentedon Dec 17, 2020
@wtrocki please jump in our Discord channel (https://discord.gg/DacEhAy) you can find the invite as a pinned message in the general channel. You are very welcome to join the weeklys.
adam-sandor commentedon Dec 17, 2020
This is my proposal:
.h4 java-operator-sdk/operator-framework/samples
.h4 java-operator-sdk/samples
.h4 operator-framework/operator-sdk-samples
metacosm commentedon Jan 22, 2021
@charlottemach you mentioned you were interested in taking this on during the last weekly call, is it still the case?