Skip to content

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

Open
wehappyfew wants to merge 1 commit into
safe-global:mainfrom
wehappyfew:feat/safe-helm-chart
Open

PR for adding a first version of the Helm chart#125
wehappyfew wants to merge 1 commit into
safe-global:mainfrom
wehappyfew:feat/safe-helm-chart

Conversation

@wehappyfew

@wehappyfew wehappyfew commented Dec 11, 2023

Copy link
Copy Markdown

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 :]

@wehappyfew wehappyfew requested review from a team, Uxio0, fmrsabino and iamacook and removed request for a team December 11, 2023 12:04
@github-actions

Copy link
Copy Markdown

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

Copy link
Copy Markdown
Author

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

@Uxio0

Uxio0 commented Dec 11, 2023

Copy link
Copy Markdown
Member

@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

wehappyfew commented Dec 12, 2023

Copy link
Copy Markdown
Author

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

Copy link
Copy Markdown
Author

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

@gatleas17 gatleas17 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants