Github user rdowner commented on the issue:
https://github.com/apache/brooklyn/pull/20
I'd rather not consider this a documentation fail that needs fixed, but a
build process that needs fixed. Is there a way to supply sensible defaults for
these parameters so that a build from
Github user rdowner commented on the issue:
https://github.com/apache/brooklyn-docs/pull/265
Thanks for your first contribution to Apache Brooklyn!
If you haven't already comnpleted and submitted an ICLA form, please could
you do this. Information can be found
[here](https
Github user rdowner commented on the issue:
https://github.com/apache/brooklyn-server/pull/975
Got it, I understand now. And Jenkins has passed this build. I'll merge.
Thanks Alex!
---
Github user rdowner commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/975#discussion_r206110391
--- Diff: karaf/features/src/main/feature/feature.xml ---
@@ -274,8 +275,40 @@
mvn:org.apache.brooklyn/brooklyn-locations
Github user rdowner commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/975#discussion_r20675
--- Diff: karaf/features/src/main/feature/feature.xml ---
@@ -274,8 +275,40 @@
mvn:org.apache.brooklyn/brooklyn-locations
Github user rdowner commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/975#discussion_r206111336
--- Diff: karaf/features/src/main/feature/feature.xml ---
@@ -274,8 +275,40 @@
mvn:org.apache.brooklyn/brooklyn-locations
Github user rdowner commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/975#discussion_r206110119
--- Diff: karaf/features/src/main/feature/feature.xml ---
@@ -274,8 +275,40 @@
mvn:org.apache.brooklyn/brooklyn-locations
Github user rdowner commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/975#discussion_r206111829
--- Diff: karaf/features/src/main/feature/feature.xml ---
@@ -274,8 +275,40 @@
mvn:org.apache.brooklyn/brooklyn-locations
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn/pull/14
Fix issue in doap_Brooklyn.rdf
As requested, update the DOAP file to correct an error. See [this post on
the mailing
list](https://lists.apache.org/thread.html
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-docs/pull/250
Update GSoC information page for current phase
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/rdowner/brooklyn-docs patch-3
Github user rdowner commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/248#discussion_r174060637
--- Diff: community/gsoc.md ---
@@ -15,10 +15,11 @@ for our project, or look at the [list of project
ideas](http://s.apache.org/gsoc
(note
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-docs/pull/248
Update GSoC information page
Update to note that applications are now open.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/rdowner
Github user rdowner commented on the issue:
https://github.com/apache/brooklyn-server/pull/909
Closing, overtaken by events.
---
Github user rdowner closed the pull request at:
https://github.com/apache/brooklyn-server/pull/909
---
Github user rdowner commented on the issue:
https://github.com/apache/brooklyn-docs/pull/247
Ta @geomacy! :+1:
---
Github user rdowner commented on the issue:
https://github.com/apache/brooklyn-docs/pull/247
Hi @geomacy, this PR is for the `website` branch (which uses Jekyll), not
`master` (which is for the user guide, using gitbook)
---
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-docs/pull/247
Fix home page installation instructions
Remove the Vagrant-based instructions, which are known to be broken in
0.12.0. Replace with .tar.gz instructions.
You can merge this pull request
Github user rdowner commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/246#discussion_r171084409
--- Diff: community/gsoc.md ---
@@ -0,0 +1,60 @@
+---
+layout: website-normal
+title: Google Summer of Code
Github user rdowner commented on the issue:
https://github.com/apache/brooklyn-docs/pull/246
Thanks @tbouron - I'll make the suggested changes and then merge.
---
Github user rdowner commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/246#discussion_r171084434
--- Diff: community/gsoc.md ---
@@ -0,0 +1,60 @@
+---
+layout: website-normal
+title: Google Summer of Code
Github user rdowner commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/246#discussion_r171084383
--- Diff: community/gsoc.md ---
@@ -0,0 +1,60 @@
+---
+layout: website-normal
+title: Google Summer of Code
Github user rdowner commented on the issue:
https://github.com/apache/brooklyn-docs/pull/246
@tbouron would appreciate it if you could give this a review, since you
have also been interacting with potential students - is there anything else
that should be added here? Thanks!
---
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-docs/pull/246
Add page with information about GSoC 2018
Add a page to the "Community" section which describes Google Summer of Code
2018, and answers some frequently asked questions.
You
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-docs/pull/243
Remove non-website stuff from `website` branch
Note that this is a PR against `website`, **not** against `master`. Please
bear this in mind when reviewing, and committer please merge
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-client/pull/66
Fix errors with `br catalog add` when run on Windows
`br catalog add` has a couple of issues when it is run on Windows. These
two commits fix these problems.
You can merge this pull
Github user rdowner commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/926#discussion_r160125927
--- Diff:
locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/pool/MachinePoolPredicates.java
---
@@ -82,40 +83,31 @@ public
Github user rdowner commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/926#discussion_r160127430
--- Diff:
locations/jclouds/src/test/java/org/apache/brooklyn/location/jclouds/JcloudsSshMachineLocationStubbedTest.java
---
@@ -76,29 +76,12
Github user rdowner commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/926#discussion_r160126361
--- Diff:
locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/JcloudsLocation.java
---
@@ -2609,7 +2609,7 @@ protected
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-server/pull/909
Bump jclouds dependency from v2.0.2 to 2.0.3
Updates to use recently released jclouds minor release 2.0.2 to 2.0.3.
Notable fixes for Apache Brooklyn include:
- Support
Github user rdowner commented on the issue:
https://github.com/apache/brooklyn-server/pull/906
I'd be in favour of defining code/doc patterns where we can do this kind of
stuff. Initializers are a bit of an odd case - catalog items such as entities,
policies etc. are all self
Github user rdowner commented on the issue:
https://github.com/apache/brooklyn-server/pull/906
Thanks for the review @ahgittin. There's already a couple of
`CreatePasswordSensorTest`s, so I've added a test for this specific behaviour
addition.
---
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-server/pull/906
CreatePassworkSensor: support "character groups"
Character groups are groups like "capital letters", "numbers", etc. This
option allows a blueprint to requi
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-docs/pull/215
Update Azure ARM documentation for Azure CLI 2.0
The existing instructions are for Azure CLI 1.0. Searching for "Azure CLI"
returns hits for 2.0 so it's likely our users will end u
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-docs/pull/211
Update _azure-ARM.md
Provides information on some additional configuration options specific to
Azue
You can merge this pull request into a Git repository by running:
$ git pull https
Github user rdowner commented on the issue:
https://github.com/apache/brooklyn-server/pull/777
@ahgittin this appears to go against our published deprecation policy and
is known to break some blueprints.
Your PR comments are making references to previous discussions, but it's
Github user rdowner commented on the issue:
https://github.com/apache/brooklyn-server/pull/721
Ready for re-review @drigodwin @aledsage
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user rdowner commented on the issue:
https://github.com/apache/brooklyn-server/pull/721
Thanks for the comments. Sorry for the delay so far, mainly due to
vacation. Having a few problems with writing a good test but will get this PR
updated as soon as I can.
---
If your
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn/pull/12
Update the DOAP ("Description Of A Project") file
Updates the list of releases and the list of maintainers (PMC)
You can merge this pull request into a Git repository by running:
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-docs/pull/193
Update 0.11.x docs - copy from master
*This is a pull request to merge into the 0.11.x branch*
This cherry-picks all commits from master since the 0.11.x branch up to
commit b835145
Github user rdowner commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/178#discussion_r115251215
--- Diff: guide/blueprints/config-files.md ---
@@ -0,0 +1,78 @@
+---
+title: Uploading Script and Configuration Files
+layout: website
Github user rdowner commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/178#discussion_r115236606
--- Diff: guide/blueprints/config-files.md ---
@@ -0,0 +1,78 @@
+---
+title: Uploading Script and Configuration Files
+layout: website
Github user rdowner commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/178#discussion_r115250946
--- Diff: guide/blueprints/config-files.md ---
@@ -0,0 +1,78 @@
+---
+title: Uploading Script and Configuration Files
+layout: website
Github user rdowner commented on the issue:
https://github.com/apache/brooklyn-server/pull/657
This is at risk of being a bike-shed discussion. Let's keep it simple.
* The ASCII art logo is not important. If it is causing debate or trouble,
then just drop completely
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-server/pull/632
Resolve release build failures due to optional JARs
Some testing resources are JARs and cannot be included in the release
build. Tests must be written to gracefully handle the absence
Github user rdowner commented on the issue:
https://github.com/apache/brooklyn-server/pull/629
Code looks good and Jenkins says OK - proceeding to merge
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-library/pull/96
LICENSE updates
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/rdowner/brooklyn-library license-update
Alternatively you can review
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-docs/pull/167
LICENSE updates
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/rdowner/brooklyn-docs license-update
Alternatively you can review
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-dist/pull/90
LICENSE updates
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/rdowner/brooklyn-dist license-update
Alternatively you can review
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn/pull/10
LICENSE updates
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/rdowner/brooklyn license-update
Alternatively you can review and apply
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-docs/pull/149
Add our security advisories to public website
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/rdowner/brooklyn-docs security-update
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-server/pull/483
Use SnakeYAML SafeConstructor by default
Overridable by setting a system property.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user rdowner commented on the issue:
https://github.com/apache/brooklyn-server/pull/276
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user rdowner commented on the issue:
https://github.com/apache/brooklyn-docs/pull/79
@aledsage @drigodwin I've addressed your review comments.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
Github user rdowner commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/79#discussion_r67850130
--- Diff: guide/ops/starting-stopping-monitoring.md ---
@@ -0,0 +1,66 @@
+---
+title: Starting, Stopping and Monitoring
+layout: website
Github user rdowner commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/79#discussion_r67849723
--- Diff: guide/ops/requirements.md ---
@@ -19,6 +19,43 @@ at least 8GB RAM and 100GB disk. The disk is just for
logs, a small amount of pe
any
Github user rdowner commented on the issue:
https://github.com/apache/brooklyn-docs/pull/79
How bizarre! Branch lives on the `rdowner` repository, but I inadvertently
opened the PR under the `richardcloudsoft` user -- and GitHub let me do it??!!
ð
---
If your project is set up
Github user rdowner commented on the pull request:
https://github.com/apache/brooklyn-client/pull/20#issuecomment-221318619
And this month's award "Most Peculiarly Named Pull Request" goes to...
ð
---
If your project is set up for it, you can reply to this emai
Github user rdowner commented on the pull request:
https://github.com/apache/brooklyn-docs/pull/47#issuecomment-212568483
Thanks @sjcorbett, since these changes are already live on the public
website (sourced from the 0.9.0 branch) they should probably be merged first!
---
If your
Github user rdowner commented on the pull request:
https://github.com/apache/brooklyn-dist/pull/21#issuecomment-197870065
@johnmccabe @ahgittin This Vagrantfile is for the release build environment
- it's not John's new Vagrantfile for end users.
---
If your project is set up
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-server/pull/64
Remove DISCLAIMER remnant
Missed one file in my earlier attempt to clean up DISCLAIMERs.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-dist/pull/20
Set RPM option to allow binaries in noarch package
Our RPM package is marked as `noarch`. RPM will complain if it finds
architecture-specific files in `noarch` package - therefore
Github user rdowner commented on the pull request:
https://github.com/apache/brooklyn-dist/pull/13#issuecomment-195388591
Ok I have it working now :smile: the good news is that once I had the RPM,
installing it was dead easy, and I instantly had a working, password-protected
Github user rdowner commented on the pull request:
https://github.com/apache/brooklyn-dist/pull/13#issuecomment-195379075
Ah good point @neykov - I didn't look closely enough at that.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user rdowner commented on the pull request:
https://github.com/apache/brooklyn-dist/pull/13#issuecomment-195374544
Hi @alrick87. I am very overdue with reviewing this, especially as it was
me who was championing an RPM package :flushed:
I'm trying this out but I'm
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-ui/pull/16
Remove Incubator DISCLAIMER
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/rdowner/brooklyn-ui remove-disclaimer
Alternatively you can
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-server/pull/57
Remove Incubator DISCLAIMER file
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/rdowner/brooklyn-server remove-disclaimer
GitHub user rdowner opened a pull request:
https://github.com/apache/brooklyn-dist/pull/17
Remove Incubator DISCLAIMER file
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/rdowner/brooklyn-dist remove-disclaimer
Alternatively
Github user rdowner commented on the pull request:
https://github.com/apache/brooklyn-client/pull/1#issuecomment-193782792
The vote passed so we have a green light to merge. See the [IP Clearance
record](https://incubator.apache.org/ip-clearance/brooklyn-cli.html).
---
If your
Github user rdowner commented on the pull request:
https://github.com/apache/incubator-brooklyn/pull/1170#issuecomment-178608060
@johnmccabe yes our release artifacts are a different repository which is
NOT moving.
---
If your project is set up for it, you can reply to this email
Github user rdowner commented on the pull request:
https://github.com/apache/incubator-brooklyn/pull/1144#issuecomment-173178599
@johnmccabe If our official docs recommend using Vagrant then those files
should be part of the Brooklyn repo IMO. It looks bad if our own documentation
Github user rdowner commented on the pull request:
https://github.com/apache/incubator-brooklyn/pull/1105#issuecomment-164406018
mea culpa for not reviewing this properly...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user rdowner commented on the pull request:
https://github.com/apache/incubator-brooklyn/pull/1105#issuecomment-164405911
fixed in
https://github.com/apache/incubator-brooklyn/commit/1fe871b76ebf13afeeab62772342e72cb70f7113
---
If your project is set up for it, you can reply
Github user rdowner commented on the pull request:
https://github.com/apache/incubator-brooklyn/pull/1105#issuecomment-164404933
fixing this now
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
73 matches
Mail list logo
