Skip to content

Improve usage doc, fix typos and follow style guide #89

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

Merged
merged 3 commits into from
Jul 23, 2025

Conversation

Rohit-0505
Copy link
Contributor

@Rohit-0505 Rohit-0505 commented Jul 8, 2025

Proposed Changes

  • Improve the usage docs and remove the reconciliation process
  • Added the missing top-level introduction in each of the usage-docs
  • fix typos, remove extra spaces
  • validated files with the standard style guide

@Rohit-0505 Rohit-0505 requested a review from a team as a code owner July 8, 2025 10:36
@github-actions github-actions bot added size/L bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request labels Jul 8, 2025
@Rohit-0505 Rohit-0505 marked this pull request as draft July 8, 2025 10:44
@Rohit-0505 Rohit-0505 self-assigned this Jul 8, 2025
@hardikdr hardikdr added this to Roadmap Jul 9, 2025
@Rohit-0505 Rohit-0505 marked this pull request as ready for review July 9, 2025 07:29
@Rohit-0505 Rohit-0505 requested a review from afritzler July 9, 2025 07:30
@@ -1,5 +1,7 @@
# Networking Resources

This guide describes the core networking resources in IronCore, such as `Networks`, `NetworkInterfaces`, `VirtualIPs`, `NAT gateways`, `LoadBalancers`, and `NetworkPolicies`. It explains how to define, configure, and manage these resources for the infrastructure.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
This guide describes the core networking resources in IronCore, such as `Networks`, `NetworkInterfaces`, `VirtualIPs`, `NAT gateways`, `LoadBalancers`, and `NetworkPolicies`. It explains how to define, configure, and manage these resources for the infrastructure.
This guide describes the core networking resources in IronCore, such as `Networks`, `NetworkInterfaces`, `VirtualIPs`, `NAT gateways`, `LoadBalancers`, and `NetworkPolicies`.


### Reconciliation Process:
- `peerings` (`list`): Are the list of network peerings with this `Network` (optional).

Copy link
Member

Choose a reason for hiding this comment

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

Please remove extra newline.

Once created and with an allocated ID, the IronCore `Network` will be patched with the corresponding provider ID of the apinet Network and set to state: `Available`.
Once the `Network` resource is in an available state. The format of a network provider ID is as follows:
`ironcore-net://<namespace>/<name>/<id>/<uid>`
- **Network creation**: [ironcore-net](https://github.com/ironcore-dev/ironcore-net) is the network plugin for IronCore that realizes the `Network` resource, When an IronCore `Network` is created, a corresponding `core.apinet.ironcore.dev/Network` is created in the ironcore-net(apinet) cluster.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- **Network creation**: [ironcore-net](https://github.com/ironcore-dev/ironcore-net) is the network plugin for IronCore that realizes the `Network` resource, When an IronCore `Network` is created, a corresponding `core.apinet.ironcore.dev/Network` is created in the ironcore-net(apinet) cluster.
- **Network creation**: [ironcore-net](https://github.com/ironcore-dev/ironcore-net) is the network plugin for IronCore that realizes the `Network` resource. When an IronCore `Network` is created, a corresponding `core.apinet.ironcore.dev/Network` is created in the ironcore-net(apinet) cluster.

- name: peering1
state: Ready
state: Available

Copy link
Member

Choose a reason for hiding this comment

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

Please remove extra newline.

@afritzler afritzler merged commit 58d77b1 into ironcore-dev:main Jul 23, 2025
7 checks passed
@github-project-automation github-project-automation bot moved this to Done in Roadmap Jul 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request size/L
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants