)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"ac5ea57b7732e900c521bd62235d7037d632719a","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add a job to run federated identity tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We were using the non-voting \"v3-only\" job to test our DevStack plugin"},{"line_number":10,"context_line":"for the Federated Identity feature - this change adds a test for this"},{"line_number":11,"context_line":"specific deployment and removes its configuration from the \"v3 only\""},{"line_number":12,"context_line":"job."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Idab0c7b8c3a60243caa0c1d47a83d58dce64bbde"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a6eadb0_53a6f710","line":12,"range":{"start_line":10,"start_character":37,"end_line":12,"end_character":4},"updated":"2016-12-13 22:42:53.000000000","message":"why?","commit_id":"3be64a5e80a6ae8c9f2e914bc1fca14cd79a0087"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"50ca973a973341b127ab5ab8f50d0752e465accf","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add a job to run federated identity tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We were using the non-voting \"v3-only\" job to test our DevStack plugin"},{"line_number":10,"context_line":"for the Federated Identity feature - this change adds a test for this"},{"line_number":11,"context_line":"specific deployment and removes its configuration from the \"v3 only\""},{"line_number":12,"context_line":"job."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Idab0c7b8c3a60243caa0c1d47a83d58dce64bbde"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a6eadb0_ae91a254","line":12,"range":{"start_line":10,"start_character":37,"end_line":12,"end_character":4},"in_reply_to":"1a6eadb0_53a6f710","updated":"2016-12-13 23:21:06.000000000","message":"since it is a significantly different environment, think the best approach is to have a separate job for it. Ideally, for each different DevStack plugin we have, we would have a separate job.","commit_id":"3be64a5e80a6ae8c9f2e914bc1fca14cd79a0087"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"b3ce4f4711c8598cdb1f64b14a8c223a9ab56c3a","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add a job to run federated identity tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We were using the non-voting \"v3-only\" job to test our DevStack plugin"},{"line_number":10,"context_line":"for the Federated Identity feature - this change adds a test for this"},{"line_number":11,"context_line":"specific deployment and removes its configuration from the \"v3 only\""},{"line_number":12,"context_line":"job."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Idab0c7b8c3a60243caa0c1d47a83d58dce64bbde"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a6eadb0_9ea07845","line":12,"range":{"start_line":10,"start_character":37,"end_line":12,"end_character":4},"in_reply_to":"1a6eadb0_6707a81c","updated":"2016-12-14 12:33:08.000000000","message":"do you agree an all-in-one job is an additional scenario? Think we need to have them separately and having all together. Example:\n\nScenario 1 - check if keystone works with federated identity\nScenario 2 - check if keystone works with LDAP as identity backend\nScenario 3 - check if keystone works with federated identity and LDAP as identity backend at the same time","commit_id":"3be64a5e80a6ae8c9f2e914bc1fca14cd79a0087"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"8c7a6c548ae59ff0f720076c5edfd302c6f759fc","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add a job to run federated identity tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We were using the non-voting \"v3-only\" job to test our DevStack plugin"},{"line_number":10,"context_line":"for the Federated Identity feature - this change adds a test for this"},{"line_number":11,"context_line":"specific deployment and removes its configuration from the \"v3 only\""},{"line_number":12,"context_line":"job."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Idab0c7b8c3a60243caa0c1d47a83d58dce64bbde"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a6eadb0_c312e708","line":12,"range":{"start_line":10,"start_character":37,"end_line":12,"end_character":4},"in_reply_to":"1a6eadb0_9e6beff0","updated":"2016-12-15 10:30:58.000000000","message":"we expect the user to use keystone against LDAP, not against LDAP, using federation, not using federation, using federation + LDAP, etc.","commit_id":"3be64a5e80a6ae8c9f2e914bc1fca14cd79a0087"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"82a73ea5052e893d3686b94ca18f1867d40157a2","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add a job to run federated identity tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We were using the non-voting \"v3-only\" job to test our DevStack plugin"},{"line_number":10,"context_line":"for the Federated Identity feature - this change adds a test for this"},{"line_number":11,"context_line":"specific deployment and removes its configuration from the \"v3 only\""},{"line_number":12,"context_line":"job."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Idab0c7b8c3a60243caa0c1d47a83d58dce64bbde"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a6eadb0_9e6beff0","line":12,"range":{"start_line":10,"start_character":37,"end_line":12,"end_character":4},"in_reply_to":"1a6eadb0_9ea07845","updated":"2016-12-15 04:43:56.000000000","message":"I\u0027d rather we just aim for scenario 3, since a user won\u0027t spin up two keystones that are configured differently, we should test the way we expect a user to use things.","commit_id":"3be64a5e80a6ae8c9f2e914bc1fca14cd79a0087"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"f033fdd24b9553b7fd0b59bf26976b553175a537","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add a job to run federated identity tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We were using the non-voting \"v3-only\" job to test our DevStack plugin"},{"line_number":10,"context_line":"for the Federated Identity feature - this change adds a test for this"},{"line_number":11,"context_line":"specific deployment and removes its configuration from the \"v3 only\""},{"line_number":12,"context_line":"job."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Idab0c7b8c3a60243caa0c1d47a83d58dce64bbde"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a6eadb0_ee1c7ac3","line":12,"range":{"start_line":10,"start_character":37,"end_line":12,"end_character":4},"in_reply_to":"1a6eadb0_ae91a254","updated":"2016-12-13 23:27:10.000000000","message":"bottom line: to keep things organized and properly separated -\u003e easier to debug if errors happen.","commit_id":"3be64a5e80a6ae8c9f2e914bc1fca14cd79a0087"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"bfc76126ec08d7450331f38b661d5d6c9ef23ff0","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add a job to run federated identity tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We were using the non-voting \"v3-only\" job to test our DevStack plugin"},{"line_number":10,"context_line":"for the Federated Identity feature - this change adds a test for this"},{"line_number":11,"context_line":"specific deployment and removes its configuration from the \"v3 only\""},{"line_number":12,"context_line":"job."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Idab0c7b8c3a60243caa0c1d47a83d58dce64bbde"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a6eadb0_6707a81c","line":12,"range":{"start_line":10,"start_character":37,"end_line":12,"end_character":4},"in_reply_to":"1a6eadb0_ee1c7ac3","updated":"2016-12-14 04:33:53.000000000","message":"add that to the commit message then ;)\n\nalso, should we sacrifice organizing different environments when we advocate that deployers do this all-in-one?\n\nlooking forward ... will ldap support be it it\u0027s own job? I always thought they\u0027ve be all-in-one","commit_id":"3be64a5e80a6ae8c9f2e914bc1fca14cd79a0087"}]}