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

installer-aws: create edge compute pool for AWS Local Zones #1232

Merged

Conversation

mtulio
Copy link
Contributor

@mtulio mtulio commented Sep 6, 2022

The enhancement proposes to introduce a new compute pool type edge to create MachineSets on a day zero operations on IPI to deploy compute resources with NoSchedule taints in AWS Local Zones locations.

The R&D is divided into three phases:

  • Phase 0 / SPLAT-635: Research options to create compute nodes on AWS Local Zones and document it as a day-0
  • Phase 1 / SPLAT-636: Create edge compute pool to generate MachineSets for node with NoSchedule taints when installing a cluster in existing VPC with AWS Local Zone subnets.
  • Phase 2 / SPLAT-657: Installer automate the network resources creation on Local Zone based on the edge compute pool.

This EP covers Phases 1 and 2.

This EP is part of the RFE-2782.

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 6, 2022
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 6, 2022

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@mtulio mtulio force-pushed the aws-local-zone-edge-machineset branch from 4d438fa to e46530d Compare September 6, 2022 13:26
@mtulio mtulio changed the title feat/installer-aws: create edge compute pool for AWS Local Zones installer-aws: create edge compute pool for AWS Local Zones Sep 8, 2022
@mtulio mtulio force-pushed the aws-local-zone-edge-machineset branch from e46530d to 64ad757 Compare September 14, 2022 22:05
@mtulio mtulio force-pushed the aws-local-zone-edge-machineset branch from 6a6a3e9 to 84458cd Compare September 16, 2022 01:43
@mtulio mtulio marked this pull request as ready for review September 16, 2022 22:20
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 16, 2022
@mtulio
Copy link
Contributor Author

mtulio commented Sep 16, 2022

Copy link
Member

@fabianofranz fabianofranz left a comment

Choose a reason for hiding this comment

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

This is looking very good, thank you @mtulio! I left a few minor comments but in my opinion this is ready for review. @sdodson and @dhellmann might have opinions and of course our friends from installer @patrickdillon and @makentenza. Tks!

@mtulio mtulio force-pushed the aws-local-zone-edge-machineset branch from 16ff2e6 to fdbdba4 Compare October 7, 2022 15:53
Copy link
Contributor Author

@mtulio mtulio left a comment

Choose a reason for hiding this comment

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

Review comments addressed to the document in my last commit. New topics have been added and/or updated:

@openshift-bot
Copy link

Inactive enhancement proposals go stale after 28d of inactivity.

See https://github.com/openshift/enhancements#life-cycle for details.

Mark the proposal as fresh by commenting /remove-lifecycle stale.
Stale proposals rot after an additional 7d of inactivity and eventually close.
Exclude this proposal from closing by commenting /lifecycle frozen.

If this proposal is safe to close now please do so with /close.

/lifecycle stale

mtulio added a commit to mtulio/installer that referenced this pull request Jun 13, 2023
Inserting the instructions to install a cluster expanding
nodes in Local Zones (new VPC), into existing documentation
of installing in existing VPC.

The Day-2 section is also added for reference of existing Local
Zone automation. The Day 2 is not part of the official documentation
delivered on 4.14, but it is mapped as an open question in the
enhancement proposal [1232](openshift/enhancements#1232).
The steps described on the KCS was validated with QE and SDN teams.
@mtulio
Copy link
Contributor Author

mtulio commented Jun 13, 2023

Presubmits were also created for two phases, awaiting the final credentials for a CI Account:

@JoelSpeed
Copy link
Contributor

I'm happy to proceed, suggest to squash the commits before we merge though (you've tagged my GH in one, should remove that)

The enhancement proposes to introduce a new compute pool type edge to create
MachineSets on a day zero operations in IPI, deploying compute resources with
`NoSchedule` taints in AWS Local Zones locations.

The R&D is divided into three phases:

- Phase 0 / SPLAT-635: Research options to create compute nodes on AWS Local
  Zones and document it as a day-0
- Phase 1 / SPLAT-636: Create edge compute pool to generate MachineSets for
  node with NoSchedule taints when installing a cluster in an existing VPC with
  AWS Local Zone subnets.
- Phase 2 / SPLAT-657: Installer automates the network resources creation on
  Local Zone based on the edge compute pool.

This EP covers Phases 1 and 2, as part of RFE-2782.
@mtulio mtulio force-pushed the aws-local-zone-edge-machineset branch from c8b832e to 15a2593 Compare June 14, 2023 13:19
@mtulio
Copy link
Contributor Author

mtulio commented Jun 14, 2023

I'm happy to proceed, suggest to squash the commits before we merge though (you've tagged my GH in one, should remove that)

For sure, squashed!

@sdodson
Copy link
Member

sdodson commented Jun 15, 2023

/approve
looks fine to me, will check w/ @patrickdillon on monday

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 15, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: sdodson

The full list of commands accepted by this bot can be found here.

The pull request process is described here

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 Jun 15, 2023
@patrickdillon
Copy link
Contributor

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jun 17, 2023
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 17, 2023

@mtulio: all tests passed!

Full PR test history. Your PR dashboard.

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/test-infra repository. I understand the commands that are listed here.

@openshift-merge-robot openshift-merge-robot merged commit 6917ab7 into openshift:master Jun 17, 2023
2 checks passed
@mtulio mtulio deleted the aws-local-zone-edge-machineset branch June 17, 2023 22:24
mtulio added a commit to mtulio/installer that referenced this pull request Jun 23, 2023
Inserting the instructions to install a cluster expanding
nodes in Local Zones (new VPC), into existing documentation
of installing in existing VPC.

The Day-2 section is also added for reference of existing Local
Zone automation. The Day 2 is not part of the official documentation
delivered on 4.14, but it is mapped as an open question in the
enhancement proposal [1232](openshift/enhancements#1232).
The steps described on the KCS was validated with QE and SDN teams.
mtulio added a commit to mtulio/installer that referenced this pull request Jul 5, 2023
Inserting the instructions to install a cluster expanding
nodes in Local Zones (new VPC), into existing documentation
of installing in existing VPC.

The Day-2 section is also added for reference of existing Local
Zone automation. The Day 2 is not part of the official documentation
delivered on 4.14, but it is mapped as an open question in the
enhancement proposal [1232](openshift/enhancements#1232).
The steps described on the KCS was validated with QE and SDN teams.
mtulio added a commit to mtulio/installer that referenced this pull request Jul 5, 2023
Inserting the instructions to install a cluster expanding
nodes in Local Zones (new VPC), into existing documentation
of installing in existing VPC.

The Day-2 section is also added for reference of existing Local
Zone automation. The Day 2 is not part of the official documentation
delivered on 4.14, but it is mapped as an open question in the
enhancement proposal [1232](openshift/enhancements#1232).
The steps described on the KCS was validated with QE and SDN teams.
mtulio added a commit to mtulio/installer that referenced this pull request Jul 5, 2023
Inserting the instructions to install a cluster expanding
nodes in Local Zones (new VPC), into existing documentation
of installing in existing VPC.

The Day-2 section is also added for reference of existing Local
Zone automation. The Day 2 is not part of the official documentation
delivered on 4.14, but it is mapped as an open question in the
enhancement proposal [1232](openshift/enhancements#1232).
The steps described on the KCS was validated with QE and SDN teams.
mtulio added a commit to mtulio/installer that referenced this pull request Jul 6, 2023
Inserting the instructions to install a cluster expanding
nodes in Local Zones (new VPC), into existing documentation
of installing in existing VPC.

The Day-2 section is also added for reference of existing Local
Zone automation. The Day 2 is not part of the official documentation
delivered on 4.14, but it is mapped as an open question in the
enhancement proposal [1232](openshift/enhancements#1232).
The steps described on the KCS was validated with QE and SDN teams.
mtulio added a commit to mtulio/installer that referenced this pull request Jul 6, 2023
Inserting the instructions to install a cluster expanding
nodes in Local Zones (new VPC), into existing documentation
of installing in existing VPC.

The Day-2 section is also added for reference of existing Local
Zone automation. The Day 2 is not part of the official documentation
delivered on 4.14, but it is mapped as an open question in the
enhancement proposal [1232](openshift/enhancements#1232).
The steps described on the KCS was validated with QE and SDN teams.
mtulio added a commit to mtulio/installer that referenced this pull request Jul 6, 2023
Inserting the instructions to install a cluster expanding
nodes in Local Zones (new VPC), into existing documentation
of installing in existing VPC.

The Day-2 section is also added for reference of existing Local
Zone automation. The Day 2 is not part of the official documentation
delivered on 4.14, but it is mapped as an open question in the
enhancement proposal [1232](openshift/enhancements#1232).
The steps described on the KCS was validated with QE and SDN teams.
mtulio added a commit to mtulio/installer that referenced this pull request Jul 6, 2023
Inserting the instructions to install a cluster expanding
nodes in Local Zones (new VPC), into existing documentation
of installing in existing VPC.

The Day-2 section is also added for reference of existing Local
Zone automation. The Day 2 is not part of the official documentation
delivered on 4.14, but it is mapped as an open question in the
enhancement proposal [1232](openshift/enhancements#1232).
The steps described on the KCS was validated with QE and SDN teams.
mtulio added a commit to mtulio/installer that referenced this pull request Jul 17, 2023
Inserting the instructions to install a cluster expanding
nodes in Local Zones (new VPC), into existing documentation
of installing in existing VPC.

The Day-2 section is also added for reference of existing Local
Zone automation. The Day 2 is not part of the official documentation
delivered on 4.14, but it is mapped as an open question in the
enhancement proposal [1232](openshift/enhancements#1232).
The steps described on the KCS was validated with QE and SDN teams.
mtulio added a commit to mtulio/installer that referenced this pull request Jul 18, 2023
Inserting the instructions to install a cluster expanding
nodes in Local Zones (new VPC), into existing documentation
of installing in existing VPC.

The Day-2 section is also added for reference of existing Local
Zone automation. The Day 2 is not part of the official documentation
delivered on 4.14, but it is mapped as an open question in the
enhancement proposal [1232](openshift/enhancements#1232).
The steps described on the KCS was validated with QE and SDN teams.
mtulio added a commit to mtulio/installer that referenced this pull request Jul 18, 2023
Inserting the instructions to install a cluster expanding
nodes in Local Zones (new VPC), into existing documentation
of installing in existing VPC.

The Day-2 section is also added for reference of existing Local
Zone automation. The Day 2 is not part of the official documentation
delivered on 4.14, but it is mapped as an open question in the
enhancement proposal [1232](openshift/enhancements#1232).
The steps described on the KCS was validated with QE and SDN teams.
mtulio added a commit to mtulio/installer that referenced this pull request Jul 19, 2023
Inserting the instructions to install a cluster expanding
nodes in Local Zones (new VPC), into existing documentation
of installing in existing VPC.

The Day-2 section is also added for reference of existing Local
Zone automation. The Day 2 is not part of the official documentation
delivered on 4.14, but it is mapped as an open question in the
enhancement proposal [1232](openshift/enhancements#1232).
The steps described on the KCS was validated with QE and SDN teams.
mtulio added a commit to mtulio/installer that referenced this pull request Jul 19, 2023
Inserting the instructions to install a cluster expanding
nodes in Local Zones (new VPC), into existing documentation
of installing in existing VPC.

The Day-2 section is also added for reference of existing Local
Zone automation. The Day 2 is not part of the official documentation
delivered on 4.14, but it is mapped as an open question in the
enhancement proposal [1232](openshift/enhancements#1232).
The steps described on the KCS was validated with QE and SDN teams.
mtulio added a commit to mtulio/installer that referenced this pull request Jul 20, 2023
Inserting the instructions to install a cluster expanding
nodes in Local Zones (new VPC), into existing documentation
of installing in existing VPC.

The Day-2 section is also added for reference of existing Local
Zone automation. The Day 2 is not part of the official documentation
delivered on 4.14, but it is mapped as an open question in the
enhancement proposal [1232](openshift/enhancements#1232).
The steps described on the KCS was validated with QE and SDN teams.
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.

None yet