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

Modify charts to represent the latest IOP configuration. #29

Open
redblom opened this issue Jun 8, 2023 · 7 comments · May be fixed by #30
Open

Modify charts to represent the latest IOP configuration. #29

redblom opened this issue Jun 8, 2023 · 7 comments · May be fixed by #30

Comments

@redblom
Copy link

redblom commented Jun 8, 2023

As discussed in the FDTF meetings:
The sciencemesh charts must be modified such that they lead to the deployment of 3 revad instances that are needed for IOP supported (OCM) federated sharing in the sciencemesh.
The configuration for these 3 instances can be found in the .toml files sciencemesh1.toml, sciencemesh2.toml, sciencemesh3.toml at https://github.com/cs3org/ocm-test-suite/tree/main/servers/revad.

@david-antos
Copy link
Contributor

Can we expect any reactions to this issue?

@david-antos
Copy link
Contributor

Well, can we expect any reactions in this century?

@dagl
Copy link
Collaborator

dagl commented Jun 14, 2023

I'll take a look at this

@david-antos
Copy link
Contributor

It would be highly appreciated.

@dagl dagl linked a pull request Jun 16, 2023 that will close this issue
@dagl
Copy link
Collaborator

dagl commented Jun 16, 2023

@david-antos I created a branch with modifications: https://github.com/sciencemesh/charts/tree/29-modify-charts-to-represent-the-latest-iop-configuration. After checkout you have to add the sciencemesh.toml files (configured for your instance) there and then it can be deployed like this:

helm repo update && helm dependency build && helm upgrade -i iop . -f values.yaml --set-file gateway.configFiles.revad\.toml=sciencemesh1.toml --set-file revad2.configFiles.revad\.toml=sciencemesh2.toml --set-file revad3.configFiles.revad\.toml=sciencemesh3.toml

Let me know if this works for you

@dagl
Copy link
Collaborator

dagl commented Jun 16, 2023

Might also need to use double "\\" instead of "\"

@marqsbla
Copy link

I used the corrected templates to deploy 3 revas. I still have problems with the deployment but I don't think it is related to the helm chargs.
What I can say, is that a bit problematic, at least for me, are different ports in *toml and the helm charts. I had to spend some time to deduce which service should connect with which, modify the ports accordingly, etcd.
Also, I believe revad3 do not open http port (only grpc). I couldn't disable http port in the helm chart definition (service, pod definition) without modifing the charts.

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 a pull request may close this issue.

4 participants