Skip to content

move status out of spec for operator api#1231

Merged
istio-testing merged 1 commit intoistio:masterfrom
morvencao:br_move_status_outof_spec_for_operator_api
Jan 11, 2020
Merged

move status out of spec for operator api#1231
istio-testing merged 1 commit intoistio:masterfrom
morvencao:br_move_status_outof_spec_for_operator_api

Conversation

@morvencao
Copy link
Copy Markdown
Member

@googlebot googlebot added the cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. label Jan 10, 2020
@istio-testing istio-testing added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jan 10, 2020
@morvencao
Copy link
Copy Markdown
Member Author

morvencao commented Jan 10, 2020

/hold

@morvencao morvencao added the do-not-merge/hold Block automatic merging of a PR. label Jan 10, 2020
@ostromart
Copy link
Copy Markdown
Contributor

LGTM, however I'm not completely sure this is the root cause of the problem. Anyway, I agree with this change so let's see what happens.

@morvencao
Copy link
Copy Markdown
Member Author

@ostromart Currently e2e-operator test is broken because of failing status update, and the reconcile loop can't be stopped.
I think istio/operator#741 can fix that temporarily.
However, after this PR is merged, we need to enhance the status update logic further.

@morvencao
Copy link
Copy Markdown
Member Author

/unhold

@morvencao morvencao removed the do-not-merge/hold Block automatic merging of a PR. label Jan 11, 2020
@istio-testing istio-testing merged commit 80fb3f4 into istio:master Jan 11, 2020
@morvencao morvencao deleted the br_move_status_outof_spec_for_operator_api branch January 11, 2020 00:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Set by the Google CLA bot to indicate the author of a PR has signed the Google CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Operator reconciler always gets CR not found reconciling error

5 participants