search

threefoldtech / tfgrid-sdk-ts

Apache License 2.0
4 stars 8 forks source link

APP: Gitea updated & Mycelium inside #2015

Closed xmonader closed 1 month ago

Mahmoud-Emad commented 5 months ago

Blocked on https://github.com/threefoldtech/tf-images/issues/206

zaelgohary commented 3 months ago

@maxux Can you promote this too? https://hub.grid.tf/petep.3bot/gitea_mycelium.flist.md

zaelgohary commented 3 months ago

@ehab-hassan This too needs a logo, please.

zaelgohary commented 3 months ago

WIP:

Added the solution but didn't test it. Currently working on adding the env vars needed and testing.

zaelgohary commented 3 months ago

WIP:

Added Gitea to the deployment list. Also, removed the env tab and added mail server. However, when I tried deploying and log the services, one of them was in a state error. The issue is blocked till the flist is fixed.

maxux commented 3 months ago

Promoted: https://hub.grid.tf/tf-official-apps/gitea_mycelium.flist.md

zaelgohary commented 3 months ago

Work completed:

Edited flist.

ehab-hassan commented 3 months ago

@zaelgohary logo white gitea

khaledyoussef24 commented 3 months ago

tested on devnet version : db2daf4 Users now can deploy a gittea solution from the dashboard. but still need a domain to visit after creating the vm and needing also to add an icon to the deployment list to visit the domain and i think it's type of solutions which will have a control panel. but the machine worked fine and the status of the tasks is running

Image Image Image Image

Mik-TF commented 2 months ago

I agree with @khaledyoussef24 here: I think we could add the possibility for the domain + Visit button to access the Gitea instance. It would be on par with the other solutions we present on the Dashboard. Also, wireguard network isn't there.

Feature Requests

Can we have these updates now or would they come at a later release? Either way is fine, I ask to coordinate with the info_grid docs.

Mik-TF commented 2 months ago

image

Does this mean we cannot run Gitea without using IPv4 network?

zaelgohary commented 2 months ago

Work completed:

Added requested changes.

zaelgohary commented 2 months ago

@Mik-TF You can go on with the docs. Same goes for Nostr.

Mik-TF commented 2 months ago

Good! Will be done this week.

khaledyoussef24 commented 2 months ago

verified and worked fine network used : devnet version : ac03039

Image Image Image Image Image Image Image Image

testcase

Mik-TF commented 2 months ago

@khaledyoussef24 one thing is not clear.

image

It says SMTP is needed, and SMTP needs IPv4.

But I deployed a gitea instance without IPv4, and it still works. But I guess SMTP isn't set up.

Should we set IPv4 as true by default (and required) for the gitea app? Then SMTP will always work. Otherwise, people will set Gitea instances without Ipv4 and then the instance won't work fully as they will not be able to receive emails.

Or perhaps when we don't set IPv4, it takes the IPv4 from the gateway? In that case, the warning (see screenshot above) is misleading.

Wdyt?

Mik-TF commented 2 months ago

Work done on info_grid's side.

zaelgohary commented 2 months ago

Work done on info_grid's side.

Gitea docs is returning 404.

image

As for the SMTP warning, I'll take a look into it.

Mik-TF commented 2 months ago

Of course, it's not yet merged: ongoing PR: https://github.com/threefoldtech/info_grid/pull/592

Great for the SMTP. Thanks!