Skip to content

Support ETHx_IP6_GATEWAY, SET_HOSTNAME on OpenNebula#1045

Merged
TheRealFalcon merged 4 commits into
canonical:mainfrom
vholer:one-ip6-gateway
Oct 7, 2021
Merged

Support ETHx_IP6_GATEWAY, SET_HOSTNAME on OpenNebula#1045
TheRealFalcon merged 4 commits into
canonical:mainfrom
vholer:one-ip6-gateway

Conversation

@vholer
Copy link
Copy Markdown
Contributor

@vholer vholer commented Sep 30, 2021 •
edited
Loading

Proposed Commit Message

summary: Support for ETHx_IP6_GATEWAY and SET_HOSTNAME on OpenNebula

OpenNebula 6.1.80 (current dev. version) is introducing new IPv6 gateway
contextualization variable ETHx_IP6_GATEWAY, which mimics existing
variable ETHx_GATEWAY6. The ETHx_GATEWAY6 used until now will
be depracated in future relase (ET spring 2022).

See:
- new variable - https://github.com/OpenNebula/one/commit/e4d2cc11b9f3c6d01b53774b831f48d9d089c1cc
- deprecation tracking issue - https://github.com/OpenNebula/one/issues/5536

Also, added support for SET_HOSTNAME context variable, which is
currently widely used variable to configure guest VM hostname. See
https://docs.opennebula.io/6.0/management_and_operations/references/template.html#context-section

Additional Context

Test Steps

Checklist:

  • My code follows the process laid out in the documentation
  • I have updated or added any unit tests accordingly
  • I have updated or added any documentation accordingly

@vholer vholer mentioned this pull request Sep 30, 2021
Open
7 tasks
@vholer vholer mentioned this pull request Oct 6, 2021
Closed
7 tasks
TheRealFalcon
TheRealFalcon reviewed Oct 6, 2021
View reviewed changes
Comment thread tests/unittests/test_datasource/test_opennebula.py Outdated
for dev in ('eth0', 'ens3'):
m_get_phys_by_mac.return_value = {MACADDR: dev}
for k in ('HOSTNAME', 'PUBLIC_IP', 'IP_PUBLIC', 'ETH0_IP'):
for k in ('SET_HOSTNAME', 'HOSTNAME', 'PUBLIC_IP', 'IP_PUBLIC', 'ETH0_IP'):
Copy link
Copy Markdown
Contributor

@TheRealFalcon TheRealFalcon Oct 6, 2021

Choose a reason for hiding this comment

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

flake8: line to long here

@TheRealFalcon TheRealFalcon merged commit 1bbb67c into canonical:main Oct 7, 2021
This was referenced May 12, 2023
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@TheRealFalcon TheRealFalcon TheRealFalcon approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vholer @TheRealFalcon