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

Resign command doesn't filter builds by profile #2404

Closed
d4rky-pl opened this issue May 30, 2024 · 7 comments
Closed

Resign command doesn't filter builds by profile #2404

d4rky-pl opened this issue May 30, 2024 · 7 comments

Comments

@d4rky-pl
Copy link

d4rky-pl commented May 30, 2024

Build/Submit details page URL

No response

Summary

When running eas build:resign --platform ios --profile testing the interactive list of builds includes builds from all profiles, not just from testing

Managed or bare?

managed

Environment

  expo-env-info 1.2.0 environment info:
    System:
      OS: macOS 14.4
      Shell: 5.9 - /bin/zsh
    Binaries:
      Node: 16.15.0 - ~/.asdf/installs/nodejs/16.15.0/bin/node
      Yarn: 1.22.15 - ~/.asdf/installs/nodejs/16.15.0/bin/yarn
      npm: 8.5.5 - ~/.asdf/plugins/nodejs/shims/npm
      Watchman: 2024.05.06.00 - /opt/homebrew/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: DriverKit 23.5, iOS 17.5, macOS 14.5, tvOS 17.5, visionOS 1.2, watchOS 10.5
      Android SDK:
        API Levels: 33, 33, 34
        Build Tools: 30.0.3, 33.0.0, 33.0.2, 34.0.0, 34.0.0
        System Images: android-33 | Google APIs ARM 64 v8a, android-34 | Google APIs ARM 64 v8a
    IDEs:
      Xcode: 15.4/15F31d - /usr/bin/xcodebuild
    npmPackages:
      @expo/webpack-config: ^18.0.1 => 18.0.2
      babel-preset-expo: ^9.3.0 => 9.3.1
      expo: ^48.0.0 => 48.0.9
      react: 18.2.0 => 18.2.0
      react-dom: 18.2.0 => 18.2.0
      react-native: 0.71.4 => 0.71.4
      react-native-web: ~0.18.11 => 0.18.12
    npmGlobalPackages:
      eas-cli: 9.1.0
    Expo Workflow: managed

Error output

No response

Reproducible demo or steps to reproduce from a blank project

  1. Create a project with profileA and profileB in eas.json
  2. Build iOS apps for both profiles
  3. Run eas build --platform ios --profile profileA
@d4rky-pl d4rky-pl added the needs review Issue is ready to be reviewed by a maintainer label May 30, 2024
@szdziedzic
Copy link
Member

When running eas build --platform ios --profile testing the interactive list of builds includes builds from all profiles, not just from testing

You mean when running eas build:list --buildProfile testing --platform ios, right?

@d4rky-pl
Copy link
Author

d4rky-pl commented Jun 5, 2024

@szdziedzic I mean build:resign, my bad. There's no --buildProfile flag according to the documentation

(sorry for closing the ticket, butter fingers 😅)

@szdziedzic
Copy link
Member

szdziedzic commented Jun 5, 2024

Oh, sorry, now I get it. Yeah, you are right, thanks for the report!

@szdziedzic
Copy link
Member

@d4rky-pl Oh, actually, I believe the name of the --profile flag is quite unfortunate and this might be an unwanted behavior.

What I mean is that --profile is actually a target profile, so it defines a profile for which you want to use credentials and env vars to resign some other eligible build. I believe that a use case for some people can be resigning build originally created using profile A using credentials and env vars from profile B that may differ. So this new filtering by the --profile flag might be unwanted in their use case.

@szdziedzic
Copy link
Member

What I think we can do here is to create 2 new flags:

  • target-profile which will do what profile does
  • source-profile for filtering

What do you think?

@d4rky-pl
Copy link
Author

d4rky-pl commented Jun 6, 2024

@szdziedzic after hearing your explanation I think it makes a lot of sense, I didn't realize there could be a use-case like this as we use the build profiles for specific environments (staging, production etc) so we would never want to mix ENV vars or credentials this way

@szdziedzic
Copy link
Member

--target-profile and source-profile flags will be released in 9.2.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants