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

Implement manage gateways in playground #1215

Conversation

MohamedElmdary
Copy link
Member

@AhmedHanafy725
Copy link
Contributor

after deploying a micro vm, couldn't list it

image

@AhmedHanafy725
Copy link
Contributor

the network name and the IP shouldn't be editable

image

@AhmedHanafy725
Copy link
Contributor

add an option for the user while deploying the gateway to allow pass-through or not. (add tool-tip to explain what it's used for)
(default: false)

Comment on lines 180 to 181
await this.tfClient.applyAllExtrinsics(ext1);
await this.tfClient.applyAllExtrinsics(ext2);
Copy link
Contributor

Choose a reason for hiding this comment

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

let's apply these extrinsics together but the new ones first on the list then the old ones

Copy link
Member Author

Choose a reason for hiding this comment

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

@AhmedHanafy725 What text should I add on the tooltip for explaining pass-through?

Copy link
Member Author

Choose a reason for hiding this comment

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

let's apply these extrinsics together but the new ones first on the list then the old ones

Merged both together

Copy link
Member Author

Choose a reason for hiding this comment

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

Added tls pass through option

@MohamedElmdary MohamedElmdary mentioned this pull request Oct 15, 2023
@MohamedElmdary
Copy link
Member Author

the network name and the IP shouldn't be editable

image

It's already readonly user can only copy them!

@MohamedElmdary
Copy link
Member Author

after deploying a micro vm, couldn't list it

image

Found the issue and fixed and tested on all solutions

@Mahmoud-Emad
Copy link
Contributor

Great job, one small comment, what do you think if we replaced the check box of the custom domain option to be the same as the custom domain as added in the other solutions, Switch I mean

@AhmedHanafy725
Copy link
Contributor

the gateway keys are not migrated while listing the instances

@AhmedHanafy725
Copy link
Contributor

when deploying a peertube instance, the name contract contains NaN

image

@MohamedElmdary MohamedElmdary added type_feature New feature or request playground labels Oct 18, 2023
@mohamedamer453
Copy link
Contributor

I was trying to test the changes but the playground failed while building the project.

image_2023-10-19_14-17-37

@MohamedElmdary
Copy link
Member Author

MohamedElmdary commented Oct 19, 2023

I was trying to test the changes but the playground failed while building the project.

image_2023-10-19_14-17-37

Should be fixed was an issue related to types but didn't show in development mode for some reason

@mohamedamer453
Copy link
Contributor

While testing i found that all deployments that i had already deployed from playground.dev.grid.tf were loaded successfully when i ran the playground from this branch except for kubernetes, when i tried to list the k8s deployment it failed to do so, however the key was sucessfully updated as seen in the polkadot ui.

image_2023-10-19_15-36-17

and when i tried to deploy a k8s deployment directly from this branch it failed to load and it also got stuck at the Waiting for deployment screen.

image_2023-10-19_15-50-15

@MohamedElmdary
Copy link
Member Author

While testing i found that all deployments that i had already deployed from playground.dev.grid.tf were loaded successfully when i ran the playground from this branch except for kubernetes, when i tried to list the k8s deployment it failed to do so, however the key was sucessfully updated as seen in the polkadot ui.

image_2023-10-19_15-36-17

and when i tried to deploy a k8s deployment directly from this branch it failed to load and it also got stuck at the Waiting for deployment screen.

image_2023-10-19_15-50-15

We didn't apply any updates to k8s but by default k8s applying migration for it's contracts so that why it cases the issue
so I applied the exact solution from vms to k8s and it should be fixed now

@mohamedamer453
Copy link
Contributor

We didn't apply any updates to k8s but by default k8s applying migration for it's contracts so that why it cases the issue so I applied the exact solution from vms to k8s and it should be fixed now

Indeed, when i checked the instances that were deployed before they were listed without any issues and then i deployed a new instance and again it got listed without any issues.

image

@AhmedHanafy725 AhmedHanafy725 merged commit 89552dc into development_group_deployment_files Oct 22, 2023
@AhmedHanafy725 AhmedHanafy725 deleted the development_group_deployment_files_ui branch October 22, 2023 07:31
AhmedHanafy725 pushed a commit that referenced this pull request Oct 22, 2023
* Update listing path and add migration

* Fix migrating

* Fix migrating path

* Check storageType before apply exts and rename deploymentPath method

* Implement manage gateways in playground (#1215)

* Add dialog to deploy gateway

* Fix gateways listing & deploying path

* Add listing gateways

* Add list & deleting gateways

* Add dialog to micro vm

* Update project name in all solutions

* Update old contracts projectName

* Fix projectName path (add missing '/')

* Add missing 'no-data-text'

* Fix listing issue

* Merge ext1/ext2 to be applied together

* Remove contractId  check

* Fix typo backends to backend

* Add tls pass through option

* Update tooltip

* Remove unload (loaded with no contracts) deployments from listing

* Use domainName component instead of new template

* Add visit action in gateways table

* Migrate gateways from old solutions

* add prefix projectName + twinId + gatewayName to gateways deployed for fullvm & microvm

* Remove prefix while listing

* Fix building errors

* Fix issue in deploying and listing k8s
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
playground type_feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants