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

search bar #33714

Closed
wants to merge 122 commits into from
Closed

search bar #33714

wants to merge 122 commits into from

Conversation

qwqtw
Copy link

@qwqtw qwqtw commented Nov 15, 2023

search.bar.demo.mov

mtyaka and others added 30 commits April 11, 2023 18:33
…_palm

[FC-0014] [palm] Add GA 4 support to edX platform
<!--

🌴🌴
🌴🌴🌴🌴         🌴 Note: the Palm master branch has been created. Please consider whether your change
    🌴🌴🌴🌴     should also be applied to Palm. If so, make another pull request against the
🌴🌴🌴🌴         open-release/palm.master branch, or ask in the #wg-build-test-release Slack channel
🌴🌴             if you have any questions or need help.

🫒🫒🫒🫒🫒🫒     🫒 Note: the Olive release is still supported.
                Please consider whether your change should be applied to Olive as well.

Please give your pull request a short but descriptive title.
Use conventional commits to separate and summarize commits logically:
https://open-edx-proposals.readthedocs.io/en/latest/oep-0051-bp-conventional-commits.html

Use this template as a guide. Omit sections that don't apply.
You may link to information rather than copy it, but only if the link is publicly
readable.  If you must linked information must be private (because it has secrets),
clearly label the link as private.

-->

## Description

This PR aims to update the version of the edx-search library to [3.5.0](https://github.com/openedx/edx-search/releases/tag/v3.5.0). This update will enable operators to have a shared ElasticSearch installation for multiple OpenedX instances. More details about this feature can be found [here](openedx/edx-search#130).

## Supporting information

In the Large Instances Working Group we added a feature to have a [shared ElasticSearch installation](openedx/openedx-k8s-harmony#4) for multiple OpenedX instances in a Kubernetes cluster. To take advantage of this feature in the Palm release, we require the upgrade of the edx-search version.
Github hosted tests weren't working because shards were not up to date.
The Github runners are working now, so switch back to using them instead
of 2U self-hosted workers which are only guaranteed to work on master
and will probably stop working on the Palm branch at some point.

This reverts commit de62e38.
Replace a relative URL with the absolute one.
chore: revert back to using GH runners for Palm
Co-authored-by: Farhaan Bukhsh <[email protected]>
(cherry picked from commit 065f894)
…ord reset (openedx#32073)

VAN-1371

Co-authored-by: Syed Sajjad  Hussain Shah <[email protected]>
(cherry picked from commit 416a502)
…openedx#32081)

There was a `requirements/pip.txt` with old versions, and a newer
`requirements/edx/pip.txt` managed via a `pip.in` file. The old one was
used in most places, but came out of sync with pip-tools.txt, which was
managed properly. Eventually this caused a `pip check` failure due to the
mismatch.

This should resolve at least part of edx/edx-arch-experiments#267

This PR moves pip.in and pip-tools.in and their corresponding pin files
up to the `requirements/` dir, since they should be shared between the edx
and sandbox environments. This also has the effect of upgrading pip to
match the version in the file we've been uselessly upgrading.

Other improvements:

- Remove `-q` option from pip and pip-sync calls, as it was hiding some
  debugging information that would have resolved this sooner.
- Depend on `pre-requirements` from `compile-requirements`, rather than
  from `upgrade`. (The base target is the one that actually needs it.)
  This also lets us remove the explicit `pip install pip-tools` line.
- Install the recompiled pip and pip-tools files right away, not after the
  loop. When we upgrade pip-tools, we want to use the upgraded version,
  not the previous version. This requires moving the pip-tools.txt
  recompilation outside of the loop and into its own explicit line.
- Don't upgrade pip if we're not running `make upgrade` (respect the
  compile options).
- Remove apparently-unneeded `--no-emit-trusted-host --no-emit-index-url`
  options (we don't pass trusted-host or index-url options).

(cherry picked from commit b852344)
* fix: upgrade pip & pip-tools versions
* fix: install libmysqlclient-dev system package again
* fix: skip reinstalling mysqlclient package
* fix: add pytz package constraint

---------

Co-authored-by: edX requirements bot <[email protected]>
(cherry picked from commit 0ec7b8c)
…2380) (openedx#32506)

* perf!: add support for caching programs on per site bases
This is a security fix that restricts capabilities to author libraries in studio to course authors/ members of an organization.
Security advisory: GHSA-3q74-3rfh-g37j
@mphilbrick211
Copy link

Hi @qtw97! Just checking in to see if you have any questions regarding submitting a CLA form? Please let us know. Thanks!

@mphilbrick211
Copy link

Hi @qtw97! Just checking in to see if you plan to pursue this pull request. Please let us know, thanks! Also, happy to help with any questions on the CLA form.

@mphilbrick211
Copy link

Hi @qtw97! I'm going to close this pull request for now. If you'd like to reopen it to work on it in the future, please let us know.

Thank you!

@openedx-webhooks
Copy link

@qtw97 Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future.

@mphilbrick211 mphilbrick211 added the closed inactivity PR was closed because the author abandoned it label Jan 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed inactivity PR was closed because the author abandoned it open-source-contribution PR author is not from Axim or 2U
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.