Skip to content

Conversation

@ferruhcihan
Copy link
Contributor

@ferruhcihan ferruhcihan commented Sep 24, 2025

📌 Summary

This PR updates Redis helm chart docs and network policies lab according to the apl-charts repository’s redis-ha chart changes.

Ticket: https://track.akamai.com/jira/browse/APL-1079
PRs: apl-api | apl-charts

🔍 Reviewer Notes

Testing:

  • Create a cluster (select g6-dedicated-8 to avoid hitting resource limits),
    or update the otomi-api app _rawValues in your cluster:
        image:
            # registry: docker.io
            # repository: linode/apl-api
            pullPolicy: Always
            tag: APL-1079

@ferruhcihan ferruhcihan requested a review from merll October 9, 2025 14:46
@jverma-git
Copy link
Collaborator

@ferruhcihan does this mean that the existing netpol automation needs to be changed ?
for example, we have this now.

    values = load_values_from_chart(otomi_charts_repo, "k8s-deployment")
    values["image"]["repository"] = image_url
    values["image"]["tag"] = "main"
    values["containerPorts"] = [{"name": "http", "containerPort": 80, "protocol": "TCP"}]
    values["env"].append({"name": "REDIS_HOST", "value": f"{redis_workload.name}-quickstart-redis-master"})

So
values["env"].append({"name": "REDIS_HOST", "value": f"{redis_workload.name}-redis-ha"}) ?

@ferruhcihan ferruhcihan changed the base branch from main to netpol-labs October 9, 2025 15:22
@ferruhcihan
Copy link
Contributor Author

@ferruhcihan does this mean that the existing netpol automation needs to be changed ? for example, we have this now.

    values = load_values_from_chart(otomi_charts_repo, "k8s-deployment")
    values["image"]["repository"] = image_url
    values["image"]["tag"] = "main"
    values["containerPorts"] = [{"name": "http", "containerPort": 80, "protocol": "TCP"}]
    values["env"].append({"name": "REDIS_HOST", "value": f"{redis_workload.name}-quickstart-redis-master"})

So values["env"].append({"name": "REDIS_HOST", "value": f"{redis_workload.name}-redis-ha"}) ?

@jverma-git Yes, REDIS_HOST will be the redis workload name. So, for both the vote and worker workloads, the line will be like: values["env"].append({"name": "REDIS_HOST", "value": f"{redis_workload.name}"}) ?

@ferruhcihan ferruhcihan changed the base branch from netpol-labs to main October 15, 2025 09:53
@ferruhcihan ferruhcihan merged commit bf92632 into main Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants