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

Add rsync environment variables to mtc controller #540

Merged
merged 1 commit into from Jan 6, 2021

Conversation

pranavgaikwad
Copy link
Collaborator

Description
Adds new environment variables in MTC container deployment to allow configuring Rsync flags.

Check each of the following when appropriate to help reviewers verify work is complete.

Modifying an existing version

  • I modified operator permissions in the OLM CSV and operator.yml
  • I modified the operator deployment in the OLM CSV and operator.yml
  • I modified operand permissions in the OLM CSV and ansible role
  • I modified CRDS in the OLM CSV and ansible role

Adding a new release version

  • I created a new z release directory in deploy/olm-catalog/konveyor-operator
  • I updated channels in the konveyor-operator.package.yaml
  • I created a new release directory in deploy/non-olm
  • I created or updated the major.minor link in deploy/non-olm
  • I updated the spec.skips parameter in the CSV

@pranavgaikwad pranavgaikwad changed the title add rsync option environment variables to mtc controller Add rsync environment variables to mtc controller Jan 5, 2021
@pranavgaikwad pranavgaikwad marked this pull request as ready for review January 5, 2021 22:27
Copy link
Collaborator

@jmontleon jmontleon left a comment

Choose a reason for hiding this comment

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

ACK

@jmontleon jmontleon added the 1.4.0 label Jan 6, 2021
@jmontleon jmontleon merged commit 103e9c0 into migtools:master Jan 6, 2021
jmontleon pushed a commit that referenced this pull request Jan 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants