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

Test samples step broken after switching to UseFederatedAuth #36467

Closed
kristapratico opened this issue Jul 12, 2024 · 3 comments · Fixed by #36499 or #36598
Closed

Test samples step broken after switching to UseFederatedAuth #36467

kristapratico opened this issue Jul 12, 2024 · 3 comments · Fixed by #36499 or #36598
Assignees
Labels
Central-EngSys This issue is owned by the Engineering System team.

Comments

@kristapratico
Copy link
Member

kristapratico commented Jul 12, 2024

The test samples step stopped running (just shows pass) after UseFederatedAuth: true is set in the tests.yml.

Not sure if this is the source of the error:

Last exit code:
Error: no "view" rule for type "text/x-python" passed its test case
(for more information, add "--debug=1" on the command line)
/usr/bin/xdg-open: 869: www-browser: not found
/usr/bin/xdg-open: 869: links2: not found
/usr/bin/xdg-open: 869: elinks: not found
/usr/bin/xdg-open: 869: links: not found
/usr/bin/xdg-open: 869: lynx: not found
/usr/bin/xdg-open: 869: w3m: not found
xdg-open: no method available for opening '/mnt/vss/_work/1/s/scripts/devops_tasks/dispatch_tox.py'

Example with eventgrid:

Added UseFederatedAuth: true on 5/31: https://github.com/Azure/azure-sdk-for-python/pull/35851/files

Test samples step before PR (5/30): https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3833023&view=logs&j=fe7804ca-eaac-501a-67e8-e2604a50cb52&t=fef01369-6249-5df0-3692-3e5dec542b9f

Test samples step after PR (7/10): https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3950597&view=logs&j=c8837ca9-7e4f-5904-07be-77c9cad87f76&t=c74bc2cc-a575-5ee1-ad8c-df8b1ef5314a

@kristapratico kristapratico added the Central-EngSys This issue is owned by the Engineering System team. label Jul 12, 2024
@scbedd
Copy link
Member

scbedd commented Jul 16, 2024

Taking a look today.

@pvaneck pvaneck reopened this Jul 23, 2024
@scbedd
Copy link
Member

scbedd commented Jul 24, 2024

Agreed. I don't think the federated samples ever worked tbh. I think this PR fixes it. #36598

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Central-EngSys This issue is owned by the Engineering System team.
Projects
3 participants