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
sidecar/compact/store/receiver - Add the prefix option to buckets #5337
Commits on Jun 9, 2022
-
Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
Signed-off-by: Maria Eduarda Duarte <dudammduarte@yahoo.com.br>
-
Signed-off-by: Maria Eduarda Duarte <dudammduarte@yahoo.com.br>
-
Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
Test more prefix cases on PrefixedBucket
Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
Only use a prefixedbucket if we have a valid prefix
Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
Add single unit test for prefixedBucket prefix
Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
test other prefixes on UsesPrefixTest
Signed-off-by: Maria Eduarda Duarte <dudammduarte@yahoo.com.br>
-
add remaining methods to UsesPrefixTest
Signed-off-by: Maria Eduarda Duarte <dudammduarte@yahoo.com.br>
-
Signed-off-by: Maria Eduarda Duarte <dudammduarte@yahoo.com.br>
-
Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
add prefix explanation to S3 docs
Signed-off-by: Maria Eduarda Duarte <dudammduarte@yahoo.com.br>
-
Conclusion of prefix sentence on docs
Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
Use DirDelim instead of magic string
Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
Add log when using prefixed bucket
Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
Remove "@" from test string to make them simpler
Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
fix BucketConfig Config type - back to interface
Signed-off-by: Maria Eduarda Duarte <dudammduarte@yahoo.com.br>
-
Signed-off-by: Maria Eduarda Duarte <dudammduarte@yahoo.com.br>
-
add missing checks in UsesPrefixTest
Signed-off-by: Maria Eduarda Duarte <dudammduarte@yahoo.com.br>
-
Signed-off-by: Maria Eduarda Duarte <dudammduarte@yahoo.com.br>
-
Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
Remove duplicated transformation of string->[]byte
Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
Add prefixed bucket on all e2e tests for S3
The idea is that if it works, we can add for all other providers. Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
Add e2e tests using prefixed bucket to all providers
Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
refactor: move validPrefix to prefixed_bucket logic
Signed-off-by: Maria Eduarda Duarte <dudammduarte@yahoo.com.br>
-
Enhance the documentation about prefix.
Signed-off-by: jademcosta <jademcosta@gmail.com>
-
Signed-off-by: jademcosta <jademcosta@gmail.com>
-
Add prefix entry on bucket config example
Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
Removing redundancies on prefix checks and tests
We already check if the prefix if not empty when creating the bucket. Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
Remove redundant YAML unmarshal
Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
Remove docs that should be auto-geneated
Signed-off-by: jademcosta <jade.costa@nubank.com.br>
-
refactor: move prefix to config root level
Signed-off-by: Maria Eduarda Duarte <dudammduarte@yahoo.com.br>
-
Signed-off-by: Maria Eduarda Duarte <dudammduarte@yahoo.com.br>
-
Signed-off-by: Maria Eduarda Duarte <dudammduarte@yahoo.com.br>