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

Backup names do not include metadata.name #253

Closed
ballad89 opened this issue Mar 14, 2019 · 7 comments
Closed

Backup names do not include metadata.name #253

ballad89 opened this issue Mar 14, 2019 · 7 comments
Milestone

Comments

@ballad89
Copy link
Contributor

apiVersion: mysql.presslabs.org/v1alpha1
kind: MysqlBackup
metadata:
  namespace: test-namespace
  name: baseline
spec:
  clusterName: mysql-cluster

Currently the above backup would be named mysql-cluster-<date>... on s3.
I think the name of the backup should be included in the name such as mysql-cluster-baseline-<date>... as it would provide a way to identify and differentiate backups.
Or it can even be called baseline-<date>... as that is the name specified.
In summary, a way to control what my snapshots are called would be nice

@AMecea
Copy link
Contributor

AMecea commented Mar 14, 2019

Hi @ballad89, thanks for opening this issue, I agree with you, I think that the backup on remote should be called baseline-<date>....

I'd be happy to review and merge a PR that does that. 😄

Here is how bucket URL is composed.

@AMecea AMecea added this to the 0.2.x milestone Mar 14, 2019
@ballad89
Copy link
Contributor Author

Cool, not a problem. Could you just walk me through how you setup and test locally ? thanks

@AMecea
Copy link
Contributor

AMecea commented Mar 15, 2019

Sure,

EDIT: In order to get the project working locally you had to install kubebuilder.

After cloning the repo in your go path (~/go/src/github.com/presslabs/mysql-operator) run:

# to install ginkgo, golangci-linter, helm, yq  (in local bin folder of the project)
make dependencies-local

# run unit and integration tests
make

# run linter
make lint

If you want to test it in minikube or on a remote cluster, you can use skaffold

# connects to your cluster and deploys an instance of mysql-operator. (make sure that no other release of the operator is running)
make skaffold-run

Note: skaffold will fail to publish images so you have to adapt the skaffold.yaml file to not publish the images if you are using minikube.

Hope this helps! Please let me know if something does not work or if you have any questions.

@ballad89
Copy link
Contributor Author

PR submitted
#257

In order to get the project working locally I had to install kubebuilder globally https://book.kubebuilder.io/getting_started/installation_and_setup.html#

@AMecea
Copy link
Contributor

AMecea commented Mar 18, 2019

Addressed by #257

@AMecea AMecea closed this as completed Mar 18, 2019
@ballad89
Copy link
Contributor Author

Any release scheduled for soon ?

@AMecea
Copy link
Contributor

AMecea commented Mar 20, 2019

We want to do a new release every two weeks. So the next version (0.2.7) is planned to be released on Monday.

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

No branches or pull requests

2 participants