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

Align other site definitions to use plans (and fix the failing docker gate) #564

Closed
mattmceuen opened this issue Jun 4, 2021 · 5 comments
Assignees
Labels
2-Manifests Relates to manifest/document set related issues bug Something isn't working priority/medium Default priority for items size m
Milestone

Comments

@mattmceuen
Copy link
Contributor

Issue #517 adjusted the test-site site definition to run deployments via phase plans, and deleted the per-phase scripts accordingly.

However, the other site definitions still need to be updated accordingly. One practical issue demonstrated by this is that the (non-voting) docker-test-site gate is failing, because its zuul config points at the deleted scripts.

@mattmceuen mattmceuen added bug Something isn't working triage Needs evaluation by project members labels Jun 4, 2021
@ratnopamc
Copy link
Contributor

Please assign to me.

@ratnopamc
Copy link
Contributor

With the deletion of per-phase scripts, the deployment of each provider test sites needs to be adjusted/validated. I think it would be better to split this issue into multiple issues for tracking.

This issue #564 can be tracked for fixing CAPD deployment/zuul failures.
Similarly, 3 separate issues can be created as below:

  • Fix/test CAPG test-site to switch to phasePlan
  • Fix/test AZ test-site to switch to phasePlan
  • Fix/test CAPO test-site to switch to phasePlan

Thanks.

@jezogwza jezogwza added 2-Manifests Relates to manifest/document set related issues priority/medium Default priority for items and removed triage Needs evaluation by project members labels Jun 9, 2021
@jezogwza jezogwza added this to the v2.2 milestone Jun 9, 2021
@jezogwza jezogwza modified the milestones: v2.2, v2.1 Jun 9, 2021
@lb4368
Copy link

lb4368 commented Jun 9, 2021

@eratnch Below are hackmd notes from today's Flight Plan call related to this issue. From this design the first step seems to be to build the base phase plans.yaml for all providers into the gating type that the test-site can eventually patch. Based upon this design would you still think we need separate issues per provider?

Normalize the phase plans across the different airshipctl sites, by setting up the same site-type inheritance for the phases entrypoint as already exists for test-site. The default phase plan in the gating type should be suitable for the general case – i.e. everything besides bare metal deployments (which have their own phases). At the e.g. docker site level, it can just consume those phase plans & use them; at the test-site level it’ll need to patch the plan to have all the bare metal phases as it has today.

@lb4368 lb4368 added the size m label Jun 16, 2021
airshipbot pushed a commit that referenced this issue Sep 16, 2021
This PS introduces phase plan for CAPD provider
 * Adds phase to merge kubeconfig
 * Patches type/gating plan to use capd phase plan
 * Patches cluster_map and executor to work with "default" namespace
 * Fixes CAPD zuul failures

Relates-To: #564
Relates-To: #580
Relates-To: #587
Change-Id: I5007970c907bc87dccf6dd9fcb052afc1b5c13f7
@ratnopamc
Copy link
Contributor

This issue can be closed. Fix for this issue is merged.

@lb4368
Copy link

lb4368 commented Oct 13, 2021

@mattmceuen Are you OK with closing this?

@lb4368 lb4368 modified the milestones: v2.1, Future Oct 27, 2021
@lb4368 lb4368 modified the milestones: Future, v2.1 Nov 4, 2021
@lb4368 lb4368 closed this as completed Nov 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2-Manifests Relates to manifest/document set related issues bug Something isn't working priority/medium Default priority for items size m
Projects
None yet
Development

No branches or pull requests

4 participants