This message was deleted.
# opal
s
This message was deleted.
o
HI @Avo Sepp - did you see this part in the docs? https://docs.opal.ac/getting-started/running-opal/run-opal-server/policy-repo-location#optional-ssh-key-for-private-repos Or do you mean something else?
a
dude how did I miss that… thank you!
💜 1
OHH I was under the tutorials section. For “Track a Git Repo”.
I think if I were to suggest a change, Track a Git Repo should have some link to the page you linked. I think I was trapped in the tutorial land and didn’t know there was more documentation about configuring Git access itself.
Or like “This page is a continuation of Policy Repo Location” or something.
o
I can see how that can get missed, I’ll add a tip bubble on the top of track git repo Thanks for the feedback! CC: @Filip , @Asaf Cohen , @Shaul Kremer
a
I notice these ENV variables are not configurable with the Helm values.yaml. Is there a good reason for this? Or can I go ahead and submit a PR to add more ENV options?
Clarification. Some of them are. But SSH key is not one.
a
Hey @Avo Sepp PRs are welcome 🙂 cc @Raz Co who can review your helm changes
permit 1
👍 1
o
@Avo Sepp Suggested improvement is live in the docs: https://docs.opal.ac/tutorials/track_a_git_repo/ Thanks again
🙌 1
a
I have finished my commit. I can’t push a branch to Permit’s opal-helm. Is there a dev fork or should I fork to my own repo and do a PR from there?
o
Yep, fork and PR is the way to go 😉 Thank you!
a
o
I believe @Raz Co will be able to review/merge tomorrow
a
Sounds good! Thanks @Or Weis and @Raz Co!
r
Looks great @Avo Sepp ! I’ll review and test it tomorrow :)
🎉 2
Hey @Avo Sepp, I reviewed your PR - and it looks great, thank you for the contribution ! Merged 😉
o
@Daniel Bass please hookup our new contributor with some OPAL / Permit swag CC: @Filip
🙌 1
d
On it! @Filip will reach out to you @Avo Sepp 🌸
r
You can get the new release here - https://github.com/permitio/opal-helm-chart/releases/tag/0.0.6 LMK if its working well for you.
a
Thank you everyone! I feel most honored.
❤️ 1