Skip to content

chore: update clamav host - #1949

Merged
jasonchung1871 merged 1 commit into
bcgov:mainfrom
jasonchung1871:update-clamav-host
Aug 25, 2026
Merged

chore: update clamav host#1949
jasonchung1871 merged 1 commit into
bcgov:mainfrom
jasonchung1871:update-clamav-host

Conversation

@jasonchung1871

Copy link
Copy Markdown
Contributor

Description

clamav hosts are updated from static ip to hostname, no longer need to set the ip address manually if the clamav deployment goes down and comes back.

Type of Change

fix (a bug fix)

build (change in build system or dependencies)
ci (change in continuous integration / deployment)

Checklist

  • I have read the CONTRIBUTING doc
  • I have checked that unit tests pass locally with my changes
  • I have run the npm script lint on the frontend and backend
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • I have approval from the product owner for the contribution in this pull request

Further comments

clamav hosts are updated from static ip to hostname
Copilot AI lite review requested due to automatic review settings August 25, 2026 16:56
@sonarqubecloud

Copy link
Copy Markdown

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the OpenShift ClamAV connection configuration to use a stable service hostname instead of static cluster IPs, so the application remains able to reach ClamAV across redeployments.

Changes:

  • Switched ClamAV HOST in dev/test/prod OpenShift param files from fixed IPs to chefs-clamav.
  • Left ClamAV port unchanged (3310) across environments.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
openshift/clamav.dev.param Updates ClamAV host to chefs-clamav for dev deployments.
openshift/clamav.test.param Updates ClamAV host to chefs-clamav for test deployments.
openshift/clamav.prod.param Updates ClamAV host to chefs-clamav for prod deployments.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@jasonchung1871
jasonchung1871 merged commit f3f8731 into bcgov:main Aug 25, 2026
7 checks passed
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.

2 participants