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

AsOwner no longer things the owner refers to a controller. It refers to the ArangoDeployment #81

Merged
merged 2 commits into from
Mar 27, 2018

Conversation

ewoutp
Copy link
Contributor

@ewoutp ewoutp commented Mar 27, 2018

This was incorrect anyway.
Fixing it makes the pods show up properly in the k8s dashboard.

Copy link
Member

@neunhoef neunhoef left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. But: I have no idea what consequences this change has. For knowledge transfer reasons, I believe I should understand this.

@ewoutp
Copy link
Contributor Author

ewoutp commented Mar 27, 2018

When you create a resource in k8s, you can attach it to an owner.
That way when you delete the owner, the things attached to it are also deleted.

The AsOwner function creates a proper owner reference. The problem was that it did set the Controller field which (according to the docs) means that the reference was to the controller (so our operator) instead of the ArangoDeployment itself. This was wrong, because the intent was to attach to the ArangoDeployment, not its controller.

@ewoutp ewoutp merged commit d981d31 into master Mar 27, 2018
@ewoutp ewoutp deleted the as-owner-fix branch March 27, 2018 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants