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: use upstreamed parts of craft-application #5239

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

bepri
Copy link
Contributor

@bepri bepri commented Feb 6, 2025

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • Have you successfully run tox run -m lint?
  • Have you successfully run tox run -e test-py310? (supported versions: py39, py310, py311, py312)

@bepri bepri self-assigned this Feb 6, 2025
@bepri bepri force-pushed the work/refactor-remote-build/CRAFT-4044 branch from affe7a0 to 4af108f Compare February 6, 2025 18:43
@mr-cal
Copy link
Collaborator

mr-cal commented Feb 7, 2025

I triggered a run for the remote build spread tests: https://github.com/canonical/snapcraft/actions/runs/13205639741

Copy link
Collaborator

@mr-cal mr-cal left a comment

Choose a reason for hiding this comment

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

Thanks for putting this up for early testing. Overall, everything is working well. I found a few things that need to be addressed in craft-application:

  1. --project is failing because this needs to be called before this.

  2. remote-build: add "Pending" as a possible status output #4956 should be upstreamed. It can be out-of-scope of this task, but we should land it prior to this refactor making its way into a Snapcraft release.

  3. Using the craft-application args makes the help messages look less cohesive. Some begin with a capital letter, others don't:

Options:
                          -h, --help:  Show this help message and exit
                       -v, --verbose:  Show debug information and be
                                       more verbose
                         -q, --quiet:  Only show warnings and errors,
                                       not progress
                         --verbosity:  Set the verbosity level to
                                       'quiet', 'brief', 'verbose',
                                       'debug' or 'trace'
                       -V, --version:  Show the application version and
                                       exit
                           --recover:  recover an interrupted build
    --launchpad-accept-public-upload:  acknowledge that uploaded code
                                       will be publicly available.
                 --launchpad-timeout:  Time in seconds to wait for
                                       launchpad to build.
                           --project:  upload to the specified Launchpad
                                       project
                         --build-for:  Comma-separated list of
                                       architectures to build for

Copy link
Collaborator

Choose a reason for hiding this comment

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

Do you mind making this change to the overview?

https://github.com/canonical/craft-application/pull/623/files

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.

2 participants