Skip to content

Conversation

@P-rple
Copy link
Contributor

@P-rple P-rple commented Oct 8, 2024

Overview of Changes

General overview of what has been changed

Modified IPTables documentation to removed errors, typos and add missing content. The added content is as follows: Docker Integration, Debugging Guidelines, and Useful commands.

Reason for Modification

This section will list the reasons modification were preformed. This justifies why the changes are necessary or the documentation should be added.

Verification

  • The documentation is completed, or does not contain work-in-progress/partially-completed sections.
  • The documentation is written in Markdown (Exception for Inject and Incident Reports).
  • Any Images are contained in a subdirectory Images.
  • Any scripts, Ansible Playbooks, Terraform scripts, etc. Have documentation explaining their purpose and use case.
  • Any scripts are tested.
  • I have assigned and notified a reviewer.

P-rple added 3 commits July 30, 2024 21:00
… added some of the stuff I was able to finalize from what I(Chisom) was working on from request b/c the CCDC crew wants to see what team members were working on, according to the discord message(s).
@P-rple P-rple requested a review from a team as a code owner October 8, 2024 15:25
@P-rple
Copy link
Contributor Author

P-rple commented Oct 8, 2024

Did not realize I never requested a pull request from the summer. This info was located in the k8branch, which was still publicly available, since the summer.

- Cluster is currenlty down....
> setup VMs on personal machine

- Migratation sucks... No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

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

Can be removed

Suggested change
- Migratation sucks...

# Updating the Contents of an Nginx Webpage in Kubernetes

## TOC
- [Updating the Contents of an Nginx Webpage in Kubernetes](#updating-the-contents-of-an-nginx-webpage-in-kubernetes)
Copy link
Contributor

Choose a reason for hiding this comment

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

The TOC can be updated to remove the document title and TOC entry.

<!- omit-from-toc --> is something that can be used if you generate this with a markdown tool. (Look up your specific tool)

- [Notes](#notes)


## Step 1: Create a ConfigMap with the New HTML Content
Copy link
Contributor

Choose a reason for hiding this comment

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

What is a Config Map and why are we writing HTML?


# Deploying an Nginx Website on Kubernetes

## TOC
Copy link
Contributor

Choose a reason for hiding this comment

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

Same TOC recommendations

```

## Step 2: Expose Nginx via a Service
To make the Nginx deployment accessible within the Kubernetes cluster, create a Service.
Copy link
Contributor

Choose a reason for hiding this comment

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

What is a Service

@DaintyJet
Copy link
Contributor

May be good to migrate this to the K8s Folder too. I do not think there was sufficient progress on the Projects to justify their own folder.

We can turn these documents into more of a quick reference, as Alex's documents are good (but long).

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.

3 participants