)]}'
{"id":"openstack%2Fmistral~718375","triplet_id":"openstack%2Fmistral~stable%2Ftrain~I47abe0904b49d72e33eb10080c71fb81980d44da","project":"openstack/mistral","branch":"stable/train","topic":"bug/1871567","hashtags":[],"change_id":"I47abe0904b49d72e33eb10080c71fb81980d44da","subject":"Add YAQL sanitizing for iterators","status":"MERGED","created":"2020-04-08 10:20:12.000000000","updated":"2020-04-08 22:54:17.000000000","submitted":"2020-04-08 22:50:01.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":3,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"718375-1586386201298-b33c4e60","meta_rev_id":"21e64a01fd88e5c5fa6de97026be44df83a38ede","_number":718375,"virtual_id_number":718375,"owner":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"value":0,"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-04-08 22:50:01.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"all":[{"value":0,"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"value":2,"date":"2020-04-08 16:51:35.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"all":[{"value":0,"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"value":1,"date":"2020-04-08 16:51:35.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-04-08 11:29:32.000000000","updated_by":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"reviewer":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"state":"REVIEWER"},{"updated":"2020-04-08 22:50:01.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"1fa1c647199799adc6ba055e80462f2127e1855c","author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"date":"2020-04-08 10:20:12.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"16438db61337201c737da1a3549677b86b691a1b","author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"date":"2020-04-08 10:36:01.000000000","message":"Patch Set 2: Published edit on patch set 1.","accounts_in_message":[],"_revision_number":2},{"id":"01fa92323a9dc413da9853fa13c06c5e3cd857c5","author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"date":"2020-04-08 10:56:15.000000000","message":"Patch Set 2:\n\nHi Andreas, I saw your recent patch about excluding sphinx 3.0.0 (https://review.opendev.org/#/c/717949/). I made the same in Mistral master branch so now it\u0027s fine. However, I\u0027m wondering what we need to do with backports. If we also add an exclusion here it will conflict with the global requirements (this exclusion is not there for stable/train) and most likely with the stable policy. Although, about stable policy I\u0027m not sure, because this is only for the doc CI job. Any ideas about how to best proceed? Without this exclusion the doc job fails.","accounts_in_message":[],"_revision_number":2},{"id":"25570b9c171bf147aa6f787ec7dd52a736a04c77","author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"date":"2020-04-08 11:29:32.000000000","message":"Patch Set 2:\n\n\u003e Hi Andreas, I saw your recent patch about excluding sphinx 3.0.0\n \u003e (https://review.opendev.org/#/c/717949/). I made the same in\n \u003e Mistral master branch so now it\u0027s fine. However, I\u0027m wondering what\n \u003e we need to do with backports. If we also add an exclusion here it\n \u003e will conflict with the global requirements (this exclusion is not\n \u003e there for stable/train) and most likely with the stable policy.\n\n \u003e Although, about stable policy I\u0027m not sure, because this is only\n \u003e for the doc CI job. Any ideas about how to best proceed? Without\n \u003e this exclusion the doc job fails.\n\nBest talk with requirements team, adding the exclusion to a stable branch should be fine IMHO. Alternative is using constraints for the docs jobs.","accounts_in_message":[],"_revision_number":2},{"id":"e7fe921f1beaf8a148f6c7ebf97f8d914aa339b0","author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"date":"2020-04-08 11:50:06.000000000","message":"Patch Set 2:\n\nOk, thanks.","accounts_in_message":[],"_revision_number":2},{"id":"330a25a46f7bb2fd9903df73a264c19939d94c5d","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-08 13:13:40.000000000","message":"Patch Set 2: Verified-1\n\n(1 comment)\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/847666a14cdc4d84bf592934ab1ea68a : SUCCESS in 6m 26s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/a5bde9db778b446aad462a9ebec711e3 : SUCCESS in 10m 00s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/44723ddc92af475eb9bd6347e70fd8a3 : SUCCESS in 10m 07s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/06add13e554e4693b7a2472a51570a4d : SUCCESS in 16m 39s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/3e7c3ea58a954dec9b6c7633193663f5 : FAILURE in 9m 57s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/3d7c6635628b43ccb31361ad4a201289 : SUCCESS in 10m 41s (non-voting)\n- mistral-devstack https://zuul.opendev.org/t/openstack/build/6c69a1cb78e24bcc8155cc6dd27ef9cc : SUCCESS in 46m 51s\n- mistral-devstack-non-apache https://zuul.opendev.org/t/openstack/build/ef35767e199b42a882eeadb061457b18 : FAILURE in 51m 20s\n- mistral-devstack-kombu https://zuul.opendev.org/t/openstack/build/f1d1bb48b4b2474cb346928b316c7215 : SUCCESS in 45m 26s\n- mistral-tox-unit-mysql https://zuul.opendev.org/t/openstack/build/23c8a53316e3474ba4830e031c3a76d8 : SUCCESS in 29m 55s\n- mistral-tox-unit-postgresql https://zuul.opendev.org/t/openstack/build/d22b6478fcbd4bc3b05414d3a8e5f751 : SUCCESS in 29m 41s\n- tripleo-ci-centos-7-scenario003-standalone https://zuul.opendev.org/t/openstack/build/253b782250c144589e093f9f7d4a3e6a : SUCCESS in 1h 19m 33s\n- tripleo-ci-centos-7-containers-multinode https://zuul.opendev.org/t/openstack/build/93da4c7790e649f99f2e5599611bde36 : SUCCESS in 2h 35m 32s\n- mistral-rally-task https://zuul.opendev.org/t/openstack/build/63fe5b74ed5949feb2a94efb4e12be40 : FAILURE in 21m 40s (non-voting)\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/c9b985e93b12495e8ad60f91e240c218 : SUCCESS in 10m 37s\n\nWarning:\n  Comments left for invalid file mistral/context.py\n  Comments left for invalid file mistral/services/workflows.py\n  Comments left for invalid file mistral/utils/expression_utils.py\n  Comments left for invalid file mistral/tests/unit/engine/base.py\n  Comments left for invalid file mistral/db/sqlalchemy/base.py\n  Comments left for invalid file mistral/tests/unit/services/test_event_engine.py\n  Comments left for invalid file mistral/tests/unit/rpc/kombu/test_kombu_hosts.py\n  Comments left for invalid file mistral/rpc/kombu/base.py\n  Comments left for invalid file mistral/tests/unit/api/v2/test_workflows.py","accounts_in_message":[],"_revision_number":2},{"id":"379c369c41e5574498e048f397b90f93f30d1101","author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"date":"2020-04-08 16:51:04.000000000","message":"Patch Set 3: Patch Set 2 was rebased","accounts_in_message":[],"_revision_number":3},{"id":"e49f8374637cd60e98b431b1de7e3e1ee5b81822","author":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"date":"2020-04-08 16:51:35.000000000","message":"Patch Set 3: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":3},{"id":"0ec3c9241b8633225f95e6aab214bd82599ae648","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-08 20:26:44.000000000","message":"Patch Set 3: Verified+1\n\n(1 comment)\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/2b935786384c4e50b905a0abf755493b : SUCCESS in 8m 01s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/165466d2a2214223b18c5bdf886b59f5 : SUCCESS in 10m 44s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/d0cb5de99cb44307a399488da1c8e0c9 : SUCCESS in 10m 48s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/561a67a17aa24cad9f8d4c5226056acf : SUCCESS in 15m 09s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/6413770f6e70478b85d85eab5f1dc7dd : SUCCESS in 9m 30s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/83e83e3dc51c482790f84abc3d7b714c : SUCCESS in 12m 09s (non-voting)\n- mistral-devstack https://zuul.opendev.org/t/openstack/build/ec9a3cca112e4d339878bc6e74443d16 : SUCCESS in 40m 53s\n- mistral-devstack-non-apache https://zuul.opendev.org/t/openstack/build/3e4d3977f8b44068a4060e1d9b592ba4 : SUCCESS in 43m 10s\n- mistral-devstack-kombu https://zuul.opendev.org/t/openstack/build/2da3c572cd404fb991f6ae81573f5fbd : SUCCESS in 45m 16s\n- mistral-tox-unit-mysql https://zuul.opendev.org/t/openstack/build/ee71b18215cb48c18c2ce96991e6fa2d : SUCCESS in 35m 22s\n- mistral-tox-unit-postgresql https://zuul.opendev.org/t/openstack/build/dd5048d271364ab3a19822c637fe5fa4 : SUCCESS in 29m 51s\n- tripleo-ci-centos-7-scenario003-standalone https://zuul.opendev.org/t/openstack/build/bd114a901ff740eaa8f8eb4e67742d75 : SUCCESS in 1h 31m 19s\n- tripleo-ci-centos-7-containers-multinode https://zuul.opendev.org/t/openstack/build/18d9252256a04da3bcb933681c20dc71 : SUCCESS in 2h 59m 53s\n- mistral-rally-task https://zuul.opendev.org/t/openstack/build/dc89684ba5254858996b77874438d95f : FAILURE in 20m 16s (non-voting)\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/0281d22d8f05408f8236732796da9039 : SUCCESS in 14m 10s\n\nWarning:\n  Comments left for invalid file mistral/context.py\n  Comments left for invalid file mistral/services/workflows.py\n  Comments left for invalid file mistral/utils/expression_utils.py\n  Comments left for invalid file mistral/tests/unit/engine/base.py\n  Comments left for invalid file mistral/db/sqlalchemy/base.py\n  Comments left for invalid file mistral/tests/unit/services/test_event_engine.py\n  Comments left for invalid file mistral/tests/unit/api/v2/test_workflows.py\n  Comments left for invalid file mistral/rpc/kombu/base.py\n  Comments left for invalid file mistral/tests/unit/rpc/kombu/test_kombu_hosts.py","accounts_in_message":[],"_revision_number":3},{"id":"48d272b4999721a514580373f82b40933eaf39dd","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-08 20:31:14.000000000","message":"Patch Set 3: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":3},{"id":"8f109003267960c664eb563c01a7e13d5925b483","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-08 22:50:01.000000000","message":"Patch Set 3: Verified+2\n\n(1 comment)\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/937c801998674f208a3d787ac15b26d5 : SUCCESS in 7m 36s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/9708a209c3ea40aeb60052e2aaf206ad : SUCCESS in 12m 43s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/061689a40f3f4b338475c2d94a2f0acd : SUCCESS in 12m 19s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/77a2e9f8c16f410cbe72da6904a5ac48 : SUCCESS in 13m 58s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/155f9cd26d9c48519af366de3e2279d0 : SUCCESS in 7m 38s\n- mistral-devstack https://zuul.opendev.org/t/openstack/build/492cb3d6cb014c06b6cc6c0d409e9fde : SUCCESS in 52m 30s\n- mistral-devstack-non-apache https://zuul.opendev.org/t/openstack/build/b0c9b600ff3045f7a9e6ff5b06efb4ae : SUCCESS in 52m 25s\n- mistral-tox-unit-mysql https://zuul.opendev.org/t/openstack/build/1f8ab3506fe04c96a97afa557e38f74e : SUCCESS in 32m 38s\n- mistral-tox-unit-postgresql https://zuul.opendev.org/t/openstack/build/e5b9acd2a89b4ea3861d49b4b2698a10 : SUCCESS in 32m 11s\n- tripleo-ci-centos-7-scenario003-standalone https://zuul.opendev.org/t/openstack/build/5f7a6ee45f80454c9334a2df74878a46 : SUCCESS in 1h 49m 39s\n- mistral-devstack-kombu https://zuul.opendev.org/t/openstack/build/ab144d6d5d164e31b72fdbfe7a4c38b9 : SUCCESS in 48m 19s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/07fce70cd3884231b3dda00ab5abdeed : SUCCESS in 13m 23s\n- tripleo-ci-centos-7-undercloud-containers https://zuul.opendev.org/t/openstack/build/04582f244b4e47e9b56d86a7b0d9ef30 : SUCCESS in 2h 16m 08s\n\nWarning:\n  Comments left for invalid file mistral/context.py\n  Comments left for invalid file mistral/utils/expression_utils.py\n  Comments left for invalid file mistral/tests/unit/engine/base.py\n  Comments left for invalid file mistral/db/sqlalchemy/base.py\n  Comments left for invalid file mistral/tests/unit/services/test_event_engine.py\n  Comments left for invalid file mistral/services/workflows.py\n  Comments left for invalid file mistral/tests/unit/rpc/kombu/test_kombu_hosts.py\n  Comments left for invalid file mistral/rpc/kombu/base.py\n  Comments left for invalid file mistral/tests/unit/api/v2/test_workflows.py","accounts_in_message":[],"_revision_number":3},{"id":"a8d6afd8a4adfb6526b524b8f0e271b0eaad4772","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-08 22:50:01.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":3},{"id":"5ee714ae7f1740a0fb3805282098c889d04145d4","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-04-08 22:54:17.000000000","message":"Patch Set 3:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/89cba08c1ed0458eaaf19d742151f7a0 : SUCCESS in 3m 32s","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"453c2a6561e543bedc2a363018dfb2c1c30dd454","revisions":{"e43aab1d45a2e546a59ffd78dff741c7806d8dc5":{"kind":"REWORK","_number":1,"created":"2020-04-08 10:20:12.000000000","uploader":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"ref":"refs/changes/75/718375/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/mistral","ref":"refs/changes/75/718375/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/mistral refs/changes/75/718375/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/mistral refs/changes/75/718375/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/mistral refs/changes/75/718375/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/mistral refs/changes/75/718375/1"}}},"commit":{"parents":[{"commit":"1e0013835c10cbbdfb747625d29cbbe9d6852948","subject":"Fix serialization of structures that might contain YAQL types","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/1e0013835c10cbbdfb747625d29cbbe9d6852948"}]}],"author":{"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","date":"2020-04-08 06:22:49.000000000","tz":420},"committer":{"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","date":"2020-04-08 10:19:58.000000000","tz":420},"subject":"Add YAQL sanitizing for iterators","message":"Add YAQL sanitizing for iterators\n\n* This patch also adds handling for iterators in the sanitizing\n  function for YAQL results. JSON serialization for iterators now\n  works correctly but the issue is that they can be used only once.\n  So, for example, if we got a result from YAQL and performed JSON\n  serialization for it, then the iterator is already empty and\n  the data structure is just corrupted. So we can\u0027t further pass\n  to executors and other subsystems. The solution is just to sanitize\n  iterators after they\u0027re just returned from YAQL.\n* Added a test to make sure that action input reaches an executor\n  not corrupted, although it gets saved into DB and hence serialized\n  into JSON.\n* Changed the Sphinx entry in requirements.txt so that the version\n  3.0.0 is now excluded (it breaks the build).\n\nCloses-Bug: #1871567\nChange-Id: I47abe0904b49d72e33eb10080c71fb81980d44da\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/e43aab1d45a2e546a59ffd78dff741c7806d8dc5"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/e43aab1d45a2e546a59ffd78dff741c7806d8dc5"}]},"branch":"refs/heads/stable/train"},"641d120b1b877a3e7320497f734191c16d48c504":{"kind":"REWORK","_number":2,"created":"2020-04-08 10:36:01.000000000","uploader":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"ref":"refs/changes/75/718375/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/mistral","ref":"refs/changes/75/718375/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/mistral refs/changes/75/718375/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/mistral refs/changes/75/718375/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/mistral refs/changes/75/718375/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/mistral refs/changes/75/718375/2"}}},"commit":{"parents":[{"commit":"1e0013835c10cbbdfb747625d29cbbe9d6852948","subject":"Fix serialization of structures that might contain YAQL types","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/1e0013835c10cbbdfb747625d29cbbe9d6852948"}]}],"author":{"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","date":"2020-04-08 06:22:49.000000000","tz":420},"committer":{"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","date":"2020-04-08 10:35:46.000000000","tz":0},"subject":"Add YAQL sanitizing for iterators","message":"Add YAQL sanitizing for iterators\n\n* This patch also adds handling for iterators in the sanitizing\n  function for YAQL results. JSON serialization for iterators now\n  works correctly but the issue is that they can be used only once.\n  So, for example, if we got a result from YAQL and performed JSON\n  serialization for it, then the iterator is already empty and\n  the data structure is just corrupted. So we can\u0027t further pass\n  to executors and other subsystems. The solution is just to sanitize\n  iterators after they\u0027re just returned from YAQL.\n* Added a test to make sure that action input reaches an executor\n  not corrupted, although it gets saved into DB and hence serialized\n  into JSON.\n* Changed the Sphinx entry in requirements.txt so that the version\n  3.0.0 is now excluded (it breaks the build).\n\nCloses-Bug: #1871567\nChange-Id: I47abe0904b49d72e33eb10080c71fb81980d44da\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/641d120b1b877a3e7320497f734191c16d48c504"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/641d120b1b877a3e7320497f734191c16d48c504"}]},"branch":"refs/heads/stable/train"},"453c2a6561e543bedc2a363018dfb2c1c30dd454":{"kind":"TRIVIAL_REBASE","_number":3,"created":"2020-04-08 16:51:04.000000000","uploader":{"_account_id":8731,"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","username":"rakhmerov"},"ref":"refs/changes/75/718375/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/mistral","ref":"refs/changes/75/718375/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/mistral refs/changes/75/718375/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/mistral refs/changes/75/718375/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/mistral refs/changes/75/718375/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/mistral refs/changes/75/718375/3"}}},"commit":{"parents":[{"commit":"fa02b097a6e511a5f705059cae6e4d4d61ad8f1e","subject":"Use constraints for docs installs","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/fa02b097a6e511a5f705059cae6e4d4d61ad8f1e"}]}],"author":{"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","date":"2020-04-08 06:22:49.000000000","tz":420},"committer":{"name":"Renat Akhmerov","email":"renat.akhmerov@gmail.com","date":"2020-04-08 16:51:04.000000000","tz":0},"subject":"Add YAQL sanitizing for iterators","message":"Add YAQL sanitizing for iterators\n\n* This patch also adds handling for iterators in the sanitizing\n  function for YAQL results. JSON serialization for iterators now\n  works correctly but the issue is that they can be used only once.\n  So, for example, if we got a result from YAQL and performed JSON\n  serialization for it, then the iterator is already empty and\n  the data structure is just corrupted. So we can\u0027t further pass\n  to executors and other subsystems. The solution is just to sanitize\n  iterators after they\u0027re just returned from YAQL.\n* Added a test to make sure that action input reaches an executor\n  not corrupted, although it gets saved into DB and hence serialized\n  into JSON.\n* Changed the Sphinx entry in requirements.txt so that the version\n  3.0.0 is now excluded (it breaks the build).\n\nCloses-Bug: #1871567\nChange-Id: I47abe0904b49d72e33eb10080c71fb81980d44da\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/453c2a6561e543bedc2a363018dfb2c1c30dd454"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/453c2a6561e543bedc2a363018dfb2c1c30dd454"}]},"branch":"refs/heads/stable/train"}},"requirements":[],"submit_records":[],"submit_requirements":[]}