ivakegg commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r492847076
##
File path: ansible/roles/logstash/templates/pipeline.conf
##
@@ -0,0 +1,57 @@
+input {
+beats {
+port => 5044
+}
+file {
+
arvindshmicrosoft commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r490638479
##
File path: ansible/roles/logstash/templates/pipeline.conf
##
@@ -0,0 +1,57 @@
+input {
+beats {
+port => 5044
+}
+file
arvindshmicrosoft commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r490638080
##
File path: lib/muchos/ec2.py
##
@@ -322,8 +323,8 @@ def terminate(self):
if path.isfile(self.config.hosts_path):
plainolneesh commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r490564302
##
File path: lib/muchos/ec2.py
##
@@ -322,8 +323,8 @@ def terminate(self):
if path.isfile(self.config.hosts_path):
o
plainolneesh commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r490563943
##
File path: lib/muchos/ec2.py
##
@@ -322,8 +323,8 @@ def terminate(self):
if path.isfile(self.config.hosts_path):
o
plainolneesh commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r490562309
##
File path: ansible/roles/logstash/templates/logstash-simple.conf
##
@@ -0,0 +1,55 @@
+input {
Review comment:
No, that was there for tes
arvindshmicrosoft commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r489703679
##
File path: ansible/roles/filebeat/tasks/main.yml
##
@@ -0,0 +1,61 @@
+---
+#
+# Licensed to the Apache Software Foundation (ASF) under one
arvindshmicrosoft commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r489703475
##
File path: ansible/roles/logstash/templates/logstash-simple-2.conf
##
@@ -0,0 +1,150 @@
+#
+# Licensed to the Apache Software Foundation (A
arvindshmicrosoft merged pull request #371:
URL: https://github.com/apache/fluo-muchos/pull/371
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abo
arvindshmicrosoft merged pull request #255:
URL: https://github.com/apache/fluo-uno/pull/255
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
arvindshmicrosoft opened a new pull request #255:
URL: https://github.com/apache/fluo-uno/pull/255
* Accumulo 1.10.0
* Hadoop 3.1.4
* Zookeeper 3.6.2
* Spark 3.0.1, 2.4.7
This is an automated message from the Apache
arvindshmicrosoft opened a new pull request #371:
URL: https://github.com/apache/fluo-muchos/pull/371
- Spark 3.0.1 and 2.4.7
- Zookeeper 3.6.2
This is an automated message from the Apache Git Service.
To respond to the me
arvindshmicrosoft commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r489130168
##
File path: ansible/roles/logstash/templates/logstash-simple-2.conf
##
@@ -0,0 +1,150 @@
+#
+# Licensed to the Apache Software Foundation (A
arvindshmicrosoft commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r489127446
##
File path: ansible/roles/logstash/templates/logstash-simple.conf
##
@@ -0,0 +1,55 @@
+input {
Review comment:
@plainolneesh could y
arvindshmicrosoft commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r489121815
##
File path: ansible/wipe.yml
##
@@ -29,7 +29,6 @@
retries: 10
delay: 3
until: delete_task is success
-
- name: "wipe grafa
plainolneesh commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r488079754
##
File path: ansible/roles/logstash/templates/logstash-simple-2.conf
##
@@ -0,0 +1,150 @@
+#
+# Licensed to the Apache Software Foundation (ASF) u
plainolneesh commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r488079885
##
File path: ansible/roles/logstash/templates/logstash-simple-2.conf
##
@@ -0,0 +1,150 @@
+#
+# Licensed to the Apache Software Foundation (ASF) u
plainolneesh opened a new issue #370:
URL: https://github.com/apache/fluo-muchos/issues/370
When I referenced the `worker_data-dirs` variable for #338, this causes a
pipeline error. `File paths must be absolute, relative path specified`.
--
ctubbsii merged pull request #197:
URL: https://github.com/apache/fluo-website/pull/197
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
ctubbsii commented on a change in pull request #197:
URL: https://github.com/apache/fluo-website/pull/197#discussion_r486394788
##
File path: _scripts/publish.sh
##
@@ -0,0 +1,36 @@
+#! /usr/bin/env bash
+
+# catch most errors
+set -eE
+trap 'echo "[ERROR] Error occurred at $BA
ctubbsii commented on a change in pull request #197:
URL: https://github.com/apache/fluo-website/pull/197#discussion_r486394788
##
File path: _scripts/publish.sh
##
@@ -0,0 +1,36 @@
+#! /usr/bin/env bash
+
+# catch most errors
+set -eE
+trap 'echo "[ERROR] Error occurred at $BA
keith-turner commented on a change in pull request #197:
URL: https://github.com/apache/fluo-website/pull/197#discussion_r486325253
##
File path: _scripts/publish.sh
##
@@ -0,0 +1,36 @@
+#! /usr/bin/env bash
+
+# catch most errors
+set -eE
+trap 'echo "[ERROR] Error occurred at
arvindshmicrosoft merged pull request #369:
URL: https://github.com/apache/fluo-muchos/pull/369
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abo
arvindshmicrosoft opened a new pull request #369:
URL: https://github.com/apache/fluo-muchos/pull/369
* Accumulo 1.10.0
* Hadoop 3.1.4 and Hadoop 2.10.0
* Zookeeper 3.6.1
This is an automated message from the Apache Git
ctubbsii opened a new pull request #197:
URL: https://github.com/apache/fluo-website/pull/197
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
plainolneesh commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r481311825
##
File path: ansible/roles/logstash/templates/logstash-simple-2.conf
##
@@ -0,0 +1,150 @@
+#
+# Licensed to the Apache Software Foundation (ASF) u
plainolneesh commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r481310725
##
File path: ansible/roles/logstash/templates/logstash-simple-2.conf
##
@@ -0,0 +1,150 @@
+#
+# Licensed to the Apache Software Foundation (ASF) u
plainolneesh commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r481310725
##
File path: ansible/roles/logstash/templates/logstash-simple-2.conf
##
@@ -0,0 +1,150 @@
+#
+# Licensed to the Apache Software Foundation (ASF) u
plainolneesh opened a new issue #368:
URL: https://github.com/apache/fluo-muchos/issues/368
Wipe currently does not stop services / erase the files associated with the
ELK stack. See #338
This is an automated messag
ctubbsii merged pull request #254:
URL: https://github.com/apache/fluo-uno/pull/254
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
ctubbsii commented on a change in pull request #254:
URL: https://github.com/apache/fluo-uno/pull/254#discussion_r477265508
##
File path: bin/uno
##
@@ -17,89 +17,33 @@
# Start: Resolve Script Directory
SOURCE="${BASH_SOURCE[0]}"
-while [[ -h "$SOURCE" ]]; do # resolve $SOU
ctubbsii commented on a change in pull request #254:
URL: https://github.com/apache/fluo-uno/pull/254#discussion_r477263613
##
File path: bin/impl/fetch.sh
##
@@ -122,13 +116,20 @@ zookeeper)
fetch_zookeeper
;;
*)
- echo "Usage: uno fetch "
- echo -e "\nPossible compon
keith-turner commented on a change in pull request #254:
URL: https://github.com/apache/fluo-uno/pull/254#discussion_r476907831
##
File path: bin/impl/fetch.sh
##
@@ -122,13 +116,20 @@ zookeeper)
fetch_zookeeper
;;
*)
- echo "Usage: uno fetch "
- echo -e "\nPossible co
ctubbsii commented on pull request #254:
URL: https://github.com/apache/fluo-uno/pull/254#issuecomment-679311347
@keith-turner I fixed the previous error, if you want to take another pass
at a review.
Some of the conditional statements in bash at the end of some scripts were
returni
ctubbsii commented on pull request #254:
URL: https://github.com/apache/fluo-uno/pull/254#issuecomment-677859599
This isn't quite ready for merging. I saw a bug where this didn't properly
trigger the install of ZooKeeper after installing Hadoop, when doing `uno setup
accumulo` *without* do
ctubbsii commented on a change in pull request #254:
URL: https://github.com/apache/fluo-uno/pull/254#discussion_r474222177
##
File path: bin/impl/install.sh
##
@@ -15,6 +15,7 @@
# See the License for the specific language governing permissions and
# limitations under the Lic
ctubbsii commented on a change in pull request #254:
URL: https://github.com/apache/fluo-uno/pull/254#discussion_r474222177
##
File path: bin/impl/install.sh
##
@@ -15,6 +15,7 @@
# See the License for the specific language governing permissions and
# limitations under the Lic
ctubbsii commented on a change in pull request #254:
URL: https://github.com/apache/fluo-uno/pull/254#discussion_r474221878
##
File path: README.md
##
@@ -188,17 +188,19 @@ applications or follow the instructions starting at the
[Configure a Fluo applic
section of the Fluo in
keith-turner commented on a change in pull request #254:
URL: https://github.com/apache/fluo-uno/pull/254#discussion_r474199567
##
File path: bin/impl/install.sh
##
@@ -15,6 +15,7 @@
# See the License for the specific language governing permissions and
# limitations under the
ctubbsii opened a new pull request #254:
URL: https://github.com/apache/fluo-uno/pull/254
* Fix numerous issues to make ShellCheck pass across `bin/*` and
`conf/uno.conf`
* Add GitHub Actions and CI script for automated ShellCheck testing
* Add missing checksum for latest Hadoop 3
ctubbsii merged pull request #252:
URL: https://github.com/apache/fluo-uno/pull/252
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
ctubbsii merged pull request #253:
URL: https://github.com/apache/fluo-uno/pull/253
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
ctubbsii opened a new pull request #253:
URL: https://github.com/apache/fluo-uno/pull/253
Adds checksum for latest 3.0 version, because it is still supported by
Accumulo, so it'd be useful to test with that version.
This i
ctubbsii opened a new pull request #252:
URL: https://github.com/apache/fluo-uno/pull/252
Show Maven version on tasks that execute `mvn`, and show stack traces,
if there are any, if `mvn` fails.
This is an automated messag
ctubbsii merged pull request #1102:
URL: https://github.com/apache/fluo/pull/1102
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 th
ctubbsii merged pull request #7:
URL: https://github.com/apache/fluo-bytes/pull/7
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 th
ctubbsii merged pull request #12:
URL: https://github.com/apache/fluo-examples/pull/12
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
ctubbsii merged pull request #367:
URL: https://github.com/apache/fluo-muchos/pull/367
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
ctubbsii merged pull request #151:
URL: https://github.com/apache/fluo-recipes/pull/151
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
ctubbsii merged pull request #196:
URL: https://github.com/apache/fluo-website/pull/196
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
ctubbsii merged pull request #20:
URL: https://github.com/apache/fluo-yarn/pull/20
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 t
ctubbsii merged pull request #251:
URL: https://github.com/apache/fluo-uno/pull/251
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
ctubbsii opened a new pull request #196:
URL: https://github.com/apache/fluo-website/pull/196
Fix a link missed from the first pass
This is an automated message from the Apache Git Service.
To respond to the message, please l
ctubbsii opened a new pull request #20:
URL: https://github.com/apache/fluo-yarn/pull/20
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 g
ctubbsii opened a new pull request #251:
URL: https://github.com/apache/fluo-uno/pull/251
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
ctubbsii opened a new pull request #151:
URL: https://github.com/apache/fluo-recipes/pull/151
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
ctubbsii opened a new pull request #367:
URL: https://github.com/apache/fluo-muchos/pull/367
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
ctubbsii opened a new pull request #12:
URL: https://github.com/apache/fluo-examples/pull/12
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
ctubbsii opened a new pull request #7:
URL: https://github.com/apache/fluo-bytes/pull/7
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
ctubbsii opened a new pull request #1102:
URL: https://github.com/apache/fluo/pull/1102
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
ctubbsii merged pull request #195:
URL: https://github.com/apache/fluo-website/pull/195
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
ctubbsii commented on pull request #195:
URL: https://github.com/apache/fluo-website/pull/195#issuecomment-673684752
This is already agreed to on the mailing list, but I wanted to post this
here for a sanity check, to make sure I didn't break any links before merging,
and because I updated
ctubbsii opened a new pull request #195:
URL: https://github.com/apache/fluo-website/pull/195
* Update links and references from the master branch to main
* Add .asf.yaml with configuration to automatically stage the site
* Update instructions for contributing and testing the site buil
arvindshmicrosoft commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r468673413
##
File path: ansible/roles/elasticsearch/tasks/main.yml
##
@@ -0,0 +1,83 @@
+---
+#
+# Licensed to the Apache Software Foundation (ASF) under
arvindshmicrosoft commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r459845654
##
File path: ansible/roles/elasticsearch/tasks/main.yml
##
@@ -0,0 +1,83 @@
+---
+#
+# Licensed to the Apache Software Foundation (ASF) under
arvindshmicrosoft commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r468672545
##
File path: ansible/roles/logstash/templates/logstash-simple-2.conf
##
@@ -0,0 +1,150 @@
+#
+# Licensed to the Apache Software Foundation (A
plainolneesh commented on pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#issuecomment-671948965
@arvindshmicrosoft thank you so much for your assistance on this issue. I
will be writing a brief readme for using the ELK stack and hopefully it will
help you and others
plainolneesh commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r468579987
##
File path: lib/muchos/existing.py
##
@@ -72,6 +72,8 @@ def sync(self):
print("- import_playbook: mesos.yml", file=site_file)
plainolneesh commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r468579221
##
File path: README.md
##
@@ -227,6 +227,8 @@ to this swarm. When this is set, docker will be installed
on all nodes of the cl
recommended that
plainolneesh commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r468576463
##
File path: ansible/wipe.yml
##
@@ -29,7 +29,6 @@
retries: 10
delay: 3
until: delete_task is success
-
- name: "wipe grafana db
milleruntime merged pull request #194:
URL: https://github.com/apache/fluo-website/pull/194
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 t
ctubbsii commented on issue #103:
URL: https://github.com/apache/fluo-website/issues/103#issuecomment-670846809
@keith-turner Was this ever done? 1.2 was released early 2018, and I noticed
this was still open and labeled with a 1.2 milestone.
--
ctubbsii commented on pull request #194:
URL: https://github.com/apache/fluo-website/pull/194#issuecomment-670846502
Oh, and this was automatically updated with `bundle update`.
This is an automated message from the Apache Gi
ctubbsii opened a new pull request #194:
URL: https://github.com/apache/fluo-website/pull/194
This updates kramdown (CVE-2020-14001)
(Note: I checked, and this doesn't actually change any content on our site,
so it's not necessary to regenerate our `asf-site` branch after this change
plainolneesh commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r465877411
##
File path: ansible/roles/logstash/templates/logstash-simple-2.conf
##
@@ -0,0 +1,150 @@
+#
+# Licensed to the Apache Software Foundation (ASF) u
plainolneesh commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r465743476
##
File path: ansible/roles/filebeat/templates/filebeat.reference.yml
##
@@ -0,0 +1,2071 @@
+#
Review comment:
This file just provides a re
ctubbsii merged pull request #250:
URL: https://github.com/apache/fluo-uno/pull/250
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
ctubbsii commented on pull request #250:
URL: https://github.com/apache/fluo-uno/pull/250#issuecomment-668088504
> @ctubbsii do you know if a similar changes in needed in Muchos?
I'm not sure. I don't know enough about Ansible to know whether the step is
already automatically skipped
keith-turner commented on pull request #250:
URL: https://github.com/apache/fluo-uno/pull/250#issuecomment-668084309
@ctubbsii do you know if a similar changes in needed in Muchos?
This is an automated message from the Apach
ctubbsii merged pull request #193:
URL: https://github.com/apache/fluo-website/pull/193
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
ctubbsii commented on pull request #250:
URL: https://github.com/apache/fluo-uno/pull/250#issuecomment-666738110
I noticed this wasn't needed any longer (for 3.3 and later) while testing
#249.
This is an automated message fr
benkelly commented on a change in pull request #193:
URL: https://github.com/apache/fluo-website/pull/193#discussion_r463300771
##
File path: pages/people.md
##
@@ -48,6 +48,7 @@ based on commits.
| [Shan Nawaz](https://github.com/shannawaz) |
[Microsoft][msf
ctubbsii commented on a change in pull request #193:
URL: https://github.com/apache/fluo-website/pull/193#discussion_r463297385
##
File path: pages/people.md
##
@@ -48,6 +48,7 @@ based on commits.
| [Shan Nawaz](https://github.com/shannawaz) |
[Microsoft][msf
ctubbsii opened a new pull request #250:
URL: https://github.com/apache/fluo-uno/pull/250
Prevent unnecessary javax.activation-api step for Hadoop 3.3 and later,
where it is already included in the Hadoop binary.
This is a
benkelly opened a new pull request #193:
URL: https://github.com/apache/fluo-website/pull/193
- for [`apache/fluo-uno#249`](https://github.com/apache/fluo-uno/pull/249)
This is an automated message from the Apache Git Service
ctubbsii merged pull request #249:
URL: https://github.com/apache/fluo-uno/pull/249
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
ctubbsii commented on pull request #249:
URL: https://github.com/apache/fluo-uno/pull/249#issuecomment-666717073
Tested and works.
Thanks for the PR @benkelly . It looks like this is your first contribution
to Fluo. If you wish to be added as a contributor to
https://fluo.apache.org
benkelly commented on pull request #249:
URL: https://github.com/apache/fluo-uno/pull/249#issuecomment-666712215
Thanks @ctubbsii !
Happy for me to merge? 🙂
This is an automated message from the Apache Git Service.
To
benkelly opened a new pull request #249:
URL: https://github.com/apache/fluo-uno/pull/249
- Hadoop v3.3.0 fixes issue such as the issue with Kerberos warnings, see
issue [HADOOP-10848](https://issues.apache.org/jira/browse/HADOOP-10848)
### Warning example:
```
WARNING: An
arvindshmicrosoft commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r462743601
##
File path: ansible/roles/filebeat/tasks/main.yml
##
@@ -0,0 +1,61 @@
+---
+#
+# Licensed to the Apache Software Foundation (ASF) under one
arvindshmicrosoft edited a comment on pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#issuecomment-666128892
@plainolneesh I was able to test this on an Azure cluster. I had to
temporarily fix some of the issues I had flagged in prior review, but after
that the Ansib
arvindshmicrosoft merged pull request #366:
URL: https://github.com/apache/fluo-muchos/pull/366
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abo
arvindshmicrosoft commented on pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#issuecomment-666128892
@plainolneesh I was able to test this on an Azure cluster. I had to
temporarily fix some of the issues I had flagged in prior review, but after
that the Ansible play
arvindshmicrosoft commented on pull request #366:
URL: https://github.com/apache/fluo-muchos/pull/366#issuecomment-665723548
Hi @Manno15 could you verify your commit is based on current master? It
looks like it is not, so ideally just merge current master, resolve the merge
conflicts and p
Manno15 commented on pull request #366:
URL: https://github.com/apache/fluo-muchos/pull/366#issuecomment-665743003
Thank you, I was wondering why some things seemed off. I though I had
previsouly updated my master but that was only local.
-
Manno15 edited a comment on pull request #366:
URL: https://github.com/apache/fluo-muchos/pull/366#issuecomment-665743003
Thank you, I was wondering why some things seemed off. I thought I had
previously updated my master but that was only local.
-
Manno15 opened a new pull request #366:
URL: https://github.com/apache/fluo-muchos/pull/366
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 t
plainolneesh commented on a change in pull request #338:
URL: https://github.com/apache/fluo-muchos/pull/338#discussion_r462309635
##
File path: lib/muchos/config.py
##
@@ -0,0 +1,647 @@
+#
Review comment:
Okay, I have removed this file.
##
File path: ansible/
ctubbsii commented on pull request #361:
URL: https://github.com/apache/fluo-muchos/pull/361#issuecomment-665276814
@plainolneesh I merged in my suggestions. Are you okay with these changes as
is, or is there more you'd like to add in before I accept this PR and merge it?
arvindshmicrosoft merged pull request #365:
URL: https://github.com/apache/fluo-muchos/pull/365
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abo
901 - 1000 of 4120 matches
Mail list logo