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

Fail if remote url address doesnt exist #527

Merged
merged 2 commits into from
Sep 11, 2024
Merged

Fail if remote url address doesnt exist #527

merged 2 commits into from
Sep 11, 2024

Conversation

Itxaka
Copy link
Member

@Itxaka Itxaka commented Sep 10, 2024

We are not failing during manual install and with a remote url if that file doesnt exists, which can lead to users thinking that they are installing from a remote url but they are not, they are just using defaults.

This will make it so it fails if the remote url doesnt exists.

Fixes kairos-io/kairos#2609

Signed-off-by: Itxaka <itxaka@kairos.io>
@Itxaka Itxaka requested a review from a team September 10, 2024 14:07
Copy link

codecov bot commented Sep 10, 2024

Codecov Report

Attention: Patch coverage is 22.22222% with 7 lines in your changes missing coverage. Please review.

Project coverage is 50.08%. Comparing base (0cad53f) to head (e432838).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
internal/agent/install.go 22.22% 5 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #527      +/-   ##
==========================================
- Coverage   50.14%   50.08%   -0.06%     
==========================================
  Files          48       48              
  Lines        4573     4582       +9     
==========================================
+ Hits         2293     2295       +2     
- Misses       2020     2025       +5     
- Partials      260      262       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@jimmykarily jimmykarily left a comment

Choose a reason for hiding this comment

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

root@localhost:~# ./kairos-agent manual-install https://www.google.com/test
configuration file not found in remote address
root@localhost:~# ./kairos-agent manual-install https://asdf
Head "https://asdf": dial tcp: lookup asdf on 10.0.2.3:53: no such host

LGTM

@Itxaka Itxaka merged commit 419f091 into main Sep 11, 2024
11 of 13 checks passed
@Itxaka Itxaka deleted the fail_remote_url_404 branch September 11, 2024 13:57
renovate bot added a commit to kairos-io/provider-kairos that referenced this pull request Sep 19, 2024
…4.1 (#638)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[github.com/kairos-io/kairos-agent/v2](https://redirect.github.com/kairos-io/kairos-agent)
| `v2.14.0` -> `v2.14.1` |
[![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fkairos-io%2fkairos-agent%2fv2/v2.14.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fkairos-io%2fkairos-agent%2fv2/v2.14.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fkairos-io%2fkairos-agent%2fv2/v2.14.0/v2.14.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fkairos-io%2fkairos-agent%2fv2/v2.14.0/v2.14.1?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>kairos-io/kairos-agent
(github.com/kairos-io/kairos-agent/v2)</summary>

###
[`v2.14.1`](https://redirect.github.com/kairos-io/kairos-agent/releases/tag/v2.14.1)

[Compare
Source](https://redirect.github.com/kairos-io/kairos-agent/compare/v2.14.0...v2.14.1)

#### What's Changed

- Bump go to 1.23.1 by
[@&#8203;Itxaka](https://redirect.github.com/Itxaka) in
[kairos-io/kairos-agent#526
- Display net info on QR code page by
[@&#8203;Itxaka](https://redirect.github.com/Itxaka) in
[kairos-io/kairos-agent#525
- chore(deps): update securego/gosec action to v2.21.1 by
[@&#8203;renovate](https://redirect.github.com/renovate) in
[kairos-io/kairos-agent#528
- fix(deps): update module github.com/kairos-io/kairos-sdk to v0.4.3 by
[@&#8203;renovate](https://redirect.github.com/renovate) in
[kairos-io/kairos-agent#529
- Fail if remote url address doesnt exist by
[@&#8203;Itxaka](https://redirect.github.com/Itxaka) in
[kairos-io/kairos-agent#527
- fix(deps): update module github.com/google/go-github/v63 to v64 by
[@&#8203;renovate](https://redirect.github.com/renovate) in
[kairos-io/kairos-agent#532
- chore(deps): update google/osv-scanner-action action to v1.8.5 by
[@&#8203;renovate](https://redirect.github.com/renovate) in
[kairos-io/kairos-agent#534
- fix(deps): update module github.com/twpayne/go-vfs/v4 to v5 by
[@&#8203;renovate](https://redirect.github.com/renovate) in
[kairos-io/kairos-agent#533
- Improve get partitions and reset spec by
[@&#8203;Itxaka](https://redirect.github.com/Itxaka) in
[kairos-io/kairos-agent#530
- fix(deps): update module k8s.io/mount-utils to v0.31.1 by
[@&#8203;renovate](https://redirect.github.com/renovate) in
[kairos-io/kairos-agent#536
- Fix upgrade size calculation inside k8s by
[@&#8203;Itxaka](https://redirect.github.com/Itxaka) in
[kairos-io/kairos-agent#537
- Do not skip /run when counting the size by
[@&#8203;Itxaka](https://redirect.github.com/Itxaka) in
[kairos-io/kairos-agent#538
- fix(deps): update module github.com/google/go-github/v63 to v64 by
[@&#8203;renovate](https://redirect.github.com/renovate) in
[kairos-io/kairos-agent#539
- fix(deps): update module github.com/twpayne/go-vfs/v4 to v5 by
[@&#8203;renovate](https://redirect.github.com/renovate) in
[kairos-io/kairos-agent#540
- fix(deps): update module github.com/kairos-io/kairos-sdk to v0.4.4 by
[@&#8203;renovate](https://redirect.github.com/renovate) in
[kairos-io/kairos-agent#541
- fix(deps): update module github.com/google/go-github/v64 to v65 by
[@&#8203;renovate](https://redirect.github.com/renovate) in
[kairos-io/kairos-agent#542
- Move to use our ghw clone by
[@&#8203;Itxaka](https://redirect.github.com/Itxaka) in
[kairos-io/kairos-agent#535
- Run tests in parallel and output github formats on workflow by
[@&#8203;Itxaka](https://redirect.github.com/Itxaka) in
[kairos-io/kairos-agent#543

**Full Changelog**:
kairos-io/kairos-agent@v2.14.0...v2.14.1

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 11pm every weekday,before 7am
every weekday,every weekend" in timezone Europe/Brussels, Automerge - At
any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend Renovate](https://mend.io/renovate/).
View the [repository job
log](https://developer.mend.io/github/kairos-io/provider-kairos).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC44MC4wIiwidXBkYXRlZEluVmVyIjoiMzguODAuMCIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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.

kairos-agent manual-install does not accept a URL to config file
2 participants