summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
38 hoursMerge "CI: Add chart build jobs"HEADmasterZuul
14 daysLog client-id in UCP API endpointsCrank, Daniel (dc6350)
Adds functionality to read context marker and end-user from request headers and log that information where available, to aid in tracing transactions that span multiple Airship components. Change-Id: I35c9e56f84f29420c4f3c081453cb81aa892fa7d Notes (review): Code-Review+1: Smruti Soumitra Khuntia <sk698p@att.com> Code-Review+2: Drew Walters <drewwalters96@gmail.com> Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Code-Review+1: Ian Pittwood <pittwoodian@gmail.com> Code-Review+1: Ahmad Mahmoudi <am495p@att.com> Code-Review+1: Andriy Shevchenko <huang.zhiping@99cloud.net> Code-Review+1: 98k <18552437190@163.com> Workflow+1: Drew Walters <drewwalters96@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 16 Apr 2019 23:11:23 +0000 Reviewed-on: https://review.openstack.org/634068 Project: openstack/airship-deckhand Branch: refs/heads/master
2019-04-04CI: Add chart build jobsDrew Walters
This commit introduces a non-voting job to lint Helm charts against the latest version of Helm toolkit from OpenStack-Helm Infra and a gate for a pinned version of Helm toolkit. Change-Id: Ia98209745641db011469d54220777110b2f6ae3a Notes (review): Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Code-Review+2: Kaspars Skels <kaspars.skels@gmail.com> Workflow+1: Kaspars Skels <kaspars.skels@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 17 Apr 2019 16:43:40 +0000 Reviewed-on: https://review.openstack.org/649441 Project: openstack/airship-deckhand Branch: refs/heads/master
2019-04-03(zuul) Fix image publish post pipelineScott Hussey
- Fix post pipeline publish job issue introduced w/ Ansible upgrade Change-Id: Ib25c2dcc08eae3db15926c272a9a6c297d9e2ed8 Notes (review): Code-Review+2: Drew Walters <drewwalters96@gmail.com> Code-Review+2: Kaspars Skels <kaspars.skels@gmail.com> Workflow+1: Drew Walters <drewwalters96@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 03 Apr 2019 21:10:31 +0000 Reviewed-on: https://review.openstack.org/649714 Project: openstack/airship-deckhand Branch: refs/heads/master
2019-03-28Merge "tools: Update Helm to v2.13.1"Zuul
2019-03-28tools: Update Helm to v2.13.1Drew Walters
Helm v2.13.1 has been released [0]. This change uplifts Helm to v2.13.1 and restores usage of the latest Helm toolkit to match other Airship projects. [0] https://github.com/helm/helm/releases/tag/v2.13.1 Depends-On: https://review.openstack.org/647812 Change-Id: Ic34f8fc1c8c3b5f8cb5ba8dc770d64502b26fdb0 Notes (review): Code-Review+1: Dustin Specker <dustin.specker@att.com> Code-Review+1: Nishant Kumar <nishant.e.kumar@ericsson.com> Code-Review+1: Dan Crank <dan.no@att.com> Code-Review+2: Matt McEuen <matt.mceuen@att.com> Code-Review+2: Scott Hussey <sthussey@att.com> Workflow+1: Scott Hussey <sthussey@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 28 Mar 2019 21:51:31 +0000 Reviewed-on: https://review.openstack.org/647822 Project: openstack/airship-deckhand Branch: refs/heads/master
2019-03-26Updating Docker Gate use of zuul.newrevAaron Sheffield
- Zuul updated ansible to 2.7, no longer allows missing variables. - Using default value when it isn't available. - airship_deckhand_path isn't working, but is a non-voting gate. Based on Aaron Sheffield's PS for Pegleg: https://review.openstack.org/#/c/645631/ Change-Id: I6fe13d2043480cd6de6785203373f35d83545196 Notes (review): Code-Review+2: Matt McEuen <matt.mceuen@att.com> Code-Review+1: Dan Crank <dan.no@att.com> Code-Review+1: PRATEEK REDDY DODDA <pd2839@att.com> Code-Review+2: Drew Walters <drewwalters96@gmail.com> Workflow+1: Drew Walters <drewwalters96@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 27 Mar 2019 15:41:32 +0000 Reviewed-on: https://review.openstack.org/645825 Project: openstack/airship-deckhand Branch: refs/heads/master
2019-03-12Merge "[FIX] Change Helm-toolkit pinning to new commit"Zuul
2019-03-12[FIX] Change Helm-toolkit pinning to new commitNishant Kumar
This PS pins version of Helm-toolkit to support recent changes made in deckhand charts to support DB initialization [0] [0] https://review.openstack.org/#/c/639218/ Change-Id: Ie8f218f780d6029bfc87937a971f342fa3968dc6 Notes (review): Code-Review+2: Scott Hussey <sthussey@att.com> Code-Review+2: Drew Walters <drewwalters96@gmail.com> Workflow+1: Drew Walters <drewwalters96@gmail.com> Code-Review+1: Michael Beaver <michaelbeaver64@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 12 Mar 2019 21:59:47 +0000 Reviewed-on: https://review.openstack.org/642884 Project: openstack/airship-deckhand Branch: refs/heads/master
2019-03-05Merge "Use helm-toolkit for DB initialization"Zuul
2019-03-04Use helm-toolkit for DB initializationNishant Kumar
Depends-On: https://review.openstack.org/#/c/635348/ Change-Id: Ie9328755d1cb2f27bfa83c0f5a6568c97befe9bf Notes (review): Code-Review+1: Drew Walters <drewwalters96@gmail.com> Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Code-Review+2: Scott Hussey <sthussey@att.com> Workflow+1: Scott Hussey <sthussey@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 05 Mar 2019 19:21:06 +0000 Reviewed-on: https://review.openstack.org/639218 Project: openstack/airship-deckhand Branch: refs/heads/master
2019-02-28[chart] Enable liveness probe in DHanthony.bellino
This is to try to address stuck deckhand-api ponds that never went to error state in an attempt to self-jolt the pod again. Change-Id: I70bf57dde5d696bddc68caab2f54826803d82d28 Notes (review): Code-Review+2: Scott Hussey <sthussey@att.com> Code-Review+1: Nishant Kumar <nishant.e.kumar@ericsson.com> Code-Review+1: Drew Walters <drewwalters96@gmail.com> Code-Review+1: Alexander Hughes <Alexander.Hughes@pm.me> Code-Review+1: Vladyslav Drok <vdrok@mirantis.com> Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Code-Review+1: Evgeniy L <eli@mirantis.com> Workflow+1: Matt McEuen <matt.mceuen@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 01 Mar 2019 23:03:46 +0000 Reviewed-on: https://review.openstack.org/638275 Project: openstack/airship-deckhand Branch: refs/heads/master
2019-02-28[ad-hoc] Update oslo.utils ver to 3.40.2Crank, Daniel (dc6350)
oslo.utils version increased to 3.40.2 to resolve requirements conflict with tox. Change-Id: I43217a4c9e4f6e6547adb1d4ffaf1d4dadd376a7 Notes (review): Code-Review+1: Drew Walters <drewwalters96@gmail.com> Code-Review+1: Anthony Bellino <ab2434@att.com> Code-Review+1: PRATEEK REDDY DODDA <pd2839@att.com> Code-Review+2: Scott Hussey <sthussey@att.com> Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Workflow+1: Felipe Monteiro <felipe.monteiro@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 01 Mar 2019 04:42:47 +0000 Reviewed-on: https://review.openstack.org/640178 Project: openstack/airship-deckhand Branch: refs/heads/master
2019-02-22Merge "docs: Add use cases for each of the mutation operations"Zuul
2019-02-14Embed UML generated diagrams into docs, fix docs buildRoman Gorshunov
Remove manual generation of UML diagrams; they will be built by Sphinx itself. Require to install PlantUML and Graphviz via bindep in openstack-tox-docs job; ReadTheDocs service already has both PlantUML and Graphviz installed for documentation builds. Change-Id: I21ab124a80e7768bc9edd891b975b4e0f8a6e50d Story: 2004747 Notes (review): Code-Review+1: Vladyslav Drok <vdrok@mirantis.com> Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Code-Review+1: Drew Walters <drewwalters96@gmail.com> Code-Review+1: Michael Beaver <michaelbeaver64@gmail.com> Code-Review+1: Nishant Kumar <nishant.e.kumar@ericsson.com> Code-Review+2: Bryan Strassner <strassner.bryan@gmail.com> Workflow+1: Bryan Strassner <strassner.bryan@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 20 Feb 2019 22:36:39 +0000 Reviewed-on: https://review.openstack.org/635357 Project: openstack/airship-deckhand Branch: refs/heads/master
2019-01-25Added filename to logging message format for troubleshooting purposeSandeep Reddy Thumma
Change-Id: I2763ae7ea62e78c9dd89659f97b149ab000d0ebb Notes (review): Code-Review+1: Drew Walters <drewwalters96@gmail.com> Code-Review+1: PRATEEK REDDY DODDA <pd2839@att.com> Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Code-Review+1: Evgeniy L <eli@mirantis.com> Code-Review+2: Matt McEuen <matt.mceuen@att.com> Workflow+1: Matt McEuen <matt.mceuen@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 25 Jan 2019 14:04:48 +0000 Reviewed-on: https://review.openstack.org/626607 Project: openstack/airship-deckhand Branch: refs/heads/master
2019-01-23Update oslo.util version in requirementsSamuel Pilla
Change-Id: I11b44544825c51c9e3a294625776bb6aa3babfd9 Notes (review): Code-Review+1: Drew Walters <drewwalters96@gmail.com> Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Workflow+1: Felipe Monteiro <felipe.monteiro@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 25 Jan 2019 01:58:05 +0000 Reviewed-on: https://review.openstack.org/632793 Project: openstack/airship-deckhand Branch: refs/heads/master
2019-01-20Merge "Add openstack-discuss"Zuul
2019-01-18schema: Fix metadata schema patternsFelipe Monteiro
This patch set fixes the schema pattern enforced by metadata_document and metadata_control. Currently, both allow a schema with either pattern: - ^metadata/Control/v\d+$ - ^metadata/Document/v\d+$ However, the metadata_control schema should only allow the former and the metadata_document schema should only allow the latter. Change-Id: Ic1b88a7158755818002de4c88cdf2d7b716f656d Notes (review): Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Code-Review+1: Rick Bartra <rb560u@att.com> Workflow+1: Felipe Monteiro <felipe.monteiro@att.com> Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 18 Jan 2019 23:05:22 +0000 Reviewed-on: https://review.openstack.org/613824 Project: openstack/airship-deckhand Branch: refs/heads/master
2019-01-18Add openstack-discuss98k
Change-Id: I5db7d7da8eba99070619b79b2f259cc320940ccc Notes (review): Code-Review+1: Drew Walters <drewwalters96@gmail.com> Code-Review+1: Andriy Shevchenko <huang.zhiping@99cloud.net> Code-Review+1: Evgeniy L <eli@mirantis.com> Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Workflow+1: Felipe Monteiro <felipe.monteiro@att.com> Code-Review+1: Filippo Inzaghi <zhao.daqing@99cloud.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Sun, 20 Jan 2019 12:34:26 +0000 Reviewed-on: https://review.openstack.org/622996 Project: openstack/airship-deckhand Branch: refs/heads/master
2019-01-18CI: Fix integration jobDrew Walters
Deckhand's integration job relies on OpenStack-Helm-Infra to deploy Kubernetes using Kubeadm. Since [0] was merged, Calico requires AppArmor to be deployed. This commit adds the same role to deploy AppArmor that exists in OpenStack-Helm-Infra to prevent inadvertent gate failures. [0] https://review.openstack.org/614805/ This also includes gate fix in https://review.openstack.org/#/c/627906/ because of a circular dependency; that change in question fixes failing CI jobs for all non-integration jobs. Change-Id: Ie3fb04ea0fbe0487ae743033315e2a4211248d32 Notes (review): Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Workflow+1: Felipe Monteiro <felipe.monteiro@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 18 Jan 2019 21:29:02 +0000 Reviewed-on: https://review.openstack.org/630749 Project: openstack/airship-deckhand Branch: refs/heads/master
2018-12-03Merge "docs(substitution): mention that all occurrences are replaced"Zuul
2018-11-30Merge "Add Python 3.6 classifier to setup.cfg"Zuul
2018-11-30docs(substitution): mention that all occurrences are replacedDustin Specker
Before it was unclear if all occurrences of a pattern were replaced in a given target. Change-Id: Ie02a54fef98a912b36b66891be8fed346ba5d9e0 Notes (review): Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Code-Review+1: Drew Walters <drewwalters96@gmail.com> Code-Review+2: Sean Eagan <sean.eagan@att.com> Workflow+1: Sean Eagan <sean.eagan@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 03 Dec 2018 23:02:31 +0000 Reviewed-on: https://review.openstack.org/620963 Project: openstack/airship-deckhand Branch: refs/heads/master
2018-11-27Merge "Revision diffing issue with revision rollback."Zuul
2018-11-27Add Python 3.6 classifier to setup.cfgVieri
Change-Id: I82eac9c1c193df278676409f8e3608fb2d4301c9 Notes (review): Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Code-Review+1: Drew Walters <drewwalters96@gmail.com> Code-Review+1: Andriy Shevchenko <huang.zhiping@99cloud.net> Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Workflow+1: Aaron Sheffield <ajs@sheffieldfamily.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 30 Nov 2018 20:12:32 +0000 Reviewed-on: https://review.openstack.org/620245 Project: openstack/airship-deckhand Branch: refs/heads/master
2018-11-21Revision diffing issue with revision rollback.Smruti Soumitra Khuntia
* Fix for diffing issue after rollback in conjunction with created and deleted buckets. * Changed rollback function to check against the full set of documents for a revision instead of just the documents at that particular revision * Created a document_delete function to encapsulate document deletion * Added additional test case to check that a rollback to something other than 0 deletes the created buckets in between Co-Authored-By: Michael Beaver <michaelbeaver64@gmail.com> Change-Id: I0d57e67d68def1f15255a8c89290e8c70deedc03 Notes (review): Code-Review+1: Smruti Soumitra Khuntia <sk698p@att.com> Code-Review+2: Bryan Strassner <bryan.strassner@gmail.com> Code-Review+1: Lev Morgan <morgan.lev@gmail.com> Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Workflow+1: Aaron Sheffield <ajs@sheffieldfamily.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 27 Nov 2018 18:56:49 +0000 Reviewed-on: https://review.openstack.org/614421 Project: openstack/airship-deckhand Branch: refs/heads/master
2018-11-16Remove proxy ARG and ENV from DockerfileBryan Strassner
Removes the proxy settings that were being established as ARG and ENV commands since these are supported automatically by Docker, and should not be set as ENV for the built container. Change-Id: Id23f0b1591ef895df193462174eab2c67e7e0e73 Notes (review): Code-Review+2: Scott Hussey <sthussey@att.com> Workflow+1: Scott Hussey <sthussey@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 16 Nov 2018 23:31:59 +0000 Reviewed-on: https://review.openstack.org/618599 Project: openstack/airship-deckhand Branch: refs/heads/master
2018-11-15Merge "[FIX] Secrets substitution issue"Zuul
2018-11-15Merge "Fix: proper ordering: tagging after build"Zuul
2018-11-15Update url in HACKING.rstzhouxinyong
Change-Id: I78c02d9c7fe24f294a894369f0d7c34cfd79d1ce Notes (review): Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Code-Review+2: Scott Hussey <sthussey@att.com> Workflow+1: Scott Hussey <sthussey@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 14 Nov 2018 17:02:36 +0000 Reviewed-on: https://review.openstack.org/617995 Project: openstack/airship-deckhand Branch: refs/heads/master
2018-11-13Merge "Create Makefile target to install Helm binary"Zuul
2018-11-13[FIX] Secrets substitution issueAhmad Mahmoudi
Fixed issue with secrets substitution, which have more than one substitution destinations. Change-Id: I6c0e9719cacc7cf4189b57379e9ebf6a8d3a4fd8 Notes (review): Code-Review+1: Aaron Sheffield <ajs@sheffieldfamily.net> Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Code-Review+2: Sean Eagan <sean.eagan@att.com> Workflow+1: Sean Eagan <sean.eagan@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 15 Nov 2018 23:47:23 +0000 Reviewed-on: https://review.openstack.org/617689 Project: openstack/airship-deckhand Branch: refs/heads/master
2018-11-13Merge "fix wrong spelling"Zuul
2018-11-13Fix: proper ordering: tagging after buildRoman Gorshunov
With this patch, if commit being merged modifies both code and non-code files, we will first build and push new image, and then will run tagging job (which will do no harm). Change-Id: Id9e9e705f9f5f69b969003c444895e1e0676c0dd Notes (review): Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Code-Review+2: Sean Eagan <sean.eagan@att.com> Workflow+1: Sean Eagan <sean.eagan@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 15 Nov 2018 21:13:41 +0000 Reviewed-on: https://review.openstack.org/617679 Project: openstack/airship-deckhand Branch: refs/heads/master
2018-11-13fix wrong spellingwangqiangbj
Change-Id: I468da7d9053f526b1d014927d27b118f65ebb4c2 Notes (review): Code-Review+2: Bryan Strassner <bryan.strassner@gmail.com> Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Workflow+1: Aaron Sheffield <ajs@sheffieldfamily.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 13 Nov 2018 15:06:38 +0000 Reviewed-on: https://review.openstack.org/617622 Project: openstack/airship-deckhand Branch: refs/heads/master
2018-11-13omit the twice occured words in layering-with-replacement-single-bucket.yamlzhouxinyong
Change-Id: I50099748bce590bba82fa98036850ef6df4e918d Notes (review): Code-Review+1: rongyaozu <rongyaozu@inspur.com> Code-Review+2: Bryan Strassner <bryan.strassner@gmail.com> Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Workflow+1: Aaron Sheffield <ajs@sheffieldfamily.net> Code-Review+1: Diogo Guerra <dy090.guerra@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 13 Nov 2018 15:05:23 +0000 Reviewed-on: https://review.openstack.org/617403 Project: openstack/airship-deckhand Branch: refs/heads/master
2018-11-09Create Makefile target to install Helm binaryDimitrios Markou
In Makefile there is no target which actually installs Helm binary. Change-Id: I74a8099d7ff9592f3a86df060a9a54e2b56f6f33 Signed-off-by: Dimitrios Markou <dm844v@att.com> Notes (review): Code-Review+2: Bryan Strassner <bryan.strassner@gmail.com> Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Workflow+1: Aaron Sheffield <ajs@sheffieldfamily.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 13 Nov 2018 17:24:35 +0000 Reviewed-on: https://review.openstack.org/617000 Project: openstack/airship-deckhand Branch: refs/heads/master
2018-11-08Minor: meaningful default labelRoman Gorshunov
Making default label meaningful and conformant to "key=value" requirement. Change-Id: I9c65feaaf7637c63e2b258a972afda773eeb1ab5 Notes (review): Code-Review+2: Matt McEuen <matt.mceuen@att.com> Code-Review+2: Scott Hussey <sthussey@att.com> Workflow+1: Scott Hussey <sthussey@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 08 Nov 2018 20:00:48 +0000 Reviewed-on: https://review.openstack.org/616662 Project: openstack/airship-deckhand Branch: refs/heads/master
2018-11-06Merge "Fix logging when "Duplicate document exists" error occurs"Zuul
2018-11-06docs: Add use cases for each of the mutation operationsFelipe Monteiro
This patch set adds use cases for each of the mutation operations used by Deckhand: * substitution * layering * replacement To help document authors decide which operations they should use. Change-Id: I8bc470c7ddc2f3c0aa9d4372a2323fa84632e369 Notes (review): Code-Review+2: Roman Gorshunov <roman.gorshunov@att.com> Code-Review+1: Drew Walters <drewwalters96@gmail.com> Code-Review+2: Bryan Strassner <strassner.bryan@gmail.com> Workflow+1: Bryan Strassner <strassner.bryan@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 22 Feb 2019 23:20:28 +0000 Reviewed-on: https://review.openstack.org/615892 Project: openstack/airship-deckhand Branch: refs/heads/master
2018-11-05Merge "docs: Add config documentation to operator's section"Zuul
2018-11-01Fix logging when "Duplicate document exists" error occursEvgeny L
Currently validation fails with "KeyError: 'schema'", which makes it hard to determine a root cause of error. Change-Id: Ifd40faf485578cc0a133e17650f8df6758a6c8ae Notes (review): Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Code-Review+1: Andrey Volkov <av903u@att.com> Code-Review+1: Vladyslav Drok <vdrok@mirantis.com> Code-Review+2: Bryan Strassner <bryan.strassner@gmail.com> Workflow+1: Bryan Strassner <bryan.strassner@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 06 Nov 2018 15:51:50 +0000 Reviewed-on: https://review.openstack.org/614641 Project: openstack/airship-deckhand Branch: refs/heads/master
2018-10-31fix: Use schema instead of metadata.schema for replacement checkFelipe Monteiro
Recently added replacement check incorrectly uses metadata.schema and metadata.name to key on the document -- but it should be schema and metadata.name, the combination of which uniquely defines a document. Change-Id: I6cd1679ad41be38cb78d65ce2763e60f7da390d2 Notes (review): Code-Review+1: Rick Bartra <rb560u@att.com> Code-Review+2: Scott Hussey <sthussey@att.com> Code-Review+1: Vladyslav Drok <vdrok@mirantis.com> Workflow+1: Craig Anderson <craig.anderson@att.com> Code-Review+2: Craig Anderson <craig.anderson@att.com> Code-Review+1: Evgeniy L <eli@mirantis.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 01 Nov 2018 22:04:19 +0000 Reviewed-on: https://review.openstack.org/614523 Project: openstack/airship-deckhand Branch: refs/heads/master
2018-10-30Merge "Validate additional 'metadata.replacement' scenarios"Zuul
2018-10-30Merge "requirements: Update pinned requirements"Zuul
2018-10-30Validate additional 'metadata.replacement' scenariosRick Bartra
This patch set adds additional documentation and unit tests to validate further replacement scenarios. In particular this commit adds an additional document check that looks for documents exisitng in different layers that contain the same name and same schema without any of them having `replacement: true` Change-Id: I7c033d32a6755f36e609789a748cbc6d4af06bc2 Notes (review): Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Workflow+1: Aaron Sheffield <ajs@sheffieldfamily.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 30 Oct 2018 21:40:18 +0000 Reviewed-on: https://review.openstack.org/612528 Project: openstack/airship-deckhand Branch: refs/heads/master
2018-10-30Fix document is_control methodRick Bartra
The document.py `is_control` method incorrectly checks if a document is a Control document. Per the documentation [0], Control documents have `metadata.schema` of `metadata/Control/v1`. This commit updates the `is_control` method to correctly check for Control documents. [0] https://github.com/openstack/airship-deckhand/blob/1d4cc81dfaa9a5280625074e569fa4ab9dc7fd09/doc/source/users/document-types.rst#control-documents Change-Id: I60ca8f31a61987b4e756784fce0f5a751639ae9e Notes (review): Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Workflow+1: Aaron Sheffield <ajs@sheffieldfamily.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 30 Oct 2018 17:33:19 +0000 Reviewed-on: https://review.openstack.org/613688 Project: openstack/airship-deckhand Branch: refs/heads/master
2018-10-29docs: Add config documentation to operator's sectionFelipe Monteiro
This PS adds configuration documentation that includes a literalinclude of the config file as well as some information on each of the cache config options as these are important on performance. Change-Id: I3b06012b8843b7bfbd46307f81397172a41d3675 Notes (review): Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Workflow+1: Craig Anderson <craig.anderson@att.com> Code-Review+2: Bryan Strassner <bryan.strassner@gmail.com> Workflow+1: Bryan Strassner <bryan.strassner@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 05 Nov 2018 18:05:49 +0000 Reviewed-on: https://review.openstack.org/610722 Project: openstack/airship-deckhand Branch: refs/heads/master
2018-10-29Merge "Validate bucket diffing works with revision rollback"Zuul