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

Updated input manifests #3074

Merged
merged 1 commit into from
Jan 10, 2023

Conversation

prudhvigodithi
Copy link
Member

@prudhvigodithi prudhvigodithi commented Jan 10, 2023

Signed-off-by: Prudhvi Godithi [email protected]

Description

Updated input manifests for 2.6.0
Creating a manual PR until the issue #3068 is addressed

Issues Resolved

N/A

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@prudhvigodithi prudhvigodithi requested a review from a team as a code owner January 10, 2023 21:22
manifests/2.6.0/opensearch-2.6.0.yml Outdated Show resolved Hide resolved
manifests/2.6.0/opensearch-dashboards-2.6.0.yml Outdated Show resolved Hide resolved
@saratvemulapalli
Copy link
Member

3.0.0 manifest is failing, looks like we split up the repos?

2023-01-10 21:41:21 ERROR    CI Manifest check failed
Traceback (most recent call last):
  File "./src/run_ci.py", line 25, in <module>
    sys.exit(main())
  File "./src/run_ci.py", line 20, in main
    CiManifests.from_file(args.manifest, args).check()
  File "/home/runner/work/opensearch-build/opensearch-build/src/ci_workflow/ci_manifest.py", line 22, in check
    self.__check__()
  File "/home/runner/work/opensearch-build/opensearch-build/src/ci_workflow/ci_input_manifest.py", line 42, in __check__
    ci_check_list.check()
  File "/home/runner/work/opensearch-build/opensearch-build/src/ci_workflow/ci_check_list_source.py", line 44, in check
    instance = klass(self.component, self.git_repo, self.target, check.args)
  File "/home/runner/work/opensearch-build/opensearch-build/src/ci_workflow/ci_check_gradle_properties.py", line 20, in __init__
    self.properties = self.__get_properties()
  File "/home/runner/work/opensearch-build/opensearch-build/src/ci_workflow/ci_check_gradle_properties.py", line 35, in __get_properties
    return PropertiesFile(self.git_repo.output(cmd))
  File "/home/runner/work/opensearch-build/opensearch-build/src/git/git_repository.py", line 80, in output
    return subprocess.check_output(command, cwd=cwd, shell=True).decode().strip()
  File "/opt/hostedtoolcache/Python/3.7.15/x64/lib/python3.7/subprocess.py", line 411, in check_output
    **kwargs).stdout
  File "/opt/hostedtoolcache/Python/3.7.15/x64/lib/python3.7/subprocess.py", line 488, in run
    with Popen(*popenargs, **kwargs) as process:
  File "/opt/hostedtoolcache/Python/3.7.15/x64/lib/python3.7/subprocess.py", line 800, in __init__
    restore_signals, start_new_session)
  File "/opt/hostedtoolcache/Python/3.7.15/x64/lib/python3.7/subprocess.py", line 1551, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: '/tmp/tmpeukd5ura/dashboards-reports/reports-scheduler': '/tmp/tmpeukd5ura/dashboards-reports/reports-scheduler'

@gaiksaya
Copy link
Member

The CI is failing for reports-scheduler. Looks like working-directory is not required anymore https://github.com/opensearch-project/opensearch-build/blob/main/manifests/3.0.0/opensearch-3.0.0.yml#L50
@rupal-bq Please confirm.
Thanks!

@zelinh
Copy link
Member

zelinh commented Jan 10, 2023

Seems like we have removed the front end code and split the repo in this PR for main branch. opensearch-project/reporting#630
Then working_directory would need to be removed.

@rupal-bq
Copy link
Contributor

rupal-bq commented Jan 10, 2023

The CI is failing for reports-scheduler. Looks like working-directory is not required anymore https://github.com/opensearch-project/opensearch-build/blob/main/manifests/3.0.0/opensearch-3.0.0.yml#L50 @rupal-bq Please confirm. Thanks!

Yes, I merged repo split changes so working-directory isn't required now.
PR for removing working-directory: #3075

@prudhvigodithi
Copy link
Member Author

prudhvigodithi commented Jan 10, 2023

The workflow still fails for manifest manifests/3.0.0/opensearch-3.0.0.yml with error

FileNotFoundError: [Errno 2] No such file or directory: '/tmp/tmpj2e28g59/dashboards-reports/reports-scheduler': '/tmp/tmpj2e28g59/dashboards-reports/reports-scheduler'

@gaiksaya
Copy link
Member

I believe it needs a rebase? @prudhvigodithi

Signed-off-by: Prudhvi Godithi <[email protected]>
@prudhvigodithi prudhvigodithi merged commit 5ca1332 into opensearch-project:main Jan 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants