{"action":"created","issue":{"url":"https://api.github.com/repos/aws/sagemaker-python-sdk/issues/965","repository_url":"https://api.github.com/repos/aws/sagemaker-python-sdk","labels_url":"https://api.github.com/repos/aws/sagemaker-python-sdk/issues/965/labels{/name}","comments_url":"https://api.github.com/repos/aws/sagemaker-python-sdk/issues/965/comments","events_url":"https://api.github.com/repos/aws/sagemaker-python-sdk/issues/965/events","html_url":"https://github.com/aws/sagemaker-python-sdk/pull/965","id":476382338,"node_id":"MDExOlB1bGxSZXF1ZXN0MzAzOTUxNzU0","number":965,"title":"fix: allow Airflow enabled estimators to use absolute path entry_point","user":{"login":"icywang86rui","id":24926915,"node_id":"MDQ6VXNlcjI0OTI2OTE1","avatar_url":"https://avatars0.githubusercontent.com/u/24926915?v=4","gravatar_id":"","url":"https://api.github.com/users/icywang86rui","html_url":"https://github.com/icywang86rui","followers_url":"https://api.github.com/users/icywang86rui/followers","following_url":"https://api.github.com/users/icywang86rui/following{/other_user}","gists_url":"https://api.github.com/users/icywang86rui/gists{/gist_id}","starred_url":"https://api.github.com/users/icywang86rui/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/icywang86rui/subscriptions","organizations_url":"https://api.github.com/users/icywang86rui/orgs","repos_url":"https://api.github.com/users/icywang86rui/repos","events_url":"https://api.github.com/users/icywang86rui/events{/privacy}","received_events_url":"https://api.github.com/users/icywang86rui/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2019-08-02T22:53:53Z","updated_at":"2019-08-02T22:57:40Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/aws/sagemaker-python-sdk/pulls/965","html_url":"https://github.com/aws/sagemaker-python-sdk/pull/965","diff_url":"https://github.com/aws/sagemaker-python-sdk/pull/965.diff","patch_url":"https://github.com/aws/sagemaker-python-sdk/pull/965.patch"},"body":"*Issue #, if available:*\r\n\r\nhttps://github.com/aws/sagemaker-python-sdk/issues/770\r\n*Description of changes:*\r\n\r\nCurrently the Airflow logic always uses basename of entry_point when\r\ntaring up the source code and upload to s3. So if only the absolute\r\npath of the script is provided the pre-processing will fail. This change\r\nfixes the issue.\r\n\r\n## Merge Checklist\r\n\r\n_Put an `x` in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request._\r\n\r\n- [ ] I have read the [CONTRIBUTING](https://github.com/aws/sagemaker-python-sdk/blob/master/CONTRIBUTING.md) doc\r\n- [ ] I used the commit message format described in [CONTRIBUTING](https://github.com/aws/sagemaker-python-sdk/blob/master/CONTRIBUTING.md#commit-message-guidlines)\r\n- [ ] I have added tests that prove my fix is effective or that my feature works (if appropriate)\r\n- [ ] I have updated any necessary [documentation](https://github.com/aws/sagemaker-python-sdk/blob/master/README.rst) (if appropriate)\r\n\r\nBy submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.\r\n"},"comment":{"url":"https://api.github.com/repos/aws/sagemaker-python-sdk/issues/comments/517866660","html_url":"https://github.com/aws/sagemaker-python-sdk/pull/965#issuecomment-517866660","issue_url":"https://api.github.com/repos/aws/sagemaker-python-sdk/issues/965","id":517866660,"node_id":"MDEyOklzc3VlQ29tbWVudDUxNzg2NjY2MA==","user":{"login":"jesterhazy","id":505258,"node_id":"MDQ6VXNlcjUwNTI1OA==","avatar_url":"https://avatars0.githubusercontent.com/u/505258?v=4","gravatar_id":"","url":"https://api.github.com/users/jesterhazy","html_url":"https://github.com/jesterhazy","followers_url":"https://api.github.com/users/jesterhazy/followers","following_url":"https://api.github.com/users/jesterhazy/following{/other_user}","gists_url":"https://api.github.com/users/jesterhazy/gists{/gist_id}","starred_url":"https://api.github.com/users/jesterhazy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jesterhazy/subscriptions","organizations_url":"https://api.github.com/users/jesterhazy/orgs","repos_url":"https://api.github.com/users/jesterhazy/repos","events_url":"https://api.github.com/users/jesterhazy/events{/privacy}","received_events_url":"https://api.github.com/users/jesterhazy/received_events","type":"User","site_admin":false},"created_at":"2019-08-02T22:57:40Z","updated_at":"2019-08-02T22:57:40Z","author_association":"CONTRIBUTOR","body":"\n### AWS CodeBuild CI Report\n\n* Result: FAILED\n* [Build Logs](https://mdk06ma93j.execute-api.us-west-2.amazonaws.com/Prod/buildlogs?key=11f74862-0388-4909-b720-8f042208f22e%2Fbuild.log) (available for 30 days)\n\n*Powered by [github-codebuild-logs](https://serverlessrepo.aws.amazon.com/applications/arn:aws:serverlessrepo:us-east-1:277187709615:applications~github-codebuild-logs), available on the [AWS Serverless Application Repository](https://aws.amazon.com/serverless/serverlessrepo/)*\n"}}
{"actor":{"display_login":"jesterhazy"}}