Skip to content

feat: allow ipv6 services#218

Merged
clementnuss merged 2 commits intopostfinance:mainfrom
borislitv:main
Sep 15, 2025
Merged

feat: allow ipv6 services#218
clementnuss merged 2 commits intopostfinance:mainfrom
borislitv:main

Conversation

@borislitv
Copy link
Copy Markdown
Contributor

No description provided.

@borislitv
Copy link
Copy Markdown
Contributor Author

@clementnuss Can you see this please?

@clementnuss
Copy link
Copy Markdown
Contributor

hi and sorry for the delay, I never got notified. I'll adapt my notifications for this project!
will merge your contributions next week. thanks!

@coveralls
Copy link
Copy Markdown

coveralls commented Sep 15, 2025

Pull Request Test Coverage Report for Build 17732415870

Details

  • 1 of 4 (25.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.9%) to 70.072%

Changes Missing Coverage Covered Lines Changed/Added Lines %
internal/servicecheck/servicecheck.go 1 4 25.0%
Totals Coverage Status
Change from base Build 17731349157: 0.9%
Covered Lines: 487
Relevant Lines: 695

💛 - Coveralls

@clementnuss clementnuss merged commit c5aa96c into postfinance:main Sep 15, 2025
3 of 5 checks passed
@clementnuss
Copy link
Copy Markdown
Contributor

the build pipeline is broken, I'll release a new kubenurse after I've had time to fix it. hopefully this week !

clementnuss pushed a commit that referenced this pull request Sep 24, 2025
* feat: allow ipv6 services

* test(servicecheck): add ipv6.google.com check

---------

Co-authored-by: Clément Nussbaumer <clement@n8r.ch>

build(gh-action): add debug information
@clementnuss
Copy link
Copy Markdown
Contributor

finally found some time to fix it!
image

@borislitv
Copy link
Copy Markdown
Contributor Author

finally found some time to fix it! image

great, thank you very much

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.

3 participants