Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds new documentation #803

Conversation

prasadtalasila
Copy link
Contributor

Builds on PR #786

  • Updates documentation on packages
  • Corrects admin documentation
  • Edits images on the user website documentation page

|Application ID|CLIENT_ID| _xx_ |
|Application Secret|CLIENT_SECRET| _xx_ |
|Callback URL|(to be directly entered in Gitlab OAuth registration)||
|Forward-auth secret|OAUTH_SECRET|_random-secret-string_ (password for forward-auth, can be changed to your preferred string) |
Copy link

Choose a reason for hiding this comment

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

Line length


- Go to the _docker_ directory

```bash
Copy link

Choose a reason for hiding this comment

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

Code block style


- Remove these three lines from the `conf.server` file

```txt
Copy link

Choose a reason for hiding this comment

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

Code block style


- Run the command for these changes to take effect:

```bash
Copy link

Choose a reason for hiding this comment

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

Code block style


**3. Add username and password:**
2. Add three lines to the `conf.server` file
Copy link

Choose a reason for hiding this comment

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

Ordered list item prefix

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The formatting is off when this page is rendered as html

  Fixes codeclimate issues on existing docs

- Go to the _docker_ directory

```bash
Copy link

Choose a reason for hiding this comment

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

Code block style


- Remove these three lines from the `conf.server` file

```txt
Copy link

Choose a reason for hiding this comment

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

Code block style


- Run the command for these changes to take effect:

```bash
Copy link

Choose a reason for hiding this comment

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

Code block style

--restart always \
mltooling/ml-workspace-minimal:0.13.2
```
```bash
Copy link

Choose a reason for hiding this comment

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

Code block style


The following code adds basic authorization for the new user.
```txt
Copy link

Choose a reason for hiding this comment

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

Code block style

Select option to generate client secret and also selection option
for trusted application.

2. **Determine Website Hostname**
Copy link

Choose a reason for hiding this comment

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

Trailing spaces

It's recommended to use a self-hosted GitLab instance, which you will use in
other parts of the DTaaS application.

3. **Determine Callback and Logout URLs**
Copy link

Choose a reason for hiding this comment

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

Trailing spaces

The logout URL is to help users logout of traefik forward-auth. The logout
URL should not be entered into Gitlab OAuth application setup.

4. **Create OAuth Application**
Copy link

Choose a reason for hiding this comment

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

Trailing spaces


![Creation of Server OAuth Application](server-oauth.png)

5. **Copy Application Credentials**
Copy link

Choose a reason for hiding this comment

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

Trailing spaces

![Server OAuth Application Credentials](server-oauth2.png)

<!-- markdownlint-disable MD013 -->
1. **Checklist: Required Information from OAuth Application:**
Copy link

Choose a reason for hiding this comment

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

Trailing spaces

  Updates libms docs
|GitLab Variable Name|Variable Name in .env of docker compose file |Default Value|
|---|---|---|
<!-- markdownlint-disable MD013 -->
| GitLab Variable Name | Variable Name in .env of docker compose file | Default Value |
Copy link

Choose a reason for hiding this comment

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

Line length

@@ -35,17 +29,21 @@ Here are the steps to get started:
homepage of the DTaaS application.
- The logout URL is where users will be directed after logging out.

**4. OAuth Application Creation:**
**4. OAuth Application Creation:**
Copy link

Choose a reason for hiding this comment

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

Trailing spaces


**4. Add 'route' for new user:**
```bash
Copy link

Choose a reason for hiding this comment

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

Code block style


Please do the following:

**1. Remove an existing user with the [DTaaS CLI](../cli.md)**
Copy link

Choose a reason for hiding this comment

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

Emphasis used instead of a header

Copy link

codeclimate bot commented Jun 5, 2024

Code Climate has analyzed commit a551e6d and detected 2 issues on this pull request.

Here's the issue category breakdown:

Category Count
Style 2

View more on Code Climate.

@prasadtalasila prasadtalasila merged commit 89c9125 into INTO-CPS-Association:feature/distributed-demo Jun 5, 2024
5 checks passed
@prasadtalasila prasadtalasila mentioned this pull request Jun 5, 2024
@prasadtalasila prasadtalasila deleted the pr-786 branch June 19, 2024 06:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants