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

Backport MTV-1645: Look for annotation on NAD to set transfer route #1315

Open
wants to merge 2 commits into
base: release-2.7
Choose a base branch
from

Conversation

mansam
Copy link
Contributor

@mansam mansam commented Jan 23, 2025

Backport of #1244 to release-2.7.

The Plan controller will now look for an annotation
`forklift.konveyor.io/route` on the NetworkAttachmentDefinition
that was specified as the transfer network for the Plan.

The annotation is expected to have a single IP address as its
value, which will be set as the default route when configuring
the secondary network on the importer pods.

The annotation is optional, but if it is present then the
Plan validator will ensure that its value is a valid IP address.

Signed-off-by: Sam Lucidi <slucidi@redhat.com>
Signed-off-by: Sam Lucidi <slucidi@redhat.com>
@mansam mansam requested review from mnecas and yaacov as code owners January 23, 2025 15:13
@mnecas mnecas added this to the 2.7.10 milestone Jan 24, 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.

2 participants