diff mbox series

[ovs-dev,4/5] Documentatoin: Update Pacemaker link.

Message ID 20240227-doc-misc-v1-4-3d166cdd7e20@ovn.org
State Accepted
Delegated to: Simon Horman
Headers show
Series Documentation: Copyright, hyperlink and spelling corrections. | expand

Checks

Context Check Description
ovsrobot/apply-robot warning apply and check: warning
ovsrobot/github-robot-_Build_and_Test success github build: passed
ovsrobot/intel-ovs-compilation success test: success

Commit Message

Simon Horman Feb. 27, 2024, 3:36 p.m. UTC
Update link to OCF Resource Agents documentation as the existing link
is broken. Also, use HTTPS.

Broken link flagged by make check-docs

Signed-off-by: Simon Horman <horms@ovn.org>
---
 Documentation/topics/integration.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

0-day Robot Feb. 27, 2024, 4:05 p.m. UTC | #1
Bleep bloop.  Greetings Simon Horman, I am a robot and I have tried out your patch.
Thanks for your contribution.

I encountered some error that I wasn't expecting.  See the details below.


checkpatch:
WARNING: Line is 88 characters long (recommended limit is 79)
#25 FILE: Documentation/topics/integration.rst:198:
<https://clusterlabs.org/pacemaker/doc/2.1/Pacemaker_Administration/html/agents.html>`__

Lines checked: 32, Warnings: 1, Errors: 0


Please check this out.  If you feel there has been an error, please email aconole@redhat.com

Thanks,
0-day Robot
Mike Pattrick Feb. 27, 2024, 5:47 p.m. UTC | #2
On Tue, Feb 27, 2024 at 10:36 AM Simon Horman <horms@ovn.org> wrote:
>
> Update link to OCF Resource Agents documentation as the existing link
> is broken. Also, use HTTPS.
>
> Broken link flagged by make check-docs
>
> Signed-off-by: Simon Horman <horms@ovn.org>

Acked-by: Mike Pattrick <mkp@redhat.com>
Eelco Chaudron Feb. 28, 2024, 9:42 a.m. UTC | #3
On 27 Feb 2024, at 16:36, Simon Horman wrote:

> Update link to OCF Resource Agents documentation as the existing link
> is broken. Also, use HTTPS.
>
> Broken link flagged by make check-docs
>
> Signed-off-by: Simon Horman <horms@ovn.org>

Thanks for checking.

Acked-by: Eelco Chaudron <echaudro@redhat.com>
Simon Horman March 1, 2024, 10:33 a.m. UTC | #4
On Tue, Feb 27, 2024 at 03:36:08PM +0000, Simon Horman wrote:
> Update link to OCF Resource Agents documentation as the existing link
> is broken. Also, use HTTPS.
> 
> Broken link flagged by make check-docs
> 
> Signed-off-by: Simon Horman <horms@ovn.org>

Not without some irony, there is a spelling mistake in
the subject of this patch.

I will plan to fix that when applying this series.

...
diff mbox series

Patch

diff --git a/Documentation/topics/integration.rst b/Documentation/topics/integration.rst
index 79bfece82113..ee83f8d43902 100644
--- a/Documentation/topics/integration.rst
+++ b/Documentation/topics/integration.rst
@@ -195,7 +195,7 @@  stalled.
 manager which can manage a defined set of resource across a set of clustered
 nodes. Pacemaker manages the resource with the help of the resource agents.
 One among the resource agent is `OCF
-<http://www.linux-ha.org/wiki/OCF_Resource_Agents>`__
+<https://clusterlabs.org/pacemaker/doc/2.1/Pacemaker_Administration/html/agents.html>`__
 
 OCF is nothing but a shell script which accepts a set of actions and returns an
 appropriate status code.