Commit Graph

50 Commits

Author SHA1 Message Date
ArkadyKanevsky de96ee2d12 Reorganize guidelines and improve consistency.sh
Reorganize guidelines into guidelines directory and create
current_guideline that is softlink to the latest approved guideline.
The same with add-ons guidelines - they are moved to guidelines
directory within add-ons one and soft links are created pointing
to the latest guidelines for each add-on.

Also cleaned up some tooling that hardwired where guidelines lived.

Change-Id: I5ad4b91b1afb44a0a6987b339f7efba14f395302
2021-06-29 22:16:59 +00:00
Martin Kopec c730eee368 Revert "Reorganize guidelines and improve consistency.sh"
This reverts commit a9fbc452a6.

Reason for revert: refstack server gives 404 on the guidelines: https://refstack.openstack.org/#/guidelines .. seems like https://review.opendev.org/c/osf/refstack/+/790940 didn't handle the update of the guidelines location everywhere - I suspect that some changes in refstack-ui are needed as well, ah

Change-Id: I2685eb8fccb4dbdcf42683f168ee2cb316137196
2021-05-25 13:27:13 +00:00
ArkadyKanevsky a9fbc452a6 Reorganize guidelines and improve consistency.sh
Reorganize guidelines into previous_guideline directory
and current_guideline that is softlink to the latest approved guideline.
Cleaned up some tooling that hardwired where guidelines lived.

Change-Id: Ia6be9ca6326718488ee5668df3806da5f76dc456
2021-05-23 10:02:02 +00:00
Artem Goncharov 6a5adfc915 Replace git.openstack.org URLs with opendev.org URLs
Change-Id: I16040e384b553ffb7d19dcda2363f3f07ce7855e
2019-05-08 12:20:31 +00:00
Mark T. Voelker 02698b5411 Fix "superceded" typo in schema
Clouds are hard.  English, doubly so.  We have a longstanding
spelling error in the schema: "superceded" should be spelled
"superseded".  This patch corrects it and updates all the documents
the typo was propagated to over time.

Change-Id: I72f24bf026919f4ccce0ce0c42e1c33c4042aca4
2017-09-12 09:38:36 -06:00
Mark T. Voelker c3e9b6b4b2 Approve 2017.01, superceding 2016.01
This patch should be approved only if and when the OpenStack
Foundation Board of Directors votes to approve the proposed 2017.01
Guideline.  It does the following:

1.) Marks 2017.01 as "approved."

2.) Marks 2016.01 as "superceded."

3.) Adjusts the "replaces" target of next.json and it's target
    approval date to reflect work begining on the next Guideline.
    Note that the Board of Directors meeting list for 2017[1]
    hasn't yet been set, so the date listed here is approximate.

[1]
https://wiki.openstack.org/wiki/Governance/Foundation#OpenStack_Board_of_Director_Meetings

Change-Id: I566e52fdc82af57a5bf24bfd3db5d10a26887127
2017-01-23 23:45:06 -05:00
Ken'ichi Ohmichi 91f1cc2f4e Fix the project name of images-v2-index
The project of images-v2-index should be glance, not nova.
This patch fixes it.

Change-Id: I3cfe851139f7a4058f1821185c983ea3971dc071
2017-01-16 09:34:02 -08:00
ghanshyam 842e46c78a Flag/Replace volume/snapshot update metadata test
Commit I8d9296f958e17d20a39a05800ac1808347d952bd, removed
this test as it is being merged to CRUD tests.

This patch flag the update metadata tests and also remove
from next guidlines.

Change-Id: Id9a53c13eb501a131d4bab48414dd8deecc17d90
Depends-On: I8d9296f958e17d20a39a05800ac1808347d952bd
2016-12-09 11:21:12 +09:00
ghanshyam c0a945ef88 Add aliases for test_create_get_delete_volume_metadata
Commit I8d9296f958e17d20a39a05800ac1808347d952bd , changed the
name of the tests on Tempest side. So this patch addes aliases.

Change-Id: I4707b4956c2b92dc76cf4720016e737edd21b8b3
Depends-On: I8d9296f958e17d20a39a05800ac1808347d952bd
2016-12-06 23:49:21 +00:00
ghanshyam 2ef7d49c01 Add aliases for test_create_get_delete_snapshot_metadata
Commit I8d9296f958e17d20a39a05800ac1808347d952bd , changed the
name of the tests on Tempest side. So this patch addes aliases.

Change-Id: I4d4432bc1d2fd8b1c68cd6d6183bc456ce3b4ff4
Depends-On: I8d9296f958e17d20a39a05800ac1808347d952bd
2016-12-05 00:50:51 +00:00
Ken'ichi Ohmichi 578ce077ff Add aliases for test_list_servers_filter_by_server_status
Since Ic19fd7bd4f0afd6a551133d7db96737487b52663 , the name has been
renamed on Tempest side. So this patch addes aliases.

Change-Id: Ie929b481d8989433922ab960d11590aba31e8374
Depends-On: Ic19fd7bd4f0afd6a551133d7db96737487b52663
2016-11-23 19:16:42 -08:00
Jenkins 8170d87c3b Merge "Flag/Replace a compute-volume-get test" 2016-10-20 08:30:58 +00:00
Ken'ichi Ohmichi d1136f5a5d Flag/Replace a compute-volume-get test
Since I7c99f61dc54ef7c437c7e4574eba935d76ef9318, QA team has removed
test_get_invalid_volume_id from Tempest because the test is duplicated
with test_volume_get_nonexistent_volume_id.

Then this patch changes the compute-volume-get test.

Change-Id: I54f9924c1c007ea2d34b7ef502df8edf8032549e
2016-10-19 10:44:44 -07:00
Ghe Rivero a7e9c5f867 Fix idempotent id names
Some cleanup of some idempotent ids that
were not properly specified

Change-Id: I9c910c96aebc94346f483f22165a88c03af8080c
2016-10-06 09:54:57 +02:00
Jenkins 0abe4b8c15 Merge "Add aliases for test_list_servers_filter_by_server_status" 2016-10-04 17:33:40 +00:00
Ken'ichi Ohmichi fb506afa93 Add aliases for test_list_servers_filter_by_server_status
Since Ieab05d793039cea1d4bb14e91ea0f4cbea3df02b , the name has been
renamed on Tempest side.
So this patch addes aliases.

Change-Id: Ibd8b3f2494046b194d77e9d563ebecef3a883b8f
2016-09-29 17:24:05 -07:00
Catherine Diep a4fd7fedaf Add aliases for test_snapshots_list_* test.
Tempest patch [1] will move these tests to a new file. This will
change the fully qualified test names.

[1] https://review.openstack.org/#/c/332066/

Change-Id: I3262f7414629d460b3660191e7f0c04198e22dd2
2016-09-29 11:59:13 -07:00
Catherine Diep 7b9b53e3bf Flag advisory tests in 2016.01 due to requirement of admin credential.
networks-l3-CRUD: a few tests in this capability still require
admin credential after refactoring [1]. More information about
DefCore tests that require admin credential can be found in [2].
volumes-v2-qos: All tests in this capability require admin
credential [3].
volumes-v2-transfer: All tests in this capability require admin
credential and multi-tenancy [4].

[1] https://review.openstack.org/#/c/335558/
[2] https://review.openstack.org/#/c/299491/
[3] https://review.openstack.org/#/c/322250/
[4] https://review.openstack.org/#/c/322253/

Change-Id: If9fcf517e4091559c1dc597cdd60fdc18201b66b
2016-08-10 00:33:17 -07:00
Mark T. Voelker 4f210354bc Flag/drop test_create_update_port_with_second_ip
In 2016.01 we introduced a new advisory test that involves creating
a Neutron port with a fixed IP address and then updating it to add
a second IP address to the same port.  However, the nunber of fixed
IP addresses that can be assigned to a port is dependent on the
plugin that Neutron is using--and not all plugins support more than
one IP address per port (the maximum number of IP's per port is
configurable in neutron.conf and clouds whose drivers do not support
multiple fixed addresses per port simply set it to 1).  Further,
some clouds limit the number of IP's per port for security reasons
to prevent malicious activity (e.g. spoofguard).

This patch adds an informational flag to the 2016.01 guideline
(where the test is advisory) and removes it from next.json.

Change-Id: Ie7d84d5d608bfc86501a5f7e097373c4565e4eb0
2016-07-15 18:11:13 -07:00
Jenkins 2d4f9649ed Merge "Flag/drop snapshot_create_with_volume_in_use test" 2016-07-15 01:07:56 +00:00
Jenkins ddcfc7a468 Merge "Flag Compute tests which depend on the os-networks extension" 2016-07-14 23:39:11 +00:00
Daryl Walleck b17276497f Flag Compute tests which depend on the os-networks extension
Certain Compute tests [1] require the fixed_network_name
configuration to be set or for a provider network to be
defined. Setting either of these triggers a code path
that uses the os-networks extension to look up the
network's id. If this extension is not enabled, then
any test that creates a server will fail. The credential
provider needs additional logic to handle the case
where this extension is not enabled.

[1] https://github.com/openstack/tempest/blob/master/tempest/api/compute/servers/test_list_server_filters.py#L270

Change-Id: I34b8cc50f35d7207d12c73f980966066eddbcf81
Related-Bug: #1592173
Related-Bug: #1600349
2016-07-14 17:50:20 -05:00
Mark T. Voelker 7e2bb629af Flag/drop snapshot_create_with_volume_in_use test
In 2016.01 we added a new advisory capability:
volumes-v2-snapshot-create-delete.  One of the tests for that Capability
involves creating a volume, attaching it to an instance, and then
creating a snapshot of the volume while it's in-use.  This is
potentially problematic for two reasons: first, it requires the use of
the force flag, which wasn't introduced until Liberty[1].  Kilo is still
covered until 2016.01 and 2016.08.  Second, snapshotting an in-use
volume is an operation which may produce corrupted data (which isn't
always well understood [2]) and therefore seems like a thing that some
clouds may not wish to offer to their clients.

Since the force flag wasn't available in all the releases covered by
the 2016.01 and 2016.08 Guidelines, we should probably wait to include
this test until a later release.  Waiting would also give us some time
to see if it's something clouds do indeed expose to their users...my
guess is that it is, but I could see why some clouds might not.

[1]
http://docs.openstack.org/admin-guide/blockstorage_volume_backups.html
[2]
https://support.rackspace.com/white-paper/allowing-snapshots-without-detaching-the-volumes/

Change-Id: I9dab755bde2f169d52503d47fcfca404b8777f77
2016-07-14 11:30:36 -07:00
Jenkins af581ea799 Merge "Flag network port tests that require admin credentials in 2016.01." 2016-07-13 16:37:53 +00:00
Jenkins f70240de66 Merge "Update gate tests to capture 2015.07 and 2016.01" 2016-07-13 15:13:04 +00:00
Jenkins 08de1db24d Merge "Update test cases corresponding to the class(execution path)" 2016-07-06 16:43:18 +00:00
Chris Hoge fcb533a6eb compute-image flags were not carried to 2015.07
Between 2015.05 and 2016.01 the compute-image flags were not
carried forward. This patch reconciles the lost flags and fixes
other errors in the existing guidelines, as well as next.

Partial-Bug: 1585773
Partial-Bug: 1599563
Change-Id: Ie405d3f24e0754a99f274aedcbd6b56adabaa5b6
2016-07-06 08:51:23 -07:00
Chris Hoge cc4ae436a1 Update gate tests to capture 2015.07 and 2016.01
Following the July 2016 board meeting, add minor update to guidelines
to move them to version 1.5 of the DefCore schema and add json
schema validation to gate job.

Change-Id: Iac667f8a87b50a15f4dfe13c1ef69efa61c10059
2016-06-29 09:52:22 -07:00
xuchao 62e3c49d69 Update test cases corresponding to the class(execution path)
Change-Id: Ic443bdc4cd5ce80029271cefbd7bef505847eb48
Closes-Bug: #1592239
2016-06-15 17:09:56 +08:00
Catherine Diep c295d6b603 Flag network port tests that require admin credentials in 2016.01.
The targeted four tests are derived from a class
(PortsAdminExtendedAttrsTestJSON) that requires admin credentials [1].

[1] https://github.com/openstack/tempest/blob/master/tempest/api/network/test_ports.py#L351

Change-Id: I916f31b25410f25a88ea7790d69f392fa34051ad
2016-06-02 14:17:29 -07:00
Chris Hoge 8d9630c939 Fix guideline errors
Fix errors in 2015.07.json, 2016.01.json, and next.json
guidelines. Fixes include removal of trailing commas,
changing empty lists to empty sets for consistency across
designated sections, and removal of duplicate test entries
from capabilities.

Co-Authored-By: Catherine Diep <cdiep@us.ibm.com>
Change-Id: Ib1a046b732f3807cabfd0c448a2cbf698a5cf44c
2016-06-02 00:21:12 -07:00
Jenkins 08b31990d4 Merge "Close coverage gap after new OpenStack releases" 2016-04-24 15:18:20 +00:00
Mark T. Voelker fbd9c0dccc Add cutoff_score to next.json, update script
We recently added a field in Schema 1.4 to record a
"cutoff_score".[1]  The field reflects the minimum score that a
Capability needs to achieve for a given Guideline in order to
warrant inclusion.  The current draft Guideline, however, hasn't yet
 been updated to include a cutoff score.

This patch adds a cutoff_score field to next.json and sets it to
a preliminary vale of 74 (which is indicative of the score we've
used in the past as a cutoff, but which can be changed by the
DefCore Committee any time between now and when the Guideline goes to
the Board for approval before it becomes binding).  The patch also
updates the tabulate_scores.py script to read the cutoff_score from the
JSON document and adds an asterisk (*) to lines in the scoring file whose
totals meet or exceed the cutoff as a visual indicator of which
Capabilities have scored highly enough to warrant inclusion in the
Guideline.  Verbage in the header of the scoring.txt file has also been
updated to reflect the meaning of the indicator.

The patch also adds the cutoff_score field to the 2016.01 document,
where it was inadvertently left off since the field was only added
to the schema in close proximity to the document being finalized for
Board approval.

[1] https://review.openstack.org/#/c/224868/

Change-Id: I73d08baae13f373c955ff94640fac43109125886
2016-04-22 12:26:37 -04:00
Chris Hoge 1acbbb2006 Flag tests scheduled for removal from Tempest
Flag tests that have been removed from the Tempest test
suite[1]. Capabilities are covered by other tests. Tests
to be permanently removed from future guidelines.

compute-auth-* capabilities removed because of empty
test sets.

[1] https://review.openstack.org/#/c/271467

Change-Id: Ib6b8ca03f39c7590163a9db710e5ca8aa373d659
2016-04-21 16:05:45 -07:00
Mark T. Voelker 9d2183c949 Close coverage gap after new OpenStack releases
Currently, each DefCore Guideline covers the most recent three
releases of the OpenStack Software at the time of the Guideline's
approval (which is generally 3-4 months after the release of a
new version of the software).  This timing means that there's a
3-4 month period during which products built on the most recent
OpenStack release may not be able to apply for an OpenStack
Powered(TM) license agreement because the version of OpenStack on
which they are built isn't covered by a Board-approved Guideline.

Whereas the OpenStack community and Foundation have an expressed
interest in enabling users to adopt new software quickly, this
patch closes the 3-4 month window by amending the 2016.01
Guideline (and future Guidelines) to allow it to cover the most
recent version of the software starting on the day that the
software is released.

Note that because this change entails a modification to a Board-
approved Guideline document, this change should not merge until
after the Board of Directors has approved the change.  The
matter will be discussed during the 2016-04-24 Board meeting.

Change-Id: I6db11762e48ff3297eb9f90c93fb68717b0f3d43
2016-04-18 12:34:16 -04:00
Chris Hoge 184ff3e19e Flag tests requiring multiple tenants and users
Guiding core priciple number 7 from the DefCore core
definition is [1]:

"7. Tests can be remotely or self-administered"

Tests that require multiple tenants and users
create a barrier to remote adminstration and
independent verification of test results. Requiring
users to enroll for multiple tenant-isolated accounts
on a public cloud creates a resource cost that
may be insurmountable for end users, especially
when comparing multiple public clouds.

In addition, tenant isolation is less a capability
for writing interoperable applications, and more
a security concern for cloud implementers. While
important, the goal of interopearbility is
orthgonal to the security goals of tenant isolation.

The flagged tests in this patch require multiple
tenant-isloated accounts, and are marked to
to be fixed where possible, or removed where not
possible.

This list was determined by successfully running
the defcore suite against a test cloud[2], then
re-running the tests configured with only one
account against the same test cloud[3].

At the January QA meetup, this test list was
refined by refactoring Server Negative tests
into two classes, one that requires multi-tenant
access, one one that does not[4]. This significantly
reduced the number of flagged tests.[5]

[1] https://github.com/openstack/defcore/blob/master/doc/source/process/CoreDefinition.rst
[2] https://refstack.openstack.org/#/results/0b8ac51d-3288-42d8-b276-4a54046e41bd
[3] https://refstack.openstack.org/#/results/01f29358-8870-4e15-8f7a-f6a082d44e29
[4] https://review.openstack.org/#/c/283819/
[5] https://etherpad.openstack.org/p/multi-tenant-defcore-tests

Change-Id: I5ec801ae48e8220abdaecebe43e77465684e8b5b
2016-04-08 09:57:40 -07:00
Catherine Diep 235ecf7d2b Flag Tempest skipped test.
Flag test "test_list_servers_filtered_by_ip_regex" which was skipped
by Tempest on Feb 2, 2016 [1] due to bug [2].

[1] https://review.openstack.org/#/c/274907/
[2] https://bugs.launchpad.net/tempest/+bug/1540645

Change-Id: Ie41ee2a8be54fd255b01301d9545eece0d8b3753
2016-03-31 14:05:54 -07:00
Catherine Diep 7c25323523 Add aliases to handle test name or test path changes.
Add aliases to the "test_nework" tests to handle test path name
update as described in [1].
Add alias to the "test_create_router_setting_tenant_id" test due
to test name changed [2].
Add test python file name ("test_api_discovery") to the path of
Keystone API discorvery tests.

[1] https://review.openstack.org/#/c/153621/
[2] https://review.openstack.org/#/c/214987/

Change-Id: I482f0b1ad6b07549aba8f2ba13e4c1a32c70c72b
2016-03-30 10:18:28 -07:00
Catherine Diep a3ea140e42 Remove duplicate test case.
Remove test case
tempest.api.image.v2.test_images.ListImagesTest.test_index_no_params
from the advisory test list because this test case has been a required
test case since 2014.07.

Change-Id: I1c70f3ca6aa762340a2eeaa9dafa25ed85087e5d
2016-03-03 16:07:39 -08:00
Catherine Diep d096a9f303 Fix value of the "required-since" in identity-v2-tokens-create.
The "identity-v2-tokens-create" capability was deprecated.  As such,
its "required-since" field should be set to empty.

Change-Id: I84890d2e5011e4e4cf3142c9d590043ca2c891d2
2016-02-22 16:39:03 -08:00
Jenkins bc586fd0c2 Merge "Use change to approved status to start general discussion" 2016-01-26 23:37:12 +00:00
Mark T. Voelker aa2ecfec44 Remove test_reboot_server_soft
This test has been flagged for some time due to a longstanding
bug [1] in Tempest (open since 2012).  In discussion with QA, it appears
that the bug isn't going to be fixed and the test is likely to be
removed.  Since the tests has been flagged for several cycles now
and the test itself is likely to be dropped, we should remove it
from the 2016.01 Guideline and future Guidelines.

[1] https://bugs.launchpad.net/tempest/+bug/1014647

Change-Id: Ia1adc9e6b3f81c65922b5167d99347d47bc95ac7
2016-01-20 08:24:54 -08:00
robhirschfeld 5741466a94 Use change to approved status to start general discussion
This change cannot land until Board approves until 2016.01
meeting.

Change-Id: Ief68c995ef1cd5498fe3649f55a19a85b21cc4be
2016-01-20 16:05:16 +00:00
Jenkins 694226a72c Merge "Change the key "flag" to "flagged" per schemna definition." 2016-01-13 03:23:36 +00:00
cdiep 5b8d1aaf2c Change the key "flag" to "flagged" per schemna definition.
Since schema version 1.3 [1], a key of "flagged" should be used
for flagged tests.

[1] https://review.openstack.org/#/c/215312/

Change-Id: Ifeac35407a589ec413748edf26bf24c73fa5e39f
2016-01-12 11:35:03 -08:00
bria4010 10562c245a Remove advisory images-v2-import capability
It's premature to include an Images v2 API import capability in DefCore
(even in an advisory capacity) given that image import functionality has
yet to be developed in Glance.

The change proposed by this patch is in accord with the TC resolution
"Compute Requirements for Images and Kernels" [0], which states that "An
OpenStack Powered Compute Cloud MUST allow end user Image Uploads".
(End user image upload is typically referred to in the industry as
"image import" in order to distinguish it from from the simple upload
facility used by trusted sources, for example, an OpenStack service like
Nova, or Glance administrators supplying public images for use in a
cloud).  I interpret the TC resolution as a forward-looking statement
saying that the capability must be exposed as soon as it's available,
not that the capability must be made available immediately using the
(possibly improper) means currently at hand.  In other words, if we want
to impose such a requirement on OpenStack deployers, then it's incumbent
upon OpenStack to provide a means by which this can responsibly be done
in an interoperable way.  That means is scheduled to be delivered in
Glance in the Mitaka cycle.

The Glance image import spec [1] contains discussion and links to even
more discussion concerning this issue.  There's also lively discussion
of the issue on the gerrit review for the spec. [2]

The images-v2-import capability should be resinstated as an advisory
capability after the Mitaka release.  (The set of tests defining the
capability, however, will have to be modified.)

[0] https://governance.openstack.org/resolutions/20151211-bring-your-own-kernel.html
[1] http://specs.openstack.org/openstack/glance-specs/specs/mitaka/approved/image-import/image-import-refactor.html
[2] https://review.openstack.org/#/c/232371/

Change-Id: I98714d97a80a14b62b936ea51ccbde50117401b5
2016-01-06 15:24:55 -05:00
cdiep e368c9e3ba Add alias to handle test name changed.
"test_rebuild_reboot_deleted_server" was renamed to
"test_rebuild_deleted_server" in patch [1] which was merged on
December 5, 2015.

[1] https://review.openstack.org/#/c/239710/

Change-Id: If64855268c167ae39661b6396d82658872309b92
2015-12-17 15:58:16 -08:00
Mark T. Voelker 6307fbde62 Migrate next and 2016.01 to Schema 1.4
At the DefCore workshop at the OpenStack Summit in Tokyo, we
discussed the need to migrate our next.json and in-review 2016.01
Guidelines to the new schema 1.4.  The new schema removes a duplicate
source of truth around the status of each capability, provides a
target date, and an alias field for tests that have been renamed.

This patch migrates the two as-yet unapproved JSON docs to the
new schema.  We've also discussed migrating older specs, which will
be done in a subsequent patch.

Change-Id: I7f0a370d4fd3e2650d70a29accdd17f4894e5a8f
2015-11-02 12:00:02 -05:00
Eddie Shvartsman 125f197265 Remove proposed advisory guidelines for floating IPs
Per discussion at Defcore working group @ Tokyo summit on Oct 28th
2016. Since various platforms do not yet support floating IPs, they
will not be an advisory guideline for 2016-01. Floating IPs will be
retained in json.next for future consideration.

Change-Id: I98beb54e8f7007b4b2d6c8c9b3cd9da87ef5a24a
2015-10-29 14:28:58 +09:00
Rob Hirschfeld c1e8b3383f Create 2016.01 REVIEW for Board Presentation
Create a new Guideline for Board and community review
from the current .next guideline.

This patch will be merged at the Board meeting.

NOTE: it is REVIEW status, not approved.  Approval
is scheduled for 2016.01.

Change-Id: I7f0152a80ae4a4619717b9ac89eb691a018847b7
2015-10-26 13:11:13 +09:00