Skip to content
This repository has been archived by the owner on Aug 30, 2023. It is now read-only.

adding private network to vm & fullvm #1181

Merged
merged 4 commits into from
Dec 27, 2022

Conversation

zaelgohary
Copy link
Contributor

Description

Adding network tab to VM & FullVm

Changes

  • Adding an advanced tab which contains network name & IP range in VM && FullVM
  • fixing some linting errors regarding previous issues

Related Issues

Copy link
Contributor

@AhmedHanafy725 AhmedHanafy725 left a comment

Choose a reason for hiding this comment

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

I don't see a change to the network field on the machine's model, is it propagated?

@zaelgohary zaelgohary marked this pull request as draft December 26, 2022 17:50
@zaelgohary
Copy link
Contributor Author

I don't see a change to the network field on the machine's model, is it propagated?

The network name & IP range gets changed as I've logged the submitted values when deployed but the change doesn't reflect in the deployment data JSON.

@AhmedHanafy725 AhmedHanafy725 marked this pull request as ready for review December 27, 2022 13:23
@Mahmoud-Emad
Copy link
Contributor

Looks good to me!

@AhmedHanafy725 AhmedHanafy725 merged commit 5798722 into development Dec 27, 2022
@AhmedHanafy725 AhmedHanafy725 deleted the development_VM_network branch December 27, 2022 14:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants