Skip to content

HBASE-22395 Document RC voting guidelines in ref guide - #297

Closed
taklwu wants to merge 1 commit into
apache:masterfrom
taklwu:HBASE-22395
Closed

HBASE-22395 Document RC voting guidelines in ref guide#297
taklwu wants to merge 1 commit into
apache:masterfrom
taklwu:HBASE-22395

Conversation

@taklwu

Copy link
Copy Markdown
Contributor

No description provided.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec66Docker mode activated.
_ Prechecks _
+1@author0The patch does not contain any @author tags.
_ master Compile Tests _
+1mvninstall262master passed
0refguide444branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Patch Compile Tests _
+1mvninstall242the patch passed
+1whitespace0The patch has no whitespace issues.
0refguide435patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Other Tests _
+1asflicense14The patch does not generate ASF License warnings.
1528
SubsystemReport/Notes
DockerClient=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/1/artifact/out/Dockerfile
GITHUB PR#297
Optional Testsdupname asflicense refguide
unameLinux e9c3c6c89995 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux
Build toolmaven
Personality/testptch/patchprocess/precommit/personality/provided.sh
git revisionmaster / 03436e6
mavenversion: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
refguidehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/1/artifact/out/branch-site/book.html
refguidehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/1/artifact/out/patch-site/book.html
Max. process+thread count96 (vs. ulimit of 10000)
modulesC: . U: .
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/1/console
Powered byApache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec159Docker mode activated.
_ Prechecks _
+1@author0The patch does not contain any @author tags.
_ master Compile Tests _
+1mvninstall268master passed
0refguide466branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Patch Compile Tests _
+1mvninstall251the patch passed
+1whitespace0The patch has no whitespace issues.
0refguide449patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Other Tests _
+1asflicense17The patch does not generate ASF License warnings.
1663
SubsystemReport/Notes
DockerClient=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/2/artifact/out/Dockerfile
GITHUB PR#297
Optional Testsdupname asflicense refguide
unameLinux 4dc115f7da10 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 10:58:50 UTC 2018 x86_64 GNU/Linux
Build toolmaven
Personality/testptch/patchprocess/precommit/personality/provided.sh
git revisionmaster / ada772a
mavenversion: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
refguidehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/2/artifact/out/branch-site/book.html
refguidehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/2/artifact/out/patch-site/book.html
Max. process+thread count78 (vs. ulimit of 10000)
modulesC: . U: .
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/2/console
Powered byApache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@busbeybusbey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this is a great improvement! thanks!

couple of formatting errors noted.

* CHANGES.md if any
* RELEASENOTES.md (release notes) if any
* Generated API compatibility report
** For what should be compatible please refer the [versioning guideline](https://hbase.apache.org/book.html#hbase.versioning), especially for items with marked as high severity

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this didn't render correctly as a link in the qabot build of the ref guide.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

fixed, thanks for calling out

http://www.apache.org/dist/hbase/KEYS
-o | --output-dir '</path>' directory which has the stdout and stderr of each verification target
----
** If you see any unit test failures, please call out the solo test result and whether it's part of flaky (nightly) tests dashboard. Link to the dashboard is e.g. https://builds.apache.org/view/H-L/view/HBase/job/HBase-Find-Flaky-Tests/job/<branch-name>/lastSuccessfulBuild/artifact/dashboard.html

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this rendered as a link to an incorrect URL. Can you make it an example of a working link with branch-2.2?

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

fixed, I changed to master branch with a note one should change the test branch accordingly such we don't have to change this ref guide often.

@taklwu
taklwuforce-pushed the HBASE-22395 branch 2 times, most recently from d5a2cb6 to 657b37fCompareJune 11, 2019 18:16
@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec80Docker mode activated.
_ Prechecks _
+1@author0The patch does not contain any @author tags.
_ master Compile Tests _
+1mvninstall262master passed
0refguide482branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Patch Compile Tests _
+1mvninstall238the patch passed
+1whitespace0The patch has no whitespace issues.
0refguide430patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Other Tests _
+1asflicense14The patch does not generate ASF License warnings.
1562
SubsystemReport/Notes
DockerClient=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/3/artifact/out/Dockerfile
GITHUB PR#297
Optional Testsdupname asflicense refguide
unameLinux 01229e4429d5 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux
Build toolmaven
Personality/testptch/patchprocess/precommit/personality/provided.sh
git revisionmaster / ada772a
mavenversion: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
refguidehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/3/artifact/out/branch-site/book.html
refguidehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/3/artifact/out/patch-site/book.html
Max. process+thread count76 (vs. ulimit of 10000)
modulesC: . U: .
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/3/console
Powered byApache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec38Docker mode activated.
_ Prechecks _
+1@author0The patch does not contain any @author tags.
_ master Compile Tests _
+1mvninstall259master passed
0refguide743branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Patch Compile Tests _
+1mvninstall250the patch passed
+1whitespace0The patch has no whitespace issues.
0refguide485patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Other Tests _
+1asflicense15The patch does not generate ASF License warnings.
1859
SubsystemReport/Notes
DockerClient=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/4/artifact/out/Dockerfile
GITHUB PR#297
Optional Testsdupname asflicense refguide
unameLinux caa3c373a2f7 4.4.0-143-generic #169~14.04.2-Ubuntu SMP Wed Feb 13 15:00:41 UTC 2019 x86_64 GNU/Linux
Build toolmaven
Personality/testptch/patchprocess/precommit/personality/provided.sh
git revisionmaster / ada772a
mavenversion: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
refguidehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/4/artifact/out/branch-site/book.html
refguidehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/4/artifact/out/patch-site/book.html
Max. process+thread count86 (vs. ulimit of 10000)
modulesC: . U: .
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/4/console
Powered byApache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@taklwu

Copy link
Copy Markdown
ContributorAuthor

I found an extra the and removed it as the last commit.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
0reexec42Docker mode activated.
_ Prechecks _
+1@author0The patch does not contain any @author tags.
_ master Compile Tests _
+1mvninstall283master passed
0refguide466branch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Patch Compile Tests _
+1mvninstall234the patch passed
+1whitespace0The patch has no whitespace issues.
0refguide463patch has no errors when building the reference guide. See footer for rendered docs, which you should manually inspect.
_ Other Tests _
+1asflicense15The patch does not generate ASF License warnings.
1569
SubsystemReport/Notes
DockerClient=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/5/artifact/out/Dockerfile
GITHUB PR#297
Optional Testsdupname asflicense refguide
unameLinux fd6d0f1e519d 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 10:58:50 UTC 2018 x86_64 GNU/Linux
Build toolmaven
Personality/testptch/patchprocess/precommit/personality/provided.sh
git revisionmaster / 934d469
mavenversion: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)
refguidehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/5/artifact/out/branch-site/book.html
refguidehttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/5/artifact/out/patch-site/book.html
Max. process+thread count76 (vs. ulimit of 10000)
modulesC: . U: .
Console outputhttps://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-297/5/console
Powered byApache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@taklwu
taklwu deleted the HBASE-22395 branch June 13, 2019 05:56
@taklwu
taklwu restored the HBASE-22395 branch June 13, 2019 05:56
@taklwu
taklwu deleted the HBASE-22395 branch June 13, 2019 05:56
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@taklwu@Apache-HBase@busbey