kevinw66 opened a new pull request, #9:
URL: https://github.com/apache/bigtop-manager/pull/9
(no comment)
--
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,
kevinw66 merged PR #8:
URL: https://github.com/apache/bigtop-manager/pull/8
--
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:
iwasakims merged PR #1276:
URL: https://github.com/apache/bigtop/pull/1276
--
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:
timyuer opened a new pull request, #7:
URL: https://github.com/apache/bigtop-manager/pull/7
(no comment)
--
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,
masatana opened a new pull request, #1276:
URL: https://github.com/apache/bigtop/pull/1276
### Description of PR
GPDB was archived and made read-only in GitHub and there was no objection to
drop GPDB.
https://lists.apache.org/thread/k5kvo9ly42lz2dz03l1or6g80r9q6tq1
kevinw66 merged PR #6:
URL: https://github.com/apache/bigtop-manager/pull/6
--
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:
kevinw66 merged PR #5:
URL: https://github.com/apache/bigtop-manager/pull/5
--
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:
kevinw66 commented on PR #5:
URL: https://github.com/apache/bigtop-manager/pull/5#issuecomment-2144586199
Thanks @guyuqi
--
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.
guyuqi commented on PR #5:
URL: https://github.com/apache/bigtop-manager/pull/5#issuecomment-2144206415
LGTM，
+1
--
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
timyuer commented on PR #1275:
URL: https://github.com/apache/bigtop/pull/1275#issuecomment-2144144458
@iwasakims @kevinw66 @guyuqi @kevinw66
Please help review this PR, Thanks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
timyuer opened a new pull request, #1275:
URL: https://github.com/apache/bigtop/pull/1275
### Description of PR
### How was this patch tested?
### For code changes:
- [ ] Does the title or this PR starts with the corresponding JIRA issue id
(e.g.
timyuer opened a new pull request, #6:
URL: https://github.com/apache/bigtop-manager/pull/6
(no comment)
--
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,
iwasakims commented on PR #1274:
URL: https://github.com/apache/bigtop/pull/1274#issuecomment-2144089358
Thanks, @sekikn. I merged this to master and branch-3.3.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
iwasakims merged PR #1274:
URL: https://github.com/apache/bigtop/pull/1274
--
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:
sekikn commented on PR #1274:
URL: https://github.com/apache/bigtop/pull/1274#issuecomment-2143917435
+1, ensured that both Rocky 8 and Fedora 38 have python2 on aarch64.
```
$ curl -sL https://github.com/apache/bigtop/pull/1274.diff | git apply
$ cd docker/bigtop-puppet
$
kevinw66 opened a new pull request, #5:
URL: https://github.com/apache/bigtop-manager/pull/5
(no comment)
--
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,
sekikn commented on PR #1270:
URL: https://github.com/apache/bigtop/pull/1270#issuecomment-2143760396
Merged into master.
--
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.
sekikn merged PR #1270:
URL: https://github.com/apache/bigtop/pull/1270
--
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:
sekikn commented on PR #1249:
URL: https://github.com/apache/bigtop/pull/1249#issuecomment-2143759156
Merged into master and branch-3.2.
--
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
sekikn merged PR #1249:
URL: https://github.com/apache/bigtop/pull/1249
--
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:
sekikn commented on PR #1249:
URL: https://github.com/apache/bigtop/pull/1249#issuecomment-2143757410
+1, tested on CentOS 7 and Rocky 8 as follows. Good catch @LiJie20190102!
```
$ curl -sL https://github.com/apache/bigtop/pull/1249.diff | git apply
$ ./gradlew flink-clean
sekikn commented on PR #1270:
URL: https://github.com/apache/bigtop/pull/1270#issuecomment-2143681589
+1, tested as follows (specifying a dummy repo URL in running
docker-hadoop.sh).
Thanks @iwasakims.
```
$ curl -sL https://github.com/apache/bigtop/pull/1270.diff | git apply
dependabot[bot] opened a new pull request, #4:
URL: https://github.com/apache/bigtop-manager/pull/4
Bumps [postcss](https://github.com/postcss/postcss) from 8.4.28 to 8.4.38.
Release notes
Sourced from https://github.com/postcss/postcss/releases;>postcss's
releases.
8.4.38
dependabot[bot] opened a new pull request, #2:
URL: https://github.com/apache/bigtop-manager/pull/2
Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from
4.4.5 to 4.5.3.
Changelog
Sourced from
dependabot[bot] opened a new pull request, #1:
URL: https://github.com/apache/bigtop-manager/pull/1
Bumps [axios](https://github.com/axios/axios) from 1.4.0 to 1.6.0.
Release notes
Sourced from https://github.com/axios/axios/releases;>axios's releases.
Release v1.6.0
dependabot[bot] opened a new pull request, #3:
URL: https://github.com/apache/bigtop-manager/pull/3
Bumps
[follow-redirects](https://github.com/follow-redirects/follow-redirects) from
1.15.2 to 1.15.6.
Commits
iwasakims opened a new pull request, #1274:
URL: https://github.com/apache/bigtop/pull/1274
https://issues.apache.org/jira/browse/BIGTOP-4114
Unfortunately we seem to need python2 for rebuilding native libraries
required for (old?) node-spss by node-gyp. `/usr/bin/python` pointing to
timyuer commented on PR #1267:
URL: https://github.com/apache/bigtop/pull/1267#issuecomment-2141508792
@sekikn
Sorry to bother, I found that `snappy-devel` causes doris to fail to compile
during the compilation process, so I can ask which components are used by this
package?
--
This
buggtb closed pull request #382: Bigtop-3070 support Spark Thrift Server in
Charm
URL: https://github.com/apache/bigtop/pull/382
--
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
iwasakims closed pull request #1271: BIGTOP-4111. Fix compilation error of GPDB
on CentOS 7 aarch64.
URL: https://github.com/apache/bigtop/pull/1271
--
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
iwasakims commented on PR #1271:
URL: https://github.com/apache/bigtop/pull/1271#issuecomment-2138609680
closing this since GPDB was archived.
--
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
JiaLiangC commented on PR #1259:
URL: https://github.com/apache/bigtop/pull/1259#issuecomment-2136327917
@iwasakims Could you help review this pr ?
--
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
masatana commented on PR #1273:
URL: https://github.com/apache/bigtop/pull/1273#issuecomment-2136189358
@iwasakims Thank you!
--
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
iwasakims merged PR #1273:
URL: https://github.com/apache/bigtop/pull/1273
--
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:
masatana opened a new pull request, #1273:
URL: https://github.com/apache/bigtop/pull/1273
### Description of PR
https://issues.apache.org/jira/browse/BIGTOP-4112
We don't have to specify `PYSPARK_PYTHON` in `/usr/bin/pyspark` as it just
execute `exec
JiaLiangC commented on PR #1272:
URL: https://github.com/apache/bigtop/pull/1272#issuecomment-2133109719
I think PYSPARK_PYTHON=python2 is unnecessary. We can simply specify python3
in installation script . Users will set a more specific Python path in their
environment settings when they
masatana commented on PR #1272:
URL: https://github.com/apache/bigtop/pull/1272#issuecomment-2133098600
Okay, I'll create a follow-up JIRA ticket.
(BTW, does [this
JiaLiangC commented on PR #1272:
URL: https://github.com/apache/bigtop/pull/1272#issuecomment-2133082713
> Oops, I haven't catched that code. And should we also handle the code here
?
>
>
masatana commented on PR #1272:
URL: https://github.com/apache/bigtop/pull/1272#issuecomment-2133079389
Oops, I haven't catched that code.
And should we also handle the code here ?
JiaLiangC commented on PR #1272:
URL: https://github.com/apache/bigtop/pull/1272#issuecomment-2133035463
@iwasakims Don't we need to handle the logic PYSPARK_PYTHON=python2 in the
spec code?
--
This is an automated message from the Apache Git Service.
To respond to the message, please
iwasakims merged PR #1272:
URL: https://github.com/apache/bigtop/pull/1272
--
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:
masatana opened a new pull request, #1272:
URL: https://github.com/apache/bigtop/pull/1272
### Description of PR
https://issues.apache.org/jira/browse/BIGTOP-4053
As discussed in the JIRA issue, I have removed the python-releated spark
tests in this PR.
Please feel
iwasakims commented on PR #1271:
URL: https://github.com/apache/bigtop/pull/1271#issuecomment-2131274614
packaging and smoke-tests of gpdb passed on CentOS 7 aarch64. I also tested
the patch on ubuntu 22.04 x86_64 for checking regression.
--
This is an automated message from the Apache
iwasakims opened a new pull request, #1271:
URL: https://github.com/apache/bigtop/pull/1271
https://issues.apache.org/jira/browse/BIGTOP-4111
```
gcc -std=gnu99 -Wall -Wmissing-prototypes -Wpointer-arith -Werror=vla
-Wendif-labels -Wmissing-format-attribute -Wformat-security
MacChen01 commented on PR #1269:
URL: https://github.com/apache/bigtop/pull/1269#issuecomment-2126792474
@iwasakims
yes, the code needs to be modified like fedora-38:
https://github.com/apache/bigtop/pull/1269/commits/42899233ba2e6adb1c1c8e740a7ce1b44d72b159
--
This is an
iwasakims commented on PR #1269:
URL: https://github.com/apache/bigtop/pull/1269#issuecomment-2126763422
@MacChen01 looks like rokcylinux-9 needs the same fix with fedora-38. Since
rokcylinux-9 is out of scope of Bigtop 3.3.0 release, I'll look into it later.
--
This is an automated
MacChen01 commented on PR #1269:
URL: https://github.com/apache/bigtop/pull/1269#issuecomment-2126554455
We verified this PR, but compile failed in rockylinux 9
https://issues.apache.org/jira/browse/BIGTOP-4110
@iwasakims @sekikn
--
This is an automated message from the Apache
iwasakims commented on PR #1269:
URL: https://github.com/apache/bigtop/pull/1269#issuecomment-2126248731
I merged this. Thanks, @sekikn.
--
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
iwasakims merged PR #1269:
URL: https://github.com/apache/bigtop/pull/1269
--
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:
sekikn commented on PR #1269:
URL: https://github.com/apache/bigtop/pull/1269#issuecomment-2126086283
+1, tested on CentOS 7. Thanks @iwasakims.
```
$ curl -sL https://github.com/apache/bigtop/pull/1269.diff | git apply
$ sudo bigtop_toolchain/bin/puppetize.sh
$ ./gradlew
iwasakims opened a new pull request, #1270:
URL: https://github.com/apache/bigtop/pull/1270
https://issues.apache.org/jira/browse/BIGTOP-4108
* Currently openEuler is not supported in [the bigtop_repo class for setting
up repository of
iwasakims merged PR #1263:
URL: https://github.com/apache/bigtop/pull/1263
--
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:
JiaLiangC commented on code in PR #1263:
URL: https://github.com/apache/bigtop/pull/1263#discussion_r1609293265
##
bigtop-deploy/puppet/manifests/python.pp:
##
@@ -14,16 +14,50 @@
# limitations under the License.
class python {
-if (($operatingsystem == 'Ubuntu' and 0
iwasakims commented on code in PR #1263:
URL: https://github.com/apache/bigtop/pull/1263#discussion_r1609257828
##
bigtop-deploy/puppet/manifests/python.pp:
##
@@ -14,16 +14,50 @@
# limitations under the License.
class python {
-if (($operatingsystem == 'Ubuntu' and 0
JiaLiangC commented on PR #1263:
URL: https://github.com/apache/bigtop/pull/1263#issuecomment-2123795380
JiaLiangC commented on code in PR #1263:
URL: https://github.com/apache/bigtop/pull/1263#discussion_r1609196909
##
bigtop-deploy/puppet/manifests/python.pp:
##
@@ -14,16 +14,57 @@
# limitations under the License.
class python {
-if (($operatingsystem == 'Ubuntu' and 0
iwasakims commented on code in PR #1263:
URL: https://github.com/apache/bigtop/pull/1263#discussion_r1609166400
##
bigtop-deploy/puppet/manifests/python.pp:
##
@@ -14,16 +14,57 @@
# limitations under the License.
class python {
-if (($operatingsystem == 'Ubuntu' and 0
JiaLiangC commented on PR #1263:
URL: https://github.com/apache/bigtop/pull/1263#issuecomment-2121612810
> I'm ok to declare some limitation on CentOS 7 in the release. CentOS 7
will be EOL on June 30, 2024.
Agreed, I'll skip setting the default Python on CentOS 7. Users can manage
iwasakims commented on PR #1263:
URL: https://github.com/apache/bigtop/pull/1263#issuecomment-2121608007
I'm ok to declare some limitation on CentOS 7 in the release. CentOS 7 will
be EOL on June 30, 2024.
--
This is an automated message from the Apache Git Service.
To respond to the
JiaLiangC commented on code in PR #1263:
URL: https://github.com/apache/bigtop/pull/1263#discussion_r1607519625
##
bigtop-deploy/puppet/manifests/python.pp:
##
@@ -14,16 +14,70 @@
# limitations under the License.
class python {
-if (($operatingsystem == 'Ubuntu' and 0
iwasakims commented on PR #1263:
URL: https://github.com/apache/bigtop/pull/1263#issuecomment-2121603024
> Even after changing the default to Python 3, you may encounter issues
where other programs that still rely on Python 2 stop working.
yeah. I know as I commented in the patch.
JiaLiangC commented on code in PR #1263:
URL: https://github.com/apache/bigtop/pull/1263#discussion_r1607515132
##
bigtop-deploy/puppet/manifests/python.pp:
##
@@ -14,16 +14,70 @@
# limitations under the License.
class python {
-if (($operatingsystem == 'Ubuntu' and 0
JiaLiangC commented on PR #1263:
URL: https://github.com/apache/bigtop/pull/1263#issuecomment-2121597494
> @JiaLiangC How about leave CentOS 7 as is (i.e. python2 as default) and
commit other part first?
@iwasakims Setting the default Python version to Python 3 on CentOS 7 is
quite
iwasakims commented on PR #1263:
URL: https://github.com/apache/bigtop/pull/1263#issuecomment-2121593539
@JiaLiangC How about leave CentOS 7 as is (i.e. python2 as default) and
commit other part first?
--
This is an automated message from the Apache Git Service.
To respond to the
iwasakims commented on code in PR #1263:
URL: https://github.com/apache/bigtop/pull/1263#discussion_r1607501918
##
bigtop-deploy/puppet/manifests/python.pp:
##
@@ -14,16 +14,70 @@
# limitations under the License.
class python {
-if (($operatingsystem == 'Ubuntu' and 0
iwasakims commented on code in PR #1263:
URL: https://github.com/apache/bigtop/pull/1263#discussion_r1607500668
##
bigtop-deploy/puppet/manifests/python.pp:
##
@@ -14,16 +14,70 @@
# limitations under the License.
class python {
-if (($operatingsystem == 'Ubuntu' and 0
iwasakims commented on PR #1264:
URL: https://github.com/apache/bigtop/pull/1264#issuecomment-2120672324
We need BIGTOP-4100 or addressing smoke-tests issues.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
iwasakims merged PR #1264:
URL: https://github.com/apache/bigtop/pull/1264
--
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:
iwasakims commented on PR #1269:
URL: https://github.com/apache/bigtop/pull/1269#issuecomment-2117519140
smoke-tests passed on fedora-38.
--
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
iwasakims commented on PR #1269:
URL: https://github.com/apache/bigtop/pull/1269#issuecomment-2117509510
packaging on fedora-38 succeeded but deployment failed. The cause seems to
be hidden bug of the manifest. Puppet 8 on Fedora 38 looks stricter than older
versions.
```
Notice:
iwasakims commented on PR #1269:
URL: https://github.com/apache/bigtop/pull/1269#issuecomment-2117399090
Packaging on fedora-38 fails due to lack of Perl module?
```
Can't locate FindBin.pm in @INC (you may need to install the FindBin module)
(@INC contains: ../../src/tools
iwasakims commented on PR #1269:
URL: https://github.com/apache/bigtop/pull/1269#issuecomment-2117398031
packaging and smoke-tests passed on centos-7, rockylinu-8 and debian-11.
(x86_64)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
iwasakims commented on PR #1269:
URL: https://github.com/apache/bigtop/pull/1269#issuecomment-2116487571
> So, do we need to install these Python dependencies during deployment?
It is user's responsibility to set up Python environment now. Since the
current coverage of the
iwasakims commented on PR #1269:
URL: https://github.com/apache/bigtop/pull/1269#issuecomment-2115286491
smoke-tests of gpdb passed on ubuntu-22.04.
--
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
iwasakims commented on PR #1269:
URL: https://github.com/apache/bigtop/pull/1269#issuecomment-2115069702
bigtop-deploy must be updated too.
```
Error: Could not find command '/usr/bin/python2'
Error:
/Stage[main]/Gpdb::Common::Install_packages/Exec[install_pip]/returns: change
iwasakims commented on PR #1269:
URL: https://github.com/apache/bigtop/pull/1269#issuecomment-2114845344
looking into the cause of test error.
```
> Task :bigtop-tests:smoke-tests:gpdb:test
TestGpdb > testGpdb STANDARD_ERROR
24/05/16 10:16:11 INFO lang.Object: Test
iwasakims commented on PR #1269:
URL: https://github.com/apache/bigtop/pull/1269#issuecomment-2114808451
Packaging was succeeded on ubuntu-22.04 and openeuler-22.03. Not yet running
smoke-tests.
--
This is an automated message from the Apache Git Service.
To respond to the message,
iwasakims opened a new pull request, #1269:
URL: https://github.com/apache/bigtop/pull/1269
https://issues.apache.org/jira/browse/BIGTOP-4106
* Since we are dropping python2 from toolchain and bigtop-deploy, packaging
stuff of GPDB must be updated to be assuming python 3.
* We
iwasakims commented on PR #1265:
URL: https://github.com/apache/bigtop/pull/1265#issuecomment-2110197742
Thanks, @kevinw66.
--
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.
kevinw66 merged PR #1265:
URL: https://github.com/apache/bigtop/pull/1265
--
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:
kevinw66 commented on PR #1265:
URL: https://github.com/apache/bigtop/pull/1265#issuecomment-2110152898
Before:
```
[root@localhost bigtop]# ./gradlew -POS=openeuler-22.03 -Pprefix=3.3.0
bigtop-slaves
..
#7 [3/5] RUN if [ -f ~/.bash_profile ]; then . ~/.bash_profile; fi
iwasakims merged PR #1268:
URL: https://github.com/apache/bigtop/pull/1268
--
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:
iwasakims commented on code in PR #1268:
URL: https://github.com/apache/bigtop/pull/1268#discussion_r1599425800
##
packages.gradle:
##
@@ -102,7 +102,7 @@ def isValidMavenBuildThreads(threads) {
}
def getPkgNameSuffix(component, type) {
-def originalSuffix =
Kapkiai commented on code in PR #1268:
URL: https://github.com/apache/bigtop/pull/1268#discussion_r1599415055
##
packages.gradle:
##
@@ -102,7 +102,7 @@ def isValidMavenBuildThreads(threads) {
}
def getPkgNameSuffix(component, type) {
-def originalSuffix =
Kapkiai commented on code in PR #1268:
URL: https://github.com/apache/bigtop/pull/1268#discussion_r1599400747
##
packages.gradle:
##
@@ -102,7 +102,7 @@ def isValidMavenBuildThreads(threads) {
}
def getPkgNameSuffix(component, type) {
-def originalSuffix =
iwasakims commented on code in PR #1268:
URL: https://github.com/apache/bigtop/pull/1268#discussion_r1599375633
##
packages.gradle:
##
@@ -102,7 +102,7 @@ def isValidMavenBuildThreads(threads) {
}
def getPkgNameSuffix(component, type) {
-def originalSuffix =
iwasakims commented on PR #1266:
URL: https://github.com/apache/bigtop/pull/1266#issuecomment-2106588225
Thanks, @yoda-mon.
--
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.
Kapkiai commented on PR #1268:
URL: https://github.com/apache/bigtop/pull/1268#issuecomment-2105982646
@guyuqi @iwasakims @sekikn
Could you help review this PR?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
Kapkiai opened a new pull request, #1268:
URL: https://github.com/apache/bigtop/pull/1268
### Description of PR
Build fails when '-DpkgSuffix' is specified when '(rpm|deb)_pkg_suffix' is
not specified in bigtop.bom
Issue introduced this bug by not checking whether the
yoda-mon merged PR #1266:
URL: https://github.com/apache/bigtop/pull/1266
--
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:
yoda-mon commented on PR #1266:
URL: https://github.com/apache/bigtop/pull/1266#issuecomment-2105662465
@iwasakims (Sorry for the late, I was on vacation)
Thank you for the information, I understood the situations related to the
deprecation of Hiera 3.
On that case, the build itself
iwasakims commented on PR #1265:
URL: https://github.com/apache/bigtop/pull/1265#issuecomment-2104311337
@sekikn Could you review this? I need to fix this to move Bigtop 3.3.0
release forward.
--
This is an automated message from the Apache Git Service.
To respond to the message, please
iwasakims commented on PR #1266:
URL: https://github.com/apache/bigtop/pull/1266#issuecomment-2104310372
@yoda-mon Could you take a look again? This issue blocks release of Bigtop
3.3.0.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
iwasakims commented on PR #1266:
URL: https://github.com/apache/bigtop/pull/1266#issuecomment-2104309475
The cause of the error turned out to be bug of the manifest rather than
Puppet version. smoke-tests of Hadoop passed on both fedora-38 and centos-7
with the updated manifest.
--
This
iwasakims commented on PR #1266:
URL: https://github.com/apache/bigtop/pull/1266#issuecomment-2104058359
The current manifest [tries to conditionaly switch hiera
file](https://github.com/apache/bigtop/blob/3808ba7d1b8d6c2504c08ed9079a9d5673ae83de/bigtop-deploy/puppet/hiera.yaml#L6)
based
iwasakims commented on PR #1266:
URL: https://github.com/apache/bigtop/pull/1266#issuecomment-2100446201
puppet 8 packaged by fedora-38 is coupled with hiera 3. I guess the error is
caused by just variable lookup settings.
```
[root@495d41ec7cd8 /]# rpm -q hiera
iwasakims commented on PR #1266:
URL: https://github.com/apache/bigtop/pull/1266#issuecomment-2099893700
Actually [deprecation of Hiera
3](https://www.puppet.com/docs/puppet/8/upgrading-from-puppet7-to-puppet8.html#upgrading-from-puppet7-to-puppet8-heira-3-deprecation)
affects
iwasakims merged PR #1248:
URL: https://github.com/apache/bigtop/pull/1248
--
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:
kevinw66 commented on PR #1248:
URL: https://github.com/apache/bigtop/pull/1248#issuecomment-2096124198
> @kevinw66 How about adding the author of #1158 as co-author on merging
this? `root@localhost.localdomain` should be updated to appropriate author too.
I think this can be edited
iwasakims commented on PR #1248:
URL: https://github.com/apache/bigtop/pull/1248#issuecomment-2095573623
@kevinw66 How about adding the author of #1158 as co-author on merging this?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
1 - 100 of 1278 matches
Mail list logo
