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

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

wehappyfew
Copy link

@wehappyfew wehappyfew commented Dec 11, 2023

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 a review from a team as a code owner December 11, 2023 12:04
@wehappyfew wehappyfew requested review from fmrsabino, Uxio0 and iamacook and removed request for a team December 11, 2023 12:04
Copy link

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
Author

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

@Uxio0
Copy link
Member

Uxio0 commented Dec 11, 2023

@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
Copy link
Author

wehappyfew commented Dec 12, 2023

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

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.

2 participants