Skip to content
This repository has been archived by the owner on Jun 25, 2024. It is now read-only.

Commit

Permalink
Ensure a valid name for AEE
Browse files Browse the repository at this point in the history
"a lowercase RFC 1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character "
ref:  openstack-k8s-operators/data-plane-adoption#176 (comment)

When Label wasn't provided it was breaking the AEE deploy

Signed-off-by: Fabricio Aguiar <[email protected]>
  • Loading branch information
fao89 committed Oct 27, 2023
1 parent 9b56e74 commit 11abc24
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions pkg/util/ansible_execution.go
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,9 @@ func AnsibleExecution(
}
if ansibleEE == nil {
executionName := fmt.Sprintf("%s-%s", label, obj.GetName())
if len(label) == 0 {
executionName = obj.GetName()
}
ansibleEE = &ansibleeev1.OpenStackAnsibleEE{
ObjectMeta: metav1.ObjectMeta{
Name: executionName,
Expand Down

0 comments on commit 11abc24

Please sign in to comment.