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

OCP/OKD cluster installation on OCI [UPI/agnostic/None/External] #26

Draft
wants to merge 39 commits into
base: main
Choose a base branch
from

Conversation

mtulio
Copy link
Owner

@mtulio mtulio commented Feb 14, 2023

Install OCP/OKD cluster in Oracle Cloud Infrastructure (OCI) using UPI

  • UPI with Platform type: None
    • Documentation
  • UPI with Platform type: External with CCM
    • Documentation
  • UPI with Platform type: External with CCM and CSI
    • Documentation
  • UPI with Platform type: None with CSI
    • Documentation

Tested Dist/OS:

  • OCP/RHCOS
  • OKD/SCOS
  • OKD/FCOS

UPI Stacks check list:

Internal stacks:

  • Config
  • Config: Manifest hook for CCM and CSI
  • Bootstrap
  • Image Mirroring
  • Destroy

Documentation:

  • Documentation: OCI Deployment using UPI and platform agnostic
  • Documentation: Steps to integrate the provider using OCI as reference: Create a diagram of the proposal.

Pre-merge:

Dependencies merged

@mtulio mtulio force-pushed the feat-added-provider-oci branch from 907af6c to b878cf5 Compare February 17, 2023 04:19
@mtulio mtulio force-pushed the feat-added-provider-oci branch 2 times, most recently from 0be3133 to eaa6196 Compare March 8, 2023 14:06
@mtulio mtulio changed the title WIP/Draft to provision stacks for OCP/OKD cluster on OCI WIP|Provision OCP/OKD cluster on OCI [UPI/agnostic/None] Mar 9, 2023
@mtulio mtulio force-pushed the feat-added-provider-oci branch from eaa6196 to f3763c0 Compare March 9, 2023 18:19
@mtulio mtulio changed the title WIP|Provision OCP/OKD cluster on OCI [UPI/agnostic/None] Provision OCP/OKD cluster on OCI [UPI/agnostic/None] Mar 9, 2023
@mtulio mtulio force-pushed the feat-added-provider-oci branch 2 times, most recently from b39c6d7 to e3d7754 Compare March 17, 2023 03:11
@mtulio mtulio changed the title Provision OCP/OKD cluster on OCI [UPI/agnostic/None] Provision OCP/OKD cluster on OCI [UPI/agnostic/None/External] Mar 21, 2023
@@ -0,0 +1,26 @@
---
# placeholder
Copy link
Owner Author

Choose a reason for hiding this comment

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

Suggested change
# placeholder

@@ -0,0 +1,26 @@
---
# placeholder
okd_cluster_destroy_instances_compartment_id: "{{ oci_compartment_id }}"
Copy link
Owner Author

Choose a reason for hiding this comment

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

The destroy flow must lookup by tags, at the same time the tags must be set for each billable resource.

Comment on lines +2 to +4
# AWS Networks
# AWS us-east-1: 10.0.0.0/16 (to 10.0.255.255/16)
# AWS <unassigned>: 10.23.0.0/16 (to 10.23.255.255/19)
Copy link
Owner Author

Choose a reason for hiding this comment

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

Suggested change
# AWS Networks
# AWS us-east-1: 10.0.0.0/16 (to 10.0.255.255/16)
# AWS <unassigned>: 10.23.0.0/16 (to 10.23.255.255/19)
# OCI Networks
# OCI us-sanjose-1: 10.0.0.0/16 (to 10.0.255.255/16)

is_enabled: yes
display_name: "{{ cluster_state.infra_id }}-igw"

nat_gateway:
Copy link
Owner Author

Choose a reason for hiding this comment

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

AWS supports many NGWs by VPC, the model of OCI subnet works (regional), multiple gateways by VCN seems to be useless.

# is_stateless: false
# source_type: CIDR_BLOCK
# description: Allow all inbound
egress_security_rules:
Copy link
Owner Author

Choose a reason for hiding this comment

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

To be reviewed. Maybe it's interesting to delegate permissions to NSG and make sure everything will works correctly. NSG is best practice.

volumeBindingMode: WaitForFirstConsumer
allowVolumeExpansion: true
reclaimPolicy: Delete
allowedTopologies:
Copy link
Owner Author

Choose a reason for hiding this comment

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

Can be removed?

Comment on lines 33 to 36
compartment_id: "{{ okd_cluster_destroy_instances_compartment_id }}"
id: "{{ inst.instances[0].id }}"
preserve_boot_volume: false
wait: no
Copy link
Owner Author

Choose a reason for hiding this comment

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

Review

Comment on lines +10 to +11

# - debug: var=_lb_out
Copy link
Owner Author

Choose a reason for hiding this comment

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

Suggested change
# - debug: var=_lb_out

Comment on lines +36 to +37

# - debug: var=_lb_del_out
Copy link
Owner Author

Choose a reason for hiding this comment

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

Suggested change
# - debug: var=_lb_del_out

ansible.builtin.include_tasks: "./{{ os_mirror_from }}.yaml"

- name: upload to provider {{ os_mirror_to_provider }}
ansible.builtin.include_tasks: "./{{ os_mirror_to_provider }}.yaml"
Copy link
Owner Author

Choose a reason for hiding this comment

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

Add statements when it's not defined

@vercel
Copy link

vercel bot commented Mar 28, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ansible-collection-okd-installer ❌ Failed (Inspect) May 23, 2023 4:51am

@vercel vercel bot temporarily deployed to Preview March 28, 2023 04:15 Inactive
@mtulio mtulio force-pushed the feat-added-provider-oci branch from 1d7f705 to b05d92c Compare March 28, 2023 04:18
@vercel vercel bot temporarily deployed to Preview March 28, 2023 04:18 Inactive
@mtulio mtulio force-pushed the feat-added-provider-oci branch from b05d92c to ae71972 Compare April 7, 2023 21:43
@vercel vercel bot temporarily deployed to Preview April 7, 2023 21:43 Inactive
---
# placeholder
okd_cluster_destroy_instances_compartment_id: "{{ oci_compartment_id }}"
okd_cluster_destroy_instances:
Copy link
Owner Author

Choose a reason for hiding this comment

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

TODO discovery by tag

@@ -1,57 +1,66 @@
---
# NOTE: we must deprecate this document in favor of 'modes':
Copy link
Owner Author

Choose a reason for hiding this comment

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

mode, action, or task?

@vercel vercel bot temporarily deployed to Preview April 17, 2023 18:22 Inactive
@vercel vercel bot temporarily deployed to Preview May 22, 2023 21:37 Inactive
mtulio added a commit that referenced this pull request May 23, 2023
* review and split non-provider intergration changes

* update cloud_loadbalancer module

* keep using amazon.s3_obj

* testing SCOS and 5.5 AWS collection

* Update mock-aws.yaml

* rename okd-scos file to 4.13

* set correct values for release version
@mtulio mtulio force-pushed the feat-added-provider-oci branch from 73222e3 to a33dc5c Compare May 23, 2023 04:51
@vercel vercel bot temporarily deployed to Preview May 23, 2023 04:51 Inactive
@mtulio mtulio force-pushed the feat-added-provider-oci branch from a33dc5c to c7cd6c7 Compare May 23, 2023 04:55
@mtulio mtulio changed the title UPI OCP/OKD cluster on OCI [UPI/agnostic/None/External] OCP/OKD cluster installation on OCI [UPI/agnostic/None/External] Sep 18, 2023
@mtulio mtulio force-pushed the feat-added-provider-oci branch from 6d8c496 to 459b509 Compare October 18, 2023 21:54
@mtulio mtulio force-pushed the feat-added-provider-oci branch from e0ece31 to 47f3db4 Compare October 23, 2023 17:08
@github-actions
Copy link

Artifacts built by CI:

@github-actions
Copy link

Artifacts built by CI:

@github-actions
Copy link

Artifacts built by CI:

@github-actions
Copy link

Artifacts built by CI:

@github-actions
Copy link

Artifacts built by CI:

@github-actions
Copy link

Artifacts built by CI:

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.

1 participant