safe-global / safe-infrastructure

One `docker-compose.yml` file to rule them all
MIT License
58 stars 96 forks source link

PR for adding a first version of the Helm chart #125

Open wehappyfew opened 10 months ago

wehappyfew commented 10 months ago

At Holograph we need to use Safe on k8s, so here it is an initial version of the Helm chart :]

Note: I'm still facing issues, so any pointers would be greatly appreciated :]

github-actions[bot] commented 10 months ago

CLA Assistant Lite bot:
Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


Kostas Demiris seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request

wehappyfew commented 10 months ago

I have read the CLA Document and I hereby sign the CLA

Uxio0 commented 10 months ago

@wehappyfew Thank you very much for your contribution! Could you let us know what issues are you facing?

@luarx @diegopazosrego Could you take care of reviewing this PR?

wehappyfew commented 10 months ago

Hey @Uxio0 ! Happy to contribute :] So, the issues I'm facing are the following: [keep in mind that I'm not a blockchain dev, and I'm still figuring out the inner works of Safe architecture :) ]

  1. I had trouble using Safe UI [safe-wallet-web] as is, cause I needed it to listen on 0.0.0.0:3000 in order to be used with an ingress in k8s.

  2. When I access the Safe UI in my browser [Brave] I get this error in the console [even when I have set the NEXT_PUBLIC_WC_PROJECT_ID var]

Screenshot 2023-12-12 at 3 48 20 PM
  1. Also, I cannot access http://localhost:8000/cfg/media and http://localhost:8000/cgw/v1/chains
Screenshot 2023-12-12 at 4 41 30 PM
  1. I dont know If 2 is responsible, but no 'Connect wallet' button is responsive, and no error is thrown and the console [or in the pod] to give me pointers :/

  2. Note: most of the config vars across the infra have sample values

  3. Also, could you please validate if the way the nginx-shared-txs volume is correctly shared between the nginx and tcs-web deployments?

All in all, I tried to translate your docker-compose file by studying the safe-infrastructure repo. Please give me any pointers if anything is off :]

Thanks again!

wehappyfew commented 10 months ago

Also, when I try to access the Django admin dashboard for the transaction service, [after I run python manage.py createsuperuser] , I get the page with broken css...

eg http://localhost:58697/admin/history/

Screenshot 2023-12-15 at 3 43 23 PM Screenshot 2023-12-15 at 3 44 16 PM