Skip to content
This repository has been archived by the owner on Mar 13, 2022. It is now read-only.

fix stream data decoding #112

Closed
wants to merge 45 commits into from
Closed

fix stream data decoding #112

wants to merge 45 commits into from

Commits on Jul 30, 2018

  1. Configuration menu
    Copy the full SHA
    529a72a View commit details
    Browse the repository at this point in the history

Commits on Aug 27, 2018

  1. Configuration menu
    Copy the full SHA
    d54efa7 View commit details
    Browse the repository at this point in the history

Commits on Dec 5, 2018

  1. Configuration menu
    Copy the full SHA
    86ae2de View commit details
    Browse the repository at this point in the history
  2. Verify Boilerplate fix

    iamneha committed Dec 5, 2018
    Configuration menu
    Copy the full SHA
    d56fdbc View commit details
    Browse the repository at this point in the history

Commits on Dec 11, 2018

  1. Make dependancy adal optional

    iamneha committed Dec 11, 2018
    Configuration menu
    Copy the full SHA
    375befb View commit details
    Browse the repository at this point in the history

Commits on Dec 12, 2018

  1. Configuration menu
    Copy the full SHA
    1637d56 View commit details
    Browse the repository at this point in the history

Commits on Jan 21, 2019

  1. fix stream data decoding

    Related to #88 and #104
    I suppose decoding complete data is better than decoding data chunks.
    dbazhal authored Jan 21, 2019
    Configuration menu
    Copy the full SHA
    989e4b3 View commit details
    Browse the repository at this point in the history
  2. complete data decoding

    dbazhal authored Jan 21, 2019
    Configuration menu
    Copy the full SHA
    545690c View commit details
    Browse the repository at this point in the history

Commits on Jan 23, 2019

  1. Merge pull request #108 from iamneha/make-adal-optional

    Make dependancy adal optional
    k8s-ci-robot authored Jan 23, 2019
    Configuration menu
    Copy the full SHA
    260f855 View commit details
    Browse the repository at this point in the history
  2. Merge pull request #107 from iamneha/verify-scripts

    Verify-boilerplate script
    k8s-ci-robot authored Jan 23, 2019
    Configuration menu
    Copy the full SHA
    8497dfb View commit details
    Browse the repository at this point in the history
  3. fix watching with a specified resource version

    The watch code reset the version to the last found in the
    response.
    When you first list existing objects and then start watching from that
    resource version the existing versions are older than the version you
    wanted and the watch starts from the wrong version after the first
    restart.
    This leads to for example already deleted objects ending in the stream
    again.
    
    Fix this by setting the minimum resource version to reset from to the
    input resource version. As long as k8s returns all objects in order in
    the watch this should work.
    We cannot use the integer value of the resource version to order it as
    one should be treat the value as opaque.
    
    Closes kubernetes-client/python#700
    juliantaylor committed Jan 23, 2019
    Configuration menu
    Copy the full SHA
    3c30a30 View commit details
    Browse the repository at this point in the history
  4. Merge pull request #109 from juliantaylor/fix-watch-reset

    fix watching with a specified resource version
    k8s-ci-robot authored Jan 23, 2019
    Configuration menu
    Copy the full SHA
    2d69e89 View commit details
    Browse the repository at this point in the history

Commits on Jan 31, 2019

  1. fix symbols definitions

    tx to micw comments
    dbazhal authored Jan 31, 2019
    Configuration menu
    Copy the full SHA
    97ad43b View commit details
    Browse the repository at this point in the history

Commits on Feb 1, 2019

  1. fix segment decode errors for watch

    ```
    Traceback (most recent call last):
      File "controller_unitproject/run.py", line 33, in <module>
        run()
      File "controller_unitproject/run.py", line 29, in run
        unitproject.run()
      File "/home/bajal/.local/share/virtualenvs/controller_unitproject-1u3cbEWP/lib/python3.7/site-packages/module_k8s_controller_sdk/controller.py", line 215, in run
        self.resync_and_watch()
      File "/home/bajal/.local/share/virtualenvs/controller_unitproject-1u3cbEWP/lib/python3.7/site-packages/module_k8s_controller_sdk/controller.py", line 209, in resync_and_watch
        from_version=last_known_resource_version
      File "/home/bajal/.local/share/virtualenvs/controller_unitproject-1u3cbEWP/lib/python3.7/site-packages/module_k8s_controller_sdk/watcher.py", line 92, in watch
        self.__watch(seconds_to_out)
      File "/home/bajal/.local/share/virtualenvs/controller_unitproject-1u3cbEWP/lib/python3.7/site-packages/module_k8s_controller_sdk/watcher.py", line 51, in __watch
        for event in stream:
      File "/home/bajal/.local/share/virtualenvs/controller_unitproject-1u3cbEWP/lib/python3.7/site-packages/kubernetes/watch/watch.py", line 130, in stream
        for line in iter_resp_lines(resp):
      File "/home/bajal/.local/share/virtualenvs/controller_unitproject-1u3cbEWP/lib/python3.7/site-packages/kubernetes/watch/watch.py", line 47, in iter_resp_lines
        seg = seg.decode('utf8')
    UnicodeDecodeError: 'utf-8' codec can't decode byte 0xd0 in position 2047: unexpected end of data
    ```
    dbazhal authored Feb 1, 2019
    Configuration menu
    Copy the full SHA
    89c802c View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    8aaa929 View commit details
    Browse the repository at this point in the history

Commits on Feb 4, 2019

  1. Configuration menu
    Copy the full SHA
    4d387d5 View commit details
    Browse the repository at this point in the history

Commits on Feb 7, 2019

  1. Update pycodestyle

    iamneha committed Feb 7, 2019
    Configuration menu
    Copy the full SHA
    0fc0d40 View commit details
    Browse the repository at this point in the history
  2. Merge pull request #115 from iamneha/pycodestylw

    Update pycodestyle
    k8s-ci-robot authored Feb 7, 2019
    Configuration menu
    Copy the full SHA
    f30d840 View commit details
    Browse the repository at this point in the history

Commits on Feb 11, 2019

  1. make ws client decode data only on OPCODE_TEXT

    Following comment #112 (comment)
    Tested on running controller.
    dbazhal authored Feb 11, 2019
    Configuration menu
    Copy the full SHA
    e413acd View commit details
    Browse the repository at this point in the history
  2. Update watch.py

    dbazhal authored Feb 11, 2019
    Configuration menu
    Copy the full SHA
    d928310 View commit details
    Browse the repository at this point in the history
  3. Update watch_test.py

    dbazhal authored Feb 11, 2019
    Configuration menu
    Copy the full SHA
    5acc854 View commit details
    Browse the repository at this point in the history
  4. Restore one-line link

    micw523 authored Feb 11, 2019
    Configuration menu
    Copy the full SHA
    0229f0a View commit details
    Browse the repository at this point in the history
  5. Merge pull request #116 from micw523/one-line-link

    Restore one-line link
    k8s-ci-robot authored Feb 11, 2019
    Configuration menu
    Copy the full SHA
    d2ac885 View commit details
    Browse the repository at this point in the history

Commits on Feb 12, 2019

  1. Configuration menu
    Copy the full SHA
    8e6f043 View commit details
    Browse the repository at this point in the history

Commits on Feb 17, 2019

  1. Merge pull request #114 from rlenferink/patch-owners

    Updated OWNERS to include link to docs
    k8s-ci-robot authored Feb 17, 2019
    Configuration menu
    Copy the full SHA
    e077f88 View commit details
    Browse the repository at this point in the history

Commits on Feb 18, 2019

  1. Configuration menu
    Copy the full SHA
    4750aa9 View commit details
    Browse the repository at this point in the history

Commits on Feb 19, 2019

  1. Added tests.

    mitar committed Feb 19, 2019
    Configuration menu
    Copy the full SHA
    ad06e5c View commit details
    Browse the repository at this point in the history

Commits on Feb 21, 2019

  1. Don't use break inside finally.

    It swallows exceptions.
    mitar committed Feb 21, 2019
    Configuration menu
    Copy the full SHA
    972a76a View commit details
    Browse the repository at this point in the history

Commits on Mar 14, 2019

  1. Merge pull request #93 from mitar/watch_and_logs

    Making watch work with read_namespaced_pod_log
    k8s-ci-robot authored Mar 14, 2019
    Configuration menu
    Copy the full SHA
    c4de8bd View commit details
    Browse the repository at this point in the history

Commits on Mar 15, 2019

  1. Configuration menu
    Copy the full SHA
    328b2d1 View commit details
    Browse the repository at this point in the history

Commits on Mar 18, 2019

  1. Merge pull request #94 from tomplus/feat/merge-kubeconfigs

    feat: merging kubeconfig files
    k8s-ci-robot authored Mar 18, 2019
    Configuration menu
    Copy the full SHA
    5c03b3b View commit details
    Browse the repository at this point in the history

Commits on Mar 30, 2019

  1. Add tests for updated pieces

    bpicolo committed Mar 30, 2019
    Configuration menu
    Copy the full SHA
    b3ddbd9 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    72a02cc View commit details
    Browse the repository at this point in the history

Commits on Apr 8, 2019

  1. Merge pull request #110 from axelsteingrimsson/add_email_scope_to_gcp…

    …_credentials_refresh
    
    Add email scope to GCP provided credential refresh
    k8s-ci-robot authored Apr 8, 2019
    Configuration menu
    Copy the full SHA
    1d5231c View commit details
    Browse the repository at this point in the history

Commits on Jun 19, 2019

  1. Merge pull request #79 from bpicolo/fix_base64_padding_for_kconfig

    Fix base64 padding for kube config
    roycaihw authored Jun 19, 2019
    2 Configuration menu
    Copy the full SHA
    487c918 View commit details
    Browse the repository at this point in the history

Commits on Jun 20, 2019

  1. fix stream data decoding

    Related to #88 and #104
    I suppose decoding complete data is better than decoding data chunks.
    dbazhal authored and Dmitry Bazhal committed Jun 20, 2019
    Configuration menu
    Copy the full SHA
    c1a0a37 View commit details
    Browse the repository at this point in the history
  2. complete data decoding

    dbazhal authored and Dmitry Bazhal committed Jun 20, 2019
    Configuration menu
    Copy the full SHA
    6cc9fa0 View commit details
    Browse the repository at this point in the history
  3. fix symbols definitions

    tx to micw comments
    dbazhal authored and Dmitry Bazhal committed Jun 20, 2019
    Configuration menu
    Copy the full SHA
    678224a View commit details
    Browse the repository at this point in the history
  4. fix segment decode errors for watch

    ```
    Traceback (most recent call last):
      File "controller_unitproject/run.py", line 33, in <module>
        run()
      File "controller_unitproject/run.py", line 29, in run
        unitproject.run()
      File "/home/bajal/.local/share/virtualenvs/controller_unitproject-1u3cbEWP/lib/python3.7/site-packages/module_k8s_controller_sdk/controller.py", line 215, in run
        self.resync_and_watch()
      File "/home/bajal/.local/share/virtualenvs/controller_unitproject-1u3cbEWP/lib/python3.7/site-packages/module_k8s_controller_sdk/controller.py", line 209, in resync_and_watch
        from_version=last_known_resource_version
      File "/home/bajal/.local/share/virtualenvs/controller_unitproject-1u3cbEWP/lib/python3.7/site-packages/module_k8s_controller_sdk/watcher.py", line 92, in watch
        self.__watch(seconds_to_out)
      File "/home/bajal/.local/share/virtualenvs/controller_unitproject-1u3cbEWP/lib/python3.7/site-packages/module_k8s_controller_sdk/watcher.py", line 51, in __watch
        for event in stream:
      File "/home/bajal/.local/share/virtualenvs/controller_unitproject-1u3cbEWP/lib/python3.7/site-packages/kubernetes/watch/watch.py", line 130, in stream
        for line in iter_resp_lines(resp):
      File "/home/bajal/.local/share/virtualenvs/controller_unitproject-1u3cbEWP/lib/python3.7/site-packages/kubernetes/watch/watch.py", line 47, in iter_resp_lines
        seg = seg.decode('utf8')
    UnicodeDecodeError: 'utf-8' codec can't decode byte 0xd0 in position 2047: unexpected end of data
    ```
    dbazhal authored and Dmitry Bazhal committed Jun 20, 2019
    Configuration menu
    Copy the full SHA
    ef65b04 View commit details
    Browse the repository at this point in the history
  5. correct decoding inside of watch

    dbazhal authored and Dmitry Bazhal committed Jun 20, 2019
    Configuration menu
    Copy the full SHA
    fc1dc36 View commit details
    Browse the repository at this point in the history
  6. make ws client decode data only on OPCODE_TEXT

    Following comment #112 (comment)
    Tested on running controller.
    dbazhal authored and Dmitry Bazhal committed Jun 20, 2019
    Configuration menu
    Copy the full SHA
    8a6f093 View commit details
    Browse the repository at this point in the history
  7. Update watch.py

    dbazhal authored and Dmitry Bazhal committed Jun 20, 2019
    Configuration menu
    Copy the full SHA
    9008491 View commit details
    Browse the repository at this point in the history
  8. Update watch_test.py

    dbazhal authored and Dmitry Bazhal committed Jun 20, 2019
    Configuration menu
    Copy the full SHA
    4295b2d View commit details
    Browse the repository at this point in the history
  9. no need to fix that

    Dmitry Bazhal committed Jun 20, 2019
    Configuration menu
    Copy the full SHA
    e8db1ba View commit details
    Browse the repository at this point in the history
  10. Merge branch 'master' of github.com:dbazhal/python-base

    Dmitry Bazhal committed Jun 20, 2019
    Configuration menu
    Copy the full SHA
    c36eada View commit details
    Browse the repository at this point in the history