Skip to content

Return an error when the node quote cannot be found#1594

Merged
achamayou merged 6 commits into
microsoft:masterfrom
achamayou:error_when_quote_unknown
Sep 9, 2020
Merged

Return an error when the node quote cannot be found#1594
achamayou merged 6 commits into
microsoft:masterfrom
achamayou:error_when_quote_unknown

Conversation

@achamayou

Copy link
Copy Markdown
Member

Fix #1571

@achamayou achamayou requested a review from a team as a code owner September 9, 2020 14:20
@ghost

ghost commented Sep 9, 2020

Copy link
Copy Markdown

error_when_quote_unknown@12528 aka 20200909.42 vs master ewma over 50 builds from 11973 to 12518
images

Comment thread tests/governance.py Outdated
def test_no_quote(network, args, notifications_queue=None, verify=True):
primary, _ = network.find_nodes()
untrusted_node = network.create_and_add_pending_node(args.package, "localhost", args)
with untrusted_node.client(ca=os.path.join(untrusted_node.common_dir, f"{untrusted_node.node_id}.pem")) as uc:

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

client() otherwise defaults to the network identity as the root CA.

@achamayou achamayou force-pushed the error_when_quote_unknown branch from 098aab9 to bb4b823 Compare September 9, 2020 16:57
@achamayou achamayou merged commit 5edc3f8 into microsoft:master Sep 9, 2020
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.

/node/quote return code is incorrect before node is trusted

2 participants