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.

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

k8s: Update negative confidential attestation k8s tests #9649

Closed
wants to merge 1 commit into from

Conversation

GabyCT
Copy link
Contributor

@GabyCT GabyCT commented May 16, 2024

This PR updates the negative confidential attestation k8s test to look for a message when we are trying to retrieve the logs from the pod.

This PR updates the negative confidential attestation k8s test
to look for a message when we are trying to retrieve the logs
from the pod.

Signed-off-by: Gabriela Cervantes <gabriela.cervantes.tellez@intel.com>
@GabyCT GabyCT added the area/confidential-containers Issues related to confidential containers (see also CCv0 branch) label May 16, 2024
@katacontainersbot katacontainersbot added the size/tiny Smallest and simplest task label May 16, 2024
cmd="kubectl logs aa-test-cc | grep -q aatest"
run $cmd
[ "$status" -eq 1 ]
kubectl logs aa-test-cc | grep -q "Failed to connect"
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you happen to have the full message that we get from the ASR? This isn't quite what I would have expected.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

mmm seems like there is still not message as I am seeing at the GHA that at the logs we still get

#   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
#                                  Dload  Upload   Total   Spent    Left  Speed
# 
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100   319  100   319    0     0    378      0 --:--:-- --:--:-- --:--:--   379

Copy link
Contributor

Choose a reason for hiding this comment

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

I think you might have to bump the version of guest components in versions.yaml to pick up the error handling change.

@GabyCT GabyCT closed this May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/confidential-containers Issues related to confidential containers (see also CCv0 branch) ok-to-test size/tiny Smallest and simplest task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants