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

✨ replace windup with analyzer-lsp again #254

Merged
merged 1 commit into from
Jul 24, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
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