[kudu-CR] docs: Add scripts to automate web site update for releases

[Attila Bukor (Code Review)]

Attila Bukor has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8800 )

Change subject: docs: Add scripts to automate web site update for releases
..


Patch Set 1:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/release_index.md.template
File docs/support/scripts/release_index.md.template:

http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/release_index.md.template@35
PS1, Line 35: 
http://www.apache.org/closer.cgi?filename=kudu/%VERSION%/apache-kudu-%VERSION%.tar.gz=download
> link should be https://archive.apache.org/dist/kudu/%VERSION%/apache-kudu-%
never mind, it seems we need to use this chooser script. It could still be 
https though


http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/release_index.md.template@36
PS1, Line 36: www
> same here, link should point to archive.apache.org instead of www.apache.or
never mind, it seems we need to use www.a.o


http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/release_index.md.template@40
PS1, Line 40: You can use the [KEYS 
file](https://www.apache.org/dist/kudu/KEYS) to verify the included GPG 
signature.
we need to add more detailed instructions on how to verify the integrity: 
https://lists.apache.org/thread.html/9cf2fcf4e919520919f1fc860231d244781cf63ee40067ed6f47494c@%3Cdev.kudu.apache.org%3E

example: https://gerrit.cloudera.org/c/11802/



--
To view, visit http://gerrit.cloudera.org:8080/8800
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If55437e0561f0bf8e5e0ec0ae6e5bfca0311fb7d
Gerrit-Change-Number: 8800
Gerrit-PatchSet: 1
Gerrit-Owner: Mike Percy 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy 
Gerrit-Comment-Date: Fri, 26 Oct 2018 21:11:33 +
Gerrit-HasComments: Yes

[kudu-CR] docs: Add scripts to automate web site update for releases

[Attila Bukor (Code Review)]

Attila Bukor has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8800 )

Change subject: docs: Add scripts to automate web site update for releases
..


Patch Set 1:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/release_index.md.template
File docs/support/scripts/release_index.md.template:

http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/release_index.md.template@35
PS1, Line 35: 
http://www.apache.org/closer.cgi?filename=kudu/%VERSION%/apache-kudu-%VERSION%.tar.gz=download
link should be 
https://archive.apache.org/dist/kudu/%VERSION%/apache-kudu-%VERSION%.tar.gz


http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/release_index.md.template@36
PS1, Line 36:   
([SHA1](https://www.apache.org/dist/kudu/%VERSION%/apache-kudu-%VERSION%.tar.gz.sha),
> this should be SHA512 now
extension is sha512 too instead of sha


http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/release_index.md.template@36
PS1, Line 36: www
same here, link should point to archive.apache.org instead of www.apache.org 
(KEYS should remain at www though as that can be updated unlike the release 
files)



--
To view, visit http://gerrit.cloudera.org:8080/8800
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If55437e0561f0bf8e5e0ec0ae6e5bfca0311fb7d
Gerrit-Change-Number: 8800
Gerrit-PatchSet: 1
Gerrit-Owner: Mike Percy 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy 
Gerrit-Comment-Date: Fri, 26 Oct 2018 19:57:18 +
Gerrit-HasComments: Yes

[kudu-CR] docs: Add scripts to automate web site update for releases

[Attila Bukor (Code Review)]

Attila Bukor has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8800 )

Change subject: docs: Add scripts to automate web site update for releases
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/prepare_release_docs.sh
File docs/support/scripts/prepare_release_docs.sh:

http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/prepare_release_docs.sh@47
PS1, Line 47: for dir in $DOCS_DIRECTORIES; do
the apidocs, cpp-client-api and docs directories in $DEST_DIR should be 
symlinks to $DEST_DIR/$RELEASE_DIR/$VERSION/{apidocs,cpp-client-api,docs} 
instead of copying them to two places



--
To view, visit http://gerrit.cloudera.org:8080/8800
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If55437e0561f0bf8e5e0ec0ae6e5bfca0311fb7d
Gerrit-Change-Number: 8800
Gerrit-PatchSet: 1
Gerrit-Owner: Mike Percy 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy 
Gerrit-Comment-Date: Wed, 24 Oct 2018 23:58:35 +
Gerrit-HasComments: Yes

[kudu-CR] docs: Add scripts to automate web site update for releases

[Attila Bukor (Code Review)]

Attila Bukor has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8800 )

Change subject: docs: Add scripts to automate web site update for releases
..


Patch Set 1:

(2 comments)

Mike, could you also add this to the RELEASING docs after the make_site.sh 
paragraph?

http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/prepare_release_docs.sh
File docs/support/scripts/prepare_release_docs.sh:

http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/prepare_release_docs.sh@68
PS1, Line 68: git commit -m "Updating web site for Kudu $VERSION release"
> It assumes you already did that. If we want it to do that we could, with th
I think it would be a good idea to document these kind of stuff. It doesn't 
seem straight-forward. Also, the docs could potentially mention that git 
worktree can be used to check out the gh-pages branch instead of multiple 
clones of the repo (Todd mentioned it the other day and it's a game changer)


http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/release_index.md.template
File docs/support/scripts/release_index.md.template:

http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/release_index.md.template@37
PS1, Line 37:   
[MD5](https://www.apache.org/dist/kudu/%VERSION%/apache-kudu-%VERSION%.tar.gz.md5),
MD5 should be removed too



--
To view, visit http://gerrit.cloudera.org:8080/8800
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If55437e0561f0bf8e5e0ec0ae6e5bfca0311fb7d
Gerrit-Change-Number: 8800
Gerrit-PatchSet: 1
Gerrit-Owner: Mike Percy 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy 
Gerrit-Comment-Date: Wed, 24 Oct 2018 21:09:27 +
Gerrit-HasComments: Yes

[kudu-CR] docs: Add scripts to automate web site update for releases

[Attila Bukor (Code Review)]

Attila Bukor has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8800 )

Change subject: docs: Add scripts to automate web site update for releases
..


Patch Set 1:

> Patch Set 1:
>
> Attila if you want you can use this for the 1.8 release regardless of whether 
> it makes it into the branch, I think.

thanks, I'll definitely try it


--
To view, visit http://gerrit.cloudera.org:8080/8800
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If55437e0561f0bf8e5e0ec0ae6e5bfca0311fb7d
Gerrit-Change-Number: 8800
Gerrit-PatchSet: 1
Gerrit-Owner: Mike Percy 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy 
Gerrit-Comment-Date: Fri, 19 Oct 2018 08:36:17 +
Gerrit-HasComments: No

[kudu-CR] docs: Add scripts to automate web site update for releases

[Attila Bukor (Code Review)]

Attila Bukor has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8800 )

Change subject: docs: Add scripts to automate web site update for releases
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/release_index.md.template
File docs/support/scripts/release_index.md.template:

http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/release_index.md.template@36
PS1, Line 36:   
([SHA1](https://www.apache.org/dist/kudu/%VERSION%/apache-kudu-%VERSION%.tar.gz.sha),
this should be SHA512 now



--
To view, visit http://gerrit.cloudera.org:8080/8800
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If55437e0561f0bf8e5e0ec0ae6e5bfca0311fb7d
Gerrit-Change-Number: 8800
Gerrit-PatchSet: 1
Gerrit-Owner: Mike Percy 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy 
Gerrit-Comment-Date: Fri, 19 Oct 2018 08:35:57 +
Gerrit-HasComments: Yes

[kudu-CR] docs: Add scripts to automate web site update for releases

[Mike Percy (Code Review)]

Mike Percy has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8800 )

Change subject: docs: Add scripts to automate web site update for releases
..


Patch Set 1:

Attila if you want you can use this for the 1.8 release regardless of whether 
it makes it into the branch, I think.


--
To view, visit http://gerrit.cloudera.org:8080/8800
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If55437e0561f0bf8e5e0ec0ae6e5bfca0311fb7d
Gerrit-Change-Number: 8800
Gerrit-PatchSet: 1
Gerrit-Owner: Mike Percy 
Gerrit-Reviewer: Attila Bukor 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 
Gerrit-Comment-Date: Wed, 10 Oct 2018 22:22:33 +
Gerrit-HasComments: No

[kudu-CR] docs: Add scripts to automate web site update for releases

[Mike Percy (Code Review)]

Mike Percy has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8800 )

Change subject: docs: Add scripts to automate web site update for releases
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/prepare_release_docs.sh
File docs/support/scripts/prepare_release_docs.sh:

http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/prepare_release_docs.sh@68
PS1, Line 68: git commit -m "Updating web site for Kudu $VERSION release"
> Should this change gerrit change be in the gh-pages branch? So you can run
It assumes you already did that. If we want it to do that we could, with the 
caveat that we would also need to set up the gerrit git trigger, but I'm not 
sure the best way to do that. Maybe we could find the current gerrit trigger 
and replicate it in the checked out version of the gh-pages tree?



--
To view, visit http://gerrit.cloudera.org:8080/8800
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If55437e0561f0bf8e5e0ec0ae6e5bfca0311fb7d
Gerrit-Change-Number: 8800
Gerrit-PatchSet: 1
Gerrit-Owner: Mike Percy 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 
Gerrit-Comment-Date: Tue, 10 Apr 2018 20:58:17 +
Gerrit-HasComments: Yes

[kudu-CR] docs: Add scripts to automate web site update for releases

[Grant Henke (Code Review)]

Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8800 )

Change subject: docs: Add scripts to automate web site update for releases
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/prepare_release_docs.sh
File docs/support/scripts/prepare_release_docs.sh:

http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/prepare_release_docs.sh@68
PS1, Line 68: git commit -m "Updating web site for Kudu $VERSION release"
> Sorry about that. Exactly right.
Should this change gerrit change be in the gh-pages branch? So you can run 
make_site.sh in a release branch, checkout to gh-pages and run this?



--
To view, visit http://gerrit.cloudera.org:8080/8800
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If55437e0561f0bf8e5e0ec0ae6e5bfca0311fb7d
Gerrit-Change-Number: 8800
Gerrit-PatchSet: 1
Gerrit-Owner: Mike Percy 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 
Gerrit-Comment-Date: Thu, 22 Mar 2018 18:13:10 +
Gerrit-HasComments: Yes

[kudu-CR] docs: Add scripts to automate web site update for releases

[Mike Percy (Code Review)]

Mike Percy has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8800 )

Change subject: docs: Add scripts to automate web site update for releases
..


Patch Set 1:

(1 comment)

> Patch Set 1:
>
> (1 comment)
>
> Could you update RELEASING.adoc to describe how this should be used?

http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/prepare_release_docs.sh
File docs/support/scripts/prepare_release_docs.sh:

http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/prepare_release_docs.sh@68
PS1, Line 68: git commit -m "Updating web site for Kudu $VERSION release"
> This needs to be in the gh-pages branch right? Does the script assume you h
Sorry about that. Exactly right.



--
To view, visit http://gerrit.cloudera.org:8080/8800
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If55437e0561f0bf8e5e0ec0ae6e5bfca0311fb7d
Gerrit-Change-Number: 8800
Gerrit-PatchSet: 1
Gerrit-Owner: Mike Percy 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 
Gerrit-Comment-Date: Thu, 22 Mar 2018 17:51:17 +
Gerrit-HasComments: Yes

[kudu-CR] docs: Add scripts to automate web site update for releases

[Grant Henke (Code Review)]

Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8800 )

Change subject: docs: Add scripts to automate web site update for releases
..


Patch Set 1:

(1 comment)

Could you update RELEASING.adoc to describe how this should be used?

http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/prepare_release_docs.sh
File docs/support/scripts/prepare_release_docs.sh:

http://gerrit.cloudera.org:8080/#/c/8800/1/docs/support/scripts/prepare_release_docs.sh@68
PS1, Line 68: git commit -m "Updating web site for Kudu $VERSION release"
This needs to be in the gh-pages branch right? Does the script assume you have 
2 clones of Kudu and source is on the release branch and dest is on gh-pages?



--
To view, visit http://gerrit.cloudera.org:8080/8800
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If55437e0561f0bf8e5e0ec0ae6e5bfca0311fb7d
Gerrit-Change-Number: 8800
Gerrit-PatchSet: 1
Gerrit-Owner: Mike Percy 
Gerrit-Reviewer: Dan Burkert 
Gerrit-Reviewer: Grant Henke 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy 
Gerrit-Comment-Date: Thu, 22 Mar 2018 17:00:23 +
Gerrit-HasComments: Yes

[kudu-CR] docs: Add scripts to automate web site update for releases

[Mike Percy (Code Review)]

Mike Percy has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/8800


Change subject: docs: Add scripts to automate web site update for releases
..

docs: Add scripts to automate web site update for releases

These scripts improve the web site deployment process for releases by
automatically placing the generated documentation in the correct
locations, updating the pointers to the new release, and creating a git
commit for all of the above.

Updated release management documentation to follow.

Change-Id: If55437e0561f0bf8e5e0ec0ae6e5bfca0311fb7d
---
A docs/support/scripts/prepare_release_docs.sh
A docs/support/scripts/release_index.md.template
A docs/support/scripts/update_release_list.pl
3 files changed, 181 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/00/8800/1
--
To view, visit http://gerrit.cloudera.org:8080/8800
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: If55437e0561f0bf8e5e0ec0ae6e5bfca0311fb7d
Gerrit-Change-Number: 8800
Gerrit-PatchSet: 1
Gerrit-Owner: Mike Percy