Fix Aquilon README #36

guillaume-philippon · 2016-10-06T10:16:44Z

Hi,

This PR is to fix documentation of Aquilon installation.

ned21 · 2016-10-06T10:36:37Z

Notifying @ajf8 @gombasg

gombasg · 2017-03-21T11:19:37Z

The ms.version.addpkg() parameters should not be changed - that would break our setup.

guillaume-philippon · 2017-03-21T11:26:17Z

I remove all non-documentation modification

jrha · 2017-03-21T16:42:47Z

LGTM

guillaume-philippon · 2017-03-21T16:44:52Z

Not sure it s enough to build a working aquilon server. Perhaps, The PR must be a base for installation hand-on and fixe what still miss

jouvin · 2017-03-22T09:50:17Z

March 17 workshop: complete and merge after the hands on session at the workshop

README fixes after March17 worshop hands-on

README: additional cleanups after the workshop hands-on

jrha · 2017-03-23T08:19:21Z

Ideally this should be pulled in to a MS branch for internal review and pushed back to Github from there.

jrha · 2017-03-23T08:20:36Z

README.md


-Run as root to install to a system directory, or run as a normal user
-to install into a user-writable location.
+If the installation is done as root and all the dependencies (liste in `/opt/aquilon/setup.py` have been installed


jrha · 2017-03-23T08:22:29Z

README.md

-Run as root to install to a system directory, or run as a normal user
-to install into a user-writable location.
+If the installation is done as root and all the dependencies (liste in `/opt/aquilon/setup.py` have been installed
+as RPM or using the `pip` command, there is no more installation steps to do and you should be able to run the


there are no more

jrha · 2017-03-23T08:23:09Z

README.md


-An environment has now been setup appropriate for running commands and development.
+ * The `cx_Oracle` install will fail if there is no local Oracle client installed - that's fine.
+ * `cdb` package installed as a RPM will be reported as missing but it should not prevent the installation to work


installation from working

jrha · 2017-03-23T08:25:16Z

README.md


-(ignore warning on administration not setup)
+Be sure to definie properly the domain associated with your realm: it must match your actual


properly definie

jrha · 2017-03-23T08:25:58Z

README.md


-and
+It is recommended not to run the broker as root. This is causing quite a number of problem with


This is causes

jouvin · 2017-03-23T08:39:40Z

@guillaume-philippon I let you fix the typos, it would be easier.

guillaume-philippon · 2017-03-23T09:50:30Z

I m waiting the feedback of @samary and @rhumroux before pushing the typo fix

jouvin · 2017-03-23T21:09:19Z

@guillaume-philippon you forgot to merge my PR to your branch with the cleanup I did yesterday evening... I guess I need to rebase it... I'm checking right now.

- PyCharm and Eclipse project information excluded

jouvin · 2017-03-23T21:06:52Z

README.md

 domain.

 If you don't run as `root`, be sure to create a keytab for the current user.

 ### Create a User to Run the Broker

-It is recommended not to run the broker as root. This is causing quite a number of problem with
+It is recommended not to run the broker as root. This cause quite a number of problem with


jouvin · 2017-03-23T21:20:39Z

PR on your source branch has been rebased on this PR and can be merged.

Git repository initialization clarifications

jouvin · 2017-03-23T21:31:32Z

It seems you need to rebase the PR on the last version of master as there is a conflict with .gitignore.

jouvin · 2017-03-23T21:36:05Z

We still need to fix the Git initialization when you are back and after testing it again. I merge this PR as it is already a good start and we'll fix the last point in a future PR.

ned21 · 2017-03-23T21:37:42Z

@jouvin I don't think you can just merge things onto this branch. These commits will get lost the next time there's a push from our internal repo. We need Gabor and Dave to determine the correct way for us to accept these patches.

jouvin · 2017-03-23T21:53:45Z

@ned21 sorry, I forgot this detail... I really apologize for that. At the same time, we need to expose a better installation documentation so I'd suggest to keep it merged and saved somewhere else (I have it in my fork and clone anyway). These changes should not conflict with MS development as I think the README file has been maintained mainly by us in fact...).

ned21 · 2017-03-23T22:21:26Z

@jouvin Definitely! Dave is out tomorrow but I'll sync up with him next week and we'll start meeting more regularly on the stand up.

- Installation guide updated with aquilon repo README.md from quattor/aquilon#36

- README contents from quattor#36 moved to Quattor web site

- README contents from quattor#36 moved to Quattor web site Fixes quattor#57.

- Installation guide updated with aquilon repo README.md from quattor/aquilon#36

- README contents from quattor#36 moved to Quattor web site Fixes quattor#57. Change-Id: I0a1a67758c478471f0ca4c81b695ea8c489d0cf1

- README contents from #36 moved to Quattor web site Fixes #57. Change-Id: I0a1a67758c478471f0ca4c81b695ea8c489d0cf1 Reviewed-By: Fred Barnes <Fred.Barnes@morganstanley.com>

…aster/by_topic/hlmpp to master * commit '4907f58394411c1865f59e9a87c43745f4b5705c': refresh_user: use a source script instead of a source list Add user type database table and related commands tests/broker/orderedsuite: cleanup import order doc: cat.xml: cleanup and make the doc accurate optparser: improve requirements and conflicts handling dbwrappers: location: add an extended get_locations function

guillaume-philippon force-pushed the docfix branch 3 times, most recently from db033cd to 8dd40d3 Compare October 6, 2016 10:31

guillaume-philippon force-pushed the docfix branch from 8dd40d3 to 850fb39 Compare March 21, 2017 11:25

jrha approved these changes Mar 21, 2017

View reviewed changes

ned21 mentioned this pull request Mar 22, 2017

README files still applicable? #31

Closed

ned21 assigned gombasg Mar 22, 2017

Fix Aquilon README

60878c1

guillaume-philippon force-pushed the docfix branch from 850fb39 to 60878c1 Compare March 22, 2017 10:14

jouvin and others added 3 commits March 22, 2017 16:51

README fixes after March17 worshop hands-on

f4344a5

Merge pull request #1 from jouvin/docfix

6ecaa18

README fixes after March17 worshop hands-on

README: additional cleanups after the workshop hands-on

3c6b6f1

guillaume-philippon force-pushed the docfix branch from c1ff446 to 3a95357 Compare March 22, 2017 16:17

Merge pull request #2 from jouvin/docfix

0516e7e

README: additional cleanups after the workshop hands-on

guillaume-philippon force-pushed the docfix branch from 3a95357 to 0516e7e Compare March 22, 2017 16:24

guillaume-philippon added 2 commits March 22, 2017 17:41

Add knc as dep requirement

6eac7cf

pan_compiler must be the .jar version

d18af2d

jouvin mentioned this pull request Mar 22, 2017

Aquilon documentation update quattor/quattor.github.com#211

Merged

jrha reviewed Mar 23, 2017

View reviewed changes

README.md Outdated

and

It is recommended not to run the broker as root. This is causing quite a number of problem with

Copy link

Member

jrha Mar 23, 2017

Choose a reason for hiding this comment

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

This is causes

Some fixes after Samir feedback

f3ad4f3

jouvin added 2 commits March 23, 2017 22:17

README: Git repo initialization clarifications

2ba2330

.gitignore updated

d79cd86

- PyCharm and Eclipse project information excluded

jouvin requested changes Mar 23, 2017

View reviewed changes

Merge pull request #3 from jouvin/docfix

e720d5d

Git repository initialization clarifications

jouvin approved these changes Mar 23, 2017

View reviewed changes

Merge branch 'upstream' into docfix

9185a64

jouvin merged commit 141c501 into quattor:upstream Mar 23, 2017

ned21 mentioned this pull request Mar 23, 2017

Revert "Fix Aquilon README" #56

Closed

jrha mentioned this pull request Mar 28, 2017

Fix Aquilon README (recreated) #61

Closed

jouvin added a commit to jouvin/quattor.github.com that referenced this pull request Mar 28, 2017

Aquilon documnentation updated based on workshop hands-on

7a108d0

- Installation guide updated with aquilon repo README.md from quattor/aquilon#36

jouvin added a commit to jouvin/aquilon that referenced this pull request Mar 28, 2017

README: updated to be a reference to Quattor web site

dce83d0

- README contents from quattor#36 moved to Quattor web site

jouvin added a commit to jouvin/aquilon that referenced this pull request Mar 28, 2017

README: updated to be a reference to Quattor web site

ac1ceff

- README contents from quattor#36 moved to Quattor web site Fixes quattor#57.

jouvin mentioned this pull request Mar 28, 2017

README: updated to be a reference to Quattor web site #62

Closed

jouvin added a commit to jouvin/aquilon that referenced this pull request Mar 28, 2017

README: updated to be a reference to Quattor web site

8ca0cfc

- README contents from quattor#36 moved to Quattor web site Fixes quattor#57.

jouvin added a commit to jouvin/quattor.github.com that referenced this pull request Mar 28, 2017

Aquilon documnentation updated based on workshop hands-on

9a92f00

- Installation guide updated with aquilon repo README.md from quattor/aquilon#36

jouvin added a commit to jouvin/quattor.github.com that referenced this pull request Mar 30, 2017

Aquilon documnentation updated based on workshop hands-on

35bfbbb

- Installation guide updated with aquilon repo README.md from quattor/aquilon#36

jrha pushed a commit to jrha/quattor.github.com that referenced this pull request Apr 12, 2017

Aquilon documnentation updated based on workshop hands-on

de6d190

- Installation guide updated with aquilon repo README.md from quattor/aquilon#36

jouvin added a commit to jouvin/aquilon that referenced this pull request Apr 27, 2017

README: updated to be a reference to Quattor web site

2d5bdfc

- README contents from quattor#36 moved to Quattor web site Fixes quattor#57. Change-Id: I0a1a67758c478471f0ca4c81b695ea8c489d0cf1

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix Aquilon README #36

Fix Aquilon README #36

guillaume-philippon commented Oct 6, 2016

ned21 commented Oct 6, 2016

gombasg commented Mar 21, 2017

guillaume-philippon commented Mar 21, 2017

jrha commented Mar 21, 2017

guillaume-philippon commented Mar 21, 2017

jouvin commented Mar 22, 2017

jrha commented Mar 23, 2017

jrha Mar 23, 2017

jrha Mar 23, 2017

jrha Mar 23, 2017

jrha Mar 23, 2017

jrha Mar 23, 2017

jouvin commented Mar 23, 2017

guillaume-philippon commented Mar 23, 2017

jouvin commented Mar 23, 2017

jouvin Mar 23, 2017

jouvin commented Mar 23, 2017

jouvin commented Mar 23, 2017

jouvin commented Mar 23, 2017

ned21 commented Mar 23, 2017

jouvin commented Mar 23, 2017

ned21 commented Mar 23, 2017


		(ignore warning on administration not setup)
		Be sure to definie properly the domain associated with your realm: it must match your actual


		and
		It is recommended not to run the broker as root. This is causing quite a number of problem with

Fix Aquilon README #36

Fix Aquilon README #36

Conversation

guillaume-philippon commented Oct 6, 2016

ned21 commented Oct 6, 2016

gombasg commented Mar 21, 2017

guillaume-philippon commented Mar 21, 2017

jrha commented Mar 21, 2017

guillaume-philippon commented Mar 21, 2017

jouvin commented Mar 22, 2017

jrha commented Mar 23, 2017

jrha Mar 23, 2017

Choose a reason for hiding this comment

jrha Mar 23, 2017

Choose a reason for hiding this comment

jrha Mar 23, 2017

Choose a reason for hiding this comment

jrha Mar 23, 2017

Choose a reason for hiding this comment

jrha Mar 23, 2017

Choose a reason for hiding this comment

jouvin commented Mar 23, 2017

guillaume-philippon commented Mar 23, 2017

jouvin commented Mar 23, 2017

jouvin Mar 23, 2017

Choose a reason for hiding this comment

jouvin commented Mar 23, 2017

jouvin commented Mar 23, 2017

jouvin commented Mar 23, 2017

ned21 commented Mar 23, 2017

jouvin commented Mar 23, 2017

ned21 commented Mar 23, 2017