Skip to content

OSDOCS-17114: microshift networking CQA pt1#107224

Open
jldohmann wants to merge 1 commit intoopenshift:mainfrom
jldohmann:OSDOCS-17114
Open

OSDOCS-17114: microshift networking CQA pt1#107224
jldohmann wants to merge 1 commit intoopenshift:mainfrom
jldohmann:OSDOCS-17114

Conversation

@jldohmann
Copy link
Copy Markdown
Contributor

@jldohmann jldohmann commented Feb 24, 2026

@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Feb 24, 2026

@jldohmann: This pull request references OSDOCS-17114 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.22.0" version, but no target version was set.

Details

In response to this:

Version(s): 4.16+

Issue: https://issues.redhat.com/browse/OSDOCS-17114

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci Bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 24, 2026
Comment thread modules/microshift-default-settings.adoc Outdated
@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Feb 24, 2026

@jldohmann: This pull request references OSDOCS-17114 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.22.0" version, but no target version was set.

Details

In response to this:

Version(s): 4.16+

Issue: https://issues.redhat.com/browse/OSDOCS-17114

Link to docs preview:

Additional information: MS networking CQA pt.1

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@ocpdocs-previewbot
Copy link
Copy Markdown

ocpdocs-previewbot commented Feb 24, 2026

@openshift-ci-robot
Copy link
Copy Markdown

openshift-ci-robot commented Feb 24, 2026

@jldohmann: This pull request references OSDOCS-17114 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.22.0" version, but no target version was set.

Details

In response to this:

Version(s): 4.16+

Issue: https://issues.redhat.com/browse/OSDOCS-17114

Link to docs preview:

Additional information: MS networking CQA pt.1

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 1, 2026
@jldohmann jldohmann force-pushed the OSDOCS-17114 branch 2 times, most recently from 6d6036a to 2cac19a Compare March 9, 2026 23:29
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Mar 9, 2026
@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Mar 24, 2026

@jldohmann: all tests passed!

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

include::snippets/technology-preview.adoc[]

[role="_abstract"]
To properly plan your single-node edge deployment, review the comprehensive default settings in {product-title}. Understanding these built-in parameters, ranging from OVN-Kubernetes Container Network Interface (CNI) settings to API and storage configurations, helps you determine if you need to customize your overall configuration to match your existing infrastructure.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
To properly plan your single-node edge deployment, review the comprehensive default settings in {product-title}. Understanding these built-in parameters, ranging from OVN-Kubernetes Container Network Interface (CNI) settings to API and storage configurations, helps you determine if you need to customize your overall configuration to match your existing infrastructure.
To properly plan your single-node edge deployment, review the comprehensive default settings in {microshift-short}. Understanding these built-in parameters, ranging from OVN-Kubernetes Container Network Interface (CNI) settings to API and storage configurations, helps you determine if you need to customize your overall configuration to match your existing infrastructure.

the product title attribute is only used in the release notes and general intro

this short desc is actually conceptually incorrect; microshift does not need to match infrastructure as this is not the way it is deployed (really cannot seem to say this enough); this sounds like single-openshift, which it is not

Suggested change
To properly plan your single-node edge deployment, review the comprehensive default settings in {product-title}. Understanding these built-in parameters, ranging from OVN-Kubernetes Container Network Interface (CNI) settings to API and storage configurations, helps you determine if you need to customize your overall configuration to match your existing infrastructure.
To plan your edge deployment, you can review the comprehensive default settings in {microshift-short}. Understanding default parameters helps you determine how you need to customize your configuration to match your edge destination.

the edge deployment is going to be a single device (ok, many times over) that runs on an image-based rhel system with microshift as the k app platform; config generally has to do with storage and security versus performance and that is based on both location and the device itself

`storage.driver`:: Specifies the storage driver to use. Default null value deploys Logical Volume Managed Storage (LVMS).
`storage.optionalCsiComponents`:: Specifies the CSI components to deploy. Default null value deploys `snapshot-controller`.
-- No newline at end of file
`storage.driver`:: Specifies the default storage to use. The default null value deploys Logical Volume Managed Storage (LVMS).
Copy link
Copy Markdown
Contributor

@ShaunaDiaz ShaunaDiaz Mar 26, 2026

Choose a reason for hiding this comment

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

Suggested change
`storage.driver`:: Specifies the default storage to use. The default null value deploys Logical Volume Managed Storage (LVMS).
`storage.driver`:: Specifies the storage driver to use. The default null value deploys Logical Volume Managed Storage (LVMS).

other types of drivers were intended to be used here as well; i would leave this--i don't know what the support status is at this point; you can ask @copejon


.{product-title} network topology
The following diagram shows an overview of the {product-title} network topology:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

the title should still be included, not deleted (did we hit a DITA rule on that one?)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yes, this kept flagging one of the AsciiDocDITA rules. if that shouldn't happen, the vale rule in the package will need to be updated to handle these cases

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants