From a24a0fa421ba9bc65446fbda4c964fc5bf48b019 Mon Sep 17 00:00:00 2001
From: Ivan Anfimov <lazekteam@gmail.com>
Date: Wed, 10 Sep 2025 23:20:37 +0000
Subject: [PATCH] docs: remove outdated information about Vagrant

and fix for url in contributors doc

Change-Id: I4c343041ea0f03f6ba8ac7dbcd0e234b41c9de89
Signed-off-by: Ivan Anfimov <lazekteam@gmail.com>
---
 doc/source/contributors/code-rules.rst | 21 ++++-----------------
 zuul.d/jobs.yaml                       |  1 -
 2 files changed, 4 insertions(+), 18 deletions(-)

diff --git a/doc/source/contributors/code-rules.rst b/doc/source/contributors/code-rules.rst
index 4b1a739ab7..6c1ecc2f78 100644
--- a/doc/source/contributors/code-rules.rst
+++ b/doc/source/contributors/code-rules.rst
@@ -26,7 +26,7 @@ General Guidelines for Submitting Code
   should always aim to result in a working build throughout the dependency
   chain. Documentation is available for `advanced gerrit usage`_ too.
 * All patch sets should adhere to the :ref:`Ansible Style Guide` listed here as
-  well as adhere to the `Ansible best practices`_ when possible.
+  well as adhere to the `Ansible playbooks`_ when possible.
 * All changes should be clearly listed in the commit message, with an
   associated bug id/blueprint along with any extra information where
   applicable.
@@ -43,9 +43,9 @@ General Guidelines for Submitting Code
   for more information.
 
 .. _Git Commit Good Practice: https://wiki.openstack.org/wiki/GitCommitMessages
-.. _workflow documented here: https://docs.openstack.org/infra/manual/developers.html#development-workflow
-.. _advanced gerrit usage: https://www.mediawiki.org/wiki/Gerrit/Advanced_usage
-.. _Ansible best practices: https://docs.ansible.com/playbooks_best_practices.html
+.. _workflow documented here: https://docs.opendev.org/opendev/infra-manual/latest/developers.html#development-workflow
+.. _advanced gerrit usage: https://docs.openstack.org/contributors/code-and-documentation/using-gerrit.html
+.. _Ansible playbooks: https://docs.ansible.com/ansible/latest/playbook_guide/playbooks_intro.html
 .. _the reno tool: https://docs.openstack.org/reno/latest/
 
 .. _documentation:
@@ -338,17 +338,6 @@ include at least the following:
   be overridden by our proposal bot should any change happen.
 * ``setup.cfg`` and ``setup.py``:
   Information about the repository used when building artifacts.
-* ``run_tests.sh``:
-  A script for developers to execute all standard tests on a
-  suitable host. This must be copied from the
-  ``openstack-ansible-tests`` repository and will be automatically
-  be overridden by our proposal bot should any change happen.
-* ``Vagrantfile``:
-  A configuration file to allow a developer to easily create a
-  test virtual machine using `Vagrant`_. This must automatically execute
-  ``run_tests.sh``. This must be copied from the
-  ``openstack-ansible-tests`` repository and will be automatically
-  be overridden by our proposal bot should any change happen.
 * ``README.rst``, ``LICENSE``, ``CONTRIBUTING.rst``:
   A set of standard files whose content is self-explanatory.
 * ``.gitignore``:
@@ -365,8 +354,6 @@ include at least the following:
 Please have a look at a role like os_cinder, os_keystone, or os_neutron
 for latest files.
 
-.. _Vagrant: https://www.vagrantup.com/
-
 Container technology independence
 ---------------------------------
 
diff --git a/zuul.d/jobs.yaml b/zuul.d/jobs.yaml
index 4a2b5cdc56..c5ba95ce52 100644
--- a/zuul.d/jobs.yaml
+++ b/zuul.d/jobs.yaml
@@ -36,7 +36,6 @@
       - ^releasenotes/.*
       - ^setup\.(cfg|py)$
       - ^tox.ini$
-      - ^Vagrantfile
     vars:
       #disable powertools/highavailabity for centos
       configure_mirrors_extra_repos: "{{ ansible_facts['os_family'] | lower != 'redhat' }}"