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

IPv6 support and tests improvements #33

Merged
merged 8 commits into from
Nov 24, 2023
Merged

IPv6 support and tests improvements #33

merged 8 commits into from
Nov 24, 2023

Conversation

dann1
Copy link
Collaborator

@dann1 dann1 commented Nov 23, 2023 •
edited
Loading

Functionality

  • ENDPOINT selection is now based on the following priority ['EXTERNAL_IP', 'IP6', 'IP']
    • if none of those parameters are found the endpoint will be an empty string as noted in the schema
  • Removed a server info call

Tests

  • Added tests for server info calls
  • Added optional automatic service cleanup enabled by default
."Found leftover services as the user github_actions"
"795: FAILED_DEPLOY"
"793: Function"
  • In case of errors any service readable by the user running the tests will be deleted
  • on the cognit infra the test user is github_actions with extremely limited access
  • if tests are run as oneadmin then every service would be deleted. This is useful when running on dev envs
  • Added tests to verify ERROR in Functions

@dann1 dann1 changed the title Fix complementary calls IPv6 support and tests improvements Nov 23, 2023
@tinova tinova merged commit 154d60f into main Nov 24, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tinova tinova tinova approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dann1 @tinova