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

fix(cvc): set the bind address using a go flag in cvc server config #332

Merged

Conversation

prateekpandey14
Copy link
Contributor

@prateekpandey14 prateekpandey14 commented May 12, 2021

Signed-off-by: prateekpandey14 prateek.pandey@mayadata.io

What this PR does ?

  • Remove the entrypoint.sh script in Dockerfile , used to set the bind address for cvc api server (backup/restore endpoints), reset/overwrite the other flags passed as args to the deployment container.

  • Updated the cvc operator deployment spec args to include the --bind flag to set the server IP using downward API status.podIP

@github-actions github-actions bot requested review from kmova and mittachaitu May 12, 2021 15:02
@prateekpandey14 prateekpandey14 requested review from shubham14bajpai and removed request for kmova May 12, 2021 15:04
@prateekpandey14 prateekpandey14 force-pushed the refact-entrypoint branch 2 times, most recently from 325df2e to 0cb9eca Compare May 12, 2021 15:26
@github-actions github-actions bot requested review from kmova and sonasingh46 May 12, 2021 15:27
@github-actions github-actions bot requested review from kmova and sonasingh46 May 12, 2021 15:51
- Remove the entrypoint.sh script in Dockerfile , used to set
  the bind address for cvc api server (backup/restore endpoints)
  reset/overwrite the other flags passed as args to the deployment
  container.

- Updated the cvc operator deployment spec args to include the
  `-bind` flag to set the server IP using downward API status.podIP

Signed-off-by: prateekpandey14 <prateek.pandey@mayadata.io>
Copy link
Contributor

@sonasingh46 sonasingh46 left a comment

Choose a reason for hiding this comment

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

Alt Text

Copy link
Contributor

@mittachaitu mittachaitu left a comment

Choose a reason for hiding this comment

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

provided minor comments

deploy/cstor-operator.yaml Outdated Show resolved Hide resolved
build/cvc-operator/Dockerfile.arm64 Show resolved Hide resolved
pkg/controllers/cstorvolumeconfig/start.go Show resolved Hide resolved
Signed-off-by: prateekpandey14 <prateek.pandey@mayadata.io>
@prateekpandey14 prateekpandey14 added area: cvc-operator bug Something isn't working labels May 17, 2021
Copy link
Contributor

@mittachaitu mittachaitu left a comment

Choose a reason for hiding this comment

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

changes are good

@prateekpandey14 prateekpandey14 merged commit e5960e0 into openebs-archive:master May 17, 2021
@prateekpandey14 prateekpandey14 deleted the refact-entrypoint branch May 17, 2021 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: cvc-operator bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants