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 kube namespace override and default to in cluster namespace #622

Merged

Conversation

chrisdoherty4
Copy link
Member

@chrisdoherty4 chrisdoherty4 commented May 13, 2022

See tinkerbell/hegel#106 for explanation.

@chrisdoherty4 chrisdoherty4 changed the title Add kube namespace override and default to incluster namespace. Add kube namespace override and default to in cluster namespace May 13, 2022
@codecov
Copy link

codecov bot commented May 13, 2022

Codecov Report

Merging #622 (bcd72d3) into main (7a8ca47) will decrease coverage by 0.12%.
The diff coverage is 0.00%.

❗ Current head bcd72d3 differs from pull request most recent head 9b173f8. Consider uploading reports for the commit 9b173f8 to get more accurate results

@@            Coverage Diff             @@
##             main     #622      +/-   ##
==========================================
- Coverage   44.65%   44.52%   -0.13%     
==========================================
  Files          61       61              
  Lines        3509     3519      +10     
==========================================
  Hits         1567     1567              
- Misses       1860     1870      +10     
  Partials       82       82              
Impacted Files Coverage Δ
server/kubernetes_api.go 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7a8ca47...9b173f8. Read the comment docs.

Copy link
Contributor

@micahhausler micahhausler left a comment

Choose a reason for hiding this comment

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

LGTM

@chrisdoherty4 chrisdoherty4 added the ready-to-merge Signal to Mergify to merge the PR. label May 16, 2022
@mergify mergify bot merged commit 7b60369 into tinkerbell:main May 17, 2022
@displague displague added this to the 0.7.0 milestone Aug 15, 2022
mergify bot added a commit that referenced this pull request Aug 29, 2022
I'm a maintainer of several other services often related to the Kuberenetes back-end/Kubernetes controllers and I'm taking ownership for a lot of release synchronization making it both appropriate and necessary for me to maintain aspects of the Tink repository.

Requirements:

- I have reviewed the [community membership guidelines](https://github.com/tinkerbell/proposals/blob/main/proposals/0024/GOVERNANCE.md)
- I have [enabled 2FA on my GitHub account](https://github.com/settings/security)
- I have subscribed to the [tinkerbell-contributors e-mail list](https://groups.google.com/g/tinkerbell-contributors)
- I am actively contributing to 1 or more Tinkerbell subprojects

Here is a list of non-trival PRs I have been the primary reviewer on:

#596
#628
#614

I have also made a number of code contributions to this repository, here are a few of them:

#638
#631
#626
#622
#612

I have also raised various issues and am driving the releasing across Tinkerbell including in this repository.

Requesting consideration of expedited responsibilities: yes

Sponsors:

- @mmlb (Maintainer)
- @micahhausler (Maintainer)
- @jacobweinstock (Core contributor in other Tinkerbell repositories)
@chrisdoherty4 chrisdoherty4 deleted the feature/add-kube-namespace-override branch May 15, 2023 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Signal to Mergify to merge the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants