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

Allow most printable ASCII chars for object label key (b-cli) #2548

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kb2ma
Copy link
Contributor

@kb2ma kb2ma commented Oct 20, 2022

Implemented as a balena-compose dependency update. See balena-io-modules/balena-compose#13 . Extends acceptable label characters to printable ASCII characters except space, single/double quotes and backtick.

Draft PR until balena-compose PR merged so uses the generated version number.

  • Updates minimum version of @balena/compose and @types/dockerode to correspond to balena-compose PR

As of 2023-03-22 this PR still is blocked on an overall system solution between clients and server. This PR is useful; however older clients likely will fail in some way if they receive a service containing the extra characters.

Updates npm-shrinkwrap.json

Major

@types/ssh2 0.5.52 --> 1.11.6, which removed its unnecessary dependency on ssh2-streams. See DefinitelyTyped/DefinitelyTyped#60982 , from 2022-06.

Minor

  • fp-ts 2.12.2 --> 2.13.1

Patch

  • docker-modem
  • io-ts
  • nan
  • pinejs-client-core
  • pinejs-client-request
  • @types/lodash

@kb2ma kb2ma added the dependencies Pull requests that update a dependency file label Oct 20, 2022
@kb2ma kb2ma requested review from Page- and dfunckt October 20, 2022 12:43
@kb2ma
Copy link
Contributor Author

kb2ma commented Oct 20, 2022

ResinCI/npm/linux-ubuntu-x86_64-node14 check fails 2 tests, both due to timeout failures > 12000 ms. However, my local tests pass on node 12 and node14. Notice time to run tests below is < 1000 ms in both cases.

  balena build
[Info] Interceptor server listening on 127.0.0.1:33601
[Info] Proxy server listening on 127.0.0.1:41493
    ✓ should create the expected tar stream (single container) (546ms)

  balena os configure
    ✓ should inject a valid config.json file (992ms)

@kb2ma
Copy link
Contributor Author

kb2ma commented Oct 20, 2022

@balena-ci retest

@kb2ma kb2ma marked this pull request as ready for review October 20, 2022 14:56
@kb2ma kb2ma marked this pull request as draft October 20, 2022 16:44
@kb2ma kb2ma changed the title Allow most printable ASCII chars for object label key Allow most printable ASCII chars for object label key (b-cli) Oct 22, 2022
@ab77 ab77 force-pushed the update_compose_label_chars branch from fa303c0 to 975df40 Compare November 11, 2022 00:52
@ab77 ab77 force-pushed the update_compose_label_chars branch from 975df40 to b843d20 Compare December 11, 2022 20:45
Implemented as a balena-compose dependency update.

Change-type: patch
Signed-off-by: Ken Bannister <[email protected]>
@kb2ma kb2ma force-pushed the update_compose_label_chars branch from b843d20 to 2782347 Compare March 22, 2023 10:26
@kb2ma
Copy link
Contributor Author

kb2ma commented Mar 22, 2023

Removed pinning dockerode to v3.3.3. That has been done in #2599 . Still waiting for merge of balena-io-modules/balena-compose#13 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant