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

Requirements for new hypershift deployment v2 #37

Merged
merged 1 commit into from
Feb 26, 2025

Conversation

morenod
Copy link
Collaborator

@morenod morenod commented Feb 24, 2025

Type of change

  • Refactor
  • New feature
  • Bug fix
  • Optimization
  • Documentation Update

Description

  • Disabled OCM token requirement for all the platforms, as hypershift do not need it
  • Refactor on credentials so we can use original azure_credentials and dns_credentials files
  • renamed default aks cluster name

Related Tickets & Documents

  • Related Issue #
  • Closes #

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.

@morenod
Copy link
Collaborator Author

morenod commented Feb 25, 2025

verified execution example:

ES_SERVER=xxxxx ES_INDEX=ripsaw-kube-burner HCP_BURNER_MC_SUBSCRIPTION=xxxxx AKS_PROM=http://xxxxx:9090 AZ_CLIENT_SECRET=xxxxx AZ_CLIENT_ID=xxxxx AZURE_PROM=https://xxxxx HCP_BURNER_AZURE_MC_KUBECONFIG=/home/morenod/.kube/kubeconfig HCP_BURNER_AZURE_MC_RESOURCE_GROUP=xxxxx HCP_BURNER_AZURE_CREDENTIALS_FILE=/home/morenod/final_azure_auth/dns_credentials HCP_BURNER_ES_URL=https://xxxxx:443 HCP_BURNER_ES_INDEX=hypershift-wrapper-timers python hcp-burner.py --config-file ./hcp-burner.ini --platform azure --subplatform hypershiftcli --wildcard-options "--infra-availability-policy HighlyAvailable --control-plane-availability-policy HighlyAvailable --base-domain mrnd.hyp.azure.rhperfscale.org --external-dns-domain mrnd.hyp.azure.rhperfscale.org --generate-ssh --azure-creds /home/morenod/final_azure_auth/dns_credentials --pull-secret /home/morenod/.docker/config.json --release-image quay.io/openshift-release-dev/ocp-release:4.18.1-x86_64 --annotations hypershift.openshift.io/pod-security-admission-label-override=baseline --instance-type Standard_D4s_v3 --fips=true --assign-service-principal-roles --dns-zone-rg-name hcp-underlay-usepmore-mgmt-1 --oidc-issuer-url xxxxx --sa-token-issuer-private-key-path /home/morenod/final_azure_auth/serviceaccount-signer.private --managed-identities-file /home/morenod/final_azure_auth/cloud_provider_hyp.json --data-plane-identities-file /home/morenod/final_azure_auth/data_plane_hyp.json"

@morenod morenod changed the title [WIP] Requirements for new hypershift deployment v2 Requirements for new hypershift deployment v2 Feb 25, 2025
@morenod morenod assigned morenod and mukrishn and unassigned morenod Feb 25, 2025
Copy link
Collaborator

@mukrishn mukrishn left a comment

Choose a reason for hiding this comment

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

lgtm

@mukrishn mukrishn merged commit 50bd804 into cloud-bulldozer:main Feb 26, 2025
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants