Skip to content
This repository was archived by the owner on Nov 27, 2024. It is now read-only.

book: fix typos#229

Merged
openshift-merge-bot[bot] merged 2 commits intokonflux-workspaces:mainfrom
filariow:doc-fix
Jul 9, 2024
Merged

book: fix typos#229
openshift-merge-bot[bot] merged 2 commits intokonflux-workspaces:mainfrom
filariow:doc-fix

Conversation

@filariow
Copy link
Member

@filariow filariow commented Jul 2, 2024

Signed-off-by: Francesco Ilario filario@redhat.com

@filariow filariow requested a review from dperaza4dustbit July 2, 2024 09:16
@openshift-ci openshift-ci bot requested a review from sadlerap July 2, 2024 09:16
@openshift-ci openshift-ci bot added the approved label Jul 2, 2024
Copy link
Contributor

@dperaza4dustbit dperaza4dustbit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Few Suggestions

@@ -5,19 +5,19 @@ In this section are detailed the main workflow implemented by this operator.
## Home Workspace

When an [KubeSaw](https://github.com/codeready-toolchain) UserSignup is approved, a Space is created by default.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When an KubeSaw UserSignup --> When a KubeSaw UserSignup


When an [KubeSaw](https://github.com/codeready-toolchain) UserSignup is approved, a Space is created by default.
This workflow monitors ensures an InternalWorkspace exists for the Space.
The operator ensures an InternalWorkspace exists for the user's default Space.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Operator" is more generic term, "controller" might be better here

A `community` InternalWorkspace is visible by every authenticated users.

If an InternalWorkspace visibility is set to `community`, the operator makes sure that a SpaceBinding exists for the special-user `kubesaw-authenticated`, the space related to the InternalWorkspace, and the role `viewer`.
If the visibility is set to `private`, the SpaceBinding is ensured not to exist.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the SpaceBinding is removed

@filariow filariow requested a review from dperaza4dustbit July 4, 2024 09:31
filariow added 2 commits July 4, 2024 11:31
Signed-off-by: Francesco Ilario <filario@redhat.com>
Signed-off-by: Francesco Ilario <filario@redhat.com>
Copy link
Contributor

@dperaza4dustbit dperaza4dustbit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci
Copy link

openshift-ci bot commented Jul 9, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dperaza4dustbit, filariow

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [dperaza4dustbit,filariow]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot bot merged commit 275d5e6 into konflux-workspaces:main Jul 9, 2024
@filariow filariow deleted the doc-fix branch July 9, 2024 14:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants