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

YUNIKORN-158. Admission controller deployment file should use the same image version as the scheduler #117

Merged
merged 2 commits into from
May 22, 2020

Conversation

yangwwei
Copy link
Contributor

Fix the hardcoded value and make sure the version is always aligned

@yangwwei yangwwei self-assigned this May 14, 2020
@yangwwei
Copy link
Contributor Author

@TaoYang526 , @sunilgovind could you help to review this?

Makefile Outdated Show resolved Hide resolved
Makefile Show resolved Hide resolved
@codecov-commenter
Copy link

Codecov Report

Merging #117 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #117   +/-   ##
=======================================
  Coverage   54.71%   54.71%           
=======================================
  Files          29       30    +1     
  Lines        2811     2811           
=======================================
  Hits         1538     1538           
  Misses       1215     1215           
  Partials       58       58           
Impacted Files Coverage Δ
pkg/common/si_helper.go 34.56% <0.00%> (ø)
pkg/common/resource.go 100.00% <0.00%> (+31.54%) ⬆️

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 b8a4a01...dad1d9f. Read the comment docs.

Copy link
Contributor

@TaoYang526 TaoYang526 left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

@TaoYang526 TaoYang526 merged commit ce2af05 into apache:master May 22, 2020
wilfred-s added a commit that referenced this pull request May 22, 2020
…e the same image version as the scheduler (#117)"

This reverts commit ce2af05.
wilfred-s added a commit that referenced this pull request May 22, 2020
The change does not fix the image for the scheduler and causes the make
to complain about unknown settings.

This reverts commit ce2af05.
@yangwwei yangwwei deleted the YUNIKORN-158 branch May 22, 2020 20:52
ayubpathan pushed a commit to ayubpathan/incubator-yunikorn-k8shim that referenced this pull request Jun 10, 2020
ayubpathan pushed a commit to ayubpathan/incubator-yunikorn-k8shim that referenced this pull request Jun 10, 2020
The change does not fix the image for the scheduler and causes the make
to complain about unknown settings.

This reverts commit ce2af05.
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 this pull request may close these issues.

3 participants