[GitHub] [yunikorn-release] craigcondit closed pull request #124: [YUNIKORN-1540] Fix default placeholder image in documentation

2023-01-18 Thread GitBox
craigcondit closed pull request #124: [YUNIKORN-1540] Fix default placeholder image in documentation URL: https://github.com/apache/yunikorn-release/pull/124 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [yunikorn-release] craigcondit opened a new pull request, #124: [YUNIKORN-1540] Fix default placeholder image in documentation

2023-01-18 Thread GitBox
craigcondit opened a new pull request, #124: URL: https://github.com/apache/yunikorn-release/pull/124 JIRA: https://issues.apache.org/jira/browse/YUNIKORN-1540 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [yunikorn-release] yuchaoran2011 merged pull request #123: [YUNIKORN-1495]Update the copyright years in NOTICE files to 2023

2023-01-14 Thread GitBox
yuchaoran2011 merged PR #123: URL: https://github.com/apache/yunikorn-release/pull/123 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [yunikorn-release] HuangTing-Yao opened a new pull request, #123: [YUNIKORN-1495]Update the copyright years in NOTICE files to 2023

2023-01-14 Thread GitBox
HuangTing-Yao opened a new pull request, #123: URL: https://github.com/apache/yunikorn-release/pull/123 update NOTICE files in root fodler and top-level-artifacts folder https://issues.apache.org/jira/browse/YUNIKORN-1495 -- This is an automated message from the Apache Git Service. To

[GitHub] [yunikorn-release] wilfred-s closed pull request #122: [YUNIKORN-1501] Update helm chart README.md to reflect K8s 1.26 support

2022-12-22 Thread GitBox
wilfred-s closed pull request #122: [YUNIKORN-1501] Update helm chart README.md to reflect K8s 1.26 support URL: https://github.com/apache/yunikorn-release/pull/122 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [yunikorn-release] craigcondit opened a new pull request, #122: [YUNIKORN-1501] Update helm chart README.md to reflect K8s 1.26 support

2022-12-22 Thread GitBox
craigcondit opened a new pull request, #122: URL: https://github.com/apache/yunikorn-release/pull/122 JIRA: https://issues.apache.org/jira/browse/YUNIKORN-1500 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [yunikorn-release] craigcondit closed pull request #121: [YUNIKORN-1474] Add LF to the end of Chart.yaml

2022-12-15 Thread GitBox
craigcondit closed pull request #121: [YUNIKORN-1474] Add LF to the end of Chart.yaml URL: https://github.com/apache/yunikorn-release/pull/121 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [yunikorn-release] craigcondit closed pull request #120: [YUNIKORN-1470] Helm: Add permission to read priority classes

2022-12-12 Thread GitBox
craigcondit closed pull request #120: [YUNIKORN-1470] Helm: Add permission to read priority classes URL: https://github.com/apache/yunikorn-release/pull/120 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [yunikorn-release] craigcondit opened a new pull request, #120: [YUNIKORN-1470] Helm: Add permission to read priority classes

2022-12-12 Thread GitBox
craigcondit opened a new pull request, #120: URL: https://github.com/apache/yunikorn-release/pull/120 JIRA: https://issues.apache.org/jira/browse/YUNIKORN-1470 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [yunikorn-release] elihschiff commented on a diff in pull request #116: [YUNIKORN-1371] Migrate to new ConfigMap configuration

2022-11-29 Thread GitBox
elihschiff commented on code in PR #116: URL: https://github.com/apache/yunikorn-release/pull/116#discussion_r1035312940 ## helm-charts/yunikorn/templates/deployment.yaml: ## @@ -24,7 +24,7 @@ metadata: release: {{ .Release.Name }} heritage: {{ .Release.Service }}

[GitHub] [yunikorn-release] craigcondit commented on a diff in pull request #116: [YUNIKORN-1371] Migrate to new ConfigMap configuration

2022-11-29 Thread GitBox
craigcondit commented on code in PR #116: URL: https://github.com/apache/yunikorn-release/pull/116#discussion_r1035309845 ## helm-charts/yunikorn/templates/deployment.yaml: ## @@ -24,7 +24,7 @@ metadata: release: {{ .Release.Name }} heritage: {{ .Release.Service }}

[GitHub] [yunikorn-release] elihschiff commented on a diff in pull request #116: [YUNIKORN-1371] Migrate to new ConfigMap configuration

2022-11-29 Thread GitBox
elihschiff commented on code in PR #116: URL: https://github.com/apache/yunikorn-release/pull/116#discussion_r1035307098 ## helm-charts/yunikorn/templates/deployment.yaml: ## @@ -24,7 +24,7 @@ metadata: release: {{ .Release.Name }} heritage: {{ .Release.Service }}

[GitHub] [yunikorn-release] craigcondit closed pull request #119: [YUNIKORN-1429] Fix syntax ingress error

2022-11-29 Thread GitBox
craigcondit closed pull request #119: [YUNIKORN-1429] Fix syntax ingress error URL: https://github.com/apache/yunikorn-release/pull/119 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [yunikorn-release] yuchaoran2011 merged pull request #118: [YUNIKORN-1429] Add ingress for Web UI

2022-11-28 Thread GitBox
yuchaoran2011 merged PR #118: URL: https://github.com/apache/yunikorn-release/pull/118 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [yunikorn-release] marload commented on a diff in pull request #118: [YUNIKORN-1429] Add ingress for Web UI

2022-11-28 Thread GitBox
marload commented on code in PR #118: URL: https://github.com/apache/yunikorn-release/pull/118#discussion_r1034286297 ## helm-charts/yunikorn/templates/ingress.yaml: ## @@ -0,0 +1,62 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license

[GitHub] [yunikorn-release] wilfred-s closed pull request #117: [YUNIKORN-1171] update .asf.yaml with new options

2022-11-28 Thread GitBox
wilfred-s closed pull request #117: [YUNIKORN-1171] update .asf.yaml with new options URL: https://github.com/apache/yunikorn-release/pull/117 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [yunikorn-release] craigcondit commented on a diff in pull request #118: [YUNIKORN-1429] Add ingress for Web UI

2022-11-28 Thread GitBox
craigcondit commented on code in PR #118: URL: https://github.com/apache/yunikorn-release/pull/118#discussion_r1034147274 ## helm-charts/yunikorn/templates/ingress.yaml: ## @@ -0,0 +1,62 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [yunikorn-release] yuchaoran2011 commented on a diff in pull request #118: [YUNIKORN-1429] Add ingress for Web UI

2022-11-27 Thread GitBox
yuchaoran2011 commented on code in PR #118: URL: https://github.com/apache/yunikorn-release/pull/118#discussion_r1033186970 ## helm-charts/yunikorn/templates/ingress.yaml: ## @@ -0,0 +1,62 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [yunikorn-release] marload opened a new pull request, #118: [YUNIKORN-1429] Add ingress for Web UI

2022-11-25 Thread GitBox
marload opened a new pull request, #118: URL: https://github.com/apache/yunikorn-release/pull/118 Ingress is required for access from outside the cluster. Create an ingress resource. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [yunikorn-release] wilfred-s opened a new pull request, #117: [YUNIKORN-1171] update .asf.yaml with new options

2022-11-24 Thread GitBox
wilfred-s opened a new pull request, #117: URL: https://github.com/apache/yunikorn-release/pull/117 Use the same settings for all code repositories. Turn off wiki on all repos. Remove incubating from the description. -- This is an automated message from the Apache Git Service. To

[GitHub] [yunikorn-release] craigcondit commented on pull request #110: [YUNIKORN-1261] Add pre-upgrade hook

2022-11-18 Thread GitBox
craigcondit commented on PR #110: URL: https://github.com/apache/yunikorn-release/pull/110#issuecomment-1320268836 Closing this as we resolved in a different way in https://issues.apache.org/jira/browse/YUNIKORN-1221. -- This is an automated message from the Apache Git Service. To

[GitHub] [yunikorn-release] craigcondit closed pull request #110: [YUNIKORN-1261] Add pre-upgrade hook

2022-11-18 Thread GitBox
craigcondit closed pull request #110: [YUNIKORN-1261] Add pre-upgrade hook URL: https://github.com/apache/yunikorn-release/pull/110 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [yunikorn-release] craigcondit closed pull request #111: [YUNIKORN-1359] Bring-your-own ConfigMap

2022-11-17 Thread GitBox
craigcondit closed pull request #111: [YUNIKORN-1359] Bring-your-own ConfigMap URL: https://github.com/apache/yunikorn-release/pull/111 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [yunikorn-release] craigcondit commented on pull request #111: [YUNIKORN-1359] Bring-your-own ConfigMap

2022-11-17 Thread GitBox
craigcondit commented on PR #111: URL: https://github.com/apache/yunikorn-release/pull/111#issuecomment-1319332412 Closing in favor of the implementation done as part of https://issues.apache.org/jira/browse/YUNIKORN-1221. -- This is an automated message from the Apache Git Service. To

[GitHub] [yunikorn-release] craigcondit closed pull request #116: [YUNIKORN-1371] Migrate to new ConfigMap configuration

2022-11-17 Thread GitBox
craigcondit closed pull request #116: [YUNIKORN-1371] Migrate to new ConfigMap configuration URL: https://github.com/apache/yunikorn-release/pull/116 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [yunikorn-release] craigcondit commented on pull request #113: [YUNIKORN-1376] update values and deployment helm chart of the admission controller

2022-11-17 Thread GitBox
craigcondit commented on PR #113: URL: https://github.com/apache/yunikorn-release/pull/113#issuecomment-1318829225 Closing in favor of #116 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [yunikorn-release] craigcondit closed pull request #113: [YUNIKORN-1376] update values and deployment helm chart of the admission controller

2022-11-17 Thread GitBox
craigcondit closed pull request #113: [YUNIKORN-1376] update values and deployment helm chart of the admission controller URL: https://github.com/apache/yunikorn-release/pull/113 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [yunikorn-release] craigcondit commented on pull request #115: [YUNIKORN-1367] Don't mount configmap into scheduler pod

2022-11-17 Thread GitBox
craigcondit commented on PR #115: URL: https://github.com/apache/yunikorn-release/pull/115#issuecomment-1318802150 Closed in favor of #116. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [yunikorn-release] craigcondit opened a new pull request, #116: [YUNIKORN-1371] Migrate to new ConfigMap configuration

2022-11-17 Thread GitBox
craigcondit opened a new pull request, #116: URL: https://github.com/apache/yunikorn-release/pull/116 Updates Helm charts to deploy new yunikorn-defaults ConfigMap, and removes legacy configuration environment variables. JIRA: https://issues.apache.org/jira/browse/YUNIKORN-1371 --

[GitHub] [yunikorn-release] craigcondit closed pull request #115: [YUNIKORN-1367] Don't mount configmap into scheduler pod

2022-11-17 Thread GitBox
craigcondit closed pull request #115: [YUNIKORN-1367] Don't mount configmap into scheduler pod URL: https://github.com/apache/yunikorn-release/pull/115 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [yunikorn-release] craigcondit opened a new pull request, #115: [YUNIKORN-1367] Don't mount configmap into scheduler pod

2022-11-15 Thread GitBox
craigcondit opened a new pull request, #115: URL: https://github.com/apache/yunikorn-release/pull/115 JIRA: https://issues.apache.org/jira/browse/YUNIKORN-1367 Now that https://issues.apache.org/jira/browse/YUNIKORN-1365 has landed, we no longer need to mount the configmap into the

[GitHub] [yunikorn-release] craigcondit closed pull request #112: [YUNIKORN-1334] Admission controller: look up user & group and add annotation if necessary

2022-11-14 Thread GitBox
craigcondit closed pull request #112: [YUNIKORN-1334] Admission controller: look up user & group and add annotation if necessary URL: https://github.com/apache/yunikorn-release/pull/112 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [yunikorn-release] craigcondit closed pull request #114: [YUNIKORN-1349] [docs] Document support for K8s 1.21-1.25

2022-11-14 Thread GitBox
craigcondit closed pull request #114: [YUNIKORN-1349] [docs] Document support for K8s 1.21-1.25 URL: https://github.com/apache/yunikorn-release/pull/114 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [yunikorn-release] pbacsko commented on pull request #112: [YUNIKORN-1334] Admission controller: look up user & group and add annotation if necessary

2022-11-14 Thread GitBox
pbacsko commented on PR #112: URL: https://github.com/apache/yunikorn-release/pull/112#issuecomment-1313848000 Note: this change has to go in order to stabilize the e2e tests for YUNIKORN-1334. -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [yunikorn-release] wilfred-s commented on pull request #113: [YUNIKORN-1376] update values and deployment helm chart of the admission controller

2022-11-10 Thread GitBox
wilfred-s commented on PR #113: URL: https://github.com/apache/yunikorn-release/pull/113#issuecomment-1311060942 If the delay of moving to the configmap is limited and manageable then that is surely preferred. We should try to prevent introducing variables and in the same release cycle

[GitHub] [yunikorn-release] craigcondit commented on pull request #113: [YUNIKORN-1376] update values and deployment helm chart of the admission controller

2022-11-10 Thread GitBox
craigcondit commented on PR #113: URL: https://github.com/apache/yunikorn-release/pull/113#issuecomment-1311042464 Can we use the configmap for this instead? I know that puts us behind a bit, but I'd rather not introduce even more admission controller env vars... -- This is an automated

[GitHub] [yunikorn-release] craigcondit commented on pull request #111: [YUNIKORN-1359] Bring-your-own ConfigMap

2022-11-03 Thread GitBox
craigcondit commented on PR #111: URL: https://github.com/apache/yunikorn-release/pull/111#issuecomment-1302530920 This is looking better, but I think we should wait until at least https://issues.apache.org/jira/browse/YUNIKORN-1365 has landed, as this will allow the scheduler to start

[GitHub] [yunikorn-release] DerekTBrown commented on pull request #111: [YUNIKORN-1359] Bring-your-own ConfigMap

2022-11-03 Thread GitBox
DerekTBrown commented on PR #111: URL: https://github.com/apache/yunikorn-release/pull/111#issuecomment-1302518514 @craigcondit I have made the requested changes. Can you take another look? -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [yunikorn-release] craigcondit commented on pull request #111: [YUNIKORN-1359] Bring-your-own ConfigMap

2022-11-01 Thread GitBox
craigcondit commented on PR #111: URL: https://github.com/apache/yunikorn-release/pull/111#issuecomment-1299455530 @DerekTBrown, this is going to be incompatible with the configuration redesign that is in progress. I've updated the original JIR A you've opened, and have added my comments

[GitHub] [yunikorn-release] DerekTBrown commented on pull request #111: [YUNIKORN-1359] Bring-your-own ConfigMap

2022-11-01 Thread GitBox
DerekTBrown commented on PR #111: URL: https://github.com/apache/yunikorn-release/pull/111#issuecomment-1299418145 @wilfred-s could you take a look at this? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [yunikorn-release] DerekTBrown commented on pull request #111: [YUNIKORN-1359] Bring-your-own ConfigMap

2022-10-18 Thread GitBox
DerekTBrown commented on PR #111: URL: https://github.com/apache/yunikorn-release/pull/111#issuecomment-1283049530 > Wouldn't the real solution then be to allow changing the config map in the deployment? Combined with the current possibility of changing the yaml file you would have full

[GitHub] [yunikorn-release] DerekTBrown commented on pull request #111: [YUNIKORN-1359] Bring-your-own ConfigMap

2022-10-18 Thread GitBox
DerekTBrown commented on PR #111: URL: https://github.com/apache/yunikorn-release/pull/111#issuecomment-1283043050 To describe the usecase a bit more; we need to be able to perform separate deployments for the infrastructure pieces (the YuniKorn scheduler) and the configuration (the queue

[GitHub] [yunikorn-release] wilfred-s commented on pull request #111: [YUNIKORN-1359] Bring-your-own ConfigMap

2022-10-18 Thread GitBox
wilfred-s commented on PR #111: URL: https://github.com/apache/yunikorn-release/pull/111#issuecomment-1282937854 Wouldn't the real solution then be to allow changing the config map in the deployment? Combined with the current possibility of changing the yaml file you would have full

[GitHub] [yunikorn-release] DerekTBrown commented on pull request #111: [YUNIKORN-1359] Bring-your-own ConfigMap

2022-10-17 Thread GitBox
DerekTBrown commented on PR #111: URL: https://github.com/apache/yunikorn-release/pull/111#issuecomment-1281724817 > How are you handling the fact that we cannot start without a configuration? We fail and exit if we cannot read the config. It would fail the helm deployment. That means the

[GitHub] [yunikorn-release] wilfred-s commented on pull request #111: [Feature] Bring-your-own ConfigMap

2022-10-17 Thread GitBox
wilfred-s commented on PR #111: URL: https://github.com/apache/yunikorn-release/pull/111#issuecomment-1281209375 How are you handling the fact that we cannot start without a configuration? We fail and exit if we cannot read the config. It would fail the helm deployment. That means the

[GitHub] [yunikorn-release] wilfred-s commented on pull request #111: [Feature] Bring-your-own ConfigMap

2022-10-15 Thread GitBox
wilfred-s commented on PR #111: URL: https://github.com/apache/yunikorn-release/pull/111#issuecomment-1279796851 Please first open a jira for this. We track everything via jira. Check the [how to contribute](https://yunikorn.apache.org/community/how_to_contribute) document on the website.

[GitHub] [yunikorn-release] DerekTBrown opened a new pull request, #111: [Feature] Bring-your-own ConfigMap

2022-10-14 Thread GitBox
DerekTBrown opened a new pull request, #111: URL: https://github.com/apache/yunikorn-release/pull/111 This PR adds a simple feature; if `Values.configuration` is empty, the chart will not create the `ConfigMap`. This enables users to create the ConfigMap in their own way. ## Test

[GitHub] [yunikorn-release] DerekTBrown commented on pull request #110: [YUNIKORN-1261] Add pre-upgrade hook

2022-10-14 Thread GitBox
DerekTBrown commented on PR #110: URL: https://github.com/apache/yunikorn-release/pull/110#issuecomment-1279475756 @wilfred-s I apologize- I put this up-for-review prematurely. I think YuniKorn is in a highly similar place to Spark Operator in terms of Helm hooks:

[GitHub] [yunikorn-release] wilfred-s commented on pull request #110: [YUNIKORN-1261] Add pre-upgrade hook

2022-10-13 Thread GitBox
wilfred-s commented on PR #110: URL: https://github.com/apache/yunikorn-release/pull/110#issuecomment-1278322727 By adding a pre-upgrade hook could we be triggering a `resource exists` error causing the upgrade to fail? Or are we relying on the default hook deletion policy

[GitHub] [yunikorn-release] DerekTBrown opened a new pull request, #110: Fix(YUNICORN-1261): Add pre-upgrade hook

2022-10-13 Thread GitBox
DerekTBrown opened a new pull request, #110: URL: https://github.com/apache/yunikorn-release/pull/110 I believe `YUNICORN-1261` is caused because we are missing a lifecycle hook for upgrades: https://helm.sh/docs/topics/charts_hooks/#the-available-hooks -- This is an automated

[GitHub] [yunikorn-release] wilfred-s closed pull request #106: [YUNIKORN-156] license check for release repo

2022-09-13 Thread GitBox
wilfred-s closed pull request #106: [YUNIKORN-156] license check for release repo URL: https://github.com/apache/yunikorn-release/pull/106 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [yunikorn-release] pbacsko closed pull request #109: [YUNIKORN-1308]validate_cluster.sh should auto-detect DOCKER_ARCH

2022-09-12 Thread GitBox
pbacsko closed pull request #109: [YUNIKORN-1308]validate_cluster.sh should auto-detect DOCKER_ARCH URL: https://github.com/apache/yunikorn-release/pull/109 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [yunikorn-release] HuangTing-Yao commented on a diff in pull request #106: [YUNIKORN-156] license check for release repo

2022-09-12 Thread GitBox
HuangTing-Yao commented on code in PR #106: URL: https://github.com/apache/yunikorn-release/pull/106#discussion_r968183845 ## check_license.sh: ## @@ -0,0 +1,38 @@ +#!/usr/bin/env bash + +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [yunikorn-release] HuangTing-Yao commented on pull request #109: [YUNIKORN-1308]validate_cluster.sh should auto-detect DOCKER_ARCH

2022-09-12 Thread GitBox
HuangTing-Yao commented on PR #109: URL: https://github.com/apache/yunikorn-release/pull/109#issuecomment-1243410087 > I do not think that we should copy this logic. Fixing the calls is a much better solution. I also noticed that we do not pick up both of the vars in all cases. > >

[GitHub] [yunikorn-release] HuangTing-Yao opened a new pull request, #109: [YUNIKORN-1308]validate_cluster.sh should auto-detect DOCKER_ARCH

2022-09-10 Thread GitBox
HuangTing-Yao opened a new pull request, #109: URL: https://github.com/apache/yunikorn-release/pull/109 Set `HOST_ARCH` and `DOCKER_ARCH` in validate_cluster.sh https://issues.apache.org/jira/browse/YUNIKORN-1308 -- This is an automated message from the Apache Git Service. To respond

[GitHub] [yunikorn-release] wilfred-s closed pull request #108: [YUNIKORN-1316] The value of userLabelKey in scheduler is empty when yunikorn is deployed by helm.

2022-09-09 Thread GitBox
wilfred-s closed pull request #108: [YUNIKORN-1316] The value of userLabelKey in scheduler is empty when yunikorn is deployed by helm. URL: https://github.com/apache/yunikorn-release/pull/108 -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [yunikorn-release] wilfred-s merged pull request #107: [YUNIKORN-1311] helm chart release for v1.1

2022-09-09 Thread GitBox
wilfred-s merged PR #107: URL: https://github.com/apache/yunikorn-release/pull/107 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [yunikorn-release] 0yukali0 opened a new pull request, #108: [YUNIKORN-1316] The value of userLabelKey in scheduler is empty when yunikorn is deployed by helm.

2022-09-09 Thread GitBox
0yukali0 opened a new pull request, #108: URL: https://github.com/apache/yunikorn-release/pull/108 ### What is this PR for? UserLabelKey is empty when yunikorn is deployed by helm. ### What type of PR is it? * [x] - Bug Fix * [ ] - Improvement * [ ] - Feature * [ ]

[GitHub] [yunikorn-release] wilfred-s opened a new pull request, #106: [YUNIKORN-156] license check for release repo

2022-08-31 Thread GitBox
wilfred-s opened a new pull request, #106: URL: https://github.com/apache/yunikorn-release/pull/106 Pre-commit worklfow to check license in release related files. Add workflow badge. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [yunikorn-release] yangwwei merged pull request #105: [YUNIKORN-1292] Fix github pages issues after moving to TLP

2022-08-23 Thread GitBox
yangwwei merged PR #105: URL: https://github.com/apache/yunikorn-release/pull/105 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [yunikorn-release] yangwwei opened a new pull request, #105: [YUNIKORN-1292] Fix github pages issues after moving to TLP

2022-08-23 Thread GitBox
yangwwei opened a new pull request, #105: URL: https://github.com/apache/yunikorn-release/pull/105 https://apache.github.io/yunikorn-release/ still has the out-of-dated issue. This was pointed out by the Spark community in this PR:

[GitHub] [yunikorn-release] yangwwei closed pull request #104: [YUNIKORN-1292] Fix github pages issues after moving to TLP

2022-08-23 Thread GitBox
yangwwei closed pull request #104: [YUNIKORN-1292] Fix github pages issues after moving to TLP URL: https://github.com/apache/yunikorn-release/pull/104 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [yunikorn-release] yangwwei opened a new pull request, #104: [YUNIKORN-1292] Fix github pages issues after moving to TLP

2022-08-23 Thread GitBox
yangwwei opened a new pull request, #104: URL: https://github.com/apache/yunikorn-release/pull/104 https://apache.github.io/yunikorn-release/ still has the out-of-dated issue. This was pointed out by the Spark community in this PR:

[GitHub] [yunikorn-release] craigcondit closed pull request #103: [YUNIKORN-1283] go dependency upgrades

2022-08-18 Thread GitBox
craigcondit closed pull request #103: [YUNIKORN-1283] go dependency upgrades URL: https://github.com/apache/yunikorn-release/pull/103 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [yunikorn-release] wilfred-s opened a new pull request, #103: [YUNIKORN-1283] go dependency upgrades

2022-08-17 Thread GitBox
wilfred-s opened a new pull request, #103: URL: https://github.com/apache/yunikorn-release/pull/103 ### What is this PR for? Update go dependencies: * golang.org/x/crypto * golang.org/x/net * golang.org/x/sys * golang.org/x/text * golang.org/x/tools ### What type

[GitHub] [yunikorn-release] wilfred-s closed pull request #102: [YUNIKORN-1256] license issues in deployments examples

2022-08-10 Thread GitBox
wilfred-s closed pull request #102: [YUNIKORN-1256] license issues in deployments examples URL: https://github.com/apache/yunikorn-release/pull/102 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [yunikorn-release] wilfred-s opened a new pull request, #102: [YUNIKORN-1256] license issues in deployments examples

2022-08-07 Thread GitBox
wilfred-s opened a new pull request, #102: URL: https://github.com/apache/yunikorn-release/pull/102 ### What is this PR for? In the k8shim the tfjob contains files that are not part of Apache. The LICENSE file should show this. It also should show correctly in the source release and

[GitHub] [yunikorn-release] wilfred-s closed pull request #101: [YUNIKORN-1250] add .asf.yaml to release repo

2022-08-03 Thread GitBox
wilfred-s closed pull request #101: [YUNIKORN-1250] add .asf.yaml to release repo URL: https://github.com/apache/yunikorn-release/pull/101 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [yunikorn-release] lowc1012 commented on pull request #101: [YUNIKORN-1250] add .asf.yaml to release repo

2022-08-02 Thread GitBox
lowc1012 commented on PR #101: URL: https://github.com/apache/yunikorn-release/pull/101#issuecomment-1203509925 Thanks @wilfred-s for the review! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [yunikorn-release] lowc1012 opened a new pull request, #101: [YUNIKORN-1250] add .asf.yaml to release repo

2022-08-02 Thread GitBox
lowc1012 opened a new pull request, #101: URL: https://github.com/apache/yunikorn-release/pull/101 Add .asf.yaml file Jira: [YUNIKORN-1250](https://issues.apache.org/jira/projects/YUNIKORN/issues/YUNIKORN-1250) -- This is an automated message from the Apache Git Service. To

[GitHub] [yunikorn-release] wilfred-s closed pull request #100: [YUNIKORN-1265] fix shellcheck issues in our shell scripts

2022-07-31 Thread GitBox
wilfred-s closed pull request #100: [YUNIKORN-1265] fix shellcheck issues in our shell scripts URL: https://github.com/apache/yunikorn-release/pull/100 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [yunikorn-release] lowc1012 opened a new pull request, #100: [YUNIKORN-1265] fix shellcheck issues in our shell scripts

2022-07-30 Thread GitBox
lowc1012 opened a new pull request, #100: URL: https://github.com/apache/yunikorn-release/pull/100 What is this PR for? Fix shellcheck issues in merge_pr.sh and validate_cluster.sh Todos Fix issues and add shellcheck into CI for other repos What is the Jira issue?

[GitHub] [yunikorn-release] wilfred-s closed pull request #99: [YUNIKORN-1262] stash local changes in merge_perge.sh

2022-07-18 Thread GitBox
wilfred-s closed pull request #99: [YUNIKORN-1262] stash local changes in merge_perge.sh URL: https://github.com/apache/yunikorn-release/pull/99 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [yunikorn-release] craigcondit closed pull request #98: [YUNIKORN-1251] Multi arch support in validation script

2022-06-30 Thread GitBox
craigcondit closed pull request #98: [YUNIKORN-1251] Multi arch support in validation script URL: https://github.com/apache/yunikorn-release/pull/98 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[GitHub] [yunikorn-release] wilfred-s opened a new pull request, #98: [YUNIKORN-1251] Multi arch support in validation script

2022-06-30 Thread GitBox
wilfred-s opened a new pull request, #98: URL: https://github.com/apache/yunikorn-release/pull/98 Add support for images build for specific architectures. Allow architecture change via the HOST_ARCH shell variable. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [yunikorn-release] wilfred-s closed pull request #96: [YUNIKORN-1239] release script error handling

2022-06-27 Thread GitBox
wilfred-s closed pull request #96: [YUNIKORN-1239] release script error handling URL: https://github.com/apache/yunikorn-release/pull/96 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [yunikorn-release] wilfred-s closed pull request #97: [YUNIKORN-1247] Update helm hook weights accordingly

2022-06-27 Thread GitBox
wilfred-s closed pull request #97: [YUNIKORN-1247] Update helm hook weights accordingly URL: https://github.com/apache/yunikorn-release/pull/97 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [yunikorn-release] craigcondit closed pull request #95: [YUNIKORN-1215] multi architecture build

2022-06-17 Thread GitBox
craigcondit closed pull request #95: [YUNIKORN-1215] multi architecture build URL: https://github.com/apache/yunikorn-release/pull/95 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

[GitHub] [yunikorn-release] craigcondit commented on a diff in pull request #95: [YUNIKORN-1215] multi architecture build

2022-06-17 Thread GitBox
craigcondit commented on code in PR #95: URL: https://github.com/apache/yunikorn-release/pull/95#discussion_r900447615 ## tools/build-image.py: ## @@ -0,0 +1,276 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See the

[GitHub] [yunikorn-release] wilfred-s opened a new pull request, #96: [YUNIKORN-1239] release script error handling

2022-06-17 Thread GitBox
wilfred-s opened a new pull request, #96: URL: https://github.com/apache/yunikorn-release/pull/96 Adding error handling to the release script Sync changes for git SHA in Makefiles into the release script -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [yunikorn-release] wilfred-s opened a new pull request, #95: [YUNIKORN-1215] multi architecture build

2022-06-17 Thread GitBox
wilfred-s opened a new pull request, #95: URL: https://github.com/apache/yunikorn-release/pull/95 Build AMD and ARM images for YuniKorn. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [yunikorn-release] craigcondit closed pull request #94: [YNIKORN-1229] fix version replacement in chart

2022-06-14 Thread GitBox
craigcondit closed pull request #94: [YNIKORN-1229] fix version replacement in chart URL: https://github.com/apache/yunikorn-release/pull/94 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [yunikorn-release] yuchaoran2011 merged pull request #93: [YUNIKORN-1236] add artifacthub-repo.yml file

2022-06-13 Thread GitBox
yuchaoran2011 merged PR #93: URL: https://github.com/apache/yunikorn-release/pull/93 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [yunikorn-release] wilfred-s opened a new pull request, #94: [YNIKORN-1229] fix version replacement in chart

2022-06-13 Thread GitBox
wilfred-s opened a new pull request, #94: URL: https://github.com/apache/yunikorn-release/pull/94 The version replacement in the values.yaml is to liberal when it comes to the selection of text to replace. This breaks the plugin image link in the chart. -- This is an automated

[GitHub] [yunikorn-release] wilfred-s opened a new pull request, #93: [YUNIKORN-1236] add artifacthub-repo.yml file

2022-06-13 Thread GitBox
wilfred-s opened a new pull request, #93: URL: https://github.com/apache/yunikorn-release/pull/93 Ownership information to claim helm repo. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [yunikorn-release] wilfred-s merged pull request #92: [YUNIKORN-993] helm chart update for v1.0.0

2022-05-04 Thread GitBox
wilfred-s merged PR #92: URL: https://github.com/apache/yunikorn-release/pull/92 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [yunikorn-release] craigcondit closed pull request #91: [YUNIKORN-1192] Update exclude list in release build

2022-04-28 Thread GitBox
craigcondit closed pull request #91: [YUNIKORN-1192] Update exclude list in release build URL: https://github.com/apache/yunikorn-release/pull/91 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [yunikorn-release] wilfred-s opened a new pull request, #91: [YUNIKORN-1192] Update exclude list in release build

2022-04-27 Thread GitBox
wilfred-s opened a new pull request, #91: URL: https://github.com/apache/yunikorn-release/pull/91 Removing non-existing entries: landmark, .travis.yml Removing .helmignore and .golangci.yml (need to be packed) Adding mac filesystem specific .DS_Store -- This is an automated message

[GitHub] [yunikorn-release] wilfred-s closed pull request #90: [YUNIKORN-1189] Support plugin mode in release validation script

2022-04-26 Thread GitBox
wilfred-s closed pull request #90: [YUNIKORN-1189] Support plugin mode in release validation script URL: https://github.com/apache/yunikorn-release/pull/90 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [yunikorn-release] craigcondit opened a new pull request, #90: [YUNIKORN-1189] Support plugin mode in release validation script

2022-04-26 Thread GitBox
craigcondit opened a new pull request, #90: URL: https://github.com/apache/yunikorn-release/pull/90 JIRA: https://issues.apache.org/jira/browse/YUNIKORN-1189 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [yunikorn-release] craigcondit closed pull request #89: [YUNIKORN-764] validations steps for a release

2022-04-18 Thread GitBox
craigcondit closed pull request #89: [YUNIKORN-764] validations steps for a release URL: https://github.com/apache/yunikorn-release/pull/89 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [yunikorn-release] wilfred-s opened a new pull request, #89: [YUNIKORN-764] validations steps for a release

2022-04-15 Thread GitBox
wilfred-s opened a new pull request, #89: URL: https://github.com/apache/yunikorn-release/pull/89 Add script to start a K8s cluster based on the release build images. Fix license headers and go version (yunikorn-core expects go 1.16) -- This is an automated message from the Apache Git

[GitHub] [yunikorn-release] craigcondit closed pull request #88: [YUNIKORN-197] update commit instructions script only

2022-03-31 Thread GitBox
craigcondit closed pull request #88: URL: https://github.com/apache/yunikorn-release/pull/88 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [yunikorn-release] wilfred-s opened a new pull request #88: [YUNIKORN-197] update commit instructions script only

2022-03-31 Thread GitBox
wilfred-s opened a new pull request #88: URL: https://github.com/apache/yunikorn-release/pull/88 Script to commit changes using command line. Referenced in the contribution documentation update in YUNIKORN-1048. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [yunikorn-release] craigcondit closed pull request #87: [YUNIKORN-1148] Update release repo to remove incubator references

2022-03-25 Thread GitBox
craigcondit closed pull request #87: URL: https://github.com/apache/yunikorn-release/pull/87 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [yunikorn-release] craigcondit opened a new pull request #87: [YUNIKORN-1148] Update release repo to remove incubator references

2022-03-25 Thread GitBox
craigcondit opened a new pull request #87: URL: https://github.com/apache/yunikorn-release/pull/87 JIRA: https://issues.apache.org/jira/browse/YUNIKORN-1148 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [incubator-yunikorn-release] wilfred-s closed pull request #86: [YUNIKORN-1130] Update helm chart example with new resource unit types

2022-03-22 Thread GitBox
wilfred-s closed pull request #86: URL: https://github.com/apache/incubator-yunikorn-release/pull/86 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe,

[GitHub] [incubator-yunikorn-release] craigcondit opened a new pull request #86: [YUNIKORN-1130] Update helm chart example with new resource unit types

2022-03-22 Thread GitBox
craigcondit opened a new pull request #86: URL: https://github.com/apache/incubator-yunikorn-release/pull/86 JIRA: https://issues.apache.org/jira/browse/YUNIKORN-1130 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [incubator-yunikorn-release] wilfred-s closed pull request #83: [YUNIKORN-638] Make placeholder image configurable

2022-03-16 Thread GitBox
wilfred-s closed pull request #83: URL: https://github.com/apache/incubator-yunikorn-release/pull/83 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe,

[GitHub] [incubator-yunikorn-release] craigcondit closed pull request #85: [YUNIKORN-1113] Allow yunikorn services to run in host network

2022-03-11 Thread GitBox
craigcondit closed pull request #85: URL: https://github.com/apache/incubator-yunikorn-release/pull/85 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe,

  1   2   3   4   5   6   7   8   >