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.

Sign up for GitHub

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

Jump to bottom

Error status being report from OPA's discovery bundle #2630

Closed
xin-styra opened this issue Aug 13, 2020 · 0 comments · Fixed by #2638
Closed

Error status being report from OPA's discovery bundle #2630

xin-styra opened this issue Aug 13, 2020 · 0 comments · Fixed by #2638
Assignees
@ashutosh-narkar
Labels
bug

Comments

@xin-styra
Copy link
Contributor

xin-styra commented Aug 13, 2020 •
edited
Loading

Expected Behavior

last_successful_request and last_request for discovery bundle in any reported status should be larger than "0001-01-01T00:00:00Z" which is the start of time. That means OPA never fetched bundle. last_successful_activation and last_successful_download should also be at least the same.

Actual Behavior

But in the same status reported, "last_successful_activation" and "last_successful_download" are updated in the same status.

"discovery": {
"code": "bundle_error",
"last_request": "0001-01-01T00:00:00Z",
"last_successful_activation": "2020-07-19T00:10:58.164191894Z",
"last_successful_download": "2020-07-19T00:10:58.162475509Z",
"last_successful_request": "0001-01-01T00:00:00Z",
"message": "server replied with HTTP 503",
"name": "discovery"
},

Steps to Reproduce the Problem

From actual usage of "version": "0.20.5"

Additional Info
@ashutosh-narkar ashutosh-narkar self-assigned this Aug 14, 2020
ashutosh-narkar added a commit to ashutosh-narkar/opa that referenced this issue Aug 19, 2020
@ashutosh-narkar
plugins/discovery: Set the last request and last successful request i…
4b3c038 
…n discovery status

This commmit sets the values for the last request and last successful request
in the discovery bundle status.

Fixes: open-policy-agent#2630

Signed-off-by: Ashutosh Narkar <anarkar4387@gmail.com>
@ashutosh-narkar ashutosh-narkar mentioned this issue Aug 19, 2020
Merged
ashutosh-narkar added a commit that referenced this issue Aug 19, 2020
@ashutosh-narkar
plugins/discovery: Set the last request and last successful request i…
30271bc 
…n discovery status

This commmit sets the values for the last request and last successful request
in the discovery bundle status.

Fixes: #2630

Signed-off-by: Ashutosh Narkar <anarkar4387@gmail.com>
tsandall pushed a commit to tsandall/opa that referenced this issue Aug 20, 2020
@ashutosh-narkar @tsandall
plugins/discovery: Set the last request and last successful request i…
7481066 
…n discovery status

This commmit sets the values for the last request and last successful request
in the discovery bundle status.

Fixes: open-policy-agent#2630

Signed-off-by: Ashutosh Narkar <anarkar4387@gmail.com>
(cherry picked from commit 30271bc)
tsandall pushed a commit that referenced this issue Aug 20, 2020
@ashutosh-narkar @tsandall
plugins/discovery: Set the last request and last successful request i…
49914dd 
…n discovery status

This commmit sets the values for the last request and last successful request
in the discovery bundle status.

Fixes: #2630

Signed-off-by: Ashutosh Narkar <anarkar4387@gmail.com>
(cherry picked from commit 30271bc)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ashutosh-narkar ashutosh-narkar

Labels
bug
Projects
Open Policy Agent
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add response header timeout and update discovery bundle status ashutosh-narkar/opa
3 participants
@ashutosh-narkar @patrick-east @xin-styra