remove ContextPathHash from driver definition

[tonistiigi]

Noticed we have https://github.com/docker/buildx/blob/master/driver/manager.go#L87 as part of driver generic properties (added with k8s driver). But this key does not seem to be driver-specific but more defines the build behavior.

I think it should be refactored so that build gets the Driver instance and then type checks against a SetStickyKey() interface that k8s driver can implement and calls it with the context path.

[9bany]

I noticed that contextPathHash is being used in

• bake
  

  buildx/commands/bake.go

  Line 90 in ae3299d

  contextPathHash, _ := os.Getwd()

• build
  

  buildx/commands/build.go

  Lines 227 to 234 in ae3299d

  	contextPathHash := options.contextPath
  	if absContextPath, err := filepath.Abs(contextPathHash); err == nil {
  	contextPathHash = absContextPath
  	}
  	b, err := builder.New(dockerCli,
  	builder.WithName(options.builder),
  	builder.WithContextPathHash(contextPathHash),
  	)

• controller
  

  buildx/controller/build/build.go

  Lines 155 to 158 in ae3299d

  	contextPathHash, err := filepath.Abs(in.ContextPath)
  	if err != nil {
  	contextPathHash = in.ContextPath
  	}

Would it be fine if I remove all of them and use contextPathHash, _ := os.Getwd() in manager.go#L87 with your approach, @tonistiigi?

Let me know if i miss anything, thanks !

[9bany]

Anyone work on it ? If no, I'm happy to fix it.