{"action":"created","issue":{"url":"https://api.github.com/repos/ansible/awx/issues/2639","repository_url":"https://api.github.com/repos/ansible/awx","labels_url":"https://api.github.com/repos/ansible/awx/issues/2639/labels{/name}","comments_url":"https://api.github.com/repos/ansible/awx/issues/2639/comments","events_url":"https://api.github.com/repos/ansible/awx/issues/2639/events","html_url":"https://github.com/ansible/awx/pull/2639","id":378826055,"node_id":"MDExOlB1bGxSZXF1ZXN0MjI5NDM4NTE5","number":2639,"title":"Update UJT with inv summary field and update schedule base route to include object to be scheduled","user":{"login":"chrismeyersfsu","id":722880,"node_id":"MDQ6VXNlcjcyMjg4MA==","avatar_url":"https://avatars3.githubusercontent.com/u/722880?v=4","gravatar_id":"","url":"https://api.github.com/users/chrismeyersfsu","html_url":"https://github.com/chrismeyersfsu","followers_url":"https://api.github.com/users/chrismeyersfsu/followers","following_url":"https://api.github.com/users/chrismeyersfsu/following{/other_user}","gists_url":"https://api.github.com/users/chrismeyersfsu/gists{/gist_id}","starred_url":"https://api.github.com/users/chrismeyersfsu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chrismeyersfsu/subscriptions","organizations_url":"https://api.github.com/users/chrismeyersfsu/orgs","repos_url":"https://api.github.com/users/chrismeyersfsu/repos","events_url":"https://api.github.com/users/chrismeyersfsu/events{/privacy}","received_events_url":"https://api.github.com/users/chrismeyersfsu/received_events","type":"User","site_admin":false},"labels":[{"id":1080816370,"node_id":"MDU6TGFiZWwxMDgwODE2Mzcw","url":"https://api.github.com/repos/ansible/awx/labels/mergeit","name":"mergeit","color":"b345d8","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2018-11-08T16:56:09Z","updated_at":"2018-11-09T18:57:19Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/ansible/awx/pulls/2639","html_url":"https://github.com/ansible/awx/pull/2639","diff_url":"https://github.com/ansible/awx/pull/2639.diff","patch_url":"https://github.com/ansible/awx/pull/2639.patch"},"body":"API: Add inventory summary to schedules resource\r\n\r\nUI: previously, schedule edit would go to its own page, which would make it so that the user could not easily get to the resource to be scheduled.\r\n\r\n---\r\n\r\nIssue filed for issue where error modal pops up when user navigates to inv sync schedule edit and they don't have access to the project the inventory source inventory file is from.  https://github.com/ansible/awx/issues/2641"},"comment":{"url":"https://api.github.com/repos/ansible/awx/issues/comments/437460001","html_url":"https://github.com/ansible/awx/pull/2639#issuecomment-437460001","issue_url":"https://api.github.com/repos/ansible/awx/issues/2639","id":437460001,"node_id":"MDEyOklzc3VlQ29tbWVudDQzNzQ2MDAwMQ==","user":{"login":"AlanCoding","id":1385596,"node_id":"MDQ6VXNlcjEzODU1OTY=","avatar_url":"https://avatars3.githubusercontent.com/u/1385596?v=4","gravatar_id":"","url":"https://api.github.com/users/AlanCoding","html_url":"https://github.com/AlanCoding","followers_url":"https://api.github.com/users/AlanCoding/followers","following_url":"https://api.github.com/users/AlanCoding/following{/other_user}","gists_url":"https://api.github.com/users/AlanCoding/gists{/gist_id}","starred_url":"https://api.github.com/users/AlanCoding/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AlanCoding/subscriptions","organizations_url":"https://api.github.com/users/AlanCoding/orgs","repos_url":"https://api.github.com/users/AlanCoding/repos","events_url":"https://api.github.com/users/AlanCoding/events{/privacy}","received_events_url":"https://api.github.com/users/AlanCoding/received_events","type":"User","site_admin":false},"created_at":"2018-11-09T18:57:19Z","updated_at":"2018-11-09T18:57:19Z","author_association":"MEMBER","body":"Steps to reproduce:\r\n\r\n - create a JT with null inventory, prompts for inventory\r\n - create a schedule for this JT\r\n - provide an inventory in prompts for schedule\r\n - check the API\r\n\r\n```json\r\n{\r\n    \"rrule\": \"DTSTART;TZID=America/New_York:20181115T000000 RRULE:FREQ=DAILY;INTERVAL=1;COUNT=1\",\r\n    \"id\": 4,\r\n    \"type\": \"schedule\",\r\n    \"url\": \"/api/v2/schedules/4/\",\r\n    \"related\": {\r\n        \"created_by\": \"/api/v2/users/1/\",\r\n        \"modified_by\": \"/api/v2/users/1/\",\r\n        \"inventory\": \"/api/v2/inventories/5/\",\r\n        \"credentials\": \"/api/v2/schedules/4/credentials/\",\r\n        \"unified_jobs\": \"/api/v2/schedules/4/jobs/\",\r\n        \"unified_job_template\": \"/api/v2/job_templates/430/\",\r\n        \"project\": \"/api/v2/projects/7/\"\r\n    },\r\n    \"summary_fields\": {\r\n        \"inventory\": {\r\n            \"id\": 5,\r\n            \"name\": \"Inventory - OriginalHusband䐧\",\r\n            \"description\": \"HateCarryMainSingStorageStealImpactCalendarCycleEquipment쐍\",\r\n            \"has_active_failures\": false,\r\n            \"total_hosts\": 0,\r\n            \"hosts_with_active_failures\": 0,\r\n            \"total_groups\": 0,\r\n            \"groups_with_active_failures\": 0,\r\n            \"has_inventory_sources\": false,\r\n            \"total_inventory_sources\": 0,\r\n            \"inventory_sources_with_failures\": 0,\r\n            \"organization_id\": 5,\r\n            \"kind\": \"\"\r\n        },\r\n        \"unified_job_template\": {\r\n            \"id\": 430,\r\n            \"name\": \"null inventory\",\r\n            \"description\": \"\",\r\n            \"unified_job_type\": \"job\"\r\n        },\r\n        \"created_by\": {\r\n            \"id\": 1,\r\n            \"username\": \"admin\",\r\n            \"first_name\": \"\",\r\n            \"last_name\": \"\"\r\n        },\r\n        \"modified_by\": {\r\n            \"id\": 1,\r\n            \"username\": \"admin\",\r\n            \"first_name\": \"\",\r\n            \"last_name\": \"\"\r\n        },\r\n        \"user_capabilities\": {\r\n            \"edit\": true,\r\n            \"delete\": true\r\n        }\r\n    },\r\n    \"created\": \"2018-11-09T18:54:49.327529Z\",\r\n    \"modified\": \"2018-11-09T18:54:49.327541Z\",\r\n    \"name\": \"fesfesfsfef\",\r\n    \"description\": \"every day for 1 time\",\r\n    \"extra_data\": {},\r\n    \"inventory\": 5,\r\n    \"job_type\": null,\r\n    \"job_tags\": null,\r\n    \"skip_tags\": null,\r\n    \"limit\": null,\r\n    \"diff_mode\": null,\r\n    \"verbosity\": null,\r\n    \"unified_job_template\": 430,\r\n    \"enabled\": true,\r\n    \"dtstart\": \"2018-11-15T05:00:00Z\",\r\n    \"dtend\": \"2018-11-15T05:00:00Z\",\r\n    \"next_run\": \"2018-11-15T05:00:00Z\",\r\n    \"timezone\": \"America/New_York\",\r\n    \"until\": \"\"\r\n}\r\n```\r\n\r\nThus, this entry becomes ambiguous with your change. If the JT also has an inventory, you don't know which is being displayed.\r\n\r\nAt least by limiting this change to inventory sources, we avoid any direct conflict with truly unpredictable server behavior. It will still be undocumented and impossible for a random observer to interpret, but at least it would solve the UI's problem without creating more problems."}}
{"actor":{"display_login":"AlanCoding"}}