Skip to content

Update AWS machineset docs#1172

Merged
openshift-merge-robot merged 1 commit into
openshift:masterfrom
sebsoto:fixAwsDocs
Aug 11, 2022
Merged

Update AWS machineset docs#1172
openshift-merge-robot merged 1 commit into
openshift:masterfrom
sebsoto:fixAwsDocs

Conversation

@sebsoto
Copy link
Copy Markdown
Contributor

@sebsoto sebsoto commented Aug 10, 2022

  • Removes outdated note on usable Windows server versions.
  • fixes formatting for variables
  • renames variable to more fitting name

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 10, 2022
@sebsoto
Copy link
Copy Markdown
Contributor Author

sebsoto commented Aug 10, 2022

/approve cancel

@openshift-ci openshift-ci Bot removed the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 10, 2022
Copy link
Copy Markdown
Contributor

@aravindhp aravindhp left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this, @sebsoto. Lets stick to MachineSet in the PR and commit messages for consistency.

Comment thread docs/machineset-aws.md Outdated
around Windows containers behind a Kubernetes load balancer becoming unreachable
[issue](https://github.com/microsoft/Windows-Containers/issues/78).

_\<windows_server_ami\>_ should be replaced with the AMI ID of a supported version of Windows Server.
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.

* Removes outdated note on usable Windows server versions.
* fixes formatting for variables
* renames variable to more fitting name
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Aug 10, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aravindhp

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:

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

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 10, 2022
@sebsoto sebsoto changed the title [docs] Update AWS machineset docs Update AWS machineset docs Aug 11, 2022
@saifshaikh48
Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Aug 11, 2022
@openshift-merge-robot openshift-merge-robot merged commit 3d54834 into openshift:master Aug 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants