kabanero-io / docs

Kabanero documentation. This repo will be archived soon.
https://kabanero.io/docs/
Apache License 2.0
7 stars 30 forks source link

Todd: doc to specify Kab CLI service metadata in kabanero operator yaml #66

Closed davco01a closed 4 years ago

davco01a commented 5 years ago

See: https://github.com/kabanero-io/kabanero-operator/wiki/Todd's-input-for-the-Kabanero-operator

bschrammIBM commented 5 years ago

@davco01a Is there an update to the docs required for this? The referring issues says : This needs to be reevaluated after discussion with the originating requestor and is in Closed status. Can this issue also be closed?

davco01a commented 5 years ago

@bschrammIBM Yes, we will need doc, I am in the process of writing up doc at this wiki right now:

https://github.com/kabanero-io/kabanero-operator/wiki/Todd's-input-for-the-Kabanero-operator

bschrammIBM commented 5 years ago

Should this be in a readme? Is it part of installation and configuration?

davco01a commented 5 years ago

@bschrammIBM recommends we add a step to:

https://kabanero.io/docs/#scripted-kabanero-foundation-setup.html

to configure the Kabanero document

davco01a commented 5 years ago
kaczyns commented 5 years ago

@davco01a You modify the CR instance of the Kabanero CRD. So if the user created one Kabanero instance then that's the one that they'd modified. Today, we only support one instance, in the Kabanero namespace, so that's the one they'd modify. And most of the fields you're referring to there are not merged yet (see PR https://github.com/kabanero-io/kabanero-operator/pull/170)

davco01a commented 5 years ago

Ok, I want to to net out which fields Todd needs to focus on for modification and document those.

bschrammIBM commented 5 years ago

@davco01a @kaczyns Can you provide a better title for this? Is it 'Configuring OpenShift to recognize the Kabanero namespace'

davco01a commented 5 years ago

@kaczyns Do you have any recommendation of what we should call deonfiguring the Kabanero CRD?

Perhaps just: Configuring the Kabanero Custom Resource Definition

kaczyns commented 5 years ago

@davco01a @bschrammIBM I'm sorry, I'm a little lost.

The install script that was mentioned earlier in https://kabanero.io/docs/#scripted-kabanero-foundation-setup.html: openshift_master_default_subdomain=my.openshift.master.default.subdomain ./install-kabanero-foundation.sh

it already installs an instance of the Kabanero object. There are no modifications that anyone would need to make at this point.

It's not until Champ clones the collections, and makes changes, that Todd would need to either go back to the Kabanero document that was created in the install script, and change it, or just delete it and make a new one.

So, I don't think the install script page is where we should be talking about modifying the Kabanero document.

When we get there, I think it should be called "Configuring the Kabanero CR instance to point at the curated collections" or something like that. And it should describe where the collection URL needs to be set, where the Github organization/team/apiUrl needs to be set.

davco01a commented 5 years ago

@kaczyns +1

jantley-ibm commented 5 years ago

@davco01a @kaczyns @bschrammIBM Looks like the best place for this is https://github.com/kabanero-io/docs/blob/master/ref/general/collection-building.adoc -- could you recommend where the new content would go in the workflow on that page?

davco01a commented 4 years ago

I thought @kaczyns gave recommendations on reordering the content of this page and placing the Kab document updates accordingly, but I don't see those changes reflected yet

kaczyns commented 4 years ago

We had been working thru this page: https://github.com/kabanero-io/docs/blob/master/ref/general/collection-building.adoc

One way we could slice this... is to insert steps between step 1 and 2. Basically: 1a) Create the initial tagged release of the cloned collections (v1) 1b) Create any org-level teams that should administer the cloned collection and add members to those teams. 1c) Alter the Kabanero document to point to that named release (v1) and add the Github: section pointing to the github org, team(s), and api url

then continue on.... but when we get to step 7 we have to figure out what to do because you can't test the changes until you've pushed them back to your remote (cloned) github repository

I think move 7 to after 9. So 7 would be push back, 8 would be make a release (not final version), and 9 would be test by altering the Kabanero document to point to the release. if errors are found iterate back and update the release. Then 10 would be to declare the release final. I think that just editing step 1 addresses this issue.

jantley-ibm commented 4 years ago

There is a pull request with the discussed changes waiting for review: https://github.com/kabanero-io/docs/pull/91

davco01a commented 4 years ago

@jantley-ibm added some comments to your PR

marikaj123 commented 4 years ago

@jantley-ibm - Does the issue contain what is needed for the ID team to provide the 'ID-reviewed' label?

davco01a commented 4 years ago

At this point in time we are really working off of:

https://github.com/kabanero-io/docs/blob/master/ref/general/collection-building.adoc

davco01a commented 4 years ago

I also have this issue open to retitle this document:

https://github.com/kabanero-io/docs/issues/119

davco01a commented 4 years ago

Additionally:

Bullet one of Step 3 needs some rework -

Specify the repository URL in the Kabanero document (object) to point to a cloned, customized index.yaml for the master collection. The URL setting is in the Spec.Collections.Repositories section of the Kabanero Custom Resource (CR) instance. You can either alter the Kabanero CR instance that already exists, or delete it and create a new one.

master collection should be curated collection

davco01a commented 4 years ago

Also some mention of step 3 about how to edit the Kabanero document would be help, e.g. oc edit or how to navigate to the CR from within the OKD console

@mtamboli ^^

marikaj123 commented 4 years ago

@mtamboli - From SVT perspective, do you have what you need? @davco01a and @kaczyns - Since ID work @jantley-ibm is complete, is the issue ready to close?

kaczyns commented 4 years ago

@davco01a I think your comments are addressed in PR #127 . Could you please review that?

SueChaplain commented 4 years ago

I'm currently writing a guide on "Working with Collections" (Champ - https://github.com/kabanero-io/docs/issues/55), which covers cloning the public collections to a local Git org/repo, understanding the content/structure of a collection, customising a stack in a collection, building a collection, testing the stack he customised, and pushing the updated collections back to GitHub. Draft here: https://github.com/kabanero-io/draft-guide-working-with-collections

The part "build a collection" is largely duplicated by the content in this reference topic, except for the custom resource definitions needed for the Operator. I think that part of the process is likely to be something Todd would do?

So I would say that adding the CRD info does not fit my guide, but the full content of what you have here mixes up a process Champ might do with one that Todd might do. Is the reference topic temporary? Is there another guide planned for this task information? Or are we leaving task information in the Docs? @bschrammIBM @jantley-ibm

davco01a commented 4 years ago

Once https://github.com/kabanero-io/docs/pull/127 is merged, then we can close this

kaczyns commented 4 years ago

127 is merged, I am closing this issue. #134 was opened to make an edit to the new page.

davco01a commented 4 years ago

closing bc 127 is merged now