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.

Sign up for GitHub

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

Jump to bottom

Updating CI env with 2022.04 jcoin.datacommons.io 1649774923 #4475

Merged
merged 2 commits into from
Apr 14, 2022
Merged

Updating CI env with 2022.04 jcoin.datacommons.io 1649774923 #4475

merged 2 commits into from
Apr 14, 2022

Conversation

PlanXCyborg
Copy link
Contributor

Applying version 2022.04 to jcoin.datacommons.io

@PlanXCyborg PlanXCyborg requested review from hughestr and a team as code owners April 12, 2022 14:48
@PlanXCyborg PlanXCyborg added the gen3-release Enable additional tests and automation label Apr 12, 2022
@PlanXCyborg
Copy link
Contributor Author

jcoin.datacommons.io

❌ etlMapping.yaml - gitops.json mismatch

  • Field error: Field [project_id] in explorerConfig.filters.tabs.fields not found in etlMapping
  • Field error: Field [project_id] in explorerConfig.table.fields not found in etlMapping
  • Field error: Field [project_id] in explorerConfig.table.fields not found in etlMapping

@PlanXCyborg
Copy link
Contributor Author

jcoin.datacommons.io/manifest.json

Deployment changes

@PlanXCyborg
Copy link
Contributor Author

jcoin.datacommons.io

❌ etlMapping.yaml - gitops.json mismatch

  • Field error: Field [project_id] in explorerConfig.filters.tabs.fields not found in etlMapping
  • Field error: Field [project_id] in explorerConfig.table.fields not found in etlMapping
  • Field error: Field [project_id] in explorerConfig.table.fields not found in etlMapping

2 similar comments
@PlanXCyborg
Copy link
Contributor Author

jcoin.datacommons.io

❌ etlMapping.yaml - gitops.json mismatch

  • Field error: Field [project_id] in explorerConfig.filters.tabs.fields not found in etlMapping
  • Field error: Field [project_id] in explorerConfig.table.fields not found in etlMapping
  • Field error: Field [project_id] in explorerConfig.table.fields not found in etlMapping

@PlanXCyborg
Copy link
Contributor Author

jcoin.datacommons.io

❌ etlMapping.yaml - gitops.json mismatch

  • Field error: Field [project_id] in explorerConfig.filters.tabs.fields not found in etlMapping
  • Field error: Field [project_id] in explorerConfig.table.fields not found in etlMapping
  • Field error: Field [project_id] in explorerConfig.table.fields not found in etlMapping

@PlanXCyborg
Copy link
Contributor Author

jcoin.datacommons.io/manifest.json

Deployment changes

paulineribeyre
paulineribeyre reviewed Apr 13, 2022
View reviewed changes
jcoin.datacommons.io/manifest.json
"spark": "707767160287.dkr.ecr.us-east-1.amazonaws.com/gen3/gen3-spark:2022.04",
"hatchery": "707767160287.dkr.ecr.us-east-1.amazonaws.com/gen3/hatchery:2022.04",
"wts": "707767160287.dkr.ecr.us-east-1.amazonaws.com/gen3/workspace-token-service:2022.04",
"manifestservice": "707767160287.dkr.ecr.us-east-1.amazonaws.com/gen3/manifestservice:2022.04",
"ambassador": "quay.io/datawire/ambassador:1.4.2",
Copy link
Contributor

Choose a reason for hiding this comment

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

ECR?

Copy link
Contributor

Choose a reason for hiding this comment

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

do we even need ambassador 🤷‍♀️

Copy link
Contributor

Choose a reason for hiding this comment

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

I am not exactly familiar with ambassador service

Copy link
Contributor

Choose a reason for hiding this comment

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

yeah let's leave it in

@atharvar28 atharvar28 merged commit 04b33a6 into master Apr 14, 2022
@atharvar28 atharvar28 deleted the chore/apply_202204_to_jcoin_datacommons_io_1649774924 branch April 14, 2022 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@atharvar28 atharvar28 atharvar28 left review comments

@paulineribeyre paulineribeyre paulineribeyre approved these changes

@hughestr hughestr Awaiting requested review from hughestr

Assignees
No one assigned
Labels
gen3-release Enable additional tests and automation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@PlanXCyborg @paulineribeyre @atharvar28