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

add arbitrary host mount list #36

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

add arbitrary host mount list #36

wants to merge 2 commits into from

Conversation

gilesknap
Copy link
Member

@marcelldls please can you validate this for me - I don't think I got the json schema change right.

We need this to mount in the Andor SDK for the the Andor3. At least that's what I"m hoping to get permission to do from the cloud team.

@MaxHerbs FYI

@gilesknap
Copy link
Member Author

@marcelldls Now I have just realised that I'd also like to mount arbitrary PVCs too.

So should we add a mountPVCs list into values.yaml, or should we just allow free form entries in the manifests 'volumes' and 'volumeMount' sections (i.e. like we did for tolerations)?

@marcelldls
Copy link
Contributor

@gilesknap I think whats different compared to tolerations is that for tolerations we dont have any defaults. I dont really like the idea of making a custom key... Im not sure if theres an easy way but I would hope we could use the standard 'volumes' and 'volumeMount' and join it with whats in the template

@gilesknap
Copy link
Member Author

I was just adding another custom key. Would you like to have a go at doing your approach? I think it makes more sense than more custom keys.

@gilesknap
Copy link
Member Author

I have a use for this today - want to try making the kernel driver for the andor camera work.

@marcelldls
Copy link
Contributor

I would like to have a go, possibly next week? Would be useful for other entries too. Also you have reminded me that the json schema for the argo charts is outdated now...

@marcelldls
Copy link
Contributor

I would like to have a go, possibly next week? Would be useful for other entries too. Also you have reminded me that the json schema for the argo charts is outdated now...

For example, resources

@gilesknap
Copy link
Member Author

OK that sounds good. I'm going to make a custom key for now so I can get Max's project working today.

But we can delete this PR once you have made your changes.

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