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

Integration for RGW host style bucket domains #1349

Closed
jpds opened this issue Dec 21, 2017 · 6 comments
Closed

Integration for RGW host style bucket domains #1349

jpds opened this issue Dec 21, 2017 · 6 comments
Assignees
Milestone

Comments

@jpds
Copy link

jpds commented Dec 21, 2017

Is this a bug report or feature request?

  • Feature Request

Feature Request

Are there any similar features already existing: No

What should the feature do: AWS has a concept of a "host style bucket" domains, where clients can be configured to go to:

host_bucket = %(bucket)s.s3.amazonaws.com

Rook could potential implement this with the help of a wildcard DNS entry which goes to a nginx/other ingress:

https://www.hastexo.com/resources/hints-and-kinks/configuring-radosgw-behave-amazon-s3/

It would seem that the RGW just needs to have a way of setting:

rgw dns name = domain.name

What would be solved through this feature: Some S3 clients expect this integration.

Does this have an impact on existing features: N/A.

@jpds jpds changed the title Integration for host style bucket domains Integration for RGW host style bucket domains Dec 21, 2017
@jpds
Copy link
Author

jpds commented Dec 21, 2017

Seems Kubernetes does not support wildcard ingress definitions (kubernetes/kubernetes#41881). For now, I'd be happy to have separate ingress definitions per bucket name which point to the same RGW instance (while using wildcard DNS).

@jpds
Copy link
Author

jpds commented Dec 21, 2017

I came across this while trying to find a fix for the issue I have in hashicorp/vault#3728 .

@jpds
Copy link
Author

jpds commented Dec 21, 2017

Potentially already implemented:

"rgw dns name": config.Host,

Prehaps Host needs to be added to the YAML/documentation?

@travisn
Copy link
Member

travisn commented Jan 2, 2018

agreed, it seems we need to expose the dns name to the object.yaml.

@travisn travisn self-assigned this Jan 25, 2018
@travisn travisn added this to To Do in v0.7 via automation Jan 25, 2018
@travisn travisn added this to the 0.7 milestone Jan 25, 2018
@travisn travisn moved this from To Do to In Progress in v0.7 Jan 25, 2018
@travisn travisn moved this from In Progress to In Review in v0.7 Jan 25, 2018
@travisn
Copy link
Member

travisn commented Jan 25, 2018

Right now the rgw dns name is set to the name of the rgw service. The side effect of this is that the hostname in the URL of s3 client requests must match that name. For now, setting the default to be blank is the right thing until it's more clear how to use the bucket names with the wildcard.

@jpds do you have any more insight into how we can configure k8s to get this to work?

@travisn travisn moved this from In Review to To Do in v0.7 Feb 1, 2018
@jbw976 jbw976 removed this from To Do in v0.7 Feb 12, 2018
@travisn
Copy link
Member

travisn commented Feb 26, 2018

@jpds please reactivate if #1437 was not sufficient

@travisn travisn closed this as completed Feb 26, 2018
@jbw976 jbw976 modified the milestones: 0.7.5, 0.8 Feb 28, 2018
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.

3 participants