search

ploigos / ploigos-software-factory-operator

33 stars 22 forks source link

Change the names of CRs used by the ploigos-software-factory-operator #107

Open dwinchell opened 3 years ago

dwinchell commented 3 years ago

TsscPlatform -> PloigosPlatform TsscPipeline -> PloigosPipeline

dwinchell commented 3 years ago

@BillBensing you good with this?

andykrohg commented 3 years ago

I believe the latest I heard on this, we're looking for TsscPlatform -> Platform and TsscPipeline -> Pipeline. Can you confirm @BillBensing

adamgoossens commented 3 years ago

Tossing in my unsolicited 2c, what about TsscPlatform -> SoftwareFactory and TsscPipeline -> SoftwareFactoryPipeline (or SoftwarePipeline) ? They make the CRs more expressive about the specific concept that they represent and that the Operator will create. Plus it aligns with the PSFO name :)

Pipeline conflicts with Tekton from a naming perspective, even though the API group is different. Users would have both Tekton Pipeline and Ploigos Pipeline in the same namespace if Tekton is their CI tool, adding a potential source of confusion for them.

adamgoossens commented 3 years ago

If there's an appetite for it, I have a local commit and branch (not yet pushed) that renames everything to SoftwareFactory and SoftwareFactoryPipeline. Local molecule tests pass. If we prefer Platform and Pipeline the change is cosmetic and quick; let me know and I'll push it up.