Placement-operator integration#1094
Open
auniyal61 wants to merge 666 commits intoopenstack-k8s-operators:mainfrom
Open
Placement-operator integration#1094auniyal61 wants to merge 666 commits intoopenstack-k8s-operators:mainfrom
auniyal61 wants to merge 666 commits intoopenstack-k8s-operators:mainfrom
Conversation
Resources created by the placement-operator are named based on the name of the PlacementAPI CR
Make sure that DeploymentReady condition is set back to False if the Deployment ReadyReplicas falls back to 0
Public/Internal service cert secrets and the CA bundle secret can be passed to configure httpd virtual hosts for tls termination. The certs are mounted to in var/lib/config-data/tls/certs/%s.crt|key and a CA bundle to /etc/pki/ca-trust/extracted/pem/tls-ca-bundle.pem. Server cert and key are intended to be moved by kolla to /etc/pki/tls/certs|private. Depends-On: openstack-k8s-operators/lib-common#428 Signed-off-by: Veronika Fisarova <vfisarov@redhat.com>
Our intention is to track service operator and lib-common dependencies via pseudoversions. However renovate does not automatically bump from a tagged version (e.g. v0.3.0) to a newer but not tagged pseudoversion. So this patch does the manual bump. After this renovate will bump the newer pseduoversions.
These fields were redundant. Crashing pods can be debugged via oc debug. Implements: OSPRH-4290
Adapt to the new version by: * adding RBAC rules for MariaDBAccount manipulation * adding Own(MariaDBAccount) to detect status changes * separating PlacementAPI CR name from the schema and MariaDBDatabase/Account name * adapting test to simulate MariaDBAccount success * adapting to new DB hostnames due to the mariadb TLS feature
This change modifies how the operator is built to use the ubi9 base image and golang tool chain. A new GO_BUILD_EXTRA_ENV_ARGS build arg is added to the dockerfile to enable the build system to inject addtional parmaters. the default value enable dynmaic linking of glibc which is required for FIPS The prow_ci.env file is extended to enable fips checkign in the prow image build job to validate this change. The Makefile is extened to allow passing build args to docker-build
We only want to support the policy.yaml file passthrough for PlacementAPI The kolla config files are modified to move the supported files in place. Implements: OSPRH-2503
Signed-off-by: Brendan Shephard <bshephar@redhat.com>
This change updates the webhook functions to comply with the new standard for controller-runtime. Signed-off-by: Brendan Shephard <bshephar@redhat.com>
This change updates the manager intialisation to comply with the new controller-runtime schema. Signed-off-by: Brendan Shephard <bshephar@redhat.com>
This change fixes the Watches declarations to comply with the new controller-runtime standards Signed-off-by: Brendan Shephard <bshephar@redhat.com>
Signed-off-by: Brendan Shephard <bshephar@redhat.com>
…er-runtime Controller runtime update
…rators/revert-144-controller-runtime Revert "Controller runtime update"
This change updates the webhook functions to comply with the new standard for controller-runtime. Signed-off-by: Brendan Shephard <bshephar@redhat.com>
Import complete placement-operator codebase into placement-import directory using subtree merge to preserve full git commit history. This merge copy placement-operator git history to nova-operator enabling git blame and git log to show original authors and commit history of placement-operator
Move placement business logic from placement-import/internal/placement to internal/placement/ to keep alongside nova business logic
Move webhook from placement-import/internal/webhook/v1beta1/ to internal/webhook/placement/v1beta1/
Move placement templates from placement-import/templates/placementapi/ to templates/placement/api/
Move placement functional tests to test/functional/placement/ Move placement kuttl tests from to test/kuttl/test-suites/placement/ Move placement kuttl config kuttl-test.yaml to test/kuttl/test-suites/placement/config.yaml Fix symlinks to point to correct relative paths for the new location.
Copy kuttl-test.yaml as test/kuttl/test-suites/placement/config.yaml to match nova test suit pattern and fields Create test/kuttl/test-suites/placement/deps/ directory with namespace and kustomizaton.yaml, create symlink for shared dependency from default/deps
Remove 'placement' prefix from filenames within placement directories to match existing naming conventions ex: placementapi_types.go → placement/api_types.go
Update all import paths in placement code from github.com/openstack-k8s-operators/placement-operator/api/v1beta1 to github.com/openstack-k8s-operators/nova-operator/api/placement/v1beta1 and github.com/openstack-k8s-operators/placement-operator/internal/placement github.com/openstack-k8s-operators/nova-operator/internal/placement package name update functional_test to placement_test
Update kustomization.yaml to include placement samples
Update controller-gen paths to process both nova and placement APIs
- Register PlacementAPI controller in cmd/main.go - Register PlacementAPI webhook in cmd/main.go - Add placement scheme and defaults initialization - Fix placement test paths after moving to test/functional/placement/ - Add placement API to test coverage in Makefile
Updated Makefile to include placement webhook paths in controller-gen
Adds getTemplateInstanceType in placement/common.go
Remove placement-import directory - all code has been moved Add generated files from 'make manifests
Contributor
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: auniyal61 The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
|
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
|
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
18 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Merge placement-operator/main into nova-operator