Bug 1372638 – The protobuf should use qps rather than ops after upgrade

Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.

Bug 1372638 - The protobuf should use qps rather than ops after upgrade

Summary:	The protobuf should use qps rather than ops after upgrade

Status:	CLOSED CURRENTRELEASE

Aliases:	None

Product:	OpenShift Container Platform
Classification:	Red Hat
Component:	Upgrade (Show other bugs)
Sub Component:
Version:	3.3.0
Hardware:	Unspecified Unspecified

Priority	medium Severity medium
Target Milestone:	---
Target Release:	---
Assigned To:	Scott Dodson
QA Contact:	Anping Li
Docs Contact:

URL:
Whiteboard:
Keywords:

Depends On:
Blocks:
	Show dependency tree / graph

Reported:	2016-09-02 05:28 EDT by Anping Li
Modified:	2016-09-19 09:32 EDT (History)
CC List:	5 users (show)

See Also:
Fixed In Version:
Doc Type:	If docs needed, set a value
Doc Text:
Story Points:	---
Clone Of:
Environment:
Last Closed:	2016-09-19 09:32:18 EDT
Type:	Bug
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Attachments	(Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

External Trackers
Tracker	ID	Priority	Status	Summary	Last Updated
Red Hat Product Errata	RHBA-2016:1933	normal	SHIPPED_LIVE	Red Hat OpenShift Container Platform 3.3 Release Advisory	2016-09-27 09:24:36 EDT

Groups: None (edit)

Description Anping Li 2016-09-02 05:28:50 EDT

Description of problem:

The bug have been fix for installation. https://bugzilla.redhat.com/show_bug.cgi?id=1364431. it also should be fixed for upgrade.

Version-Release number of selected component (if applicable):
atomic-openshift-utils-3.3.20-1.git.0.d15a8dc.el7.noarch

How reproducible:
always

Steps to Reproduce:
1. check the master-config.yaml after upgrade.
grep -A 4 externalKubernetesClientConnectionOverrides master-config.yaml
  externalKubernetesClientConnectionOverrides:
    acceptContentTypes: application/vnd.kubernetes.protobuf,application/json
    burst: 400
    contentType: application/vnd.kubernetes.protobuf
    ops: 200
2. check node-config.yaml after upgrade

grep -A 4 masterClientConnectionOverrides node-config.yaml
masterClientConnectionOverrides:
  acceptContentTypes: application/vnd.kubernetes.protobuf,application/json
  burst: 40
  contentType: application/vnd.kubernetes.protobuf
  ops: 20

Actual results:


Expected results:
It should be   qps: 20

Additional info:

Comment 1 Jordan Liggitt 2016-09-02 09:14:49 EDT

fix in https://github.com/openshift/openshift-ansible/pull/2400

Comment 2 Scott Dodson 2016-09-02 13:56:29 EDT

backported to enterprise-3.3 branch here https://github.com/openshift/openshift-ansible/pull/2403

Comment 4 Anping Li 2016-09-13 23:18:22 EDT

qps was used in configure files now.

---
> kubeletArguments:
> masterClientConnectionOverrides:
>   acceptContentTypes: application/vnd.kubernetes.protobuf,application/json
>   burst: 40
>   contentType: application/vnd.kubernetes.protobuf
>   qps: 20

Note You need to log in before you can comment on or make changes to this bug.