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

chore: fix error message in cli dev/deploy #697

Merged
merged 1 commit into from
Mar 29, 2024
Merged

chore: fix error message in cli dev/deploy #697

merged 1 commit into from
Mar 29, 2024

Conversation

cmwylie19
Copy link
Collaborator

Description

Changes the error message to be readable by the user:

└─[0] <git:(main✈) > npx ts-node ../src/cli.ts deploy -i pepr:dev --confirm

  dist/pepr-static-test.js            3.6kb  100.0%
   ├ capabilities/hello-pepr.ts       2.8kb   77.4%
   ├ package.json                     657b    17.9%
   └ pepr.ts                           46b     1.3%

  dist/pepr-static-test.js.LEGAL.txt    0b   100.0%

Registered Pepr Capability "hello-pepr"
{"level":30,"time":1711655887945,"pid":57319,"hostname":"Cases-MacBook-Pro.local","msg":"Establishing connection to Kubernetes"}
Error deploying module: No currently active cluster
{"level":30,"time":1711655887946,"pid":57319,"hostname":"Cases-MacBook-Pro.local","msg":"Applying pepr-system namespace"}

Related Issue

Fixes #696

Relates to #

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Other (security config, docs update, etc)

Checklist before merging

Signed-off-by: Case Wylie <cmwylie19@defenseunicorns.com>
Copy link
Collaborator

@btlghrants btlghrants left a comment

Choose a reason for hiding this comment

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

LGTM

@btlghrants btlghrants merged commit 84b9726 into main Mar 29, 2024
11 checks passed
@btlghrants btlghrants deleted the 696 branch March 29, 2024 14:53
mjnagel pushed a commit to defenseunicorns/uds-core that referenced this pull request Apr 2, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence | Type |
Update |
|---|---|---|---|---|---|---|---|
| [pepr](https://togithub.com/defenseunicorns/pepr) | [`0.28.6` ->
`0.28.7`](https://renovatebot.com/diffs/npm/pepr/0.28.6/0.28.7) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/pepr/0.28.7?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/pepr/0.28.7?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/pepr/0.28.6/0.28.7?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/pepr/0.28.6/0.28.7?slim=true)](https://docs.renovatebot.com/merge-confidence/)
| dependencies | patch |
|
[registry1.dso.mil/ironbank/opensource/defenseunicorns/pepr/controller](https://togithub.com/defenseunicorns/pepr)
([source](https://repo1.dso.mil/dsop/opensource/defenseunicorns/pepr/controller))
| `v0.28.6` -> `v0.28.7` |
[![age](https://developer.mend.io/api/mc/badges/age/docker/registry1.dso.mil%2fironbank%2fopensource%2fdefenseunicorns%2fpepr%2fcontroller/v0.28.7?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/docker/registry1.dso.mil%2fironbank%2fopensource%2fdefenseunicorns%2fpepr%2fcontroller/v0.28.7?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/docker/registry1.dso.mil%2fironbank%2fopensource%2fdefenseunicorns%2fpepr%2fcontroller/v0.28.6/v0.28.7?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/docker/registry1.dso.mil%2fironbank%2fopensource%2fdefenseunicorns%2fpepr%2fcontroller/v0.28.6/v0.28.7?slim=true)](https://docs.renovatebot.com/merge-confidence/)
| | patch |

---

### Release Notes

<details>
<summary>defenseunicorns/pepr (pepr)</summary>

###
[`v0.28.7`](https://togithub.com/defenseunicorns/pepr/releases/tag/v0.28.7)

[Compare
Source](https://togithub.com/defenseunicorns/pepr/compare/v0.28.6...v0.28.7)

#### What's Changed

- chore: fix error message in cli dev/deploy by
[@&#8203;cmwylie19](https://togithub.com/cmwylie19) in
[defenseunicorns/pepr#697
- chore: add test for when capability is schedule by
[@&#8203;cmwylie19](https://togithub.com/cmwylie19) in
[defenseunicorns/pepr#693
- chore: fixed typo by
[@&#8203;naveensrinivasan](https://togithub.com/naveensrinivasan) in
[defenseunicorns/pepr#682
- chore: bump actions/dependency-review-action from 4.2.3 to 4.2.4 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[defenseunicorns/pepr#686
- chore: bump the production-dependencies group with 1 update by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[defenseunicorns/pepr#687
- chore: bump the production-dependencies group with 1 update by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[defenseunicorns/pepr#691
- chore: bump actions/dependency-review-action from 4.2.4 to 4.2.5 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[defenseunicorns/pepr#692

**Full Changelog**:
defenseunicorns/pepr@v0.28.6...v0.28.7

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

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

---

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

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/defenseunicorns/uds-core).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yNjkuMiIsInVwZGF0ZWRJblZlciI6IjM3LjI2OS4yIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
robmcelvenny pushed a commit to owen-grady/uds-core-slim-dev that referenced this pull request Jun 3, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence | Type |
Update |
|---|---|---|---|---|---|---|---|
| [pepr](https://togithub.com/defenseunicorns/pepr) | [`0.28.6` ->
`0.28.7`](https://renovatebot.com/diffs/npm/pepr/0.28.6/0.28.7) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/pepr/0.28.7?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/pepr/0.28.7?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/pepr/0.28.6/0.28.7?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/pepr/0.28.6/0.28.7?slim=true)](https://docs.renovatebot.com/merge-confidence/)
| dependencies | patch |
|
[registry1.dso.mil/ironbank/opensource/defenseunicorns/pepr/controller](https://togithub.com/defenseunicorns/pepr)
([source](https://repo1.dso.mil/dsop/opensource/defenseunicorns/pepr/controller))
| `v0.28.6` -> `v0.28.7` |
[![age](https://developer.mend.io/api/mc/badges/age/docker/registry1.dso.mil%2fironbank%2fopensource%2fdefenseunicorns%2fpepr%2fcontroller/v0.28.7?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/docker/registry1.dso.mil%2fironbank%2fopensource%2fdefenseunicorns%2fpepr%2fcontroller/v0.28.7?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/docker/registry1.dso.mil%2fironbank%2fopensource%2fdefenseunicorns%2fpepr%2fcontroller/v0.28.6/v0.28.7?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/docker/registry1.dso.mil%2fironbank%2fopensource%2fdefenseunicorns%2fpepr%2fcontroller/v0.28.6/v0.28.7?slim=true)](https://docs.renovatebot.com/merge-confidence/)
| | patch |

---

### Release Notes

<details>
<summary>defenseunicorns/pepr (pepr)</summary>

###
[`v0.28.7`](https://togithub.com/defenseunicorns/pepr/releases/tag/v0.28.7)

[Compare
Source](https://togithub.com/defenseunicorns/pepr/compare/v0.28.6...v0.28.7)

#### What's Changed

- chore: fix error message in cli dev/deploy by
[@&#8203;cmwylie19](https://togithub.com/cmwylie19) in
[defenseunicorns/pepr#697
- chore: add test for when capability is schedule by
[@&#8203;cmwylie19](https://togithub.com/cmwylie19) in
[defenseunicorns/pepr#693
- chore: fixed typo by
[@&#8203;naveensrinivasan](https://togithub.com/naveensrinivasan) in
[defenseunicorns/pepr#682
- chore: bump actions/dependency-review-action from 4.2.3 to 4.2.4 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[defenseunicorns/pepr#686
- chore: bump the production-dependencies group with 1 update by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[defenseunicorns/pepr#687
- chore: bump the production-dependencies group with 1 update by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[defenseunicorns/pepr#691
- chore: bump actions/dependency-review-action from 4.2.4 to 4.2.5 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[defenseunicorns/pepr#692

**Full Changelog**:
defenseunicorns/pepr@v0.28.6...v0.28.7

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

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

---

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

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/defenseunicorns/uds-core).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yNjkuMiIsInVwZGF0ZWRJblZlciI6IjM3LjI2OS4yIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

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
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

unreadable error message in deploy and dev CLI commands
2 participants