Skip to content

Commit

Permalink
✨ replace windup with analyzer-lsp again (#254)
Browse files Browse the repository at this point in the history
Signed-off-by: David Zager <david.j.zager@gmail.com>
  • Loading branch information
djzager authored Jul 24, 2023
1 parent 1f2aac0 commit ab7c353
Show file tree
Hide file tree
Showing 14 changed files with 64 additions and 175 deletions.
20 changes: 10 additions & 10 deletions .github/actions/install-tackle/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,20 +22,20 @@ inputs:
description: "image urd for admin addon"
required: false
default: "quay.io/konveyor/tackle2-addon:latest"
tackle-addon-windup-image:
description: "image url for windup addon"
tackle-addon-analyzer-image:
description: "image url for analyzer addon"
required: false
default: "quay.io/konveyor/tackle2-addon-windup:latest"
default: "quay.io/konveyor/tackle2-addon-analyzer:latest"
tackle-image-pull-policy:
description: "Image Pull Policy"
required: false
default: "Always"
tackle-windup-container-cpu:
description: "The CPU request value for the windup task containers"
tackle-analyzer-container-cpu:
description: "The CPU request value for the analyzer task containers"
required: false
default: 0
tackle-windup-container-memory:
description: "The memory request value for the windup task containers"
tackle-analyzer-container-memory:
description: "The memory request value for the analyzer task containers"
required: false
default: 0
# TODO(djzager): uncomment when supported
Expand All @@ -55,10 +55,10 @@ runs:
export TACKLE_PATHFINDER_IMAGE="${{ inputs.tackle-pathfinder-image }}"
export TACKLE_UI_IMAGE="${{ inputs.tackle-ui-image }}"
export TACKLE_ADDON_ADMIN_IMAGE="${{ inputs.tackle-addon-admin-image }}"
export TACKLE_ADDON_WINDUP_IMAGE="${{ inputs.tackle-addon-windup-image }}"
export TACKLE_ADDON_ANALYZER_IMAGE="${{ inputs.tackle-addon-analyzer-image }}"
export TACKLE_IMAGE_PULL_POLICY="${{ inputs.tackle-image-pull-policy }}"
export TACKLE_WINDUP_CONTAINER_REQUESTS_MEMORY="${{ inputs.tackle-windup-container-memory }}"
export TACKLE_WINDUP_CONTAINER_REQUESTS_CPU="${{ inputs.tackle-windup-container-cpu }}"
export TACKLE_ANALYZER_CONTAINER_REQUESTS_MEMORY="${{ inputs.tackle-analyzer-container-memory }}"
export TACKLE_ANALYZER_CONTAINER_REQUESTS_CPU="${{ inputs.tackle-analyzer-container-cpu }}"
make install-tackle
working-directory: ${{ github.action_path }}/../../..
shell: bash
4 changes: 2 additions & 2 deletions .github/workflows/create-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ jobs:
- konveyor/tackle2-hub
- konveyor/tackle2-ui
- konveyor/tackle2-addon
- konveyor/tackle2-addon-windup
- konveyor/tackle2-addon-analyzer
- konveyor/operator
- konveyor/tackle-pathfinder
- konveyor/tackle-keycloak-theme
Expand All @@ -63,7 +63,7 @@ jobs:
- konveyor/tackle2-hub
- konveyor/tackle2-ui
- konveyor/tackle2-addon
- konveyor/tackle2-addon-windup
- konveyor/tackle2-addon-analyzer
- konveyor/tackle2-operator
- konveyor/tackle-pathfinder
- konveyor/tackle-keycloak-init
Expand Down
21 changes: 5 additions & 16 deletions bundle/manifests/konveyor-operator.clusterserviceversion.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,17 +4,6 @@ metadata:
annotations:
alm-examples: |-
[
{
"apiVersion": "tackle.konveyor.io/v1alpha1",
"kind": "Addon",
"metadata": {
"name": "move2kube-plan",
"namespace": "konveyor-tackle"
},
"spec": {
"image": "quay.io/konveyor/tackle-addon-move2kube-plan:latest"
}
},
{
"apiVersion": "tackle.konveyor.io/v1alpha1",
"kind": "Tackle",
Expand All @@ -31,7 +20,7 @@ metadata:
categories: Modernization & Migration
certified: "false"
containerImage: quay.io/konveyor/tackle2-operator:latest
createdAt: "2023-06-08T20:09:08Z"
createdAt: "2023-07-24T15:04:42Z"
description: Konveyor is an open-source application modernization platform that
helps organizations safely and predictably modernize applications to Kubernetes
at scale.
Expand Down Expand Up @@ -192,8 +181,8 @@ spec:
value: quay.io/konveyor/tackle2-ui:latest
- name: RELATED_IMAGE_ADDON_ADMIN
value: quay.io/konveyor/tackle2-addon:latest
- name: RELATED_IMAGE_ADDON_WINDUP
value: quay.io/konveyor/tackle2-addon-windup:latest
- name: RELATED_IMAGE_ADDON_ANALYZER
value: quay.io/konveyor/tackle2-addon-analyzer:latest
image: quay.io/konveyor/tackle2-operator:latest
imagePullPolicy: Always
livenessProbe:
Expand Down Expand Up @@ -382,6 +371,6 @@ spec:
name: tackle-ui
- image: quay.io/konveyor/tackle2-addon:latest
name: addon-admin
- image: quay.io/konveyor/tackle2-addon-windup:latest
name: addon-windup
- image: quay.io/konveyor/tackle2-addon-analyzer:latest
name: addon-analyzer
version: 99.0.0
4 changes: 2 additions & 2 deletions config/manager/manager.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -58,8 +58,8 @@ spec:
value: quay.io/konveyor/tackle2-ui:latest
- name: RELATED_IMAGE_ADDON_ADMIN
value: quay.io/konveyor/tackle2-addon:latest
- name: RELATED_IMAGE_ADDON_WINDUP
value: quay.io/konveyor/tackle2-addon-windup:latest
- name: RELATED_IMAGE_ADDON_ANALYZER
value: quay.io/konveyor/tackle2-addon-analyzer:latest
name: tackle-operator
image: controller:latest # this will be replaced with IMG by `make bundle`
imagePullPolicy: Always
Expand Down
1 change: 0 additions & 1 deletion config/samples/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
## Append samples you want in your CSV to this file as resources ##
resources:
- tackle_v1alpha1_tackle.yaml
- tackle_v1alpha1_addon.yaml
#+kubebuilder:scaffold:manifestskustomizesamples
7 changes: 0 additions & 7 deletions config/samples/tackle_v1alpha1_addon.yaml

This file was deleted.

12 changes: 6 additions & 6 deletions hack/install-tackle.sh
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,10 @@ TACKLE_PATHFINDER_IMAGE="${TACKLE_PATHFINDER_IMAGE:-quay.io/konveyor/tackle-path
TACKLE_UI_IMAGE="${TACKLE_UI_IMAGE:-quay.io/konveyor/tackle2-ui:latest}"
TACKLE_UI_INGRESS_CLASS_NAME="${TACKLE_UI_INGRESS_CLASS_NAME:-nginx}"
TACKLE_ADDON_ADMIN_IMAGE="${TACKLE_ADDON_ADMIN_IMAGE:-quay.io/konveyor/tackle2-addon:latest}"
TACKLE_ADDON_WINDUP_IMAGE="${TACKLE_ADDON_WINDUP_IMAGE:-quay.io/konveyor/tackle2-addon-windup:latest}"
TACKLE_ADDON_ANALYZER_IMAGE="${TACKLE_ADDON_ANALYZER_IMAGE:-quay.io/konveyor/tackle2-addon-analyzer:latest}"
TACKLE_IMAGE_PULL_POLICY="${TACKLE_IMAGE_PULL_POLICY:-Always}"
TACKLE_WINDUP_CONTAINER_REQUESTS_MEMORY="${TACKLE_WINDUP_CONTAINER_REQUESTS_MEMORY:-0}"
TACKLE_WINDUP_CONTAINER_REQUESTS_CPU="${TACKLE_WINDUP_CONTAINER_REQUESTS_CPU:-0}"
TACKLE_ANALYZER_CONTAINER_REQUESTS_MEMORY="${TACKLE_ANALYZER_CONTAINER_REQUESTS_MEMORY:-0}"
TACKLE_ANALYZER_CONTAINER_REQUESTS_CPU="${TACKLE_ANALYZER_CONTAINER_REQUESTS_CPU:-0}"

TACKLE_FEATURE_AUTH_REQUIRED="${TACKLE_FEATURE_AUTH_REQUIRED:-false}"

Expand Down Expand Up @@ -90,10 +90,10 @@ spec:
ui_image_fqin: ${TACKLE_UI_IMAGE}
ui_ingress_class_name: ${TACKLE_UI_INGRESS_CLASS_NAME}
admin_fqin: ${TACKLE_ADDON_ADMIN_IMAGE}
windup_fqin: ${TACKLE_ADDON_WINDUP_IMAGE}
analyzer_fqin: ${TACKLE_ADDON_ANALYZER_IMAGE}
image_pull_policy: ${TACKLE_IMAGE_PULL_POLICY}
windup_container_requests_memory: ${TACKLE_WINDUP_CONTAINER_REQUESTS_MEMORY}
windup_container_requests_cpu: ${TACKLE_WINDUP_CONTAINER_REQUESTS_CPU}
analyzer_container_requests_memory: ${TACKLE_ANALYZER_CONTAINER_REQUESTS_MEMORY}
analyzer_container_requests_cpu: ${TACKLE_ANALYZER_CONTAINER_REQUESTS_CPU}
EOF
# Wait for reconcile to finish
kubectl wait \
Expand Down
28 changes: 13 additions & 15 deletions roles/tackle/defaults/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,22 +24,20 @@ hub_deployment_name: "{{ hub_service_name }}"
hub_deployment_replicas: "1"
hub_deployment_strategy: "Recreate"
hub_container_name: "{{ hub_service_name }}"
hub_container_limits_cpu: "500m"
hub_container_limits_memory: "800Mi"
hub_container_requests_cpu: "100m"
hub_container_requests_memory: "350Mi"
hub_container_limits_cpu: "1"
hub_container_limits_memory: "2Gi"
hub_container_requests_cpu: "500m"
hub_container_requests_memory: "1Gi"
hub_database_filename: "hub.db"
hub_database_volume_name: "{{ hub_service_name }}-database"
hub_database_volume_size: "5Gi"
hub_database_volume_size: "10Gi"
hub_database_volume_path: "/database"
hub_database_volume_claim_name: "{{ hub_service_name }}-database-volume-claim"
hub_bucket_volume_name: "{{ hub_service_name }}-bucket"
hub_bucket_volume_size: "100Gi"
hub_bucket_volume_path: "/buckets"
hub_bucket_volume_claim_name: "{{ hub_service_name }}-bucket-volume-claim"
hub_bucket_volume_claim_mode: ReadWriteOnce
hub_seed_configmap_name: "{{ hub_service_name }}-seed"
hub_seed_configmap_path: "/seed"
hub_addon_working_path: "/working"
hub_task_sa: "{{ app_name }}-hub"
hub_tls_enabled: false
Expand Down Expand Up @@ -179,14 +177,14 @@ admin_service_name: "{{ app_name }}-{{ admin_name }}-{{ admin_component_name }}"
admin_container_requests_cpu: "50m"
admin_container_requests_memory: "50Mi"

windup_fqin: "{{ lookup('env', 'RELATED_IMAGE_ADDON_WINDUP') }}"
windup_name: "windup"
windup_component_name: "addon"
windup_service_name: "{{ app_name }}-{{ windup_name }}-{{ windup_component_name }}"
windup_container_limits_cpu: "1"
windup_container_limits_memory: "6Gi"
windup_container_requests_cpu: "1"
windup_container_requests_memory: "4Gi"
analyzer_fqin: "{{ lookup('env', 'RELATED_IMAGE_ADDON_ANALYZER') }}"
analyzer_name: "analyzer"
analyzer_component_name: "addon"
analyzer_service_name: "{{ app_name }}-{{ analyzer_name }}-{{ analyzer_component_name }}"
analyzer_container_limits_cpu: "1"
analyzer_container_limits_memory: "4Gi"
analyzer_container_requests_cpu: "1"
analyzer_container_requests_memory: "2Gi"

cache_name: "cache"
cache_data_volume_size: "100Gi"
Expand Down
9 changes: 2 additions & 7 deletions roles/tackle/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -358,11 +358,6 @@
state: present
definition: "{{ lookup('template', 'persistentvolumeclaim-hub-bucket.yml.j2') }}"

- name: "Setup Hub Seed ConfigMap"
k8s:
state: present
definition: "{{ lookup('template', 'configmap-hub-seed.yml.j2') }}"

- name: "Check if Hub Secret exists already so we don't update it"
k8s_info:
api_version: v1
Expand Down Expand Up @@ -472,7 +467,7 @@
- name: "Create Windup Addon CR"
k8s:
state: present
definition: "{{ lookup('template', 'customresource-addon-windup.yml.j2') }}"
definition: "{{ lookup('template', 'customresource-addon-analyzer.yml.j2') }}"

- name: "Create Network Policy"
k8s:
Expand Down Expand Up @@ -520,4 +515,4 @@
definition: "{{ lookup('template', 'servicemonitor-hub.yml.j2') }}"
when:
- hub_metrics_enabled|bool
- openshift_cluster|bool
- openshift_cluster|bool
77 changes: 0 additions & 77 deletions roles/tackle/templates/configmap-hub-seed.yml.j2

This file was deleted.

20 changes: 20 additions & 0 deletions roles/tackle/templates/customresource-addon-analyzer.yml.j2
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
---
kind: Addon
apiVersion: tackle.konveyor.io/v1alpha1
metadata:
name: {{ analyzer_name }}
namespace: {{ app_namespace }}
labels:
app.kubernetes.io/name: {{ analyzer_service_name }}
app.kubernetes.io/component: {{ analyzer_component_name }}
app.kubernetes.io/part-of: {{ app_name }}
spec:
image: {{ analyzer_fqin }}
imagePullPolicy: {{ image_pull_policy }}
resources:
limits:
cpu: {{ analyzer_container_limits_cpu }}
memory: {{ analyzer_container_limits_memory }}
requests:
cpu: {{ analyzer_container_requests_cpu }}
memory: {{ analyzer_container_requests_memory }}
20 changes: 0 additions & 20 deletions roles/tackle/templates/customresource-addon-windup.yml.j2

This file was deleted.

8 changes: 0 additions & 8 deletions roles/tackle/templates/deployment-hub.yml.j2
Original file line number Diff line number Diff line change
Expand Up @@ -81,8 +81,6 @@ spec:
value: "{{ hub_addon_working_path }}"
- name: DB_PATH
value: "{{ hub_database_volume_path }}/{{ hub_database_filename }}"
- name: DB_SEED_PATH
value: "{{ hub_seed_configmap_path }}"
- name: BUCKET_PATH
value: "{{ hub_bucket_volume_path }}"
- name: BUCKET_PVC
Expand Down Expand Up @@ -187,9 +185,6 @@ spec:
mountPath: {{ hub_database_volume_path }}
- name: {{ hub_bucket_volume_name }}
mountPath: {{ hub_bucket_volume_path }}
- name: {{ hub_seed_configmap_name }}
mountPath: {{ hub_seed_configmap_path }}
readOnly: true
{% if rwx_supported|bool %}
- name: {{ cache_data_volume_name }}
mountPath: {{ cache_mount_path }}
Expand All @@ -210,9 +205,6 @@ spec:
- name: {{ hub_bucket_volume_name }}
persistentVolumeClaim:
claimName: {{ hub_bucket_volume_claim_name }}
- name: {{ hub_seed_configmap_name }}
configMap:
name: {{ hub_seed_configmap_name }}
{% if hub_tls_enabled|bool %}
- name: {{ hub_tls_secret_name }}
secret:
Expand Down
Loading

0 comments on commit ab7c353

Please sign in to comment.