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

Hardcoded namespace in jobservice.redis_pool #1916

Open
payxn opened this issue Jan 22, 2025 · 2 comments
Open

Hardcoded namespace in jobservice.redis_pool #1916

payxn opened this issue Jan 22, 2025 · 2 comments
Assignees

Comments

@payxn
Copy link

payxn commented Jan 22, 2025

Hi Harbor team,

I noticed that in the Harbor Helm chart, the jobservice configuration includes a hardcoded namespace:

redis_pool:  
  redis_url: "{{ template "harbor.redis.urlForJobservice" . }}"
  namespace: "harbor_job_service_namespace"  

Currently, the namespace value is hardcoded as "harbor_job_service_namespace".

I encountered issues with the garbage collection job, which was consistently in an ERROR state with the following message:

2025-01-21T12:05:04Z [ERROR] [/jobservice/runner/redis.go:123]: Job 'GARBAGE_COLLECTION:556a5514773eda909642fa2d' exit with error: run error: error retrieving 'blobs::*' keys

Initially, I thought that "harbor_job_service_namespace" was meant to be a reference that shouldn't be customized. However, after changing it to match the current Redis namespace, the garbage collection jobs started working correctly.

Could you please confirm if my understanding is correct and whether this variable should indeed be configurable?

Thanks in advance for your support!

@MinerYang
Copy link
Collaborator

MinerYang commented Jan 27, 2025

Hi @payxn ,

How do you install and use the redis? Jobservice load the config from a internal config yaml file by default,so if jobservice start and running normally without any error msg, this issue suppose not occurs due to the namespace config.
BTW, have you set any env JOB_SERVICE_POOL_REDIS_NAMESPACE inside your container?

@MinerYang MinerYang self-assigned this Jan 27, 2025
@payxn
Copy link
Author

payxn commented Jan 30, 2025

Hello,

We install redis with the chart with the default values. And we don't have JOB_SERVICE_POOL_REDIS_NAMESPACE env var in our container.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants