-
Notifications
You must be signed in to change notification settings - Fork 101
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
Adding support for adoption scenario 1 #818
base: main
Are you sure you want to change the base?
Adding support for adoption scenario 1 #818
Conversation
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/c5c21e7abf014fe09fd09f29112f63c2 ✔️ openstack-k8s-operators-content-provider SUCCESS in 57m 02s |
Makefile
Outdated
@@ -35,6 +35,22 @@ CLEANUP_DIR_CMD ?= rm -Rf | |||
# be also cloned from OPENSTACK_REPO/OPENSTACK_BRANCH. | |||
CHECKOUT_FROM_OPENSTACK_REF ?= false | |||
|
|||
ADOPTED_SCENARIO ?=false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
names of variables here should not refer to adoption. There's a general case for deploying CP and DP services in different ranges. We should be able to stand up greenfield deployment with different ranges.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This applies both to the code and the commit title / message.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please give a playbook of commands that one would use to deploy the setup as you intend? Please add them to commit message.
Please also fix pre-commits.
Makefile
Outdated
@@ -35,6 +35,22 @@ CLEANUP_DIR_CMD ?= rm -Rf | |||
# be also cloned from OPENSTACK_REPO/OPENSTACK_BRANCH. | |||
CHECKOUT_FROM_OPENSTACK_REF ?= false | |||
|
|||
ADOPTED_SCENARIO ?=false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This applies both to the code and the commit title / message.
devsetup/scripts/standalone.sh
Outdated
@@ -217,6 +217,16 @@ dns_server: ${PRIMARY_RESOLV_CONF_ENTRY} | |||
compute_driver: ${COMPUTE_DRIVER} | |||
sriov_agent: ${EDPM_COMPUTE_SRIOV_ENABLED} | |||
dhcp_agent: ${EDPM_COMPUTE_DHCP_AGENT_ENABLED} | |||
# Adoption related stuff | |||
adoption_scenario: ${ADOPTION_SCENARIO_A} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
instead of referring to "adoption" here, please refer to "host_routes" that should be added instead; e.g. storage_host_routes
, tenant_host_routes
, etc.
scripts/gen-netatt.sh
Outdated
if [ -z "${DEPLOY_DIR}" ]; then | ||
echo "Please set DEPLOY_DIR"; exit 1 | ||
fi | ||
function check_var_setted () { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
setted -> set
scripts/gen-nncp.sh
Outdated
if [ -z "${DEPLOY_DIR}" ]; then | ||
echo "Please set DEPLOY_DIR"; exit 1 | ||
fi | ||
function check_var_setted () { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
scripts/gen-nncp.sh
Outdated
@@ -145,6 +133,25 @@ EOF_CAT | |||
next-hop-interface: ${BRIDGE_NAME} | |||
EOF_CAT | |||
done | |||
fi | |||
if [ -n "$ADOPTION_CASE_A" ]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be smth like (pseudo-code)
if INTERNALAPI_HOST_ROUTES:
echo INTERNALAPI_HOST_ROUTES >>
if TENANT_HOST_ROUTES:
...
etc. etc.
next-hop-interface: ${INTERFACE}.${tenant_vlan_id} | ||
EOF_CAT | ||
fi | ||
if [ -n "$IPV6_ENABLED" ]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hm. why is it needed?
08dcc5e
to
d870686
Compare
f0b68ff
to
4e9acfb
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/68f215690f5042079e33575183d8afee ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 38m 10s |
4e9acfb
to
d28295b
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/c2a9060024d747bb950c8ee43e94b81b ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 38m 15s |
acbb1ae
to
296dd1c
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/e664007fab804645b3184d51e04f4172 ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 46m 08s |
303c31d
to
8f099cc
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/f201b76d01f447bdab030c2eb9a6165a ✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 38m 02s |
recheck |
07f88df
to
e28ad08
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/ce81e17c11b545b8a34edaef5d32f2d9 ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 10m 17s |
e28ad08
to
e69bd16
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/661344149bd14f6788c276224778c41c ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 21m 53s |
recheck |
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/3e8be8aa1a3a4bbab5e6d3af0d2aecac ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 43m 19s |
recheck |
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
e69bd16
to
179ad80
Compare
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: averdagu, fao89 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
179ad80
to
790a225
Compare
Build failed (check pipeline). Post https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/64c0f0b2b3be4ef5ae747b53520e1b6c ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 54m 28s |
recheck |
242e1cf
to
afc4d32
Compare
Dataplane adoption docs supports doing the adoption while wallaby environment uses different subnets than next generation. This commit enables Makefiles in order to prepare the environment with different subnets between environments. Related: OSPRH-5881
afc4d32
to
a7f1cb4
Compare
Dataplane adoption docs supports doing the adoption while wallaby environment uses different subnets than next generation. This commit enables Makefiles in order to prepare the environment with different subnets between environments.
Related: OSPRH-5881