Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[3.5 Backport] Resolve deprecation warnings in Contiv roles #4930

Merged

Conversation

mtnbikenc
Copy link
Member

Backports #4479

@mtnbikenc mtnbikenc self-assigned this Jul 28, 2017
@mtnbikenc
Copy link
Member Author

aos-ci-test

@mtnbikenc mtnbikenc requested a review from tbielawa July 28, 2017 18:55
@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.5_NOT_containerized, aos-ci-jenkins/OS_3.5_NOT_containerized_e2e_tests" for f39ab34 (logs)

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.5_containerized, aos-ci-jenkins/OS_3.5_containerized_e2e_tests" for f39ab34 (logs)

@openshift-bot
Copy link

error: "aos-ci-jenkins/OS_3.5_containerized, aos-ci-jenkins/OS_3.5_containerized_e2e_tests" for f39ab34 (logs)

@mtnbikenc
Copy link
Member Author

aos-ci-test

@openshift-bot
Copy link

error: aos-ci-jenkins/OS_3.5_containerized for f39ab34 (logs)

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.5_NOT_containerized, aos-ci-jenkins/OS_3.5_NOT_containerized_e2e_tests" for f39ab34 (logs)

@tbielawa
Copy link
Contributor

tbielawa commented Aug 1, 2017

Looks like CI fail was unrelated to this PR

[EnvInject] - Variables injected successfully.
[openshift-ansible-slave] $ python /tmp/hudson3005405940977483119.py
Updating status for f39ab34c7ea57efd2ceff9c7b4ccfee2ed6a426c
Failed to update commit status [HTTP 405]
{'date': 'Mon, 31 Jul 2017 16:59:40 GMT', 'x-github-request-id': 'FF2D:3089:21B9946:530C578:597F61FC', 'content-length': '166', 'content-type': 'text/html', 'server': 'GitHub.com'}
Traceback (most recent call last):
  File "/tmp/hudson3005405940977483119.py", line 39, in <module>
    print r.json()
  File "/usr/lib/python2.7/site-packages/requests/models.py", line 819, in json
    return json.loads(self.text, **kwargs)
  File "/usr/lib64/python2.7/site-packages/simplejson/__init__.py", line 501, in loads
    return _default_decoder.decode(s)
  File "/usr/lib64/python2.7/site-packages/simplejson/decoder.py", line 370, in decode
    obj, end = self.raw_decode(s)
  File "/usr/lib64/python2.7/site-packages/simplejson/decoder.py", line 393, in raw_decode
    return self.scan_once(s, idx=_w(s, idx).end())
simplejson.scanner.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
Build step 'Execute Python script' marked build as failure

Copy link
Contributor

@tbielawa tbielawa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 for implementing new guidlines

@mtnbikenc
Copy link
Member Author

aos-ci-test

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.5_NOT_containerized, aos-ci-jenkins/OS_3.5_NOT_containerized_e2e_tests" for f39ab34 (logs)

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.5_containerized, aos-ci-jenkins/OS_3.5_containerized_e2e_tests" for f39ab34 (logs)

@sdodson sdodson merged commit 0013483 into openshift:release-1.5 Aug 2, 2017
@mtnbikenc mtnbikenc deleted the release-1.5-contiv-warnings branch August 2, 2017 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants