Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 11, 2026

The Azure Relay Tests workflow was failing because azure/login@v1 doesn't accept client-secret as a standalone parameter. The action requires credentials in JSON format via the creds parameter.

Changes

Workflow (.github/workflows/azure-relay-tests.yml):

  • Updated 3 Azure Login steps from individual parameters to JSON credentials:
    # Before
    - name: Azure Login
      uses: azure/login@v1
      with:
        client-id: ${{ secrets.AZURE_CLIENT_ID }}
        tenant-id: ${{ secrets.AZURE_TENANT_ID }}
        subscription-id: ${{ secrets.AZURE_SUBSCRIPTION_ID }}
        client-secret: ${{ secrets.AZURE_CLIENT_SECRET }}  # Invalid parameter
    
    # After
    - name: Azure Login
      uses: azure/login@v1
      with:
        creds: ${{ secrets.AZURE_CREDENTIALS }}
  • Removed unused AZURE_SUBSCRIPTION_ID environment variable (already in JSON)

Documentation:

  • Updated azure/docs/SECRETS.md: Primary method now uses single AZURE_CREDENTIALS JSON secret
  • Updated azure/docs/MIGRATION_GUIDE.md: Instructions for creating service principal with --sdk-auth
  • Updated azure/IMPLEMENTATION_SUMMARY.md: Corrected prerequisites

Required Setup

Repository admin must configure AZURE_CREDENTIALS secret:

az ad sp create-for-rbac \
  --name "pluresdb-github-actions" \
  --role contributor \
  --scopes /subscriptions/{subscription-id} \
  --sdk-auth

Then add the entire JSON output as AZURE_CREDENTIALS secret in GitHub repository settings.

Original prompt

This section details on the original issue you should resolve

<issue_title>Azure Relay Tests Failed - test</issue_title>
<issue_description>The Azure relay tests failed for the test environment.

Details:

  • Workflow: Azure Relay Tests
  • Run ID: 20888287721
  • Environment: test
  • Node Count: 3

Please review the workflow logs:
https://github.com/plures/pluresdb/actions/runs/20888287721

Next Steps:

  1. Review test failures in the workflow logs
  2. Check Azure container logs for errors
  3. Verify network connectivity between nodes
  4. Review recent code changes that may have affected P2P relay

Related Documentation:

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 5 commits January 11, 2026 06:42
- Changed azure/login@v1 to use 'creds' parameter with JSON credentials
- Updated all three Azure Login steps in the workflow
- Removed invalid 'client-secret' parameter that was causing failures
- Updated SECRETS.md with correct configuration instructions
- Updated MIGRATION_GUIDE.md with proper migration steps

Co-authored-by: kayodebristol <3579196+kayodebristol@users.noreply.github.com>
…uirement

Co-authored-by: kayodebristol <3579196+kayodebristol@users.noreply.github.com>
The subscription ID is already included in the AZURE_CREDENTIALS JSON,
so there's no need for a separate environment variable or secret.

Co-authored-by: kayodebristol <3579196+kayodebristol@users.noreply.github.com>
Co-authored-by: kayodebristol <3579196+kayodebristol@users.noreply.github.com>
Co-authored-by: kayodebristol <3579196+kayodebristol@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix Azure relay tests for test environment Fix Azure Login authentication in relay tests workflow Jan 11, 2026
Copilot AI requested a review from kayodebristol January 11, 2026 06:51
@kayodebristol kayodebristol marked this pull request as ready for review January 11, 2026 11:53
@kayodebristol kayodebristol merged commit ee89431 into main Jan 11, 2026
3 checks passed
@kayodebristol kayodebristol deleted the copilot/fix-azure-relay-tests branch January 11, 2026 11:53
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.

Azure Relay Tests Failed - test

2 participants