2019-05-16 Meeting notes

Date & Location

Thursday 2019/05/16 @ 9 AM PST

Location: 158-210A and WebEx (https://jpl.webex.com/jpl/j.php?MTID=m8ba5f1719bfbed57ce25de777072ad4b)

Participants

  • @Frank Greguska

  • @Sujen Shah

  • @Lan Dang

  • @Gerald Manipon

  • @Namrata Malarout

  • @Hook Hua

  • @Mohammed Karim

Potential Topics

  • Code process (fork -> PR, branch -> PR)?

  • Ticket process (when to open ticket, ticket type, approvals)?

  • Testing expectations

  • When/how patch gets integrated and built

  • When/how a project should integrate the patch

  • Release process of hysds components

  • Discuss existing HySDS API design, best practices, and future development, so we can build automation/operation/reporting tools against APIs instead of directly accessing HySDS components or Elasticsearch.

  • Find a SIRI intern for fall to help organize/consolidate documentation on Atlassian wiki

Goals

  • Code Process

  • PR approvals? Currently can’t request approvers

  • PUNT – Decision: When to squash

  • PUNT – Decision: When to rebase

  • Decision: Should individuals Fork & PR, have a list of committers/mergers, or something else?

    • need to identify set of people with “admin” authority on repos

    • devs should be able to create feature branches

    • circleci should build feature branches

    • protect master and develop

    • up to dev if they want to fork & PR against feature branch

    • PR must have Jira --- should document how git/jira integration works

  • API Development

    • Recommendation to use Domain models so that we can work with the objects across all source code.

    • middleware code that abstracts the backend

    • API/library for external users (Pele) goes through middleware code

Discussion topics

Time

Item

Discussion Lead

Notes

Time

Item

Discussion Lead

Notes

:05-:10

Intro/Previous Actions

@Frank Greguska

Start meeting. Briefly cover previous action items

 

Circle ci can be used across all repos, some repos have it on already: hysds, hysds-dockerfiles, sciflo, prov_es, hysds_commons

:10-:25

Code Process

@Frank Greguska

Discussion on branching/forking workflow that we want to follow.

https://nvie.com/posts/a-successful-git-branching-model/

Yes, use this workflow

python2 fixes will be hot fixes

not all hot fixes necessarily make it to develop

 

 

:25-:40

API development

@Lan Dang

Discuss current HySDS APIs, design, and best practices to have more consistency in API design, and possibly consolidate or encourage use of common API for some use cases.

https://hysds-core.atlassian.net/wiki/spaces/v3/pages/172785723

:40-:55

How Projects Contribute to Core

@Sujen Shah

:55

Wrap up

@Frank Greguska

Wrap up meeting, decide on timeframe for next meetup

 

Notes

  •  

Action items

@Frank Greguska Send out Doodle for next meetup
@Frank Greguska Identify set of people with “admin” authority on repos
@Sujen Shah Document how git/jira integration works. What level of integration do we have enabled and how is it triggered?

Decisions

  1. We are following the workflow described here: https://nvie.com/posts/a-successful-git-branching-model/
  2. Devs should be able to create feature branches
  3. CircleCI should build feature branches
  4. Master & Develop branches should be protected
  5. Developer’s discretion if they would rather fork & PR or commit directly to a feature branch
  6. Python2 fixes will be hot fixes from python2 branch and should try to be python3 compatible
  7. Hot fixes should have a PR for getting back into develop
  8. There is a need for a middleware abstraction the warrants further discussion

 

Note: JPL employees can also get answers to HySDS questions at Stack Overflow Enterprise: